blindsidenetworks / moodle-mod_bigbluebuttonbn

backup/moodle2/backup_bigbluebuttonbn_activity_task.class.php

Spacing +4 added lines, -4 removed lines

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

backup/moodle2/restore_bigbluebuttonbn_stepslib.php

Spacing +2 added lines, -2 removed lines

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

backup/moodle2/restore_bigbluebuttonbn_activity_task.class.php

Spacing +1 added lines, -1 removed lines

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

backup/moodle2/backup_bigbluebuttonbn_stepslib.php

Spacing +2 added lines, -2 removed lines

@@ -40,12 +40,12 @@
                             'moderatorpass', 'viewerpass', 'wait', 'record', 'tagging',
                             'welcome', 'voicebridge', 'openingtime', 'closingtime',
                             'timecreated', 'timemodified', 'presentation', 'participants',
-                            'userlimit', ));
+                            'userlimit',));
 
         $logs = new backup_nested_element('logs');
 
         $log = new backup_nested_element('log', array('id'), array(
-                'courseid', 'bigbluebuttonbnid', 'userid', 'timecreated', 'meetingid', 'log', 'meta', ));
+                'courseid', 'bigbluebuttonbnid', 'userid', 'timecreated', 'meetingid', 'log', 'meta',));
 
         // Build the tree.
         $bigbluebuttonbn->add_child($logs);

settings.php

Spacing +3 added lines, -3 removed lines

@@ -27,7 +27,7 @@ 
 
 global $CFG;
 
-require_once(dirname(__FILE__).'/locallib.php');
+require_once(dirname(__FILE__) . '/locallib.php');
 
 if ($ADMIN->fulltree) {
     if (!isset($CFG->bigbluebuttonbn['server_url']) ||
@@ -258,8 +258,8 @@ 
         } else {
             $settings->add(new admin_setting_heading('bigbluebuttonbn_feature_preuploadpresentation',
                 get_string('config_feature_preuploadpresentation', 'bigbluebuttonbn'),
-                get_string('config_feature_preuploadpresentation_description', 'bigbluebuttonbn').'<br><br>'.
-                '<div class="form-defaultinfo">'.get_string('config_warning_curl_not_installed', 'bigbluebuttonbn').'</div><br>'
+                get_string('config_feature_preuploadpresentation_description', 'bigbluebuttonbn') . '<br><br>' .
+                '<div class="form-defaultinfo">' . get_string('config_warning_curl_not_installed', 'bigbluebuttonbn') . '</div><br>'
                 ));
         }
     }

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);
@@ -92,12 +92,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 . ']';
         }
         bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getEndMeetingURL($meetingid));
-        redirect('index.php?id='.$id);
+        redirect('index.php?id=' . $id);
     }
 }
 
@@ -114,7 +114,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);
 
@@ -135,13 +135,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;
     }
 
@@ -160,7 +160,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 = '';
@@ -197,7 +197,7 @@ 
         $attendeecount = 0;
         foreach ($meetinginfo['attendees']->attendee as $attendee) {
             if ($attendee->role == $role) {
-                $attendeelist .= ($attendeecount++ > 0 ? ', ' : '').$attendee->fullName;
+                $attendeelist .= ($attendeecount++ > 0 ? ', ' : '') . $attendee->fullName;
             }
         }
     }
@@ -220,15 +220,15 @@ 
     $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;

lib.php

Indentation +9 added lines, -9 removed lines

@@ -208,7 +208,7 @@ 
 
     $logs = $DB->get_records('bigbluebuttonbn_logs',
         array('bigbluebuttonbnid' => $bigbluebuttonbn->id, 'log' => BIGBLUEBUTTONBN_LOG_EVENT_CREATE, 'meta' => "{\"record\":true}")
-      );
+        );
     $log->meta = "{\"has_recordings\":false}";
     if (!empty($logs)) {
         $log->meta = "{\"has_recordings\":true}";
@@ -233,9 +233,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').' '.
@@ -256,9 +256,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;
 }
@@ -331,12 +331,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;

Doc Comments +14 added lines, -1 removed lines

@@ -227,7 +227,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;
@@ -317,6 +317,9 @@ 
     }
 }
 
+/**
+ * @param integer $now
+ */
 function bigbluebuttonbn_print_overview_element($bigbluebuttonbn, $now) {
     global $CFG;
 
@@ -542,6 +545,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;
@@ -558,6 +565,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) {

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];
     }
 
@@ -175,7 +175,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_is_meeting_running($meetingid)) {
@@ -235,11 +235,11 @@ 
     $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');
     }
 
@@ -258,7 +258,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;
 }
@@ -328,16 +328,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;
 }
@@ -425,7 +425,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.\mod_bigbluebuttonbn\locallib\config::get('shared_secret'));
+        $meetingid = sha1($CFG->wwwroot . $bigbluebuttonbn->id . \mod_bigbluebuttonbn\locallib\config::get('shared_secret'));
         $DB->set_field('bigbluebuttonbn', 'meetingid', $meetingid, array('id' => $bigbluebuttonbn->id));
 
         $action = get_string('mod_form_field_notification_msg_created', 'bigbluebuttonbn');
@@ -507,7 +507,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' => $bigbluebuttonbnid));
@@ -564,7 +564,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()) {

import_view.php

Indentation +2 added lines, -2 removed lines

@@ -96,8 +96,8 @@
     $output .= html_writer::start_tag('br');
     $output .= html_writer::tag('input', '',
         array('type' => 'button', 'class' => 'btn btn-secondary',
-              'value' => get_string('view_recording_button_return', 'bigbluebuttonbn'),
-              'onclick' => 'window.location=\''.$CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$cm->id.'\''));
+                'value' => get_string('view_recording_button_return', 'bigbluebuttonbn'),
+                'onclick' => 'window.location=\''.$CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$cm->id.'\''));
 
     // JavaScript for locales.
     $PAGE->requires->strings_for_js(array_keys(bigbluebuttonbn_get_strings_for_js()), 'bigbluebuttonbn');

Spacing +3 added lines, -3 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');
 
 $bn = required_param('bn', PARAM_INT);
 $tc = optional_param('tc', 0, PARAM_INT);
@@ -97,7 +97,7 @@ 
     $output .= html_writer::tag('input', '',
         array('type' => 'button', 'class' => 'btn btn-secondary',
               'value' => get_string('view_recording_button_return', 'bigbluebuttonbn'),
-              'onclick' => 'window.location=\''.$CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$cm->id.'\''));
+              'onclick' => 'window.location=\'' . $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $cm->id . '\''));
 
     // JavaScript for locales.
     $PAGE->requires->strings_for_js(array_keys(bigbluebuttonbn_get_strings_for_js()), 'bigbluebuttonbn');

bbb_view.php

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);
@@ -97,7 +97,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;
         }
 
@@ -113,17 +113,17 @@ 
             // 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;
         }
 
@@ -158,7 +158,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();
@@ -201,7 +201,7 @@ 
     }
     if ($durationtime > 0) {
         $data['duration'] = $durationtime;
-        $data['welcome'] .= '<br><br>'.str_replace('%duration%', ''.$durationtime, get_string('bbbdurationwarning', 'bigbluebuttonbn'));
+        $data['welcome'] .= '<br><br>' . str_replace('%duration%', '' . $durationtime, get_string('bbbdurationwarning', 'bigbluebuttonbn'));
     }
 
     $voicebridge = intval($bbbsession['voicebridge']);
@@ -247,7 +247,7 @@ 
 
     if ($bbbsession['userlimit'] > 0 && intval($meetinginfo['participantCount']) >= $bbbsession['userlimit']) {
         // No more users allowed to join.
-        header('Location: '.$bbbsession['logoutURL']);
+        header('Location: ' . $bbbsession['logoutURL']);
         return;
     }
 
@@ -266,20 +266,20 @@ 
     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();
 }

Switch Indentation +81 added lines, -81 removed lines

@@ -62,106 +62,106 @@
 }
 
 switch (strtolower($action)) {
-    case 'logout':
-        if (isset($errors) && $errors != '') {
-            bigbluebutton_bbb_view_errors($errors, $id);
-            break;
-        }
+        case 'logout':
+            if (isset($errors) && $errors != '') {
+                bigbluebutton_bbb_view_errors($errors, $id);
+                break;
+            }
 
-        if (is_null($bbbsession)) {
-            bigbluebutton_bbb_view_close_window_manually();
-            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);
+            // 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);
+            // 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');
+            // 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;
-        }
+            // 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']);
-            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 (!$response) {
-            // The server is unreachable.
-            if ($bbbsession['administrator']) {
-                print_error('view_error_unable_join', 'bigbluebuttonbn',
-                    $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+            // 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;
             }
-            if ($bbbsession['moderator']) {
-                print_error('view_error_unable_join_teacher', 'bigbluebuttonbn',
+
+            // 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 (!$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');
+                    break;
+                }
+                $printerrorkey = bigbluebuttonbn_get_error_key($response['messageKey'], 'view_error_create');
+                print_error($printerrorkey, 'bigbluebuttonbn');
+                break;
+            }
 
-        if ($response['returncode'] == 'FAILED') {
-            // The meeting was not created.
-            if (!$printerrorkey) {
-                print_error($response['message'], 'bigbluebuttonbn');
+            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');
-            break;
-        }
 
-        if ($response['hasBeenForciblyEnded'] == 'true') {
-            print_error(get_string('index_error_forciblyended', '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;
+        case 'playback':
+            if ($href == '') {
+                bigbluebutton_bbb_view_close_window();
+                return;
+            }
+            // 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 'playback':
-        if ($href == '') {
+        default:
             bigbluebutton_bbb_view_close_window();
-            return;
-        }
-        // 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();
 }
 
 function bigbluebutton_bbb_view_close_window() {

Indentation +14 added lines, -14 removed lines

@@ -178,10 +178,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'] = 'false';
@@ -221,14 +221,14 @@ 
     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')) {
@@ -236,8 +236,8 @@ 
             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'];