get_my_courses() callers update - check of the $fields param - pass null instead

The default way to call get_my_courses() is now to pass null in the $fields
param to get the (cached copy) of the useful fields. Cuts down enormously
on DB traffic, memory footprint, and session size.
This commit is contained in:
martinlanghoff 2007-05-06 05:32:40 +00:00
parent 4dbca99edf
commit 0a66585b3d
4 changed files with 5 additions and 5 deletions

View File

@ -1247,10 +1247,10 @@ function calendar_get_default_courses($ignoreref = false) {
}
if (isset($CFG->adminseesall)) {
$courses = get_my_courses($USER->id, 'visible DESC', '*', $CFG->adminseesall);
$courses = get_my_courses($USER->id, 'visible DESC', null, $CFG->adminseesall);
}
else {
$courses = get_my_courses($USER->id, 'visible DESC', '*', false);
$courses = get_my_courses($USER->id, 'visible DESC', null, false);
}
// Make sure global events are included
$courses[0] = true;

View File

@ -1300,7 +1300,7 @@ function forum_get_readable_forums($userid, $courseid=0) {
// If no course is specified, then the user can see SITE + his courses.
// And admins can see all courses, so pass the $doanything flag enabled
$courses1 = get_records('course', 'id', SITEID);
$courses2 = get_my_courses($userid, 'visible DESC,sortorder ASC', '*', true);
$courses2 = get_my_courses($userid, 'visible DESC,sortorder ASC', null, true);
$courses = array_merge($courses1, $courses2);
}
if (!$courses) {

View File

@ -55,7 +55,7 @@
/// The main overview in the middle of the page
// limits the number of courses showing up
$courses = get_my_courses($USER->id, 'visible DESC,sortorder ASC', '*', false, 21);
$courses = get_my_courses($USER->id, 'visible DESC,sortorder ASC', null, false, 21);
$site = get_site();
$course = $site; //just in case we need the old global $course hack

View File

@ -300,7 +300,7 @@
profile_display_fields($user->id);
if ($mycourses = get_my_courses($user->id,'visible DESC,sortorder ASC', '*', false, 21)) {
if ($mycourses = get_my_courses($user->id,'visible DESC,sortorder ASC', null, false, 21)) {
$shown=0;
$courselisting = '';
foreach ($mycourses as $mycourse) {