From: defacer <defacer>
Date: Fri, 28 Jan 2005 00:02:33 +0000 (+0000)
Subject: Fixing incorrect use of optional_param.
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=2335781f1c62ab39d7579d0735e65e72f5e26158;p=moodle.git

Fixing incorrect use of optional_param.
---

diff --git a/course/view.php b/course/view.php
index e3493a375f..58bd86073d 100644
--- a/course/view.php
+++ b/course/view.php
@@ -6,24 +6,23 @@
     require_once('lib.php');
     require_once($CFG->libdir.'/blocklib.php');
 
-    optional_variable($id);
-    optional_variable($name);
+    $id          = optional_param('id', 0, PARAM_INT);
+    $name        = optional_param('name');
+    $blockaction = optional_param('blockaction');
+    $instanceid  = optional_param('instanceid', 0, PARAM_INT);
+    $blockid     = optional_param('blockid',    0, PARAM_INT);
 
-    optional_param('blockaction');
-    optional_param('instanceid', 0, PARAM_INT);
-    optional_param('blockid',    0, PARAM_INT);
-
-    if (!$id and !$name) {
+    if (empty($id) && empty($name)) {
         error("Must specify course id or short name");
     }
 
-    if (!empty($_GET['name'])) {
-        if (! ($course = get_record("course", "shortname", $name)) ) {
-            error("That's an invalid short course name");
+    if (!empty($name)) {
+        if (! ($course = get_record('course', 'shortname', $name)) ) {
+            error('Invalid short course name');
         }
     } else {
-        if (! ($course = get_record("course", "id", $id)) ) {
-            error("That's an invalid course id");
+        if (! ($course = get_record('course', 'id', $id)) ) {
+            error('Invalid course id');
         }
     }