Minor -fix edit prerequisite

main/lp/learnpath.class.php

@@ -1539,56 +1539,53 @@ class learnpath
      *
      * @param int    $id              Element ID
      * @param string $prerequisite_id Prerequisite Element ID
-     * @param int    $mastery_score   Prerequisite min score
+     * @param int    $minScore        Prerequisite min score
-     * @param int    $max_score       Prerequisite max score
+     * @param int    $maxScore        Prerequisite max score
      *
      * @return bool True on success, false on error
      */
     public function edit_item_prereq(
         $id,
         $prerequisite_id,
-        $mastery_score = 0,
+        $minScore = 0,
-        $max_score = 100
+        $maxScore = 100
     ) {
-        $course_id = api_get_course_int_id();
         if ($this->debug > 0) {
-            error_log('In learnpath::edit_item_prereq('.$id.','.$prerequisite_id.','.$mastery_score.','.$max_score.')', 0);
+            error_log('In learnpath::edit_item_prereq('.$id.','.$prerequisite_id.','.$minScore.','.$maxScore.')', 0);
-        }
-
-        if (empty($id) || ($id != strval(intval($id))) || empty($prerequisite_id)) {
-            return false;
         }
 
+        $id = (int) $id;
         $prerequisite_id = (int) $prerequisite_id;
-        $tbl_lp_item = Database::get_course_table(TABLE_LP_ITEM);
 
-        if (!is_numeric($mastery_score) || $mastery_score < 0) {
+        if (empty($id) || empty($prerequisite_id)) {
-            $mastery_score = 0;
+            return false;
         }
 
-        if (!is_numeric($max_score) || $max_score < 0) {
+        if (empty($minScore) || $minScore < 0) {
-            $max_score = 100;
+            $minScore = 0;
         }
 
-        /*if ($mastery_score > $max_score) {
+        if (empty($maxScore) || $maxScore < 0) {
-            $max_score = $mastery_score;
+            $maxScore = 100;
-        }*/
+        }
 
-        if (!is_numeric($prerequisite_id)) {
+        if (empty($prerequisite_id)) {
             $prerequisite_id = 'NULL';
         }
 
-        $mastery_score = floatval($mastery_score);
+        $minScore = floatval($minScore);
-        $max_score = floatval($max_score);
+        $maxScore = floatval($maxScore);
 
+        $tbl_lp_item = Database::get_course_table(TABLE_LP_ITEM);
         $sql = " UPDATE $tbl_lp_item
                  SET
                     prerequisite = $prerequisite_id ,
-                    prerequisite_min_score = $mastery_score ,
+                    prerequisite_min_score = $minScore ,
-                    prerequisite_max_score = $max_score
+                    prerequisite_max_score = $maxScore
                  WHERE iid = $id";
+
         Database::query($sql);
-        // TODO: Update the item object (can be ignored for now because refreshed).
+
         return true;
     }