Fixing return page after submiting changes in the LP see BT#5763

main/newscorm/learnpath.class.php

@@ -6061,8 +6061,7 @@ class learnpath
             ICON_SIZE_MEDIUM
         ).'</a>';*/
 
-        $return .= '<a href="lp_controller.php?'.api_get_cidreq(
-        ).'&amp;action=edit&amp;lp_id='.$_SESSION['oLP']->lp_id.'">'.Display :: return_icon(
+        $return .= '<a href="lp_controller.php?referer=add_item&'.api_get_cidreq().'&amp;action=edit&amp;lp_id='.$_SESSION['oLP']->lp_id.'">'.Display :: return_icon(
             'settings.png',
             get_lang('CourseSettings'),
             '',

main/newscorm/lp_controller.php

@@ -455,7 +455,6 @@ switch ($action) {
                     $_SESSION['oLP']->edit_document($_course);
                 }
                 $is_success = true;
-
                 $url = api_get_self().'?action=add_item&type=step&lp_id='.intval($_SESSION['oLP']->lp_id);
                 header('Location: '.$url);
                 exit;
@@ -752,7 +751,18 @@ switch ($action) {
                     }
                 }
             }
-            $url = api_get_self().'?action=add_item&type=step&lp_id='.intval($_SESSION['oLP']->lp_id).'&'.api_get_cidreq();
+
+
+            $referer = isset($_REQUEST['referer']) ? $_REQUEST['referer'] : 'add_item';
+            switch ($referer) {
+                case 'lplist':
+                    $url = api_get_self().'?action=list&'.api_get_cidreq();
+                    break;
+                default:
+                case 'add_item':
+                    $url = api_get_self().'?action=add_item&type=step&lp_id='.intval($_SESSION['oLP']->lp_id).'&'.api_get_cidreq();
+                    break;
+            }
             header('Location: '.$url);
             exit;
         }

main/newscorm/lp_edit.php

@@ -69,6 +69,9 @@ $form->addRule('lp_name', get_lang('ThisFieldIsRequired'), 'required');
 
 $form->addElement('hidden', 'lp_encoding');
 
+$referer = isset($_REQUEST['referer']) ? Security::remove_XSS($_REQUEST['referer']) : null;
+$form->addElement('hidden', 'referer', $referer);
+
 $items = learnpath::get_category_from_course_into_select(api_get_course_int_id());
 
 if (!empty($items)) {

main/newscorm/lp_list.php

@@ -288,7 +288,7 @@ foreach ($categories as $item) {
             if ($is_allowed_to_edit) {
                 // EDIT LP
                 if ($current_session == $details['lp_session']) {
-                    $dsp_edit_lp = '<a href="lp_controller.php?'.api_get_cidreq().'&action=edit&lp_id='.$id.'">'.Display::return_icon('settings.png', get_lang('CourseSettings'), '', ICON_SIZE_SMALL).'</a>';
+                    $dsp_edit_lp = '<a href="lp_controller.php?referer=lplist&'.api_get_cidreq().'&action=edit&lp_id='.$id.'">'.Display::return_icon('settings.png', get_lang('CourseSettings'), '', ICON_SIZE_SMALL).'</a>';
                 } else {
                     $dsp_edit_lp = Display::return_icon('settings_na.png', get_lang('CourseSettings'), '', ICON_SIZE_SMALL);
                 }