blindsidenetworks / moodle-mod_bigbluebuttonbn

classes/event/bigbluebuttonbn_meeting_created.php

Spacing +3 added lines, -3 removed lines

@@ -52,11 +52,11 @@ 
      * @return string
      */
     public function get_description() {
-        $a = (object) array('userid' => $this->userid,
+        $a = (object)array('userid' => $this->userid,
                             'bigbluebuttonbnid' => $this->objectid,
                             'courseid' => $this->contextinstanceid);
 
-        return "The user with id '$a->userid' created a bigbluebutton meeting for".
+        return "The user with id '$a->userid' created a bigbluebutton meeting for" .
         "the bigbluebuttonbn activity with id '$a->bigbluebuttonbnid' for the course id '$a->courseid'.";
     }
 
@@ -67,7 +67,7 @@ 
      */
     protected function get_legacy_logdata() {
         return array($this->courseid, 'bigbluebuttonbn', 'meeting created',
-                'view.php?pageid='.$this->objectid, 'BigBlueButtonBN meeting created',
+                'view.php?pageid=' . $this->objectid, 'BigBlueButtonBN meeting created',
                 $this->contextinstanceid);
     }
 

classes/event/bigbluebuttonbn_meeting_joined.php

Spacing +4 added lines, -4 removed lines

@@ -52,12 +52,12 @@ 
      * @return string
      */
     public function get_description() {
-        $a = (object) array('userid' => $this->userid,
+        $a = (object)array('userid' => $this->userid,
                             'bigbluebuttonbnid' => $this->objectid,
                             'courseid' => $this->contextinstanceid);
 
-        return "The user with id '$a->userid' has joined a bigbluebutton meeting for ".
-        "the bigbluebuttonbn activity with id '$a->bigbluebuttonbnid' for the course id ".
+        return "The user with id '$a->userid' has joined a bigbluebutton meeting for " .
+        "the bigbluebuttonbn activity with id '$a->bigbluebuttonbnid' for the course id " .
         "'$a->courseid'.";
     }
 
@@ -68,7 +68,7 @@ 
      */
     protected function get_legacy_logdata() {
         return array($this->courseid, 'bigbluebuttonbn', 'meeting joined',
-                'view.php?pageid='.$this->objectid, 'BigBlueButtonBN meeting joined',
+                'view.php?pageid=' . $this->objectid, 'BigBlueButtonBN meeting joined',
                 $this->contextinstanceid);
     }
 

backup/moodle2/backup_bigbluebuttonbn_activity_task.class.php

Spacing +4 added lines, -4 removed lines

@@ -23,7 +23,7 @@ 
 
 defined('MOODLE_INTERNAL') || die;
 
-require_once($CFG->dirroot.'/mod/bigbluebuttonbn/backup/moodle2/backup_bigbluebuttonbn_stepslib.php');
+require_once($CFG->dirroot . '/mod/bigbluebuttonbn/backup/moodle2/backup_bigbluebuttonbn_stepslib.php');
 
 /**
  * bigbluebuttonbn backup task that provides all the settings and steps to perform one
@@ -53,14 +53,14 @@ 
     public static function encode_content_links($content) {
         global $CFG;
 
-        $base = preg_quote($CFG->wwwroot.'/mod/bigbluebuttonbn', '#');
+        $base = preg_quote($CFG->wwwroot . '/mod/bigbluebuttonbn', '#');
 
         // Link to the list of bigbluebuttonbns.
-        $pattern = '#('.$base."\/index.php\?id\=)([0-9]+)#";
+        $pattern = '#(' . $base . "\/index.php\?id\=)([0-9]+)#";
         $content = preg_replace($pattern, '$@BIGBLUEBUTTONBNINDEX*$2@$', $content);
 
         // Link to bigbluebuttonbn view by moduleid.
-        $pattern = '#('.$base."\/view.php\?id\=)([0-9]+)#";
+        $pattern = '#(' . $base . "\/view.php\?id\=)([0-9]+)#";
         $content = preg_replace($pattern, '$@BIGBLUEBUTTONBNVIEWBYID*$2@$', $content);
 
         return $content;

backup/moodle2/restore_bigbluebuttonbn_stepslib.php

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
     protected function process_bigbluebuttonbn($data) {
         global $DB;
 
-        $data = (object) $data;
+        $data = (object)$data;
         $data->course = $this->get_courseid();
 
         $data->timemodified = $this->apply_date_offset($data->timemodified);
@@ -59,7 +59,7 @@ 
     protected function process_bigbluebuttonbn_logs($data) {
         global $DB;
 
-        $data = (object) $data;
+        $data = (object)$data;
         // Apply modifications.
         $data->courseid = $this->get_mappingid('course', $data->courseid);
         $data->bigbluebuttonbnid = $this->get_new_parentid('bigbluebuttonbn');

backup/moodle2/restore_bigbluebuttonbn_activity_task.class.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 
 defined('MOODLE_INTERNAL') || die();
 
-require_once($CFG->dirroot.'/mod/bigbluebuttonbn/backup/moodle2/restore_bigbluebuttonbn_stepslib.php');
+require_once($CFG->dirroot . '/mod/bigbluebuttonbn/backup/moodle2/restore_bigbluebuttonbn_stepslib.php');
 
 /**
  * bigbluebuttonbn restore task that provides all the settings and steps to perform one

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>'
                 ));
         }
     }

bbb_broker.php

Doc Comments +6 added lines

@@ -175,6 +175,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) {
@@ -195,6 +198,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'])) {

Indentation +3 added lines, -3 removed lines

@@ -554,11 +554,11 @@
         'recording_import' => ['id' => 'The recordingID must be specified.'],
         'recording_ready' => [
             'signed_parameters' => 'A JWT encoded string must be included as [signed_parameters].'
-          ],
+            ],
         'live_session_events' => [
             'signed_parameters' => 'A JWT encoded string must be included as [signed_parameters].'
-          ]
-      ];
+            ]
+        ];
 
     if (!isset($params['callback'])) {
         return 'This call must include a javascript callback.';

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;
     }
@@ -450,8 +450,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;
     }
 
@@ -469,8 +469,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);
     }
 }
 
@@ -485,7 +485,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;
     }
 
@@ -493,7 +493,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'])) {
@@ -510,8 +510,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;
     }
 
@@ -570,7 +570,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.';
     }
 
     foreach ($requiredparams[$action] as $param => $message) {

mod_form.php

Indentation +4 added lines, -4 removed lines

@@ -203,8 +203,8 @@ 
             $this->bigbluebuttonbn_mform_add_element($mform, $field['type'], $field['name'], $field['data_type'],
                 $field['description_key'], 0, ['maxlength' => 4, 'size' => 6],
                 ['message' => get_string('mod_form_field_voicebridge_format_error', 'bigbluebuttonbn'),
-                 'type' => 'numeric', 'rule' => '####', 'validator' => 'server']
-              );
+                    'type' => 'numeric', 'rule' => '####', 'validator' => 'server']
+                );
         } else {
             $this->bigbluebuttonbn_mform_add_element($mform, $field['type'], $field['name'], $field['data_type'],
                 $field['description_key'], 0, ['maxlength' => 4, 'size' => 6]);
@@ -246,7 +246,7 @@ 
             $field['description_key'], $cfg['recordings_html_default']);
 
         $field = ['type' => 'hidden', 'name' => 'recordings_deleted_activities', 'data_type' => PARAM_INT,
-                  'description_key' => null];
+                    'description_key' => null];
         if ($cfg['recordings_deleted_activities_editable']) {
             $field['type'] = 'checkbox';
             $field['description_key'] = 'mod_form_field_recordings_deleted_activities';
@@ -301,7 +301,7 @@ 
             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')).'  '.

Spacing +9 added lines, -9 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 {
 
@@ -38,7 +38,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');
         }
 
         $bigbluebuttonbn = null;
@@ -301,12 +301,12 @@ 
             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')).'  '.
-            '<input value="'.get_string('mod_form_field_participant_list_action_add', 'bigbluebuttonbn').
-            '" class="btn btn-secondary" type="button" id="id_addselectionid" '.
+                array('id' => 'bigbluebuttonbn_participant_selection', 'disabled' => 'disabled')) . '&nbsp;&nbsp;' .
+            '<input value="' . get_string('mod_form_field_participant_list_action_add', 'bigbluebuttonbn') .
+            '" class="btn btn-secondary" type="button" id="id_addselectionid" ' .
             'onclick="M.mod_bigbluebuttonbn.modform.participant_add(); return 0;" />'
         );
 
@@ -323,7 +323,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)
         );
 
@@ -356,7 +356,7 @@ 
         }
 
         $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)) {