Merge pull request #169 from catalyst/issue137-V1-debugging-noise

Issue #137: Cleanup debugging noise
catalyst · Mar 22, 2021 · ba4c5b1 · ba4c5b1
2 parents 526f2e3 + aab56d0
commit ba4c5b1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/classes/form/config.php b/classes/form/config.php
@@ -145,24 +145,20 @@ public function definition() {
             $mform->setDefault('dividerselector', 'filler');
         }
 
-        $localid = -1;
-
         foreach ($records as $record) {
 
             $locked = false;
 
             // Local records set in config.php will be locked for editing.
             if (isset($record->local)) {
-                $record->id = $localid;
                 $locked = true;
 
                 $mform->addElement(
                     "hidden",
-                    "locked[{$localid}]",
+                    "locked[{$record->id}]",
                     $locked
                 );
-                $mform->setType("locked[{$localid}]", PARAM_INT);
-                $localid--;
+                $mform->setType("locked[{$record->id}]", PARAM_INT);
             }
 
             $id = $record->id;

diff --git a/classes/local/envbarlib.php b/classes/local/envbarlib.php
@@ -143,8 +143,10 @@ public static function get_records() {
 
             // Converting them to stdClass and adding a local flag.
             foreach ($items as $key => $value) {
-                $value['local'] = true;
-                $items[$key] = (object) $value;
+                $record = (object) $value;
+                $record->id = $key . 'LOCAL';
+                $record->local = true;
+                $items[$key] = $record;
             }
 
             $result = array_merge($items, $result);