jfederico / moodle-mod_bigbluebuttonbn

backup/moodle2/backup_bigbluebuttonbn_stepslib.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
         $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);

bbb_ajax.php

Spacing +1 added lines, -1 removed lines

@@ -126,7 +126,7 @@
         echo $completionvalidate;
         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');
 } catch (Exception $e) {
     header('HTTP/1.0 500 Internal Server Error. '.$e->getMessage());
 }

brokerlib.php

Spacing +6 added lines, -6 removed lines

@@ -626,7 +626,7 @@ 
         // Pull the Bearer from the headers.
         if (!array_key_exists('Authorization', $headers)) {
             $msg = 'Authorization failed';
-            header('HTTP/1.0 400 Bad Request. ' . $msg);
+            header('HTTP/1.0 400 Bad Request. '.$msg);
             return;
         }
         $authorization = explode(" ", $headers['Authorization']);
@@ -641,8 +641,8 @@ 
         // Convert JSON string to a JSON object.
         $jsonobj = json_decode($jsonstr);
     } catch (Exception $e) {
-        $msg = 'Caught exception: ' . $e->getMessage();
-        header('HTTP/1.0 400 Bad Request. ' . $msg);
+        $msg = 'Caught exception: '.$e->getMessage();
+        header('HTTP/1.0 400 Bad Request. '.$msg);
         return;
     }
 
@@ -651,7 +651,7 @@ 
     $meetingidelements = explode('-', $meetingidelements[0]);
     if (!isset($bigbluebuttonbn) || $bigbluebuttonbn->meetingid != $meetingidelements[0]) {
         $msg = 'The activity may have been deleted';
-        header('HTTP/1.0 410 Gone. ' . $msg);
+        header('HTTP/1.0 410 Gone. '.$msg);
         return;
     }
 
@@ -664,11 +664,11 @@ 
         // Process the events.
         bigbluebuttonbn_process_meeting_events($bigbluebuttonbn, $jsonobj);
         $msg = 'Enqueued.';
-        header('HTTP/1.0 202 Accepted. ' . $msg);
+        header('HTTP/1.0 202 Accepted. '.$msg);
         return;
     }
     $msg = 'Already processed.';
-    header('HTTP/1.0 202 Accepted. ' . $msg);
+    header('HTTP/1.0 202 Accepted. '.$msg);
 }
 
 /**

import_view.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
     print_error('view_error_invalid_session', plugin::COMPONENT);
 }
 
-if (!(boolean)\mod_bigbluebuttonbn\locallib\config::importrecordings_enabled()) {
+if (!(boolean) \mod_bigbluebuttonbn\locallib\config::importrecordings_enabled()) {
     print_error('view_message_importrecordings_disabled', plugin::COMPONENT);
 }
 

mod_form.php

Indentation +11 added lines, -11 removed lines

@@ -309,8 +309,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]);
@@ -380,7 +380,7 @@ 
     private function bigbluebuttonbn_mform_add_block_room_recordings(&$mform, $cfg) {
         $recordingsettings = false;
         $field = ['type' => 'hidden', 'name' => 'recordings_html', 'data_type' => PARAM_INT,
-                  'description_key' => null];
+                    'description_key' => null];
         if ($cfg['recordings_html_editable']) {
             $field['type'] = 'checkbox';
             $field['description_key'] = 'mod_form_field_recordings_html';
@@ -389,7 +389,7 @@ 
         $this->bigbluebuttonbn_mform_add_element($mform, $field['type'], $field['name'], $field['data_type'],
             $field['description_key'], $cfg['recordings_html_default']);
         $field = ['type' => 'hidden', 'name' => 'recordings_deleted', 'data_type' => PARAM_INT,
-                  'description_key' => null];
+                    'description_key' => null];
         if ($cfg['recordings_deleted_editable']) {
             $field['type'] = 'checkbox';
             $field['description_key'] = 'mod_form_field_recordings_deleted';
@@ -398,7 +398,7 @@ 
         $this->bigbluebuttonbn_mform_add_element($mform, $field['type'], $field['name'], $field['data_type'],
             $field['description_key'], $cfg['recordings_deleted_default']);
         $field = ['type' => 'hidden', 'name' => 'recordings_imported', 'data_type' => PARAM_INT,
-                  'description_key' => null];
+                    'description_key' => null];
         if ($cfg['importrecordings_enabled'] && $cfg['recordings_imported_editable']) {
             $field['type'] = 'checkbox';
             $field['description_key'] = 'mod_form_field_recordings_imported';
@@ -407,7 +407,7 @@ 
         $this->bigbluebuttonbn_mform_add_element($mform, $field['type'], $field['name'], $field['data_type'],
             $field['description_key'], $cfg['recordings_imported_default']);
         $field = ['type' => 'hidden', 'name' => 'recordings_preview', 'data_type' => PARAM_INT,
-                  'description_key' => null];
+                    'description_key' => null];
         if ($cfg['recordings_preview_editable']) {
             $field['type'] = 'checkbox';
             $field['description_key'] = 'mod_form_field_recordings_preview';
@@ -484,7 +484,7 @@ 
         $htmlselectiontype = 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.participantSelectionSet(); return 0;'));
+                    'onchange' => 'M.mod_bigbluebuttonbn.modform.participantSelectionSet(); return 0;'));
         $htmlselectionoptions = html_writer::select($participantselection['options'], 'bigbluebuttonbn_participant_selection',
             $participantselection['selected'], array(),
             array('id' => 'bigbluebuttonbn_participant_selection', 'disabled' => 'disabled'));
@@ -492,7 +492,7 @@ 
             'type' => 'button', 'class' => 'btn btn-secondary',
             'value' => get_string('mod_form_field_participant_list_action_add', 'bigbluebuttonbn'),
             'onclick' => 'M.mod_bigbluebuttonbn.modform.participantAdd(); return 0;'
-          ));
+            ));
         $htmladdparticipant = html_writer::tag('div',
             $htmlselectiontype . '  ' . $htmlselectionoptions . '  ' . $htmlselectioninput, null);
         $mform->addElement('html', "\n\n");
@@ -534,9 +534,9 @@ 
             $field['type'] = 'select';
             $field['data_type'] = PARAM_TEXT;
             $field['description_key'] = 'mod_form_field_block_clienttype';
-             $choices = array(BIGBLUEBUTTON_CLIENTTYPE_FLASH => get_string('mod_form_block_clienttype_flash', 'bigbluebuttonbn'),
-                             BIGBLUEBUTTON_CLIENTTYPE_HTML5 => get_string('mod_form_block_clienttype_html5', 'bigbluebuttonbn'));
-             $mform->addElement('header', 'clienttypeselection', get_string('mod_form_block_clienttype', 'bigbluebuttonbn'));
+                $choices = array(BIGBLUEBUTTON_CLIENTTYPE_FLASH => get_string('mod_form_block_clienttype_flash', 'bigbluebuttonbn'),
+                                BIGBLUEBUTTON_CLIENTTYPE_HTML5 => get_string('mod_form_block_clienttype_html5', 'bigbluebuttonbn'));
+                $mform->addElement('header', 'clienttypeselection', get_string('mod_form_block_clienttype', 'bigbluebuttonbn'));
             $this->bigbluebuttonbn_mform_add_element($mform, $field['type'], $field['name'], $field['data_type'],
                                     $field['description_key'], $cfg['clienttype_default'], $choices);
             return;

Spacing +11 added lines, -11 removed lines

@@ -99,8 +99,8 @@ 
         $PAGE->requires->strings_for_js(array_keys(bigbluebuttonbn_get_strings_for_js()), 'bigbluebuttonbn');
         $jsvars['participantData'] = bigbluebuttonbn_get_participant_data($context, $bigbluebuttonbn);
         $jsvars['participantList'] = $participantlist;
-        $jsvars['iconsEnabled'] = (boolean)$cfg['recording_icons_enabled'];
-        $jsvars['pixIconDelete'] = (string)$OUTPUT->pix_icon('t/delete', get_string('delete'), 'moodle');
+        $jsvars['iconsEnabled'] = (boolean) $cfg['recording_icons_enabled'];
+        $jsvars['pixIconDelete'] = (string) $OUTPUT->pix_icon('t/delete', get_string('delete'), 'moodle');
         $PAGE->requires->yui_module('moodle-mod_bigbluebuttonbn-modform',
             'M.mod_bigbluebuttonbn.modform.init', array($jsvars));
     }
@@ -171,7 +171,7 @@ 
      */
     public function add_completion_rules() {
         $mform = $this->_form;
-        if (!bigbluebuttonbn_is_bn_server() || !(boolean)\mod_bigbluebuttonbn\locallib\config::get('meetingevents_enabled')) {
+        if (!bigbluebuttonbn_is_bn_server() || !(boolean) \mod_bigbluebuttonbn\locallib\config::get('meetingevents_enabled')) {
             return [];
         }
 
@@ -179,7 +179,7 @@ 
         $attendance['grouplabel'] = get_string('completionattendancegroup', 'bigbluebuttonbn');
         $attendance['rulelabel'] = get_string('completionattendance', 'bigbluebuttonbn');
         $attendance['group'] = [
-            $mform->createElement('checkbox', 'completionattendanceenabled', '', $attendance['rulelabel'] . ' '),
+            $mform->createElement('checkbox', 'completionattendanceenabled', '', $attendance['rulelabel'].' '),
             $mform->createElement('text', 'completionattendance', '', ['size' => 3]),
             $mform->createElement('static', 'completionattendanceunit', ' ', get_string('minutes', 'bigbluebuttonbn'))
         ];
@@ -197,11 +197,11 @@ 
         $engagement['pollvotes'] = get_string('completionengagementpollvotes', 'bigbluebuttonbn');
         $engagement['emojis'] = get_string('completionengagementemojis', 'bigbluebuttonbn');
         $engagement['group'] = [
-            $mform->createElement('checkbox', 'completionengagementchats', '', $engagement['chatlabel'] . '  '),
-            $mform->createElement('checkbox', 'completionengagementtalks', '', $engagement['talklabel'] . '  '),
-            $mform->createElement('checkbox', 'completionengagementraisehand', '', $engagement['raisehand'] . '  '),
-            $mform->createElement('checkbox', 'completionengagementpollvotes', '', $engagement['pollvotes'] . '  '),
-            $mform->createElement('checkbox', 'completionengagementemojis', '', $engagement['emojis'] . '  '),
+            $mform->createElement('checkbox', 'completionengagementchats', '', $engagement['chatlabel'].'  '),
+            $mform->createElement('checkbox', 'completionengagementtalks', '', $engagement['talklabel'].'  '),
+            $mform->createElement('checkbox', 'completionengagementraisehand', '', $engagement['raisehand'].'  '),
+            $mform->createElement('checkbox', 'completionengagementpollvotes', '', $engagement['pollvotes'].'  '),
+            $mform->createElement('checkbox', 'completionengagementemojis', '', $engagement['emojis'].'  '),
         ];
         $mform->addGroup($engagement['group'], 'completionengagementgroup', $engagement['grouplabel'], [' '], false);
         $mform->addHelpButton('completionengagementgroup', 'completionengagementgroup', 'bigbluebuttonbn');
@@ -248,7 +248,7 @@ 
      * @return void
      */
     private function bigbluebuttonbn_mform_add_block_profiles(&$mform, $profiles) {
-        if ((boolean)\mod_bigbluebuttonbn\locallib\config::recordings_enabled()) {
+        if ((boolean) \mod_bigbluebuttonbn\locallib\config::recordings_enabled()) {
             $mform->addElement('select', 'type', get_string('mod_form_field_instanceprofiles', 'bigbluebuttonbn'),
                 bigbluebuttonbn_get_instance_profiles_array($profiles),
                 array('onchange' => 'M.mod_bigbluebuttonbn.modform.updateInstanceTypeProfile(this);'));
@@ -487,7 +487,7 @@ 
             'onclick' => 'M.mod_bigbluebuttonbn.modform.participantAdd(); return 0;'
           ));
         $htmladdparticipant = html_writer::tag('div',
-            $htmlselectiontype . '  ' . $htmlselectionoptions . '  ' . $htmlselectioninput, null);
+            $htmlselectiontype.'  '.$htmlselectionoptions.'  '.$htmlselectioninput, null);
         $mform->addElement('html', "\n\n");
         $mform->addElement('static', 'static_add_participant',
             get_string('mod_form_field_participant_add', 'bigbluebuttonbn'), $htmladdparticipant);

view.php

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
 bigbluebuttonbn_view_bbbsession_set($PAGE->context, $bbbsession);
 
 // Validates if the BigBlueButton server is working.
-$serverversion = bigbluebuttonbn_get_server_version();  // In locallib.
+$serverversion = bigbluebuttonbn_get_server_version(); // In locallib.
 if ($serverversion === null) {
     $errmsg = 'view_error_unable_join_student';
     $errurl = '/course/view.php';

db/mobile.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
             'coursebigbluebuttonbn' => array( // Handler unique name (can be anything).
                 'displaydata' => array(
                     'title' => 'pluginname',
-                    'icon' => $CFG->wwwroot . '/mod/bigbluebuttonbn/pix/icon.gif',
+                    'icon' => $CFG->wwwroot.'/mod/bigbluebuttonbn/pix/icon.gif',
                     'class' => '',
                 ),
                 'delegate' => 'CoreCourseModuleDelegate', // Delegate (where to display the link to the add-on).

bbb_broker.php

Spacing +3 added lines, -3 removed lines

@@ -43,8 +43,8 @@ 
 // The endpoints for recording_ready and meeting_events callbacks must be moved to services (CONTRIB-7440).
 // But in order to support the transition, requests other than the callbacks are redirected to bbb_ajax.php.
 if ($params['action'] != 'recording_ready' && $params['action'] != 'meeting_events') {
-    $url = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_ajax.php?' . http_build_query($params, '', '&');
-    header("Location: " . $url);
+    $url = $CFG->wwwroot.'/mod/bigbluebuttonbn/bbb_ajax.php?'.http_build_query($params, '', '&');
+    header("Location: ".$url);
     exit;
 }
 
@@ -71,7 +71,7 @@ 
         bigbluebuttonbn_broker_meeting_events($bigbluebuttonbn);
         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');
 } catch (Exception $e) {
     header('HTTP/1.0 500 Internal Server Error. '.$e->getMessage());
 }

bbb_view.php

Indentation +5 added lines, -5 removed lines

@@ -309,10 +309,10 @@ 
  */
 function bigbluebuttonbn_bbb_view_create_meeting_data(&$bbbsession) {
     $data = ['meetingID' => $bbbsession['meetingid'],
-              'name' => bigbluebuttonbn_html2text($bbbsession['meetingname'], 64),
-              'attendeePW' => $bbbsession['viewerPW'],
-              'moderatorPW' => $bbbsession['modPW'],
-              'logoutURL' => $bbbsession['logoutURL'],
+                'name' => bigbluebuttonbn_html2text($bbbsession['meetingname'], 64),
+                'attendeePW' => $bbbsession['viewerPW'],
+                'moderatorPW' => $bbbsession['modPW'],
+                'logoutURL' => $bbbsession['logoutURL'],
             ];
     $data['record'] = bigbluebuttonbn_bbb_view_create_meeting_data_record($bbbsession['record']);
     // Check if auto_start_record is enable.
@@ -334,7 +334,7 @@ 
             '%duration%',
             (string) $durationtime,
             get_string('bbbdurationwarning', 'bigbluebuttonbn')
-          );
+            );
     }
     $voicebridge = intval($bbbsession['voicebridge']);
     if ($voicebridge > 0 && $voicebridge < 79999) {

Spacing +12 added lines, -12 removed lines

@@ -85,15 +85,15 @@ 
     $bbbsession['serverversion'] = (string) $serverversion;
 
     // 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 .
-        '&bn=' . $bbbsession['bigbluebuttonbn']->id;
-    $bbbsession['recordingReadyURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_broker.php?action=recording_' .
-        'ready&bigbluebuttonbn=' . $bbbsession['bigbluebuttonbn']->id;
-    $bbbsession['meetingEventsURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_broker.php?action=meeting' .
-        '_events&bigbluebuttonbn=' . $bbbsession['bigbluebuttonbn']->id;
-    $bbbsession['joinURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_view.php?action=join&id=' . $cm->id .
-        '&bn=' . $bbbsession['bigbluebuttonbn']->id;
+    $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.
+        '&bn='.$bbbsession['bigbluebuttonbn']->id;
+    $bbbsession['recordingReadyURL'] = $CFG->wwwroot.'/mod/bigbluebuttonbn/bbb_broker.php?action=recording_'.
+        'ready&bigbluebuttonbn='.$bbbsession['bigbluebuttonbn']->id;
+    $bbbsession['meetingEventsURL'] = $CFG->wwwroot.'/mod/bigbluebuttonbn/bbb_broker.php?action=meeting'.
+        '_events&bigbluebuttonbn='.$bbbsession['bigbluebuttonbn']->id;
+    $bbbsession['joinURL'] = $CFG->wwwroot.'/mod/bigbluebuttonbn/bbb_view.php?action=join&id='.$cm->id.
+        '&bn='.$bbbsession['bigbluebuttonbn']->id;
 
     // Check status and set extra values.
     $activitystatus = bigbluebuttonbn_view_get_activity_status($bbbsession);
@@ -155,7 +155,7 @@ 
         $lastaccess = json_decode($lastaccess->meta);
         // If the user acceded from Timeline it should be redirected to the Dashboard.
         if (isset($lastaccess->origin) && $lastaccess->origin == BIGBLUEBUTTON_ORIGIN_TIMELINE) {
-            redirect($CFG->wwwroot . '/my/');
+            redirect($CFG->wwwroot.'/my/');
         }
         // Close the tab or window where BBB was opened.
         bigbluebuttonbn_bbb_view_close_window();
@@ -357,7 +357,7 @@ 
  * @return string
  */
 function bigbluebuttonbn_bbb_view_create_meeting_data_record($record) {
-    if ((boolean)\mod_bigbluebuttonbn\locallib\config::recordings_enabled() && $record) {
+    if ((boolean) \mod_bigbluebuttonbn\locallib\config::recordings_enabled() && $record) {
         return 'true';
     }
     return 'false';
@@ -370,7 +370,7 @@ 
  * @return integer
  */
 function bigbluebuttonbn_bbb_view_create_meeting_data_duration($closingtime) {
-    if ((boolean)\mod_bigbluebuttonbn\locallib\config::get('scheduled_duration_enabled')) {
+    if ((boolean) \mod_bigbluebuttonbn\locallib\config::get('scheduled_duration_enabled')) {
         return bigbluebuttonbn_get_duration($closingtime);
     }
     return 0;