From: moodler <moodler>
Date: Tue, 19 Sep 2006 14:23:43 +0000 (+0000)
Subject: Slight extensions to get_my_courses() API to make it more useful in general
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=33f857405b08ff5edec260ef11dc261871036244;p=moodle.git

Slight extensions to get_my_courses() API to make it more useful in general
---

diff --git a/lib/datalib.php b/lib/datalib.php
index 7b20baabd9..8a7d20ee60 100644
--- a/lib/datalib.php
+++ b/lib/datalib.php
@@ -653,14 +653,15 @@ function get_courses_page($categoryid="all", $sort="c.sortorder ASC", $fields="c
  *
  * @uses $CFG
  * @param int $userid The user of interest
- * @param string $sort ?
- * @return object {@link $COURSE} records
+ * @param string $sort the sortorder in the course table
+ * @param string $fields  the fields to return
+ * @return array {@link $COURSE} of course objects
  */
-function get_my_courses($userid, $sort='visible DESC,sortorder ASC') {
+function get_my_courses($userid, $sort='visible DESC,sortorder ASC', $fields='*') {
 
     $mycourses = array();
 
-    $rs = get_recordset('course', '', '', $sort, '*');
+    $rs = get_recordset('course', '', '', $sort, $fields);
 
     if ($rs && $rs->RecordCount() > 0) {
         while (!$rs->EOF) {
@@ -673,7 +674,7 @@ function get_my_courses($userid, $sort='visible DESC,sortorder ASC') {
                 $context = get_context_instance(CONTEXT_COURSE, $course->id);
                 if (has_capability('moodle/course:view', $context, $userid) && 
                     ($course->visible || has_capability('moodle/course:viewhiddencourses', $context, $userid))) {
-                    $mycourses[] = $course;
+                    $mycourses[$course->id] = $course;
                 }
             }