Rollback the previous change: I introduced a bug by mistake! :/

But the system is a bit... strange... shouldn't the "editing mode" flag be
propagated in the session variables or something? It confused me having to
click "Edit" a second time after I 'd just clicked it in the course view.

diff --git a/course/groups.php b/course/groups.php
index 684b462cea3299f1e05a20065a9e9397d5f7a257..2d3161c9a15e904d779d7bb2f19c21e2dcecbab1 100644 (file)
--- a/course/groups.php
+++ b/course/groups.php
@@ -20,10 +20,12 @@
     }
 
     if (isteacheredit($course->id)) {
-        if (isset($_GET['edit']) && $edit == "on") {
-            $USER->groupsediting = true;
-        } else {
-            $USER->groupsediting = false;
+        if (isset($_GET['edit'])) {
+            if($edit == "on") {
+                $USER->groupsediting = true;
+            } else {
+                $USER->groupsediting = false;
+            }
         }
     } else {
         $USER->groupsediting = false;