MDL-18727 param types ARE required

This commit is contained in:
skodak 2009-03-28 15:51:42 +00:00
parent 08a09f54ed
commit e9a0ee701d
5 changed files with 12 additions and 12 deletions

View file

@ -26,7 +26,7 @@
* Page to be read in by the flex application. * Page to be read in by the flex application.
* Outputs data for a visulasation in tab format. * Outputs data for a visulasation in tab format.
*/ */
/// Get a session id from the URI request and make a cookie /// Get a session id from the URI request and make a cookie
/// for it temparaly. This is needed as the flex application will /// for it temparaly. This is needed as the flex application will
/// not have the users oringal cookie and only the session information /// 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/lib.php';
require_once $CFG->dirroot.'/grade/report/visual/lib.php'; require_once $CFG->dirroot.'/grade/report/visual/lib.php';
$courseid = required_param('id'); $courseid = required_param('id', PARAM_INT);
$visid = optional_param('visid'); $visid = optional_param('visid', '', PARAM_ACTION);
/// basic access checks /// basic access checks
if (!$course = $DB->get_record('course', array('id' => $courseid))) { if (!$course = $DB->get_record('course', array('id' => $courseid))) {
print_error('nocourseid'); print_error('nocourseid');
} }
require_login($course); require_login($course);
$context = get_context_instance(CONTEXT_COURSE, $course->id); $context = get_context_instance(CONTEXT_COURSE, $course->id);

View file

@ -31,12 +31,12 @@ require_once $CFG->libdir.'/gradelib.php';
require_once $CFG->dirroot.'/grade/lib.php'; require_once $CFG->dirroot.'/grade/lib.php';
require_once $CFG->dirroot.'/grade/report/visual/lib.php'; require_once $CFG->dirroot.'/grade/report/visual/lib.php';
$courseid = required_param('id'); $courseid = required_param('id', PARAM_INT);
$visid = optional_param('visid'); $visid = optional_param('visid', '', PARAM_ACTION);
/// basic access checks /// basic access checks
if (!$course = $DB->get_record('course', array('id' => $courseid))) { if (!$course = $DB->get_record('course', array('id' => $courseid))) {
print_error('nocourseid'); print_error('nocourseid');
} }
require_login($course); require_login($course);
$context = get_context_instance(CONTEXT_COURSE, $course->id); $context = get_context_instance(CONTEXT_COURSE, $course->id);

View file

@ -37,7 +37,7 @@ $courseid = required_param('id', PARAM_INT);
/// Make sure they can even access this course /// Make sure they can even access this course
if (!$course = $DB->get_record('course', array('id' => $courseid))) { if (!$course = $DB->get_record('course', array('id' => $courseid))) {
print_error('nocourseid'); print_error('nocourseid');
} }
require_login($course); require_login($course);

View file

@ -32,7 +32,7 @@ require_once $CFG->dirroot.'/grade/lib.php';
require_once $CFG->dirroot.'/grade/report/visual/lib.php'; require_once $CFG->dirroot.'/grade/report/visual/lib.php';
$courseid = required_param('id', PARAM_INT); $courseid = required_param('id', PARAM_INT);
$visid = optional_param('visid'); $visid = optional_param('visid', '', PARAM_ACTION);
/// basic access checks /// basic access checks
if (!$course = $DB->get_record('course', array('id' => $courseid))) { if (!$course = $DB->get_record('course', array('id' => $courseid))) {

View file

@ -43,12 +43,12 @@ require_once '../../../config.php';
require_once $CFG->dirroot.'/grade/report/visual/lib.php'; require_once $CFG->dirroot.'/grade/report/visual/lib.php';
require_once $CFG->libdir.'/phpxml/xml.php'; require_once $CFG->libdir.'/phpxml/xml.php';
$visname = required_param('visid'); $visname = required_param('visid', PARAM_ACTION);
$courseid = required_param('id'); $courseid = required_param('id', PARAM_INT);
/// basic access checks /// basic access checks
if (!$course = $DB->get_record('course', array('id' => $courseid))) { if (!$course = $DB->get_record('course', array('id' => $courseid))) {
print_error('nocourseid'); print_error('nocourseid');
} }
require_login($course); require_login($course);
$context = get_context_instance(CONTEXT_COURSE, $course->id); $context = get_context_instance(CONTEXT_COURSE, $course->id);