MDL-18727 param types ARE required
authorskodak <skodak>
Sat, 28 Mar 2009 15:51:42 +0000 (15:51 +0000)
committerskodak <skodak>
Sat, 28 Mar 2009 15:51:42 +0000 (15:51 +0000)
grade/report/visual/data.php
grade/report/visual/index.php
grade/report/visual/preferences.php
grade/report/visual/print.php
grade/report/visual/visual_settings.php

index e460f57241366c64ddccca9c6def1fc8302bd315..ecf0a503494e59aa1b8b4454529cdb6e8d47d68b 100644 (file)
@@ -26,7 +26,7 @@
  * Page to be read in by the flex application.
  * Outputs data for a visulasation in tab format.
  */
+
 /// Get a session id from the URI request and make a cookie
 /// for it temparaly. This is needed as the flex application will
 /// not have the users oringal cookie and only the session information
@@ -43,12 +43,12 @@ require_once $CFG->libdir.'/gradelib.php';
 require_once $CFG->dirroot.'/grade/lib.php';
 require_once $CFG->dirroot.'/grade/report/visual/lib.php';
 
-$courseid = required_param('id');
-$visid = optional_param('visid');
+$courseid = required_param('id', PARAM_INT);
+$visid = optional_param('visid', '', PARAM_ACTION);
 
 /// basic access checks
 if (!$course = $DB->get_record('course', array('id' => $courseid))) {
-        print_error('nocourseid');
+    print_error('nocourseid');
 }
 require_login($course);
 $context = get_context_instance(CONTEXT_COURSE, $course->id);
index 5d35831348fa5e91747a8faf6e79dbe8124cffce..0ff4f4485c1dfe5c27705bee2c9e571db6e0f3df 100644 (file)
@@ -31,12 +31,12 @@ require_once $CFG->libdir.'/gradelib.php';
 require_once $CFG->dirroot.'/grade/lib.php';
 require_once $CFG->dirroot.'/grade/report/visual/lib.php';
 
-$courseid = required_param('id');
-$visid = optional_param('visid');
+$courseid = required_param('id', PARAM_INT);
+$visid    = optional_param('visid', '', PARAM_ACTION);
 
 /// basic access checks
 if (!$course = $DB->get_record('course', array('id' => $courseid))) {
-        print_error('nocourseid');
+    print_error('nocourseid');
 }
 require_login($course);
 $context = get_context_instance(CONTEXT_COURSE, $course->id);
index 30ff069182a17ea1d9fe341c1f878f3997e8e7a6..187d2e315be8c3969d99730e184cc60c1292acb8 100755 (executable)
@@ -37,7 +37,7 @@ $courseid = required_param('id', PARAM_INT);
 
 /// Make sure they can even access this course
 if (!$course = $DB->get_record('course', array('id' => $courseid))) {
-        print_error('nocourseid');
+    print_error('nocourseid');
 }
 
 require_login($course);
index e585ab41b3fcbcda3301868bcf33e7b6276e9da7..860aabc7d51b52bafb1d1cf393170b9b588a0d0a 100644 (file)
@@ -32,7 +32,7 @@ require_once $CFG->dirroot.'/grade/lib.php';
 require_once $CFG->dirroot.'/grade/report/visual/lib.php';
 
 $courseid = required_param('id', PARAM_INT);
-$visid = optional_param('visid');
+$visid    = optional_param('visid', '', PARAM_ACTION);
 
 /// basic access checks
 if (!$course = $DB->get_record('course', array('id' => $courseid))) {
index bf6fbffac11d1529fc74b68290b7f56366158d09..d3db8f2139041c0c7f78c997c5ccb04172093dc5 100644 (file)
@@ -43,12 +43,12 @@ require_once '../../../config.php';
 require_once $CFG->dirroot.'/grade/report/visual/lib.php';
 require_once $CFG->libdir.'/phpxml/xml.php';
 
-$visname = required_param('visid');
-$courseid = required_param('id');
+$visname  = required_param('visid', PARAM_ACTION);
+$courseid = required_param('id', PARAM_INT);
 
 /// basic access checks
 if (!$course = $DB->get_record('course', array('id' => $courseid))) {
-        print_error('nocourseid');
+    print_error('nocourseid');
 }
 require_login($course);
 $context = get_context_instance(CONTEXT_COURSE, $course->id);