Fix the disabledNote(s) field for previous releases of the plugin

* We need to now use disabledNotes but still allow for disabledNote field
moodlehq · Aug 26, 2024 · cb18d84 · cb18d84
1 parent 24e7ea9
commit cb18d84
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/application/src/Controller/ApiController.php b/application/src/Controller/ApiController.php
@@ -87,7 +87,8 @@ public function meetingEnd(string $serverID, Request $request): XmlResponse
         'disableMic',
         'disablePrivateChat',
         'disablePublicChat',
-        'disableNotes',
+        'disableNotes', // This is the real name of the setting (see MDL-82389).
+        'disableNote', // This was the previous name (a typo), but we need to keep it here for backward compatibility.
         'lockedLayout',
         'hideUserList',
         'lockOnJoin',

diff --git a/application/src/Entity/Meeting.php b/application/src/Entity/Meeting.php
@@ -619,13 +619,21 @@ public function setLockSetting(string $lockName, bool $value) {
         if (property_exists($this, $lockName)) {
             $this->$lockName = $value;
         }
+        // Keep the old behavior for disableNote.
+        if ($lockName === 'disableNote') {
+            $this->disableNotes = $value;
+        }
     }
 
     public function getLockSetting(string $lockName): bool {
         if (property_exists($this, $lockName)) {
             // If lockOnJoin not set to true, this is false for every lock.
             return $this->lockOnJoin && $this->$lockName;
         }
+        // Keep the old behavior for disableNote.
+        if ($lockName === 'disableNote') {
+            return $this->disableNotes;
+        }
         return false;
     }