From 25c3f91a52ff6720a2d4d6d26af4f27865ccf30c Mon Sep 17 00:00:00 2001
From: poltawski <poltawski>
Date: Tue, 1 Jan 2008 23:03:48 +0000
Subject: [PATCH] MDL-12789 - dont unnecessarily iterate over all groups

---
 enrol/manual/enrol.php | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/enrol/manual/enrol.php b/enrol/manual/enrol.php
index 46030f87b1e..1f97c66fae1 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;
 }