blindsidenetworks / moodle-mod_bigbluebuttonbn

view.php

Spacing +91 added lines, -91 removed lines

@@ -9,12 +9,12 @@ 
  * @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
-$group  = optional_param('group', 0, PARAM_INT);    // group instance ID
+$id = required_param('id', PARAM_INT); // Course Module ID, or
+$b  = optional_param('n', 0, PARAM_INT); // bigbluebuttonbn instance ID
+$group  = optional_param('group', 0, PARAM_INT); // group instance ID
 
 if ($id) {
     $cm = get_coursemodule_from_id('bigbluebuttonbn', $id, 0, false, MUST_EXIST);
@@ -31,7 +31,7 @@ 
 require_login($course, true, $cm);
 
 $version_major = bigbluebuttonbn_get_moodle_version_major();
-if ( $version_major < '2013111800' ) {
+if ($version_major < '2013111800') {
     //This is valid before v2.6
     $module = $DB->get_record('modules', array('name' => 'bigbluebuttonbn'));
     $module_version = $module->version;
@@ -56,7 +56,7 @@ 
 $bbbsession['roles'] = get_user_roles($context, $USER->id, true);
 
 // User roles
-if( $bigbluebuttonbn->participants == null || $bigbluebuttonbn->participants == "" || $bigbluebuttonbn->participants == "[]" ){
+if ($bigbluebuttonbn->participants == null || $bigbluebuttonbn->participants == "" || $bigbluebuttonbn->participants == "[]") {
     //The room that is being used comes from a previous version
     $bbbsession['moderator'] = has_capability('mod/bigbluebuttonbn:moderate', $context);
 } else {
@@ -76,16 +76,16 @@ 
 // Database info related to the activity
 $bbbsession['meetingdescription'] = $bigbluebuttonbn->intro;
 $bbbsession['welcome'] = $bigbluebuttonbn->welcome;
-if( !isset($bbbsession['welcome']) || $bbbsession['welcome'] == '') {
+if (!isset($bbbsession['welcome']) || $bbbsession['welcome'] == '') {
     $bbbsession['welcome'] = get_string('mod_form_field_welcome_default', 'bigbluebuttonbn'); 
 }
 
 $bbbsession['userlimit'] = intval($bigbluebuttonbn->userlimit);
-$bbbsession['voicebridge'] = ($bigbluebuttonbn->voicebridge > 0)? 70000 + $bigbluebuttonbn->voicebridge: $bigbluebuttonbn->voicebridge;
+$bbbsession['voicebridge'] = ($bigbluebuttonbn->voicebridge > 0) ? 70000 + $bigbluebuttonbn->voicebridge : $bigbluebuttonbn->voicebridge;
 $bbbsession['wait'] = $bigbluebuttonbn->wait;
 $bbbsession['record'] = $bigbluebuttonbn->record;
-if( $bigbluebuttonbn->record )
-    $bbbsession['welcome'] .= '<br><br>'.get_string('bbbrecordwarning', 'bigbluebuttonbn');
+if ($bigbluebuttonbn->record)
+    $bbbsession['welcome'] .= '<br><br>' . get_string('bbbrecordwarning', 'bigbluebuttonbn');
 $bbbsession['tagging'] = $bigbluebuttonbn->tagging;
 
 $bbbsession['openingtime'] = $bigbluebuttonbn->openingtime;
@@ -104,29 +104,29 @@ 
 $bbbsession['originServerName'] = $parsedUrl['host'];
 $bbbsession['originServerUrl'] = $CFG->wwwroot;
 $bbbsession['originServerCommonName'] = '';
-$bbbsession['originTag'] = 'moodle-mod_bigbluebuttonbn ('.$module_version.')';
+$bbbsession['originTag'] = 'moodle-mod_bigbluebuttonbn (' . $module_version . ')';
 ////////////////////////////////////////////////
 /////   BigBlueButton Session Setup Ends   /////
 ////////////////////////////////////////////////
 
 // Validates if the BigBlueButton server is running
 $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' );
-    else if ( $bbbsession['moderator'] )
-        print_error( 'view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course );
+if (!isset($serverVersion)) { //Server is not working
+    if ($bbbsession['administrator'])
+        print_error('view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
+    else if ($bbbsession['moderator'])
+        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);
 } else {
-    $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getMeetingsURL( $bbbsession['endpoint'], $bbbsession['shared_secret'] ) );
-    if ( !isset($xml) || !isset($xml->returncode) || $xml->returncode == 'FAILED' ){ // The shared secret is wrong
-        if ( $bbbsession['administrator'] )
-            print_error( 'view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
-        else if ( $bbbsession['moderator'] )
-            print_error( 'view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course );
+    $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getMeetingsURL($bbbsession['endpoint'], $bbbsession['shared_secret']));
+    if (!isset($xml) || !isset($xml->returncode) || $xml->returncode == 'FAILED') { // The shared secret is wrong
+        if ($bbbsession['administrator'])
+            print_error('view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
+        else if ($bbbsession['moderator'])
+            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);
     }
 }
 
@@ -136,12 +136,12 @@ 
 
 // 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);
 
-if( $bigbluebuttonbn->newwindow == 1 ) {
+if ($bigbluebuttonbn->newwindow == 1) {
     $PAGE->blocks->show_only_fake_blocks();
 
 } else {
@@ -149,14 +149,14 @@ 
 }
 
 // Validate if the user is in a role allowed to join
-if ( !has_capability('moodle/category:manage', $context) && !has_capability('mod/bigbluebuttonbn:join', $context) ) {
+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();
@@ -164,10 +164,10 @@ 
 }
 
 // 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';
-$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';
+$bbbsession['joinURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_view.php?action=join&id=' . $id . '&bigbluebuttonbn=' . $bbbsession['bigbluebuttonbn']->id;
 
 $bigbluebuttonbn_view = '';
 
@@ -176,20 +176,20 @@ 
 
 /// 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;
+if ($groupmode == NOGROUPS) {  //No groups mode
+    $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();
 
     $bbbsession['group'] = groups_get_activity_group($bbbsession['cm'], true);
-    if ($groupmode == SEPARATEGROUPS ) {
-        groups_print_activity_menu($cm, $CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$bbbsession['cm']->id, false, true);
-        if( $bbbsession['group'] == 0 ) {
-            if ( $bbbsession['administrator'] ) {
+    if ($groupmode == SEPARATEGROUPS) {
+        groups_print_activity_menu($cm, $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $bbbsession['cm']->id, false, true);
+        if ($bbbsession['group'] == 0) {
+            if ($bbbsession['administrator']) {
                 $my_groups = groups_get_all_groups($bbbsession['course']->id);
             } else {
                 $my_groups = groups_get_activity_allowed_groups($bbbsession['cm']);
@@ -199,15 +199,15 @@ 
         }
 
     } else {
-        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);
     }
 
-    $bbbsession['meetingid'] = $bbbsession['bigbluebuttonbn']->meetingid.'-'.$bbbsession['course']->id.'-'.$bbbsession['bigbluebuttonbn']->id.'['.$bbbsession['group'].']';
-    if( $bbbsession['group'] > 0 )
+    $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'];
@@ -215,7 +215,7 @@ 
 $bbbsession['contextActivityTags'] = "";
 
 $now = time();
-if (!empty($bigbluebuttonbn->openingtime) && $now < $bigbluebuttonbn->openingtime ) {
+if (!empty($bigbluebuttonbn->openingtime) && $now < $bigbluebuttonbn->openingtime) {
     //CALLING BEFORE
     $bigbluebuttonbn_view = 'before';
 
@@ -247,7 +247,7 @@ 
         'action' => $bigbluebuttonbn_view,
         'meetingid' => $bbbsession['meetingid'],
         'bigbluebuttonbnid' => $bbbsession['bigbluebuttonbn']->id,
-        'ping_interval' => ($waitformoderator_ping_interval > 0? $waitformoderator_ping_interval * 1000: 15000),
+        'ping_interval' => ($waitformoderator_ping_interval > 0 ? $waitformoderator_ping_interval * 1000 : 15000),
         'userlimit' => $bbbsession['userlimit'],
         'locales' => bigbluebuttonbn_get_locales_for_ui()
     );
@@ -264,7 +264,7 @@ 
 // Finish the page
 echo $OUTPUT->footer();
 
-function bigbluebuttonbn_view_joining($bbbsession){
+function bigbluebuttonbn_view_joining($bbbsession) {
     global $CFG, $DB, $OUTPUT;
 
     echo $OUTPUT->heading($bbbsession['meetingname'], 3);
@@ -273,29 +273,29 @@ 
     echo '<br><span id="status_bar"></span><br>';
     echo '<span id="control_panel"></span>';
     echo $OUTPUT->box_end();
-    if( $bbbsession['tagging'] && ($bbbsession['administrator'] || $bbbsession['moderator']) ){
-        echo ''.
-          '<div id="panelContent" class="hidden">'.
-          '  <div class="yui3-widget-bd">'.
-          '    <form>'.
-          '      <fieldset>'.
-          '        <input type="hidden" name="join" id="meeting_join_url" value="">'.
-          '        <input type="hidden" name="message" id="meeting_message" value="">'.
-          '        <div>'.
-          '          <label for="name">'.get_string('view_recording_name', 'bigbluebuttonbn').'</label><br/>'.
-          '          <input type="text" name="name" id="recording_name" placeholder="">'.
-          '        </div><br>'.
-          '        <div>'.
-          '          <label for="description">'.get_string('view_recording_description', 'bigbluebuttonbn').'</label><br/>'.
-          '          <input type="text" name="description" id="recording_description" value="" placeholder="">'.
-          '        </div><br>'.
-          '        <div>'.
-          '          <label for="tags">'.get_string('view_recording_tags', 'bigbluebuttonbn').'</label><br/>'.
-          '          <input type="text" name="tags" id="recording_tags" value="" placeholder="">'.
-          '        </div>'.
-          '      </fieldset>'.
-          '    </form>'.
-          '  </div>'.
+    if ($bbbsession['tagging'] && ($bbbsession['administrator'] || $bbbsession['moderator'])) {
+        echo '' .
+          '<div id="panelContent" class="hidden">' .
+          '  <div class="yui3-widget-bd">' .
+          '    <form>' .
+          '      <fieldset>' .
+          '        <input type="hidden" name="join" id="meeting_join_url" value="">' .
+          '        <input type="hidden" name="message" id="meeting_message" value="">' .
+          '        <div>' .
+          '          <label for="name">' . get_string('view_recording_name', 'bigbluebuttonbn') . '</label><br/>' .
+          '          <input type="text" name="name" id="recording_name" placeholder="">' .
+          '        </div><br>' .
+          '        <div>' .
+          '          <label for="description">' . get_string('view_recording_description', 'bigbluebuttonbn') . '</label><br/>' .
+          '          <input type="text" name="description" id="recording_description" value="" placeholder="">' .
+          '        </div><br>' .
+          '        <div>' .
+          '          <label for="tags">' . get_string('view_recording_tags', 'bigbluebuttonbn') . '</label><br/>' .
+          '          <input type="text" name="tags" id="recording_tags" value="" placeholder="">' .
+          '        </div>' .
+          '      </fieldset>' .
+          '    </form>' .
+          '  </div>' .
           '</div>';
     }
 
@@ -306,19 +306,19 @@ 
     bigbluebuttonbn_view_recordings($bbbsession);
 }
 
-function bigbluebuttonbn_view_before( $bbbsession ){
+function bigbluebuttonbn_view_before($bbbsession) {
     global $CFG, $DB, $OUTPUT;
 
     echo $OUTPUT->heading(get_string('view_message_conference_not_started', 'bigbluebuttonbn'), 3);
 
     echo '<table>';
     if ($bbbsession['openingtime']) {
-        echo '<tr><td class="c0">'.get_string('mod_form_field_openingtime','bigbluebuttonbn').':</td>';
-        echo '    <td class="c1">'.userdate($bbbsession['openingtime']).'</td></tr>';
+        echo '<tr><td class="c0">' . get_string('mod_form_field_openingtime', 'bigbluebuttonbn') . ':</td>';
+        echo '    <td class="c1">' . userdate($bbbsession['openingtime']) . '</td></tr>';
     }
     if ($bbbsession['closingtime']) {
-        echo '<tr><td class="c0">'.get_string('mod_form_field_closingtime','bigbluebuttonbn').':</td>';
-        echo '    <td class="c1">'.userdate($bbbsession['closingtime']).'</td></tr>';
+        echo '<tr><td class="c0">' . get_string('mod_form_field_closingtime', 'bigbluebuttonbn') . ':</td>';
+        echo '    <td class="c1">' . userdate($bbbsession['closingtime']) . '</td></tr>';
     }
     echo '</table>';
 }
@@ -328,13 +328,13 @@ 
 
     echo $OUTPUT->heading(get_string('view_message_conference_has_ended', 'bigbluebuttonbn'), 3);
 
-    if( !is_null($bbbsession['presentation']['url']) ) {
+    if (!is_null($bbbsession['presentation']['url'])) {
         $attributes = array('title' => $bbbsession['presentation']['name']);
         $icon = new pix_icon($bbbsession['presentation']['icon'], $bbbsession['presentation']['mimetype_description']);
 
-        echo '<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>';
+        echo '<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>';
     }
 
     bigbluebuttonbn_view_recordings($bbbsession);
@@ -343,10 +343,10 @@ 
 function bigbluebuttonbn_view_recordings($bbbsession) {
     global $CFG;
 
-    if( isset($bbbsession['record']) && $bbbsession['record'] ) {
-        $output = html_writer::tag('h4', get_string('view_section_title_recordings', 'bigbluebuttonbn') );
+    if (isset($bbbsession['record']) && $bbbsession['record']) {
+        $output = html_writer::tag('h4', get_string('view_section_title_recordings', 'bigbluebuttonbn'));
 
-        $meetingID='';
+        $meetingID = '';
         $results = bigbluebuttonbn_getRecordedMeetings($bbbsession['course']->id, $bbbsession['bigbluebuttonbn']->id);
 
         //if( $recordingsbn->include_deleted_activities ) {
@@ -354,7 +354,7 @@ 
         //    $results = array_merge($results, $results_deleted);
         //}
 
-        if( $results ){
+        if ($results) {
             //Eliminates duplicates
             $mIDs = array();
             foreach ($results as $result) {
@@ -369,7 +369,7 @@ 
 
         // Get actual recordings
         //$recordings = bigbluebuttonbn_getRecordingsArray($bbbsession['meetingid'], $bbbsession['endpoint'], $bbbsession['shared_secret']);
-        if ( $meetingID != '' ) {
+        if ($meetingID != '') {
             $recordings = bigbluebuttonbn_getRecordingsArray($meetingID, $bbbsession['endpoint'], $bbbsession['shared_secret']);
         } else {
             $recordings = Array();
@@ -377,12 +377,12 @@ 
         // Get recording links
         $recordings_imported = bigbluebuttonbn_getRecordingsImportedArray($bbbsession['course']->id, $bbbsession['bigbluebuttonbn']->id);
         // Merge the recordings
-        $recordings = array_merge( $recordings, $recordings_imported );
+        $recordings = array_merge($recordings, $recordings_imported);
         // Render the table
-        $output .= bigbluebutton_output_recording_table($bbbsession, $recordings)."\n";
+        $output .= bigbluebutton_output_recording_table($bbbsession, $recordings) . "\n";
 
-        if ( $bbbsession['managerecordings'] && bigbluebuttonbn_get_cfg_importrecordings_enabled() ) {
-            $button_import_recordings = html_writer::tag( 'input', '', array('type' => 'button', 'value' => get_string('view_recording_button_import', 'bigbluebuttonbn'), 'onclick' => 'window.location=\''.$CFG->wwwroot.'/mod/bigbluebuttonbn/import_view.php?bn='.$bbbsession['bigbluebuttonbn']->id.'\'') );
+        if ($bbbsession['managerecordings'] && bigbluebuttonbn_get_cfg_importrecordings_enabled()) {
+            $button_import_recordings = html_writer::tag('input', '', array('type' => 'button', 'value' => get_string('view_recording_button_import', 'bigbluebuttonbn'), 'onclick' => 'window.location=\'' . $CFG->wwwroot . '/mod/bigbluebuttonbn/import_view.php?bn=' . $bbbsession['bigbluebuttonbn']->id . '\''));
             $output .= html_writer::start_tag('br');
             $output .= html_writer::tag('span', $button_import_recordings, ['id'=>"import_recording_links_button"]);
             $output .= html_writer::tag('span', '', ['id'=>"import_recording_links_table"]);

Braces +24 added lines, -18 removed lines

@@ -84,8 +84,9 @@ 
 $bbbsession['voicebridge'] = ($bigbluebuttonbn->voicebridge > 0)? 70000 + $bigbluebuttonbn->voicebridge: $bigbluebuttonbn->voicebridge;
 $bbbsession['wait'] = $bigbluebuttonbn->wait;
 $bbbsession['record'] = $bigbluebuttonbn->record;
-if( $bigbluebuttonbn->record )
+if( $bigbluebuttonbn->record ) {
     $bbbsession['welcome'] .= '<br><br>'.get_string('bbbrecordwarning', 'bigbluebuttonbn');
+}
 $bbbsession['tagging'] = $bigbluebuttonbn->tagging;
 
 $bbbsession['openingtime'] = $bigbluebuttonbn->openingtime;
@@ -112,22 +113,24 @@ 
 // Validates if the BigBlueButton server is running
 $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' );
-    else if ( $bbbsession['moderator'] )
-        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 );
-} else {
-    $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getMeetingsURL( $bbbsession['endpoint'], $bbbsession['shared_secret'] ) );
-    if ( !isset($xml) || !isset($xml->returncode) || $xml->returncode == 'FAILED' ){ // The shared secret is wrong
-        if ( $bbbsession['administrator'] )
+    if ( $bbbsession['administrator'] ) {
             print_error( 'view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
-        else if ( $bbbsession['moderator'] )
+    } else if ( $bbbsession['moderator'] ) {
             print_error( 'view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course );
-        else
+    } else {
             print_error( 'view_error_unable_join_student', 'bigbluebuttonbn', $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course );
     }
+    } else {
+    $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getMeetingsURL( $bbbsession['endpoint'], $bbbsession['shared_secret'] ) );
+    if ( !isset($xml) || !isset($xml->returncode) || $xml->returncode == 'FAILED' ){ // The shared secret is wrong
+        if ( $bbbsession['administrator'] ) {
+                    print_error( 'view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
+        } else if ( $bbbsession['moderator'] ) {
+                    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 );
+        }
+    }
 }
 
 // Mark viewed by user (if required)
@@ -203,10 +206,11 @@ 
     }
 
     $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');
+    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.')';
 }
 // Metadata (context)
@@ -362,7 +366,9 @@ 
             }
             //Generates the meetingID string
             foreach ($mIDs as $mID) {
-                if (strlen($meetingID) > 0) $meetingID .= ',';
+                if (strlen($meetingID) > 0) {
+                    $meetingID .= ',';
+                }
                 $meetingID .= $mID;
             }
         }