blindsidenetworks / moodle-mod_bigbluebuttonbn

bbb_broker.php

Doc Comments +9 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) {
     if ($running && ($bbbsession['administrator'] || $bbbsession['moderator'])) {
         return array("can_end" => true);
@@ -337,6 +343,9 @@ 
     return "{$params['callback']}({$callbackresponsedata});";
 }
 
+/**
+ * @param string $action
+ */
 function bigbluebuttonbn_broker_recording_action_perform($action, $params, $recordings) {
     if ($action == 'recording_publish') {
         return bigbluebuttonbn_broker_recording_action_publish($params, $recordings);

Spacing +16 added lines, -16 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;
 }
 
@@ -283,7 +283,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']) || empty($params['meta'])) {
         return $callbackresponse;
     }
@@ -545,8 +545,8 @@ 
         $decodedparameters = JWT::decode($params['signed_parameters'], (string)\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;
     }
 
@@ -564,8 +564,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);
     }
 }
 
@@ -580,7 +580,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;
     }
 
@@ -588,7 +588,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'])) {
@@ -606,8 +606,8 @@ 
         $decodedparameters = JWT::decode($params['signed_parameters'], (string)\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;
     }
 
@@ -645,7 +645,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

Indentation +56 added lines, -56 removed lines

@@ -94,9 +94,9 @@ 
  */
 function bigbluebuttonbn_get_join_url($meetingid, $username, $pw, $logouturl, $configtoken = null, $userid = null) {
     $data = ['meetingID' => $meetingid,
-              'fullName' => $username,
-              'password' => $pw,
-              'logoutURL' => $logouturl,
+                'fullName' => $username,
+                'password' => $pw,
+                'logoutURL' => $logouturl,
             ];
     if (!is_null($configtoken)) {
         $data['configToken'] = $configtoken;
@@ -177,23 +177,23 @@ 
 function bigbluebuttonbn_get_meeting_info_array($meetingid) {
     $xml = bigbluebuttonbn_wrap_xml_load_file(
         \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('getMeetingInfo', ['meetingID' => $meetingid])
-      );
+        );
     if ($xml && $xml->returncode == 'SUCCESS' && empty($xml->messageKey)) {
         // Meeting info was returned.
         return array('returncode' => $xml->returncode,
-                     'meetingID' => $xml->meetingID,
-                     'moderatorPW' => $xml->moderatorPW,
-                     'attendeePW' => $xml->attendeePW,
-                     'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded,
-                     'running' => $xml->running,
-                     'recording' => $xml->recording,
-                     'startTime' => $xml->startTime,
-                     'endTime' => $xml->endTime,
-                     'participantCount' => $xml->participantCount,
-                     'moderatorCount' => $xml->moderatorCount,
-                     'attendees' => $xml->attendees,
-                     'metadata' => $xml->metadata,
-                   );
+                        'meetingID' => $xml->meetingID,
+                        'moderatorPW' => $xml->moderatorPW,
+                        'attendeePW' => $xml->attendeePW,
+                        'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded,
+                        'running' => $xml->running,
+                        'recording' => $xml->recording,
+                        'startTime' => $xml->startTime,
+                        'endTime' => $xml->endTime,
+                        'participantCount' => $xml->participantCount,
+                        'moderatorCount' => $xml->moderatorCount,
+                        'attendees' => $xml->attendees,
+                        'metadata' => $xml->metadata,
+                    );
     }
     if ($xml) {
         // Either failure or success without meeting info.
@@ -249,7 +249,7 @@ 
         // Do getRecordings is executed using a method GET (supported by all versions of BBB).
         $xml = bigbluebuttonbn_wrap_xml_load_file(
             \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('getRecordings', ['meetingID' => implode(',', $mids)])
-          );
+            );
         if ($xml && $xml->returncode == 'SUCCESS' && isset($xml->recordings)) {
             // If there were meetings already created.
             foreach ($xml->recordings->recording as $recording) {
@@ -309,7 +309,7 @@ 
 function bigbluebuttonbn_get_default_config_xml() {
     $xml = bigbluebuttonbn_wrap_xml_load_file(
         \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('getDefaultConfigXML')
-      );
+        );
     return $xml;
 }
 
@@ -378,7 +378,7 @@ 
     foreach ($ids as $id) {
         $xml = bigbluebuttonbn_wrap_xml_load_file(
             \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('deleteRecordings', ['recordID' => $id])
-          );
+            );
         if ($xml && $xml->returncode != 'SUCCESS') {
             return false;
         }
@@ -395,7 +395,7 @@ 
     foreach ($ids as $id) {
         $xml = bigbluebuttonbn_wrap_xml_load_file(
             \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('publishRecordings', ['recordID' => $id, 'publish' => $publish])
-          );
+            );
         if ($xml && $xml->returncode != 'SUCCESS') {
             return false;
         }
@@ -412,7 +412,7 @@ 
     foreach ($ids as $id) {
         $xml = bigbluebuttonbn_wrap_xml_load_file(
             \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('updateRecordings', ['recordID' => $id] + (array) $params)
-          );
+            );
         if ($xml && $xml->returncode != 'SUCCESS') {
             return false;
         }
@@ -427,7 +427,7 @@ 
 function bigbluebuttonbn_end_meeting($meetingid, $modpw) {
     $xml = bigbluebuttonbn_wrap_xml_load_file(
         \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('end', ['meetingID' => $meetingid, 'password' => $modpw])
-      );
+        );
     if ($xml) {
         // If the xml packet returned failure it displays the message to the user.
         return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey);
@@ -442,7 +442,7 @@ 
 function bigbluebuttonbn_is_meeting_running($meetingid) {
     $xml = bigbluebuttonbn_wrap_xml_load_file(
         \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('isMeetingRunning', ['meetingID' => $meetingid])
-      );
+        );
     if ($xml && $xml->returncode == 'SUCCESS') {
         return ($xml->running == 'true');
     }
@@ -452,7 +452,7 @@ 
 function bigbluebuttonbn_get_server_version() {
     $xml = bigbluebuttonbn_wrap_xml_load_file(
         \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url()
-      );
+        );
     if ($xml && $xml->returncode == 'SUCCESS') {
         return $xml->version;
     }
@@ -507,10 +507,10 @@ 
 
         $options = array();
         $options['CURLOPT_HTTPHEADER'] = array(
-                 'Content-Type: '.$contenttype,
-                 'Content-Length: '.strlen($data),
-                 'Content-Language: en-US',
-               );
+                    'Content-Type: '.$contenttype,
+                    'Content-Length: '.strlen($data),
+                    'Content-Language: en-US',
+                );
 
         return $c->post($url, $data, $options);
     }
@@ -581,13 +581,13 @@ 
 
 function bigbluebuttonbn_role_unknown() {
     return array(
-              "id" => "0",
-              "name" => "",
-              "shortname" => "unknown",
-              "description" => "",
-              "sortorder" => "0",
-              "archetype" => "guest",
-              "localname" => "Unknown"
+                "id" => "0",
+                "name" => "",
+                "shortname" => "unknown",
+                "description" => "",
+                "sortorder" => "0",
+                "archetype" => "guest",
+                "localname" => "Unknown"
             );
 }
 
@@ -596,16 +596,16 @@ 
         'all' => array(
             'name' => get_string('mod_form_field_participant_list_type_all', 'bigbluebuttonbn'),
             'children' => []
-          )
-      );
+            )
+        );
     $data['role'] = array(
         'name' => get_string('mod_form_field_participant_list_type_role', 'bigbluebuttonbn'),
         'children' => bigbluebuttonbn_get_roles_select($context)
-      );
+        );
     $data['user'] = array(
         'name' => get_string('mod_form_field_participant_list_type_user', 'bigbluebuttonbn'),
         'children' => bigbluebuttonbn_get_users_select($context)
-      );
+        );
     return $data;
 }
 
@@ -635,9 +635,9 @@ 
             continue;
         }
         $participantlistarray[] = array(
-              'selectiontype' => 'role',
-              'selectionid' => $moderatordefault,
-              'role' => BIGBLUEBUTTONBN_ROLE_MODERATOR);
+                'selectiontype' => 'role',
+                'selectionid' => $moderatordefault,
+                'role' => BIGBLUEBUTTONBN_ROLE_MODERATOR);
     }
     return $participantlistarray;
 }
@@ -668,11 +668,11 @@ 
             'all' => get_string('mod_form_field_participant_list_type_all', 'bigbluebuttonbn'),
             'role' => get_string('mod_form_field_participant_list_type_role', 'bigbluebuttonbn'),
             'user' => get_string('mod_form_field_participant_list_type_user', 'bigbluebuttonbn'),
-          ],
+            ],
         'type_selected' => 'all',
         'options' => ['all' => '---------------'],
         'selected' => 'all',
-      ];
+        ];
 }
 
 function bigbluebuttonbn_is_moderator($context, $participants, $userid = null, $userroles = null) {
@@ -862,7 +862,7 @@ 
         $eventproperties['other'] = $options['other'];
     }
     $event = call_user_func_array('\mod_bigbluebuttonbn\event\bigbluebuttonbn_'.$eventtype.'::create',
-      array($eventproperties));
+        array($eventproperties));
     $event->trigger();
 }
 
@@ -907,7 +907,7 @@ 
     // Ping again and refresh the cache.
     $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)));
     return $meetinginfo;
 }
@@ -1143,8 +1143,8 @@ 
     }
     $id = 'playbacks-'.$recording['recordID'];
     $recordingtypes = html_writer::start_tag('div', array('id' => $id, 'data-imported' => $dataimported,
-          'data-meetingid' => $recording['meetingID'], 'data-recordingid' => $recording['recordID'],
-          'title' => $title, $visibility => $visibility));
+            'data-meetingid' => $recording['meetingID'], 'data-recordingid' => $recording['recordID'],
+            'title' => $title, $visibility => $visibility));
     foreach ($recording['playbacks'] as $playback) {
         $recordingtypes .= bigbluebuttonbn_get_recording_data_row_type($recording, $bigbluebuttonbnid,
             $playback).' ';
@@ -1158,7 +1158,7 @@ 
     $title = get_string('view_recording_format_'.$playback['type'], 'bigbluebuttonbn');
     $onclick = 'M.mod_bigbluebuttonbn.recordings.recording_play(this);';
     $href = $CFG->wwwroot.'/mod/bigbluebuttonbn/bbb_view.php?action=play&bn='.$bigbluebuttonbnid.
-      '&mid='.$recording['meetingID'].'&rid='.$recording['recordID'].'&rtype='.$playback['type'];
+        '&mid='.$recording['meetingID'].'&rid='.$recording['recordID'].'&rtype='.$playback['type'];
     if (!isset($recording['imported']) || !isset($recording['protected']) || $recording['protected'] === 'false') {
         $href .= '&href='.urlencode(trim($playback['url']));
     }
@@ -1170,7 +1170,7 @@ 
         'data-target' => $playback['type'],
         'data-href' => $href,
         'class' => 'btn btn-sm btn-default'
-      );
+        );
     return $OUTPUT->action_link('#', $title, null, $linkattributes);
 }
 
@@ -1254,7 +1254,7 @@ 
             'onclick' => $onclick,
             'data-action' => $data['action'],
             'data-links' => bigbluebuttonbn_get_count_recording_imported_instances($recording['recordID'])
-          );
+            );
         return $OUTPUT->action_icon('#', $icon, null, $linkattributes, false);
     }
     // With text for $manageaction.
@@ -1359,7 +1359,7 @@ 
         $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;
     }
@@ -1601,8 +1601,8 @@ 
     $activitytime = '';
     if ($time) {
         $activitytime = calendar_day_representation($time).' '.
-          get_string('mod_form_field_notification_msg_at', 'bigbluebuttonbn').' '.
-          calendar_time_representation($time);
+            get_string('mod_form_field_notification_msg_at', 'bigbluebuttonbn').' '.
+            calendar_time_representation($time);
     }
     return $activitytime;
 }
@@ -1739,7 +1739,7 @@ 
         $owner = array('0' => get_string('mod_form_field_participant_list_type_owner', 'bigbluebuttonbn'));
         $renderer->render_group_element('participant_moderator_default',
             $renderer->render_group_element_configmultiselect('participant_moderator_default', array_keys($owner), array_merge($owner, $roles))
-          );
+            );
     }
     // Configuration for "send notifications" feature.
     if ((boolean)\mod_bigbluebuttonbn\settings\renderer::section_send_notifications_shown()) {

Spacing +99 added lines, -99 removed lines

@@ -27,7 +27,7 @@ 
 
 global $CFG;
 
-require_once(dirname(__FILE__).'/lib.php');
+require_once(dirname(__FILE__) . '/lib.php');
 
 const BIGBLUEBUTTONBN_FORCED = true;
 
@@ -79,7 +79,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);
 }
@@ -119,8 +119,8 @@ 
     $data = null;
     if (!is_null($pname) && !is_null($purl)) {
         $method = BIGBLUEBUTTONBN_METHOD_POST;
-        $data = "<?xml version='1.0' encoding='UTF-8'?><modules><module name='presentation'><document url='".
-            $purl."' /></module></modules>";
+        $data = "<?xml version='1.0' encoding='UTF-8'?><modules><module name='presentation'><document url='" .
+            $purl . "' /></module></modules>";
     }
     $xml = bigbluebuttonbn_wrap_xml_load_file($createmeetingurl, $method, $data);
     if ($xml) {
@@ -159,7 +159,7 @@ 
                                 'moderatorPW' => $meeting->moderatorPW,
                                 'attendeePW' => $meeting->attendeePW,
                                 'hasBeenForciblyEnded' => $meeting->hasBeenForciblyEnded,
-                                'running' => $meeting->running, );
+                                'running' => $meeting->running,);
         }
         return $meetings;
     }
@@ -299,7 +299,7 @@ 
         // Override imported flag with actual ID.
         $recording['imported'] = $recordimported->id;
         if (isset($recordimported->protected)) {
-            $recording['protected'] = (string) $recordimported->protected;
+            $recording['protected'] = (string)$recordimported->protected;
         }
         $recordsimportedarray[$recording['recordID']] = $recording;
     }
@@ -315,36 +315,36 @@ 
 
 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' => trim((string) $format->url), 'length' => (string) $format->length);
+        $playbackarray[(string)$format->type] = array('type' => (string)$format->type,
+            'url' => trim((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' => trim((string) $image));
+                $imagearray = array('url' => trim((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;
     }
     return $recordingarray + $metadataarray;
 }
@@ -355,7 +355,7 @@ 
         if (is_object($value)) {
             $value = '';
         }
-        $metadataarray['meta_'.$key] = $value;
+        $metadataarray['meta_' . $key] = $value;
     }
     return $metadataarray;
 }
@@ -411,7 +411,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;
@@ -478,7 +478,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;
         }
@@ -489,7 +489,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;
@@ -507,8 +507,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',
                );
 
@@ -523,9 +523,9 @@ 
     if ($userroles) {
         $where = '';
         foreach ($userroles as $value) {
-            $where .= (empty($where) ? ' WHERE' : ' OR').' id='.$value->roleid;
+            $where .= (empty($where) ? ' WHERE' : ' OR') . ' id=' . $value->roleid;
         }
-        $userroles = $DB->get_records_sql('SELECT * FROM {role}'.$where);
+        $userroles = $DB->get_records_sql('SELECT * FROM {role}' . $where);
     }
     return $userroles;
 }
@@ -536,7 +536,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);
     }
@@ -544,7 +544,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));
     }
@@ -552,7 +552,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;
     }
@@ -560,7 +560,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);
     }
@@ -568,7 +568,7 @@ 
 }
 
 function bigbluebuttonbn_get_role($id) {
-    $roles = (array) role_get_names();
+    $roles = (array)role_get_names();
     if (is_numeric($id)) {
         return (object)$roles[$id];
     }
@@ -648,11 +648,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;
         }
@@ -691,10 +691,10 @@ 
     if (empty($userroles)) {
         $userroles = get_user_roles($context, $userid, true);
     }
-    return bigbluebuttonbn_is_moderator_validator($participantlist, $userid , $userroles);
+    return bigbluebuttonbn_is_moderator_validator($participantlist, $userid, $userroles);
 }
 
-function bigbluebuttonbn_is_moderator_validator($participantlist, $userid , $userroles) {
+function bigbluebuttonbn_is_moderator_validator($participantlist, $userid, $userroles) {
     // Iterate participant rules.
     foreach ($participantlist as $participant) {
         if (bigbluebuttonbn_is_moderator_validate_rule($participant, $userid, $userroles)) {
@@ -739,9 +739,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;
@@ -795,7 +795,7 @@ 
 function bigbluebuttonbn_generate_nonce() {
     $mt = microtime();
     $rand = mt_rand();
-    return md5($mt.$rand);
+    return md5($mt . $rand);
 }
 
 function bigbluebuttonbn_random_password($length = 8) {
@@ -806,41 +806,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
     );
 }
 
@@ -861,7 +861,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();
 }
@@ -902,10 +902,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)));
@@ -980,8 +980,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;
 }
 
@@ -990,7 +990,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');
@@ -1003,7 +1003,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 '';
@@ -1053,7 +1053,7 @@ 
     global $USER;
     $starttime = $starttime - ($starttime % 1000);
     // Set formatted date.
-    $dateformat = get_string('strftimerecentfull', 'langconfig').' %Z';
+    $dateformat = get_string('strftimerecentfull', 'langconfig') . ' %Z';
     return userdate($starttime / 1000, $dateformat, usertimezone($USER->timezone));
 }
 
@@ -1069,7 +1069,7 @@ 
 function bigbluebuttonbn_get_recording_data_row_actionbar($recording, $tools) {
     $actionbar = '';
     foreach ($tools as $tool) {
-        if ( $tool == 'protect' && !isset($recording['protected']) ) {
+        if ($tool == 'protect' && !isset($recording['protected'])) {
             continue;
         }
         $buttonpayload = bigbluebuttonbn_get_recording_data_row_actionbar_payload($recording, $tool);
@@ -1113,7 +1113,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) {
@@ -1141,13 +1141,13 @@ 
     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-meetingid' => $recording['meetingID'], 'data-recordingid' => $recording['recordID'],
           'title' => $title, $visibility => $visibility));
     foreach ($recording['playbacks'] as $playback) {
         $recordingtypes .= bigbluebuttonbn_get_recording_data_row_type($recording, $bigbluebuttonbnid,
-            $playback).'&#32;';
+            $playback) . '&#32;';
     }
     $recordingtypes .= html_writer::end_tag('div');
     return $recordingtypes;
@@ -1155,12 +1155,12 @@ 
 
 function bigbluebuttonbn_get_recording_data_row_type($recording, $bigbluebuttonbnid, $playback) {
     global $CFG, $OUTPUT;
-    $title = get_string('view_recording_format_'.$playback['type'], 'bigbluebuttonbn');
+    $title = get_string('view_recording_format_' . $playback['type'], 'bigbluebuttonbn');
     $onclick = 'M.mod_bigbluebuttonbn.recordings.recording_play(this);';
-    $href = $CFG->wwwroot.'/mod/bigbluebuttonbn/bbb_view.php?action=play&bn='.$bigbluebuttonbnid.
-      '&mid='.$recording['meetingID'].'&rid='.$recording['recordID'].'&rtype='.$playback['type'];
+    $href = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_view.php?action=play&bn=' . $bigbluebuttonbnid .
+      '&mid=' . $recording['meetingID'] . '&rid=' . $recording['recordID'] . '&rtype=' . $playback['type'];
     if (!isset($recording['imported']) || !isset($recording['protected']) || $recording['protected'] === 'false') {
-        $href .= '&href='.urlencode(trim($playback['url']));
+        $href .= '&href=' . urlencode(trim($playback['url']));
     }
     $id = 'recording-play-' . $playback['type'] . '-' . $recording['recordID'];
     $linkattributes = array(
@@ -1242,11 +1242,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(
@@ -1328,7 +1328,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);
@@ -1343,7 +1343,7 @@ 
         return;
     }
     $row = new html_table_row();
-    $row->id = 'recording-td-'.$recording['recordID'];
+    $row->id = 'recording-td-' . $recording['recordID'];
     $row->attributes['data-imported'] = 'false';
     $texthead = '';
     $texttail = '';
@@ -1367,7 +1367,7 @@ 
 }
 
 function bigbluebuttonbn_include_recording_table_row($bbbsession, $recording) {
-    if ( isset($recording['imported']) || !isset($bbbsession['group']) || $recording['meetingID'] == $bbbsession['meetingid'] ) {
+    if (isset($recording['imported']) || !isset($bbbsession['group']) || $recording['meetingID'] == $bbbsession['meetingid']) {
         return true;
     }
     return false;
@@ -1380,9 +1380,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);
 }
 
@@ -1472,15 +1472,15 @@ 
 
 function bigbluebuttonbn_get_recordings_sql_selectdeleted($courseid, $bigbluebuttonbnid = null, $subset = true) {
     if ($bigbluebuttonbnid === null) {
-        return "courseid = '{$courseid}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_DELETE.
+        return "courseid = '{$courseid}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_DELETE .
             "' AND meta like '%has_recordings%' AND meta like '%true%'";
     }
     if ($subset) {
-        return "bigbluebuttonbnid = '{$bigbluebuttonbnid}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_DELETE.
+        return "bigbluebuttonbnid = '{$bigbluebuttonbnid}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_DELETE .
             "' AND meta like '%has_recordings%' AND meta like '%true%'";
     }
-    return "courseid = '{$courseid}' AND bigbluebuttonbnid <> '{$bigbluebuttonbnid}' AND log = '".
-        BIGBLUEBUTTONBN_LOG_EVENT_DELETE."' AND meta like '%has_recordings%' AND meta like '%true%'";
+    return "courseid = '{$courseid}' AND bigbluebuttonbnid <> '{$bigbluebuttonbnid}' AND log = '" .
+        BIGBLUEBUTTONBN_LOG_EVENT_DELETE . "' AND meta like '%has_recordings%' AND meta like '%true%'";
 }
 
 function bigbluebuttonbn_get_allrecordings($courseid, $bigbluebuttonbnid = null, $subset = true,
@@ -1525,7 +1525,7 @@ 
     }
     // 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.
@@ -1600,8 +1600,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);
     }
     return $activitytime;
@@ -1711,7 +1711,7 @@ 
         // This feature only works if curl is installed.
         $preuploadpresentationdescrip = get_string('config_preuploadpresentation_description', 'bigbluebuttonbn');
         if (!extension_loaded('curl')) {
-            $preuploadpresentationdescrip .= '<div class="form-defaultinfo">'.get_string('config_warning_curl_not_installed', 'bigbluebuttonbn').'</div><br>';
+            $preuploadpresentationdescrip .= '<div class="form-defaultinfo">' . get_string('config_warning_curl_not_installed', 'bigbluebuttonbn') . '</div><br>';
         }
         $renderer->render_group_header('preuploadpresentation', null, $preuploadpresentationdescrip);
         if (extension_loaded('curl')) {

classes/locallib/config.php

Indentation +23 added lines, -23 removed lines

@@ -119,28 +119,28 @@
      */
     public static function get_options() {
         return [
-               'version_major' => self::get_moodle_version_major(),
-               'voicebridge_editable' => self::get('voicebridge_editable'),
-               'waitformoderator_default' => self::get('waitformoderator_default'),
-               'waitformoderator_editable' => self::get('waitformoderator_editable'),
-               'userlimit_default' => self::get('userlimit_default'),
-               'userlimit_editable' => self::get('userlimit_editable'),
-               'preuploadpresentation_enabled' => self::get('preuploadpresentation_enabled'),
-               'sendnotifications_enabled' => self::get('sendnotifications_enabled'),
-               'recordings_enabled' => self::get('recordings_enabled'),
-               'recordings_html_default' => self::get('recordings_html_default'),
-               'recordings_html_editable' => self::get('recordings_html_editable'),
-               'recordings_deleted_default' => self::get('recordings_deleted_default'),
-               'recordings_deleted_editable' => self::get('recordings_deleted_editable'),
-               'recordings_imported_default' => self::get('recordings_imported_default'),
-               'recordings_imported_editable' => self::get('recordings_imported_editable'),
-               'recording_default' => self::get('recording_default'),
-               'recording_editable' => self::get('recording_editable'),
-               'recording_icons_enabled' => self::get('recording_icons_enabled'),
-               'general_warning_message' => self::get('general_warning_message'),
-               'general_warning_button_text' => self::get('general_warning_button_text'),
-               'general_warning_button_href' => self::get('general_warning_button_href'),
-               'general_warning_button_class' => self::get('general_warning_button_class'),
-          ];
+                'version_major' => self::get_moodle_version_major(),
+                'voicebridge_editable' => self::get('voicebridge_editable'),
+                'waitformoderator_default' => self::get('waitformoderator_default'),
+                'waitformoderator_editable' => self::get('waitformoderator_editable'),
+                'userlimit_default' => self::get('userlimit_default'),
+                'userlimit_editable' => self::get('userlimit_editable'),
+                'preuploadpresentation_enabled' => self::get('preuploadpresentation_enabled'),
+                'sendnotifications_enabled' => self::get('sendnotifications_enabled'),
+                'recordings_enabled' => self::get('recordings_enabled'),
+                'recordings_html_default' => self::get('recordings_html_default'),
+                'recordings_html_editable' => self::get('recordings_html_editable'),
+                'recordings_deleted_default' => self::get('recordings_deleted_default'),
+                'recordings_deleted_editable' => self::get('recordings_deleted_editable'),
+                'recordings_imported_default' => self::get('recordings_imported_default'),
+                'recordings_imported_editable' => self::get('recordings_imported_editable'),
+                'recording_default' => self::get('recording_default'),
+                'recording_editable' => self::get('recording_editable'),
+                'recording_icons_enabled' => self::get('recording_icons_enabled'),
+                'general_warning_message' => self::get('general_warning_message'),
+                'general_warning_button_text' => self::get('general_warning_button_text'),
+                'general_warning_button_href' => self::get('general_warning_button_href'),
+                'general_warning_button_class' => self::get('general_warning_button_class'),
+            ];
     }
 }

Spacing +5 added lines, -5 removed lines

@@ -44,8 +44,8 @@ 
      */
     public static function defaultvalues() {
         return array(
-            'server_url' => (string) BIGBLUEBUTTONBN_DEFAULT_SERVER_URL,
-            'server_url' => (string) BIGBLUEBUTTONBN_DEFAULT_SHARED_SECRET,
+            'server_url' => (string)BIGBLUEBUTTONBN_DEFAULT_SERVER_URL,
+            'server_url' => (string)BIGBLUEBUTTONBN_DEFAULT_SHARED_SECRET,
             'importrecordings_enabled' => 'false',
             'voicebridge_editable' => 'false',
             'importrecordings_enabled' => 'false',
@@ -99,10 +99,10 @@ 
     public static function get($setting) {
         global $CFG;
         if (isset($CFG->bigbluebuttonbn[$setting])) {
-            return (string) $CFG->bigbluebuttonbn[$setting];
+            return (string)$CFG->bigbluebuttonbn[$setting];
         }
-        if (isset($CFG->{'bigbluebuttonbn_'.$setting})) {
-            return (string)$CFG->{'bigbluebuttonbn_'.$setting};
+        if (isset($CFG->{'bigbluebuttonbn_' . $setting})) {
+            return (string)$CFG->{'bigbluebuttonbn_' . $setting};
         }
         return  self::defaultvalue($setting);
     }

classes/settings/renderer.php

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
 defined('MOODLE_INTERNAL') || die();
 
 require_once($CFG->dirroot . '/mod/bigbluebuttonbn/locallib.php');
-require_once($CFG->libdir.'/adminlib.php');
+require_once($CFG->libdir . '/adminlib.php');
 
 class renderer {
 
@@ -45,7 +45,7 @@ 
             $itemname = get_string('config_' . $name, 'bigbluebuttonbn');
         }
         if ($itemdescription == null) {
-            $itemdescription = get_string('config_' .$name . '_description', 'bigbluebuttonbn');
+            $itemdescription = get_string('config_' . $name . '_description', 'bigbluebuttonbn');
         }
         $item = new \admin_setting_heading('bigbluebuttonbn_config_' . $name, $itemname, $itemdescription);
         $this->settings->add($item);

config-dist.php

Indentation +13 added lines, -13 removed lines

@@ -94,7 +94,7 @@ 
  * recordings from a different activity even from a different course.
  **/
 
- /*
+    /*
  * When the value is set to 1 (checked) the bigbluebuttonbn rooms or
  * activities will have the 'import recordings' capability enabled.
  */
@@ -115,7 +115,7 @@ 
  * 'Join session' button enabled
  **/
 
- /*
+    /*
  * When the value is set to 1 (checked) the bigbluebuttonbn rooms or
  * activities will have the 'wait for moderator' capability enabled by
  * default.
@@ -149,7 +149,7 @@ 
  * 5. CONFIGURATION FOR "STATIC VOICE BRIDGE" FEATURE
  *
  **/
- /*
+    /*
  * A conference voice bridge number can be permanently assigned to a room
  * or activity.
  */
@@ -159,7 +159,7 @@ 
  * 6. CONFIGURATION FOR "PRE-UPLOAD PRESENTATION" FEATURE
  *
  **/
- /*
+    /*
  * Since version 0.8, BigBluebutton has an implementation for allowing
  * preuploading presentation. When this feature is enabled, users creating or
  * editing a room or activity can upload a PDF or Office document to the
@@ -176,7 +176,7 @@ 
  * applied to each room or activity, or globally.
  **/
 
- /*
+    /*
  * The number of users allowed in a session by default when a new room or
  * conference is added. If the number is set to 0, no limit is established.
  * $CFG->bigbluebuttonbn['userlimit_default'] = 0;
@@ -197,7 +197,7 @@ 
  * By default only the owner is assigned.
  **/
 
- /*
+    /*
  * The values for this parameter can be 'owner' and/or any of the roles defined in
  * Moodle (including the custom parameters). The value used will be the key for the role.
  * [owner|manager|coursecreator|editingteacher|teacher|student|guest|user|frontpage|ANY_CUSTOM_ROLE]
@@ -208,7 +208,7 @@ 
  * 9. CONFIGURATION FOR "NOTIFICATION SENDING" FEATURE
  *
  **/
- /*
+    /*
  * When the value is set to 1 (checked) the 'notification sending'
  * capability can be used by the user creating or editing the room or
  * activity.
@@ -222,7 +222,7 @@ 
  * 10. CONFIGURATION FOR "RECORDING READY" FEATURE
  *
  **/
- /*
+    /*
  * When the value is set to 1 (checked) the 'notify users when recording ready'
  * capability is enabled, meaning that a message will be sent to all enrolled
  * users in a course when a recording is ready
@@ -246,7 +246,7 @@ 
  * 12. GENERAL CONFIGURATION FOR RECORDINGS UI
  *
  **/
- /*
+    /*
  * When the value is set to 1 (checked) the bigbluebuttonbn resources
  * will show the recodings in an html table by default.
  * $CFG->bigbluebuttonbn['recordings_html_default'] = 0;
@@ -270,22 +270,22 @@ 
  * $CFG->bigbluebuttonbn['recordings_deleted_editable'] = 0;
  */
 
- /*
+    /*
   * When the value is set to 1 (checked) the bigbluebuttonbn resources for recordings
   * will show only the imported links as part of the list.
   * $CFG->bigbluebuttonbn['recordings_imported_default'] = 0;
   */
 
- /*
+    /*
   * When the value is set to 1 (checked) the 'show only imported links'
   * capability can be enabled/disabled by the user creating or editing the resource for recordings.
   * $CFG->bigbluebuttonbn['recordings_imported_editable'] = 1;
   */
 
-  /*
+    /*
    *  CONFIGURATION FOR FEATURES OFFERED BY BN SERVERS
   ** ------------------------------------------------------------------ **/
-  /*
+    /*
    * When general_warning_message value is different than "", the string is shown
    * as a warning message to privileged users (administrators and Teachers or users allowed to edit).
    * $CFG->bigbluebuttonbn['general_warning_message'] = "This will may have recordings enabled when upgraded to premium.";

bbb_view.php

Switch Indentation +79 added lines, -79 removed lines

@@ -61,104 +61,104 @@
 }
 
 switch (strtolower($action)) {
-    case 'logout':
-        if (isset($errors) && $errors != '') {
-            bigbluebutton_bbb_view_errors($errors, $id);
-            break;
-        }
-
-        if (is_null($bbbsession)) {
-            bigbluebutton_bbb_view_close_window_manually();
-            break;
-        }
+        case 'logout':
+            if (isset($errors) && $errors != '') {
+                bigbluebutton_bbb_view_errors($errors, $id);
+                break;
+            }
 
-        // Moodle event logger: Create an event for meeting left.
-        bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_LEFT, $bigbluebuttonbn, $cm);
+            if (is_null($bbbsession)) {
+                bigbluebutton_bbb_view_close_window_manually();
+                break;
+            }
 
-        // Update the cache.
-        $meetinginfo = bigbluebuttonbn_get_meeting_info($bbbsession['meetingid'], BIGBLUEBUTTONBN_FORCED);
+            // Moodle event logger: Create an event for meeting left.
+            bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_LEFT, $bigbluebuttonbn, $cm);
 
-        // Close the tab or window where BBB was opened.
-        bigbluebutton_bbb_view_close_window();
-        break;
-    case 'join':
-        if (is_null($bbbsession)) {
-            print_error('view_error_unable_join', 'bigbluebuttonbn');
-            break;
-        }
+            // Update the cache.
+            $meetinginfo = bigbluebuttonbn_get_meeting_info($bbbsession['meetingid'], BIGBLUEBUTTONBN_FORCED);
 
-        // See if the session is in progress.
-        if (bigbluebuttonbn_is_meeting_running($bbbsession['meetingid'])) {
-            // Since the meeting is already running, we just join the session.
-            bigbluebutton_bbb_view_join_meeting($bbbsession, $cm, $bigbluebuttonbn);
+            // Close the tab or window where BBB was opened.
+            bigbluebutton_bbb_view_close_window();
             break;
-        }
+        case 'join':
+            if (is_null($bbbsession)) {
+                print_error('view_error_unable_join', 'bigbluebuttonbn');
+                break;
+            }
 
-        // If user is not administrator nor moderator (user is steudent) and waiting is required.
-        if (!$bbbsession['administrator'] && !$bbbsession['moderator'] && $bbbsession['wait']) {
-            header('Location: '.$bbbsession['logoutURL']);
-            break;
-        }
+            // See if the session is in progress.
+            if (bigbluebuttonbn_is_meeting_running($bbbsession['meetingid'])) {
+                // Since the meeting is already running, we just join the session.
+                bigbluebutton_bbb_view_join_meeting($bbbsession, $cm, $bigbluebuttonbn);
+                break;
+            }
 
-        // As the meeting doesn't exist, try to create it.
-        $response = bigbluebuttonbn_get_create_meeting_array(
-            bigbluebutton_bbb_view_create_meeting_data($bbbsession, $bigbluebuttonbn),
-            bigbluebutton_bbb_view_create_meeting_metadata($bbbsession),
-            $bbbsession['presentation']['name'],
-            $bbbsession['presentation']['url']
-        );
-
-        if (!$response) {
-            // The server is unreachable.
-            if ($bbbsession['administrator']) {
-                print_error('view_error_unable_join', 'bigbluebuttonbn',
-                    $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+            // If user is not administrator nor moderator (user is steudent) and waiting is required.
+            if (!$bbbsession['administrator'] && !$bbbsession['moderator'] && $bbbsession['wait']) {
+                header('Location: '.$bbbsession['logoutURL']);
                 break;
             }
-            if ($bbbsession['moderator']) {
-                print_error('view_error_unable_join_teacher', 'bigbluebuttonbn',
+
+            // As the meeting doesn't exist, try to create it.
+            $response = bigbluebuttonbn_get_create_meeting_array(
+                bigbluebutton_bbb_view_create_meeting_data($bbbsession, $bigbluebuttonbn),
+                bigbluebutton_bbb_view_create_meeting_metadata($bbbsession),
+                $bbbsession['presentation']['name'],
+                $bbbsession['presentation']['url']
+            );
+
+            if (!$response) {
+                // The server is unreachable.
+                if ($bbbsession['administrator']) {
+                    print_error('view_error_unable_join', 'bigbluebuttonbn',
+                        $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+                    break;
+                }
+                if ($bbbsession['moderator']) {
+                    print_error('view_error_unable_join_teacher', 'bigbluebuttonbn',
+                        $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+                    break;
+                }
+
+                print_error('view_error_unable_join_student', 'bigbluebuttonbn',
                     $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
                 break;
             }
 
-            print_error('view_error_unable_join_student', 'bigbluebuttonbn',
-                $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
-            break;
-        }
+            if ($response['returncode'] == 'FAILED') {
+                // The meeting was not created.
+                if (!$printerrorkey) {
+                    print_error($response['message'], 'bigbluebuttonbn');
+                    break;
+                }
+                $printerrorkey = bigbluebuttonbn_get_error_key($response['messageKey'], 'view_error_create');
+                print_error($printerrorkey, 'bigbluebuttonbn');
+                break;
+            }
 
-        if ($response['returncode'] == 'FAILED') {
-            // The meeting was not created.
-            if (!$printerrorkey) {
-                print_error($response['message'], 'bigbluebuttonbn');
+            if ($response['hasBeenForciblyEnded'] == 'true') {
+                print_error(get_string('index_error_forciblyended', 'bigbluebuttonbn'));
                 break;
             }
-            $printerrorkey = bigbluebuttonbn_get_error_key($response['messageKey'], 'view_error_create');
-            print_error($printerrorkey, 'bigbluebuttonbn');
-            break;
-        }
 
-        if ($response['hasBeenForciblyEnded'] == 'true') {
-            print_error(get_string('index_error_forciblyended', 'bigbluebuttonbn'));
+            // Moodle event logger: Create an event for meeting created.
+            bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_CREATED, $bigbluebuttonbn, $cm);
+            // Internal logger: Insert a record with the meeting created.
+            bigbluebuttonbn_logs($bbbsession, BIGBLUEBUTTONBN_LOG_EVENT_CREATE);
+            // Since the meeting is already running, we just join the session.
+            bigbluebutton_bbb_view_join_meeting($bbbsession, $cm, $bigbluebuttonbn);
             break;
-        }
+        case 'play':
+            $href = bigbluebutton_bbb_view_playback_href($href, $mid, $rid, $rtype);
 
-        // Moodle event logger: Create an event for meeting created.
-        bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_CREATED, $bigbluebuttonbn, $cm);
-        // Internal logger: Insert a record with the meeting created.
-        bigbluebuttonbn_logs($bbbsession, BIGBLUEBUTTONBN_LOG_EVENT_CREATE);
-        // Since the meeting is already running, we just join the session.
-        bigbluebutton_bbb_view_join_meeting($bbbsession, $cm, $bigbluebuttonbn);
-        break;
-    case 'play':
-        $href = bigbluebutton_bbb_view_playback_href($href, $mid, $rid, $rtype);
-
-        // Moodle event logger: Create an event for meeting left.
-        bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_RECORDING_VIEWED, $bigbluebuttonbn, $cm, ['other' => $rid]);
-        // Execute the redirect.
-        header('Location: '.urldecode($href));
-        break;
-    default:
-        bigbluebutton_bbb_view_close_window();
+            // Moodle event logger: Create an event for meeting left.
+            bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_RECORDING_VIEWED, $bigbluebuttonbn, $cm, ['other' => $rid]);
+            // Execute the redirect.
+            header('Location: '.urldecode($href));
+            break;
+        default:
+            bigbluebutton_bbb_view_close_window();
 }
 
 function bigbluebutton_bbb_view_playback_href($href, $mid, $rid, $rtype) {

view.php

Spacing +47 added lines, -47 removed lines

@@ -23,8 +23,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');
 
 $id = required_param('id', PARAM_INT);
 $bn = optional_param('n', 0, PARAM_INT);
@@ -56,21 +56,21 @@ 
 if (is_null($serverversion)) {
     if ($bbbsession['administrator']) {
         print_error('view_error_unable_join', 'bigbluebuttonbn',
-            $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+            $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
         exit;
     }
 
     if ($bbbsession['moderator']) {
         print_error('view_error_unable_join_teacher', 'bigbluebuttonbn',
-            $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course);
+            $CFG->wwwroot . '/course/view.php?id=' . $bigbluebuttonbn->course);
         exit;
     }
 
     print_error('view_error_unable_join_student', 'bigbluebuttonbn',
-        $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course);
+        $CFG->wwwroot . '/course/view.php?id=' . $bigbluebuttonbn->course);
     exit;
 }
-$bbbsession['serverversion'] = (string) $serverversion;
+$bbbsession['serverversion'] = (string)$serverversion;
 
 // Mark viewed by user (if required).
 $completion = new completion_info($course);
@@ -78,7 +78,7 @@ 
 
 // Print the page header.
 $PAGE->set_context($context);
-$PAGE->set_url($CFG->wwwroot.'/mod/bigbluebuttonbn/view.php', array('id' => $cm->id));
+$PAGE->set_url($CFG->wwwroot . '/mod/bigbluebuttonbn/view.php', array('id' => $cm->id));
 $PAGE->set_title(format_string($bigbluebuttonbn->name));
 $PAGE->set_cacheable(false);
 $PAGE->set_heading($course->fullname);
@@ -88,11 +88,11 @@ 
 if (!has_capability('moodle/category:manage', $context) && !has_capability('mod/bigbluebuttonbn:join', $context)) {
     echo $OUTPUT->header();
     if (isguestuser()) {
-        echo $OUTPUT->confirm('<p>'.get_string('view_noguests', 'bigbluebuttonbn').'</p>'.get_string('liketologin'),
-            get_login_url(), $CFG->wwwroot.'/course/view.php?id='.$course->id);
+        echo $OUTPUT->confirm('<p>' . get_string('view_noguests', 'bigbluebuttonbn') . '</p>' . get_string('liketologin'),
+            get_login_url(), $CFG->wwwroot . '/course/view.php?id=' . $course->id);
     } else {
-        echo $OUTPUT->confirm('<p>'.get_string('view_nojoin', 'bigbluebuttonbn').'</p>'.get_string('liketologin'),
-            get_login_url(), $CFG->wwwroot.'/course/view.php?id='.$course->id);
+        echo $OUTPUT->confirm('<p>' . get_string('view_nojoin', 'bigbluebuttonbn') . '</p>' . get_string('liketologin'),
+            get_login_url(), $CFG->wwwroot . '/course/view.php?id=' . $course->id);
     }
     echo $OUTPUT->footer();
     exit;
@@ -100,7 +100,7 @@ 
 
 // Operation URLs.
 $bbbsession['bigbluebuttonbnURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $bbbsession['cm']->id;
-$bbbsession['logoutURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_view.php?action=logout&id='.$id .
+$bbbsession['logoutURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_view.php?action=logout&id=' . $id .
     '&bn=' . $bbbsession['bigbluebuttonbn']->id;
 $bbbsession['recordingReadyURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_broker.php?action=recording_' .
     'ready&bigbluebuttonbn=' . $bbbsession['bigbluebuttonbn']->id;
@@ -120,7 +120,7 @@ 
 echo $OUTPUT->footer();
 
 // Shows version as a comment.
-echo '<!-- '.$bbbsession['originTag'].' -->'."\n";
+echo '<!-- ' . $bbbsession['originTag'] . ' -->' . "\n";
 
 // Initialize session variable used across views.
 $SESSION->bigbluebuttonbn_bbbsession = $bbbsession;
@@ -150,7 +150,7 @@ 
     $bbbsession['viewerPW'] = $bigbluebuttonbn->viewerpass;
 
     // Database info related to the activity.
-    $bbbsession['meetingid'] = $bbbsession['bigbluebuttonbn']->meetingid.'-'.$bbbsession['course']->id.'-'.
+    $bbbsession['meetingid'] = $bbbsession['bigbluebuttonbn']->meetingid . '-' . $bbbsession['course']->id . '-' .
         $bbbsession['bigbluebuttonbn']->id;
     $bbbsession['meetingname'] = $bbbsession['bigbluebuttonbn']->name;
     $bbbsession['meetingdescription'] = $bigbluebuttonbn->intro;
@@ -171,7 +171,7 @@ 
         $bbbsession['welcome'] = get_string('mod_form_field_welcome_default', 'bigbluebuttonbn');
     }
     if ($bigbluebuttonbn->record) {
-        $bbbsession['welcome'] .= '<br><br>'.get_string('bbbrecordwarning', 'bigbluebuttonbn');
+        $bbbsession['welcome'] .= '<br><br>' . get_string('bbbrecordwarning', 'bigbluebuttonbn');
     }
 
     $bbbsession['openingtime'] = $bigbluebuttonbn->openingtime;
@@ -187,7 +187,7 @@ 
     $bbbsession['originServerName'] = $parsedurl['host'];
     $bbbsession['originServerUrl'] = $CFG->wwwroot;
     $bbbsession['originServerCommonName'] = '';
-    $bbbsession['originTag'] = 'moodle-mod_bigbluebuttonbn ('.get_config('mod_bigbluebuttonbn', 'version').')';
+    $bbbsession['originTag'] = 'moodle-mod_bigbluebuttonbn (' . get_config('mod_bigbluebuttonbn', 'version') . ')';
 }
 
 function bigbluebuttonbn_view_bbbsession_roles($context, $userid) {
@@ -244,10 +244,10 @@ 
     $bbbsession['group'] = groups_get_activity_group($bbbsession['cm'], true);
 
     // Assign group default values.
-    $bbbsession['meetingid'] = $bbbsession['bigbluebuttonbn']->meetingid.'-'.$bbbsession['course']->id.'-'.
-        $bbbsession['bigbluebuttonbn']->id.'['.$bbbsession['group'].']';
+    $bbbsession['meetingid'] = $bbbsession['bigbluebuttonbn']->meetingid . '-' . $bbbsession['course']->id . '-' .
+        $bbbsession['bigbluebuttonbn']->id . '[' . $bbbsession['group'] . ']';
     $groupname = get_string('allparticipants');
-    $bbbsession['meetingname'] = $bbbsession['bigbluebuttonbn']->name.' ('.$groupname.')';
+    $bbbsession['meetingname'] = $bbbsession['bigbluebuttonbn']->name . ' (' . $groupname . ')';
 
     if (count($groups) == 0) {
         // Only the All participants group exists.
@@ -258,10 +258,10 @@ 
         $bbbsession['group'] = array_values($groups)[0]->id;
     }
 
-    $bbbsession['meetingid'] = $bbbsession['bigbluebuttonbn']->meetingid.'-'.$bbbsession['course']->id.'-'.
-        $bbbsession['bigbluebuttonbn']->id.'['.$bbbsession['group'].']';
+    $bbbsession['meetingid'] = $bbbsession['bigbluebuttonbn']->meetingid . '-' . $bbbsession['course']->id . '-' .
+        $bbbsession['bigbluebuttonbn']->id . '[' . $bbbsession['group'] . ']';
     $groupname = groups_get_group_name($bbbsession['group']);
-    $bbbsession['meetingname'] = $bbbsession['bigbluebuttonbn']->name.' ('.$groupname.')';
+    $bbbsession['meetingname'] = $bbbsession['bigbluebuttonbn']->name . ' (' . $groupname . ')';
 
     if (count($groups) == 1) {
         // There only one group and the user has access to.
@@ -269,12 +269,12 @@ 
     }
 
     echo $OUTPUT->box_start('generalbox boxaligncenter');
-    echo '<br><div class="alert alert-warning">'.get_string('view_groups_selection_warning', 'bigbluebuttonbn').
+    echo '<br><div class="alert alert-warning">' . get_string('view_groups_selection_warning', 'bigbluebuttonbn') .
       '</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>';
 }
 
@@ -317,7 +317,7 @@ 
                 'M.mod_bigbluebuttonbn.recordings.init', array($jsvars));
     }
 
-    echo $output.html_writer::empty_tag('br').html_writer::empty_tag('br').html_writer::empty_tag('br');
+    echo $output . html_writer::empty_tag('br') . html_writer::empty_tag('br') . html_writer::empty_tag('br');
     $PAGE->requires->yui_module('moodle-mod_bigbluebuttonbn-broker', 'M.mod_bigbluebuttonbn.broker.init', array($jsvars));
 }
 
@@ -341,21 +341,21 @@ 
     return $output;
 }
 
-function bigbluebuttonbn_view_render_warning($message, $type='info', $href='', $text='', $class='') {
+function bigbluebuttonbn_view_render_warning($message, $type = 'info', $href = '', $text = '', $class = '') {
     global $OUTPUT;
     $output = "\n";
     // Evaluates if config_warning is enabled.
     if (empty($message)) {
         return $output;
     }
-    $output .= $OUTPUT->box_start('box boxalignleft adminerror alert alert-' . $type . ' alert-block fade in', 'bigbluebuttonbn_view_general_warning')."\n";
-    $output .= '  <button type="button" class="close" data-dismiss="alert">&times;</button>'.$message."\n";
-    $output .= '  <div class="singlebutton">'."\n";
+    $output .= $OUTPUT->box_start('box boxalignleft adminerror alert alert-' . $type . ' alert-block fade in', 'bigbluebuttonbn_view_general_warning') . "\n";
+    $output .= '  <button type="button" class="close" data-dismiss="alert">&times;</button>' . $message . "\n";
+    $output .= '  <div class="singlebutton">' . "\n";
     if (!empty($href)) {
         $output .= bigbluebutton_view_render_general_warning_button($href, $text, $class);
     }
-    $output .= '  </div>'."\n";
-    $output .= $OUTPUT->box_end()."\n";
+    $output .= '  </div>' . "\n";
+    $output .= $OUTPUT->box_end() . "\n";
     return $output;
 }
 
@@ -366,11 +366,11 @@ 
     if ($class == '') {
         $class = 'btn btn-secondary';
     }
-    $output  = '  <form method="post" action="' . $href . '" class="form-inline">'."\n";
-    $output .= '      <button type="submit" class="' . $class . '"'."\n";
-    $output .= '          title=""'."\n";
-    $output .= '          >' . $text . '</button>'."\n";
-    $output .= '  </form>'."\n";
+    $output  = '  <form method="post" action="' . $href . '" class="form-inline">' . "\n";
+    $output .= '      <button type="submit" class="' . $class . '"' . "\n";
+    $output .= '          title=""' . "\n";
+    $output .= '          >' . $text . '</button>' . "\n";
+    $output .= '  </form>' . "\n";
     return $output;
 }
 
@@ -380,12 +380,12 @@ 
     // JavaScript variables for room.
     $openingtime = '';
     if ($bbbsession['openingtime']) {
-        $openingtime = get_string('mod_form_field_openingtime', 'bigbluebuttonbn').': '.
+        $openingtime = get_string('mod_form_field_openingtime', 'bigbluebuttonbn') . ': ' .
             userdate($bbbsession['openingtime']);
     }
     $closingtime = '';
     if ($bbbsession['closingtime']) {
-        $closingtime = get_string('mod_form_field_closingtime', 'bigbluebuttonbn').': '.
+        $closingtime = get_string('mod_form_field_closingtime', 'bigbluebuttonbn') . ': ' .
             userdate($bbbsession['closingtime']);
     }
     $jsvars += array(
@@ -402,7 +402,7 @@ 
     $output .= $OUTPUT->box_end();
 
     $output .= $OUTPUT->box_start('generalbox boxaligncenter', 'bigbluebuttonbn_view_action_button_box');
-    $output .= '<br><br><span id="join_button"></span>&nbsp;<span id="end_button"></span>'."\n";
+    $output .= '<br><br><span id="join_button"></span>&nbsp;<span id="end_button"></span>' . "\n";
     $output .= $OUTPUT->box_end();
 
     if ($activity == 'ended') {
@@ -428,7 +428,7 @@ 
 
     // Get recordings.
     $recordings = array();
-    if ( bigbluebuttonbn_view_include_recordings($bbbsession) ) {
+    if (bigbluebuttonbn_view_include_recordings($bbbsession)) {
         $recordings = bigbluebuttonbn_get_recordings(
             $bbbsession['course']->id, $bigbluebuttonbnid, $showroom,
             $bbbsession['bigbluebuttonbn']->recordings_deleted
@@ -459,7 +459,7 @@ 
     // If there are meetings with recordings load the data to the table.
     if ($bbbsession['bigbluebuttonbn']->recordings_html) {
         // Render a plain html table.
-        return bigbluebutton_output_recording_table($bbbsession, $recordings)."\n";
+        return bigbluebutton_output_recording_table($bbbsession, $recordings) . "\n";
     }
 
     // JavaScript variables for recordings with YUI.
@@ -479,8 +479,8 @@ 
         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.'\''));
+              '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'));
@@ -495,10 +495,10 @@ 
         $attributes = array('title' => $bbbsession['presentation']['name']);
         $icon = new pix_icon($bbbsession['presentation']['icon'], $bbbsession['presentation']['mimetype_description']);
 
-        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>';
+        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>';
     }
 
     return '';

settings.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
 
 global $CFG;
 
-require_once(dirname(__FILE__).'/locallib.php');
+require_once(dirname(__FILE__) . '/locallib.php');
 
 if ($ADMIN->fulltree) {
     // Configuration for BigBlueButton.

mod_form.php

Spacing +8 added lines, -8 removed lines

@@ -26,8 +26,8 @@ 
 
 defined('MOODLE_INTERNAL') || die();
 
-require_once(dirname(__FILE__).'/locallib.php');
-require_once($CFG->dirroot.'/course/moodleform_mod.php');
+require_once(dirname(__FILE__) . '/locallib.php');
+require_once($CFG->dirroot . '/course/moodleform_mod.php');
 
 class mod_bigbluebuttonbn_mod_form extends moodleform_mod {
 
@@ -37,7 +37,7 @@ 
         $serverversion = bigbluebuttonbn_get_server_version();
         if (is_null($serverversion)) {
             print_error('general_error_unable_connect', 'bigbluebuttonbn',
-                $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+                $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
             return;
         }
         // Context.
@@ -110,7 +110,7 @@ 
                 );
                 $defaultvalues['presentation'] = $draftitemid;
             } catch (Exception $e) {
-                debugging('Presentation could not be loaded: '.$e->getMessage(), DEBUG_DEVELOPER);
+                debugging('Presentation could not be loaded: ' . $e->getMessage(), DEBUG_DEVELOPER);
                 return null;
             }
         }
@@ -282,10 +282,10 @@ 
             html_writer::select($participantselection['type_options'], 'bigbluebuttonbn_participant_selection_type',
                 $participantselection['type_selected'], array(),
                 array('id' => 'bigbluebuttonbn_participant_selection_type',
-                      'onchange' => 'M.mod_bigbluebuttonbn.modform.participant_selection_set(); return 0;')).'  '.
+                      'onchange' => 'M.mod_bigbluebuttonbn.modform.participant_selection_set(); return 0;')) . '  ' .
             html_writer::select($participantselection['options'], 'bigbluebuttonbn_participant_selection',
                 $participantselection['selected'], array(),
-                array('id' => 'bigbluebuttonbn_participant_selection', 'disabled' => 'disabled')).'  '.
+                array('id' => 'bigbluebuttonbn_participant_selection', 'disabled' => 'disabled')) . '  ' .
             html_writer::tag('input', '', array('id' => 'id_addselectionid', 'type' => 'button', 'class' => 'btn btn-secondary',
                 'value' => get_string('mod_form_field_participant_list_action_add', 'bigbluebuttonbn'),
                 'onclick' => 'M.mod_bigbluebuttonbn.modform.participant_add(); return 0;'
@@ -302,7 +302,7 @@ 
         // Render elements for participant list.
         $htmllist = html_writer::tag('div',
             html_writer::label(get_string('mod_form_field_participant_list', 'bigbluebuttonbn'),
-                'bigbluebuttonbn_participant_list').
+                'bigbluebuttonbn_participant_list') .
             html_writer::table($table)
         );
         $mform->addElement('html', "\n\n");
@@ -332,7 +332,7 @@ 
             return;
         }
         $mform->addElement($type, $name, get_string($descriptionkey, 'bigbluebuttonbn'), $options);
-        if (get_string_manager()->string_exists($descriptionkey.'_help', 'bigbluebuttonbn')) {
+        if (get_string_manager()->string_exists($descriptionkey . '_help', 'bigbluebuttonbn')) {
             $mform->addHelpButton($name, $descriptionkey, 'bigbluebuttonbn');
         }
         if (!empty($rule)) {