lots of incorrect required_param()'s - there MUST be only two parameters not three!!
authorskodak <skodak>
Wed, 8 Mar 2006 00:08:51 +0000 (00:08 +0000)
committerskodak <skodak>
Wed, 8 Mar 2006 00:08:51 +0000 (00:08 +0000)
14 files changed:
course/participation.php
course/statsgraph.php
mod/choice/index.php
mod/forum/subscribe.php
mod/glossary/exportentry.php
mod/quiz/edit.php
mod/resource/index.php
mod/scorm/aicc.php
mod/scorm/api.php
mod/scorm/datamodel.php
mod/scorm/index.php
mod/scorm/loadSCO.php
mod/scorm/player.php
mod/scorm/validate.php

index acf8c3c..cfbc28b 100644 (file)
@@ -5,7 +5,7 @@
 
     define('DEFAULT_PAGE_SIZE', 20);
     
-    $id         = required_param('id',0,PARAM_INT); // course id.
+    $id         = required_param('id',PARAM_INT); // course id.
     $moduleid   = optional_param('moduleid',0,PARAM_INT); // module id.
     $oldmod     = optional_param('oldmod',0,PARAM_INT);
     $teachers   = optional_param('teachers',0,PARAM_BOOL); // show teachers? off = students.
index ddce74c..1943a3e 100644 (file)
@@ -4,11 +4,11 @@
     require_once($CFG->dirroot.'/lib/statslib.php');
     require_once($CFG->dirroot.'/lib/graphlib.php');
 
-    $courseid = required_param('course',0,PARAM_INT);
-    $report = required_param('report',0,PARAM_INT);
-    $time = required_param('time',0,PARAM_INT);
-    $userid = optional_param('userid',0,PARAM_INT);
-    $mode = required_param('mode',STATS_MODE_GENERAL,PARAM_INT);
+    $courseid = required_param('course',PARAM_INT);
+    $report = required_param('report',PARAM_INT);
+    $time = required_param('time',PARAM_INT);
+    $userid = optional_param('userid', 0, PARAM_INT);
+    $mode = required_param('mode',PARAM_INT);
 
     if (!$course = get_record("course","id",$courseid)) {
         error("That's an invalid course id");
index 86c76bc..970d016 100644 (file)
@@ -3,7 +3,7 @@
     require_once("../../config.php");
     require_once("lib.php");
 
-    $id = required_param('id',0,PARAM_INT);   // course
+    $id = required_param('id',PARAM_INT);   // course
 
     if (! $course = get_record("course", "id", $id)) {
         error("Course ID is incorrect");
index 187e4da..323f299 100644 (file)
@@ -5,7 +5,7 @@
     require_once("../../config.php");
     require_once("lib.php");
 
-    $id = required_param('id',0,PARAM_INT);      // The forum to subscribe or unsubscribe to
+    $id = required_param('id',PARAM_INT);      // The forum to subscribe or unsubscribe to
     $force = optional_param('force','',PARAM_ALPHA);  // Force everyone to be subscribed to this forum?
     $user = optional_param('user',0,PARAM_INT);
 
index 06ab91d..a1695eb 100644 (file)
@@ -3,7 +3,7 @@
     require_once('lib.php');
 
     $id    = required_param('id', PARAM_INT);           // course module ID
-    $entry = required_param('entry', 0, PARAM_INT);     // Entry ID
+    $entry = required_param('entry', PARAM_INT);     // Entry ID
     $confirm = optional_param('confirm', 0, PARAM_INT); // confirmation
 
     $hook = optional_param('hook', '', PARAM_ALPHANUM);
index 1c6edd3..5c9bc35 100644 (file)
@@ -219,7 +219,7 @@ if (self.name == 'editquestion') {
 
     if (isset($_REQUEST['repaginate']) and confirm_sesskey()) { /// Re-paginate the quiz
         if (isset($_REQUEST['questionsperpage'])) {
-            $modform->questionsperpage = required_param('questionsperpage', 1, PARAM_INT);
+            $modform->questionsperpage = required_param('questionsperpage', PARAM_INT);
             if (!set_field('quiz', 'questionsperpage', $modform->questionsperpage, 'id', $modform->id)) {
                 error('Could not save number of questions per page');
             }
index 4331f0b..221ce10 100644 (file)
@@ -2,7 +2,7 @@
 
     require_once("../../config.php");
 
-    $id = required_param( 'id', 0, PARAM_INT ); // course
+    $id = required_param( 'id', PARAM_INT ); // course
 
     if (!empty($CFG->forcelogin)) {
         require_login();
index 6c61e6b..450b467 100755 (executable)
@@ -2,8 +2,8 @@
     require_once('../../config.php');
     require_once('locallib.php');
 
-    $command = required_param('command', '', PARAM_ALPHA);
-    $sessionid = required_param('session_id', '', PARAM_ALPHANUM);
+    $command = required_param('command', PARAM_ALPHA);
+    $sessionid = required_param('session_id', PARAM_ALPHANUM);
     $aiccdata = optional_param('aicc_data', '', PARAM_RAW);
 
     require_login();
index c8afcb4..2d47e1b 100644 (file)
@@ -5,9 +5,9 @@
 
     $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
+    $scoid = required_param('scoid', PARAM_INT);  // sco ID
     $mode = optional_param('mode', '', PARAM_ALPHA); // navigation mode
-    $attempt = required_param('attempt', '', PARAM_INT); // new attempt
+    $attempt = required_param('attempt', PARAM_INT); // new attempt
 
     if (!empty($id)) {
         if (! $cm = get_record("course_modules", "id", $id)) {
index 5e3b1f8..a253d98 100755 (executable)
@@ -4,8 +4,8 @@
     
     $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
-    $attempt = required_param('attempt', '', PARAM_INT);  // attempt number
+    $scoid = required_param('scoid', PARAM_INT);  // sco ID
+    $attempt = required_param('attempt', PARAM_INT);  // attempt number
 
     if (!empty($id)) {
         if (! $cm = get_record("course_modules", "id", $id)) {
index 098b52c..faffd41 100755 (executable)
@@ -2,7 +2,7 @@
 
     require_once("../../config.php");
 
-    $id = required_param('id', '', PARAM_INT);   // course id
+    $id = required_param('id', PARAM_INT);   // course id
 
     if (!empty($id)) {
         if (! $course = get_record("course", "id", $id)) {
index 71901ed..2b303b4 100755 (executable)
@@ -4,7 +4,7 @@
 
     $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
+    $scoid = required_param('scoid', PARAM_INT); // sco ID
 
     if (!empty($id)) {
         if (! $cm = get_record("course_modules", "id", $id)) {
index 9167a3d..cb3e56f 100755 (executable)
@@ -10,7 +10,7 @@
     //
     $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
+    $scoid = required_param('scoid', PARAM_INT);  // sco ID
     $mode = optional_param('mode', 'normal', PARAM_ALPHA); // navigation mode
     $currentorg = optional_param('currentorg', '', PARAM_RAW); // selected organization
     $newattempt = optional_param('newattempt', 'off', PARAM_ALPHA); // the user request to start a new attempt
index f1e2232..d1a6988 100755 (executable)
@@ -3,8 +3,8 @@
     require_once("../../config.php");
     require_once('locallib.php');
 
-    $courseid = required_param('id', '', PARAM_INT);             // Course Module ID, or
-    $reference = required_param('reference', '', PARAM_PATH);    // Package path
+    $courseid = required_param('id', PARAM_INT);             // Course Module ID, or
+    $reference = required_param('reference', PARAM_PATH);    // Package path
     $scormid = optional_param('instance', '', PARAM_INT);       // scorm ID
 
     require_login($courseid, false);