blindsidenetworks-ps · jfederico · Mar 20, 2024 · Nov 5, 2023
diff --git a/backup/moodle2/backup_bigbluebuttonbn_stepslib.php b/backup/moodle2/backup_bigbluebuttonbn_stepslib.php
@@ -51,18 +51,18 @@ protected function define_structure() {
             'recordings_html', 'recordings_imported', 'recordings_preview',
             'clienttype', 'muteonstart', 'completionattendance',
             'completionengagementchats', 'completionengagementtalks', 'completionengagementraisehand',
-            'completionengagementpollvotes', 'completionengagementemojis']);
+            'completionengagementpollvotes', 'completionengagementemojis', ]);
 
         $logs = new backup_nested_element('logs');
 
         $log = new backup_nested_element('log', ['id'], [
-            'courseid', 'bigbluebuttonbnid', 'userid', 'timecreated', 'meetingid', 'log', 'meta']);
+            'courseid', 'bigbluebuttonbnid', 'userid', 'timecreated', 'meetingid', 'log', 'meta', ]);
 
         $recordings = new backup_nested_element('recordings');
 
         $recording = new backup_nested_element('recording', ['id'], [
             'courseid', 'bigbluebuttonbnid', 'groupid', 'recordingid', 'headlesss', 'imported', 'status', 'importeddata',
-            'timecreated']);
+            'timecreated', ]);
 
         // Build the tree.
         $bigbluebuttonbn->add_child($logs);

diff --git a/classes/broker.php b/classes/broker.php
@@ -34,10 +34,10 @@ class broker {
     protected $requiredparams = [
         'recording_ready' => [
             'bigbluebuttonbn' => 'The BigBlueButtonBN instance ID must be specified.',
-            'signed_parameters' => 'A JWT encoded string must be included as [signed_parameters].'
+            'signed_parameters' => 'A JWT encoded string must be included as [signed_parameters].',
         ],
         'meeting_events' => [
-            'bigbluebuttonbn' => 'The BigBlueButtonBN instance ID must be specified.'
+            'bigbluebuttonbn' => 'The BigBlueButtonBN instance ID must be specified.',
         ],
     ];
 
@@ -89,7 +89,7 @@ public static function process_recording_ready(instance $instance, array $params
             $decodedparameters = JWT::decode(
                 $params['signed_parameters'],
                 config::get('shared_secret'),
-                array('HS256')
+                ['HS256'],
             );
         } catch (Exception $e) {
             $error = 'Caught exception: ' . $e->getMessage();
@@ -153,7 +153,7 @@ public static function process_meeting_events(instance $instance) {
             $token = JWT::decode(
                 $authorization[1],
                 config::get('shared_secret'),
-                array('HS512')
+                ['HS512'],
             );
 
             // Get JSON string from the body.

diff --git a/classes/event/base.php b/classes/event/base.php
@@ -71,7 +71,7 @@ public function get_description() {
             'courseid' => $this->courseid,
             'objectid' => $this->objectid,
             'contextinstanceid' => $this->contextinstanceid,
-            'other' => $this->other
+            'other' => $this->other,
         ];
         $string = $this->description;
         foreach ($vars as $key => $value) {

diff --git a/classes/event/events.php b/classes/event/events.php
@@ -46,6 +46,6 @@ class events {
         'recording_unpublish' => 'recording_unpublished',
         'recording_edit' => 'recording_edited',
         'recording_play' => 'recording_viewed',
-        'live_session' => 'live_session'
+        'live_session' => 'live_session',
     ];
 }
diff --git a/classes/external/completion_validate.php b/classes/external/completion_validate.php
@@ -85,15 +85,15 @@ public static function execute(
                     'item' => 'mod_bigbluebuttonbn',
                     'itemid' => $instance->get_instance_id(),
                     'warningcode' => 'nopermissions',
-                    'message' => get_string('nopermissions', 'error', 'completion_validate')
+                    'message' => get_string('nopermissions', 'error', 'completion_validate'),
                 ];
             }
         } else {
             $result['warnings'][] = [
                 'item' => 'mod_bigbluebuttonbn',
                 'itemid' => $bigbluebuttonbnid,
                 'warningcode' => 'indexerrorbbtn',
-                'message' => get_string('index_error_bbtn', 'mod_bigbluebuttonbn', $bigbluebuttonbnid)
+                'message' => get_string('index_error_bbtn', 'mod_bigbluebuttonbn', $bigbluebuttonbnid),
             ];
         }
         // We might want to return a status here or some warnings.
@@ -108,7 +108,7 @@ public static function execute(
      */
     public static function execute_returns(): external_single_structure {
         return new external_single_structure([
-                'warnings' => new \external_warnings()
+                'warnings' => new \external_warnings(),
             ]
         );
     }

diff --git a/classes/external/end_meeting.php b/classes/external/end_meeting.php
@@ -105,9 +105,9 @@ public static function execute(
                         'item' => $instance->get_meeting_name(),
                         'itemid' => $instance->get_instance_id(),
                         'warningcode' => 'notFound',
-                        'message' => $e->getMessage()
-                    ]
-                ]
+                        'message' => $e->getMessage(),
+                    ],
+                ],
             ];
         }
         logger::log_meeting_ended_event($instance);
@@ -125,7 +125,7 @@ public static function execute(
      */
     public static function execute_returns(): external_single_structure {
         return new external_single_structure([
-            'warnings' => new \external_warnings()
+            'warnings' => new \external_warnings(),
         ]);
     }
 }
diff --git a/classes/external/get_bigbluebuttonbns_by_courses.php b/classes/external/get_bigbluebuttonbns_by_courses.php
@@ -101,7 +101,7 @@ public static function execute($courseids = []) {
 
         $result = [
             'bigbluebuttonbns' => $returnedbigbluebuttonbns,
-            'warnings' => $warnings
+            'warnings' => $warnings,
         ];
         return $result;
     }

diff --git a/classes/external/get_join_url.php b/classes/external/get_join_url.php
@@ -94,7 +94,7 @@ public static function execute(
                 'item' => 'mod_bigbluebuttonbn',
                 'itemid' => $instance->get_instance_id(),
                 'warningcode' => $e->errorcode,
-                'message' => $e->getMessage()
+                'message' => $e->getMessage(),
             ];
         }
         return $result;
@@ -109,7 +109,7 @@ public static function execute(
     public static function execute_returns(): external_single_structure {
         return new external_single_structure([
             'join_url' => new external_value(PARAM_RAW, 'Can join session', VALUE_OPTIONAL),
-            'warnings' => new \external_warnings()
+            'warnings' => new \external_warnings(),
         ]);
     }
 }
diff --git a/classes/external/get_recordings.php b/classes/external/get_recordings.php
@@ -97,7 +97,7 @@ public static function execute(
                 'item' => $bigbluebuttonbnid,
                 'warningcode' => 'nosuchinstance',
                 'message' => get_string('nosuchinstance', 'mod_bigbluebuttonbn',
-                    (object) ['id' => $bigbluebuttonbnid, 'entity' => 'bigbluebuttonbn'])
+                    (object) ['id' => $bigbluebuttonbnid, 'entity' => 'bigbluebuttonbn']),
             ];
         } else {
             $typeprofiles = bigbluebutton_proxy::get_instance_type_profiles();
@@ -122,7 +122,7 @@ public static function execute(
                 $returnval['warnings'][] = [
                     'item' => $bigbluebuttonbnid,
                     'warningcode' => 'instanceprofilewithoutrecordings',
-                    'message' => get_string('instanceprofilewithoutrecordings', 'mod_bigbluebuttonbn')
+                    'message' => get_string('instanceprofilewithoutrecordings', 'mod_bigbluebuttonbn'),
                 ];
             }
         }
@@ -154,7 +154,7 @@ public static function execute_returns(): external_single_structure {
                 ])),
                 'data' => new external_value(PARAM_RAW), // For now it will be json encoded.
             ], '', VALUE_OPTIONAL),
-            'warnings' => new external_warnings()
+            'warnings' => new external_warnings(),
         ]);
     }
 }
diff --git a/classes/external/get_recordings_to_import.php b/classes/external/get_recordings_to_import.php
@@ -105,7 +105,7 @@ public static function execute(
             'sourcebigbluebuttonbnid' => $sourcebigbluebuttonbnid,
             'sourcecourseid' => $sourcecourseid,
             'tools' => $tools,
-            'groupid' => $groupid
+            'groupid' => $groupid,
         ]);
 
         $tools = explode(',', $tools ?? 'protect,unprotect,publish,unpublish,delete');
@@ -200,7 +200,7 @@ public static function execute_returns(): external_single_structure {
                 ])),
                 'data' => new external_value(PARAM_RAW), // For now it will be json encoded.
             ], '', VALUE_OPTIONAL),
-            'warnings' => new external_warnings()
+            'warnings' => new external_warnings(),
         ]);
     }
 }
diff --git a/classes/external/view_bigbluebuttonbn.php b/classes/external/view_bigbluebuttonbn.php
@@ -81,9 +81,9 @@ public static function execute($instanceid) {
                         'itemid' => 0,
                         'warningcode' => 'nosuchinstance',
                         'message' => get_string('nosuchinstance', 'mod_bigbluebuttonbn',
-                            (object) ['id' => $instanceid, 'entity' => 'bigbluebuttonbn'])
-                    ]
-                ]
+                            (object) ['id' => $instanceid, 'entity' => 'bigbluebuttonbn']),
+                    ],
+                ],
             ];
         }
         $context = context_module::instance($instance->get_cm_id());
@@ -94,7 +94,7 @@ public static function execute($instanceid) {
         bigbluebuttonbn_view($instance->get_instance_data(), $instance->get_course(), $instance->get_cm(), $context);
         return [
             'status' => true,
-            'warnings' => []
+            'warnings' => [],
         ];
     }
 
@@ -107,7 +107,7 @@ public static function execute($instanceid) {
     public static function execute_returns() {
         return new external_single_structure([
                 'status' => new external_value(PARAM_BOOL, 'status: true if success'),
-                'warnings' => new external_warnings()
+                'warnings' => new external_warnings(),
             ]
         );
     }

diff --git a/classes/local/bigbluebutton/recordings/recording_action.php b/classes/local/bigbluebutton/recordings/recording_action.php
@@ -49,7 +49,7 @@ public static function delete(recording $recording) {
         // As the recordingid was not identified as imported recording link, execute delete on a real recording.
         // Step 1, delete imported links associated to the recording.
         $recordingstodelete = recording::get_records(['recordingid' => $recording->get('recordingid'),
-            'imported' => true]);
+            'imported' => true, ]);
         foreach ($recordingstodelete as $rec) {
             $rec->delete();
         }

diff --git a/classes/local/config.php b/classes/local/config.php
@@ -42,7 +42,7 @@ class config {
     const CHECKSUM_ALGORITHMS = [
         self::DEFAULT_CHECKSUM_ALGORITHM,
         'SHA256',
-        'SHA512'
+        'SHA512',
     ];
 
     /**
@@ -117,7 +117,7 @@ protected static function defaultvalues() {
             'lockonjoin_default' => true,
             'lockonjoin_editable' => false,
             'welcome_default' => '',
-            'checksum_algorithm' => self::DEFAULT_CHECKSUM_ALGORITHM
+            'checksum_algorithm' => self::DEFAULT_CHECKSUM_ALGORITHM,
         ];
     }
 

diff --git a/classes/local/helpers/mod_helper.php b/classes/local/helpers/mod_helper.php
@@ -174,7 +174,7 @@ protected static function process_post_save_event(stdClass $bigbluebuttonbn): vo
         $eventid = $DB->get_field('event', 'id', [
             'modulename' => 'bigbluebuttonbn',
             'instance' => $bigbluebuttonbn->id,
-            'eventtype' => logger::EVENT_MEETING_START
+            'eventtype' => logger::EVENT_MEETING_START,
         ]);
 
         // Delete the event from calendar when/if openingtime is NOT set.

diff --git a/classes/local/helpers/reset.php b/classes/local/helpers/reset.php
@@ -101,7 +101,7 @@ public static function reset_events($courseid) {
     public static function reset_getstatus(string $item): array {
         return ['component' => get_string('modulenameplural', 'bigbluebuttonbn'),
             'item' => get_string("removed{$item}", 'bigbluebuttonbn'),
-            'error' => false];
+            'error' => false, ];
     }
 
     /**

diff --git a/classes/local/helpers/roles.php b/classes/local/helpers/roles.php
@@ -209,12 +209,12 @@ public static function get_participant_data(context $context, ?stdClass $bbactiv
         $data = [
             'all' => [
                 'name' => get_string('mod_form_field_participant_list_type_all', 'bigbluebuttonbn'),
-                'children' => []
+                'children' => [],
             ],
         ];
         $data['role'] = [
             'name' => get_string('mod_form_field_participant_list_type_role', 'bigbluebuttonbn'),
-            'children' => self::get_roles_select($context, true)
+            'children' => self::get_roles_select($context, true),
         ];
         $data['user'] = [
             'name' => get_string('mod_form_field_participant_list_type_user', 'bigbluebuttonbn'),
@@ -253,7 +253,7 @@ public static function get_participant_list(?stdClass $bigbluebuttonbn, context
      * Returns an array to populate a list of participants used in mod_form.php with default values.
      *
      * @param context $context
-     * @param int $ownerid
+     * @param int|null $ownerid
      *
      * @return array
      */
@@ -271,14 +271,14 @@ protected static function get_participant_list_default(context $context, ?int $o
                     $participantlist[] = [
                         'selectiontype' => 'user',
                         'selectionid' => (string) $ownerid,
-                        'role' => self::ROLE_MODERATOR];
+                        'role' => self::ROLE_MODERATOR, ];
                 }
                 continue;
             }
             $participantlist[] = [
                 'selectiontype' => 'role',
                 'selectionid' => $defaultrule,
-                'role' => self::ROLE_MODERATOR];
+                'role' => self::ROLE_MODERATOR, ];
         }
         return $participantlist;
     }
@@ -329,7 +329,7 @@ public static function get_participant_selection_data(): array {
      *
      * @param context $context
      * @param array $participantlist
-     * @param int $userid
+     * @param int|null $userid
      *
      * @return bool
      */
@@ -420,7 +420,7 @@ public static function participant_joined(string $meetingid, bool $ismoderator):
             $meetinginfo->moderatorCount += 1;
         }
         $cache->set($meetingid, ['creation_time' => $result['creation_time'],
-            'meeting_info' => json_encode($meetinginfo)]);
+            'meeting_info' => json_encode($meetinginfo), ]);
     }
 
     /**

diff --git a/classes/local/helpers/user_info.php b/classes/local/helpers/user_info.php
@@ -36,7 +36,7 @@ class user_info {
      */
     const EVENT_TO_WATCH = [
         'join' => logger::EVENT_JOIN,
-        'play_recording' => logger::EVENT_PLAYED
+        'play_recording' => logger::EVENT_PLAYED,
     ];
 
     /**

diff --git a/classes/local/proxy/bigbluebutton_proxy.php b/classes/local/proxy/bigbluebutton_proxy.php
@@ -69,7 +69,7 @@ public static function get_join_url(
             'fullName' => $username,
             'password' => $pw,
             'logoutURL' => $logouturl,
-            'role' => $role
+            'role' => $role,
         ];
 
         if (!is_null($configtoken)) {
@@ -262,20 +262,20 @@ public static function get_instance_type_profiles(): array {
             instance::TYPE_ALL => [
                 'id' => instance::TYPE_ALL,
                 'name' => get_string('instance_type_default', 'bigbluebuttonbn'),
-                'features' => ['all']
+                'features' => ['all'],
             ],
             instance::TYPE_ROOM_ONLY => [
                 'id' => instance::TYPE_ROOM_ONLY,
                 'name' => get_string('instance_type_room_only', 'bigbluebuttonbn'),
                 'features' => ['showroom', 'welcomemessage', 'voicebridge', 'waitformoderator', 'userlimit',
                     'recording', 'sendnotifications', 'lock', 'preuploadpresentation', 'permissions', 'schedule', 'groups',
                     'modstandardelshdr', 'availabilityconditionsheader', 'tagshdr', 'competenciessection',
-                    'completionattendance', 'completionengagement', 'availabilityconditionsheader']
+                    'completionattendance', 'completionengagement', 'availabilityconditionsheader', ],
             ],
             instance::TYPE_RECORDING_ONLY => [
                 'id' => instance::TYPE_RECORDING_ONLY,
                 'name' => get_string('instance_type_recording_only', 'bigbluebuttonbn'),
-                'features' => ['showrecordings', 'importrecordings', 'availabilityconditionsheader']
+                'features' => ['showrecordings', 'importrecordings', 'availabilityconditionsheader'],
             ],
         ];
         return $instanceprofiles;
@@ -439,7 +439,7 @@ public static function create_meeting(
             'meetingID' => (string) $xml->meetingID,
             'internalMeetingID' => (string) $xml->internalMeetingID,
             'attendeePW' => (string) $xml->attendeePW,
-            'moderatorPW' => (string) $xml->moderatorPW
+            'moderatorPW' => (string) $xml->moderatorPW,
         ];
     }
 

diff --git a/classes/local/proxy/curl.php b/classes/local/proxy/curl.php
@@ -92,7 +92,7 @@ public function post($url, $params = '', $options = []) {
                 'Content-Type: ' . $this->get_content_type(),
                 'Content-Length: ' . strlen($params),
                 'Content-Language: en-US',
-            ]
+            ],
         ]);
 
         return $this->handle_response(parent::post($url, $params, $options));