Skip to content
Permalink
Browse files

MDL-27824 - course category: add check test for ->coursesperpage value.

  • Loading branch information...
Rossiani Wijaya
Rossiani Wijaya committed Aug 30, 2012
1 parent 6109f21 commit 98068e1d8c0e3f6272218daebd21419749666e0b
Showing with 9 additions and 1 deletion.
  1. +9 −1 course/category.php
@@ -31,7 +31,6 @@
$id = required_param('id', PARAM_INT); // Category id
$page = optional_param('page', 0, PARAM_INT); // which page to show
$perpage = optional_param('perpage', $CFG->coursesperpage, PARAM_INT); // how many per page
$categoryedit = optional_param('categoryedit', -1, PARAM_BOOL);
$hide = optional_param('hide', 0, PARAM_INT);
$show = optional_param('show', 0, PARAM_INT);
@@ -41,6 +40,15 @@
$resort = optional_param('resort', 0, PARAM_BOOL);
$sesskey = optional_param('sesskey', '', PARAM_RAW);
// MDL-27824 - This is a temporary fix until we have the proper
// way to check/initialize $CFG value.
if (isset($CFG->coursesperpage)) {
$defaultperpage = $CFG->coursesperpage;
} else {
$defaultperpage = 20;
}
$perpage = optional_param('perpage', $defaultperpage, PARAM_INT); // how many per page
if (empty($id)) {
print_error("unknowcategory");
}

0 comments on commit 98068e1

Please sign in to comment.
You can’t perform that action at this time.