blindsidenetworks / moodle-mod_bigbluebuttonbn

view.php

Doc Comments +7 added lines

@@ -276,6 +276,9 @@ 
     $PAGE->requires->js_init_call('M.mod_bigbluebuttonbn.view_init', array(), false, $jsmodule);
 }
 
+/**
+ * @param boolean $showrecordings
+ */
 function bigbluebuttonbn_view_show_rooms($bbbsession, $activity, $showrecordings, &$jsvars)
 {
     global $OUTPUT;
@@ -321,6 +324,10 @@ 
     return $output;
 }
 
+/**
+ * @param boolean $showroom
+ * @param string[] $jsdependences
+ */
 function bigbluebuttonbn_view_show_recordings($bbbsession, $showroom, &$jsvars, &$jsdependences)
 {
 

Indentation +5 added lines, -5 removed lines

@@ -326,9 +326,9 @@ 
 
     // Get recordings
     $recordings = bigbluebuttonbn_get_recordings($bbbsession['course']->id,
-                                                 $showroom ? $bbbsession['bigbluebuttonbn']->id : null,
-                                                 $showroom,
-                                                 $bbbsession['bigbluebuttonbn']->recordings_deleted_activities);
+                                                    $showroom ? $bbbsession['bigbluebuttonbn']->id : null,
+                                                    $showroom,
+                                                    $bbbsession['bigbluebuttonbn']->recordings_deleted_activities);
 
     if (!isset($recordings) || empty($recordings) || array_key_exists('messageKey', $recordings)) {
         // There are no recordings to be shown.
@@ -339,7 +339,7 @@ 
     // JavaScript variables for recordings
     $jsvars += array(
             'recordings_html' => $bbbsession['bigbluebuttonbn']->recordings_html == '1',
-          );
+            );
 
     // If there are meetings with recordings load the data to the table
     if ($bbbsession['bigbluebuttonbn']->recordings_html) {
@@ -351,7 +351,7 @@ 
     $jsvars += array(
             'columns' => bigbluebuttonbn_get_recording_columns($bbbsession),
             'data' => bigbluebuttonbn_get_recording_data($bbbsession, $recordings),
-          );
+            );
 
     // JavaScript dependences for recordings with YUI
     $jsdependences += array('datatable', 'datatable-sort', 'datatable-paginator', 'datatype-number');

Spacing +54 added lines, -54 removed lines

@@ -22,8 +22,8 @@ 
  * @copyright 2010-2015 Blindside Networks Inc
  * @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); // Course Module ID, or
 $b = optional_param('n', 0, PARAM_INT); // bigbluebuttonbn instance ID
@@ -88,7 +88,7 @@ 
 $bbbsession['wait'] = $bigbluebuttonbn->wait;
 $bbbsession['record'] = $bigbluebuttonbn->record;
 if ($bigbluebuttonbn->record) {
-    $bbbsession['welcome'] .= '<br><br>'.get_string('bbbrecordwarning', 'bigbluebuttonbn');
+    $bbbsession['welcome'] .= '<br><br>' . get_string('bbbrecordwarning', 'bigbluebuttonbn');
 }
 $bbbsession['tagging'] = $bigbluebuttonbn->tagging;
 
@@ -108,7 +108,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') . ')';
 ////////////////////////////////////////////////
 /////   BigBlueButton Session Setup Ends   /////
 ////////////////////////////////////////////////
@@ -117,11 +117,11 @@ 
 $serverVersion = bigbluebuttonbn_getServerVersion($bbbsession['endpoint']);
 if (!isset($serverVersion)) { //Server is not working
     if ($bbbsession['administrator']) {
-        print_error('view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+        print_error('view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
     } elseif ($bbbsession['moderator']) {
-        print_error('view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course);
+        print_error('view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot . '/course/view.php?id=' . $bigbluebuttonbn->course);
     } else {
-        print_error('view_error_unable_join_student', 'bigbluebuttonbn', $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course);
+        print_error('view_error_unable_join_student', 'bigbluebuttonbn', $CFG->wwwroot . '/course/view.php?id=' . $bigbluebuttonbn->course);
     }
 }
 
@@ -131,7 +131,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);
@@ -141,11 +141,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();
@@ -153,29 +153,29 @@ 
 }
 
 // Operation URLs
-$bbbsession['courseURL'] = $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course;
-$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='.$id.'&bigbluebuttonbn='.$bbbsession['bigbluebuttonbn']->id;
+$bbbsession['courseURL'] = $CFG->wwwroot . '/course/view.php?id=' . $bigbluebuttonbn->course;
+$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=' . $id . '&bigbluebuttonbn=' . $bbbsession['bigbluebuttonbn']->id;
 
 // Output starts here
 echo $OUTPUT->header();
 /// Shows version as a comment
 echo '
-<!-- '.$bbbsession['originTag'].' -->'."\n";
+<!-- '.$bbbsession['originTag'] . ' -->' . "\n";
 
 /// find out current groups mode
 $groupmode = groups_get_activity_groupmode($bbbsession['cm']);
 if ($groupmode == NOGROUPS) {  //No groups mode
-    $bbbsession['meetingid'] = $bbbsession['bigbluebuttonbn']->meetingid.'-'.$bbbsession['course']->id.'-'.$bbbsession['bigbluebuttonbn']->id;
+    $bbbsession['meetingid'] = $bbbsession['bigbluebuttonbn']->meetingid . '-' . $bbbsession['course']->id . '-' . $bbbsession['bigbluebuttonbn']->id;
     $bbbsession['meetingname'] = $bbbsession['bigbluebuttonbn']->name;
 } else {                                        // Separate or visible groups mode
     echo $OUTPUT->box_start('generalbox boxaligncenter');
-    echo '<br><div class="alert alert-warning">'.get_string('view_groups_selection_warning', 'bigbluebuttonbn').'</div>';
+    echo '<br><div class="alert alert-warning">' . get_string('view_groups_selection_warning', 'bigbluebuttonbn') . '</div>';
     echo $OUTPUT->box_end();
 
-    groups_print_activity_menu($cm, $CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$bbbsession['cm']->id);
+    groups_print_activity_menu($cm, $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $bbbsession['cm']->id);
     if ($groupmode == SEPARATEGROUPS) {
         $groups = groups_get_activity_allowed_groups($bbbsession['cm']);
         $current_group = current($groups);
@@ -185,13 +185,13 @@ 
         $bbbsession['group'] = groups_get_activity_group($bbbsession['cm'], true);
     }
 
-    $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'] . ']';
     if ($bbbsession['group'] > 0) {
         $group_name = groups_get_group_name($bbbsession['group']);
     } else {
         $group_name = get_string('allparticipants');
     }
-    $bbbsession['meetingname'] = $bbbsession['bigbluebuttonbn']->name.' ('.$group_name.')';
+    $bbbsession['meetingname'] = $bbbsession['bigbluebuttonbn']->name . ' (' . $group_name . ')';
 }
 // Metadata (context)
 $bbbsession['contextActivityName'] = $bbbsession['meetingname'];
@@ -260,7 +260,7 @@ 
         }
     }
 
-    $output .= html_writer::empty_tag('br').html_writer::empty_tag('br').html_writer::empty_tag('br');
+    $output .= html_writer::empty_tag('br') . html_writer::empty_tag('br') . html_writer::empty_tag('br');
 
     echo $output;
 
@@ -285,11 +285,11 @@ 
     //JavaScript variables for room
     $openingtime = '';
     if ($bbbsession['openingtime']) {
-        $openingtime = get_string('mod_form_field_openingtime', 'bigbluebuttonbn').': '.userdate($bbbsession['openingtime']);
+        $openingtime = get_string('mod_form_field_openingtime', 'bigbluebuttonbn') . ': ' . userdate($bbbsession['openingtime']);
     }
     $closingtime = '';
     if ($bbbsession['closingtime']) {
-        $closingtime = get_string('mod_form_field_closingtime', 'bigbluebuttonbn').': '.userdate($bbbsession['closingtime']);
+        $closingtime = get_string('mod_form_field_closingtime', 'bigbluebuttonbn') . ': ' . userdate($bbbsession['closingtime']);
     }
     $jsvars += array(
         'meetingid' => $bbbsession['meetingid'],
@@ -305,7 +305,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') {
@@ -344,7 +344,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
@@ -366,7 +366,7 @@ 
 
     $output = '';
 
-    $btn_import_links = html_writer::tag('input', '', array('type' => 'button', 'value' => get_string('view_recording_button_import', 'bigbluebuttonbn'), 'class' => 'btn btn-secondary', 'onclick' => 'window.location=\''.$CFG->wwwroot.'/mod/bigbluebuttonbn/import_view.php?bn='.$bbbsession['bigbluebuttonbn']->id.'\''));
+    $btn_import_links = html_writer::tag('input', '', array('type' => 'button', 'value' => get_string('view_recording_button_import', 'bigbluebuttonbn'), 'class' => 'btn btn-secondary', 'onclick' => 'window.location=\'' . $CFG->wwwroot . '/mod/bigbluebuttonbn/import_view.php?bn=' . $bbbsession['bigbluebuttonbn']->id . '\''));
     $output .= html_writer::start_tag('br');
     $output .= html_writer::tag('span', $btn_import_links, array('id' => 'import_recording_links_button'));
     $output .= html_writer::tag('span', '', array('id' => 'import_recording_links_table'));
@@ -377,28 +377,28 @@ 
 function bigbluebuttonbn_view_joining($bbbsession)
 {
     if ($bbbsession['tagging'] && ($bbbsession['administrator'] || $bbbsession['moderator'])) {
-        return ''.
-            '<div id="panelContent" class="hidden">'."\n".
-            '  <div class="yui3-widget-bd">'."\n".
-            '    <form>'."\n".
-            '      <fieldset>'."\n".
-            '        <input type="hidden" name="join" id="meeting_join_url" value="">'."\n".
-            '        <input type="hidden" name="message" id="meeting_message" value="">'."\n".
-            '        <div>'."\n".
-            '          <label for="name">'.get_string('view_recording_name', 'bigbluebuttonbn').'</label><br/>'."\n".
-            '          <input type="text" name="name" id="recording_name" placeholder="">'."\n".
-            '        </div><br>'."\n".
-            '        <div>'."\n".
-            '          <label for="description">'.get_string('view_recording_description', 'bigbluebuttonbn').'</label><br/>'."\n".
-            '          <input type="text" name="description" id="recording_description" value="" placeholder="">'."\n".
-            '        </div><br>'."\n".
-            '        <div>'."\n".
-            '          <label for="tags">'.get_string('view_recording_tags', 'bigbluebuttonbn').'</label><br/>'."\n".
-            '          <input type="text" name="tags" id="recording_tags" value="" placeholder="">'."\n".
-            '        </div>'."\n".
-            '      </fieldset>'."\n".
-            '    </form>'."\n".
-            '  </div>'."\n".
+        return '' .
+            '<div id="panelContent" class="hidden">' . "\n" .
+            '  <div class="yui3-widget-bd">' . "\n" .
+            '    <form>' . "\n" .
+            '      <fieldset>' . "\n" .
+            '        <input type="hidden" name="join" id="meeting_join_url" value="">' . "\n" .
+            '        <input type="hidden" name="message" id="meeting_message" value="">' . "\n" .
+            '        <div>' . "\n" .
+            '          <label for="name">' . get_string('view_recording_name', 'bigbluebuttonbn') . '</label><br/>' . "\n" .
+            '          <input type="text" name="name" id="recording_name" placeholder="">' . "\n" .
+            '        </div><br>' . "\n" .
+            '        <div>' . "\n" .
+            '          <label for="description">' . get_string('view_recording_description', 'bigbluebuttonbn') . '</label><br/>' . "\n" .
+            '          <input type="text" name="description" id="recording_description" value="" placeholder="">' . "\n" .
+            '        </div><br>' . "\n" .
+            '        <div>' . "\n" .
+            '          <label for="tags">' . get_string('view_recording_tags', 'bigbluebuttonbn') . '</label><br/>' . "\n" .
+            '          <input type="text" name="tags" id="recording_tags" value="" placeholder="">' . "\n" .
+            '        </div>' . "\n" .
+            '      </fieldset>' . "\n" .
+            '    </form>' . "\n" .
+            '  </div>' . "\n" .
             '</div>';
     }
 
@@ -413,9 +413,9 @@ 
         $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 '';

bbb_broker.php

Spacing +26 added lines, -26 removed lines

@@ -21,8 +21,8 @@ 
  * @copyright 2015 Blindside Networks Inc
  * @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;
 
@@ -101,7 +101,7 @@ 
                         if ($bbbsession['administrator'] || $bbbsession['moderator']) {
                             $end_button_text = get_string('view_conference_action_end', 'bigbluebuttonbn');
                             $can_end = true;
-                            $status_can_end = '"can_end": true, "end_button_text": "'.$end_button_text.'", ';
+                            $status_can_end = '"can_end": true, "end_button_text": "' . $end_button_text . '", ';
                         }
                     } else {
                         // If user is administrator, moderator or if is viewer and no waiting is required
@@ -112,7 +112,7 @@ 
                         } else {
                             $initial_message = get_string('view_message_conference_not_started', 'bigbluebuttonbn');
                             if ($bbbsession['wait']) {
-                                $initial_message .= ' '.get_string('view_message_conference_wait_for_moderator', 'bigbluebuttonbn');
+                                $initial_message .= ' ' . get_string('view_message_conference_wait_for_moderator', 'bigbluebuttonbn');
                             }
                             $can_join = false;
                         }
@@ -122,10 +122,10 @@ 
                         } else {
                             $can_tag = false;
                         }
-                        $status_can_end = '"can_tag": '.($can_tag ? 'true' : 'false').', ';
+                        $status_can_end = '"can_tag": ' . ($can_tag ? 'true' : 'false') . ', ';
                     }
 
-                    echo $params['callback'].'({ "running": '.($meeting_running ? 'true' : 'false').', "info": '.json_encode($meeting_info).', "status": {"can_join": '.($can_join ? 'true' : 'false').',"join_url": "'.$bbbsession['joinURL'].'","join_button_text": "'.$join_button_text.'", '.$status_can_end.$status_can_tag.'"message": "'.$initial_message.'"} });';
+                    echo $params['callback'] . '({ "running": ' . ($meeting_running ? 'true' : 'false') . ', "info": ' . json_encode($meeting_info) . ', "status": {"can_join": ' . ($can_join ? 'true' : 'false') . ',"join_url": "' . $bbbsession['joinURL'] . '","join_button_text": "' . $join_button_text . '", ' . $status_can_end . $status_can_tag . '"message": "' . $initial_message . '"} });';
                     break;
                 case 'meeting_end':
                     if ($bbbsession['administrator'] || $bbbsession['moderator']) {
@@ -138,7 +138,7 @@ 
                         // Update the cache
                         bigbluebuttonbn_get_meeting_info($params['id'], BIGBLUEBUTTONBN_FORCED);
 
-                        echo $params['callback'].'({ "status": true });';
+                        echo $params['callback'] . '({ "status": true });';
                     } else {
                         error_log('ERROR: User not authorized to execute end command');
                         header('HTTP/1.0 401 Unauthorized. User not authorized to execute end command');
@@ -148,9 +148,9 @@ 
                     if ($bbbsession['managerecordings']) {
                         if (isset($params['id']) && $params['id'] != '') {
                             $recordings_imported_all = bigbluebuttonbn_get_recording_imported_instances($params['id']);
-                            echo $params['callback'].'({ "status": "true", "links": '.count($recordings_imported_all).'});';
+                            echo $params['callback'] . '({ "status": "true", "links": ' . count($recordings_imported_all) . '});';
                         } else {
-                            echo $params['callback'].'({"status": "false"});';
+                            echo $params['callback'] . '({"status": "false"});';
                         }
                     } else {
                         error_log('ERROR: User not authorized to execute end command');
@@ -165,17 +165,17 @@ 
                             // Look up for an update on the imported recording
                             $recording = $recordings[$params['id']];
                             if (isset($recording) && !empty($recording) && !array_key_exists('messageKey', $recording)) {  // The recording was found
-                                echo $params['callback'].'({ "status": "true", "published": "'.$recording['published'].'"});';
+                                echo $params['callback'] . '({ "status": "true", "published": "' . $recording['published'] . '"});';
                             } else {
-                                echo $params['callback'].'({ "status": "false" });';
+                                echo $params['callback'] . '({ "status": "false" });';
                             }
                         // As the recordingid was not identified as imported recording link, look up for a real recording
                         } else {
                             $recording = bigbluebuttonbn_getRecordingsArray($params['idx'], $params['id']);
                             if (isset($recording) && !empty($recording) && array_key_exists($params['id'], $recording)) {  // The recording was found
-                                echo $params['callback'].'({ "status": "true", "published": "'.$recording[$params['id']]['published'].'"});';
+                                echo $params['callback'] . '({ "status": "true", "published": "' . $recording[$params['id']]['published'] . '"});';
                             } else {
-                                echo $params['callback'].'({"status": "false"});';
+                                echo $params['callback'] . '({"status": "false"});';
                             }
                         }
                     } else {
@@ -287,9 +287,9 @@ 
                     try {
                         $decodedParameters = JWT::decode($params['signed_parameters'], $shared_secret, array('HS256'));
                     } catch (Exception $e) {
-                        $error = 'Caught exception: '.$e->getMessage();
+                        $error = 'Caught exception: ' . $e->getMessage();
                         error_log($error);
-                        header('HTTP/1.0 400 Bad Request. '.$error);
+                        header('HTTP/1.0 400 Bad Request. ' . $error);
 
                         return;
                     }
@@ -305,16 +305,16 @@ 
 
                             return;
                         } catch (Exception $e) {
-                            $error = 'Caught exception: '.$e->getMessage();
+                            $error = 'Caught exception: ' . $e->getMessage();
                             error_log($error);
-                            header('HTTP/1.0 503 Service Unavailable. '.$error);
+                            header('HTTP/1.0 503 Service Unavailable. ' . $error);
 
                             return;
                         }
                     } else {
-                        $error = 'Caught exception: '.$e->getMessage();
+                        $error = 'Caught exception: ' . $e->getMessage();
                         error_log($error);
-                        header('HTTP/1.0 410 Gone. '.$error);
+                        header('HTTP/1.0 410 Gone. ' . $error);
 
                         return;
                     }
@@ -326,7 +326,7 @@ 
                         if (isset($importrecordings[$params['id']])) {
                             $importrecordings[$params['id']]['imported'] = true;
                             $overrides['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($bigbluebuttonbn)) {
@@ -339,7 +339,7 @@ 
                         } else {
                             $error = "Recording {$params['id']} could not be found. It can not be imported";
                             error_log($error);
-                            header('HTTP/1.0 404 Not found. '.$error);
+                            header('HTTP/1.0 404 Not found. ' . $error);
 
                             return;
                         }
@@ -350,9 +350,9 @@ 
                     try {
                         $decodedParameters = JWT::decode($params['signed_parameters'], $shared_secret, array('HS256'));
                     } catch (Exception $e) {
-                        $error = 'Caught exception: '.$e->getMessage();
+                        $error = 'Caught exception: ' . $e->getMessage();
                         error_log($error);
-                        header('HTTP/1.0 400 Bad Request. '.$error);
+                        header('HTTP/1.0 400 Bad Request. ' . $error);
 
                         return;
                     }
@@ -390,15 +390,15 @@ 
                     break;
             }
         } catch (Exception $e) {
-            error_log('BBB_BROKER ERROR: '.$e->getCode().', '.$e->getMessage());
-            header('HTTP/1.0 500 Internal Server Error. '.$e->getMessage());
+            error_log('BBB_BROKER ERROR: ' . $e->getCode() . ', ' . $e->getMessage());
+            header('HTTP/1.0 500 Internal Server Error. ' . $e->getMessage());
 
             return;
         }
     }
 } else {
     error_log(json_encode($error));
-    header('HTTP/1.0 400 Bad Request. '.$error);
+    header('HTTP/1.0 400 Bad Request. ' . $error);
 
     return;
 }

lib.php

Indentation +8 added lines, -8 removed lines

@@ -225,9 +225,9 @@ 
     global $DB;
 
     $completed = $DB->count_records('bigbluebuttonbn_logs', array('courseid' => $course->id,
-                                                              'bigbluebuttonbnid' => $bigbluebuttonbn->id,
-                                                              'userid' => $user->id,
-                                                              'log' => 'Join', ), '*');
+                                                                'bigbluebuttonbnid' => $bigbluebuttonbn->id,
+                                                                'userid' => $user->id,
+                                                                'log' => 'Join', ), '*');
 
     if ($completed > 0) {
         return fullname($user).' '.get_string('view_message_has_joined', 'bigbluebuttonbn').' '.get_string('view_message_session_for', 'bigbluebuttonbn').' '.(string) $completed.' '.get_string('view_message_times', 'bigbluebuttonbn');
@@ -246,9 +246,9 @@ 
 {
     global $DB;
     $completed = $DB->count_recorda('bigbluebuttonbn_logs', array('courseid' => $course->id,
-                                                              'bigbluebuttonbnid' => $bigbluebuttonbn->id,
-                                                              'userid' => $user->id,
-                                                              'log' => 'Join', ), '*', IGNORE_MULTIPLE);
+                                                                'bigbluebuttonbnid' => $bigbluebuttonbn->id,
+                                                                'userid' => $user->id,
+                                                                'log' => 'Join', ), '*', IGNORE_MULTIPLE);
 
     return $completed > 0;
 }
@@ -315,11 +315,11 @@ 
             $str = '<div class="bigbluebuttonbn overview">'."\n";
             $str .= '  <div class="name">'.get_string('modulename', 'bigbluebuttonbn').':&nbsp;'."\n";
             $str .= '    <a '.$classes.'href="'.$CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$bigbluebuttonbn->coursemodule.
-              '">'.$bigbluebuttonbn->name.'</a>'."\n";
+                '">'.$bigbluebuttonbn->name.'</a>'."\n";
             $str .= '  </div>'."\n";
             $str .= '  <div class="info">'.get_string($start, 'bigbluebuttonbn').': '.userdate($bigbluebuttonbn->openingtime).'</div>'."\n";
             $str .= '  <div class="info">'.get_string('ends_at', 'bigbluebuttonbn').': '.userdate($bigbluebuttonbn->closingtime)
-              .'</div>'."\n";
+                .'</div>'."\n";
             $str .= '</div>'."\n";
 
             if (empty($htmlarray[$bigbluebuttonbn->course]['bigbluebuttonbn'])) {

Spacing +61 added lines, -61 removed lines

@@ -26,23 +26,23 @@ 
 
 global $BIGBLUEBUTTONBN_CFG, $CFG;
 
-require_once $CFG->dirroot.'/calendar/lib.php';
-require_once $CFG->dirroot.'/message/lib.php';
-require_once $CFG->dirroot.'/mod/lti/OAuth.php';
-require_once $CFG->libdir.'/accesslib.php';
-require_once $CFG->libdir.'/completionlib.php';
-require_once $CFG->libdir.'/datalib.php';
-require_once $CFG->libdir.'/coursecatlib.php';
-require_once $CFG->libdir.'/enrollib.php';
-require_once $CFG->libdir.'/filelib.php';
-require_once $CFG->libdir.'/formslib.php';
-
-require_once dirname(__FILE__).'/vendor/firebase/php-jwt/src/JWT.php';
-
-if (file_exists(dirname(__FILE__).'/config.php')) {
-    require_once dirname(__FILE__).'/config.php';
+require_once $CFG->dirroot . '/calendar/lib.php';
+require_once $CFG->dirroot . '/message/lib.php';
+require_once $CFG->dirroot . '/mod/lti/OAuth.php';
+require_once $CFG->libdir . '/accesslib.php';
+require_once $CFG->libdir . '/completionlib.php';
+require_once $CFG->libdir . '/datalib.php';
+require_once $CFG->libdir . '/coursecatlib.php';
+require_once $CFG->libdir . '/enrollib.php';
+require_once $CFG->libdir . '/filelib.php';
+require_once $CFG->libdir . '/formslib.php';
+
+require_once dirname(__FILE__) . '/vendor/firebase/php-jwt/src/JWT.php';
+
+if (file_exists(dirname(__FILE__) . '/config.php')) {
+    require_once dirname(__FILE__) . '/config.php';
     if (isset($BIGBLUEBUTTONBN_CFG)) {
-        $CFG = (object) array_merge((array) $CFG, (array) $BIGBLUEBUTTONBN_CFG);
+        $CFG = (object)array_merge((array)$CFG, (array)$BIGBLUEBUTTONBN_CFG);
     }
 } else {
     $BIGBLUEBUTTONBN_CFG = new stdClass();
@@ -72,19 +72,19 @@ 
     }
 
     $features = array(
-        (string) FEATURE_IDNUMBER => true,
-        (string) FEATURE_GROUPS => true,
-        (string) FEATURE_GROUPINGS => true,
-        (string) FEATURE_GROUPMEMBERSONLY => true,
-        (string) FEATURE_MOD_INTRO => true,
-        (string) FEATURE_BACKUP_MOODLE2 => true,
-        (string) FEATURE_COMPLETION_TRACKS_VIEWS => true,
-        (string) FEATURE_GRADE_HAS_GRADE => false,
-        (string) FEATURE_GRADE_OUTCOMES => false,
-        (string) FEATURE_SHOW_DESCRIPTION => true,
+        (string)FEATURE_IDNUMBER => true,
+        (string)FEATURE_GROUPS => true,
+        (string)FEATURE_GROUPINGS => true,
+        (string)FEATURE_GROUPMEMBERSONLY => true,
+        (string)FEATURE_MOD_INTRO => true,
+        (string)FEATURE_BACKUP_MOODLE2 => true,
+        (string)FEATURE_COMPLETION_TRACKS_VIEWS => true,
+        (string)FEATURE_GRADE_HAS_GRADE => false,
+        (string)FEATURE_GRADE_OUTCOMES => false,
+        (string)FEATURE_SHOW_DESCRIPTION => true,
     );
 
-    if (isset($features[(string) $feature])) {
+    if (isset($features[(string)$feature])) {
         return $features[$feature];
     }
 
@@ -167,7 +167,7 @@ 
     }
 
     // End the session associated with this instance (if it's running)
-    $meetingID = $bigbluebuttonbn->meetingid.'-'.$bigbluebuttonbn->course.'-'.$bigbluebuttonbn->id;
+    $meetingID = $bigbluebuttonbn->meetingid . '-' . $bigbluebuttonbn->course . '-' . $bigbluebuttonbn->id;
     $modPW = $bigbluebuttonbn->moderatorpass;
 
     if (bigbluebuttonbn_isMeetingRunning($meetingID)) {
@@ -227,10 +227,10 @@ 
     $completed = $DB->count_records('bigbluebuttonbn_logs', array('courseid' => $course->id,
                                                               'bigbluebuttonbnid' => $bigbluebuttonbn->id,
                                                               'userid' => $user->id,
-                                                              'log' => 'Join', ), '*');
+                                                              'log' => 'Join',), '*');
 
     if ($completed > 0) {
-        return fullname($user).' '.get_string('view_message_has_joined', 'bigbluebuttonbn').' '.get_string('view_message_session_for', 'bigbluebuttonbn').' '.(string) $completed.' '.get_string('view_message_times', 'bigbluebuttonbn');
+        return fullname($user) . ' ' . get_string('view_message_has_joined', 'bigbluebuttonbn') . ' ' . get_string('view_message_session_for', 'bigbluebuttonbn') . ' ' . (string)$completed . ' ' . get_string('view_message_times', 'bigbluebuttonbn');
     }
 
     return '';
@@ -248,7 +248,7 @@ 
     $completed = $DB->count_recorda('bigbluebuttonbn_logs', array('courseid' => $course->id,
                                                               'bigbluebuttonbnid' => $bigbluebuttonbn->id,
                                                               'userid' => $user->id,
-                                                              'log' => 'Join', ), '*', IGNORE_MULTIPLE);
+                                                              'log' => 'Join',), '*', IGNORE_MULTIPLE);
 
     return $completed > 0;
 }
@@ -312,15 +312,15 @@ 
             if ($bigbluebuttonbn->visible) {
                 $classes = 'class="dimmed" ';
             }
-            $str = '<div class="bigbluebuttonbn overview">'."\n";
-            $str .= '  <div class="name">'.get_string('modulename', 'bigbluebuttonbn').':&nbsp;'."\n";
-            $str .= '    <a '.$classes.'href="'.$CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$bigbluebuttonbn->coursemodule.
-              '">'.$bigbluebuttonbn->name.'</a>'."\n";
-            $str .= '  </div>'."\n";
-            $str .= '  <div class="info">'.get_string($start, 'bigbluebuttonbn').': '.userdate($bigbluebuttonbn->openingtime).'</div>'."\n";
-            $str .= '  <div class="info">'.get_string('ends_at', 'bigbluebuttonbn').': '.userdate($bigbluebuttonbn->closingtime)
-              .'</div>'."\n";
-            $str .= '</div>'."\n";
+            $str = '<div class="bigbluebuttonbn overview">' . "\n";
+            $str .= '  <div class="name">' . get_string('modulename', 'bigbluebuttonbn') . ':&nbsp;' . "\n";
+            $str .= '    <a ' . $classes . 'href="' . $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $bigbluebuttonbn->coursemodule .
+              '">' . $bigbluebuttonbn->name . '</a>' . "\n";
+            $str .= '  </div>' . "\n";
+            $str .= '  <div class="info">' . get_string($start, 'bigbluebuttonbn') . ': ' . userdate($bigbluebuttonbn->openingtime) . '</div>' . "\n";
+            $str .= '  <div class="info">' . get_string('ends_at', 'bigbluebuttonbn') . ': ' . userdate($bigbluebuttonbn->closingtime)
+              .'</div>' . "\n";
+            $str .= '</div>' . "\n";
 
             if (empty($htmlarray[$bigbluebuttonbn->course]['bigbluebuttonbn'])) {
                 $htmlarray[$bigbluebuttonbn->course]['bigbluebuttonbn'] = '';
@@ -417,7 +417,7 @@ 
     // Now that an id was assigned, generate and set the meetingid property based on
     // [Moodle Instance + Activity ID + BBB Secret] (but only for new activities)
     if (isset($bigbluebuttonbn->add) && !empty($bigbluebuttonbn->add)) {
-        $meetingid = sha1($CFG->wwwroot.$bigbluebuttonbn->id.bigbluebuttonbn_get_cfg_shared_secret());
+        $meetingid = sha1($CFG->wwwroot . $bigbluebuttonbn->id . bigbluebuttonbn_get_cfg_shared_secret());
         $DB->set_field('bigbluebuttonbn', 'meetingid', $meetingid, array('id' => $bigbluebuttonbn->id));
 
         $action = get_string('mod_form_field_notification_msg_created', 'bigbluebuttonbn');
@@ -491,7 +491,7 @@ 
         // Get the first (and only) file.
         $file = reset($files);
         // Set the presentation column in the bigbluebuttonbn table.
-        $DB->set_field('bigbluebuttonbn', 'presentation', '/'.$file->get_filename(), array('id' => $bigbluebuttonbn_id));
+        $DB->set_field('bigbluebuttonbn', 'presentation', '/' . $file->get_filename(), array('id' => $bigbluebuttonbn_id));
     } else {
         // Set the presentation column in the bigbluebuttonbn table.
         $DB->set_field('bigbluebuttonbn', 'presentation', '', array('id' => $bigbluebuttonbn_id));
@@ -532,7 +532,7 @@ 
         return false;
     }
 
-    $fullpath = "/$context->id/mod_bigbluebuttonbn/$filearea/0/".$filename;
+    $fullpath = "/$context->id/mod_bigbluebuttonbn/$filearea/0/" . $filename;
     $fs = get_file_storage();
     if (!$file = $fs->get_file_by_hash(sha1($fullpath)) or $file->is_directory()) {
         return false;
@@ -642,19 +642,19 @@ 
 
 function bigbluebuttonbn_notification_msg_html($msg)
 {
-    $message_text = '<p>'.$msg->activity_type.' &quot;'.$msg->activity_title.'&quot; '.get_string('email_body_notification_meeting_has_been', 'bigbluebuttonbn').' '.$msg->action.'.</p>'."\n";
-    $message_text .= '<p><b>'.$msg->activity_title.'</b> '.get_string('email_body_notification_meeting_details', 'bigbluebuttonbn').':'."\n";
-    $message_text .= '<table border="0" style="margin: 5px 0 0 20px"><tbody>'."\n";
-    $message_text .= '<tr><td style="font-weight:bold;color:#555;">'.get_string('email_body_notification_meeting_title', 'bigbluebuttonbn').': </td><td>'."\n";
-    $message_text .= $msg->activity_title.'</td></tr>'."\n";
-    $message_text .= '<tr><td style="font-weight:bold;color:#555;">'.get_string('email_body_notification_meeting_description', 'bigbluebuttonbn').': </td><td>'."\n";
-    $message_text .= $msg->activity_description.'</td></tr>'."\n";
-    $message_text .= '<tr><td style="font-weight:bold;color:#555;">'.get_string('email_body_notification_meeting_start_date', 'bigbluebuttonbn').': </td><td>'."\n";
-    $message_text .= $msg->activity_openingtime.'</td></tr>'."\n";
-    $message_text .= '<tr><td style="font-weight:bold;color:#555;">'.get_string('email_body_notification_meeting_end_date', 'bigbluebuttonbn').': </td><td>'."\n";
-    $message_text .= $msg->activity_closingtime.'</td></tr>'."\n";
-    $message_text .= '<tr><td style="font-weight:bold;color:#555;">'.$msg->action.' '.get_string('email_body_notification_meeting_by', 'bigbluebuttonbn').': </td><td>'."\n";
-    $message_text .= $msg->activity_owner.'</td></tr></tbody></table></p>'."\n";
+    $message_text = '<p>' . $msg->activity_type . ' &quot;' . $msg->activity_title . '&quot; ' . get_string('email_body_notification_meeting_has_been', 'bigbluebuttonbn') . ' ' . $msg->action . '.</p>' . "\n";
+    $message_text .= '<p><b>' . $msg->activity_title . '</b> ' . get_string('email_body_notification_meeting_details', 'bigbluebuttonbn') . ':' . "\n";
+    $message_text .= '<table border="0" style="margin: 5px 0 0 20px"><tbody>' . "\n";
+    $message_text .= '<tr><td style="font-weight:bold;color:#555;">' . get_string('email_body_notification_meeting_title', 'bigbluebuttonbn') . ': </td><td>' . "\n";
+    $message_text .= $msg->activity_title . '</td></tr>' . "\n";
+    $message_text .= '<tr><td style="font-weight:bold;color:#555;">' . get_string('email_body_notification_meeting_description', 'bigbluebuttonbn') . ': </td><td>' . "\n";
+    $message_text .= $msg->activity_description . '</td></tr>' . "\n";
+    $message_text .= '<tr><td style="font-weight:bold;color:#555;">' . get_string('email_body_notification_meeting_start_date', 'bigbluebuttonbn') . ': </td><td>' . "\n";
+    $message_text .= $msg->activity_openingtime . '</td></tr>' . "\n";
+    $message_text .= '<tr><td style="font-weight:bold;color:#555;">' . get_string('email_body_notification_meeting_end_date', 'bigbluebuttonbn') . ': </td><td>' . "\n";
+    $message_text .= $msg->activity_closingtime . '</td></tr>' . "\n";
+    $message_text .= '<tr><td style="font-weight:bold;color:#555;">' . $msg->action . ' ' . get_string('email_body_notification_meeting_by', 'bigbluebuttonbn') . ': </td><td>' . "\n";
+    $message_text .= $msg->activity_owner . '</td></tr></tbody></table></p>' . "\n";
 }
 
 function bigbluebuttonbn_notification_send($sender, $bigbluebuttonbn, $message = '')
@@ -669,17 +669,17 @@ 
     $msg->user_name = fullname($sender);
     $msg->user_email = $sender->email;
     $msg->course_name = "$course->fullname";
-    $message .= '<p><hr/><br/>'.get_string('email_footer_sent_by', 'bigbluebuttonbn').' '.$msg->user_name.'('.$msg->user_email.') ';
-    $message .= get_string('email_footer_sent_from', 'bigbluebuttonbn').' '.$msg->course_name.'.</p>';
+    $message .= '<p><hr/><br/>' . get_string('email_footer_sent_by', 'bigbluebuttonbn') . ' ' . $msg->user_name . '(' . $msg->user_email . ') ';
+    $message .= get_string('email_footer_sent_from', 'bigbluebuttonbn') . ' ' . $msg->course_name . '.</p>';
 
     $users = get_enrolled_users($context);
     foreach ($users as $user) {
         if ($user->id != $sender->id) {
             $messageid = message_post_message($sender, $user, $message, FORMAT_HTML);
             if (!empty($messageid)) {
-                error_log('Msg to '.$msg->user_name.' was sent.');
+                error_log('Msg to ' . $msg->user_name . ' was sent.');
             } else {
-                error_log('Msg to '.$msg->user_name.' was NOT sent.');
+                error_log('Msg to ' . $msg->user_name . ' was NOT sent.');
             }
         }
     }
@@ -704,7 +704,7 @@ 
 {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
 
-    return isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_server_url) ? trim(trim($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_server_url), '/').'/' : (isset($CFG->bigbluebuttonbn_server_url) ? trim(trim($CFG->bigbluebuttonbn_server_url), '/').'/' : 'http://test-install.blindsidenetworks.com/bigbluebutton/');
+    return isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_server_url) ? trim(trim($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_server_url), '/') . '/' : (isset($CFG->bigbluebuttonbn_server_url) ? trim(trim($CFG->bigbluebuttonbn_server_url), '/') . '/' : 'http://test-install.blindsidenetworks.com/bigbluebutton/');
 }
 
 function bigbluebuttonbn_get_cfg_shared_secret()

bbb_view.php

Indentation +9 added lines, -9 removed lines

@@ -93,14 +93,14 @@ 
 
                 /// Prepare the metadata
                 $metadata = array('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' => (isset($name) && $name != '') ? $name : $bbbsession['contextActivityName'],
-                                  'bbb-recording-description' => (isset($description) && $description != '') ? $description : $bbbsession['contextActivityDescription'],
-                                  'bbb-recording-tags' => (isset($tags) && $tags != '') ? $tags : $bbbsession['contextActivityTags'],
+                                    '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' => (isset($name) && $name != '') ? $name : $bbbsession['contextActivityName'],
+                                    'bbb-recording-description' => (isset($description) && $description != '') ? $description : $bbbsession['contextActivityDescription'],
+                                    'bbb-recording-tags' => (isset($tags) && $tags != '') ? $tags : $bbbsession['contextActivityTags'],
                                 );
 
                 if (bigbluebuttonbn_server_offers_bn_capabilities()) {
@@ -138,7 +138,7 @@ 
                         $metadata,
                         $bbbsession['presentation']['name'],
                         $bbbsession['presentation']['url']
-                      );
+                        );
 
                 if (!$response) {
                     // If the server is unreachable, then prompts the user of the necessary action

Spacing +10 added lines, -10 removed lines

@@ -21,8 +21,8 @@ 
  * @copyright 2015 Blindside Networks Inc
  * @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 = optional_param('id', 0, PARAM_INT); // course_module ID, or
 $bn = optional_param('bn', 0, PARAM_INT); // bigbluebuttonbn instance ID
@@ -87,7 +87,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;
                 }
 
@@ -118,7 +118,7 @@ 
                 if (bigbluebuttonbn_get_cfg_scheduled_duration_enabled()) {
                     $durationtime = bigbluebuttonbn_get_duration($bigbluebuttonbn->closingtime);
                     if ($durationtime > 0) {
-                        $bbbsession['welcome'] .= '<br><br>'.str_replace('%duration%', ''.$durationtime, get_string('bbbdurationwarning', 'bigbluebuttonbn'));
+                        $bbbsession['welcome'] .= '<br><br>' . str_replace('%duration%', '' . $durationtime, get_string('bbbdurationwarning', 'bigbluebuttonbn'));
                     }
                 }
                 /// Execute the create command
@@ -149,7 +149,7 @@ 
                     if ($bbbsession['moderator']) {
                         $print_error_key = 'view_error_unable_join_teacher';
                     }
-                    print_error($print_error_key, 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+                    print_error($print_error_key, 'bigbluebuttonbn', $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
                     break;
                 } elseif ($response['returncode'] == 'FAILED') {
                     // The meeting was not created
@@ -201,7 +201,7 @@ 
 
     if ($bbbsession['userlimit'] > 0 && intval($meeting_info['participantCount']) >= $bbbsession['userlimit']) {
         // No more users allowed to join
-        header('Location: '.$bbbsession['logoutURL']);
+        header('Location: ' . $bbbsession['logoutURL']);
 
         return;
     }
@@ -219,20 +219,20 @@ 
     //// Before executing the redirect, increment the number of participants
     bigbluebuttonbn_participant_joined($bbbsession['meetingid'], ($bbbsession['administrator'] || $bbbsession['moderator']));
     //// Execute the redirect
-    header('Location: '.$join_url);
+    header('Location: ' . $join_url);
 }
 
 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);
+    print_error('view_error_bigbluebutton', 'bigbluebuttonbn', $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $id, $msgErrors, $sErrors);
     echo $OUTPUT->footer();
 }

index.php

Spacing +17 added lines, -17 removed lines

@@ -22,8 +22,8 @@ 
  * @copyright 2010-2017 Blindside Networks Inc
  * @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); // Course Module ID, or
 $a = optional_param('a', 0, PARAM_INT); // bigbluebuttonbn instance ID
@@ -98,12 +98,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 . ']';
         }
         $getArray = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getEndMeetingURL($meetingID, $bigbluebuttonbn->moderatorpass));
-        redirect('index.php?id='.$id);
+        redirect('index.php?id=' . $id);
     }
 }
 
@@ -123,7 +123,7 @@ 
         $can_moderate = ($administrator || $moderator);
 
         //Add a the data for the bigbluebuttonbn instance
-        $groupObj = (groups_get_activity_groupmode($cm) > 0) ? (object) array('id' => 0, 'name' => get_string('allparticipants')) : null;
+        $groupObj = (groups_get_activity_groupmode($cm) > 0) ? (object)array('id' => 0, 'name' => get_string('allparticipants')) : null;
         $table->data[] = bigbluebuttonbn_index_display_room($can_moderate, $course, $bigbluebuttonbn, $groupObj);
 
         //Add a the data for the groups belonging to the bigbluebuttonbn instance, if any
@@ -144,13 +144,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;
     }
 
@@ -171,7 +171,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 = '';
@@ -210,7 +210,7 @@ 
         $attendee_count = 0;
         foreach ($meetingInfo['attendees']->attendee as $attendee) {
             if ($attendee->role == $role) {
-                $attendeeList .= ($attendee_count++ > 0 ? ', ' : '').$attendee->fullName;
+                $attendeeList .= ($attendee_count++ > 0 ? ', ' : '') . $attendee->fullName;
             }
         }
     }
@@ -234,14 +234,14 @@ 
     $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;

locallib.php

Indentation +53 added lines, -53 removed lines

@@ -75,9 +75,9 @@ 
 function bigbluebuttonbn_getJoinURL($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)) {
@@ -93,11 +93,11 @@ 
 function bigbluebuttonbn_getCreateMeetingURL($name, $meetingID, $attendeePW, $moderatorPW, $welcome, $logoutURL, $record = 'false', $duration = 0, $voiceBridge = 0, $maxParticipants = 0, $metadata = array())
 {
     $data = ['meetingID' => $meetingID,
-              'name' => $name,
-              'attendeePW' => $attendeePW,
-              'moderatorPW' => $moderatorPW,
-              'logoutURL' => $logoutURL,
-              'record' => $record,
+                'name' => $name,
+                'attendeePW' => $attendeePW,
+                'moderatorPW' => $moderatorPW,
+                'logoutURL' => $logoutURL,
+                'record' => $record,
             ];
 
     $voiceBridge = intval($voiceBridge);
@@ -291,19 +291,19 @@ 
     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) {
@@ -612,10 +612,10 @@ 
 
         $options = array();
         $options['CURLOPT_HTTPHEADER'] = array(
-                 'Content-Type: '.$content_type,
-                 'Content-Length: '.strlen($data),
-                 'Content-Language: en-US',
-               );
+                    'Content-Type: '.$content_type,
+                    'Content-Length: '.strlen($data),
+                    'Content-Language: en-US',
+                );
 
         return $c->post($url, $data, $options);
     }
@@ -687,8 +687,8 @@ 
     $participant_list = json_decode($bigbluebuttonbn->participants);
     foreach ($participant_list as $participant) {
         $participant_list_array[] = array('selectiontype' => $participant->selectiontype,
-                                          'selectionid' => $participant->selectionid,
-                                          'role' => $participant->role, );
+                                            'selectionid' => $participant->selectionid,
+                                            'role' => $participant->role, );
     }
 
     return $participant_list_array;
@@ -700,8 +700,8 @@ 
 
     $participant_list_array = array();
     $participant_list_array[] = array('selectiontype' => 'all',
-                                       'selectionid' => 'all',
-                                       'role' => BIGBLUEBUTTONBN_ROLE_VIEWER, );
+                                        'selectionid' => 'all',
+                                        'role' => BIGBLUEBUTTONBN_ROLE_VIEWER, );
 
     $moderator_defaults = explode(',', bigbluebuttonbn_get_cfg_moderator_default());
     foreach ($moderator_defaults as $moderator_default) {
@@ -710,8 +710,8 @@ 
             foreach ($users as $user) {
                 if ($user->id == $USER->id) {
                     $participant_list_array[] = array('selectiontype' => 'user',
-                                                       'selectionid' => $USER->id,
-                                                       'role' => BIGBLUEBUTTONBN_ROLE_MODERATOR, );
+                                                        'selectionid' => $USER->id,
+                                                        'role' => BIGBLUEBUTTONBN_ROLE_MODERATOR, );
                     break;
                 }
             }
@@ -719,8 +719,8 @@ 
         }
 
         $participant_list_array[] = array('selectiontype' => 'role',
-                                          'selectionid' => $moderator_default,
-                                          'role' => BIGBLUEBUTTONBN_ROLE_MODERATOR, );
+                                            'selectionid' => $moderator_default,
+                                            'role' => BIGBLUEBUTTONBN_ROLE_MODERATOR, );
     }
 
     return $participant_list_array;
@@ -1642,24 +1642,24 @@ 
 function bigbluebuttonbn_get_cfg_options()
 {
     return [
-          'voicebridge_editable' => bigbluebuttonbn_get_cfg_voicebridge_editable(),
-          'recording_default' => bigbluebuttonbn_get_cfg_recording_default(),
-          'recording_editable' => bigbluebuttonbn_get_cfg_recording_editable(),
-          'recording_tagging_default' => bigbluebuttonbn_get_cfg_recording_tagging_default(),
-          'recording_tagging_editable' => bigbluebuttonbn_get_cfg_recording_tagging_editable(),
-          'waitformoderator_default' => bigbluebuttonbn_get_cfg_waitformoderator_default(),
-          'waitformoderator_editable' => bigbluebuttonbn_get_cfg_waitformoderator_editable(),
-          'userlimit_default' => bigbluebuttonbn_get_cfg_userlimit_default(),
-          'userlimit_editable' => bigbluebuttonbn_get_cfg_userlimit_editable(),
-          'preuploadpresentation_enabled' => bigbluebuttonbn_get_cfg_preuploadpresentation_enabled(),
-          'sendnotifications_enabled' => bigbluebuttonbn_get_cfg_sendnotifications_enabled(),
-          'recordings_html_default' => bigbluebuttonbn_get_cfg_recordings_html_default(),
-          'recordings_html_editable' => bigbluebuttonbn_get_cfg_recordings_html_editable(),
-          'recordings_deleted_activities_default' => bigbluebuttonbn_get_cfg_recordings_deleted_activities_default(),
-          'recordings_deleted_activities_editable' => bigbluebuttonbn_get_cfg_recordings_deleted_activities_editable(),
-          'recording_icons_enabled' => bigbluebuttonbn_get_cfg_recording_icons_enabled(),
-          'instance_type_enabled' => bigbluebuttonbn_recordings_enabled(),
-          'instance_type_default' => BIGBLUEBUTTONBN_TYPE_ALL,
+            'voicebridge_editable' => bigbluebuttonbn_get_cfg_voicebridge_editable(),
+            'recording_default' => bigbluebuttonbn_get_cfg_recording_default(),
+            'recording_editable' => bigbluebuttonbn_get_cfg_recording_editable(),
+            'recording_tagging_default' => bigbluebuttonbn_get_cfg_recording_tagging_default(),
+            'recording_tagging_editable' => bigbluebuttonbn_get_cfg_recording_tagging_editable(),
+            'waitformoderator_default' => bigbluebuttonbn_get_cfg_waitformoderator_default(),
+            'waitformoderator_editable' => bigbluebuttonbn_get_cfg_waitformoderator_editable(),
+            'userlimit_default' => bigbluebuttonbn_get_cfg_userlimit_default(),
+            'userlimit_editable' => bigbluebuttonbn_get_cfg_userlimit_editable(),
+            'preuploadpresentation_enabled' => bigbluebuttonbn_get_cfg_preuploadpresentation_enabled(),
+            'sendnotifications_enabled' => bigbluebuttonbn_get_cfg_sendnotifications_enabled(),
+            'recordings_html_default' => bigbluebuttonbn_get_cfg_recordings_html_default(),
+            'recordings_html_editable' => bigbluebuttonbn_get_cfg_recordings_html_editable(),
+            'recordings_deleted_activities_default' => bigbluebuttonbn_get_cfg_recordings_deleted_activities_default(),
+            'recordings_deleted_activities_editable' => bigbluebuttonbn_get_cfg_recordings_deleted_activities_editable(),
+            'recording_icons_enabled' => bigbluebuttonbn_get_cfg_recording_icons_enabled(),
+            'instance_type_enabled' => bigbluebuttonbn_recordings_enabled(),
+            'instance_type_default' => BIGBLUEBUTTONBN_TYPE_ALL,
         ];
 }
 
@@ -1935,8 +1935,8 @@ 
     $activity_time = '';
     if ($time) {
         $activity_time = 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 $activity_time;

Spacing +63 added lines, -63 removed lines

@@ -26,7 +26,7 @@ 
 
 global $BIGBLUEBUTTONBN_CFG, $CFG;
 
-require_once dirname(__FILE__).'/lib.php';
+require_once dirname(__FILE__) . '/lib.php';
 
 const BIGBLUEBUTTONBN_FORCED = true;
 
@@ -65,7 +65,7 @@ 
     if (isset($meta)) {
         $log->meta = $meta;
     } elseif ($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);
@@ -187,19 +187,19 @@ 
 
 function bigbluebuttonbn_bigbluebutton_action_url($action, $data = array(), $metadata = array())
 {
-    $base_url = bigbluebuttonbn_get_cfg_server_url().'api/'.$action.'?';
+    $base_url = bigbluebuttonbn_get_cfg_server_url() . 'api/' . $action . '?';
 
     $params = '';
 
     foreach ($data as $key => $value) {
-        $params .= '&'.$key.'='.urlencode($value);
+        $params .= '&' . $key . '=' . urlencode($value);
     }
 
     foreach ($metadata as $key => $value) {
-        $params .= '&'.'meta_'.$key.'='.urlencode($value);
+        $params .= '&' . 'meta_' . $key . '=' . urlencode($value);
     }
 
-    return $base_url.$params.'&checksum='.sha1($action.$params.bigbluebuttonbn_get_cfg_shared_secret());
+    return $base_url . $params . '&checksum=' . sha1($action . $params . bigbluebuttonbn_get_cfg_shared_secret());
 }
 
 function bigbluebuttonbn_getCreateMeetingArray($meetingName, $meetingID, $welcomeString, $mPW, $aPW, $logoutURL, $record = 'false', $duration = 0, $voiceBridge = 0, $maxParticipants = 0, $metadata = array(), $presentation_name = null, $presentation_url = null)
@@ -210,7 +210,7 @@ 
 
     if (!is_null($presentation_name) && !is_null($presentation_url)) {
         $method = BIGBLUEBUTTONBN_METHOD_POST;
-        $data = "<?xml version='1.0' encoding='UTF-8'?><modules><module name='presentation'><document url='".$presentation_url."' /></module></modules>";
+        $data = "<?xml version='1.0' encoding='UTF-8'?><modules><module name='presentation'><document url='" . $presentation_url . "' /></module></modules>";
     }
 
     $xml = bigbluebuttonbn_wrap_xml_load_file($create_meeting_url, $method, $data);
@@ -256,7 +256,7 @@ 
                                 'moderatorPW' => $meeting->moderatorPW,
                                 'attendeePW' => $meeting->attendeePW,
                                 'hasBeenForciblyEnded' => $meeting->hasBeenForciblyEnded,
-                                'running' => $meeting->running, );
+                                'running' => $meeting->running,);
         }
 
         return $meetings;
@@ -379,11 +379,11 @@ 
 {
     global $DB;
 
-    $select = "courseid = '{$courseID}' AND bigbluebuttonbnid <> '{$bigbluebuttonbnID}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_IMPORT."'";
+    $select = "courseid = '{$courseID}' AND bigbluebuttonbnid <> '{$bigbluebuttonbnID}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_IMPORT . "'";
     if ($bigbluebuttonbnID === null) {
-        $select = "courseid = '{$courseID}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_IMPORT."'";
+        $select = "courseid = '{$courseID}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_IMPORT . "'";
     } elseif ($subset) {
-        $select = "bigbluebuttonbnid = '{$bigbluebuttonbnID}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_IMPORT."'";
+        $select = "bigbluebuttonbnid = '{$bigbluebuttonbnID}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_IMPORT . "'";
     }
     $records_imported = $DB->get_records_select('bigbluebuttonbn_logs', $select);
 
@@ -413,7 +413,7 @@ 
 function bigbluebuttonbn_getDefaultConfigXMLArray()
 {
     $default_config_xml = bigbluebuttonbn_getDefaultConfigXML();
-    $default_config_xml_array = (array) $default_config_xml;
+    $default_config_xml_array = (array)$default_config_xml;
 
     return $default_config_xml_array;
 }
@@ -423,18 +423,18 @@ 
     //Add formats
     $playbackArray = array();
     foreach ($recording->playback->format as $format) {
-        $playbackArray[(string) $format->type] = array('type' => (string) $format->type, 'url' => (string) $format->url, 'length' => (string) $format->length);
+        $playbackArray[(string)$format->type] = array('type' => (string)$format->type, 'url' => (string)$format->url, 'length' => (string)$format->length);
         //Add preview per format when existing
         if ($format->preview) {
             $imagesArray = array();
             foreach ($format->preview->images->image as $image) {
-                $imageArray = array('url' => (string) $image);
+                $imageArray = array('url' => (string)$image);
                 foreach ($image->attributes() as $attKey => $attValue) {
-                    $imageArray[$attKey] = (string) $attValue;
+                    $imageArray[$attKey] = (string)$attValue;
                 }
                 array_push($imagesArray, $imageArray);
             }
-            $playbackArray[(string) $format->type]['preview'] = $imagesArray;
+            $playbackArray[(string)$format->type]['preview'] = $imagesArray;
         }
     }
 
@@ -445,10 +445,10 @@ 
         if (is_object($value)) {
             $value = '';
         }
-        $metadataArray['meta_'.$key] = $value;
+        $metadataArray['meta_' . $key] = $value;
     }
 
-    $recordingArrayValue = 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) + $metadataArray;
+    $recordingArrayValue = 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) + $metadataArray;
 
     return $recordingArrayValue;
 }
@@ -528,7 +528,7 @@ 
 
 function bigbluebuttonbn_getServerVersion()
 {
-    $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_get_cfg_server_url().'api');
+    $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_get_cfg_server_url() . 'api');
     if ($xml && $xml->returncode == 'SUCCESS') {
         return $xml->version;
     }
@@ -553,7 +553,7 @@ 
 function bigbluebuttonbn_wrap_xml_load_file($url, $method = BIGBLUEBUTTONBN_METHOD_GET, $data = null, $content_type = 'text/xml')
 {
     if (bigbluebuttonbn_debugdisplay()) {
-        error_log('Request to: '.$url);
+        error_log('Request to: ' . $url);
     }
 
     if (extension_loaded('curl')) {
@@ -566,7 +566,7 @@ 
         }
 
         if (bigbluebuttonbn_debugdisplay()) {
-            error_log('Response: '.$response);
+            error_log('Response: ' . $response);
         }
 
         $previous = libxml_use_internal_errors(true);
@@ -576,7 +576,7 @@ 
             return $xml;
         } catch (Exception $e) {
             libxml_use_internal_errors($previous);
-            $error = 'Caught exception: '.$e->getMessage();
+            $error = 'Caught exception: ' . $e->getMessage();
             error_log($error);
 
             return null;
@@ -588,12 +588,12 @@ 
     try {
         $response = simplexml_load_file($url, 'SimpleXMLElement', LIBXML_NOCDATA | LIBXML_NOBLANKS);
         if (bigbluebuttonbn_debugdisplay()) {
-            error_log('Response processed: '.$response->asXML());
+            error_log('Response processed: ' . $response->asXML());
         }
 
         return $response;
     } catch (Exception $e) {
-        $error = 'Caught exception: '.$e->getMessage();
+        $error = 'Caught exception: ' . $e->getMessage();
         error_log($error);
         libxml_use_internal_errors($previous);
 
@@ -612,8 +612,8 @@ 
 
         $options = array();
         $options['CURLOPT_HTTPHEADER'] = array(
-                 'Content-Type: '.$content_type,
-                 'Content-Length: '.strlen($data),
+                 'Content-Type: ' . $content_type,
+                 'Content-Length: ' . strlen($data),
                  'Content-Language: en-US',
                );
 
@@ -631,9 +631,9 @@ 
     if ($user_roles) {
         $where = '';
         foreach ($user_roles as $key => $value) {
-            $where .= (empty($where) ? ' WHERE' : ' AND').' id='.$value->roleid;
+            $where .= (empty($where) ? ' WHERE' : ' AND') . ' id=' . $value->roleid;
         }
-        $user_roles = $DB->get_records_sql('SELECT * FROM {role}'.$where);
+        $user_roles = $DB->get_records_sql('SELECT * FROM {role}' . $where);
     }
 
     return $user_roles;
@@ -688,7 +688,7 @@ 
     foreach ($participant_list as $participant) {
         $participant_list_array[] = array('selectiontype' => $participant->selectiontype,
                                           'selectionid' => $participant->selectionid,
-                                          'role' => $participant->role, );
+                                          'role' => $participant->role,);
     }
 
     return $participant_list_array;
@@ -701,7 +701,7 @@ 
     $participant_list_array = array();
     $participant_list_array[] = array('selectiontype' => 'all',
                                        'selectionid' => 'all',
-                                       'role' => BIGBLUEBUTTONBN_ROLE_VIEWER, );
+                                       'role' => BIGBLUEBUTTONBN_ROLE_VIEWER,);
 
     $moderator_defaults = explode(',', bigbluebuttonbn_get_cfg_moderator_default());
     foreach ($moderator_defaults as $moderator_default) {
@@ -711,7 +711,7 @@ 
                 if ($user->id == $USER->id) {
                     $participant_list_array[] = array('selectiontype' => 'user',
                                                        'selectionid' => $USER->id,
-                                                       'role' => BIGBLUEBUTTONBN_ROLE_MODERATOR, );
+                                                       'role' => BIGBLUEBUTTONBN_ROLE_MODERATOR,);
                     break;
                 }
             }
@@ -720,7 +720,7 @@ 
 
         $participant_list_array[] = array('selectiontype' => 'role',
                                           'selectionid' => $moderator_default,
-                                          'role' => BIGBLUEBUTTONBN_ROLE_MODERATOR, );
+                                          'role' => BIGBLUEBUTTONBN_ROLE_MODERATOR,);
     }
 
     return $participant_list_array;
@@ -778,9 +778,9 @@ 
     $is_unique = 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)) {
             $is_unique = false;
@@ -845,7 +845,7 @@ 
     $mt = microtime();
     $rand = mt_rand();
 
-    return md5($mt.$rand);
+    return md5($mt . $rand);
 }
 
 function bigbluebuttonbn_random_password($length = 8)
@@ -955,7 +955,7 @@ 
     }
 
     //Ping again and refresh the cache
-    $meeting_info = (array) bigbluebuttonbn_getMeetingInfo($meetingid);
+    $meeting_info = (array)bigbluebuttonbn_getMeetingInfo($meetingid);
     $cache->set($meetingid, array('creation_time' => time(), 'meeting_info' => json_encode($meeting_info)));
 
     return $meeting_info;
@@ -1051,7 +1051,7 @@ 
         case 'moodle_event':
             break;
         default:
-            return bigbluebuttonbn_add_error($error, 'Action '.$params['action'].' can not be performed.');
+            return bigbluebuttonbn_add_error($error, 'Action ' . $params['action'] . ' can not be performed.');
     }
 
     return '';
@@ -1077,8 +1077,8 @@ 
  */
 function bigbluebuttonbn_setConfigXMLParams($meetingID, $configXML)
 {
-    $params = 'configXML='.urlencode($configXML).'&meetingID='.urlencode($meetingID);
-    $config_xml_params = $params.'&checksum='.sha1('setConfigXML'.$params.bigbluebuttonbn_get_cfg_shared_secret());
+    $params = 'configXML=' . urlencode($configXML) . '&meetingID=' . urlencode($meetingID);
+    $config_xml_params = $params . '&checksum=' . sha1('setConfigXML' . $params . bigbluebuttonbn_get_cfg_shared_secret());
 
     return $config_xml_params;
 }
@@ -1089,7 +1089,7 @@ 
  */
 function bigbluebuttonbn_setConfigXML($meetingID, $configXML)
 {
-    $url_default_config = bigbluebuttonbn_get_cfg_server_url().'api/setConfigXML?';
+    $url_default_config = bigbluebuttonbn_get_cfg_server_url() . 'api/setConfigXML?';
     $config_xml_params = bigbluebuttonbn_setConfigXMLParams($meetingID, $configXML);
     $xml = bigbluebuttonbn_wrap_xml_load_file($url_default_config, BIGBLUEBUTTONBN_METHOD_POST, $config_xml_params, 'application/x-www-form-urlencoded');
 
@@ -1103,7 +1103,7 @@ 
 function bigbluebuttonbn_setConfigXMLArray($meetingID, $configXML)
 {
     $config_xml = bigbluebuttonbn_setConfigXML($meetingID, $configXML);
-    $config_xml_array = (array) $config_xml;
+    $config_xml_array = (array)$config_xml;
 
     return $config_xml_array;
 }
@@ -1145,7 +1145,7 @@ 
         $row->date = floatval($recording['startTime']);
 
         // Set formatted date
-        $dateformat = get_string('strftimerecentfull', 'langconfig').' %Z';
+        $dateformat = get_string('strftimerecentfull', 'langconfig') . ' %Z';
         $row->date_formatted = userdate($startTime / 1000, $dateformat, usertimezone($USER->timezone));
 
         // Set formatted duration
@@ -1213,7 +1213,7 @@ 
 
     $attributes = 'data-imported="false"';
     if (isset($recording['imported'])) {
-        $attributes = 'data-imported="true" title="'.get_string('view_recording_link_warning', 'bigbluebuttonbn').'"';
+        $attributes = 'data-imported="true" title="' . get_string('view_recording_link_warning', 'bigbluebuttonbn') . '"';
     }
 
     $visibility = '';
@@ -1221,9 +1221,9 @@ 
         $visibility = 'hidden ';
     }
 
-    $recording_types = '<div id="playbacks-'.$recording['recordID'].'" '.$attributes.' '.$visibility.'>';
+    $recording_types = '<div id="playbacks-' . $recording['recordID'] . '" ' . $attributes . ' ' . $visibility . '>';
     foreach ($recording['playbacks'] as $playback) {
-        $recording_types .= $OUTPUT->action_link($playback['url'], get_string('view_recording_format_'.$playback['type'], 'bigbluebuttonbn'), null, array('title' => get_string('view_recording_format_'.$playback['type'], 'bigbluebuttonbn'), 'target' => '_new')).'&#32;';
+        $recording_types .= $OUTPUT->action_link($playback['url'], get_string('view_recording_format_' . $playback['type'], 'bigbluebuttonbn'), null, array('title' => get_string('view_recording_format_' . $playback['type'], 'bigbluebuttonbn'), 'target' => '_new')) . '&#32;';
     }
     $recording_types .= '</div>';
 
@@ -1258,12 +1258,12 @@ 
 {
     global $OUTPUT;
 
-    $onclick = 'M.mod_bigbluebuttonbn.broker_manageRecording("'.$manage_action.'", "'.$recording['recordID'].'", "'.$recording['meetingID'].'");';
+    $onclick = 'M.mod_bigbluebuttonbn.broker_manageRecording("' . $manage_action . '", "' . $recording['recordID'] . '", "' . $recording['meetingID'] . '");';
     if (bigbluebuttonbn_get_cfg_recording_icons_enabled()) {
         //With icon for $manage_action
-        $icon_attributes = array('id' => 'recording-btn-'.$manage_action.'-'.$recording['recordID']);
-        $icon = new pix_icon('i/'.$manage_tag, get_string($manage_tag), 'moodle', $icon_attributes);
-        $link_attributes = array('id' => 'recording-link-'.$manage_action.'-'.$recording['recordID'], 'onclick' => $onclick);
+        $icon_attributes = array('id' => 'recording-btn-' . $manage_action . '-' . $recording['recordID']);
+        $icon = new pix_icon('i/' . $manage_tag, get_string($manage_tag), 'moodle', $icon_attributes);
+        $link_attributes = array('id' => 'recording-link-' . $manage_action . '-' . $recording['recordID'], 'onclick' => $onclick);
 
         return $OUTPUT->action_icon('#', $icon, null, $link_attributes, false);
     }
@@ -1335,7 +1335,7 @@ 
     $table->data = array();
 
     ///Initialize table headers
-    $table->head = array($playback.$recording, $description, $preview, $date, $duration);
+    $table->head = array($playback . $recording, $description, $preview, $date, $duration);
     $table->align = array('left', 'left', 'left', 'left', 'left', 'center');
     if ($bbbsession['managerecordings']) {
         $table->head[] = $actionbar;
@@ -1346,7 +1346,7 @@ 
     if (isset($recordings) && !array_key_exists('messageKey', $recordings)) {  // There are recordings for this meeting
         foreach ($recordings as $recording) {
             $row = new html_table_row();
-            $row->id = 'recording-td-'.$recording['recordID'];
+            $row->id = 'recording-td-' . $recording['recordID'];
             $row->attributes['data-imported'] = 'false';
             if (isset($recording['imported'])) {
                 $row->attributes['data-imported'] = 'true';
@@ -1378,7 +1378,7 @@ 
     /// Build the message_body
     $msg->activity_type = '';
     $msg->activity_title = $bigbluebuttonbn->name;
-    $message_text = '<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>';
+    $message_text = '<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, $message_text);
 }
@@ -1741,9 +1741,9 @@ 
         $select = '';
         foreach ($bigbluebuttonbns as $key => $bigbluebuttonbn) {
             $select .= strlen($select) == 0 ? '(' : ' OR ';
-            $select .= 'bigbluebuttonbnid='.$bigbluebuttonbn->id;
+            $select .= 'bigbluebuttonbnid=' . $bigbluebuttonbn->id;
         }
-        $select .= ") AND log='".BIGBLUEBUTTONBN_LOG_EVENT_CREATE."'";
+        $select .= ") AND log='" . BIGBLUEBUTTONBN_LOG_EVENT_CREATE . "'";
 
         //Execute select for loading records based on existent bigbluebuttonbns
         $records = $DB->get_records_select($table, $select);
@@ -1751,7 +1751,7 @@ 
         //Remove duplicates
         $unique_records = array();
         foreach ($records as $key => $record) {
-            $record_key = $record->meetingid.','.$record->bigbluebuttonbnid.','.$record->meta;
+            $record_key = $record->meetingid . ',' . $record->bigbluebuttonbnid . ',' . $record->meta;
             if (array_search($record_key, $unique_records) === true) {
                 unset($records[$key]);
                 continue;
@@ -1790,7 +1790,7 @@ 
 {
     global $CFG;
 
-    return (bool) $CFG->debugdisplay;
+    return (bool)$CFG->debugdisplay;
 }
 
 function bigbluebuttonbn_html2text($html, $len)
@@ -1810,7 +1810,7 @@ 
     $tags = '';
     $tags_array = core_tag_tag::get_item_tags_array('core', 'course_modules', $id);
     foreach ($tags_array as $tag) {
-        $tags .= ($tags == '') ? $tag : ','.$tag;
+        $tags .= ($tags == '') ? $tag : ',' . $tag;
     }
 
     return $tags;
@@ -1832,13 +1832,13 @@ 
 
     // Gather the bigbluebuttonbnids whose meetingids should be included in the getRecordings request
     $select = "id <> '{$bigbluebuttonbnID}' AND course = '{$courseID}'";
-    $select_deleted = "courseid = '{$courseID}' AND bigbluebuttonbnid <> '{$bigbluebuttonbnID}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_DELETE."' AND meta like '%has_recordings%' AND meta like '%true%'";
+    $select_deleted = "courseid = '{$courseID}' AND bigbluebuttonbnid <> '{$bigbluebuttonbnID}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_DELETE . "' AND meta like '%has_recordings%' AND meta like '%true%'";
     if ($bigbluebuttonbnID === null) {
         $select = "course = '{$courseID}'";
-        $select_deleted = "courseid = '{$courseID}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_DELETE."' AND meta like '%has_recordings%' AND meta like '%true%'";
+        $select_deleted = "courseid = '{$courseID}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_DELETE . "' AND meta like '%has_recordings%' AND meta like '%true%'";
     } elseif ($subset) {
         $select = "id = '{$bigbluebuttonbnID}'";
-        $select_deleted = "bigbluebuttonbnid = '{$bigbluebuttonbnID}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_DELETE."' AND meta like '%has_recordings%' AND meta like '%true%'";
+        $select_deleted = "bigbluebuttonbnid = '{$bigbluebuttonbnID}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_DELETE . "' AND meta like '%has_recordings%' AND meta like '%true%'";
     }
     $bigbluebuttonbns = $DB->get_records_select_menu('bigbluebuttonbn', $select, null, 'id', 'id, meetingid');
 
@@ -1856,7 +1856,7 @@ 
     if (!empty($bigbluebuttonbns)) {
         //Prepare select for loading records based on existent bigbluebuttonbns
         $sql = 'SELECT DISTINCT meetingid, bigbluebuttonbnid FROM {bigbluebuttonbn_logs} WHERE ';
-        $sql .= '(bigbluebuttonbnid='.implode(' OR bigbluebuttonbnid=', array_keys($bigbluebuttonbns)).')';
+        $sql .= '(bigbluebuttonbnid=' . implode(' OR bigbluebuttonbnid=', array_keys($bigbluebuttonbns)) . ')';
         //Include only Create events and exclude those with record not true
         $sql .= ' AND log = ? AND meta LIKE ? AND meta LIKE ?';
         //Execute select for loading records based on existent bigbluebuttonbns
@@ -1934,8 +1934,8 @@ 
 {
     $activity_time = '';
     if ($time) {
-        $activity_time = calendar_day_representation($time).' '.
-          get_string('mod_form_field_notification_msg_at', 'bigbluebuttonbn').' '.
+        $activity_time = calendar_day_representation($time) . ' ' .
+          get_string('mod_form_field_notification_msg_at', 'bigbluebuttonbn') . ' ' .
           calendar_time_representation($time);
     }