Merge from MOODLE_14_STABLE: moodle--eduforge--1.3.3--patch-201 - optimized calls to get_courses() to retrieve only required fields (martinlanghoff)
This commit is contained in:
+1
-1
@@ -168,7 +168,7 @@
|
||||
$movecourse = NULL;
|
||||
$swapcourse = NULL;
|
||||
|
||||
$courses = get_courses($category->id);
|
||||
$courses = get_courses($category->id,'c.sortorder ASC', 'c.id,c.sortorder');
|
||||
|
||||
if (isset($moveup)) {
|
||||
if ($movecourse = get_record("course", "id", $moveup)) {
|
||||
|
||||
+1
-1
@@ -245,7 +245,7 @@
|
||||
}
|
||||
|
||||
/// Find any orphan courses that don't yet have a valid category and set to default
|
||||
if ($courses = get_courses()) {
|
||||
if ($courses = get_courses(NULL,NULL,'c.category,c.id,c.sortorder')) {
|
||||
foreach ($courses as $course) {
|
||||
if ($course->category and !isset($categories[$course->category])) {
|
||||
set_field("course", "category", $default, "id", $course->id);
|
||||
|
||||
+4
-4
@@ -1149,7 +1149,7 @@ function print_category_info($category, $depth) {
|
||||
echo "\n\n".'<table border="0" cellpadding="3" cellspacing="0" width="100%">';
|
||||
|
||||
if ($CFG->frontpage == FRONTPAGECOURSELIST) {
|
||||
$courses = get_courses($category->id);
|
||||
$courses = get_courses($category->id, NULL, 'c.id,c.sortorder,c.visible,c.fullname,c.shortname,c.password,c.summary');
|
||||
|
||||
echo "<tr>";
|
||||
|
||||
@@ -1224,14 +1224,14 @@ function print_courses($category, $width="100%") {
|
||||
$categories = get_categories(0); // Parent = 0 ie top-level categories only
|
||||
if (count($categories) == 1) {
|
||||
$category = array_shift($categories);
|
||||
$courses = get_courses($category->id);
|
||||
$courses = get_courses($category->id, NULL, 'c.id,c.sortorder,c.visible,c.fullname,c.shortname,c.password,c.summary');
|
||||
} else {
|
||||
$courses = get_courses("all");
|
||||
$courses = get_courses("all", NULL, 'c.id,c.sortorder,c.visible,c.fullname,c.shortname,c.password,c.summary');
|
||||
}
|
||||
unset($categories);
|
||||
} else {
|
||||
$categories = get_categories($category->id); // sub categories
|
||||
$courses = get_courses($category->id);
|
||||
$courses = get_courses($category->id, NULL, 'c.id,c.sortorder,c.visible,c.fullname,c.shortname,c.password,c.summary');
|
||||
}
|
||||
|
||||
if ($courses) {
|
||||
|
||||
Reference in New Issue
Block a user