blindsidenetworks / moodle-mod_bigbluebuttonbn

index.php

Doc Comments +3 added lines

@@ -206,6 +206,9 @@
     return $users;
 }
 
+/**
+ * @param string $role
+ */
 function displayBigBlueButtonRoomsAttendeeList($meetingInfo, $role)
 {
     $attendeeList = '';

Spacing +16 added lines, -16 removed lines

@@ -22,8 +22,8 @@ 
  * @copyright 2010-2015 Blindside Networks Inc
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v2 or later
  */
-require_once dirname(dirname(dirname(__FILE__))).'/config.php';
-require_once dirname(__FILE__).'/locallib.php';
+require_once dirname(dirname(dirname(__FILE__))) . '/config.php';
+require_once dirname(__FILE__) . '/locallib.php';
 
 $id = required_param('id', PARAM_INT); // Course Module ID, or
 $a = optional_param('a', 0, PARAM_INT); // bigbluebuttonbn instance ID
@@ -101,13 +101,13 @@ 
 
         echo get_string('index_ending', 'bigbluebuttonbn');
 
-        $meetingID = $bigbluebuttonbn->meetingid.'-'.$course->id.'-'.$bigbluebuttonbn->id;
+        $meetingID = $bigbluebuttonbn->meetingid . '-' . $course->id . '-' . $bigbluebuttonbn->id;
         if ($g != '0') {
-            $getArray = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getEndMeetingURL($meetingID.'['.$g.']', $bigbluebuttonbn->moderatorpass, $endpoint, $shared_secret));
+            $getArray = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getEndMeetingURL($meetingID . '[' . $g . ']', $bigbluebuttonbn->moderatorpass, $endpoint, $shared_secret));
         } else {
             $getArray = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getEndMeetingURL($meetingID, $bigbluebuttonbn->moderatorpass, $endpoint, $shared_secret));
         }
-        redirect('index.php?id='.$id);
+        redirect('index.php?id=' . $id);
     }
 }
 
@@ -127,7 +127,7 @@ 
         $can_moderate = ($administrator || $moderator);
 
         //Add a the data for the bigbluebuttonbn instance
-        $groupObj = (groups_get_activity_groupmode($cm) > 0) ? (object) array('id' => 0, 'name' => get_string('allparticipants')) : null;
+        $groupObj = (groups_get_activity_groupmode($cm) > 0) ? (object)array('id' => 0, 'name' => get_string('allparticipants')) : null;
         $table->data[] = displayBigBlueButtonRooms($endpoint, $shared_secret, $can_moderate, $course, $bigbluebuttonbn, $groupObj);
 
         //Add a the data for the groups belonging to the bigbluebuttonbn instance, if any
@@ -148,9 +148,9 @@ 
 /// Functions
 function displayBigBlueButtonRooms($endpoint, $shared_secret, $moderator, $course, $bigbluebuttonbn, $groupObj = null)
 {
-    $meetingID = $bigbluebuttonbn->meetingid.'-'.$course->id.'-'.$bigbluebuttonbn->id;
+    $meetingID = $bigbluebuttonbn->meetingid . '-' . $course->id . '-' . $bigbluebuttonbn->id;
     if ($groupObj) {
-        $meetingID .= '['.$groupObj->id.']';
+        $meetingID .= '[' . $groupObj->id . ']';
         $group = $groupObj->name;
     }
 
@@ -174,7 +174,7 @@ 
 
     // Output Users in the meeting
 
-    $joinURL = '<a href="view.php?id='.$bigbluebuttonbn->coursemodule.($groupObj ? '&group='.$groupObj->id : '').'">'.format_string($bigbluebuttonbn->name).'</a>';
+    $joinURL = '<a href="view.php?id=' . $bigbluebuttonbn->coursemodule . ($groupObj ? '&group=' . $groupObj->id : '') . '">' . format_string($bigbluebuttonbn->name) . '</a>';
     $group = '';
     $users = '';
     $viewerList = '';
@@ -214,7 +214,7 @@ 
         $attendee_count = 0;
         foreach ($meetingInfo['attendees']->attendee as $attendee) {
             if ($attendee->role == $role) {
-                $attendeeList .= ($attendee_count++ > 0 ? ', ' : '').$attendee->fullName;
+                $attendeeList .= ($attendee_count++ > 0 ? ', ' : '') . $attendee->fullName;
             }
         }
     }
@@ -238,14 +238,14 @@ 
     $actions = '';
 
     if ($moderator) {
-        $actions .= '<form name="form1" method="post" action="">'.'/n';
-        $actions .= '  <INPUT type="hidden" name="id" value="'.$course->id.'">'.'/n';
-        $actions .= '  <INPUT type="hidden" name="a" value="'.$bigbluebuttonbn->id.'">'.'/n';
+        $actions .= '<form name="form1" method="post" action="">' . '/n';
+        $actions .= '  <INPUT type="hidden" name="id" value="' . $course->id . '">' . '/n';
+        $actions .= '  <INPUT type="hidden" name="a" value="' . $bigbluebuttonbn->id . '">' . '/n';
         if ($groupObj != null) {
-            $actions .= '  <INPUT type="hidden" name="g" value="'.$groupObj->id.'">'.'/n';
+            $actions .= '  <INPUT type="hidden" name="g" value="' . $groupObj->id . '">' . '/n';
         }
-        $actions .= '  <INPUT type="submit" name="submit" value="end" onclick="return confirm(\''.get_string('index_confirm_end', 'bigbluebuttonbn').'\')">'.'/n';
-        $actions .= '</form>'.'/n';
+        $actions .= '  <INPUT type="submit" name="submit" value="end" onclick="return confirm(\'' . get_string('index_confirm_end', 'bigbluebuttonbn') . '\')">' . '/n';
+        $actions .= '</form>' . '/n';
     }
 
     return $actions;

lib.php

Doc Comments +9 added lines, -1 removed lines

@@ -220,7 +220,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)
 {
@@ -544,6 +544,11 @@ 
     send_stored_file($file, 0, 0, $forcedownload, $options); // download MUST be forced - security!
 }
 
+/**
+ * @param stdClass $course
+ * @param stdClass $cm
+ * @param stdClass $context
+ */
 function bigbluebuttonbn_pluginfile_filename($course, $cm, $context, $args)
 {
     global $DB;
@@ -642,6 +647,9 @@ 
     bigbluebuttonbn_notification_send($USER, $bigbluebuttonbn, bigbluebuttonbn_notification_msg_html($msg));
 }
 
+/**
+ * @param stdClass $msg
+ */
 function bigbluebuttonbn_notification_msg_html($msg)
 {
     $message_text = '<p>'.$msg->activity_type.' &quot;'.$msg->activity_title.'&quot; '.get_string('email_body_notification_meeting_has_been', 'bigbluebuttonbn').' '.$msg->action.'.</p>'."\n";

Indentation +8 added lines, -8 removed lines

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

Spacing +61 added lines, -61 removed lines

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

locallib.php

Doc Comments +4 added lines

@@ -1343,6 +1343,10 @@
     return $row;
 }
 
+/**
+ * @param string $manage_action
+ * @param string $manage_tag
+ */
 function bigbluebuttonbn_actionbar_render($manage_action, $manage_tag, $recording)
 {
     global $OUTPUT;

Indentation +20 added lines, -20 removed lines

@@ -1765,24 +1765,24 @@ 
 function bigbluebuttonbn_get_cfg_options()
 {
     return [
-          'voicebridge_editable' => bigbluebuttonbn_get_cfg_voicebridge_editable(),
-          'recording_default' => bigbluebuttonbn_get_cfg_recording_default(),
-          'recording_editable' => bigbluebuttonbn_get_cfg_recording_editable(),
-          'recording_tagging_default' => bigbluebuttonbn_get_cfg_recording_tagging_default(),
-          'recording_tagging_editable' => bigbluebuttonbn_get_cfg_recording_tagging_editable(),
-          'waitformoderator_default' => bigbluebuttonbn_get_cfg_waitformoderator_default(),
-          'waitformoderator_editable' => bigbluebuttonbn_get_cfg_waitformoderator_editable(),
-          'userlimit_default' => bigbluebuttonbn_get_cfg_userlimit_default(),
-          'userlimit_editable' => bigbluebuttonbn_get_cfg_userlimit_editable(),
-          'preuploadpresentation_enabled' => bigbluebuttonbn_get_cfg_preuploadpresentation_enabled(),
-          'sendnotifications_enabled' => bigbluebuttonbn_get_cfg_sendnotifications_enabled(),
-          'recordings_html_default' => bigbluebuttonbn_get_cfg_recordings_html_default(),
-          'recordings_html_editable' => bigbluebuttonbn_get_cfg_recordings_html_editable(),
-          'recordings_deleted_activities_default' => bigbluebuttonbn_get_cfg_recordings_deleted_activities_default(),
-          'recordings_deleted_activities_editable' => bigbluebuttonbn_get_cfg_recordings_deleted_activities_editable(),
-          'recording_icons_enabled' => bigbluebuttonbn_get_cfg_recording_icons_enabled(),
-          'instance_type_enabled' => bigbluebuttonbn_recordings_enabled(),
-          'instance_type_default' => BIGBLUEBUTTONBN_TYPE_ALL,
+            'voicebridge_editable' => bigbluebuttonbn_get_cfg_voicebridge_editable(),
+            'recording_default' => bigbluebuttonbn_get_cfg_recording_default(),
+            'recording_editable' => bigbluebuttonbn_get_cfg_recording_editable(),
+            'recording_tagging_default' => bigbluebuttonbn_get_cfg_recording_tagging_default(),
+            'recording_tagging_editable' => bigbluebuttonbn_get_cfg_recording_tagging_editable(),
+            'waitformoderator_default' => bigbluebuttonbn_get_cfg_waitformoderator_default(),
+            'waitformoderator_editable' => bigbluebuttonbn_get_cfg_waitformoderator_editable(),
+            'userlimit_default' => bigbluebuttonbn_get_cfg_userlimit_default(),
+            'userlimit_editable' => bigbluebuttonbn_get_cfg_userlimit_editable(),
+            'preuploadpresentation_enabled' => bigbluebuttonbn_get_cfg_preuploadpresentation_enabled(),
+            'sendnotifications_enabled' => bigbluebuttonbn_get_cfg_sendnotifications_enabled(),
+            'recordings_html_default' => bigbluebuttonbn_get_cfg_recordings_html_default(),
+            'recordings_html_editable' => bigbluebuttonbn_get_cfg_recordings_html_editable(),
+            'recordings_deleted_activities_default' => bigbluebuttonbn_get_cfg_recordings_deleted_activities_default(),
+            'recordings_deleted_activities_editable' => bigbluebuttonbn_get_cfg_recordings_deleted_activities_editable(),
+            'recording_icons_enabled' => bigbluebuttonbn_get_cfg_recording_icons_enabled(),
+            'instance_type_enabled' => bigbluebuttonbn_recordings_enabled(),
+            'instance_type_default' => BIGBLUEBUTTONBN_TYPE_ALL,
         ];
 }
 
@@ -2066,8 +2066,8 @@ 
     $activity_time = '';
     if ($time) {
         $activity_time = calendar_day_representation($time).' '.
-          get_string('mod_form_field_notification_msg_at', 'bigbluebuttonbn').' '.
-          calendar_time_representation($time);
+            get_string('mod_form_field_notification_msg_at', 'bigbluebuttonbn').' '.
+            calendar_time_representation($time);
     }
 
     return $activity_time;

Spacing +99 added lines, -99 removed lines

@@ -26,7 +26,7 @@ 
 
 global $BIGBLUEBUTTONBN_CFG, $CFG;
 
-require_once dirname(__FILE__).'/lib.php';
+require_once dirname(__FILE__) . '/lib.php';
 
 const BIGBLUEBUTTONBN_FORCED = true;
 
@@ -65,7 +65,7 @@ 
     if (isset($meta)) {
         $log->meta = $meta;
     } elseif ($event == BIGBLUEBUTTONBN_LOG_EVENT_CREATE) {
-        $log->meta = '{"record":'.($bbbsession['record'] ? 'true' : 'false').'}';
+        $log->meta = '{"record":' . ($bbbsession['record'] ? 'true' : 'false') . '}';
     }
 
     $DB->insert_record('bigbluebuttonbn_logs', $log);
@@ -74,67 +74,67 @@ 
 //  BigBlueButton API Calls  //
 function bigbluebuttonbn_getJoinURL($meetingID, $userName, $PW, $SALT, $URL, $logoutURL, $configToken = null, $userId = null)
 {
-    $url_join = $URL.'api/join?';
-    $params = 'meetingID='.urlencode($meetingID).'&fullName='.urlencode($userName).'&password='.urlencode($PW).'&logoutURL='.urlencode($logoutURL);
+    $url_join = $URL . 'api/join?';
+    $params = 'meetingID=' . urlencode($meetingID) . '&fullName=' . urlencode($userName) . '&password=' . urlencode($PW) . '&logoutURL=' . urlencode($logoutURL);
     if (!is_null($userId)) {
-        $params .= '&userID='.urlencode($userId);
+        $params .= '&userID=' . urlencode($userId);
     }
     if (!is_null($configToken)) {
-        $params .= '&configToken='.$configToken;
+        $params .= '&configToken=' . $configToken;
     }
-    $url = $url_join.$params.'&checksum='.sha1('join'.$params.$SALT);
+    $url = $url_join . $params . '&checksum=' . sha1('join' . $params . $SALT);
 
     return $url;
 }
 
 function bigbluebuttonbn_getCreateMeetingURL($name, $meetingID, $attendeePW, $moderatorPW, $welcome, $logoutURL, $SALT, $URL, $record = 'false', $duration = 0, $voiceBridge = 0, $maxParticipants = 0, $metadata = array())
 {
-    $url_create = $URL.'api/create?';
+    $url_create = $URL . 'api/create?';
 
-    $params = 'name='.urlencode($name).'&meetingID='.urlencode($meetingID).'&attendeePW='.urlencode($attendeePW).'&moderatorPW='.urlencode($moderatorPW).'&logoutURL='.urlencode($logoutURL).'&record='.$record;
+    $params = 'name=' . urlencode($name) . '&meetingID=' . urlencode($meetingID) . '&attendeePW=' . urlencode($attendeePW) . '&moderatorPW=' . urlencode($moderatorPW) . '&logoutURL=' . urlencode($logoutURL) . '&record=' . $record;
 
     $voiceBridge = intval($voiceBridge);
     if ($voiceBridge > 0 && $voiceBridge < 79999) {
-        $params .= '&voiceBridge='.$voiceBridge;
+        $params .= '&voiceBridge=' . $voiceBridge;
     }
 
     $duration = intval($duration);
     if ($duration > 0) {
-        $params .= '&duration='.$duration;
+        $params .= '&duration=' . $duration;
     }
 
     $maxParticipants = intval($maxParticipants);
     if ($maxParticipants > 0) {
-        $params .= '&maxParticipants='.$maxParticipants;
+        $params .= '&maxParticipants=' . $maxParticipants;
     }
 
     if (trim($welcome)) {
-        $params .= '&welcome='.urlencode($welcome);
+        $params .= '&welcome=' . urlencode($welcome);
     }
 
     foreach ($metadata as $key => $value) {
-        $params .= '&'.$key.'='.urlencode($value);
+        $params .= '&' . $key . '=' . urlencode($value);
     }
 
-    $url = $url_create.$params.'&checksum='.sha1('create'.$params.$SALT);
+    $url = $url_create . $params . '&checksum=' . sha1('create' . $params . $SALT);
 
     return $url;
 }
 
 function bigbluebuttonbn_getIsMeetingRunningURL($meetingID, $URL, $SALT)
 {
-    $base_url = $URL.'api/isMeetingRunning?';
-    $params = 'meetingID='.urlencode($meetingID);
-    $url = $base_url.$params.'&checksum='.sha1('isMeetingRunning'.$params.$SALT);
+    $base_url = $URL . 'api/isMeetingRunning?';
+    $params = 'meetingID=' . urlencode($meetingID);
+    $url = $base_url . $params . '&checksum=' . sha1('isMeetingRunning' . $params . $SALT);
 
     return $url;
 }
 
 function bigbluebuttonbn_getMeetingInfoURL($meetingID, $modPW, $URL, $SALT)
 {
-    $base_url = $URL.'api/getMeetingInfo?';
-    $params = 'meetingID='.urlencode($meetingID).'&password='.urlencode($modPW);
-    $url = $base_url.$params.'&checksum='.sha1('getMeetingInfo'.$params.$SALT);
+    $base_url = $URL . 'api/getMeetingInfo?';
+    $params = 'meetingID=' . urlencode($meetingID) . '&password=' . urlencode($modPW);
+    $url = $base_url . $params . '&checksum=' . sha1('getMeetingInfo' . $params . $SALT);
 
     return $url;
 }
@@ -145,8 +145,8 @@ 
  */
 function bigbluebuttonbn_getMeetingsURL($URL, $SALT)
 {
-    $base_url = $URL.'api/getMeetings?';
-    $url = $base_url.'&checksum='.sha1('getMeetings'.$SALT);
+    $base_url = $URL . 'api/getMeetings?';
+    $url = $base_url . '&checksum=' . sha1('getMeetings' . $SALT);
 
     return $url;
 }
@@ -157,9 +157,9 @@ 
  */
 function bigbluebuttonbn_getEndMeetingURL($meetingID, $modPW, $URL, $SALT)
 {
-    $base_url = $URL.'api/end?';
-    $params = 'meetingID='.urlencode($meetingID).'&password='.urlencode($modPW);
-    $url = $base_url.$params.'&checksum='.sha1('end'.$params.$SALT);
+    $base_url = $URL . 'api/end?';
+    $params = 'meetingID=' . urlencode($meetingID) . '&password=' . urlencode($modPW);
+    $url = $base_url . $params . '&checksum=' . sha1('end' . $params . $SALT);
 
     return $url;
 }
@@ -171,9 +171,9 @@ 
  */
 function bigbluebuttonbn_getRecordingsURL($meetingID, $URL, $SALT)
 {
-    $base_url_record = $URL.'api/getRecordings?';
-    $params = 'meetingID='.urlencode($meetingID);
-    $url = $base_url_record.$params.'&checksum='.sha1('getRecordings'.$params.$SALT);
+    $base_url_record = $URL . 'api/getRecordings?';
+    $params = 'meetingID=' . urlencode($meetingID);
+    $url = $base_url_record . $params . '&checksum=' . sha1('getRecordings' . $params . $SALT);
 
     return $url;
 }
@@ -184,9 +184,9 @@ 
  */
 function bigbluebuttonbn_getDeleteRecordingsURL($recordID, $URL, $SALT)
 {
-    $url_delete = $URL.'api/deleteRecordings?';
-    $params = 'recordID='.urlencode($recordID);
-    $url = $url_delete.$params.'&checksum='.sha1('deleteRecordings'.$params.$SALT);
+    $url_delete = $URL . 'api/deleteRecordings?';
+    $params = 'recordID=' . urlencode($recordID);
+    $url = $url_delete . $params . '&checksum=' . sha1('deleteRecordings' . $params . $SALT);
 
     return $url;
 }
@@ -198,30 +198,30 @@ 
  */
 function bigbluebuttonbn_getPublishRecordingsURL($recordID, $set, $URL, $SALT)
 {
-    $url_publish = $URL.'api/publishRecordings?';
-    $params = 'recordID='.$recordID.'&publish='.$set;
-    $url = $url_publish.$params.'&checksum='.sha1('publishRecordings'.$params.$SALT);
+    $url_publish = $URL . 'api/publishRecordings?';
+    $params = 'recordID=' . $recordID . '&publish=' . $set;
+    $url = $url_publish . $params . '&checksum=' . sha1('publishRecordings' . $params . $SALT);
 
     return $url;
 }
 
 function bigbluebuttonbn_getUpdateRecordingsURL($recordID, $URL, $SALT, $metadata = array())
 {
-    $url_update = $URL.'api/updateRecordings?';
-    $params = 'recordID='.$recordID;
+    $url_update = $URL . 'api/updateRecordings?';
+    $params = 'recordID=' . $recordID;
     foreach ($metadata as $key => $value) {
-        $params .= '&'.'meta_'.$key.'='.urlencode($value);
+        $params .= '&' . 'meta_' . $key . '=' . urlencode($value);
     }
-    $url = $url_update.$params.'&checksum='.sha1('updateRecordings'.$params.$SALT);
+    $url = $url_update . $params . '&checksum=' . sha1('updateRecordings' . $params . $SALT);
 
     return $url;
 }
 
 function bigbluebuttonbn_getDefaultConfigXMLURL($URL, $SALT)
 {
-    $url_default_config = $URL.'api/getDefaultConfigXML?';
+    $url_default_config = $URL . 'api/getDefaultConfigXML?';
     $params = '';
-    $url = $url_default_config.$params.'&checksum='.sha1('getDefaultConfigXML'.$params.$SALT);
+    $url = $url_default_config . $params . '&checksum=' . sha1('getDefaultConfigXML' . $params . $SALT);
 
     return $url;
 }
@@ -232,7 +232,7 @@ 
     if (!is_null($presentation_name) && !is_null($presentation_url)) {
         $xml = bigbluebuttonbn_wrap_xml_load_file($create_meeting_url,
                 BIGBLUEBUTTONBN_METHOD_POST,
-                "<?xml version='1.0' encoding='UTF-8'?><modules><module name='presentation'><document url='".$presentation_url."' /></module></modules>"
+                "<?xml version='1.0' encoding='UTF-8'?><modules><module name='presentation'><document url='" . $presentation_url . "' /></module></modules>"
                 );
     } else {
         $xml = bigbluebuttonbn_wrap_xml_load_file($create_meeting_url);
@@ -389,11 +389,11 @@ 
     global $DB;
 
     if ($bigbluebuttonbnID === null) {
-        $select = "courseid = '{$courseID}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_IMPORT."'";
+        $select = "courseid = '{$courseID}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_IMPORT . "'";
     } elseif ($subset) {
-        $select = "bigbluebuttonbnid = '{$bigbluebuttonbnID}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_IMPORT."'";
+        $select = "bigbluebuttonbnid = '{$bigbluebuttonbnID}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_IMPORT . "'";
     } else {
-        $select = "courseid = '{$courseID}' AND bigbluebuttonbnid <> '{$bigbluebuttonbnID}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_IMPORT."'";
+        $select = "courseid = '{$courseID}' AND bigbluebuttonbnid <> '{$bigbluebuttonbnID}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_IMPORT . "'";
     }
     $records_imported = $DB->get_records_select('bigbluebuttonbn_logs', $select);
 
@@ -425,7 +425,7 @@ 
 function bigbluebuttonbn_getDefaultConfigXMLArray($URL, $SALT)
 {
     $default_config_xml = bigbluebuttonbn_getDefaultConfigXML($URL, $SALT);
-    $default_config_xml_array = (array) $default_config_xml;
+    $default_config_xml_array = (array)$default_config_xml;
 
     return $default_config_xml_array;
 }
@@ -436,18 +436,18 @@ 
     //Add formats
     $playbackArray = array();
     foreach ($recording->playback->format as $format) {
-        $playbackArray[(string) $format->type] = array('type' => (string) $format->type, 'url' => (string) $format->url, 'length' => (string) $format->length);
+        $playbackArray[(string)$format->type] = array('type' => (string)$format->type, 'url' => (string)$format->url, 'length' => (string)$format->length);
         //Add preview per format when existing
         if ($format->preview) {
             $imagesArray = array();
             foreach ($format->preview->images->image as $image) {
-                $imageArray = array('url' => (string) $image);
+                $imageArray = array('url' => (string)$image);
                 foreach ($image->attributes() as $attKey => $attValue) {
-                    $imageArray[$attKey] = (string) $attValue;
+                    $imageArray[$attKey] = (string)$attValue;
                 }
                 array_push($imagesArray, $imageArray);
             }
-            $playbackArray[(string) $format->type]['preview'] = $imagesArray;
+            $playbackArray[(string)$format->type]['preview'] = $imagesArray;
         }
     }
 
@@ -458,10 +458,10 @@ 
         if (is_object($value)) {
             $value = '';
         }
-        $metadataArray['meta_'.$key] = $value;
+        $metadataArray['meta_' . $key] = $value;
     }
 
-    $recordingArrayValue = array('recordID' => (string) $recording->recordID, 'meetingID' => (string) $recording->meetingID, 'meetingName' => (string) $recording->name, 'published' => (string) $recording->published, 'startTime' => (string) $recording->startTime, 'endTime' => (string) $recording->endTime, 'playbacks' => $playbackArray) + $metadataArray;
+    $recordingArrayValue = array('recordID' => (string)$recording->recordID, 'meetingID' => (string)$recording->meetingID, 'meetingName' => (string)$recording->name, 'published' => (string)$recording->published, 'startTime' => (string)$recording->startTime, 'endTime' => (string)$recording->endTime, 'playbacks' => $playbackArray) + $metadataArray;
 
     return $recordingArrayValue;
 }
@@ -549,7 +549,7 @@ 
 {
     $endpoint = is_null($URL) ? bigbluebuttonbn_get_cfg_server_url() : $URL;
 
-    $xml = bigbluebuttonbn_wrap_xml_load_file($endpoint.'api');
+    $xml = bigbluebuttonbn_wrap_xml_load_file($endpoint . 'api');
     if ($xml && $xml->returncode == 'SUCCESS') {
         return $xml->version;
     } else {
@@ -574,7 +574,7 @@ 
 function bigbluebuttonbn_wrap_xml_load_file($url, $method = BIGBLUEBUTTONBN_METHOD_GET, $data = null, $content_type = 'text/xml')
 {
     if (bigbluebuttonbn_debugdisplay()) {
-        error_log('Request to: '.$url);
+        error_log('Request to: ' . $url);
     }
 
     if (extension_loaded('curl')) {
@@ -585,8 +585,8 @@ 
                 if (!is_array($data)) {
                     $options = array();
                     $options['CURLOPT_HTTPHEADER'] = array(
-                            'Content-Type: '.$content_type,
-                            'Content-Length: '.strlen($data),
+                            'Content-Type: ' . $content_type,
+                            'Content-Length: ' . strlen($data),
                             'Content-Language: en-US',
                         );
                     $response = $c->post($url, $data, $options);
@@ -602,7 +602,7 @@ 
 
         if ($response) {
             if (bigbluebuttonbn_debugdisplay()) {
-                error_log('Response: '.$response);
+                error_log('Response: ' . $response);
             }
             $previous = libxml_use_internal_errors(true);
             try {
@@ -611,7 +611,7 @@ 
                 return $xml;
             } catch (Exception $e) {
                 libxml_use_internal_errors($previous);
-                $error = 'Caught exception: '.$e->getMessage();
+                $error = 'Caught exception: ' . $e->getMessage();
                 error_log($error);
 
                 return null;
@@ -626,12 +626,12 @@ 
         try {
             $response = simplexml_load_file($url, 'SimpleXMLElement', LIBXML_NOCDATA | LIBXML_NOBLANKS);
             if (bigbluebuttonbn_debugdisplay()) {
-                error_log('Response processed: '.$response->asXML());
+                error_log('Response processed: ' . $response->asXML());
             }
 
             return $response;
         } catch (Exception $e) {
-            $error = 'Caught exception: '.$e->getMessage();
+            $error = 'Caught exception: ' . $e->getMessage();
             error_log($error);
             libxml_use_internal_errors($previous);
 
@@ -648,9 +648,9 @@ 
     if ($user_roles) {
         $where = '';
         foreach ($user_roles as $key => $value) {
-            $where .= (empty($where) ? ' WHERE' : ' AND').' id='.$value->roleid;
+            $where .= (empty($where) ? ' WHERE' : ' AND') . ' id=' . $value->roleid;
         }
-        $user_roles = $DB->get_records_sql('SELECT * FROM {role}'.$where);
+        $user_roles = $DB->get_records_sql('SELECT * FROM {role}' . $where);
     }
 
     return $user_roles;
@@ -713,13 +713,13 @@ 
             foreach ($participant_list as $participant) {
                 $participant_list_array[] = array('selectiontype' => $participant->selectiontype,
                                                     'selectionid' => $participant->selectionid,
-                                                    'role' => $participant->role, );
+                                                    'role' => $participant->role,);
             }
         }
     } else {
         $participant_list_array[] = array('selectiontype' => 'all',
                                             'selectionid' => 'all',
-                                            'role' => BIGBLUEBUTTONBN_ROLE_VIEWER, );
+                                            'role' => BIGBLUEBUTTONBN_ROLE_VIEWER,);
 
         $moderator_defaults = bigbluebuttonbn_get_cfg_moderator_default();
         if (!isset($moderator_defaults)) {
@@ -734,14 +734,14 @@ 
                     if ($user->id == $USER->id) {
                         $participant_list_array[] = array('selectiontype' => 'user',
                                                             'selectionid' => $USER->id,
-                                                            'role' => BIGBLUEBUTTONBN_ROLE_MODERATOR, );
+                                                            'role' => BIGBLUEBUTTONBN_ROLE_MODERATOR,);
                         break;
                     }
                 }
             } else {
                 $participant_list_array[] = array('selectiontype' => 'role',
                                                     'selectionid' => $moderator_default,
-                                                    'role' => BIGBLUEBUTTONBN_ROLE_MODERATOR, );
+                                                    'role' => BIGBLUEBUTTONBN_ROLE_MODERATOR,);
             }
         }
     }
@@ -804,9 +804,9 @@ 
     $is_unique = true;
     if ($voicebridge != 0) {
         $table = 'bigbluebuttonbn';
-        $select = 'voicebridge = '.$voicebridge;
+        $select = 'voicebridge = ' . $voicebridge;
         if ($id) {
-            $select .= ' AND id <> '.$id;
+            $select .= ' AND id <> ' . $id;
         }
         if ($DB->get_records_select($table, $select)) {
             $is_unique = false;
@@ -872,7 +872,7 @@ 
     $mt = microtime();
     $rand = mt_rand();
 
-    return md5($mt.$rand);
+    return md5($mt . $rand);
 }
 
 function bigbluebuttonbn_random_password($length = 8)
@@ -1048,7 +1048,7 @@ 
         }
         if (isset($moderatorPW)) {
             //Ping again and refresh the cache
-            $meeting_info = (array) bigbluebuttonbn_getMeetingInfo($meetingid, $moderatorPW, $endpoint, $shared_secret);
+            $meeting_info = (array)bigbluebuttonbn_getMeetingInfo($meetingid, $moderatorPW, $endpoint, $shared_secret);
             $cache->set($meetingid, array('creation_time' => time(), 'meeting_info' => json_encode($meeting_info)));
         }
     }
@@ -1154,7 +1154,7 @@ 
             case 'moodle_event':
                 break;
             default:
-                $error = bigbluebuttonbn_add_error($error, 'Action '.$params['action'].' can not be performed.');
+                $error = bigbluebuttonbn_add_error($error, 'Action ' . $params['action'] . ' can not be performed.');
         }
     }
 
@@ -1180,8 +1180,8 @@ 
  */
 function bigbluebuttonbn_setConfigXMLParams($meetingID, $configXML, $SALT)
 {
-    $params = 'configXML='.urlencode($configXML).'&meetingID='.urlencode($meetingID);
-    $config_xml_params = $params.'&checksum='.sha1('setConfigXML'.$params.$SALT);
+    $params = 'configXML=' . urlencode($configXML) . '&meetingID=' . urlencode($meetingID);
+    $config_xml_params = $params . '&checksum=' . sha1('setConfigXML' . $params . $SALT);
 
     return $config_xml_params;
 }
@@ -1192,7 +1192,7 @@ 
  */
 function bigbluebuttonbn_setConfigXML($meetingID, $configXML, $URL, $SALT)
 {
-    $url_default_config = $URL.'api/setConfigXML?';
+    $url_default_config = $URL . 'api/setConfigXML?';
     $config_xml_params = bigbluebuttonbn_setConfigXMLParams($meetingID, $configXML, $SALT);
     $xml = bigbluebuttonbn_wrap_xml_load_file($url_default_config, BIGBLUEBUTTONBN_METHOD_POST, $config_xml_params, 'application/x-www-form-urlencoded');
 
@@ -1206,7 +1206,7 @@ 
 function bigbluebuttonbn_setConfigXMLArray($meetingID, $configXML, $URL, $SALT)
 {
     $config_xml = bigbluebuttonbn_setConfigXML($meetingID, $configXML, $URL, $SALT);
-    $config_xml_array = (array) $config_xml;
+    $config_xml_array = (array)$config_xml;
 
     return $config_xml_array;
 }
@@ -1263,19 +1263,19 @@ 
 
         //Set recording_types
         if (isset($recording['imported'])) {
-            $attributes = 'data-imported="true" title='.get_string('view_recording_link_warning', 'bigbluebuttonbn');
+            $attributes = 'data-imported="true" title=' . get_string('view_recording_link_warning', 'bigbluebuttonbn');
         } else {
             $attributes = 'data-imported="false"';
         }
 
         $recording_types = '';
         if ($recording['published'] == 'true') {
-            $recording_types .= '<div id="playbacks-'.$recording['recordID'].'" '.$attributes.'>';
+            $recording_types .= '<div id="playbacks-' . $recording['recordID'] . '" ' . $attributes . '>';
         } else {
-            $recording_types .= '<div id="playbacks-'.$recording['recordID'].'" '.$attributes.'" hidden>';
+            $recording_types .= '<div id="playbacks-' . $recording['recordID'] . '" ' . $attributes . '" hidden>';
         }
         foreach ($recording['playbacks'] as $playback) {
-            $recording_types .= $OUTPUT->action_link($playback['url'], get_string('view_recording_format_'.$playback['type'], 'bigbluebuttonbn'), null, array('title' => get_string('view_recording_format_'.$playback['type'], 'bigbluebuttonbn'), 'target' => '_new')).'&#32;';
+            $recording_types .= $OUTPUT->action_link($playback['url'], get_string('view_recording_format_' . $playback['type'], 'bigbluebuttonbn'), null, array('title' => get_string('view_recording_format_' . $playback['type'], 'bigbluebuttonbn'), 'target' => '_new')) . '&#32;';
         }
         $recording_types .= '</div>';
 
@@ -1323,7 +1323,7 @@ 
         }
 
         //Set corresponding format
-        $dateformat = get_string('strftimerecentfull', 'langconfig').' %Z';
+        $dateformat = get_string('strftimerecentfull', 'langconfig') . ' %Z';
         $formattedStartDate = userdate($startTime / 1000, $dateformat, usertimezone($USER->timezone));
 
         $row = new stdClass();
@@ -1349,12 +1349,12 @@ 
 
     $actionbar = '';
 
-    $onclick = 'M.mod_bigbluebuttonbn.broker_manageRecording("'.$manage_action.'", "'.$recording['recordID'].'", "'.$recording['meetingID'].'");';
+    $onclick = 'M.mod_bigbluebuttonbn.broker_manageRecording("' . $manage_action . '", "' . $recording['recordID'] . '", "' . $recording['meetingID'] . '");';
     if (bigbluebuttonbn_get_cfg_recording_icons_enabled()) {
         //With icon for $action
-        $icon_attributes = array('id' => 'recording-btn-'.$action.'-'.$recording['recordID']);
-        $icon = new pix_icon('i/'.$manage_tag, get_string($manage_tag), 'moodle', $icon_attributes);
-        $link_attributes = array('id' => 'recording-link-'.$manage_action.'-'.$recording['recordID'], 'onclick' => $onclick);
+        $icon_attributes = array('id' => 'recording-btn-' . $action . '-' . $recording['recordID']);
+        $icon = new pix_icon('i/' . $manage_tag, get_string($manage_tag), 'moodle', $icon_attributes);
+        $link_attributes = array('id' => 'recording-link-' . $manage_action . '-' . $recording['recordID'], 'onclick' => $onclick);
         $actionbar .= $OUTPUT->action_icon('#', $icon, null, $link_attributes, false);
     } else {
         //With text for $action
@@ -1430,7 +1430,7 @@ 
         $table->head = array($view_recording_playback, $view_recording_recording, $view_recording_description, $view_recording_preview, $view_recording_date, $view_recording_duration, $view_recording_actionbar);
         $table->align = array('left', 'left', 'left', 'left', 'left', 'center', 'left');
     } else {
-        $table->head = array($view_recording_playback.$view_recording_recording, $view_recording_description, $view_recording_preview, $view_recording_date, $view_recording_duration);
+        $table->head = array($view_recording_playback . $view_recording_recording, $view_recording_description, $view_recording_preview, $view_recording_date, $view_recording_duration);
         $table->align = array('left', 'left', 'left', 'left', 'left', 'center');
     }
 
@@ -1438,7 +1438,7 @@ 
     if (isset($recordings) && !array_key_exists('messageKey', $recordings)) {  // There are recordings for this meeting
         foreach ($recordings as $recording) {
             $row = new html_table_row();
-            $row->id = 'recording-td-'.$recording['recordID'];
+            $row->id = 'recording-td-' . $recording['recordID'];
             if (isset($recording['imported'])) {
                 $row->attributes['data-imported'] = 'true';
                 $row->attributes['title'] = get_string('view_recording_link_warning', 'bigbluebuttonbn');
@@ -1473,7 +1473,7 @@ 
     /// Build the message_body
     $msg->activity_type = '';
     $msg->activity_title = $bigbluebuttonbn->name;
-    $message_text = '<p>'.get_string('email_body_recording_ready_for', 'bigbluebuttonbn').' '.$msg->activity_type.' &quot;'.$msg->activity_title.'&quot; '.get_string('email_body_recording_ready_is_ready', 'bigbluebuttonbn').'.</p>';
+    $message_text = '<p>' . get_string('email_body_recording_ready_for', 'bigbluebuttonbn') . ' ' . $msg->activity_type . ' &quot;' . $msg->activity_title . '&quot; ' . get_string('email_body_recording_ready_is_ready', 'bigbluebuttonbn') . '.</p>';
 
     bigbluebuttonbn_send_notification($sender, $bigbluebuttonbn, $message_text);
 }
@@ -1864,9 +1864,9 @@ 
         $select = '';
         foreach ($bigbluebuttonbns as $key => $bigbluebuttonbn) {
             $select .= strlen($select) == 0 ? '(' : ' OR ';
-            $select .= 'bigbluebuttonbnid='.$bigbluebuttonbn->id;
+            $select .= 'bigbluebuttonbnid=' . $bigbluebuttonbn->id;
         }
-        $select .= ") AND log='".BIGBLUEBUTTONBN_LOG_EVENT_CREATE."'";
+        $select .= ") AND log='" . BIGBLUEBUTTONBN_LOG_EVENT_CREATE . "'";
 
         //Execute select for loading records based on existent bigbluebuttonbns
         $records = $DB->get_records_select($table, $select);
@@ -1874,7 +1874,7 @@ 
         //Remove duplicates
         $unique_records = array();
         foreach ($records as $key => $record) {
-            $record_key = $record->meetingid.','.$record->bigbluebuttonbnid.','.$record->meta;
+            $record_key = $record->meetingid . ',' . $record->bigbluebuttonbnid . ',' . $record->meta;
             if (array_search($record_key, $unique_records) === false) {
                 array_push($unique_records, $record_key);
             } else {
@@ -1915,7 +1915,7 @@ 
 {
     global $CFG;
 
-    return (bool) $CFG->debugdisplay;
+    return (bool)$CFG->debugdisplay;
 }
 
 function bigbluebuttonbn_html2text($html, $len)
@@ -1923,7 +1923,7 @@ 
     $text = strip_tags($html);
     $text = str_replace('&nbsp;', ' ', $text);
     if (strlen($text) > $len) {
-        $text = substr($text, 0, $len).'...';
+        $text = substr($text, 0, $len) . '...';
     } else {
         $text = substr($text, 0, $len);
     }
@@ -1936,7 +1936,7 @@ 
     $tags = '';
     $tags_array = core_tag_tag::get_item_tags_array('core', 'course_modules', $id);
     foreach ($tags_array as $key => $value) {
-        $tags .= ($tags == '') ? $value : ','.$value;
+        $tags .= ($tags == '') ? $value : ',' . $value;
     }
 
     return $tags;
@@ -1969,11 +1969,11 @@ 
     // Consider logs from deleted bigbluebuttonbn instances whose meetingids should be included in the getRecordings request
     if ($include_deleted) {
         if ($bigbluebuttonbnID === null) {
-            $select = "courseid = '{$courseID}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_DELETE."' AND meta like '%has_recordings%' AND meta like '%true%'";
+            $select = "courseid = '{$courseID}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_DELETE . "' AND meta like '%has_recordings%' AND meta like '%true%'";
         } elseif ($subset) {
-            $select = "bigbluebuttonbnid = '{$bigbluebuttonbnID}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_DELETE."' AND meta like '%has_recordings%' AND meta like '%true%'";
+            $select = "bigbluebuttonbnid = '{$bigbluebuttonbnID}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_DELETE . "' AND meta like '%has_recordings%' AND meta like '%true%'";
         } else {
-            $select = "courseid = '{$courseID}' AND bigbluebuttonbnid <> '{$bigbluebuttonbnID}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_DELETE."' AND meta like '%has_recordings%' AND meta like '%true%'";
+            $select = "courseid = '{$courseID}' AND bigbluebuttonbnid <> '{$bigbluebuttonbnID}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_DELETE . "' AND meta like '%has_recordings%' AND meta like '%true%'";
         }
         $bigbluebuttonbns_deleted = $DB->get_records_select_menu('bigbluebuttonbn_logs', $select, null, 'bigbluebuttonbnid', 'bigbluebuttonbnid, meetingid');
         if (!empty($bigbluebuttonbns_deleted)) {
@@ -1986,7 +1986,7 @@ 
     if (!empty($bigbluebuttonbns)) {
         //Prepare select for loading records based on existent bigbluebuttonbns
         $sql = 'SELECT DISTINCT meetingid, bigbluebuttonbnid FROM {bigbluebuttonbn_logs} WHERE ';
-        $sql .= '(bigbluebuttonbnid='.implode(' OR bigbluebuttonbnid=', array_keys($bigbluebuttonbns)).')';
+        $sql .= '(bigbluebuttonbnid=' . implode(' OR bigbluebuttonbnid=', array_keys($bigbluebuttonbns)) . ')';
         //Include only Create events and exclude those with record not true
         $sql .= ' AND log = ? AND meta LIKE ? AND meta LIKE ?';
         //Execute select for loading records based on existent bigbluebuttonbns
@@ -2065,8 +2065,8 @@ 
 {
     $activity_time = '';
     if ($time) {
-        $activity_time = calendar_day_representation($time).' '.
-          get_string('mod_form_field_notification_msg_at', 'bigbluebuttonbn').' '.
+        $activity_time = calendar_day_representation($time) . ' ' .
+          get_string('mod_form_field_notification_msg_at', 'bigbluebuttonbn') . ' ' .
           calendar_time_representation($time);
     }
 

vendor/firebase/php-jwt/src/JWT.php

Doc Comments +1 added lines, -1 removed lines

@@ -174,7 +174,7 @@
      * Sign a string with a given key and algorithm.
      *
      * @param string            $msg    The message to sign
-     * @param string|resource   $key    The secret key
+     * @param string   $key    The secret key
      * @param string            $alg    The signing algorithm.
      *                                  Supported algorithms are 'HS256', 'HS384', 'HS512' and 'RS256'
      *

Spacing +5 added lines, -5 removed lines

@@ -156,7 +156,7 @@ 
         if ($keyId !== null) {
             $header['kid'] = $keyId;
         }
-        if ( isset($head) && is_array($head) ) {
+        if (isset($head) && is_array($head)) {
             $header = array_merge($head, $header);
         }
         $segments = array();
@@ -188,7 +188,7 @@ 
             throw new DomainException('Algorithm not supported');
         }
         list($function, $algorithm) = static::$supported_algs[$alg];
-        switch($function) {
+        switch ($function) {
             case 'hash_hmac':
                 return hash_hmac($algorithm, $msg, $key, true);
             case 'openssl':
@@ -222,7 +222,7 @@ 
         }
 
         list($function, $algorithm) = static::$supported_algs[$alg];
-        switch($function) {
+        switch ($function) {
             case 'openssl':
                 $success = openssl_verify($msg, $signature, $key, $algorithm);
                 if (!$success) {
@@ -270,8 +270,8 @@ 
              * manually detect large ints in the JSON string and quote them (thus converting
              *them to strings) before decoding, hence the preg_replace() call.
              */
-            $max_int_length = strlen((string) PHP_INT_MAX) - 1;
-            $json_without_bigints = preg_replace('/:\s*(-?\d{'.$max_int_length.',})/', ': "$1"', $input);
+            $max_int_length = strlen((string)PHP_INT_MAX) - 1;
+            $json_without_bigints = preg_replace('/:\s*(-?\d{' . $max_int_length . ',})/', ': "$1"', $input);
             $obj = json_decode($json_without_bigints);
         }
 

backup/moodle2/backup_bigbluebuttonbn_stepslib.php

Spacing +2 added lines, -2 removed lines

@@ -37,12 +37,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);

backup/moodle2/restore_bigbluebuttonbn_stepslib.php

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
     {
         global $DB;
 
-        $data = (object) $data;
+        $data = (object)$data;
         $data->course = $this->get_courseid();
 
         $data->timemodified = $this->apply_date_offset($data->timemodified);
@@ -58,7 +58,7 @@ 
     {
         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

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

backup/moodle2/backup_bigbluebuttonbn_activity_task.class.php

Spacing +4 added lines, -4 removed lines

@@ -20,7 +20,7 @@ 
  */
 defined('MOODLE_INTERNAL') || die;
 
-require_once $CFG->dirroot.'/mod/bigbluebuttonbn/backup/moodle2/backup_bigbluebuttonbn_stepslib.php'; // Because it exists (must)
+require_once $CFG->dirroot . '/mod/bigbluebuttonbn/backup/moodle2/backup_bigbluebuttonbn_stepslib.php'; // Because it exists (must)
 
 /**
  * bigbluebuttonbn backup task that provides all the settings and steps to perform one
@@ -53,14 +53,14 @@ 
     {
         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;

bbb_broker.php

Spacing +26 added lines, -26 removed lines

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