Browse Source

Fixing LP category redirections

Julio Montoya 12 years ago
parent
commit
b810d8d71a
2 changed files with 18 additions and 9 deletions
  1. 6 4
      main/newscorm/learnpath.class.php
  2. 12 5
      main/newscorm/lp_add_category.php

+ 6 - 4
main/newscorm/learnpath.class.php

@@ -9312,10 +9312,12 @@ EOD;
         global $app;
         $em = $app['orm.em'];
         $item = $em->find('Entity\EntityCLpCategory', $params['id']);
-        $item->setName($params['name']);
-        $item->setCId($params['c_id']);
-        $em->persist($item);
-        $em->flush();
+        if ($item) {
+            $item->setName($params['name']);
+            $item->setCId($params['c_id']);
+            $em->persist($item);
+            $em->flush();
+        }
     }
 
     static function get_categories($course_id) {

+ 12 - 5
main/newscorm/lp_add_category.php

@@ -100,11 +100,7 @@ if (!empty($gradebook) && $gradebook=='view') {
 
 $interbreadcrumb[] = array('url' => 'lp_controller.php?action=list', 'name' => get_lang('LearningPaths'));
 
-Display::display_header(get_lang('LearnpathAddLearnpath'), 'Path');
 
-echo '<div class="actions">';
-echo '<a href="lp_controller.php?'.api_get_cidreq().'">'.Display::return_icon('back.png', get_lang('ReturnToLearningPaths'),'',ICON_SIZE_MEDIUM).'</a>';
-echo '</div>';
 
 $form = new FormValidator('lp_add_category', 'post', 'lp_controller.php');
 
@@ -123,10 +119,16 @@ $form->addElement('style_submit_button', 'Submit', get_lang('Save'),'class="save
 
 if ($form->validate()) {
     $values = $form->getSubmitValues();
-    if (isset($values['id'])) {
+    if (!empty($values['id'])) {
         learnpath::update_category($values);
+        $url = api_get_self().'?action=list&'.api_get_cidreq();
+        header('Location: '.$url);
+        exit;
     } else {
         learnpath::create_category($values);
+        $url = api_get_self().'?action=list&'.api_get_cidreq();
+        header('Location: '.$url);
+        exit;
     }
 } else {
     $id = isset($_REQUEST['id']) ? $_REQUEST['id'] : null;
@@ -141,6 +143,11 @@ if ($form->validate()) {
     }
 }
 
+Display::display_header(get_lang('LearnpathAddLearnpath'), 'Path');
+
+echo '<div class="actions">';
+echo '<a href="lp_controller.php?'.api_get_cidreq().'">'.Display::return_icon('back.png', get_lang('ReturnToLearningPaths'),'',ICON_SIZE_MEDIUM).'</a>';
+echo '</div>';
 
 $form->display();
 // Footer