blindsidenetworks / moodle-mod_bigbluebuttonbn

bbb_broker.php

Doc Comments +15 added lines

@@ -181,6 +181,9 @@ 
     return "{$params['callback']}({$callbackresponsedata});";
 }
 
+/**
+ * @param boolean $running
+ */
 function bigbluebuttonbn_broker_meeting_info_can_join($bbbsession, $running, $participantcount) {
     $status = array("can_join" => false);
     if ($running) {
@@ -201,6 +204,9 @@ 
     return $status;
 }
 
+/**
+ * @param boolean $running
+ */
 function bigbluebuttonbn_broker_meeting_info_can_end($bbbsession, $running) {
     $status = array("can_end" => false);
     if ($running && ($bbbsession['administrator'] || $bbbsession['moderator'])) {
@@ -348,6 +354,9 @@ 
     return "{$params['callback']}({$callbackresponsedata});";
 }
 
+/**
+ * @param string $action
+ */
 function bigbluebuttonbn_broker_recording_action_perform($action, $bbbsession, $params, $recordings) {
     if ($action == 'recording_publish') {
         return bigbluebuttonbn_broker_recording_action_publish($bbbsession, $params, $recordings);
@@ -377,6 +386,9 @@ 
     return bigbluebuttonbn_broker_recording_action_publishunprotect($bbbsession, $params, $recordings, 'unprotect');
 }
 
+/**
+ * @param string $action
+ */
 function bigbluebuttonbn_broker_recording_action_publishunprotect($bbbsession, $params, $recordings, $action) {
     $status = true;
     if (bigbluebuttonbn_broker_recording_is_imported($recordings, $params['id'])) {
@@ -408,6 +420,9 @@ 
     return bigbluebuttonbn_broker_recording_action_unpublishprotect($bbbsession, $params, $recordings, 'protect');
 }
 
+/**
+ * @param string $action
+ */
 function bigbluebuttonbn_broker_recording_action_unpublishprotect($bbbsession, $params, $recordings, $action) {
     global $DB;
 

Indentation +17 added lines, -17 removed lines

@@ -414,10 +414,10 @@ 
     if (bigbluebuttonbn_broker_recording_is_imported($recordings, $params['id'])) {
         // Execute unpublish or protect on imported recording link.
         return array(
-          'status' => bigbluebuttonbn_publish_recording_imported(
-              $params['id'], $bbbsession['bigbluebuttonbn']->id, false
+            'status' => bigbluebuttonbn_publish_recording_imported(
+                $params['id'], $bbbsession['bigbluebuttonbn']->id, false
             )
-          );
+            );
     }
 
     // As the recordingid was not identified as imported recording link, execute unpublish on a real recording.
@@ -437,26 +437,26 @@ 
     }
     // Second: Execute the real unpublish.
     return array(
-      'status' => bigbluebuttonbn_publish_recordings($params['id'], 'false')
-      );
+        'status' => bigbluebuttonbn_publish_recordings($params['id'], 'false')
+        );
 }
 
 function bigbluebuttonbn_broker_recording_action_edit($bbbsession, $params, $recordings) {
     if (bigbluebuttonbn_broker_recording_is_imported($recordings, $params['id'])) {
         // Execute update on imported recording link.
         return array(
-          'status' => bigbluebuttonbn_update_recording_imported(
-              $params['id'], $bbbsession['bigbluebuttonbn']->id, json_decode($params['meta'], true)
+            'status' => bigbluebuttonbn_update_recording_imported(
+                $params['id'], $bbbsession['bigbluebuttonbn']->id, json_decode($params['meta'], true)
             )
-          );
+            );
     }
 
     // As the recordingid was not identified as imported recording link, execute update on a real recording.
     // (No need to update imported links as the update only affects the actual recording).
     // Execute update on actual recording.
     return array(
-      'status' => bigbluebuttonbn_update_recordings($params['id'], json_decode($params['meta']))
-      );
+        'status' => bigbluebuttonbn_update_recordings($params['id'], json_decode($params['meta']))
+        );
 }
 
 function bigbluebuttonbn_broker_recording_action_delete($bbbsession, $params, $recordings) {
@@ -465,10 +465,10 @@ 
     if (bigbluebuttonbn_broker_recording_is_imported($recordings, $params['id'])) {
         // Execute delete on imported recording link.
         return array(
-          'status' => bigbluebuttonbn_delete_recording_imported(
-              $recordings[$params['id']]['imported']
+            'status' => bigbluebuttonbn_delete_recording_imported(
+                $recordings[$params['id']]['imported']
             )
-          );
+            );
     }
 
     // As the recordingid was not identified as imported recording link, execute delete on a real recording.
@@ -483,8 +483,8 @@ 
     }
     // Execute the actual delete.
     return array(
-      'status' => bigbluebuttonbn_delete_recordings($params['id'])
-      );
+        'status' => bigbluebuttonbn_delete_recordings($params['id'])
+        );
 }
 
 function bigbluebuttonbn_broker_recording_ready($params, $bigbluebuttonbn) {
@@ -625,10 +625,10 @@ 
     $params['recording_import'] = ['id' => 'The recordingID must be specified.'];
     $params['recording_ready'] = [
             'signed_parameters' => 'A JWT encoded string must be included as [signed_parameters].'
-          ];
+            ];
     $params['live_session_events'] = [
             'signed_parameters' => 'A JWT encoded string must be included as [signed_parameters].'
-          ];
+            ];
     return $params;
 }
 

Spacing +19 added lines, -19 removed lines

@@ -22,8 +22,8 @@ 
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v2 or later
  */
 
-require_once(dirname(dirname(dirname(__FILE__))).'/config.php');
-require_once(dirname(__FILE__).'/locallib.php');
+require_once(dirname(dirname(dirname(__FILE__))) . '/config.php');
+require_once(dirname(__FILE__) . '/locallib.php');
 
 use \Firebase\JWT\JWT;
 
@@ -39,13 +39,13 @@ 
 $params['meta'] = optional_param('meta', '', PARAM_TEXT);
 
 if (empty($params['action'])) {
-    header('HTTP/1.0 400 Bad Request. Parameter ['.$params['action'].'] was not included');
+    header('HTTP/1.0 400 Bad Request. Parameter [' . $params['action'] . '] was not included');
     return;
 }
 
 $error = bigbluebuttonbn_broker_validate_parameters($params);
 if (!empty($error)) {
-    header('HTTP/1.0 400 Bad Request. '.$error);
+    header('HTTP/1.0 400 Bad Request. ' . $error);
     return;
 }
 
@@ -139,11 +139,11 @@ 
         return;
     }
 
-    header('HTTP/1.0 400 Bad request. The action '. $a . ' doesn\'t exist');
+    header('HTTP/1.0 400 Bad request. The action ' . $a . ' doesn\'t exist');
     return;
 
 } catch (Exception $e) {
-    header('HTTP/1.0 500 Internal Server Error. '.$e->getMessage());
+    header('HTTP/1.0 500 Internal Server Error. ' . $e->getMessage());
     return;
 }
 
@@ -290,7 +290,7 @@ 
 function bigbluebuttonbn_broker_recording_info_current($recording, $params) {
     $callbackresponse['status'] = true;
     $callbackresponse['found'] = true;
-    $callbackresponse['published'] = (string) $recording['published'];
+    $callbackresponse['published'] = (string)$recording['published'];
     if (!isset($params['meta'])) {
         return $callbackresponse;
     }
@@ -383,7 +383,7 @@ 
         // Execute publish or unprotect on imported recording link, if the real recording is published.
         $realrecordings = bigbluebuttonbn_get_recordings_array(
             $recordings[$params['id']]['meetingID'], $recordings[$params['id']]['recordID']);
-        $status = ($realrecordings[$params['id']][$action.'ed'] === 'true');
+        $status = ($realrecordings[$params['id']][$action . 'ed'] === 'true');
         if ($status) {
             // Only if the physical recording is published, execute publish on imported recording link.
             bigbluebuttonbn_publish_recording_imported($params['id'], $bbbsession['bigbluebuttonbn']->id, true);
@@ -395,7 +395,7 @@ 
 
     $response = array('status' => $status);
     if (!$status) {
-        $response['message'] = get_string('view_recording_'.$action.'_link_error', 'bigbluebuttonbn');
+        $response['message'] = get_string('view_recording_' . $action . '_link_error', 'bigbluebuttonbn');
     }
     return $response;
 }
@@ -428,7 +428,7 @@ 
         foreach ($importedall as $key => $record) {
             $meta = json_decode($record->meta, true);
             // Prepare data for the update.
-            $meta['recording'][$action.'ed'] = 'false';
+            $meta['recording'][$action . 'ed'] = 'false';
             $importedall[$key]->meta = json_encode($meta);
 
             // Proceed with the update.
@@ -494,8 +494,8 @@ 
         $decodedparameters = JWT::decode($params['signed_parameters'], \mod_bigbluebuttonbn\locallib\config::get('shared_secret'),
             array('HS256'));
     } catch (Exception $e) {
-        $error = 'Caught exception: '.$e->getMessage();
-        header('HTTP/1.0 400 Bad Request. '.$error);
+        $error = 'Caught exception: ' . $e->getMessage();
+        header('HTTP/1.0 400 Bad Request. ' . $error);
         return;
     }
 
@@ -513,8 +513,8 @@ 
         bigbluebuttonbn_send_notification_recording_ready($bigbluebuttonbn);
         header('HTTP/1.0 202 Accepted');
     } catch (Exception $e) {
-        $error = 'Caught exception: '.$e->getMessage();
-        header('HTTP/1.0 503 Service Unavailable. '.$error);
+        $error = 'Caught exception: ' . $e->getMessage();
+        header('HTTP/1.0 503 Service Unavailable. ' . $error);
     }
 }
 
@@ -529,7 +529,7 @@ 
     $importrecordings = $SESSION->bigbluebuttonbn_importrecordings;
     if (!isset($importrecordings[$params['id']])) {
         $error = "Recording {$params['id']} could not be found. It can not be imported";
-        header('HTTP/1.0 404 Not found. '.$error);
+        header('HTTP/1.0 404 Not found. ' . $error);
         return;
     }
 
@@ -537,7 +537,7 @@ 
 
     $importrecordings[$params['id']]['imported'] = true;
     $overrides = array('meetingid' => $importrecordings[$params['id']]['meetingID']);
-    $meta = '{"recording":'.json_encode($importrecordings[$params['id']]).'}';
+    $meta = '{"recording":' . json_encode($importrecordings[$params['id']]) . '}';
     bigbluebuttonbn_logs($bbbsession, BIGBLUEBUTTONBN_LOG_EVENT_IMPORT, $overrides, $meta);
     // Moodle event logger: Create an event for recording imported.
     if (isset($bbbsession['bigbluebutton']) && isset($bbbsession['cm'])) {
@@ -555,8 +555,8 @@ 
         $decodedparameters = JWT::decode($params['signed_parameters'], \mod_bigbluebuttonbn\locallib\config::get('shared_secret'),
             array('HS256'));
     } catch (Exception $e) {
-        $error = 'Caught exception: '.$e->getMessage();
-        header('HTTP/1.0 400 Bad Request. '.$error);
+        $error = 'Caught exception: ' . $e->getMessage();
+        header('HTTP/1.0 400 Bad Request. ' . $error);
         return;
     }
 
@@ -594,7 +594,7 @@ 
 
     $action = strtolower($params['action']);
     if (!array_key_exists($action, $requiredparams)) {
-        return 'Action '.$params['action'].' can not be performed.';
+        return 'Action ' . $params['action'] . ' can not be performed.';
     }
 
     return bigbluebuttonbn_broker_validate_parameters_message($params, $requiredparams[$action]);

locallib.php

Doc Comments +15 added lines, -2 removed lines

@@ -56,6 +56,9 @@ 
 const BIGBLUEBUTTON_EVENT_RECORDING_EDITED = 'recording_edited';
 const BIGBLUEBUTTON_EVENT_RECORDING_VIEWED = 'recording_viewed';
 
+/**
+ * @param string $meta
+ */
 function bigbluebuttonbn_logs(array $bbbsession, $event, array $overrides = [], $meta = null) {
     global $DB;
 
@@ -585,6 +588,12 @@ 
     }
 }
 
+/**
+ * @param string $url
+ * @param string $data
+ *
+ * @return string
+ */
 function bigbluebuttonbn_wrap_xml_load_file_curl_request($url, $method = BIGBLUEBUTTONBN_METHOD_GET,
     $data = null, $contenttype = 'text/xml') {
     $c = new curl();
@@ -760,6 +769,9 @@ 
       ];
 }
 
+/**
+ * @param string $participants
+ */
 function bigbluebuttonbn_is_moderator($context, $participants, $userid = null, $userroles = null) {
     global $USER;
     if (empty($userid)) {
@@ -1023,8 +1035,6 @@ 
 }
 
 /**
- * @param string $recordingid
- * @param string $bigbluebuttonbnid
  */
 function bigbluebuttonbn_delete_recording_imported($id) {
     global $DB;
@@ -1475,6 +1485,9 @@ 
     return html_writer::div(html_writer::table($table), '', array('id' => 'bigbluebuttonbn_html_table'));
 }
 
+/**
+ * @param integer $len
+ */
 function bigbluebuttonbn_html2text($html, $len) {
     $text = strip_tags($html);
     $text = str_replace(' ', ' ', $text);

Spacing +97 added lines, -97 removed lines

@@ -27,7 +27,7 @@ 
 
 global $CFG;
 
-require_once(dirname(__FILE__).'/lib.php');
+require_once(dirname(__FILE__) . '/lib.php');
 
 const BIGBLUEBUTTONBN_FORCED = true;
 
@@ -76,7 +76,7 @@ 
     if (isset($meta)) {
         $log->meta = $meta;
     } else if ($event == BIGBLUEBUTTONBN_LOG_EVENT_CREATE) {
-        $log->meta = '{"record":'.($bbbsession['record'] ? 'true' : 'false').'}';
+        $log->meta = '{"record":' . ($bbbsession['record'] ? 'true' : 'false') . '}';
     }
 
     $DB->insert_record('bigbluebuttonbn_logs', $log);
@@ -121,8 +121,8 @@ 
 
     if (!is_null($pname) && !is_null($purl)) {
         $method = BIGBLUEBUTTONBN_METHOD_POST;
-        $data = "<?xml version='1.0' encoding='UTF-8'?><modules><module name='" . $pname . "'><document url='".
-            $purl."' /></module></modules>";
+        $data = "<?xml version='1.0' encoding='UTF-8'?><modules><module name='" . $pname . "'><document url='" .
+            $purl . "' /></module></modules>";
     }
 
     $xml = bigbluebuttonbn_wrap_xml_load_file($createmeetingurl, $method, $data);
@@ -167,7 +167,7 @@ 
                                 'moderatorPW' => $meeting->moderatorPW,
                                 'attendeePW' => $meeting->attendeePW,
                                 'hasBeenForciblyEnded' => $meeting->hasBeenForciblyEnded,
-                                'running' => $meeting->running, );
+                                'running' => $meeting->running,);
         }
 
         return $meetings;
@@ -345,37 +345,37 @@ 
 function bigbluebuttonbn_get_default_config_xml_array() {
     $defaultconfigxml = bigbluebuttonbn_getDefaultConfigXML();
 
-    return (array) $defaultconfigxml;
+    return (array)$defaultconfigxml;
 }
 
 function bigbluebuttonbn_get_recording_array_value($recording) {
     // Add formats.
     $playbackarray = array();
     foreach ($recording->playback->format as $format) {
-        $playbackarray[(string) $format->type] = array('type' => (string) $format->type,
-            'url' => (string) $format->url, 'length' => (string) $format->length);
+        $playbackarray[(string)$format->type] = array('type' => (string)$format->type,
+            'url' => (string)$format->url, 'length' => (string)$format->length);
         // Add preview per format when existing.
         if ($format->preview) {
             $imagesarray = array();
             foreach ($format->preview->images->image as $image) {
-                $imagearray = array('url' => (string) $image);
+                $imagearray = array('url' => (string)$image);
                 foreach ($image->attributes() as $attkey => $attvalue) {
-                    $imagearray[$attkey] = (string) $attvalue;
+                    $imagearray[$attkey] = (string)$attvalue;
                 }
                 array_push($imagesarray, $imagearray);
             }
-            $playbackarray[(string) $format->type]['preview'] = $imagesarray;
+            $playbackarray[(string)$format->type]['preview'] = $imagesarray;
         }
     }
 
     // Add the metadata to the recordings array.
     $metadataarray = bigbluebuttonbn_get_recording_array_meta(get_object_vars($recording->metadata));
-    $recordingarray = array('recordID' => (string) $recording->recordID,
-        'meetingID' => (string) $recording->meetingID, 'meetingName' => (string) $recording->name,
-        'published' => (string) $recording->published, 'startTime' => (string) $recording->startTime,
-        'endTime' => (string) $recording->endTime, 'playbacks' => $playbackarray);
+    $recordingarray = array('recordID' => (string)$recording->recordID,
+        'meetingID' => (string)$recording->meetingID, 'meetingName' => (string)$recording->name,
+        'published' => (string)$recording->published, 'startTime' => (string)$recording->startTime,
+        'endTime' => (string)$recording->endTime, 'playbacks' => $playbackarray);
     if (isset($recording->protected)) {
-        $recordingarray['protected'] = (string) $recording->protected;
+        $recordingarray['protected'] = (string)$recording->protected;
     }
     // Force protected.
     $recordingarray['protected'] = 'false';
@@ -388,7 +388,7 @@ 
         if (is_object($value)) {
             $value = '';
         }
-        $metadataarray['meta_'.$key] = $value;
+        $metadataarray['meta_' . $key] = $value;
     }
     return $metadataarray;
 }
@@ -446,7 +446,7 @@ 
     $ids = explode(',', $recordids);
     foreach ($ids as $id) {
         $xml = bigbluebuttonbn_wrap_xml_load_file(
-            \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('updateRecordings', ['recordID' => $id] + (array) $params)
+            \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('updateRecordings', ['recordID' => $id] + (array)$params)
           );
         if ($xml && $xml->returncode != 'SUCCESS') {
             return false;
@@ -566,7 +566,7 @@ 
             return $xml;
         } catch (Exception $e) {
             libxml_use_internal_errors($previous);
-            $error = 'Caught exception: '.$e->getMessage();
+            $error = 'Caught exception: ' . $e->getMessage();
             debugging($error, DEBUG_DEVELOPER);
             return null;
         }
@@ -578,7 +578,7 @@ 
         $response = simplexml_load_file($url, 'SimpleXMLElement', LIBXML_NOCDATA | LIBXML_NOBLANKS);
         return $response;
     } catch (Exception $e) {
-        $error = 'Caught exception: '.$e->getMessage();
+        $error = 'Caught exception: ' . $e->getMessage();
         debugging($error, DEBUG_DEVELOPER);
         libxml_use_internal_errors($previous);
         return null;
@@ -596,8 +596,8 @@ 
 
         $options = array();
         $options['CURLOPT_HTTPHEADER'] = array(
-                 'Content-Type: '.$contenttype,
-                 'Content-Length: '.strlen($data),
+                 'Content-Type: ' . $contenttype,
+                 'Content-Length: ' . strlen($data),
                  'Content-Language: en-US',
                );
 
@@ -614,9 +614,9 @@ 
     if ($userroles) {
         $where = '';
         foreach ($userroles as $value) {
-            $where .= (empty($where) ? ' WHERE' : ' AND').' id='.$value->roleid;
+            $where .= (empty($where) ? ' WHERE' : ' AND') . ' id=' . $value->roleid;
         }
-        $userroles = $DB->get_records_sql('SELECT * FROM {role}'.$where);
+        $userroles = $DB->get_records_sql('SELECT * FROM {role}' . $where);
     }
 
     return $userroles;
@@ -629,7 +629,7 @@ 
 }
 
 function bigbluebuttonbn_get_users(context $context = null) {
-    $users = (array) get_enrolled_users($context,'',0,'u.*',null,0,0,true);
+    $users = (array)get_enrolled_users($context, '', 0, 'u.*', null, 0, 0, true);
     foreach ($users as $key => $value) {
         $users[$key] = fullname($value);
     }
@@ -637,7 +637,7 @@ 
 }
 
 function bigbluebuttonbn_get_users_select(context $context = null) {
-    $users = (array) get_enrolled_users($context,'',0,'u.*',null,0,0,true);
+    $users = (array)get_enrolled_users($context, '', 0, 'u.*', null, 0, 0, true);
     foreach ($users as $key => $value) {
         $users[$key] = array('id' => $value->id, 'name' => fullname($value));
     }
@@ -645,7 +645,7 @@ 
 }
 
 function bigbluebuttonbn_get_roles(context $context = null) {
-    $roles = (array) role_get_names($context);
+    $roles = (array)role_get_names($context);
     foreach ($roles as $key => $value) {
         $roles[$key] = $value->localname;
     }
@@ -653,7 +653,7 @@ 
 }
 
 function bigbluebuttonbn_get_roles_select(context $context = null) {
-    $roles = (array) role_get_names($context);
+    $roles = (array)role_get_names($context);
     foreach ($roles as $key => $value) {
         $roles[$key] = array('id' => $value->id, 'name' => $value->localname);
     }
@@ -661,7 +661,7 @@ 
 }
 
 function bigbluebuttonbn_get_role($id) {
-    $roles = (array) role_get_names();
+    $roles = (array)role_get_names();
     if (is_numeric($id)) {
         return (object)$roles[$id];
     }
@@ -707,11 +707,11 @@ 
         return array();
     }
     foreach ($rules as $key => $rule) {
-        if ( $rule['selectiontype'] !== 'role' || is_numeric($rule['selectionid']) ) {
+        if ($rule['selectiontype'] !== 'role' || is_numeric($rule['selectionid'])) {
             continue;
         }
         $role = bigbluebuttonbn_get_role($rule['selectionid']);
-        if ( $role == null ) {
+        if ($role == null) {
             unset($rules[$key]);
             continue;
         }
@@ -820,9 +820,9 @@ 
     $isunique = true;
     if ($voicebridge != 0) {
         $table = 'bigbluebuttonbn';
-        $select = 'voicebridge = '.$voicebridge;
+        $select = 'voicebridge = ' . $voicebridge;
         if ($id) {
-            $select .= ' AND id <> '.$id;
+            $select .= ' AND id <> ' . $id;
         }
         if ($DB->get_records_select($table, $select)) {
             $isunique = false;
@@ -883,7 +883,7 @@ 
 function bigbluebuttonbn_generate_nonce() {
     $mt = microtime();
     $rand = mt_rand();
-    return md5($mt.$rand);
+    return md5($mt . $rand);
 }
 
 function bigbluebuttonbn_random_password($length = 8) {
@@ -894,41 +894,41 @@ 
 
 function bigbluebuttonbn_events() {
     return array(
-        (string) BIGBLUEBUTTON_EVENT_ACTIVITY_VIEWED,
-        (string) BIGBLUEBUTTON_EVENT_ACTIVITY_MANAGEMENT_VIEWED,
-        (string) BIGBLUEBUTTON_EVENT_LIVE_SESSION,
-        (string) BIGBLUEBUTTON_EVENT_MEETING_CREATED,
-        (string) BIGBLUEBUTTON_EVENT_MEETING_ENDED,
-        (string) BIGBLUEBUTTON_EVENT_MEETING_JOINED,
-        (string) BIGBLUEBUTTON_EVENT_MEETING_LEFT,
-        (string) BIGBLUEBUTTON_EVENT_RECORDING_DELETED,
-        (string) BIGBLUEBUTTON_EVENT_RECORDING_IMPORTED,
-        (string) BIGBLUEBUTTON_EVENT_RECORDING_PROTECTED,
-        (string) BIGBLUEBUTTON_EVENT_RECORDING_PUBLISHED,
-        (string) BIGBLUEBUTTON_EVENT_RECORDING_UNPROTECTED,
-        (string) BIGBLUEBUTTON_EVENT_RECORDING_UNPUBLISHED,
-        (string) BIGBLUEBUTTON_EVENT_RECORDING_EDITED,
-        (string) BIGBLUEBUTTON_EVENT_RECORDING_VIEWED
+        (string)BIGBLUEBUTTON_EVENT_ACTIVITY_VIEWED,
+        (string)BIGBLUEBUTTON_EVENT_ACTIVITY_MANAGEMENT_VIEWED,
+        (string)BIGBLUEBUTTON_EVENT_LIVE_SESSION,
+        (string)BIGBLUEBUTTON_EVENT_MEETING_CREATED,
+        (string)BIGBLUEBUTTON_EVENT_MEETING_ENDED,
+        (string)BIGBLUEBUTTON_EVENT_MEETING_JOINED,
+        (string)BIGBLUEBUTTON_EVENT_MEETING_LEFT,
+        (string)BIGBLUEBUTTON_EVENT_RECORDING_DELETED,
+        (string)BIGBLUEBUTTON_EVENT_RECORDING_IMPORTED,
+        (string)BIGBLUEBUTTON_EVENT_RECORDING_PROTECTED,
+        (string)BIGBLUEBUTTON_EVENT_RECORDING_PUBLISHED,
+        (string)BIGBLUEBUTTON_EVENT_RECORDING_UNPROTECTED,
+        (string)BIGBLUEBUTTON_EVENT_RECORDING_UNPUBLISHED,
+        (string)BIGBLUEBUTTON_EVENT_RECORDING_EDITED,
+        (string)BIGBLUEBUTTON_EVENT_RECORDING_VIEWED
     );
 }
 
 function bigbluebuttonbn_events_action() {
     return array(
-        'view' => (string) BIGBLUEBUTTON_EVENT_ACTIVITY_VIEWED,
-        'view_management' => (string) BIGBLUEBUTTON_EVENT_ACTIVITY_MANAGEMENT_VIEWED,
-        'live_action' => (string) BIGBLUEBUTTON_EVENT_LIVE_SESSION,
-        'meeting_create' => (string) BIGBLUEBUTTON_EVENT_MEETING_CREATED,
-        'meeting_end' => (string) BIGBLUEBUTTON_EVENT_MEETING_ENDED,
-        'meeting_join' => (string) BIGBLUEBUTTON_EVENT_MEETING_JOINED,
-        'meeting_left' => (string) BIGBLUEBUTTON_EVENT_MEETING_LEFT,
-        'recording_delete' => (string) BIGBLUEBUTTON_EVENT_RECORDING_DELETED,
-        'recording_import' => (string) BIGBLUEBUTTON_EVENT_RECORDING_IMPORTED,
-        'recording_protect' => (string) BIGBLUEBUTTON_EVENT_RECORDING_PROTECTED,
-        'recording_publish' => (string) BIGBLUEBUTTON_EVENT_RECORDING_PUBLISHED,
-        'recording_unprotect' => (string) BIGBLUEBUTTON_EVENT_RECORDING_UNPROTECTED,
-        'recording_unpublish' => (string) BIGBLUEBUTTON_EVENT_RECORDING_UNPUBLISHED,
-        'recording_edit' => (string) BIGBLUEBUTTON_EVENT_RECORDING_EDITED,
-        'recording_play' => (string) BIGBLUEBUTTON_EVENT_RECORDING_VIEWED
+        'view' => (string)BIGBLUEBUTTON_EVENT_ACTIVITY_VIEWED,
+        'view_management' => (string)BIGBLUEBUTTON_EVENT_ACTIVITY_MANAGEMENT_VIEWED,
+        'live_action' => (string)BIGBLUEBUTTON_EVENT_LIVE_SESSION,
+        'meeting_create' => (string)BIGBLUEBUTTON_EVENT_MEETING_CREATED,
+        'meeting_end' => (string)BIGBLUEBUTTON_EVENT_MEETING_ENDED,
+        'meeting_join' => (string)BIGBLUEBUTTON_EVENT_MEETING_JOINED,
+        'meeting_left' => (string)BIGBLUEBUTTON_EVENT_MEETING_LEFT,
+        'recording_delete' => (string)BIGBLUEBUTTON_EVENT_RECORDING_DELETED,
+        'recording_import' => (string)BIGBLUEBUTTON_EVENT_RECORDING_IMPORTED,
+        'recording_protect' => (string)BIGBLUEBUTTON_EVENT_RECORDING_PROTECTED,
+        'recording_publish' => (string)BIGBLUEBUTTON_EVENT_RECORDING_PUBLISHED,
+        'recording_unprotect' => (string)BIGBLUEBUTTON_EVENT_RECORDING_UNPROTECTED,
+        'recording_unpublish' => (string)BIGBLUEBUTTON_EVENT_RECORDING_UNPUBLISHED,
+        'recording_edit' => (string)BIGBLUEBUTTON_EVENT_RECORDING_EDITED,
+        'recording_play' => (string)BIGBLUEBUTTON_EVENT_RECORDING_VIEWED
     );
 }
 
@@ -949,7 +949,7 @@ 
     if (array_key_exists('other', $options)) {
         $eventproperties['other'] = $options['other'];
     }
-    $event = call_user_func_array('\mod_bigbluebuttonbn\event\bigbluebuttonbn_'.$eventtype.'::create',
+    $event = call_user_func_array('\mod_bigbluebuttonbn\event\bigbluebuttonbn_' . $eventtype . '::create',
       array($eventproperties));
     $event->trigger();
 }
@@ -990,10 +990,10 @@ 
     $now = time();
     if (!$forced && isset($result) && $now < ($result['creation_time'] + $cachettl)) {
         // Use the value in the cache.
-        return (array) json_decode($result['meeting_info']);
+        return (array)json_decode($result['meeting_info']);
     }
     // Ping again and refresh the cache.
-    $meetinginfo = (array) bigbluebuttonbn_wrap_xml_load_file(
+    $meetinginfo = (array)bigbluebuttonbn_wrap_xml_load_file(
         \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('getMeetingInfo', ['meetingID' => $meetingid])
       );
     $cache->set($meetingid, array('creation_time' => time(), 'meeting_info' => json_encode($meetinginfo)));
@@ -1038,8 +1038,8 @@ 
  * @param string $configxml
  */
 function bigbluebuttonbn_set_config_xml_params($meetingid, $configxml) {
-    $params = 'configXML='.urlencode($configxml).'&meetingID='.urlencode($meetingid);
-    $configxmlparams = $params.'&checksum='.sha1('setConfigXML'.$params.\mod_bigbluebuttonbn\locallib\config::get('shared_secret'));
+    $params = 'configXML=' . urlencode($configxml) . '&meetingID=' . urlencode($meetingid);
+    $configxmlparams = $params . '&checksum=' . sha1('setConfigXML' . $params . \mod_bigbluebuttonbn\locallib\config::get('shared_secret'));
     return $configxmlparams;
 }
 
@@ -1048,7 +1048,7 @@ 
  * @param string $configxml
  */
 function bigbluebuttonbn_set_config_xml($meetingid, $configxml) {
-    $urldefaultconfig = \mod_bigbluebuttonbn\locallib\config::get('server_url').'api/setConfigXML?';
+    $urldefaultconfig = \mod_bigbluebuttonbn\locallib\config::get('server_url') . 'api/setConfigXML?';
     $configxmlparams = bigbluebuttonbn_set_config_xml_params($meetingid, $configxml);
     $xml = bigbluebuttonbn_wrap_xml_load_file($urldefaultconfig, BIGBLUEBUTTONBN_METHOD_POST,
         $configxmlparams, 'application/x-www-form-urlencoded');
@@ -1061,7 +1061,7 @@ 
  */
 function bigbluebuttonbn_set_config_xml_array($meetingid, $configxml) {
     $configxml = bigbluebuttonbn_setConfigXML($meetingid, $configxml);
-    $configxmlarray = (array) $configxml;
+    $configxmlarray = (array)$configxml;
     if ($configxmlarray['returncode'] != 'SUCCESS') {
         debugging('BigBlueButton was not able to set the custom config.xml file', DEBUG_DEVELOPER);
         return '';
@@ -1090,7 +1090,7 @@ 
         $row->date = $starttime;
         $starttime = $starttime - ($starttime % 1000);
         // Set formatted date.
-        $dateformat = get_string('strftimerecentfull', 'langconfig').' %Z';
+        $dateformat = get_string('strftimerecentfull', 'langconfig') . ' %Z';
         $row->date_formatted = userdate($starttime / 1000, $dateformat, usertimezone($USER->timezone));
         // Set formatted duration.
         $firstplayback = array_values($recording['playbacks'])[0];
@@ -1152,7 +1152,7 @@ 
         $visibility = 'hidden ';
     }
     $recordingpreview = html_writer::start_tag('div',
-        array('id' => 'preview-'.$recording['recordID'], $visibility => $visibility));
+        array('id' => 'preview-' . $recording['recordID'], $visibility => $visibility));
     foreach ($recording['playbacks'] as $playback) {
         if (isset($playback['preview'])) {
             foreach ($playback['preview'] as $image) {
@@ -1181,19 +1181,19 @@ 
     if ($recording['published'] === 'false') {
         $visibility = 'hidden ';
     }
-    $id = 'playbacks-'.$recording['recordID'];
+    $id = 'playbacks-' . $recording['recordID'];
     $recordingtypes = html_writer::start_tag('div', array('id' => $id, 'data-imported' => $dataimported,
           'data-recordingid' => $recording['recordID'], 'data-meetingid' => $recording['meetingID'],
           'title' => $title, $visibility => $visibility));
     foreach ($recording['playbacks'] as $playback) {
         $onclick = 'M.mod_bigbluebuttonbn.recordings.recording_play(this);';
-        $href = $CFG->wwwroot.'/mod/bigbluebuttonbn/bbb_view.php?action=playback&bn='.$bigbluebuttonbnid.
-            '&href='.urlencode($playback['url']).'&rid='.$recording['recordID'];
-        $linkattributes = array('title' => get_string('view_recording_format_'.$playback['type'], 'bigbluebuttonbn'),
+        $href = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_view.php?action=playback&bn=' . $bigbluebuttonbnid .
+            '&href=' . urlencode($playback['url']) . '&rid=' . $recording['recordID'];
+        $linkattributes = array('title' => get_string('view_recording_format_' . $playback['type'], 'bigbluebuttonbn'),
             'class' => 'btn btn-sm btn-default', 'onclick' => $onclick,
             'data-action' => 'play', 'data-href' => $href);
-        $recordingtypes .= $OUTPUT->action_link('#', get_string('view_recording_format_'.$playback['type'],
-            'bigbluebuttonbn'), null, $linkattributes).'&#32;';
+        $recordingtypes .= $OUTPUT->action_link('#', get_string('view_recording_format_' . $playback['type'],
+            'bigbluebuttonbn'), null, $linkattributes) . '&#32;';
     }
     $recordingtypes .= html_writer::end_tag('div');
     return $recordingtypes;
@@ -1277,11 +1277,11 @@ 
         $target .= '-' . $data['target'];
     }
     $id = 'recording-' . $target . '-' . $recording['recordID'];
-    $onclick = 'M.mod_bigbluebuttonbn.recordings.recording_'.$data['action'].'(this);';
+    $onclick = 'M.mod_bigbluebuttonbn.recordings.recording_' . $data['action'] . '(this);';
     if ((boolean)\mod_bigbluebuttonbn\locallib\config::get('recording_icons_enabled')) {
         // With icon for $manageaction.
         $iconattributes = array('id' => $id, 'class' => 'iconsmall');
-        $icon = new pix_icon('i/'.$data['tag'],
+        $icon = new pix_icon('i/' . $data['tag'],
             get_string('view_recording_list_actionbar_' . $data['action'], 'bigbluebuttonbn'),
             'moodle', $iconattributes);
         $linkattributes = array(
@@ -1371,7 +1371,7 @@ 
     if (isset($recordings) && !array_key_exists('messageKey', $recordings)) {
         // There are recordings for this meeting.
         foreach ($recordings as $recording) {
-            if ( !bigbluebuttonbn_include_recording_table_row($bbbsession, $recording) ) {
+            if (!bigbluebuttonbn_include_recording_table_row($bbbsession, $recording)) {
                 continue;
             }
             bigbluebuttonbn_get_recording_table_row($bbbsession, $recording, $tools, $table);
@@ -1383,7 +1383,7 @@ 
 
 function bigbluebuttonbn_get_recording_table_row($bbbsession, $recording, $tools, &$table) {
     $row = new html_table_row();
-    $row->id = 'recording-td-'.$recording['recordID'];
+    $row->id = 'recording-td-' . $recording['recordID'];
     $row->attributes['data-imported'] = 'false';
     $texthead = '';
     $texttail = '';
@@ -1397,9 +1397,9 @@ 
     $rowdata = bigbluebuttonbn_get_recording_data_row($bbbsession, $recording, $tools);
     if ($rowdata != null) {
         $rowdata->date_formatted = str_replace(' ', '&nbsp;', $rowdata->date_formatted);
-        $row->cells = array($texthead.$rowdata->recording.$texttail,
-            $texthead.$rowdata->activity.$texttail, $texthead.$rowdata->description.$texttail,
-            $rowdata->preview, $texthead.$rowdata->date_formatted.$texttail,
+        $row->cells = array($texthead . $rowdata->recording . $texttail,
+            $texthead . $rowdata->activity . $texttail, $texthead . $rowdata->description . $texttail,
+            $rowdata->preview, $texthead . $rowdata->date_formatted . $texttail,
             $rowdata->duration_formatted);
         if ($bbbsession['managerecordings']) {
             $row->cells[] = $rowdata->actionbar;
@@ -1409,7 +1409,7 @@ 
 }
 
 function bigbluebuttonbn_include_recording_table_row($bbbsession, $recording) {
-    return !( !isset($recording['imported']) && isset($bbbsession['group']) && $recording['meetingID'] != $bbbsession['meetingid'] );
+    return !(!isset($recording['imported']) && isset($bbbsession['group']) && $recording['meetingID'] != $bbbsession['meetingid']);
 }
 
 function bigbluebuttonbn_send_notification_recording_ready($bigbluebuttonbn) {
@@ -1421,9 +1421,9 @@ 
     // Build the message_body.
     $msg->activity_type = '';
     $msg->activity_title = $bigbluebuttonbn->name;
-    $messagetext = '<p>'.get_string('email_body_recording_ready_for', 'bigbluebuttonbn').' '.
-        $msg->activity_type.' &quot;'.$msg->activity_title.'&quot; '.
-        get_string('email_body_recording_ready_is_ready', 'bigbluebuttonbn').'.</p>';
+    $messagetext = '<p>' . get_string('email_body_recording_ready_for', 'bigbluebuttonbn') . ' ' .
+        $msg->activity_type . ' &quot;' . $msg->activity_title . '&quot; ' .
+        get_string('email_body_recording_ready_is_ready', 'bigbluebuttonbn') . '.</p>';
 
     bigbluebuttonbn_send_notification($sender, $bigbluebuttonbn, $messagetext);
 }
@@ -1516,15 +1516,15 @@ 
 
     // Gather the bigbluebuttonbnids whose meetingids should be included in the getRecordings request'.
     $select = "id <> '{$bigbluebuttonbnid}' AND course = '{$courseid}'";
-    $selectdeleted = "courseid = '{$courseid}' AND bigbluebuttonbnid <> '{$bigbluebuttonbnid}' AND log = '".
-        BIGBLUEBUTTONBN_LOG_EVENT_DELETE."' AND meta like '%has_recordings%' AND meta like '%true%'";
+    $selectdeleted = "courseid = '{$courseid}' AND bigbluebuttonbnid <> '{$bigbluebuttonbnid}' AND log = '" .
+        BIGBLUEBUTTONBN_LOG_EVENT_DELETE . "' AND meta like '%has_recordings%' AND meta like '%true%'";
     if ($bigbluebuttonbnid === null) {
         $select = "course = '{$courseid}'";
-        $selectdeleted = "courseid = '{$courseid}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_DELETE.
+        $selectdeleted = "courseid = '{$courseid}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_DELETE .
             "' AND meta like '%has_recordings%' AND meta like '%true%'";
     } else if ($subset) {
         $select = "id = '{$bigbluebuttonbnid}'";
-        $selectdeleted = "bigbluebuttonbnid = '{$bigbluebuttonbnid}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_DELETE.
+        $selectdeleted = "bigbluebuttonbnid = '{$bigbluebuttonbnid}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_DELETE .
             "' AND meta like '%has_recordings%' AND meta like '%true%'";
     }
     $bigbluebuttonbns = $DB->get_records_select_menu('bigbluebuttonbn', $select, null, 'id', 'id, meetingid');
@@ -1546,7 +1546,7 @@ 
     if (!empty($bigbluebuttonbns)) {
         // Prepare select for loading records based on existent bigbluebuttonbns.
         $sql = 'SELECT DISTINCT meetingid, bigbluebuttonbnid FROM {bigbluebuttonbn_logs} WHERE ';
-        $sql .= '(bigbluebuttonbnid='.implode(' OR bigbluebuttonbnid=', array_keys($bigbluebuttonbns)).')';
+        $sql .= '(bigbluebuttonbnid=' . implode(' OR bigbluebuttonbnid=', array_keys($bigbluebuttonbns)) . ')';
         // Include only Create events and exclude those with record not true.
         $sql .= ' AND log = ? AND meta LIKE ? AND meta LIKE ?';
         // Execute select for loading records based on existent bigbluebuttonbns.
@@ -1638,8 +1638,8 @@ 
 function bigbluebuttonbn_format_activity_time($time) {
     $activitytime = '';
     if ($time) {
-        $activitytime = calendar_day_representation($time).' '.
-          get_string('mod_form_field_notification_msg_at', 'bigbluebuttonbn').' '.
+        $activitytime = calendar_day_representation($time) . ' ' .
+          get_string('mod_form_field_notification_msg_at', 'bigbluebuttonbn') . ' ' .
           calendar_time_representation($time);
     }
 

view.php

Indentation +10 added lines, -10 removed lines

@@ -273,11 +273,11 @@ 
 
     echo $OUTPUT->box_start('generalbox boxaligncenter');
     echo '<br><div class="alert alert-warning">'.get_string('view_groups_selection_warning', 'bigbluebuttonbn').
-      '</div>';
+        '</div>';
     echo $OUTPUT->box_end();
 
     groups_print_activity_menu(
-      $bbbsession['cm'], $CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$bbbsession['cm']->id);
+        $bbbsession['cm'], $CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$bbbsession['cm']->id);
     echo '<br><br>';
 }
 
@@ -372,7 +372,7 @@ 
     $recordings = bigbluebuttonbn_get_recordings(
         $bbbsession['course']->id, $bigbluebuttonbnid, $showroom,
         $bbbsession['bigbluebuttonbn']->recordings_deleted_activities
-      );
+        );
 
     if (empty($recordings) || array_key_exists('messageKey', $recordings)) {
         // There are no recordings to be shown.
@@ -384,7 +384,7 @@ 
     // JavaScript variables for recordings.
     $jsvars += array(
             'recordings_html' => $bbbsession['bigbluebuttonbn']->recordings_html == '1',
-          );
+            );
 
     // If there are meetings with recordings load the data to the table.
     if ($bbbsession['bigbluebuttonbn']->recordings_html) {
@@ -396,7 +396,7 @@ 
     $jsvars += array(
             'columns' => bigbluebuttonbn_get_recording_columns($bbbsession),
             'data' => bigbluebuttonbn_get_recording_data($bbbsession, $recordings),
-          );
+            );
 
     // Render a YUI table.
     return html_writer::div('', '', array('id' => 'bigbluebuttonbn_yui_table'));
@@ -407,10 +407,10 @@ 
 
     $button = html_writer::tag('input', '',
         array('type' => 'button',
-              'value' => get_string('view_recording_button_import', 'bigbluebuttonbn'),
-              'class' => 'btn btn-secondary',
-              'onclick' => 'window.location=\''.$CFG->wwwroot.'/mod/bigbluebuttonbn/import_view.php?bn='.
-                  $bbbsession['bigbluebuttonbn']->id.'\''));
+                'value' => get_string('view_recording_button_import', 'bigbluebuttonbn'),
+                'class' => 'btn btn-secondary',
+                'onclick' => 'window.location=\''.$CFG->wwwroot.'/mod/bigbluebuttonbn/import_view.php?bn='.
+                    $bbbsession['bigbluebuttonbn']->id.'\''));
     $output = html_writer::start_tag('br');
     $output .= html_writer::tag('span', $button, array('id' => 'import_recording_links_button'));
     $output .= html_writer::tag('span', '', array('id' => 'import_recording_links_table'));
@@ -428,7 +428,7 @@ 
         return '<h4>'.get_string('view_section_title_presentation', 'bigbluebuttonbn').'</h4>'.
                 ''.$OUTPUT->action_icon($bbbsession['presentation']['url'], $icon, null, array(), false).''.
                 ''.$OUTPUT->action_link($bbbsession['presentation']['url'],
-                      $bbbsession['presentation']['name'], null, $attributes).'<br><br>';
+                        $bbbsession['presentation']['name'], null, $attributes).'<br><br>';
     }
 
     return '';