changed all the POST with option/required_param functions
[moodle.git] / mod / scorm / datamodel.php
index e691cb7..f7bbe1b 100755 (executable)
@@ -2,35 +2,37 @@
     require_once('../../config.php');
     require_once('lib.php');
     
-    optional_variable($id);    // Course Module ID, or
-    optional_variable($a);     // scorm IDa
+    $id = optional_param('id', '', PARAM_INT);       // Course Module ID, or
+    $a = optional_param('a', '', PARAM_INT);         // scorm ID
+    $scoid = required_param('scoid', '', PARAM_INT);  // sco ID
 
-    if ($id) {
-        if (! $cm = get_record('course_modules', 'id', $id)) {
-            error('Course Module ID was incorrect');
+    if (!empty($id)) {
+        if (! $cm = get_record("course_modules", "id", $id)) {
+            error("Course Module ID was incorrect");
         }
-        if (! $course = get_record('course', 'id', $cm->course)) {
-            error('Course is misconfigured');
+        if (! $course = get_record("course", "id", $cm->course)) {
+            error("Course is misconfigured");
         }
-        if (! $scorm = get_record('scorm', 'id', $cm->instance)) {
-            error('Course module is incorrect');
+        if (! $scorm = get_record("scorm", "id", $cm->instance)) {
+            error("Course module is incorrect");
         }
-    } else {
-        if (! $scorm = get_record('scorm', 'id', $a)) {
-            error('Course module is incorrect');
+    } else if (!empty($a)) {
+        if (! $scorm = get_record("scorm", "id", $a)) {
+            error("Course module is incorrect");
         }
-        if (! $course = get_record('course', 'id', $scorm->course)) {
-            error('Course is misconfigured');
+        if (! $course = get_record("course", "id", $scorm->course)) {
+            error("Course is misconfigured");
         }
-        if (! $cm = get_coursemodule_from_instance('scorm', $scorm->id, $course->id)) {
-            error('Course Module ID was incorrect');
+        if (! $cm = get_coursemodule_from_instance("scorm", $scorm->id, $course->id)) {
+            error("Course Module ID was incorrect");
         }
+    } else {
+        error('A required parameter is missing');
     }
 
     require_login($course->id, false, $cm);
     
-    if (confirm_sesskey() && (isset($SESSION->scorm_scoid))) {
-        $scoid = $SESSION->scorm_scoid;
+    if (confirm_sesskey() && (!empty($scoid))) {
         $result = true;
         foreach ($_POST as $element => $value) {
             if (substr($element,0,3) == 'cmi') {