From: poltawski <poltawski>
Date: Tue, 1 Jan 2008 23:03:48 +0000 (+0000)
Subject: MDL-12789 - dont unnecessarily iterate over all groups
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=25c3f91a52ff6720a2d4d6d26af4f27865ccf30c;p=moodle.git

MDL-12789 - dont unnecessarily iterate over all groups
---

diff --git a/enrol/manual/enrol.php b/enrol/manual/enrol.php
index 46030f87b1..1f97c66fae 100644
--- a/enrol/manual/enrol.php
+++ b/enrol/manual/enrol.php
@@ -188,15 +188,16 @@ function check_entry($form, $course) {
 * @param    password  the submitted enrolment key
 */
 function check_group_entry ($courseid, $password) {
-    $ingroup = false;
-    if (($groups = groups_get_all_groups($courseid))) {
+
+    if ($groups = groups_get_all_groups($courseid)) {
         foreach ($groups as $group) {
             if ( !empty($group->enrolmentkey) and (stripslashes($password) == $group->enrolmentkey) ) {
-                $ingroup = $group->id;
+                return $group->id;
             }
         }
     }
-    return $ingroup;
+
+    return false;
 }