blindsidenetworks / moodle-mod_bigbluebuttonbn

backup/moodle2/backup_bigbluebuttonbn_stepslib.php

Spacing +2 added lines, -2 removed lines

@@ -40,12 +40,12 @@
                             'moderatorpass', 'viewerpass', 'wait', 'record', 'tagging',
                             'welcome', 'voicebridge', 'openingtime', 'closingtime',
                             'timecreated', 'timemodified', 'presentation', 'participants',
-                            'userlimit', ));
+                            'userlimit',));
 
         $logs = new backup_nested_element('logs');
 
         $log = new backup_nested_element('log', array('id'), array(
-                'courseid', 'bigbluebuttonbnid', 'userid', 'timecreated', 'meetingid', 'log', 'meta', ));
+                'courseid', 'bigbluebuttonbnid', 'userid', 'timecreated', 'meetingid', 'log', 'meta',));
 
         // Build the tree.
         $bigbluebuttonbn->add_child($logs);

settings.php

Spacing +3 added lines, -3 removed lines

@@ -27,7 +27,7 @@ 
 
 global $CFG;
 
-require_once(dirname(__FILE__).'/locallib.php');
+require_once(dirname(__FILE__) . '/locallib.php');
 
 if ($ADMIN->fulltree) {
     if (!isset($CFG->bigbluebuttonbn['server_url']) ||
@@ -258,8 +258,8 @@ 
         } else {
             $settings->add(new admin_setting_heading('bigbluebuttonbn_feature_preuploadpresentation',
                 get_string('config_feature_preuploadpresentation', 'bigbluebuttonbn'),
-                get_string('config_feature_preuploadpresentation_description', 'bigbluebuttonbn').'<br><br>'.
-                '<div class="form-defaultinfo">'.get_string('config_warning_curl_not_installed', 'bigbluebuttonbn').'</div><br>'
+                get_string('config_feature_preuploadpresentation_description', 'bigbluebuttonbn') . '<br><br>' .
+                '<div class="form-defaultinfo">' . get_string('config_warning_curl_not_installed', 'bigbluebuttonbn') . '</div><br>'
                 ));
         }
     }

classes/event/bigbluebuttonbn_live_session_event.php

Spacing +4 added lines, -4 removed lines

@@ -52,13 +52,13 @@ 
      * @return string
      */
     public function get_description() {
-        $a = (object) array('userid' => $this->userid,
+        $a = (object)array('userid' => $this->userid,
                             'bigbluebuttonbnid' => $this->objectid,
                             'courseid' => $this->contextinstanceid,
                             'action' => $this->other);
 
-        return "The user with id '$a->userid' triggered action $a->action in a ".
-        "bigbluebutton meeting for the bigbluebuttonbn activity with id ".
+        return "The user with id '$a->userid' triggered action $a->action in a " .
+        "bigbluebutton meeting for the bigbluebuttonbn activity with id " .
         "'$a->bigbluebuttonbnid' for the course id '$a->courseid'.";
     }
 
@@ -69,7 +69,7 @@ 
      */
     protected function get_legacy_logdata() {
         return array($this->courseid, 'bigbluebuttonbn', 'meeting left',
-                'view.php?pageid='.$this->objectid, 'BigBlueButtonBN meeting live session event',
+                'view.php?pageid=' . $this->objectid, 'BigBlueButtonBN meeting live session event',
                 $this->contextinstanceid);
     }
 

config-dist.php

Indentation +9 added lines, -9 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
@@ -245,7 +245,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;

index.php

Spacing +18 added lines, -18 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);
 $a = optional_param('a', 0, PARAM_INT);
@@ -92,12 +92,12 @@ 
 
         echo get_string('index_ending', 'bigbluebuttonbn');
 
-        $meetingid = $bigbluebuttonbn->meetingid.'-'.$course->id.'-'.$bigbluebuttonbn->id;
+        $meetingid = $bigbluebuttonbn->meetingid . '-' . $course->id . '-' . $bigbluebuttonbn->id;
         if ($g != '0') {
-            $meetingid .= '['.$g.']';
+            $meetingid .= '[' . $g . ']';
         }
         bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getEndMeetingURL($meetingid));
-        redirect('index.php?id='.$id);
+        redirect('index.php?id=' . $id);
     }
 }
 
@@ -114,7 +114,7 @@ 
         // Add a the data for the bigbluebuttonbn instance.
         $groupobj = null;
         if (groups_get_activity_groupmode($cm) > 0) {
-            $groupobj = (object) array('id' => 0, 'name' => get_string('allparticipants'));
+            $groupobj = (object)array('id' => 0, 'name' => get_string('allparticipants'));
         }
         $table->data[] = bigbluebuttonbn_index_display_room($canmoderate, $course, $bigbluebuttonbn, $groupobj);
 
@@ -135,13 +135,13 @@ 
 
 // Functions.
 function bigbluebuttonbn_index_display_room($moderator, $course, $bigbluebuttonbn, $groupobj = null) {
-    $meetingid = $bigbluebuttonbn->meetingid.'-'.$course->id.'-'.$bigbluebuttonbn->id;
+    $meetingid = $bigbluebuttonbn->meetingid . '-' . $course->id . '-' . $bigbluebuttonbn->id;
     $paramgroup = '';
     $groupname = '';
 
     if ($groupobj) {
-        $meetingid .= '['.$groupobj->id.']';
-        $paramgroup = '&group='.$groupobj->id;
+        $meetingid .= '[' . $groupobj->id . ']';
+        $paramgroup = '&group=' . $groupobj->id;
         $groupname = $groupobj->name;
     }
 
@@ -160,7 +160,7 @@ 
     }
 
     // Output Users in the meeting.
-    $joinurl = '<a href="view.php?id='.$bigbluebuttonbn->coursemodule.$paramgroup.'">'.format_string($bigbluebuttonbn->name).'</a>';
+    $joinurl = '<a href="view.php?id=' . $bigbluebuttonbn->coursemodule . $paramgroup . '">' . format_string($bigbluebuttonbn->name) . '</a>';
     $group = $groupname;
     $users = '';
     $viewerlist = '';
@@ -197,7 +197,7 @@ 
         $attendeecount = 0;
         foreach ($meetinginfo['attendees']->attendee as $attendee) {
             if ($attendee->role == $role) {
-                $attendeelist .= ($attendeecount++ > 0 ? ', ' : '').$attendee->fullName;
+                $attendeelist .= ($attendeecount++ > 0 ? ', ' : '') . $attendee->fullName;
             }
         }
     }
@@ -220,15 +220,15 @@ 
     $actions = '';
 
     if ($moderator) {
-        $actions .= '<form name="form1" method="post" action="">'.'/n';
-        $actions .= '  <INPUT type="hidden" name="id" value="'.$course->id.'">'.'/n';
-        $actions .= '  <INPUT type="hidden" name="a" value="'.$bigbluebuttonbn->id.'">'.'/n';
+        $actions .= '<form name="form1" method="post" action="">' . '/n';
+        $actions .= '  <INPUT type="hidden" name="id" value="' . $course->id . '">' . '/n';
+        $actions .= '  <INPUT type="hidden" name="a" value="' . $bigbluebuttonbn->id . '">' . '/n';
         if ($groupobj != null) {
-            $actions .= '  <INPUT type="hidden" name="g" value="'.$groupobj->id.'">'.'/n';
+            $actions .= '  <INPUT type="hidden" name="g" value="' . $groupobj->id . '">' . '/n';
         }
-        $actions .= '  <INPUT type="submit" name="submit" value="end" onclick="return confirm(\''.
-            get_string('index_confirm_end', 'bigbluebuttonbn').'\')">'.'/n';
-        $actions .= '</form>'.'/n';
+        $actions .= '  <INPUT type="submit" name="submit" value="end" onclick="return confirm(\'' .
+            get_string('index_confirm_end', 'bigbluebuttonbn') . '\')">' . '/n';
+        $actions .= '</form>' . '/n';
     }
 
     return $actions;

db/upgrade.php

Indentation +36 added lines, -36 removed lines

@@ -36,23 +36,23 @@ 
 
         // Change welcome, allow null.
         $fielddefinition = array('type' => XMLDB_TYPE_TEXT,
-                                  'precision' => null,
-                                  'unsigned' => null,
-                                  'notnull' => XMLDB_NOTNULL,
-                                  'sequence' => null,
-                                  'default' => null,
-                                  'previous' => 'type');
+                                    'precision' => null,
+                                    'unsigned' => null,
+                                    'notnull' => XMLDB_NOTNULL,
+                                    'sequence' => null,
+                                    'default' => null,
+                                    'previous' => 'type');
         xmldb_bigbluebuttonbn_add_change_field($dbman, 'bigbluebuttonbn', 'welcome',
             $fielddefinition);
 
         // Change userid definition in bigbluebuttonbn_log.
         $fielddefinition = array('type' => XMLDB_TYPE_INTEGER,
-                                  'precision' => '10',
-                                  'unsigned' => null,
-                                  'notnull' => XMLDB_NOTNULL,
-                                  'sequence' => null,
-                                  'default' => null,
-                                  'previous' => 'bigbluebuttonbnid');
+                                    'precision' => '10',
+                                    'unsigned' => null,
+                                    'notnull' => XMLDB_NOTNULL,
+                                    'sequence' => null,
+                                    'default' => null,
+                                    'previous' => 'bigbluebuttonbnid');
         xmldb_bigbluebuttonbn_add_change_field($dbman, 'bigbluebuttonbn_log', 'userid',
             $fielddefinition);
 
@@ -78,34 +78,34 @@ 
 
         // Add field type.
         $fielddefinition = array('type' => XMLDB_TYPE_INTEGER,
-                                  'precision' => '2',
-                                  'unsigned' => null,
-                                  'notnull' => XMLDB_NOTNULL,
-                                  'sequence' => null,
-                                  'default' => 0,
-                                  'previous' => 'id');
+                                    'precision' => '2',
+                                    'unsigned' => null,
+                                    'notnull' => XMLDB_NOTNULL,
+                                    'sequence' => null,
+                                    'default' => 0,
+                                    'previous' => 'id');
         xmldb_bigbluebuttonbn_add_change_field($dbman, 'bigbluebuttonbn', 'type',
             $fielddefinition);
 
         // Add field recordings_html.
         $fielddefinition = array('type' => XMLDB_TYPE_INTEGER,
-                                  'precision' => '1',
-                                  'unsigned' => null,
-                                  'notnull' => XMLDB_NOTNULL,
-                                  'sequence' => null,
-                                  'default' => 0,
-                                  'previous' => null);
+                                    'precision' => '1',
+                                    'unsigned' => null,
+                                    'notnull' => XMLDB_NOTNULL,
+                                    'sequence' => null,
+                                    'default' => 0,
+                                    'previous' => null);
         xmldb_bigbluebuttonbn_add_change_field($dbman, 'bigbluebuttonbn', 'recordings_html',
             $fielddefinition);
 
         // Add field recordings_deleted_activities.
         $fielddefinition = array('type' => XMLDB_TYPE_INTEGER,
-                                  'precision' => '1',
-                                  'unsigned' => null,
-                                  'notnull' => XMLDB_NOTNULL,
-                                  'sequence' => null,
-                                  'default' => 1,
-                                  'previous' => null);
+                                    'precision' => '1',
+                                    'unsigned' => null,
+                                    'notnull' => XMLDB_NOTNULL,
+                                    'sequence' => null,
+                                    'default' => 1,
+                                    'previous' => null);
         xmldb_bigbluebuttonbn_add_change_field($dbman, 'bigbluebuttonbn', 'recordings_deleted_activities',
             $fielddefinition);
 
@@ -122,12 +122,12 @@ 
     $table = new xmldb_table($tablename);
     $field = new xmldb_field($fieldname);
     $field->set_attributes($fielddefinition['type'],
-                           $fielddefinition['precision'],
-                           $fielddefinition['unsigned'],
-                           $fielddefinition['notnull'],
-                           $fielddefinition['sequence'],
-                           $fielddefinition['default'],
-                           $fielddefinition['previous']);
+                            $fielddefinition['precision'],
+                            $fielddefinition['unsigned'],
+                            $fielddefinition['notnull'],
+                            $fielddefinition['sequence'],
+                            $fielddefinition['default'],
+                            $fielddefinition['previous']);
     if ($dbman->field_exists($table, $field)) {
         $dbman->change_field_type($table, $field, true, true);
         $dbman->change_field_precision($table, $field, true, true);

bbb_broker.php

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;
 }
 
@@ -133,11 +133,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;
 }
 
@@ -282,7 +282,7 @@ 
 
 function bigbluebuttonbn_broker_recording_info_current($recording, $params) {
     $callbackresponse['status'] = true;
-    $callbackresponse['published'] = (string) $recording['published'];
+    $callbackresponse['published'] = (string)$recording['published'];
     if (!isset($params['meta'])) {
         return $callbackresponse;
     }
@@ -451,8 +451,8 @@ 
         $decodedparameters = JWT::decode($params['signed_parameters'], bigbluebuttonbn_get_cfg_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;
     }
 
@@ -470,8 +470,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);
     }
 }
 
@@ -486,7 +486,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;
     }
 
@@ -494,7 +494,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'])) {
@@ -511,8 +511,8 @@ 
         $decodedparameters = JWT::decode($params['signed_parameters'], bigbluebuttonbn_get_cfg_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;
     }
 
@@ -550,7 +550,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]);

Doc Comments +9 added lines

@@ -179,6 +179,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) {
@@ -199,6 +202,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'])) {
@@ -345,6 +351,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);

Indentation +17 added lines, -17 removed lines

@@ -389,10 +389,10 @@ 
     if (bigbluebuttonbn_broker_recording_is_imported($recordings, $params['id'])) {
         // Execute unpublish 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.
@@ -412,26 +412,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) {
@@ -440,10 +440,10 @@ 
     if (bigbluebuttonbn_broker_recording_is_imported($recordings, $params['id'])) {
         // Execute delete on imported recording link.
         return array(
-          'status' => bigbluebuttonbn_delete_recording_imported(
-              $params['id'], $bbbsession['bigbluebuttonbn']->id
+            'status' => bigbluebuttonbn_delete_recording_imported(
+                $params['id'], $bbbsession['bigbluebuttonbn']->id
             )
-          );
+            );
     }
 
     // As the recordingid was not identified as imported recording link, execute delete on a real recording.
@@ -458,8 +458,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) {
@@ -600,10 +600,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;
 }
 

import_view.php

Indentation +2 added lines, -2 removed lines

@@ -96,8 +96,8 @@
     $output .= html_writer::start_tag('br');
     $output .= html_writer::tag('input', '',
         array('type' => 'button', 'class' => 'btn btn-secondary',
-              'value' => get_string('view_recording_button_return', 'bigbluebuttonbn'),
-              'onclick' => 'window.location=\''.$CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$cm->id.'\''));
+                'value' => get_string('view_recording_button_return', 'bigbluebuttonbn'),
+                'onclick' => 'window.location=\''.$CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$cm->id.'\''));
 
     // JavaScript for locales.
     $PAGE->requires->strings_for_js(array_keys(bigbluebuttonbn_get_strings_for_js()), 'bigbluebuttonbn');

Spacing +3 added lines, -3 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');
 
 $bn = required_param('bn', PARAM_INT);
 $tc = optional_param('tc', 0, PARAM_INT);
@@ -97,7 +97,7 @@ 
     $output .= html_writer::tag('input', '',
         array('type' => 'button', 'class' => 'btn btn-secondary',
               'value' => get_string('view_recording_button_return', 'bigbluebuttonbn'),
-              'onclick' => 'window.location=\''.$CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$cm->id.'\''));
+              'onclick' => 'window.location=\'' . $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $cm->id . '\''));
 
     // JavaScript for locales.
     $PAGE->requires->strings_for_js(array_keys(bigbluebuttonbn_get_strings_for_js()), 'bigbluebuttonbn');

bbb_view.php

Indentation +14 added lines, -14 removed lines

@@ -166,10 +166,10 @@ 
 
 function bigbluebutton_bbb_view_create_meeting_data(&$bbbsession, $bigbluebuttonbn) {
     $data = ['meetingID' => $bbbsession['meetingid'],
-              'name' => $bbbsession['meetingname'],
-              'attendeePW' => $bbbsession['viewerPW'],
-              'moderatorPW' => $bbbsession['modPW'],
-              'logoutURL' => $bbbsession['logoutURL'],
+                'name' => $bbbsession['meetingname'],
+                'attendeePW' => $bbbsession['viewerPW'],
+                'moderatorPW' => $bbbsession['modPW'],
+                'logoutURL' => $bbbsession['logoutURL'],
             ];
 
     $data['record'] = 'false';
@@ -206,16 +206,16 @@ 
 }
 
 function bigbluebutton_bbb_view_create_meeting_metadata(&$bbbsession) {
-      $metadata = ['bbb-origin' => $bbbsession['origin'],
-                   'bbb-origin-version' => $bbbsession['originVersion'],
-                   'bbb-origin-server-name' => $bbbsession['originServerName'],
-                   'bbb-origin-server-common-name' => $bbbsession['originServerCommonName'],
-                   'bbb-origin-tag' => $bbbsession['originTag'],
-                   'bbb-context' => $bbbsession['course']->fullname,
-                   'bbb-recording-name' => $bbbsession['meetingname'],
-                   'bbb-recording-description' => bigbluebuttonbn_html2text($bbbsession['meetingdescription'], 64),
-                   'bbb-recording-tags' => bigbluebuttonbn_get_tags($bbbsession['cm']->id), // Same as $id.
-                  ];
+        $metadata = ['bbb-origin' => $bbbsession['origin'],
+                    'bbb-origin-version' => $bbbsession['originVersion'],
+                    'bbb-origin-server-name' => $bbbsession['originServerName'],
+                    'bbb-origin-server-common-name' => $bbbsession['originServerCommonName'],
+                    'bbb-origin-tag' => $bbbsession['originTag'],
+                    'bbb-context' => $bbbsession['course']->fullname,
+                    'bbb-recording-name' => $bbbsession['meetingname'],
+                    'bbb-recording-description' => bigbluebuttonbn_html2text($bbbsession['meetingdescription'], 64),
+                    'bbb-recording-tags' => bigbluebuttonbn_get_tags($bbbsession['cm']->id), // Same as $id.
+                    ];
 
     if (bigbluebuttonbn_is_bn_server()) {
         if (bigbluebuttonbn_get_cfg_recordingready_enabled()) {

Switch Indentation +81 added lines, -81 removed lines

@@ -62,106 +62,106 @@
 }
 
 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;
-        }
-
-        // 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 'playback':
-        if ($href == '') {
+        case 'playback':
+            if ($href == '') {
+                bigbluebutton_bbb_view_close_window();
+                return;
+            }
+            // 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();
-            return;
-        }
-        // 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_close_window() {

Spacing +13 added lines, -13 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');
 
 $action = required_param('action', PARAM_TEXT);
 $id = optional_param('id', 0, PARAM_INT);
@@ -97,7 +97,7 @@ 
 
         // 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']);
+            header('Location: ' . $bbbsession['logoutURL']);
             break;
         }
 
@@ -113,17 +113,17 @@ 
             // The server is unreachable.
             if ($bbbsession['administrator']) {
                 print_error('view_error_unable_join', 'bigbluebuttonbn',
-                    $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+                    $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');
+                    $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
                 break;
             }
 
             print_error('view_error_unable_join_student', 'bigbluebuttonbn',
-                $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+                $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
             break;
         }
 
@@ -158,7 +158,7 @@ 
         // 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));
+        header('Location: ' . urldecode($href));
         break;
     default:
         bigbluebutton_bbb_view_close_window();
@@ -201,7 +201,7 @@ 
     }
     if ($durationtime > 0) {
         $data['duration'] = $durationtime;
-        $data['welcome'] .= '<br><br>'.str_replace('%duration%', ''.$durationtime, get_string('bbbdurationwarning', 'bigbluebuttonbn'));
+        $data['welcome'] .= '<br><br>' . str_replace('%duration%', '' . $durationtime, get_string('bbbdurationwarning', 'bigbluebuttonbn'));
     }
 
     $voicebridge = intval($bbbsession['voicebridge']);
@@ -247,7 +247,7 @@ 
 
     if ($bbbsession['userlimit'] > 0 && intval($meetinginfo['participantCount']) >= $bbbsession['userlimit']) {
         // No more users allowed to join.
-        header('Location: '.$bbbsession['logoutURL']);
+        header('Location: ' . $bbbsession['logoutURL']);
         return;
     }
 
@@ -266,20 +266,20 @@ 
     bigbluebuttonbn_participant_joined($bbbsession['meetingid'],
         ($bbbsession['administrator'] || $bbbsession['moderator']));
     // Execute the redirect.
-    header('Location: '.$joinurl);
+    header('Location: ' . $joinurl);
 }
 
 function bigbluebutton_bbb_view_errors($serrors, $id) {
     global $CFG, $OUTPUT;
 
-    $errors = (array) json_decode(urldecode($serrors));
+    $errors = (array)json_decode(urldecode($serrors));
     $msgerrors = '';
     foreach ($errors as $error) {
-        $msgerrors .= html_writer::tag('p', $error->{'message'}, array('class' => 'alert alert-danger'))."\n";
+        $msgerrors .= html_writer::tag('p', $error->{'message'}, array('class' => 'alert alert-danger')) . "\n";
     }
 
     echo $OUTPUT->header();
     print_error('view_error_bigbluebutton', 'bigbluebuttonbn',
-        $CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$id, $msgerrors, $serrors);
+        $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $id, $msgerrors, $serrors);
     echo $OUTPUT->footer();
 }