From: nfreear <nfreear>
Date: Thu, 22 Feb 2007 12:59:39 +0000 (+0000)
Subject: Fixes for Bug MDL-8617 "Implement groupings & course modules..."
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=1a9e3005df65f00723c5fc7a9f24942ebaa8e434;p=moodle.git

Fixes for Bug MDL-8617 "Implement groupings & course modules..."
---

diff --git a/group/db/dbgroupinglib.php b/group/db/dbgroupinglib.php
index aa06598334..9971ba23a9 100644
--- a/group/db/dbgroupinglib.php
+++ b/group/db/dbgroupinglib.php
@@ -124,20 +124,19 @@ function groups_db_get_grouping_settings($groupingid) {
  * @return int The id of the grouping or false if there is no such id recorded 
  * or if an error occurred. 
  */
-function groups_db_get_grouping_for_coursemodule($coursemoduleid) {
-	if (!$coursemoduleid) {
-		$groupingid = false;
-	} else {
-		$record = get_record('course_modules', 'id', $coursemoduleid, 'id, ' .
-				             'groupingid');
-		if (!$record) {
-			$groupingid = false;
-		} else {
-			$groupingid = $record->groupingid;
-		}
-	}
-	
-	return $groupingid;
+function groups_db_get_grouping_for_coursemodule($cm) {
+	if (is_object($cm) and isset($cm->course) and isset($cm->groupingid)) {
+        //Do NOT rely on cm->module!
+        return $cm->groupingid;
+    } elseif (is_numeric($cm)) {
+        // Treat param as the course module ID.
+        $coursemoduleid = $cm;
+        $record = get_record('course_modules', 'id', $coursemoduleid, 'id, groupingid');
+        if ($record and isset($record->groupingid)) {
+            return $record->groupingid;
+        }
+    }
+    return false;
 }