blindsidenetworks / moodle-mod_bigbluebuttonbn

index.php

Spacing +50 added lines, -50 removed lines

@@ -8,12 +8,12 @@ 
  * @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
-$g  = optional_param('g', 0, PARAM_INT);    // group instance ID
+$id = required_param('id', PARAM_INT); // Course Module ID, or
+$a  = optional_param('a', 0, PARAM_INT); // bigbluebuttonbn instance ID
+$g  = optional_param('g', 0, PARAM_INT); // group instance ID
 
 if ($id) {
     $course = $DB->get_record('course', array('id'=>$id), '*', MUST_EXIST);
@@ -36,7 +36,7 @@ 
 $PAGE->navbar->add(get_string('modulename', 'bigbluebuttonbn'), "index.php?id=$course->id");
 
 /// Get all the appropriate data
-if (! $bigbluebuttonbns = get_all_instances_in_course('bigbluebuttonbn', $course)) {
+if (!$bigbluebuttonbns = get_all_instances_in_course('bigbluebuttonbn', $course)) {
     notice('There are no instances of bigbluebuttonbn', "../../course/view.php?id=$course->id");
 }
 
@@ -44,18 +44,18 @@ 
 $timenow            = time();
 $strweek            = get_string('week');
 $strtopic           = get_string('topic');
-$heading_name       = get_string('index_heading_name', 'bigbluebuttonbn' );
-$heading_group      = get_string('index_heading_group', 'bigbluebuttonbn' );
+$heading_name       = get_string('index_heading_name', 'bigbluebuttonbn');
+$heading_group      = get_string('index_heading_group', 'bigbluebuttonbn');
 $heading_users      = get_string('index_heading_users', 'bigbluebuttonbn');
 $heading_viewer     = get_string('index_heading_viewer', 'bigbluebuttonbn');
-$heading_moderator  = get_string('index_heading_moderator', 'bigbluebuttonbn' );
-$heading_actions    = get_string('index_heading_actions', 'bigbluebuttonbn' );
-$heading_recording  = get_string('index_heading_recording', 'bigbluebuttonbn' );
+$heading_moderator  = get_string('index_heading_moderator', 'bigbluebuttonbn');
+$heading_actions    = get_string('index_heading_actions', 'bigbluebuttonbn');
+$heading_recording  = get_string('index_heading_recording', 'bigbluebuttonbn');
 
 $table = new html_table();
 
-$table->head  = array ($strweek, $heading_name, $heading_group, $heading_users, $heading_viewer, $heading_moderator, $heading_recording, $heading_actions );
-$table->align = array ('center', 'left', 'center', 'center', 'center',  'center', 'center' );
+$table->head  = array($strweek, $heading_name, $heading_group, $heading_users, $heading_viewer, $heading_moderator, $heading_recording, $heading_actions);
+$table->align = array('center', 'left', 'center', 'center', 'center', 'center', 'center');
 
 $endpoint = bigbluebuttonbn_get_cfg_server_url();
 $shared_secret = bigbluebuttonbn_get_cfg_shared_secret();
@@ -66,14 +66,14 @@ 
     //
     // A request to end the meeting
     //
-    if (! $bigbluebuttonbn = $DB->get_record('bigbluebuttonbn', array('id' => $a), '*', MUST_EXIST) ) {
+    if (!$bigbluebuttonbn = $DB->get_record('bigbluebuttonbn', array('id' => $a), '*', MUST_EXIST)) {
         print_error("BigBlueButton ID $a is incorrect");
     }
     $course = $DB->get_record('course', array('id' => $bigbluebuttonbn->course), '*', MUST_EXIST);
     $cm = get_coursemodule_from_instance('bigbluebuttonbn', $bigbluebuttonbn->id, $course->id, false, MUST_EXIST);
 
     //User roles
-    if( $bigbluebuttonbn->participants == null || $bigbluebuttonbn->participants == "" || $bigbluebuttonbn->participants == "[]" ){
+    if ($bigbluebuttonbn->participants == null || $bigbluebuttonbn->participants == "" || $bigbluebuttonbn->participants == "[]") {
         //The room that is being used comes from a previous version
         $moderator = has_capability('mod/bigbluebuttonbn:moderate', $context);
     } else {
@@ -81,19 +81,19 @@ 
     }
     $administrator = has_capability('moodle/category:manage', $context);
 
-    if( $moderator || $administrator ) {
+    if ($moderator || $administrator) {
         bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_ENDED, $bigbluebuttonbn, $context, $cm);
 
         echo get_string('index_ending', 'bigbluebuttonbn');
 
-        $meetingID = $bigbluebuttonbn->meetingid.'-'.$course->id.'-'.$bigbluebuttonbn->id;
+        $meetingID = $bigbluebuttonbn->meetingid . '-' . $course->id . '-' . $bigbluebuttonbn->id;
         $modPW = $bigbluebuttonbn->moderatorpass;
-        if( $g != '0'  ) {
-            $getArray = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getEndMeetingURL( $meetingID.'['.$g.']', $modPW, $endpoint, $shared_secret ) );
+        if ($g != '0') {
+            $getArray = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getEndMeetingURL($meetingID . '[' . $g . ']', $modPW, $endpoint, $shared_secret));
         } else {
-            $getArray = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getEndMeetingURL( $meetingID, $modPW, $endpoint, $shared_secret ));
+            $getArray = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getEndMeetingURL($meetingID, $modPW, $endpoint, $shared_secret));
         }
-	   redirect('index.php?id='.$id);
+	   redirect('index.php?id=' . $id);
     }
 }
 
@@ -101,7 +101,7 @@ 
     $cm = get_coursemodule_from_id('bigbluebuttonbn', $bigbluebuttonbn->coursemodule, 0, false, MUST_EXIST);
 
     //User roles
-    if( $bigbluebuttonbn->participants == null || $bigbluebuttonbn->participants == "" || $bigbluebuttonbn->participants == "[]" ){
+    if ($bigbluebuttonbn->participants == null || $bigbluebuttonbn->participants == "" || $bigbluebuttonbn->participants == "[]") {
         //The room that is being used comes from a previous version
         $moderator = has_capability('mod/bigbluebuttonbn:moderate', $context);
     } else {
@@ -109,11 +109,11 @@ 
     }
     $administrator = has_capability('moodle/category:manage', $context);
 
-    if ( groups_get_activity_groupmode($cm) > 0 ){
-        $table->data[] = displayBigBlueButtonRooms($endpoint, $shared_secret, ($administrator || $moderator), $course, $bigbluebuttonbn, (object) array('id'=>0, 'name'=>get_string('allparticipants')));
+    if (groups_get_activity_groupmode($cm) > 0) {
+        $table->data[] = displayBigBlueButtonRooms($endpoint, $shared_secret, ($administrator || $moderator), $course, $bigbluebuttonbn, (object)array('id'=>0, 'name'=>get_string('allparticipants')));
         $groups = groups_get_activity_allowed_groups($cm);
-        if( isset($groups)) {
-            foreach( $groups as $group){
+        if (isset($groups)) {
+            foreach ($groups as $group) {
                 $table->data[] = displayBigBlueButtonRooms($endpoint, $shared_secret, ($administrator || $moderator), $course, $bigbluebuttonbn, $group);
             }
         }
@@ -130,7 +130,7 @@ 
 echo $OUTPUT->footer();
 
 /// Functions
-function displayBigBlueButtonRooms($endpoint, $shared_secret, $moderator, $course, $bigbluebuttonbn, $groupObj = null ){
+function displayBigBlueButtonRooms($endpoint, $shared_secret, $moderator, $course, $bigbluebuttonbn, $groupObj = null) {
     $joinURL = null;
     $group = "-";
     $users = "-";
@@ -140,22 +140,22 @@ 
     $moderatorList = "-";
     $recording = "-";
 
-    if ( !$bigbluebuttonbn->visible ) {
+    if (!$bigbluebuttonbn->visible) {
         // Nothing to do
     } else {
         $modPW = $bigbluebuttonbn->moderatorpass;
         $attPW = $bigbluebuttonbn->viewerpass;
 
-        $meetingID = $bigbluebuttonbn->meetingid.'-'.$course->id.'-'.$bigbluebuttonbn->id;
+        $meetingID = $bigbluebuttonbn->meetingid . '-' . $course->id . '-' . $bigbluebuttonbn->id;
         //
         // Output Users in the meeting
         //
-        if( $groupObj == null ){
-            $meetingInfo = bigbluebuttonbn_getMeetingInfoArray( $meetingID, $modPW, $endpoint, $shared_secret );
-            $joinURL = '<a href="view.php?id='.$bigbluebuttonbn->coursemodule.'">'.format_string($bigbluebuttonbn->name).'</a>';
+        if ($groupObj == null) {
+            $meetingInfo = bigbluebuttonbn_getMeetingInfoArray($meetingID, $modPW, $endpoint, $shared_secret);
+            $joinURL = '<a href="view.php?id=' . $bigbluebuttonbn->coursemodule . '">' . format_string($bigbluebuttonbn->name) . '</a>';
         } else {
-            $meetingInfo = bigbluebuttonbn_getMeetingInfoArray( $meetingID.'['.$groupObj->id.']', $modPW, $endpoint, $shared_secret );
-            $joinURL = '<a href="view.php?id='.$bigbluebuttonbn->coursemodule.'&group='.$groupObj->id.'">'.format_string($bigbluebuttonbn->name).'</a>';
+            $meetingInfo = bigbluebuttonbn_getMeetingInfoArray($meetingID . '[' . $groupObj->id . ']', $modPW, $endpoint, $shared_secret);
+            $joinURL = '<a href="view.php?id=' . $bigbluebuttonbn->coursemodule . '&group=' . $groupObj->id . '">' . format_string($bigbluebuttonbn->name) . '</a>';
             $group = $groupObj->name;
         }
         
@@ -163,7 +163,7 @@ 
             //
             // The server was unreachable
             //
-            print_error( get_string( 'index_error_unable_display', 'bigbluebuttonbn' ));
+            print_error(get_string('index_error_unable_display', 'bigbluebuttonbn'));
             return;
         }
 
@@ -172,7 +172,7 @@ 
             // There was an error returned
             //
             if ($meetingInfo['messageKey'] == "checksumError") {
-                print_error( get_string( 'index_error_checksum', 'bigbluebuttonbn' ));
+                print_error(get_string('index_error_checksum', 'bigbluebuttonbn'));
                 return;
             }
 
@@ -184,39 +184,39 @@ 
                 //
                 // There was an error
                 //
-                $users = $meetingInfo['messageKey'].": ".$meetingInfo['message'];
+                $users = $meetingInfo['messageKey'] . ": " . $meetingInfo['message'];
             }
         } else {
             //
             // The meeting info was returned
             //
             if ($meetingInfo['running'] == 'true') {
-                if ( $moderator ) {
-                    if( $groupObj == null ) {
-                        $actions = '<form name="form1" method="post" action=""><INPUT type="hidden" name="id" value="'.$course->id.'"><INPUT type="hidden" name="a" value="'.$bigbluebuttonbn->id.'"><INPUT type="submit" name="submit" value="end" onclick="return confirm(\''. get_string('index_confirm_end', 'bigbluebuttonbn' ).'\')"></form>';
+                if ($moderator) {
+                    if ($groupObj == null) {
+                        $actions = '<form name="form1" method="post" action=""><INPUT type="hidden" name="id" value="' . $course->id . '"><INPUT type="hidden" name="a" value="' . $bigbluebuttonbn->id . '"><INPUT type="submit" name="submit" value="end" onclick="return confirm(\'' . get_string('index_confirm_end', 'bigbluebuttonbn') . '\')"></form>';
                     } else {
-                        $actions = '<form name="form1" method="post" action=""><INPUT type="hidden" name="id" value="'.$course->id.'"><INPUT type="hidden" name="a" value="'.$bigbluebuttonbn->id.'"><INPUT type="hidden" name="g" value="'.$groupObj->id.'"><INPUT type="submit" name="submit" value="end" onclick="return confirm(\''. get_string('index_confirm_end', 'bigbluebuttonbn' ).'\')"></form>';
+                        $actions = '<form name="form1" method="post" action=""><INPUT type="hidden" name="id" value="' . $course->id . '"><INPUT type="hidden" name="a" value="' . $bigbluebuttonbn->id . '"><INPUT type="hidden" name="g" value="' . $groupObj->id . '"><INPUT type="submit" name="submit" value="end" onclick="return confirm(\'' . get_string('index_confirm_end', 'bigbluebuttonbn') . '\')"></form>';
                     }
                 }
-                if ( isset($meetingInfo['recording']) && $meetingInfo['recording'] == 'true' ){ // if it has been set when meeting created, set the variable on/off
-                    $recording = get_string('index_enabled', 'bigbluebuttonbn' );
+                if (isset($meetingInfo['recording']) && $meetingInfo['recording'] == 'true') { // if it has been set when meeting created, set the variable on/off
+                    $recording = get_string('index_enabled', 'bigbluebuttonbn');
                 }
                  
                 $xml = $meetingInfo['attendees'];
-                if (count( $xml ) && count( $xml->attendee ) ) {
-                    $users = count( $xml->attendee );
+                if (count($xml) && count($xml->attendee)) {
+                    $users = count($xml->attendee);
                     $viewer_count = 0;
                     $moderator_count = 0;
-                    foreach ( $xml->attendee as $attendee ) {
-                        if ($attendee->role == "MODERATOR" ) {
-                            if ( $viewer_count++ > 0 ) {
+                    foreach ($xml->attendee as $attendee) {
+                        if ($attendee->role == "MODERATOR") {
+                            if ($viewer_count++ > 0) {
                                 $moderatorList .= ", ";
                             } else {
                                 $moderatorList = "";
                             }
                             $moderatorList .= $attendee->fullName;
                         } else {
-                            if ( $moderator_count++ > 0 ) {
+                            if ($moderator_count++ > 0) {
                                 $viewerList .= ", ";
                             } else {
                                 $viewerList = "";
@@ -228,6 +228,6 @@ 
             }
         }
 
-        return array ($bigbluebuttonbn->section, $joinURL, $group, $users, $viewerList, $moderatorList, $recording, $actions );
+        return array($bigbluebuttonbn->section, $joinURL, $group, $users, $viewerList, $moderatorList, $recording, $actions);
     }
 }
\ No newline at end of file

JWT.php

Spacing +4 added lines, -4 removed lines

@@ -150,7 +150,7 @@ 
             throw new DomainException('Algorithm not supported');
         }
         list($function, $algorithm) = self::$supported_algs[$alg];
-        switch($function) {
+        switch ($function) {
             case 'hash_hmac':
                 return hash_hmac($algorithm, $msg, $key, true);
             case 'openssl':
@@ -181,7 +181,7 @@ 
         }
 
         list($function, $algorithm) = self::$supported_algs[$alg];
-        switch($function) {
+        switch ($function) {
             case 'openssl':
                 $success = openssl_verify($msg, $signature, $key, $algorithm);
                 if (!$success) {
@@ -228,8 +228,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);
         }
 

import_view.php

Spacing +12 added lines, -12 removed lines

@@ -8,11 +8,11 @@ 
  * @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);     // bigbluebuttonbn instance ID
-$tc = optional_param('tc', 0, PARAM_INT);  // target course ID
+$bn = required_param('bn', PARAM_INT); // bigbluebuttonbn instance ID
+$tc = optional_param('tc', 0, PARAM_INT); // target course ID
 
 if ($bn) {
     $bigbluebuttonbn = $DB->get_record('bigbluebuttonbn', array('id' => $bn), '*', MUST_EXIST);
@@ -26,7 +26,7 @@ 
 
 require_login($course, true, $cm);
 
-if ( isset($SESSION) && isset($SESSION->bigbluebuttonbn_bbbsession)) {
+if (isset($SESSION) && isset($SESSION->bigbluebuttonbn_bbbsession)) {
     $bbbsession = $SESSION->bigbluebuttonbn_bbbsession;
 }
 
@@ -47,7 +47,7 @@ 
 
 $options = bigbluebuttonbn_import_get_courses_for_select($bbbsession);
 $selected = bigbluebuttonbn_selected_course($options, $tc);
-if( empty($options) ) {
+if (empty($options)) {
     $output .= html_writer::tag('div', get_string('view_error_import_no_courses', 'bigbluebuttonbn'));
 
 } else {
@@ -58,14 +58,14 @@ 
     $recordings = bigbluebuttonbn_import_exlcude_recordings_already_imported($bbbsession['course']->id, $bbbsession['bigbluebuttonbn']->id, $recordings);
     //store remaining recordings (indexed) in a session variable
     $SESSION->bigbluebuttonbn_importrecordings = bigbluebuttonbn_index_recordings($recordings);
-    if( empty($recordings) ) {
+    if (empty($recordings)) {
         $output .= html_writer::tag('div', get_string('view_error_import_no_recordings', 'bigbluebuttonbn'));
     } else {
-        $output .= html_writer::tag('span', '', ['id' => 'import_recording_links_table' ,'name'=>'import_recording_links_table']);
+        $output .= html_writer::tag('span', '', ['id' => 'import_recording_links_table', 'name'=>'import_recording_links_table']);
         $output .= bigbluebutton_output_recording_table($bbbsession, $recordings, ['importing']);
     }
     $output .= html_writer::start_tag('br');
-    $output .= html_writer::tag( 'input', '', array('type' => 'button', 'value' => get_string('view_recording_button_return', 'bigbluebuttonbn'), 'onclick' => 'window.location=\''.$CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$cm->id.'\'') );
+    $output .= html_writer::tag('input', '', array('type' => 'button', 'value' => get_string('view_recording_button_return', 'bigbluebuttonbn'), 'onclick' => 'window.location=\'' . $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $cm->id . '\''));
 
     $jsvars = array(
         'bn' => $bn,
@@ -87,10 +87,10 @@ 
 // finally, render the output
 echo $output;
 
-function bigbluebuttonbn_selected_course($options, $tc='') {
-    if( empty($options) ) {
+function bigbluebuttonbn_selected_course($options, $tc = '') {
+    if (empty($options)) {
         $selected = '';
-    } else if(array_key_exists($tc, $options)) {
+    } else if (array_key_exists($tc, $options)) {
         $selected = $tc;
     } else {
         $selected = '';

backup/moodle2/backup_bigbluebuttonbn_activity_task.class.php

Spacing +3 added lines, -3 removed lines

@@ -54,14 +54,14 @@
     static public 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;

bbb_view.php

Spacing +38 added lines, -38 removed lines

@@ -8,11 +8,11 @@ 
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v2 or later
  */
 
-require_once(dirname(dirname(dirname(__FILE__))).'/config.php');
-require_once(dirname(__FILE__).'/locallib.php');
+require_once(dirname(dirname(dirname(__FILE__))) . '/config.php');
+require_once(dirname(__FILE__) . '/locallib.php');
 
-$id = optional_param('id', 0, PARAM_INT);  // course_module ID, or
-$bn = optional_param('bn', 0, PARAM_INT);  // bigbluebuttonbn instance ID
+$id = optional_param('id', 0, PARAM_INT); // course_module ID, or
+$bn = optional_param('bn', 0, PARAM_INT); // bigbluebuttonbn instance ID
 $action = required_param('action', PARAM_TEXT);
 $name = optional_param('name', '', PARAM_TEXT);
 $description = optional_param('description', '', PARAM_TEXT);
@@ -43,7 +43,7 @@ 
 
 require_login($course, true, $cm);
 
-if ( isset($SESSION) && isset($SESSION->bigbluebuttonbn_bbbsession)) {
+if (isset($SESSION) && isset($SESSION->bigbluebuttonbn_bbbsession)) {
     $bbbsession = $SESSION->bigbluebuttonbn_bbbsession;
 }
 switch (strtolower($action)) {
@@ -51,7 +51,7 @@ 
         if (isset($errors) && $errors != '') {
           bigbluebutton_bbb_view_errors($errors, $id);
 
-        } else if ( isset($bbbsession) && !is_null($bbbsession) ) {
+        } else if (isset($bbbsession) && !is_null($bbbsession)) {
             /// Moodle event logger: Create an event for meeting left
             bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_LEFT, $bigbluebuttonbn, $context, $cm);
 
@@ -66,15 +66,15 @@ 
         }
         break;
     case 'join':
-        if ( isset($bbbsession) && !is_null($bbbsession) ) {
+        if (isset($bbbsession) && !is_null($bbbsession)) {
             //See if the session is in progress
-            if( bigbluebuttonbn_isMeetingRunning( $bbbsession['meetingid'], $bbbsession['endpoint'], $bbbsession['shared_secret'] ) ) {
+            if (bigbluebuttonbn_isMeetingRunning($bbbsession['meetingid'], $bbbsession['endpoint'], $bbbsession['shared_secret'])) {
                 /// Since the meeting is already running, we just join the session
                 bigbluebutton_bbb_view_execute_join($bbbsession, $cm, $context, $bigbluebuttonbn);
 
             } else {
                 // If user is administrator, moderator or if is viewer and no waiting is required
-                if( $bbbsession['administrator'] || $bbbsession['moderator'] || !$bbbsession['wait'] ) {
+                if ($bbbsession['administrator'] || $bbbsession['moderator'] || !$bbbsession['wait']) {
                     /// Prepare the metadata
                     $metadata = array("meta_bn-origin" => $bbbsession['origin'],
                             "meta_bbb-origin-version" => $bbbsession['originVersion'],
@@ -82,20 +82,20 @@ 
                             "meta_bbb-origin-server-common-name" => $bbbsession['originServerCommonName'],
                             "meta_bbb-origin-tag" => $bbbsession['originTag'],
                             "meta_bbb-context" => $bbbsession['course']->fullname,
-                            "meta_bbb-recording-name" => (isset($name) && $name != '')? $name: $bbbsession['contextActivityName'],
-                            "meta_bbb-recording-description" => (isset($description) && $description != '')? $description: $bbbsession['contextActivityDescription'],
-                            "meta_bbb-recording-tags" => (isset($tags) && $tags != '')? $tags: $bbbsession['contextActivityTags'],
+                            "meta_bbb-recording-name" => (isset($name) && $name != '') ? $name : $bbbsession['contextActivityName'],
+                            "meta_bbb-recording-description" => (isset($description) && $description != '') ? $description : $bbbsession['contextActivityDescription'],
+                            "meta_bbb-recording-tags" => (isset($tags) && $tags != '') ? $tags : $bbbsession['contextActivityTags'],
                     );
 
-                    if ( bigbluebuttonbn_server_offers_bn_capabilities() && bigbluebuttonbn_get_cfg_recordingready_enabled() ) {
+                    if (bigbluebuttonbn_server_offers_bn_capabilities() && bigbluebuttonbn_get_cfg_recordingready_enabled()) {
                         $metadata["meta_bn-recording-ready-url"] = $bbbsession['recordingReadyURL'];
                     }
 
                     /// Set the duration for the meeting
-                    if ( bigbluebuttonbn_get_cfg_scheduled_duration_enabled() ) {
+                    if (bigbluebuttonbn_get_cfg_scheduled_duration_enabled()) {
                         $durationtime = bigbluebuttonbn_get_duration($bigbluebuttonbn->openingtime, $bigbluebuttonbn->closingtime);
-                        if( $durationtime > 0 )
-                            $bbbsession['welcome'] .= '<br><br>'.str_replace("%duration%", ''.$durationtime, get_string('bbbdurationwarning', 'bigbluebuttonbn'));
+                        if ($durationtime > 0)
+                            $bbbsession['welcome'] .= '<br><br>' . str_replace("%duration%", '' . $durationtime, get_string('bbbdurationwarning', 'bigbluebuttonbn'));
                     } else {
                         $durationtime = 0;
                     }
@@ -109,7 +109,7 @@ 
                             $bbbsession['shared_secret'],
                             $bbbsession['endpoint'],
                             $bbbsession['logoutURL'],
-                            $bbbsession['record']? 'true': 'false',
+                            $bbbsession['record'] ? 'true' : 'false',
                             $durationtime,
                             $bbbsession['voicebridge'],
                             $bbbsession['userlimit'],
@@ -120,25 +120,25 @@ 
 
                     if (!$response) {
                         // If the server is unreachable, then prompts the user of the necessary action
-                        if ( $bbbsession['administrator'] ) {
-                            print_error( 'view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
-                        } else if ( $bbbsession['moderator'] ) {
-                            print_error( 'view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
+                        if ($bbbsession['administrator']) {
+                            print_error('view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
+                        } else if ($bbbsession['moderator']) {
+                            print_error('view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
                         } else {
-                            print_error( 'view_error_unable_join_student', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
+                            print_error('view_error_unable_join_student', 'bigbluebuttonbn', $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
                         }
 
-                    } else if( $response['returncode'] == "FAILED" ) {
+                    } else if ($response['returncode'] == "FAILED") {
                         // The meeting was not created
-                        $error_key = bigbluebuttonbn_get_error_key( $response['messageKey'], 'view_error_create' );
-                        if( !$error_key ) {
-                            print_error( $response['message'], 'bigbluebuttonbn' );
+                        $error_key = bigbluebuttonbn_get_error_key($response['messageKey'], 'view_error_create');
+                        if (!$error_key) {
+                            print_error($response['message'], 'bigbluebuttonbn');
                         } else {
-                            print_error( $error_key, 'bigbluebuttonbn' );
+                            print_error($error_key, 'bigbluebuttonbn');
                         }
 
-                    } else if ($response['hasBeenForciblyEnded'] == "true"){
-                        print_error( get_string( 'index_error_forciblyended', 'bigbluebuttonbn' ));
+                    } else if ($response['hasBeenForciblyEnded'] == "true") {
+                        print_error(get_string('index_error_forciblyended', 'bigbluebuttonbn'));
 
                     } else { ///////////////Everything is ok /////////////////////
                         /// Moodle event logger: Create an event for meeting created
@@ -150,12 +150,12 @@ 
                     }
 
                 } else {
-                    header('Location: '.$bbbsession['logoutURL'] );
+                    header('Location: ' . $bbbsession['logoutURL']);
                 }
             }
 
         } else {
-            print_error( 'view_error_unable_join', 'bigbluebuttonbn' );
+            print_error('view_error_unable_join', 'bigbluebuttonbn');
         }
         break;
     default:
@@ -179,9 +179,9 @@ 
 function bigbluebutton_bbb_view_execute_join($bbbsession, $cm, $context, $bigbluebuttonbn) {
     //// Update the cache
     $meeting_info = bigbluebuttonbn_bbb_broker_get_meeting_info($bbbsession['meetingid'], $bbbsession['modPW'], true);
-    if( $bbbsession['userlimit'] == 0 || intval($meeting_info['participantCount']) < $bbbsession['userlimit']  ) {
+    if ($bbbsession['userlimit'] == 0 || intval($meeting_info['participantCount']) < $bbbsession['userlimit']) {
         //// Build the URL
-        if( $bbbsession['administrator'] || $bbbsession['moderator'] ) {
+        if ($bbbsession['administrator'] || $bbbsession['moderator']) {
             $password = $bbbsession['modPW'];
         } else {
             $password = $bbbsession['viewerPW'];
@@ -192,25 +192,25 @@ 
         /// Internal logger: Instert a record with the meeting created
         bigbluebuttonbn_logs($bbbsession, BIGBLUEBUTTONBN_LOG_EVENT_JOIN);
         //// Before executing the redirect, increment the number of participants
-        bigbluebuttonbn_bbb_broker_participant_joined($bbbsession['meetingid'], ($bbbsession['administrator'] || $bbbsession['moderator']) );
+        bigbluebuttonbn_bbb_broker_participant_joined($bbbsession['meetingid'], ($bbbsession['administrator'] || $bbbsession['moderator']));
         //// Execute the redirect
-        header('Location: '.$join_url );
+        header('Location: ' . $join_url);
 
     } else {
-        header('Location: '.$bbbsession['logoutURL'] );
+        header('Location: ' . $bbbsession['logoutURL']);
     }
 }
 
 function bigbluebutton_bbb_view_errors($sErrors, $id) {
     global $CFG, $OUTPUT, $PAGE;
 
-    $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";
     }
 
     echo $OUTPUT->header();
-    print_error( 'view_error_bigbluebutton', 'bigbluebuttonbn', $CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$id, $msgErrors, $sErrors );
+    print_error('view_error_bigbluebutton', 'bigbluebuttonbn', $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $id, $msgErrors, $sErrors);
     echo $OUTPUT->footer();
 }

bbb_broker.php

Spacing +64 added lines, -64 removed lines

@@ -8,15 +8,15 @@ 
  * @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');
 
 global $PAGE, $USER, $CFG, $SESSION, $DB;
 
-$params['action']  = optional_param('action', '', PARAM_TEXT);
+$params['action'] = optional_param('action', '', PARAM_TEXT);
 $params['callback'] = optional_param('callback', '', PARAM_TEXT);
-$params['id'] = optional_param('id', '', PARAM_TEXT);    //recordID, the BBB recordID
-$params['idx'] = optional_param('idx', '', PARAM_TEXT);  //meetingID, the BBB meetingID
+$params['id'] = optional_param('id', '', PARAM_TEXT); //recordID, the BBB recordID
+$params['idx'] = optional_param('idx', '', PARAM_TEXT); //meetingID, the BBB meetingID
 $params['bigbluebuttonbn'] = optional_param('bigbluebuttonbn', 0, PARAM_INT);
 $params['signed_parameters'] = optional_param('signed_parameters', '', PARAM_TEXT);
 
@@ -25,13 +25,13 @@ 
 
 $error = '';
 
-if( empty($params['action']) ) {
+if (empty($params['action'])) {
     $error = bigbluebuttonbn_bbb_broker_add_error($error, "Parameter [action] was not included");
 
 } else {
     $error = bigbluebuttonbn_bbb_broker_validate_parameters($params);
 
-    if( empty($error) && $params['action'] != "recording_ready" ) {
+    if (empty($error) && $params['action'] != "recording_ready") {
 
         if ($params['bigbluebuttonbn'] != 0) {
             $bigbluebuttonbn = $DB->get_record('bigbluebuttonbn', array('id' => $params['bigbluebuttonbn']), '*', MUST_EXIST);
@@ -40,7 +40,7 @@ 
             $context = bigbluebuttonbn_get_context_module($cm->id);
         }
 
-        if ( isset($SESSION->bigbluebuttonbn_bbbsession) && !is_null($SESSION->bigbluebuttonbn_bbbsession) ) {
+        if (isset($SESSION->bigbluebuttonbn_bbbsession) && !is_null($SESSION->bigbluebuttonbn_bbbsession)) {
             $bbbsession = $SESSION->bigbluebuttonbn_bbbsession;
         } else {
             $error = bigbluebuttonbn_bbb_broker_add_error($error, "No session variable set");
@@ -49,7 +49,7 @@ 
 }
 
 header('Content-Type: application/javascript; charset=utf-8');
-if ( empty($error) ) {
+if (empty($error)) {
 
     if (!isloggedin() && $PAGE->course->id == SITEID) {
         $userid = guest_user()->id;
@@ -58,21 +58,21 @@ 
     }
     $hascourseaccess = ($PAGE->course->id == SITEID) || can_access_course($PAGE->course, $userid);
 
-    if( !$hascourseaccess ){
+    if (!$hascourseaccess) {
         header("HTTP/1.0 401 Unauthorized");
         return;
     } else {
         try {
-            switch ( strtolower($params['action']) ){
+            switch (strtolower($params['action'])) {
                 case 'meeting_info':
                     $meeting_info = bigbluebuttonbn_bbb_broker_get_meeting_info($params['id'], $bbbsession['modPW']);
                     $meeting_running = bigbluebuttonbn_bbb_broker_is_meeting_running($meeting_info); 
 
                     $status_can_end = '';
                     $status_can_tag = '';
-                    if( $meeting_running ) {
+                    if ($meeting_running) {
                         $join_button_text = get_string('view_conference_action_join', 'bigbluebuttonbn');
-                        if( $bbbsession['userlimit'] == 0 || $meeting_info->participantCount < $bbbsession['userlimit'] ) {
+                        if ($bbbsession['userlimit'] == 0 || $meeting_info->participantCount < $bbbsession['userlimit']) {
                             $initial_message = get_string('view_message_conference_in_progress', 'bigbluebuttonbn');
                             $can_join = true;
 
@@ -81,50 +81,50 @@ 
                             $can_join = false;
                         }
 
-                        if( $bbbsession['administrator'] || $bbbsession['moderator'] ) {
+                        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
-                        if ( $bbbsession['administrator'] || $bbbsession['moderator'] || !$bbbsession['wait'] ) {
+                        if ($bbbsession['administrator'] || $bbbsession['moderator'] || !$bbbsession['wait']) {
                             $initial_message = get_string('view_message_conference_room_ready', 'bigbluebuttonbn');
                             $join_button_text = get_string('view_conference_action_join', 'bigbluebuttonbn');
                             $can_join = true;
 
                         } 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');
+                            if ($bbbsession['wait']) {
+                                $initial_message .= ' ' . get_string('view_message_conference_wait_for_moderator', 'bigbluebuttonbn');
                             }
                             $join_button_text = get_string('view_conference_action_lineup', 'bigbluebuttonbn');
                             $can_join = false;
                         }
 
-                        if( $bbbsession['tagging'] && ($bbbsession['administrator'] || $bbbsession['moderator']) ) {
+                        if ($bbbsession['tagging'] && ($bbbsession['administrator'] || $bbbsession['moderator'])) {
                             $can_tag = true;
 
                         } 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'] ) {
+                    if ($bbbsession['administrator'] || $bbbsession['moderator']) {
                         //Execute the end command
                         $meeting_info = bigbluebuttonbn_bbb_broker_do_end_meeting($params['id'], $bbbsession['modPW']);
                         // Moodle event logger: Create an event for meeting ended
-                        if( isset($bigbluebuttonbn) )
+                        if (isset($bigbluebuttonbn))
                             bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_ENDED, $bigbluebuttonbn, $context, $cm);
                         // Update the cache
                         $meeting_info = bigbluebuttonbn_bbb_broker_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");
@@ -133,26 +133,26 @@ 
                 case 'recording_list':
                     break;
                 case 'recording_info':
-                    if( $bbbsession['managerecordings'] ) {
+                    if ($bbbsession['managerecordings']) {
                         //Retrieve the array of imported recordings
-                        $recordings_imported = bigbluebuttonbn_getRecordingsImportedArray($bbbsession['course']->id, isset($bbbsession['bigbluebuttonbn'])?$bbbsession['bigbluebuttonbn']->id: NULL);
+                        $recordings_imported = bigbluebuttonbn_getRecordingsImportedArray($bbbsession['course']->id, isset($bbbsession['bigbluebuttonbn']) ? $bbbsession['bigbluebuttonbn']->id : NULL);
                         $recordings_indexed = bigbluebuttonbn_index_recordings($recordings_imported);
-                        if( isset($recordings_indexed[$params['id']]) ) {
+                        if (isset($recordings_indexed[$params['id']])) {
                             //Look up for an update on the imported recording
                             $recording = $recordings_indexed[$params['id']];
-                            if ( isset($recording) && !empty($recording) && !array_key_exists('messageKey', $recording)) {  // The recording was found
-                                echo $params['callback'].'({ "status": "true", "published": "'.$recording['published'].'"});';
+                            if (isset($recording) && !empty($recording) && !array_key_exists('messageKey', $recording)) {  // The recording was found
+                                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_getRecordingArray($params['id'], $params['idx'], $endpoint, $shared_secret);
-                            if ( isset($recording) && !empty($recording) && !array_key_exists('messageKey', $recording)) {  // The recording was found
-                                echo $params['callback'].'({ "status": "true", "published": "'.$recording['published'].'"});';
+                            if (isset($recording) && !empty($recording) && !array_key_exists('messageKey', $recording)) {  // The recording was found
+                                echo $params['callback'] . '({ "status": "true", "published": "' . $recording['published'] . '"});';
                             } else {
-                                echo $params['callback'].'({ "status": "false" });';
+                                echo $params['callback'] . '({ "status": "false" });';
                             }
                         }
 
@@ -162,16 +162,16 @@ 
                     }
                     break;
                 case 'recording_publish':
-                    if( $bbbsession['managerecordings'] ) {
+                    if ($bbbsession['managerecordings']) {
                         $status = true;
                         //Retrieve the array of imported recordings for the current course and activity
-                        $recordings_imported = bigbluebuttonbn_getRecordingsImportedArray($bbbsession['course']->id, isset($bbbsession['bigbluebuttonbn'])?$bbbsession['bigbluebuttonbn']->id: NULL);
+                        $recordings_imported = bigbluebuttonbn_getRecordingsImportedArray($bbbsession['course']->id, isset($bbbsession['bigbluebuttonbn']) ? $bbbsession['bigbluebuttonbn']->id : NULL);
                         $recordings_imported_indexed = bigbluebuttonbn_index_recordings($recordings_imported);
-                        if( isset($recordings_imported_indexed[$params['id']]) ) {
+                        if (isset($recordings_imported_indexed[$params['id']])) {
                             $recordings = bigbluebuttonbn_getRecordingsArray($recordings_imported_indexed[$params['id']]['meetingID'], $bbbsession['endpoint'], $bbbsession['shared_secret']);
                             $recordings_indexed = bigbluebuttonbn_index_recordings($recordings);
                             $recording = $recordings_indexed[$params['id']];
-                            if ( $recording['published'] === 'true' ) {
+                            if ($recording['published'] === 'true') {
                                 // Only if the physical recording is published, execute publish on imported recording link
                                 $meeting_info = bigbluebuttonbn_bbb_broker_do_publish_recording_imported($params['id'], $bbbsession['course']->id, $bbbsession['bigbluebuttonbn']->id, true);
                             } else {
@@ -183,11 +183,11 @@ 
                             $meeting_info = bigbluebuttonbn_bbb_broker_do_publish_recording($params['id'], true);
                         }
 
-                        if ( $status ) {
+                        if ($status) {
                             $callback_response['status'] = "true";
 
                             // Moodle event logger: Create an event for recording published
-                            if( isset($bigbluebuttonbn) ) {
+                            if (isset($bigbluebuttonbn)) {
                                 bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_RECORDING_PUBLISHED, $bigbluebuttonbn, $context, $cm);
                             }
 
@@ -205,11 +205,11 @@ 
                     }
                     break;
                 case 'recording_unpublish':
-                    if( $bbbsession['managerecordings'] ) {
+                    if ($bbbsession['managerecordings']) {
                         //Retrieve the array of imported recordings for the current course and activity
-                        $recordings_imported = bigbluebuttonbn_getRecordingsImportedArray($bbbsession['course']->id, isset($bbbsession['bigbluebuttonbn'])?$bbbsession['bigbluebuttonbn']->id: NULL);
+                        $recordings_imported = bigbluebuttonbn_getRecordingsImportedArray($bbbsession['course']->id, isset($bbbsession['bigbluebuttonbn']) ? $bbbsession['bigbluebuttonbn']->id : NULL);
                         $recordings_indexed = bigbluebuttonbn_index_recordings($recordings_imported);
-                        if( isset($recordings_indexed[$params['id']]) ) {
+                        if (isset($recordings_indexed[$params['id']])) {
                             // Execute unpublish on imported recording link
                             $meeting_info = bigbluebuttonbn_bbb_broker_do_publish_recording_imported($params['id'], $bbbsession['course']->id, $bbbsession['bigbluebuttonbn']->id, false);
                         } else {
@@ -217,7 +217,7 @@ 
                             // First: Unpublish imported links associated to the recording 
                             $recordings_imported_all = bigbluebuttonbn_getRecordingsImportedAllInstances($params['id']);
 
-                            if( count($recordings_imported_all) > 0 ) {
+                            if (count($recordings_imported_all) > 0) {
                                 foreach ($recordings_imported_all as $key => $record) {
                                     $meta = json_decode($record->meta, true);
                                     // Prepare data for the update
@@ -233,7 +233,7 @@ 
                         }
 
                         // Moodle event logger: Create an event for recording unpublished
-                        if( isset($bigbluebuttonbn) ) {
+                        if (isset($bigbluebuttonbn)) {
                             bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_RECORDING_UNPUBLISHED, $bigbluebuttonbn, $context, $cm);
                         }
 
@@ -247,11 +247,11 @@ 
                     }
                     break;
                 case 'recording_delete':
-                    if( $bbbsession['managerecordings'] ) {
+                    if ($bbbsession['managerecordings']) {
                         //Retrieve the array of imported recordings
-                        $recordings_imported = bigbluebuttonbn_getRecordingsImportedArray($bbbsession['course']->id, isset($bbbsession['bigbluebuttonbn'])?$bbbsession['bigbluebuttonbn']->id: NULL);
+                        $recordings_imported = bigbluebuttonbn_getRecordingsImportedArray($bbbsession['course']->id, isset($bbbsession['bigbluebuttonbn']) ? $bbbsession['bigbluebuttonbn']->id : NULL);
                         $recordings_indexed = bigbluebuttonbn_index_recordings($recordings_imported);
-                        if( isset($recordings_indexed[$params['id']]) ) {
+                        if (isset($recordings_indexed[$params['id']])) {
                             // Execute unpublish on imported recording link
                             bigbluebuttonbn_bbb_broker_do_delete_recording_imported($params['id'], $bbbsession['course']->id, $bbbsession['bigbluebuttonbn']->id);
                         } else {
@@ -259,7 +259,7 @@ 
                             // First: Delete imported links associated to the recording 
                             $recordings_imported_all = bigbluebuttonbn_getRecordingsImportedAllInstances($params['id']);
 
-                            if( count($recordings_imported_all) > 0 ) {
+                            if (count($recordings_imported_all) > 0) {
                                 foreach ($recordings_imported_all as $key => $record) {
                                     // Execute delete
                                     $DB->delete_records("bigbluebuttonbn_logs", array('id' => $key));
@@ -270,7 +270,7 @@ 
                         }
 
                         // Moodle event logger: Create an event for recording deleted
-                        if( isset($bigbluebuttonbn) ) {
+                        if (isset($bigbluebuttonbn)) {
                             bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_RECORDING_DELETED, $bigbluebuttonbn, $context, $cm);
                         }
 
@@ -289,9 +289,9 @@ 
                         $decoded_parameters = 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;
                     }
 
@@ -302,9 +302,9 @@ 
                         $bigbluebuttonbn = $DB->get_record('bigbluebuttonbn', array('id' => $meeting_id_elements[2]), '*', MUST_EXIST);
 
                     } catch (Exception $e) {
-                        $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;
                     }
 
@@ -314,22 +314,22 @@ 
                         header("HTTP/1.0 202 Accepted");
                         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;
                     }
                     break;
                 case 'recording_import':
-                    if( $bbbsession['managerecordings'] ) {
+                    if ($bbbsession['managerecordings']) {
                         $importrecordings = $SESSION->bigbluebuttonbn_importrecordings;
-                        if( isset($importrecordings[$params['id']]) ) {
+                        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) ) {
+                            if (isset($bigbluebuttonbn)) {
                                 bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_RECORDING_IMPORTED, $bigbluebuttonbn, $context, $cm);
                             }
 
@@ -340,7 +340,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;
                         }
                     }
@@ -351,14 +351,14 @@ 
                     break;
             }
 
-        } catch(Exception $e) {
-            error_log("BBB_BROKER ERROR: ".$e->getCode().", ".$e->getMessage());
-            header("HTTP/1.0 502 Bad Gateway. ".$e->getMessage());
+        } catch (Exception $e) {
+            error_log("BBB_BROKER ERROR: " . $e->getCode() . ", " . $e->getMessage());
+            header("HTTP/1.0 502 Bad Gateway. " . $e->getMessage());
             return;
         }
     }
 
 } else {
-    header("HTTP/1.0 400 Bad Request. ".$error);
+    header("HTTP/1.0 400 Bad Request. " . $error);
     return;
 }
\ No newline at end of file

classes/event/bigbluebuttonbn_meeting_ended.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
      * @return string
      */
     public function get_description() {
-        $a = (object) array('userid' => $this->userid, 'bigbluebuttonbnid' => $this->objectid, 'courseid' => $this->contextinstanceid);
+        $a = (object)array('userid' => $this->userid, 'bigbluebuttonbnid' => $this->objectid, 'courseid' => $this->contextinstanceid);
         return get_string('event_meeting_ended_description', 'bigbluebuttonbn', $a);
     }
 

classes/event/bigbluebuttonbn_recording_published.php

Spacing +2 added lines, -2 removed lines

@@ -38,8 +38,8 @@
      * @return string
      */
     public function get_description() {
-        $rid = isset($this->other['rid'])? $this->other['rid']: '';
-        $a = (object) array('userid' => $this->userid, 'recordingid' => $rid, 'courseid' => $this->contextinstanceid);
+        $rid = isset($this->other['rid']) ? $this->other['rid'] : '';
+        $a = (object)array('userid' => $this->userid, 'recordingid' => $rid, 'courseid' => $this->contextinstanceid);
         return get_string('event_recording_published_description', 'bigbluebuttonbn', $a);
     }
 

classes/event/bigbluebuttonbn_recording_deleted.php

Spacing +2 added lines, -2 removed lines

@@ -38,8 +38,8 @@
      * @return string
      */
     public function get_description() {
-        $rid = isset($this->other['rid'])? $this->other['rid']: '';
-        $a = (object) array('userid' => $this->userid, 'recordingid' => $rid, 'courseid' => $this->contextinstanceid);
+        $rid = isset($this->other['rid']) ? $this->other['rid'] : '';
+        $a = (object)array('userid' => $this->userid, 'recordingid' => $rid, 'courseid' => $this->contextinstanceid);
         return get_string('event_recording_published_description', 'bigbluebuttonbn', $a);
     }