blindsidenetworks / moodle-mod_bigbluebuttonbn

classes/locallib/config.php

Indentation +24 added lines, -24 removed lines

@@ -120,29 +120,29 @@
      */
     public static function get_options() {
         return [
-               'version_major' => self::get_moodle_version_major(),
-               'voicebridge_editable' => self::get('voicebridge_editable'),
-               'waitformoderator_default' => self::get('waitformoderator_default'),
-               'waitformoderator_editable' => self::get('waitformoderator_editable'),
-               'userlimit_default' => self::get('userlimit_default'),
-               'userlimit_editable' => self::get('userlimit_editable'),
-               'preuploadpresentation_enabled' => self::get('preuploadpresentation_enabled'),
-               'sendnotifications_enabled' => self::get('sendnotifications_enabled'),
-               'recordings_enabled' => self::get('recordings_enabled'),
-               'recordings_html_default' => self::get('recordings_html_default'),
-               'recordings_html_editable' => self::get('recordings_html_editable'),
-               'recordings_deleted_default' => self::get('recordings_deleted_default'),
-               'recordings_deleted_editable' => self::get('recordings_deleted_editable'),
-               'recordings_imported_default' => self::get('recordings_imported_default'),
-               'recordings_imported_editable' => self::get('recordings_imported_editable'),
-               'recording_default' => self::get('recording_default'),
-               'recording_editable' => self::get('recording_editable'),
-               'recording_icons_enabled' => self::get('recording_icons_enabled'),
-               'general_warning_message' => self::get('general_warning_message'),
-               'general_warning_box_type' => self::get('general_warning_box_type'),
-               'general_warning_button_text' => self::get('general_warning_button_text'),
-               'general_warning_button_href' => self::get('general_warning_button_href'),
-               'general_warning_button_class' => self::get('general_warning_button_class'),
-          ];
+                'version_major' => self::get_moodle_version_major(),
+                'voicebridge_editable' => self::get('voicebridge_editable'),
+                'waitformoderator_default' => self::get('waitformoderator_default'),
+                'waitformoderator_editable' => self::get('waitformoderator_editable'),
+                'userlimit_default' => self::get('userlimit_default'),
+                'userlimit_editable' => self::get('userlimit_editable'),
+                'preuploadpresentation_enabled' => self::get('preuploadpresentation_enabled'),
+                'sendnotifications_enabled' => self::get('sendnotifications_enabled'),
+                'recordings_enabled' => self::get('recordings_enabled'),
+                'recordings_html_default' => self::get('recordings_html_default'),
+                'recordings_html_editable' => self::get('recordings_html_editable'),
+                'recordings_deleted_default' => self::get('recordings_deleted_default'),
+                'recordings_deleted_editable' => self::get('recordings_deleted_editable'),
+                'recordings_imported_default' => self::get('recordings_imported_default'),
+                'recordings_imported_editable' => self::get('recordings_imported_editable'),
+                'recording_default' => self::get('recording_default'),
+                'recording_editable' => self::get('recording_editable'),
+                'recording_icons_enabled' => self::get('recording_icons_enabled'),
+                'general_warning_message' => self::get('general_warning_message'),
+                'general_warning_box_type' => self::get('general_warning_box_type'),
+                'general_warning_button_text' => self::get('general_warning_button_text'),
+                'general_warning_button_href' => self::get('general_warning_button_href'),
+                'general_warning_button_class' => self::get('general_warning_button_class'),
+            ];
     }
 }

Spacing +4 added lines, -4 removed lines

@@ -55,8 +55,8 @@ 
      */
     public static function defaultvalues() {
         return array(
-            'server_url' => (string) BIGBLUEBUTTONBN_DEFAULT_SERVER_URL,
-            'shared_secret' => (string) BIGBLUEBUTTONBN_DEFAULT_SHARED_SECRET,
+            'server_url' => (string)BIGBLUEBUTTONBN_DEFAULT_SERVER_URL,
+            'shared_secret' => (string)BIGBLUEBUTTONBN_DEFAULT_SHARED_SECRET,
             'voicebridge_editable' => 'false',
             'importrecordings_enabled' => 'false',
             'importrecordings_from_deleted_enabled' => 'false',
@@ -119,8 +119,8 @@ 
         if (isset($CFG->bigbluebuttonbn[$setting])) {
             return (string)$CFG->bigbluebuttonbn[$setting];
         }
-        if (isset($CFG->{'bigbluebuttonbn_'.$setting})) {
-            return (string)$CFG->{'bigbluebuttonbn_'.$setting};
+        if (isset($CFG->{'bigbluebuttonbn_' . $setting})) {
+            return (string)$CFG->{'bigbluebuttonbn_' . $setting};
         }
         return  self::defaultvalue($setting);
     }

bbb_view.php

Indentation +15 added lines, -15 removed lines

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

Switch Indentation +75 added lines, -75 removed lines

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

Spacing +13 added lines, -13 removed lines

@@ -23,8 +23,8 @@ 
  * @author    Jesus Federico  (jesus [at] blindsidenetworks [dt] com)
  */
 
-require_once(dirname(dirname(dirname(__FILE__))).'/config.php');
-require_once(dirname(__FILE__).'/locallib.php');
+require_once(dirname(dirname(dirname(__FILE__))) . '/config.php');
+require_once(dirname(__FILE__) . '/locallib.php');
 
 $action = required_param('action', PARAM_TEXT);
 $id = optional_param('id', 0, PARAM_INT);
@@ -91,7 +91,7 @@ 
         }
         // If user is not administrator nor moderator (user is steudent) and waiting is required.
         if (!$bbbsession['administrator'] && !$bbbsession['moderator'] && $bbbsession['wait']) {
-            header('Location: '.$bbbsession['logoutURL']);
+            header('Location: ' . $bbbsession['logoutURL']);
             break;
         }
         // As the meeting doesn't exist, try to create it.
@@ -105,16 +105,16 @@ 
             // The server is unreachable.
             if ($bbbsession['administrator']) {
                 print_error('view_error_unable_join', 'bigbluebuttonbn',
-                    $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+                    $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
                 break;
             }
             if ($bbbsession['moderator']) {
                 print_error('view_error_unable_join_teacher', 'bigbluebuttonbn',
-                    $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+                    $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
                 break;
             }
             print_error('view_error_unable_join_student', 'bigbluebuttonbn',
-                $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+                $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
             break;
         }
         if ($response['returncode'] == 'FAILED') {
@@ -143,7 +143,7 @@ 
         // Moodle event logger: Create an event for meeting left.
         bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_RECORDING_VIEWED, $bigbluebuttonbn, $cm, ['other' => $rid]);
         // Execute the redirect.
-        header('Location: '.urldecode($href));
+        header('Location: ' . urldecode($href));
         break;
     default:
         bigbluebutton_bbb_view_close_window();
@@ -228,7 +228,7 @@ 
         $data['welcome'] .= '<br><br>';
         $data['welcome'] .= str_replace(
             '%duration%',
-            (string) $durationtime,
+            (string)$durationtime,
             get_string('bbbdurationwarning', 'bigbluebuttonbn')
           );
     }
@@ -316,7 +316,7 @@ 
     $meetinginfo = bigbluebuttonbn_get_meeting_info($bbbsession['meetingid'], BIGBLUEBUTTONBN_UPDATE_CACHE);
     if ($bbbsession['userlimit'] > 0 && intval($meetinginfo['participantCount']) >= $bbbsession['userlimit']) {
         // No more users allowed to join.
-        header('Location: '.$bbbsession['logoutURL']);
+        header('Location: ' . $bbbsession['logoutURL']);
         return;
     }
     // Build the URL.
@@ -334,7 +334,7 @@ 
     bigbluebuttonbn_participant_joined($bbbsession['meetingid'],
         ($bbbsession['administrator'] || $bbbsession['moderator']));
     // Execute the redirect.
-    header('Location: '.$joinurl);
+    header('Location: ' . $joinurl);
 }
 
 /**
@@ -346,13 +346,13 @@ 
  */
 function bigbluebutton_bbb_view_errors($serrors, $id) {
     global $CFG, $OUTPUT;
-    $errors = (array) json_decode(urldecode($serrors));
+    $errors = (array)json_decode(urldecode($serrors));
     $msgerrors = '';
     foreach ($errors as $error) {
-        $msgerrors .= html_writer::tag('p', $error->{'message'}, array('class' => 'alert alert-danger'))."\n";
+        $msgerrors .= html_writer::tag('p', $error->{'message'}, array('class' => 'alert alert-danger')) . "\n";
     }
     echo $OUTPUT->header();
     print_error('view_error_bigbluebutton', 'bigbluebuttonbn',
-        $CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$id, $msgerrors, $serrors);
+        $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $id, $msgerrors, $serrors);
     echo $OUTPUT->footer();
 }

bbb_broker.php

Indentation +18 added lines, -18 removed lines

@@ -366,26 +366,26 @@ 
             return array(
                 'status' => false,
                 'message' => get_string('view_recording_publish_link_deleted', 'bigbluebuttonbn')
-              );
+                );
         }
         if ($realrecordings[$params['id']]['published'] !== 'true') {
             return array(
                 'status' => false,
                 'message' => get_string('view_recording_publish_link_not_published', 'bigbluebuttonbn')
-              );
+                );
         }
         return array(
             'status' => bigbluebuttonbn_publish_recording_imported(
                 $recordings[$params['id']]['imported'], true
             )
-          );
+            );
     }
     // As the recordingid was not identified as imported recording link, execute actual publish.
     return array(
         'status' => bigbluebuttonbn_publish_recordings(
             $params['id'], 'true'
         )
-      );
+        );
 }
 
 /**
@@ -401,26 +401,26 @@ 
             return array(
                 'status' => false,
                 'message' => get_string('view_recording_unprotect_link_deleted', 'bigbluebuttonbn')
-              );
+                );
         }
         if ($realrecordings[$params['id']]['protected'] === 'true') {
             return array(
                 'status' => false,
                 'message' => get_string('view_recording_unprotect_link_not_unprotected', 'bigbluebuttonbn')
-              );
+                );
         }
         return array(
             'status' => bigbluebuttonbn_protect_recording_imported(
                 $recordings[$params['id']]['imported'], false
             )
-          );
+            );
     }
     // As the recordingid was not identified as imported recording link, execute actual uprotect.
     return array(
         'status' => bigbluebuttonbn_update_recordings(
             $params['id'], array('protect' => 'false')
         )
-      );
+        );
 }
 
 /**
@@ -434,7 +434,7 @@ 
             'status' => bigbluebuttonbn_publish_recording_imported(
                 $recordings[$params['id']]['imported'], false
             )
-          );
+            );
     }
     // As the recordingid was not identified as imported recording link, execute unpublish on a real recording.
     // First: Unpublish imported links associated to the recording.
@@ -452,7 +452,7 @@ 
         'status' => bigbluebuttonbn_publish_recordings(
             $params['id'], 'false'
         )
-      );
+        );
 }
 
 /**
@@ -466,7 +466,7 @@ 
             'status' => bigbluebuttonbn_protect_recording_imported(
                 $recordings[$params['id']]['imported'], true
             )
-          );
+            );
     }
     // As the recordingid was not identified as imported recording link, execute protect on a real recording.
     // First: Protect imported links associated to the recording.
@@ -484,7 +484,7 @@ 
         'status' => bigbluebuttonbn_update_recordings(
             $params['id'], array('protect' => 'true')
         )
-      );
+        );
 }
 
 /**
@@ -498,7 +498,7 @@ 
             'status' => bigbluebuttonbn_delete_recording_imported(
                 $recordings[$params['id']]['imported']
             )
-          );
+            );
     }
     // As the recordingid was not identified as imported recording link, execute delete on a real recording.
     // First: Delete imported links associated to the recording.
@@ -514,7 +514,7 @@ 
         'status' => bigbluebuttonbn_delete_recordings(
             $params['id']
         )
-      );
+        );
 }
 
 /**
@@ -527,7 +527,7 @@ 
             'status' => bigbluebuttonbn_update_recording_imported(
                 $recordings[$params['id']]['imported'], json_decode($params['meta'], true)
             )
-          );
+            );
     }
 
     // As the recordingid was not identified as imported recording link, execute update on a real recording.
@@ -537,7 +537,7 @@ 
         'status' => bigbluebuttonbn_update_recordings(
             $params['id'], json_decode($params['meta'])
         )
-      );
+        );
 }
 
 /**
@@ -682,10 +682,10 @@ 
     $params['recording_import'] = ['id' => 'The recordingID must be specified.'];
     $params['recording_ready'] = [
             'signed_parameters' => 'A JWT encoded string must be included as [signed_parameters].'
-          ];
+            ];
     $params['live_session_events'] = [
             'signed_parameters' => 'A JWT encoded string must be included as [signed_parameters].'
-          ];
+            ];
     return $params;
 }
 

Spacing +16 added lines, -16 removed lines

@@ -23,8 +23,8 @@ 
  * @author    Jesus Federico  (jesus [at] blindsidenetworks [dt] com)
  */
 
-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;
 
@@ -40,13 +40,13 @@ 
 $params['meta'] = optional_param('meta', '', PARAM_TEXT);
 
 if (empty($params['action'])) {
-    header('HTTP/1.0 400 Bad Request. Parameter ['.$params['action'].'] was not included');
+    header('HTTP/1.0 400 Bad Request. Parameter [' . $params['action'] . '] was not included');
     return;
 }
 
 $error = bigbluebuttonbn_broker_validate_parameters($params);
 if (!empty($error)) {
-    header('HTTP/1.0 400 Bad Request. '.$error);
+    header('HTTP/1.0 400 Bad Request. ' . $error);
     return;
 }
 
@@ -130,11 +130,11 @@ 
         bigbluebuttonbn_broker_live_session_events($params, $bigbluebuttonbn, $cm);
         return;
     }
-    header('HTTP/1.0 400 Bad request. The action '. $a . ' doesn\'t exist');
+    header('HTTP/1.0 400 Bad request. The action ' . $a . ' doesn\'t exist');
     return;
 
 } catch (Exception $e) {
-    header('HTTP/1.0 500 Internal Server Error. '.$e->getMessage());
+    header('HTTP/1.0 500 Internal Server Error. ' . $e->getMessage());
     return;
 }
 
@@ -320,7 +320,7 @@ 
 function bigbluebuttonbn_broker_recording_info_current($recording, $params) {
     $callbackresponse['status'] = true;
     $callbackresponse['found'] = true;
-    $callbackresponse['published'] = (string) $recording['published'];
+    $callbackresponse['published'] = (string)$recording['published'];
     if (!isset($params['meta']) || empty($params['meta'])) {
         return $callbackresponse;
     }
@@ -650,8 +650,8 @@ 
         $decodedparameters = JWT::decode($params['signed_parameters'],
             \mod_bigbluebuttonbn\locallib\config::get('shared_secret'), array('HS256'));
     } catch (Exception $e) {
-        $error = 'Caught exception: '.$e->getMessage();
-        header('HTTP/1.0 400 Bad Request. '.$error);
+        $error = 'Caught exception: ' . $e->getMessage();
+        header('HTTP/1.0 400 Bad Request. ' . $error);
         return;
     }
     // Validate that the bigbluebuttonbn activity corresponds to the meeting_id received.
@@ -667,8 +667,8 @@ 
         bigbluebuttonbn_send_notification_recording_ready($bigbluebuttonbn);
         header('HTTP/1.0 202 Accepted');
     } catch (Exception $e) {
-        $error = 'Caught exception: '.$e->getMessage();
-        header('HTTP/1.0 503 Service Unavailable. '.$error);
+        $error = 'Caught exception: ' . $e->getMessage();
+        header('HTTP/1.0 503 Service Unavailable. ' . $error);
     }
 }
 
@@ -689,13 +689,13 @@ 
     $importrecordings = $SESSION->bigbluebuttonbn_importrecordings;
     if (!isset($importrecordings[$params['id']])) {
         $error = "Recording {$params['id']} could not be found. It can not be imported";
-        header('HTTP/1.0 404 Not found. '.$error);
+        header('HTTP/1.0 404 Not found. ' . $error);
         return;
     }
     $callbackresponse = array('status' => true);
     $importrecordings[$params['id']]['imported'] = true;
     $overrides = array('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($bbbsession['bigbluebutton']) && isset($bbbsession['cm'])) {
@@ -721,8 +721,8 @@ 
         $decodedparameters = JWT::decode($params['signed_parameters'],
             \mod_bigbluebuttonbn\locallib\config::get('shared_secret'), array('HS256'));
     } catch (Exception $e) {
-        $error = 'Caught exception: '.$e->getMessage();
-        header('HTTP/1.0 400 Bad Request. '.$error);
+        $error = 'Caught exception: ' . $e->getMessage();
+        header('HTTP/1.0 400 Bad Request. ' . $error);
         return;
     }
     // Validate that the bigbluebuttonbn activity corresponds to the meeting_id received.
@@ -763,7 +763,7 @@ 
     }
     $action = strtolower($params['action']);
     if (!array_key_exists($action, $requiredparams)) {
-        return 'Action '.$params['action'].' can not be performed.';
+        return 'Action ' . $params['action'] . ' can not be performed.';
     }
     return bigbluebuttonbn_broker_validate_parameters_message($params, $requiredparams[$action]);
 }

lib.php

Doc Comments +3 added lines, -3 removed lines

@@ -97,7 +97,7 @@ 
  * @uses FEATURE_GRADE_OUTCOMES
  * @uses FEATURE_SHOW_DESCRIPTION
  * @param string $feature
- * @return mixed True if yes (some features may use other values)
+ * @return null|boolean True if yes (some features may use other values)
  */
 function bigbluebuttonbn_supports($feature) {
     if (!$feature) {
@@ -239,7 +239,7 @@ 
  * @param object $mod
  * @param object $bigbluebuttonbn
  *
- * @return bool
+ * @return string
  */
 function bigbluebuttonbn_user_outline($course, $user, $mod, $bigbluebuttonbn) {
     global $DB;
@@ -542,7 +542,7 @@ 
  * @param stdClass $context       context object
  * @param string   $filearea      file area
  *
- * @return false|null false if file not valid
+ * @return boolean false if file not valid
  */
 function bigbluebuttonbn_pluginfile_valid($context, $filearea) {
     if ($context->contextlevel != CONTEXT_MODULE) {

Indentation +3 added lines, -3 removed lines

@@ -68,7 +68,7 @@ 
  * Remove this block when restored
  */
 
- /** @var BIGBLUEBUTTONBN_DEFAULT_SERVER_URL string of default bigbluebutton server url */
+    /** @var BIGBLUEBUTTONBN_DEFAULT_SERVER_URL string of default bigbluebutton server url */
 const BIGBLUEBUTTONBN_DEFAULT_SERVER_URL = 'http://test-install.blindsidenetworks.com/bigbluebutton/';
 /** @var BIGBLUEBUTTONBN_DEFAULT_SHARED_SECRET string of default bigbluebutton server shared secret */
 const BIGBLUEBUTTONBN_DEFAULT_SHARED_SECRET = '8cd8ef52e8e101574e400365b55e11a6';
@@ -345,12 +345,12 @@ 
     $str  = '<div class="bigbluebuttonbn overview">'."\n";
     $str .= '  <div class="name">'.get_string('modulename', 'bigbluebuttonbn').':&nbsp;'."\n";
     $str .= '    <a '.$classes.'href="'.$CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$bigbluebuttonbn->coursemodule.
-      '">'.$bigbluebuttonbn->name.'</a>'."\n";
+        '">'.$bigbluebuttonbn->name.'</a>'."\n";
     $str .= '  </div>'."\n";
     $str .= '  <div class="info">'.get_string($start, 'bigbluebuttonbn').': '.userdate($bigbluebuttonbn->openingtime).
         '</div>'."\n";
     $str .= '  <div class="info">'.get_string('ends_at', 'bigbluebuttonbn').': '.userdate($bigbluebuttonbn->closingtime)
-      .'</div>'."\n";
+        .'</div>'."\n";
     $str .= '</div>'."\n";
     return $str;
 }

Spacing +44 added lines, -44 removed lines

@@ -28,32 +28,32 @@ 
 
 global $CFG;
 
-require_once($CFG->dirroot.'/calendar/lib.php');
-require_once($CFG->dirroot.'/message/lib.php');
-require_once($CFG->dirroot.'/mod/lti/OAuth.php');
-require_once($CFG->libdir.'/accesslib.php');
-require_once($CFG->libdir.'/completionlib.php');
-require_once($CFG->libdir.'/datalib.php');
-require_once($CFG->libdir.'/coursecatlib.php');
-require_once($CFG->libdir.'/enrollib.php');
-require_once($CFG->libdir.'/filelib.php');
-require_once($CFG->libdir.'/formslib.php');
-
-if (file_exists(dirname(__FILE__).'/vendor/firebase/php-jwt/src/JWT.php')) {
-    require_once(dirname(__FILE__).'/vendor/firebase/php-jwt/src/JWT.php');
+require_once($CFG->dirroot . '/calendar/lib.php');
+require_once($CFG->dirroot . '/message/lib.php');
+require_once($CFG->dirroot . '/mod/lti/OAuth.php');
+require_once($CFG->libdir . '/accesslib.php');
+require_once($CFG->libdir . '/completionlib.php');
+require_once($CFG->libdir . '/datalib.php');
+require_once($CFG->libdir . '/coursecatlib.php');
+require_once($CFG->libdir . '/enrollib.php');
+require_once($CFG->libdir . '/filelib.php');
+require_once($CFG->libdir . '/formslib.php');
+
+if (file_exists(dirname(__FILE__) . '/vendor/firebase/php-jwt/src/JWT.php')) {
+    require_once(dirname(__FILE__) . '/vendor/firebase/php-jwt/src/JWT.php');
 }
 
 if (!isset($CFG->bigbluebuttonbn)) {
     $CFG->bigbluebuttonbn = array();
 }
 
-if (file_exists(dirname(__FILE__).'/config.php')) {
-    require_once(dirname(__FILE__).'/config.php');
+if (file_exists(dirname(__FILE__) . '/config.php')) {
+    require_once(dirname(__FILE__) . '/config.php');
     // Old BigBlueButtonBN cfg schema. For backward compatibility.
     global $BIGBLUEBUTTONBN_CFG;
 
     if (isset($BIGBLUEBUTTONBN_CFG)) {
-        foreach ((array) $BIGBLUEBUTTONBN_CFG as $key => $value) {
+        foreach ((array)$BIGBLUEBUTTONBN_CFG as $key => $value) {
             $cfgkey = str_replace("bigbluebuttonbn_", "", $key);
             $CFG->bigbluebuttonbn[$cfgkey] = $value;
         }
@@ -104,18 +104,18 @@ 
         return null;
     }
     $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];
     }
     return null;
@@ -215,7 +215,7 @@ 
     $log->timecreated = time();
     $log->log = BIGBLUEBUTTONBN_LOG_EVENT_DELETE;
     $sql  = "SELECT * FROM {bigbluebuttonbn_logs} ";
-    $sql .= "WHERE bigbluebuttonbnid = ? AND log = ? AND ". $DB->sql_compare_text('meta') . " = ?";
+    $sql .= "WHERE bigbluebuttonbnid = ? AND log = ? AND " . $DB->sql_compare_text('meta') . " = ?";
     $logs = $DB->get_records_sql($sql, array($bigbluebuttonbn->id, BIGBLUEBUTTONBN_LOG_EVENT_CREATE, "{\"record\":true}"));
     $log->meta = "{\"has_recordings\":false}";
     if (!empty($logs)) {
@@ -244,10 +244,10 @@ 
 function bigbluebuttonbn_user_outline($course, $user, $mod, $bigbluebuttonbn) {
     global $DB;
     $completed = $DB->count_records('bigbluebuttonbn_logs', array('courseid' => $course->id,
-        'bigbluebuttonbnid' => $bigbluebuttonbn->id, 'userid' => $user->id, 'log' => 'Join', ), '*');
+        'bigbluebuttonbnid' => $bigbluebuttonbn->id, 'userid' => $user->id, 'log' => 'Join',), '*');
     if ($completed > 0) {
-        return fullname($user).' '.get_string('view_message_has_joined', 'bigbluebuttonbn').' '.
-            get_string('view_message_session_for', 'bigbluebuttonbn').' '.(string) $completed.' '.
+        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 '';
@@ -267,7 +267,7 @@ 
 function bigbluebuttonbn_user_complete($course, $user, $mod, $bigbluebuttonbn) {
     global $DB;
     $completed = $DB->count_records('bigbluebuttonbn_logs', array('courseid' => $course->id,
-        'bigbluebuttonbnid' => $bigbluebuttonbn->id, 'userid' => $user->id, 'log' => 'Join', ),
+        'bigbluebuttonbnid' => $bigbluebuttonbn->id, 'userid' => $user->id, 'log' => 'Join',),
         '*', IGNORE_MULTIPLE);
     return $completed > 0;
 }
@@ -342,16 +342,16 @@ 
     if ($bigbluebuttonbn->visible) {
         $classes = 'class="dimmed" ';
     }
-    $str  = '<div class="bigbluebuttonbn overview">'."\n";
-    $str .= '  <div class="name">'.get_string('modulename', 'bigbluebuttonbn').':&nbsp;'."\n";
-    $str .= '    <a '.$classes.'href="'.$CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$bigbluebuttonbn->coursemodule.
-      '">'.$bigbluebuttonbn->name.'</a>'."\n";
-    $str .= '  </div>'."\n";
-    $str .= '  <div class="info">'.get_string($start, 'bigbluebuttonbn').': '.userdate($bigbluebuttonbn->openingtime).
-        '</div>'."\n";
-    $str .= '  <div class="info">'.get_string('ends_at', 'bigbluebuttonbn').': '.userdate($bigbluebuttonbn->closingtime)
-      .'</div>'."\n";
-    $str .= '</div>'."\n";
+    $str  = '<div class="bigbluebuttonbn overview">' . "\n";
+    $str .= '  <div class="name">' . get_string('modulename', 'bigbluebuttonbn') . ':&nbsp;' . "\n";
+    $str .= '    <a ' . $classes . 'href="' . $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $bigbluebuttonbn->coursemodule .
+      '">' . $bigbluebuttonbn->name . '</a>' . "\n";
+    $str .= '  </div>' . "\n";
+    $str .= '  <div class="info">' . get_string($start, 'bigbluebuttonbn') . ': ' . userdate($bigbluebuttonbn->openingtime) .
+        '</div>' . "\n";
+    $str .= '  <div class="info">' . get_string('ends_at', 'bigbluebuttonbn') . ': ' . userdate($bigbluebuttonbn->closingtime)
+      .'</div>' . "\n";
+    $str .= '</div>' . "\n";
     return $str;
 }
 
@@ -505,7 +505,7 @@ 
     if (count($files) == 1) {
         // Get the first (and only) file.
         $file = reset($files);
-        $filesrc = '/'.$file->get_filename();
+        $filesrc = '/' . $file->get_filename();
     }
     return $filesrc;
 }
@@ -573,7 +573,7 @@ 
     if (!$filename) {
         return false;
     }
-    $fullpath = "/$context->id/mod_bigbluebuttonbn/$filearea/0/".$filename;
+    $fullpath = "/$context->id/mod_bigbluebuttonbn/$filearea/0/" . $filename;
     $fs = get_file_storage();
     $file = $fs->get_file_by_hash(sha1($fullpath));
     if (!$file || $file->is_directory()) {

import_view.php

Indentation +2 added lines, -2 removed lines

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

Spacing +3 added lines, -3 removed lines

@@ -23,8 +23,8 @@ 
  * @author    Jesus Federico  (jesus [at] blindsidenetworks [dt] com)
  */
 
-require_once(dirname(dirname(dirname(__FILE__))).'/config.php');
-require_once(dirname(__FILE__).'/locallib.php');
+require_once(dirname(dirname(dirname(__FILE__))) . '/config.php');
+require_once(dirname(__FILE__) . '/locallib.php');
 
 $bn = required_param('bn', PARAM_INT);
 $tc = optional_param('tc', 0, PARAM_INT);
@@ -78,7 +78,7 @@ 
     $body .= html_writer::tag('input', '',
         array('type' => 'button', 'class' => 'btn btn-secondary',
               'value' => get_string('view_recording_button_return', 'bigbluebuttonbn'),
-              'onclick' => 'window.location=\''.$CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$cm->id.'\''));
+              'onclick' => 'window.location=\'' . $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $cm->id . '\''));
     // JavaScript for locales.
     $PAGE->requires->strings_for_js(array_keys(bigbluebuttonbn_get_strings_for_js()), 'bigbluebuttonbn');
     // Require JavaScript modules.

mod_form.php

Indentation +8 added lines, -8 removed lines

@@ -104,7 +104,7 @@ 
                 'alt' => $contents,
                 'title' => $contents,
                 'src' => $jsvars['pix_icon_delete']
-              );
+                );
             $jsvars['pix_icon_delete'] = html_writer::tag('img', '', $options);
         }
         $PAGE->requires->yui_module('moodle-mod_bigbluebuttonbn-modform',
@@ -224,8 +224,8 @@ 
             $this->bigbluebuttonbn_mform_add_element($mform, $field['type'], $field['name'], $field['data_type'],
                 $field['description_key'], 0, ['maxlength' => 4, 'size' => 6],
                 ['message' => get_string('mod_form_field_voicebridge_format_error', 'bigbluebuttonbn'),
-                 'type' => 'numeric', 'rule' => '####', 'validator' => 'server']
-              );
+                    'type' => 'numeric', 'rule' => '####', 'validator' => 'server']
+                );
         } else {
             $this->bigbluebuttonbn_mform_add_element($mform, $field['type'], $field['name'], $field['data_type'],
                 $field['description_key'], 0, ['maxlength' => 4, 'size' => 6]);
@@ -263,7 +263,7 @@ 
      */
     private function bigbluebuttonbn_mform_add_block_room_recordings(&$mform, $cfg) {
         $field = ['type' => 'hidden', 'name' => 'recordings_html', 'data_type' => PARAM_INT,
-                  'description_key' => null];
+                    'description_key' => null];
         if ($cfg['recordings_html_editable']) {
             $field['type'] = 'checkbox';
             $field['description_key'] = 'mod_form_field_recordings_html';
@@ -271,7 +271,7 @@ 
         $this->bigbluebuttonbn_mform_add_element($mform, $field['type'], $field['name'], $field['data_type'],
             $field['description_key'], $cfg['recordings_html_default']);
         $field = ['type' => 'hidden', 'name' => 'recordings_deleted', 'data_type' => PARAM_INT,
-                  'description_key' => null];
+                    'description_key' => null];
         if ($cfg['recordings_deleted_editable']) {
             $field['type'] = 'checkbox';
             $field['description_key'] = 'mod_form_field_recordings_deleted';
@@ -279,7 +279,7 @@ 
         $this->bigbluebuttonbn_mform_add_element($mform, $field['type'], $field['name'], $field['data_type'],
             $field['description_key'], $cfg['recordings_deleted_default']);
         $field = ['type' => 'hidden', 'name' => 'recordings_imported', 'data_type' => PARAM_INT,
-                  'description_key' => null];
+                    'description_key' => null];
         if ($cfg['recordings_imported_editable']) {
             $field['type'] = 'checkbox';
             $field['description_key'] = 'mod_form_field_recordings_imported';
@@ -349,7 +349,7 @@ 
         $htmlselectiontype = html_writer::select($participantselection['type_options'],
             'bigbluebuttonbn_participant_selection_type', $participantselection['type_selected'], array(),
             array('id' => 'bigbluebuttonbn_participant_selection_type',
-                  'onchange' => 'M.mod_bigbluebuttonbn.modform.participantSelectionSet(); return 0;'));
+                    'onchange' => 'M.mod_bigbluebuttonbn.modform.participantSelectionSet(); return 0;'));
         $htmlselectionoptions = html_writer::select($participantselection['options'], 'bigbluebuttonbn_participant_selection',
             $participantselection['selected'], array(),
             array('id' => 'bigbluebuttonbn_participant_selection', 'disabled' => 'disabled'));
@@ -357,7 +357,7 @@ 
             'type' => 'button', 'class' => 'btn btn-secondary',
             'value' => get_string('mod_form_field_participant_list_action_add', 'bigbluebuttonbn'),
             'onclick' => 'M.mod_bigbluebuttonbn.modform.participantAdd(); return 0;'
-          ));
+            ));
         $htmladdparticipant = html_writer::tag('div',
             $htmlselectiontype . '  ' . $htmlselectionoptions . '  ' . $htmlselectioninput, null);
         $mform->addElement('html', "\n\n");

Spacing +5 added lines, -5 removed lines

@@ -26,8 +26,8 @@ 
 
 defined('MOODLE_INTERNAL') || die();
 
-require_once(dirname(__FILE__).'/locallib.php');
-require_once($CFG->dirroot.'/course/moodleform_mod.php');
+require_once(dirname(__FILE__) . '/locallib.php');
+require_once($CFG->dirroot . '/course/moodleform_mod.php');
 
 /**
  * Moodle class for mod_form.
@@ -48,7 +48,7 @@ 
         $serverversion = bigbluebuttonbn_get_server_version();
         if (is_null($serverversion)) {
             print_error('general_error_unable_connect', 'bigbluebuttonbn',
-                $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+                $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
             return;
         }
         // Context.
@@ -127,7 +127,7 @@ 
                 );
                 $defaultvalues['presentation'] = $draftitemid;
             } catch (Exception $e) {
-                debugging('Presentation could not be loaded: '.$e->getMessage(), DEBUG_DEVELOPER);
+                debugging('Presentation could not be loaded: ' . $e->getMessage(), DEBUG_DEVELOPER);
                 return;
             }
         }
@@ -423,7 +423,7 @@ 
             return;
         }
         $mform->addElement($type, $name, get_string($descriptionkey, 'bigbluebuttonbn'), $options);
-        if (get_string_manager()->string_exists($descriptionkey.'_help', 'bigbluebuttonbn')) {
+        if (get_string_manager()->string_exists($descriptionkey . '_help', 'bigbluebuttonbn')) {
             $mform->addHelpButton($name, $descriptionkey, 'bigbluebuttonbn');
         }
         if (!empty($rule)) {

classes/locallib/notifier.php

Spacing +26 added lines, -26 removed lines

@@ -47,7 +47,7 @@ 
     public static function notification_process($context, $bigbluebuttonbn, $action) {
         global $USER;
         // Prepare message.
-        $msg = (object) array();
+        $msg = (object)array();
         // Build the message_body.
         $msg->action = $action;
         $msg->activity_type = '';
@@ -72,26 +72,26 @@ 
      * @return string
      */
     public static function notification_msg_html($msg) {
-        $messagetext = '<p>'.$msg->activity_type.' &quot;'.$msg->activity_title.'&quot; '.
-            get_string('email_body_notification_meeting_has_been', 'bigbluebuttonbn').' '.$msg->action.'.</p>'."\n";
-        $messagetext .= '<p><b>'.$msg->activity_title.'</b> '.
-            get_string('email_body_notification_meeting_details', 'bigbluebuttonbn').':'."\n";
-        $messagetext .= '<table border="0" style="margin: 5px 0 0 20px"><tbody>'."\n";
-        $messagetext .= '<tr><td style="font-weight:bold;color:#555;">'.
-            get_string('email_body_notification_meeting_title', 'bigbluebuttonbn').': </td><td>'."\n";
-        $messagetext .= $msg->activity_title.'</td></tr>'."\n";
-        $messagetext .= '<tr><td style="font-weight:bold;color:#555;">'.
-            get_string('email_body_notification_meeting_description', 'bigbluebuttonbn').': </td><td>'."\n";
-        $messagetext .= $msg->activity_description.'</td></tr>'."\n";
-        $messagetext .= '<tr><td style="font-weight:bold;color:#555;">'.
-            get_string('email_body_notification_meeting_start_date', 'bigbluebuttonbn').': </td><td>'."\n";
-        $messagetext .= $msg->activity_openingtime.'</td></tr>'."\n";
-        $messagetext .= '<tr><td style="font-weight:bold;color:#555;">'.
-            get_string('email_body_notification_meeting_end_date', 'bigbluebuttonbn').': </td><td>'."\n";
-        $messagetext .= $msg->activity_closingtime.'</td></tr>'."\n";
-        $messagetext .= '<tr><td style="font-weight:bold;color:#555;">'.$msg->action.' '.
-            get_string('email_body_notification_meeting_by', 'bigbluebuttonbn').': </td><td>'."\n";
-        $messagetext .= $msg->activity_owner.'</td></tr></tbody></table></p>'."\n";
+        $messagetext = '<p>' . $msg->activity_type . ' &quot;' . $msg->activity_title . '&quot; ' .
+            get_string('email_body_notification_meeting_has_been', 'bigbluebuttonbn') . ' ' . $msg->action . '.</p>' . "\n";
+        $messagetext .= '<p><b>' . $msg->activity_title . '</b> ' .
+            get_string('email_body_notification_meeting_details', 'bigbluebuttonbn') . ':' . "\n";
+        $messagetext .= '<table border="0" style="margin: 5px 0 0 20px"><tbody>' . "\n";
+        $messagetext .= '<tr><td style="font-weight:bold;color:#555;">' .
+            get_string('email_body_notification_meeting_title', 'bigbluebuttonbn') . ': </td><td>' . "\n";
+        $messagetext .= $msg->activity_title . '</td></tr>' . "\n";
+        $messagetext .= '<tr><td style="font-weight:bold;color:#555;">' .
+            get_string('email_body_notification_meeting_description', 'bigbluebuttonbn') . ': </td><td>' . "\n";
+        $messagetext .= $msg->activity_description . '</td></tr>' . "\n";
+        $messagetext .= '<tr><td style="font-weight:bold;color:#555;">' .
+            get_string('email_body_notification_meeting_start_date', 'bigbluebuttonbn') . ': </td><td>' . "\n";
+        $messagetext .= $msg->activity_openingtime . '</td></tr>' . "\n";
+        $messagetext .= '<tr><td style="font-weight:bold;color:#555;">' .
+            get_string('email_body_notification_meeting_end_date', 'bigbluebuttonbn') . ': </td><td>' . "\n";
+        $messagetext .= $msg->activity_closingtime . '</td></tr>' . "\n";
+        $messagetext .= '<tr><td style="font-weight:bold;color:#555;">' . $msg->action . ' ' .
+            get_string('email_body_notification_meeting_by', 'bigbluebuttonbn') . ': </td><td>' . "\n";
+        $messagetext .= $msg->activity_owner . '</td></tr></tbody></table></p>' . "\n";
         return $messagetext;
     }
 
@@ -108,14 +108,14 @@ 
         global $DB;
         $course = $DB->get_record('course', array('id' => $bigbluebuttonbn->course), '*', MUST_EXIST);
         // Complete message.
-        $msg = (object) array();
+        $msg = (object)array();
         $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>';
-        $users = (array) get_enrolled_users($context, '', 0, 'u.*', null, 0, 0, true);
+        $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 = (array)get_enrolled_users($context, '', 0, 'u.*', null, 0, 0, true);
         foreach ($users as $user) {
             if ($user->id != $sender->id) {
                 message_post_message($sender, $user, $message, FORMAT_HTML);

index.php

Spacing +18 added lines, -18 removed lines

@@ -24,8 +24,8 @@ 
  * @author    Fred Dixon  (ffdixon [at] blindsidenetworks [dt] com)
  */
 
-require_once(dirname(dirname(dirname(__FILE__))).'/config.php');
-require_once(dirname(__FILE__).'/locallib.php');
+require_once(dirname(dirname(dirname(__FILE__))) . '/config.php');
+require_once(dirname(__FILE__) . '/locallib.php');
 
 $id = required_param('id', PARAM_INT);
 $a = optional_param('a', 0, PARAM_INT);
@@ -88,12 +88,12 @@ 
     if ($moderator || $administrator) {
         bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_ENDED, $bigbluebuttonbn, $cm);
         echo get_string('index_ending', 'bigbluebuttonbn');
-        $meetingid = $bigbluebuttonbn->meetingid.'-'.$course->id.'-'.$bigbluebuttonbn->id;
+        $meetingid = $bigbluebuttonbn->meetingid . '-' . $course->id . '-' . $bigbluebuttonbn->id;
         if ($g != '0') {
-            $meetingid .= '['.$g.']';
+            $meetingid .= '[' . $g . ']';
         }
         bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getEndMeetingURL($meetingid));
-        redirect('index.php?id='.$id);
+        redirect('index.php?id=' . $id);
     }
 }
 
@@ -107,7 +107,7 @@ 
         // Add a the data for the bigbluebuttonbn instance.
         $groupobj = null;
         if (groups_get_activity_groupmode($cm) > 0) {
-            $groupobj = (object) array('id' => 0, 'name' => get_string('allparticipants'));
+            $groupobj = (object)array('id' => 0, 'name' => get_string('allparticipants'));
         }
         $table->data[] = bigbluebuttonbn_index_display_room($canmoderate, $course, $bigbluebuttonbn, $groupobj);
         // Add a the data for the groups belonging to the bigbluebuttonbn instance, if any.
@@ -133,12 +133,12 @@ 
  * @return array
  */
 function bigbluebuttonbn_index_display_room($moderator, $course, $bigbluebuttonbn, $groupobj = null) {
-    $meetingid = $bigbluebuttonbn->meetingid.'-'.$course->id.'-'.$bigbluebuttonbn->id;
+    $meetingid = $bigbluebuttonbn->meetingid . '-' . $course->id . '-' . $bigbluebuttonbn->id;
     $paramgroup = '';
     $groupname = '';
     if ($groupobj) {
-        $meetingid .= '['.$groupobj->id.']';
-        $paramgroup = '&group='.$groupobj->id;
+        $meetingid .= '[' . $groupobj->id . ']';
+        $paramgroup = '&group=' . $groupobj->id;
         $groupname = $groupobj->name;
     }
     $meetinginfo = bigbluebuttonbn_get_meeting_info_array($meetingid);
@@ -153,7 +153,7 @@ 
         return;
     }
     // Output Users in the meeting.
-    $joinurl = '<a href="view.php?id='.$bigbluebuttonbn->coursemodule.$paramgroup.'">'.format_string($bigbluebuttonbn->name).'</a>';
+    $joinurl = '<a href="view.php?id=' . $bigbluebuttonbn->coursemodule . $paramgroup . '">' . format_string($bigbluebuttonbn->name) . '</a>';
     $group = $groupname;
     $users = '';
     $viewerlist = '';
@@ -198,7 +198,7 @@ 
         $attendeecount = 0;
         foreach ($meetinginfo['attendees']->attendee as $attendee) {
             if ($attendee->role == $role) {
-                $attendeelist .= ($attendeecount++ > 0 ? ', ' : '').$attendee->fullName;
+                $attendeelist .= ($attendeecount++ > 0 ? ', ' : '') . $attendee->fullName;
             }
         }
     }
@@ -232,15 +232,15 @@ 
 function bigbluebuttonbn_index_display_room_actions($moderator, $course, $bigbluebuttonbn, $groupobj = null) {
     $actions = '';
     if ($moderator) {
-        $actions .= '<form name="form1" method="post" action="">'.'/n';
-        $actions .= '  <INPUT type="hidden" name="id" value="'.$course->id.'">'.'/n';
-        $actions .= '  <INPUT type="hidden" name="a" value="'.$bigbluebuttonbn->id.'">'.'/n';
+        $actions .= '<form name="form1" method="post" action="">' . '/n';
+        $actions .= '  <INPUT type="hidden" name="id" value="' . $course->id . '">' . '/n';
+        $actions .= '  <INPUT type="hidden" name="a" value="' . $bigbluebuttonbn->id . '">' . '/n';
         if ($groupobj != null) {
-            $actions .= '  <INPUT type="hidden" name="g" value="'.$groupobj->id.'">'.'/n';
+            $actions .= '  <INPUT type="hidden" name="g" value="' . $groupobj->id . '">' . '/n';
         }
-        $actions .= '  <INPUT type="submit" name="submit" value="end" onclick="return confirm(\''.
-            get_string('index_confirm_end', 'bigbluebuttonbn').'\')">'.'/n';
-        $actions .= '</form>'.'/n';
+        $actions .= '  <INPUT type="submit" name="submit" value="end" onclick="return confirm(\'' .
+            get_string('index_confirm_end', 'bigbluebuttonbn') . '\')">' . '/n';
+        $actions .= '</form>' . '/n';
     }
     return $actions;
 }

classes/settings/renderer.php

Spacing +2 added lines, -2 removed lines

@@ -28,7 +28,7 @@ 
 defined('MOODLE_INTERNAL') || die();
 
 require_once($CFG->dirroot . '/mod/bigbluebuttonbn/locallib.php');
-require_once($CFG->libdir.'/adminlib.php');
+require_once($CFG->libdir . '/adminlib.php');
 
 /**
  * Helper class for rendering HTML for settings.php.
@@ -66,7 +66,7 @@ 
             $itemname = get_string('config_' . $name, 'bigbluebuttonbn');
         }
         if ($itemdescription === null) {
-            $itemdescription = get_string('config_' .$name . '_description', 'bigbluebuttonbn');
+            $itemdescription = get_string('config_' . $name . '_description', 'bigbluebuttonbn');
         }
         $item = new \admin_setting_heading('bigbluebuttonbn_config_' . $name, $itemname, $itemdescription);
         $this->settings->add($item);