MDL-41157 calendar: improve calendar_get_course_cached
authorDan Poltawski <dan@moodle.com>
Wed, 14 Aug 2013 02:46:19 +0000 (10:46 +0800)
committerDan Poltawski <dan@moodle.com>
Wed, 14 Aug 2013 03:05:27 +0000 (11:05 +0800)
The function get_course() introduced in 2.5.1 will use $COURSE and
$SITE to avoid an uncessary DB query, simplifying the logic of this
function and improving perf.

calendar/lib.php

index 50ef3a6..cde7b74 100644 (file)
@@ -1391,14 +1391,8 @@ function calendar_get_module_cached(&$coursecache, $modulename, $instance) {
  * @return stdClass $coursecache[$courseid] return the specific course cache
  */
 function calendar_get_course_cached(&$coursecache, $courseid) {
-    global $COURSE, $DB;
-
     if (!isset($coursecache[$courseid])) {
-        if ($courseid == $COURSE->id) {
-            $coursecache[$courseid] = $COURSE;
-        } else {
-            $coursecache[$courseid] = $DB->get_record('course', array('id'=>$courseid));
-        }
+        $coursecache[$courseid] = get_course($courseid);
     }
     return $coursecache[$courseid];
 }