Some fixes -refs #7107

main/inc/lib/link.lib.php

@@ -51,8 +51,13 @@ class Link extends Model
         $linkUrl = Database::escape_string($linkUrl);
         $linkId = intval($linkId);
         $courseId = api_get_course_int_id();
-        $query = "UPDATE $tblLink SET url= '$linkUrl' WHERE id=$linkId AND c_id=$courseId";
-        $resLink = Database::query($query);
+        $session_id = api_get_session_id();
+        if ($linkUrl != '') {
+            $query = "UPDATE $tblLink SET url= '$linkUrl' WHERE id=$linkId AND c_id=$courseId AND session_id=$session_id";
+            $resLink = Database::query($query);
+        } else {
+            return false;
+        }
     }
 }
 

main/newscorm/learnpath.class.php

@@ -1132,7 +1132,7 @@ class learnpath
      * @param   array   The array resulting of the $_FILES[mp3] element
      * @return	boolean	True on success, false on error
      */
-    public function edit_item($id, $parent, $previous, $title, $description, $prerequisites = 0, $audio = null, $url = '', $max_time_allowed = 0)
+    public function edit_item($id, $parent, $previous, $title, $description, $prerequisites = 0, $audio = null, $max_time_allowed = 0, $url = '')
     {
         $course_id = api_get_course_int_id();
         if ($this->debug > 0) {

main/newscorm/lp_controller.php

@@ -582,7 +582,7 @@ switch ($action) {
                 if (isset($_FILES['mp3'])) {
                     $audio = $_FILES['mp3'];
                 }
-                $_SESSION['oLP']->edit_item($_REQUEST['id'], $_POST['parent'], $_POST['previous'], $post_title, $_POST['description'], $_POST['prerequisites'], $audio, $_POST['url'], $_POST['maxTimeAllowed']);
+                $_SESSION['oLP']->edit_item($_REQUEST['id'], $_POST['parent'], $_POST['previous'], $post_title, $_POST['description'], $_POST['prerequisites'], $audio, $_POST['maxTimeAllowed'], $_POST['url']);
 
                 if (isset($_POST['content_lp'])) {
                     $_SESSION['oLP']->edit_document($_course);