From 24a88abd00a79684d2c9e6bee25b7cfe93299181 Mon Sep 17 00:00:00 2001
From: skodak <skodak>
Date: Sat, 28 Mar 2009 15:18:38 +0000
Subject: [PATCH] MDL-18726 dml cleanup

---
 grade/report/stats/index.php                  |  6 +-
 grade/report/stats/lib.php                    | 34 +++------
 grade/report/stats/preferences.php            |  8 +--
 grade/report/stats/print.php                  |  9 +--
 grade/report/visual/data.php                  |  7 +-
 grade/report/visual/index.php                 |  7 +-
 grade/report/visual/lib.php                   | 72 ++++++-------------
 grade/report/visual/preferences.php           | 12 +---
 grade/report/visual/print.php                 | 11 +--
 grade/report/visual/visual_settings.php       |  7 +-
 .../visual_grade_distribution_bar.php         | 12 ++--
 11 files changed, 51 insertions(+), 134 deletions(-)

diff --git a/grade/report/stats/index.php b/grade/report/stats/index.php
index 1e9cb9cc9f..891c8422ae 100755
--- a/grade/report/stats/index.php
+++ b/grade/report/stats/index.php
@@ -34,14 +34,12 @@ require_once $CFG->libdir.'/gradelib.php';
 require_once $CFG->dirroot.'/grade/lib.php';
 require_once $CFG->dirroot.'/grade/report/stats/lib.php';
 
-$courseid = required_param('id');
+$courseid = required_param('id', PARAM_INT);
 $toggle = optional_param('toggle', NULL, PARAM_INT);
 $toggle_type = optional_param('toggle_type', 0, PARAM_ALPHANUM);
 
 /// basic access checks
-$course = $DB->get_record('course', array('id' => $courseid));
-
-if (!$course) {
+if (!$course = $DB->get_record('course', array('id' => $courseid))) {
     print_error('nocourseid');
 }
 
diff --git a/grade/report/stats/lib.php b/grade/report/stats/lib.php
index a67b58ca4d..30abffd867 100755
--- a/grade/report/stats/lib.php
+++ b/grade/report/stats/lib.php
@@ -194,13 +194,9 @@ class grade_report_stats extends grade_report {
             $this->users = array();
             $this->userselect_params = array();
         } else {
-            if(isset($DB) && !is_null($DB)) {
-                list($usql, $params) = $DB->get_in_or_equal(array_keys($this->users));
-                $this->userselect = "AND g.userid $usql";
-                $this->userselect_params = $params;
-            }else{
-                $this->userselect = 'AND g.userid in ('.implode(',', array_keys($this->users)).')';
-            }
+            list($usql, $params) = $DB->get_in_or_equal(array_keys($this->users));
+            $this->userselect = "AND g.userid $usql";
+            $this->userselect_params = $params;
         }
 
         return $this->users;
@@ -234,25 +230,15 @@ class grade_report_stats extends grade_report {
 
         $params = array();
 
-        if(isset($DB) && !is_null($DB)) {
-            $params = array_merge(array($this->courseid), $this->userselect_params);
-
-            /// please note that we must fetch all grade_grades fields if we want to contruct grade_grade object from it!
-            $sql = "SELECT g.*
-                  FROM {grade_items} gi,
-                       {grade_grades} g
-                 WHERE g.itemid = gi.id AND gi.courseid = ? {$this->userselect}";
+        $params = array_merge(array($this->courseid), $this->userselect_params);
 
-            $grades = $DB->get_records_sql($sql, $params);
-        } else {
-            /// please note that we must fetch all grade_grades fields if we want to contruct grade_grade object from it!
-            $sql = "SELECT g.*
-                  FROM {grade_items} gi,
-                       {grade_grades} g
-                 WHERE g.itemid = gi.id AND gi.courseid = {$this->courseid} {$this->userselect}";
+        /// please note that we must fetch all grade_grades fields if we want to contruct grade_grade object from it!
+        $sql = "SELECT g.*
+              FROM {grade_items} gi,
+                   {grade_grades} g
+             WHERE g.itemid = gi.id AND gi.courseid = ? {$this->userselect}";
 
-            $grades = $DB->get_records_sql($sql);
-        }
+        $grades = $DB->get_records_sql($sql, $params);
 
         $userids = array_keys($this->users);
 
diff --git a/grade/report/stats/preferences.php b/grade/report/stats/preferences.php
index 669de0c3b3..20450c3c07 100755
--- a/grade/report/stats/preferences.php
+++ b/grade/report/stats/preferences.php
@@ -36,13 +36,7 @@ $courseid = required_param('id', PARAM_INT);
 
 
 /// Make sure they can even access this course
-if(isset($DB) && !is_null($DB)) {
-    $course = $DB->get_record('course', array('id' => $courseid));
-} else {
-    $course = get_record('course', 'id', $courseid);
-}
-
-if (!$course) {
+if (!$course = $DB->get_record('course', array('id' => $courseid))) {
     print_error('nocourseid');
 }
 
diff --git a/grade/report/stats/print.php b/grade/report/stats/print.php
index de9033a657..bf55173361 100644
--- a/grade/report/stats/print.php
+++ b/grade/report/stats/print.php
@@ -37,13 +37,8 @@ $courseid = required_param('id');
 $reportname = get_string('modulename', 'gradereport_stats');
 
 /// basic access checks
-if(isset($DB) && !is_null($DB)) {
-    $course = $DB->get_record('course', array('id' => $courseid));
-} else {
-    $course = get_record('course', 'id', $courseid);
-}
-if (!$course) {
-        print_error('nocourseid');
+if (!$course = $DB->get_record('course', array('id' => $courseid))) {
+    print_error('nocourseid');
 }
 require_login($course);
 $context = get_context_instance(CONTEXT_COURSE, $course->id);
diff --git a/grade/report/visual/data.php b/grade/report/visual/data.php
index ee8e7afb1b..e460f57241 100644
--- a/grade/report/visual/data.php
+++ b/grade/report/visual/data.php
@@ -47,12 +47,7 @@ $courseid = required_param('id');
 $visid = optional_param('visid');
 
 /// basic access checks
-if(isset($DB) && !is_null($DB)) {
-    $course = $DB->get_record('course', array('id' => $courseid));
-} else {
-    $course = get_record('course', 'id', $courseid);
-}
-if (!$course) {
+if (!$course = $DB->get_record('course', array('id' => $courseid))) {
         print_error('nocourseid');
 }
 require_login($course);
diff --git a/grade/report/visual/index.php b/grade/report/visual/index.php
index 06c064b8ee..5d35831348 100644
--- a/grade/report/visual/index.php
+++ b/grade/report/visual/index.php
@@ -35,12 +35,7 @@ $courseid = required_param('id');
 $visid = optional_param('visid');
 
 /// basic access checks
-if(isset($DB) && !is_null($DB)) {
-    $course = $DB->get_record('course', array('id' => $courseid));
-} else {
-    $course = get_record('course', 'id', $courseid);
-}
-if (!$course) {
+if (!$course = $DB->get_record('course', array('id' => $courseid))) {
         print_error('nocourseid');
 }
 require_login($course);
diff --git a/grade/report/visual/lib.php b/grade/report/visual/lib.php
index 2875a94322..d02796779b 100755
--- a/grade/report/visual/lib.php
+++ b/grade/report/visual/lib.php
@@ -224,44 +224,28 @@ class grade_report_visual extends grade_report {
     public function load_users() {
         global $CFG, $DB;
 
-        if(isset($DB) && !is_null($DB)) {
-            $params = array();
-            list($usql, $gbr_params) = $DB->get_in_or_equal(explode(',', $this->gradebookroles));
-
-            $sql = "SELECT u.id, u.firstname, u.lastname, u.imagealt, u.picture, u.idnumber, u.username
-                    FROM {user} u
-                        JOIN {role_assignments} ra ON u.id = ra.userid
-                        $this->groupsql
-                    WHERE ra.roleid $usql
-                        $this->groupwheresql
-                        AND ra.contextid ".get_related_contexts_string($this->context);
-
-            $params = array_merge($gbr_params, $this->groupwheresql_params);
-            $this->users = $DB->get_records_sql($sql, $params);
-        } else {
-            $sql = "SELECT u.id, u.firstname, u.lastname, u.imagealt, u.picture, u.idnumber
-                      FROM {user) u
-                           JOIN {role_assignments} ra ON u.id = ra.userid
-                           $this->groupsql
-                     WHERE ra.roleid in ($this->gradebookroles)
-                           $this->groupwheresql
-                           AND ra.contextid ".get_related_contexts_string($this->context);
-
-            $this->users = $DB->get_records_sql($sql);
-        }
+        $params = array();
+        list($usql, $gbr_params) = $DB->get_in_or_equal(explode(',', $this->gradebookroles));
+
+        $sql = "SELECT u.id, u.firstname, u.lastname, u.imagealt, u.picture, u.idnumber, u.username
+                FROM {user} u
+                    JOIN {role_assignments} ra ON u.id = ra.userid
+                    $this->groupsql
+                WHERE ra.roleid $usql
+                    $this->groupwheresql
+                    AND ra.contextid ".get_related_contexts_string($this->context);
+
+        $params = array_merge($gbr_params, $this->groupwheresql_params);
+        $this->users = $DB->get_records_sql($sql, $params);
 
         if (empty($this->users)) {
             $this->userselect = '';
             $this->users = array();
             $this->userselect_params = array();
         } else {
-            if(isset($DB) && !is_null($DB)) {
-                list($usql, $params) = $DB->get_in_or_equal(array_keys($this->users));
-                $this->userselect = "AND g.userid $usql";
-                $this->userselect_params = $params;
-            }else{
-                $this->userselect = 'AND g.userid in ('.implode(',', array_keys($this->users)).')';
-            }
+            list($usql, $params) = $DB->get_in_or_equal(array_keys($this->users));
+            $this->userselect = "AND g.userid $usql";
+            $this->userselect_params = $params;
         }
 
         return $this->users;
@@ -277,25 +261,15 @@ class grade_report_visual extends grade_report {
 
         $params = array();
 
-        if(isset($DB) && !is_null($DB)) {
-            $params = array_merge(array($this->courseid), $this->userselect_params);
-
-            /// please note that we must fetch all grade_grades fields if we want to contruct grade_grade object from it!
-            $sql = "SELECT g.*
-                  FROM {grade_items} gi,
-                       {grade_grades} g
-                 WHERE g.itemid = gi.id AND gi.courseid = ? {$this->userselect}";
+        $params = array_merge(array($this->courseid), $this->userselect_params);
 
-            $grades = $DB->get_records_sql($sql, $params);
-        } else {
-            /// please note that we must fetch all grade_grades fields if we want to contruct grade_grade object from it!
-            $sql = "SELECT g.*
-                  FROM {grade_items} gi,
-                       {grade_grades} g
-                 WHERE g.itemid = gi.id AND gi.courseid = {$this->courseid} {$this->userselect}";
+        /// please note that we must fetch all grade_grades fields if we want to contruct grade_grade object from it!
+        $sql = "SELECT g.*
+              FROM {grade_items} gi,
+                   {grade_grades} g
+             WHERE g.itemid = gi.id AND gi.courseid = ? {$this->userselect}";
 
-            $grades = get_records_sql($sql);
-        }
+        $grades = $DB->get_records_sql($sql, $params);
 
         $userids = array_keys($this->users);
 
diff --git a/grade/report/visual/preferences.php b/grade/report/visual/preferences.php
index 884fec6977..30ff069182 100755
--- a/grade/report/visual/preferences.php
+++ b/grade/report/visual/preferences.php
@@ -36,17 +36,11 @@ $courseid = required_param('id', PARAM_INT);
 
 
 /// Make sure they can even access this course
-if(isset($DB) && !is_null($DB)) {
-    $course = $DB->get_record('course', array('id' => $courseid));
-} else {
-    $course = get_record('course', 'id', $courseid);
+if (!$course = $DB->get_record('course', array('id' => $courseid))) {
+        print_error('nocourseid');
 }
 
-if (!$course) {
-    print_error('nocourseid');
-}
-
-require_login($course->id);
+require_login($course);
 
 $context = get_context_instance(CONTEXT_COURSE, $course->id);
 $systemcontext = get_context_instance(CONTEXT_SYSTEM);
diff --git a/grade/report/visual/print.php b/grade/report/visual/print.php
index 679cfe2db1..e585ab41b3 100644
--- a/grade/report/visual/print.php
+++ b/grade/report/visual/print.php
@@ -31,17 +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');
+$courseid = required_param('id', PARAM_INT);
 $visid = optional_param('visid');
 
 /// basic access checks
-if(isset($DB) && !is_null($DB)) {
-    $course = $DB->get_record('course', array('id' => $courseid));
-} else {
-    $course = get_record('course', 'id', $courseid);
-}
-if (!$course) {
-        print_error('nocourseid');
+if (!$course = $DB->get_record('course', array('id' => $courseid))) {
+    print_error('nocourseid');
 }
 require_login($course);
 $context = get_context_instance(CONTEXT_COURSE, $course->id);
diff --git a/grade/report/visual/visual_settings.php b/grade/report/visual/visual_settings.php
index 85b2210a1b..bf6fbffac1 100644
--- a/grade/report/visual/visual_settings.php
+++ b/grade/report/visual/visual_settings.php
@@ -47,12 +47,7 @@ $visname = required_param('visid');
 $courseid = required_param('id');
 
 /// basic access checks
-if(isset($DB) && !is_null($DB)) {
-    $course = $DB->get_record('course', array('id' => $courseid));
-} else {
-    $course = get_record('course', 'id', $courseid);
-}
-if (!$course) {
+if (!$course = $DB->get_record('course', array('id' => $courseid))) {
         print_error('nocourseid');
 }
 require_login($course);
diff --git a/grade/report/visual/visualizations/visual_grade_distribution_bar.php b/grade/report/visual/visualizations/visual_grade_distribution_bar.php
index f789dd58f9..ef94681433 100644
--- a/grade/report/visual/visualizations/visual_grade_distribution_bar.php
+++ b/grade/report/visual/visualizations/visual_grade_distribution_bar.php
@@ -70,19 +70,15 @@ class grade_distribution_bar extends visualization {
             $options[$groupkey] = grade_report_visual::truncate($group->name);
         }
         $options[0] = 'All Groups';
-        
-        if(isset($DB) && !is_null($DB)) {
-            $course = $DB->get_record('course', array('id' => required_param('id')));
-        } else {
-            $course = get_record('course', 'id', required_param('id'));
-        }
-        if (!$course) {
+
+// TODO: move the required_param elsewhere!!
+        if (!$course = $DB->get_record('course', array('id' => required_param('id', PARAM_INT)))) {
             print_error('nocourseid');
         }
         
         $active = groups_get_course_group($course, true);
         
-        if(!$active) {
+        if (!$active) {
             $active = 0;
         }
 
-- 
2.39.5