blindsidenetworks / moodle-mod_bigbluebuttonbn

lib.php

Doc Comments +12 added lines, -4 removed lines

@@ -219,7 +219,7 @@ 
  * $return->time = the time they did it
  * $return->info = a short text description.
  *
- * @return bool
+ * @return string
  */
 function bigbluebuttonbn_user_outline($course, $user, $mod, $bigbluebuttonbn) {
     global $DB;
@@ -462,9 +462,7 @@ 
  * that was uploaded, or if there is none, set the
  * presentation field to blank.
  *
- * @param int      $bigbluebuttonbnid the bigbluebuttonbn id
- * @param stdClass $context            the context
- * @param int      $draftitemid        the draft item
+ * @param int      $bigbluebuttonbn the bigbluebuttonbn id
  */
 function bigbluebuttonbn_update_media_file(&$bigbluebuttonbn) {
     global $DB;
@@ -517,6 +515,10 @@ 
     send_stored_file($file, 0, 0, $forcedownload, $options); // download MUST be forced - security!
 }
 
+/**
+ * @param stdClass $context
+ * @param string $filearea
+ */
 function bigbluebuttonbn_pluginfile_valid($context, $filearea) {
     if ($context->contextlevel != CONTEXT_MODULE) {
         return false;
@@ -533,6 +535,12 @@ 
     return true;
 }
 
+/**
+ * @param stdClass $course
+ * @param stdClass $cm
+ * @param stdClass $context
+ * @param string $filearea
+ */
 function bigbluebuttonbn_pluginfile_file($course, $cm, $context, $filearea, $args) {
     $filename = bigbluebuttonbn_pluginfile_filename($course, $cm, $context, $args);
     if (!$filename) {

Indentation +8 added lines, -8 removed lines

@@ -224,9 +224,9 @@ 
 function bigbluebuttonbn_user_outline($course, $user, $mod, $bigbluebuttonbn) {
     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.' '.
@@ -244,9 +244,9 @@ 
 function bigbluebuttonbn_user_complete($course, $user, $mod, $bigbluebuttonbn) {
     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;
 }
 
@@ -317,12 +317,12 @@ 
     $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";
     return $str;
 }

Spacing +45 added lines, -45 removed lines

@@ -27,32 +27,32 @@ 
 
 global $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');
-
-if (file_exists(dirname(__FILE__).'/vendor/firebase/php-jwt/src/JWT.php')) {
-    require_once(dirname(__FILE__).'/vendor/firebase/php-jwt/src/JWT.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');
+
+if (file_exists(dirname(__FILE__) . '/vendor/firebase/php-jwt/src/JWT.php')) {
+    require_once(dirname(__FILE__) . '/vendor/firebase/php-jwt/src/JWT.php');
 }
 
 if (!isset($CFG->bigbluebuttonbn)) {
     $CFG->bigbluebuttonbn = array();
 }
 
-if (file_exists(dirname(__FILE__).'/config.php')) {
-    require_once(dirname(__FILE__).'/config.php');
+if (file_exists(dirname(__FILE__) . '/config.php')) {
+    require_once(dirname(__FILE__) . '/config.php');
     // Old BigBlueButtonBN cfg schema. For backward compatibility.
     global $BIGBLUEBUTTONBN_CFG;
 
     if (isset($BIGBLUEBUTTONBN_CFG)) {
-        foreach ((array) $BIGBLUEBUTTONBN_CFG as $key => $value) {
+        foreach ((array)$BIGBLUEBUTTONBN_CFG as $key => $value) {
             $cfgkey = str_replace("bigbluebuttonbn_", "", $key);
             $CFG->bigbluebuttonbn[$cfgkey] = $value;
         }
@@ -83,19 +83,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];
     }
 
@@ -125,7 +125,7 @@ 
         $DB->set_field('bigbluebuttonbn', 'meetingid', $meetingid, array('id' => $data->id));
         // Assuming the inserts work, we get to the following line.
         $transaction->allow_commit();
-    } catch(Exception $e) {
+    } catch (Exception $e) {
         $transaction->rollback($e);
     }
     if ($data->id) {
@@ -154,7 +154,7 @@ 
         // Assuming the inserts work, we get to the following line.
         $transaction->allow_commit();
         $updated = true;
-    } catch(Exception $e) {
+    } catch (Exception $e) {
         $transaction->rollback($e);
         $updated = false;
     }
@@ -226,10 +226,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.' '.
+        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 '';
@@ -246,7 +246,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;
 }
 
@@ -314,16 +314,16 @@ 
     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";
     return $str;
 }
 
@@ -482,7 +482,7 @@ 
     if (count($files) == 1) {
         // Get the first (and only) file.
         $file = reset($files);
-        $filesrc = '/'.$file->get_filename();
+        $filesrc = '/' . $file->get_filename();
     }
     // Set the presentation column in the bigbluebuttonbn table.
     $DB->set_field('bigbluebuttonbn', 'presentation', $filesrc, array('id' => $bigbluebuttonbn->id));
@@ -539,7 +539,7 @@ 
         return false;
     }
 
-    $fullpath = "/$context->id/mod_bigbluebuttonbn/$filearea/0/".$filename;
+    $fullpath = "/$context->id/mod_bigbluebuttonbn/$filearea/0/" . $filename;
     $fs = get_file_storage();
     $file = $fs->get_file_by_hash(sha1($fullpath));
     if (!$file || $file->is_directory()) {

bbb_view.php

Indentation +15 added lines, -15 removed lines

@@ -182,10 +182,10 @@ 
 
 function bigbluebutton_bbb_view_create_meeting_data(&$bbbsession, $bigbluebuttonbn) {
     $data = ['meetingID' => $bbbsession['meetingid'],
-              'name' => $bbbsession['meetingname'],
-              'attendeePW' => $bbbsession['viewerPW'],
-              'moderatorPW' => $bbbsession['modPW'],
-              'logoutURL' => $bbbsession['logoutURL'],
+                'name' => $bbbsession['meetingname'],
+                'attendeePW' => $bbbsession['viewerPW'],
+                'moderatorPW' => $bbbsession['modPW'],
+                'logoutURL' => $bbbsession['logoutURL'],
             ];
     $data['record'] = bigbluebutton_bbb_view_create_meeting_data_record($bbbsession['record']);
     $data['welcome'] = trim($bbbsession['welcome']);
@@ -198,7 +198,7 @@ 
             '%duration%',
             (string) $durationtime,
             get_string('bbbdurationwarning', 'bigbluebuttonbn')
-          );
+            );
     }
     $voicebridge = intval($bbbsession['voicebridge']);
     if ($voicebridge > 0 && $voicebridge < 79999) {
@@ -228,22 +228,22 @@ 
 function bigbluebutton_bbb_view_create_meeting_metadata(&$bbbsession) {
     global $USER;
     $metadata = ['bbb-origin' => $bbbsession['origin'],
-                 'bbb-origin-version' => $bbbsession['originVersion'],
-                 'bbb-origin-server-name' => $bbbsession['originServerName'],
-                 'bbb-origin-server-common-name' => $bbbsession['originServerCommonName'],
-                 'bbb-origin-tag' => $bbbsession['originTag'],
-                 'bbb-context' => $bbbsession['course']->fullname,
-                 'bbb-recording-name' => $bbbsession['meetingname'],
-                 'bbb-recording-description' => bigbluebuttonbn_html2text($bbbsession['meetingdescription'], 64),
-                 'bbb-recording-tags' => bigbluebuttonbn_get_tags($bbbsession['cm']->id), // Same as $id.
+                    'bbb-origin-version' => $bbbsession['originVersion'],
+                    'bbb-origin-server-name' => $bbbsession['originServerName'],
+                    'bbb-origin-server-common-name' => $bbbsession['originServerCommonName'],
+                    'bbb-origin-tag' => $bbbsession['originTag'],
+                    'bbb-context' => $bbbsession['course']->fullname,
+                    'bbb-recording-name' => $bbbsession['meetingname'],
+                    'bbb-recording-description' => bigbluebuttonbn_html2text($bbbsession['meetingdescription'], 64),
+                    'bbb-recording-tags' => bigbluebuttonbn_get_tags($bbbsession['cm']->id), // Same as $id.
                 ];
     if ((boolean)\mod_bigbluebuttonbn\locallib\config::get('recordingstatus_enabled')) {
         $metadata["bn-recording-status"] = json_encode(
             array(
                 'email' => array('"' . fullname($USER) . '" <' . $USER->email . '>'),
                 'context' => $bbbsession['bigbluebuttonbnURL']
-              )
-          );
+                )
+            );
     }
     if ((boolean)\mod_bigbluebuttonbn\locallib\config::get('recordingready_enabled')) {
         $metadata['bn-recording-ready-url'] = $bbbsession['recordingReadyURL'];

Switch Indentation +75 added lines, -75 removed lines

@@ -61,91 +61,91 @@
 }
 
 switch (strtolower($action)) {
-    case 'logout':
-        if (isset($errors) && $errors != '') {
-            bigbluebutton_bbb_view_errors($errors, $id);
-            break;
-        }
-        if (is_null($bbbsession)) {
-            bigbluebutton_bbb_view_close_window_manually();
-            break;
-        }
-        // Moodle event logger: Create an event for meeting left.
-        bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_LEFT, $bigbluebuttonbn, $cm);
-        // Update the cache.
-        $meetinginfo = bigbluebuttonbn_get_meeting_info($bbbsession['meetingid'], BIGBLUEBUTTONBN_FORCED);
-        // Close the tab or window where BBB was opened.
-        bigbluebutton_bbb_view_close_window();
-        break;
-    case 'join':
-        if (is_null($bbbsession)) {
-            print_error('view_error_unable_join', 'bigbluebuttonbn');
-            break;
-        }
-        // See if the session is in progress.
-        if (bigbluebuttonbn_is_meeting_running($bbbsession['meetingid'])) {
-            // Since the meeting is already running, we just join the session.
-            bigbluebutton_bbb_view_join_meeting($bbbsession, $cm, $bigbluebuttonbn);
-            break;
-        }
-        // 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']);
+        case 'logout':
+            if (isset($errors) && $errors != '') {
+                bigbluebutton_bbb_view_errors($errors, $id);
+                break;
+            }
+            if (is_null($bbbsession)) {
+                bigbluebutton_bbb_view_close_window_manually();
+                break;
+            }
+            // Moodle event logger: Create an event for meeting left.
+            bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_LEFT, $bigbluebuttonbn, $cm);
+            // Update the cache.
+            $meetinginfo = bigbluebuttonbn_get_meeting_info($bbbsession['meetingid'], BIGBLUEBUTTONBN_FORCED);
+            // Close the tab or window where BBB was opened.
+            bigbluebutton_bbb_view_close_window();
             break;
-        }
-        // As the meeting doesn't exist, try to create it.
-        $response = bigbluebuttonbn_get_create_meeting_array(
-            bigbluebutton_bbb_view_create_meeting_data($bbbsession, $bigbluebuttonbn),
-            bigbluebutton_bbb_view_create_meeting_metadata($bbbsession),
-            $bbbsession['presentation']['name'],
-            $bbbsession['presentation']['url']
-        );
-        if (empty($response)) {
-            // The server is unreachable.
-            if ($bbbsession['administrator']) {
-                print_error('view_error_unable_join', 'bigbluebuttonbn',
-                    $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+        case 'join':
+            if (is_null($bbbsession)) {
+                print_error('view_error_unable_join', 'bigbluebuttonbn');
+                break;
+            }
+            // See if the session is in progress.
+            if (bigbluebuttonbn_is_meeting_running($bbbsession['meetingid'])) {
+                // Since the meeting is already running, we just join the session.
+                bigbluebutton_bbb_view_join_meeting($bbbsession, $cm, $bigbluebuttonbn);
                 break;
             }
-            if ($bbbsession['moderator']) {
-                print_error('view_error_unable_join_teacher', 'bigbluebuttonbn',
+            // If user is not administrator nor moderator (user is steudent) and waiting is required.
+            if (!$bbbsession['administrator'] && !$bbbsession['moderator'] && $bbbsession['wait']) {
+                header('Location: '.$bbbsession['logoutURL']);
+                break;
+            }
+            // As the meeting doesn't exist, try to create it.
+            $response = bigbluebuttonbn_get_create_meeting_array(
+                bigbluebutton_bbb_view_create_meeting_data($bbbsession, $bigbluebuttonbn),
+                bigbluebutton_bbb_view_create_meeting_metadata($bbbsession),
+                $bbbsession['presentation']['name'],
+                $bbbsession['presentation']['url']
+            );
+            if (empty($response)) {
+                // The server is unreachable.
+                if ($bbbsession['administrator']) {
+                    print_error('view_error_unable_join', 'bigbluebuttonbn',
+                        $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+                    break;
+                }
+                if ($bbbsession['moderator']) {
+                    print_error('view_error_unable_join_teacher', 'bigbluebuttonbn',
+                        $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+                    break;
+                }
+                print_error('view_error_unable_join_student', 'bigbluebuttonbn',
                     $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
                 break;
             }
-            print_error('view_error_unable_join_student', 'bigbluebuttonbn',
-                $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
-            break;
-        }
-        if ($response['returncode'] == 'FAILED') {
-            // The meeting was not created.
-            if (!$printerrorkey) {
-                print_error($response['message'], 'bigbluebuttonbn');
+            if ($response['returncode'] == 'FAILED') {
+                // The meeting was not created.
+                if (!$printerrorkey) {
+                    print_error($response['message'], 'bigbluebuttonbn');
+                    break;
+                }
+                $printerrorkey = bigbluebuttonbn_get_error_key($response['messageKey'], 'view_error_create');
+                print_error($printerrorkey, 'bigbluebuttonbn');
+                break;
+            }
+            if ($response['hasBeenForciblyEnded'] == 'true') {
+                print_error(get_string('index_error_forciblyended', 'bigbluebuttonbn'));
                 break;
             }
-            $printerrorkey = bigbluebuttonbn_get_error_key($response['messageKey'], 'view_error_create');
-            print_error($printerrorkey, 'bigbluebuttonbn');
+            // Moodle event logger: Create an event for meeting created.
+            bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_CREATED, $bigbluebuttonbn, $cm);
+            // Internal logger: Insert a record with the meeting created.
+            bigbluebuttonbn_logs($bbbsession, BIGBLUEBUTTONBN_LOG_EVENT_CREATE);
+            // Since the meeting is already running, we just join the session.
+            bigbluebutton_bbb_view_join_meeting($bbbsession, $cm, $bigbluebuttonbn);
             break;
-        }
-        if ($response['hasBeenForciblyEnded'] == 'true') {
-            print_error(get_string('index_error_forciblyended', 'bigbluebuttonbn'));
+        case 'play':
+            $href = bigbluebutton_bbb_view_playback_href($href, $mid, $rid, $rtype);
+            // Moodle event logger: Create an event for meeting left.
+            bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_RECORDING_VIEWED, $bigbluebuttonbn, $cm, ['other' => $rid]);
+            // Execute the redirect.
+            header('Location: '.urldecode($href));
             break;
-        }
-        // Moodle event logger: Create an event for meeting created.
-        bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_CREATED, $bigbluebuttonbn, $cm);
-        // Internal logger: Insert a record with the meeting created.
-        bigbluebuttonbn_logs($bbbsession, BIGBLUEBUTTONBN_LOG_EVENT_CREATE);
-        // Since the meeting is already running, we just join the session.
-        bigbluebutton_bbb_view_join_meeting($bbbsession, $cm, $bigbluebuttonbn);
-        break;
-    case 'play':
-        $href = bigbluebutton_bbb_view_playback_href($href, $mid, $rid, $rtype);
-        // Moodle event logger: Create an event for meeting left.
-        bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_RECORDING_VIEWED, $bigbluebuttonbn, $cm, ['other' => $rid]);
-        // Execute the redirect.
-        header('Location: '.urldecode($href));
-        break;
-    default:
-        bigbluebutton_bbb_view_close_window();
+        default:
+            bigbluebutton_bbb_view_close_window();
 }
 
 function bigbluebutton_bbb_view_playback_href($href, $mid, $rid, $rtype) {

Spacing +13 added lines, -13 removed lines

@@ -22,8 +22,8 @@ 
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v2 or later
  */
 
-require_once(dirname(dirname(dirname(__FILE__))).'/config.php');
-require_once(dirname(__FILE__).'/locallib.php');
+require_once(dirname(dirname(dirname(__FILE__))) . '/config.php');
+require_once(dirname(__FILE__) . '/locallib.php');
 
 $action = required_param('action', PARAM_TEXT);
 $id = optional_param('id', 0, PARAM_INT);
@@ -90,7 +90,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;
         }
         // As the meeting doesn't exist, try to create it.
@@ -104,16 +104,16 @@ 
             // The server is unreachable.
             if ($bbbsession['administrator']) {
                 print_error('view_error_unable_join', 'bigbluebuttonbn',
-                    $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+                    $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
                 break;
             }
             if ($bbbsession['moderator']) {
                 print_error('view_error_unable_join_teacher', 'bigbluebuttonbn',
-                    $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+                    $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
                 break;
             }
             print_error('view_error_unable_join_student', 'bigbluebuttonbn',
-                $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+                $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
             break;
         }
         if ($response['returncode'] == 'FAILED') {
@@ -142,7 +142,7 @@ 
         // Moodle event logger: Create an event for meeting left.
         bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_RECORDING_VIEWED, $bigbluebuttonbn, $cm, ['other' => $rid]);
         // Execute the redirect.
-        header('Location: '.urldecode($href));
+        header('Location: ' . urldecode($href));
         break;
     default:
         bigbluebutton_bbb_view_close_window();
@@ -196,7 +196,7 @@ 
         $data['welcome'] .= '<br><br>';
         $data['welcome'] .= str_replace(
             '%duration%',
-            (string) $durationtime,
+            (string)$durationtime,
             get_string('bbbdurationwarning', 'bigbluebuttonbn')
           );
     }
@@ -259,7 +259,7 @@ 
     $meetinginfo = bigbluebuttonbn_get_meeting_info($bbbsession['meetingid'], BIGBLUEBUTTONBN_FORCED);
     if ($bbbsession['userlimit'] > 0 && intval($meetinginfo['participantCount']) >= $bbbsession['userlimit']) {
         // No more users allowed to join.
-        header('Location: '.$bbbsession['logoutURL']);
+        header('Location: ' . $bbbsession['logoutURL']);
         return;
     }
     // Build the URL.
@@ -277,18 +277,18 @@ 
     bigbluebuttonbn_participant_joined($bbbsession['meetingid'],
         ($bbbsession['administrator'] || $bbbsession['moderator']));
     // Execute the redirect.
-    header('Location: '.$joinurl);
+    header('Location: ' . $joinurl);
 }
 
 function bigbluebutton_bbb_view_errors($serrors, $id) {
     global $CFG, $OUTPUT;
-    $errors = (array) json_decode(urldecode($serrors));
+    $errors = (array)json_decode(urldecode($serrors));
     $msgerrors = '';
     foreach ($errors as $error) {
-        $msgerrors .= html_writer::tag('p', $error->{'message'}, array('class' => 'alert alert-danger'))."\n";
+        $msgerrors .= html_writer::tag('p', $error->{'message'}, array('class' => 'alert alert-danger')) . "\n";
     }
     echo $OUTPUT->header();
     print_error('view_error_bigbluebutton', 'bigbluebuttonbn',
-        $CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$id, $msgerrors, $serrors);
+        $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $id, $msgerrors, $serrors);
     echo $OUTPUT->footer();
 }

index.php

Spacing +18 added lines, -18 removed lines

@@ -23,8 +23,8 @@ 
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v2 or later
  */
 
-require_once(dirname(dirname(dirname(__FILE__))).'/config.php');
-require_once(dirname(__FILE__).'/locallib.php');
+require_once(dirname(dirname(dirname(__FILE__))) . '/config.php');
+require_once(dirname(__FILE__) . '/locallib.php');
 
 $id = required_param('id', PARAM_INT);
 $a = optional_param('a', 0, PARAM_INT);
@@ -87,12 +87,12 @@ 
     if ($moderator || $administrator) {
         bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_ENDED, $bigbluebuttonbn, $cm);
         echo get_string('index_ending', 'bigbluebuttonbn');
-        $meetingid = $bigbluebuttonbn->meetingid.'-'.$course->id.'-'.$bigbluebuttonbn->id;
+        $meetingid = $bigbluebuttonbn->meetingid . '-' . $course->id . '-' . $bigbluebuttonbn->id;
         if ($g != '0') {
-            $meetingid .= '['.$g.']';
+            $meetingid .= '[' . $g . ']';
         }
         bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getEndMeetingURL($meetingid));
-        redirect('index.php?id='.$id);
+        redirect('index.php?id=' . $id);
     }
 }
 
@@ -106,7 +106,7 @@ 
         // Add a the data for the bigbluebuttonbn instance.
         $groupobj = null;
         if (groups_get_activity_groupmode($cm) > 0) {
-            $groupobj = (object) array('id' => 0, 'name' => get_string('allparticipants'));
+            $groupobj = (object)array('id' => 0, 'name' => get_string('allparticipants'));
         }
         $table->data[] = bigbluebuttonbn_index_display_room($canmoderate, $course, $bigbluebuttonbn, $groupobj);
         // Add a the data for the groups belonging to the bigbluebuttonbn instance, if any.
@@ -124,12 +124,12 @@ 
 
 // 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;
     }
     $meetinginfo = bigbluebuttonbn_get_meeting_info_array($meetingid);
@@ -144,7 +144,7 @@ 
         return;
     }
     // 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 = '';
@@ -176,7 +176,7 @@ 
         $attendeecount = 0;
         foreach ($meetinginfo['attendees']->attendee as $attendee) {
             if ($attendee->role == $role) {
-                $attendeelist .= ($attendeecount++ > 0 ? ', ' : '').$attendee->fullName;
+                $attendeelist .= ($attendeecount++ > 0 ? ', ' : '') . $attendee->fullName;
             }
         }
     }
@@ -195,15 +195,15 @@ 
 function bigbluebuttonbn_index_display_room_actions($moderator, $course, $bigbluebuttonbn, $groupobj = null) {
     $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;
 }