blindsidenetworks / moodle-mod_bigbluebuttonbn

classes/output/mobile.php

Unused Use Statements -1 removed lines

@@ -27,7 +27,6 @@
 
 defined('MOODLE_INTERNAL') || die();
 
-use context_module;
 use mod_bigbluebuttonbn\locallib\bigbluebutton;
 
 require_once($CFG->dirroot . '/mod/bigbluebuttonbn/locallib.php');

Spacing +4 added lines, -4 removed lines

@@ -56,7 +56,7 @@ 
 
         global $OUTPUT, $SESSION;
 
-        $args = (object) $args;
+        $args = (object)$args;
         $viewinstance = bigbluebuttonbn_view_validator($args->cmid, null);
         if (!$viewinstance) {
             $error = get_string('view_error_url_missing_parameters', 'bigbluebuttonbn');
@@ -164,7 +164,7 @@ 
             }
             if ($response['returncode'] == 'FAILED') {
                 // The meeting could not be created.
-                $errorkey = bigbluebuttonbn_get_error_key($response['messageKey'],  'view_error_create');
+                $errorkey = bigbluebuttonbn_get_error_key($response['messageKey'], 'view_error_create');
                 $e = get_string($errorkey, 'bigbluebuttonbn');
                 return(self::mobile_print_error($e));
             }
@@ -177,7 +177,7 @@ 
             bigbluebuttonbn_event_log(\mod_bigbluebuttonbn\event\events::$events['meeting_create'], $bigbluebuttonbn);
             // Insert a record that meeting was created.
             $overrides = array('meetingid' => $bbbsession['meetingid']);
-            $meta = '{"record":'.($bbbsession['record'] ? 'true' : 'false').'}';
+            $meta = '{"record":' . ($bbbsession['record'] ? 'true' : 'false') . '}';
             bigbluebuttonbn_log($bbbsession['bigbluebuttonbn'], BIGBLUEBUTTONBN_LOG_EVENT_CREATE, $overrides, $meta);
         }
 
@@ -203,7 +203,7 @@ 
 
         $data = array(
             'bigbluebuttonbn' => $bigbluebuttonbn,
-            'bbbsession' => (object) $bbbsession,
+            'bbbsession' => (object)$bbbsession,
             'msjgroup' => $msjgroup,
             'urltojoin' => $urltojoin,
             'cmid' => $cm->id,

view.php

Spacing +6 added lines, -6 removed lines

@@ -27,9 +27,9 @@ 
 
 use mod_bigbluebuttonbn\plugin;
 
-require(__DIR__.'/../../config.php');
-require_once(__DIR__.'/locallib.php');
-require_once(__DIR__.'/viewlib.php');
+require(__DIR__ . '/../../config.php');
+require_once(__DIR__ . '/locallib.php');
+require_once(__DIR__ . '/viewlib.php');
 
 $id = required_param('id', PARAM_INT);
 $bn = optional_param('bn', 0, PARAM_INT);
@@ -58,7 +58,7 @@ 
 mod_bigbluebuttonbn\locallib\bigbluebutton::view_bbbsession_set($PAGE->context, $bbbsession);
 
 // Validates if the BigBlueButton server is working.
-$serverversion = bigbluebuttonbn_get_server_version();  // In locallib.
+$serverversion = bigbluebuttonbn_get_server_version(); // In locallib.
 if ($serverversion === null) {
     $errmsg = 'view_error_unable_join_student';
     $errurl = '/course/view.php';
@@ -72,7 +72,7 @@ 
     }
     print_error($errmsg, plugin::COMPONENT, new moodle_url($errurl, $errurlparams));
 }
-$bbbsession['serverversion'] = (string) $serverversion;
+$bbbsession['serverversion'] = (string)$serverversion;
 
 // Mark viewed by user (if required).
 $completion = new completion_info($course);
@@ -116,7 +116,7 @@ 
 echo $OUTPUT->footer();
 
 // Shows version as a comment.
-echo '<!-- '.$bbbsession['originTag'].' -->'."\n";
+echo '<!-- ' . $bbbsession['originTag'] . ' -->' . "\n";
 
 // Initialize session variable used across views.
 $SESSION->bigbluebuttonbn_bbbsession = $bbbsession;
\ No newline at end of file

bbb_view.php

Spacing +20 added lines, -20 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');
 
 global $SESSION;
 
@@ -63,16 +63,16 @@ 
     if (is_null($serverversion)) {
         if ($bbbsession['administrator']) {
             print_error('view_error_unable_join', 'bigbluebuttonbn',
-                $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+                $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
             exit;
         }
         if ($bbbsession['moderator']) {
             print_error('view_error_unable_join_teacher', 'bigbluebuttonbn',
-                $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course);
+                $CFG->wwwroot . '/course/view.php?id=' . $bigbluebuttonbn->course);
             exit;
         }
         print_error('view_error_unable_join_student', 'bigbluebuttonbn',
-            $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course);
+            $CFG->wwwroot . '/course/view.php?id=' . $bigbluebuttonbn->course);
         exit;
     }
 
@@ -97,8 +97,8 @@ 
         }
 
         // Assign group default values.
-        $bbbsession['meetingid'] .= '['.$bbbsession['group'].']';
-        $bbbsession['meetingname'] .= ' ('.$groupname.')';
+        $bbbsession['meetingid'] .= '[' . $bbbsession['group'] . ']';
+        $bbbsession['meetingname'] .= ' (' . $groupname . ')';
     }
 
     // Initialize session variable used across views.
@@ -163,7 +163,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.
@@ -177,16 +177,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') {
@@ -207,7 +207,7 @@ 
         bigbluebuttonbn_event_log(\mod_bigbluebuttonbn\event\events::$events['meeting_create'], $bigbluebuttonbn);
         // Internal logger: Insert a record with the meeting created.
         $overrides = array('meetingid' => $bbbsession['meetingid']);
-        $meta = '{"record":'.($bbbsession['record'] ? 'true' : 'false').'}';
+        $meta = '{"record":' . ($bbbsession['record'] ? 'true' : 'false') . '}';
         bigbluebuttonbn_log($bbbsession['bigbluebuttonbn'], BIGBLUEBUTTONBN_LOG_EVENT_CREATE, $overrides, $meta);
         // Since the meeting is already running, we just join the session.
         bigbluebuttonbn_bbb_view_join_meeting($bbbsession, $bigbluebuttonbn, $origin);
@@ -221,7 +221,7 @@ 
         $overrides = array('meetingid' => $bbbsession['meetingid']);
         bigbluebuttonbn_log($bbbsession['bigbluebuttonbn'], BIGBLUEBUTTONBN_LOG_EVENT_PLAYED, $overrides);
         // Execute the redirect.
-        header('Location: '.urldecode($href));
+        header('Location: ' . urldecode($href));
         break;
     default:
         bigbluebuttonbn_bbb_view_close_window();
@@ -315,7 +315,7 @@ 
         $data['welcome'] .= '<br><br>';
         $data['welcome'] .= str_replace(
             '%duration%',
-            (string) $durationtime,
+            (string)$durationtime,
             get_string('bbbdurationwarning', 'bigbluebuttonbn')
           );
     }
@@ -381,7 +381,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.
@@ -395,13 +395,13 @@ 
     bigbluebuttonbn_event_log(\mod_bigbluebuttonbn\event\events::$events['meeting_join'], $bigbluebuttonbn);
     // Internal logger: Instert a record with the meeting created.
     $overrides = array('meetingid' => $bbbsession['meetingid']);
-    $meta = '{"origin":'.$origin.'}';
+    $meta = '{"origin":' . $origin . '}';
     bigbluebuttonbn_log($bbbsession['bigbluebuttonbn'], BIGBLUEBUTTONBN_LOG_EVENT_JOIN, $overrides, $meta);
     // Before executing the redirect, increment the number of participants.
     bigbluebuttonbn_participant_joined($bbbsession['meetingid'],
         ($bbbsession['administrator'] || $bbbsession['moderator']));
     // Execute the redirect.
-    header('Location: '.$joinurl);
+    header('Location: ' . $joinurl);
 }
 
 /**
@@ -413,13 +413,13 @@ 
  */
 function bigbluebuttonbn_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();
 }

classes/locallib/mobileview.php

Spacing +1 added lines, -1 removed lines

@@ -113,7 +113,7 @@
             $data['welcome'] .= '<br><br>';
             $data['welcome'] .= str_replace(
                 '%duration%',
-                (string) $durationtime,
+                (string)$durationtime,
                 get_string('bbbdurationwarning', 'bigbluebuttonbn')
             );
         }

classes/locallib/bigbluebutton.php

Spacing +7 added lines, -7 removed lines

@@ -149,7 +149,7 @@ 
         self::view_bbbsession_set($context, $bbbsession);
 
         $serverversion = bigbluebuttonbn_get_server_version();
-        $bbbsession['serverversion'] = (string) $serverversion;
+        $bbbsession['serverversion'] = (string)$serverversion;
 
         // Operation URLs.
         $bbbsession['bigbluebuttonbnURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $cm->id;
@@ -187,12 +187,12 @@ 
         $session['importrecordings'] = ($session['managerecordings']);
         $session['modPW'] = $session['bigbluebuttonbn']->moderatorpass;
         $session['viewerPW'] = $session['bigbluebuttonbn']->viewerpass;
-        $session['meetingid'] = $session['bigbluebuttonbn']->meetingid.'-'.$session['course']->id.'-'.
+        $session['meetingid'] = $session['bigbluebuttonbn']->meetingid . '-' . $session['course']->id . '-' .
             $session['bigbluebuttonbn']->id;
         $session['meetingname'] = $session['bigbluebuttonbn']->name;
         $session['meetingdescription'] = $session['bigbluebuttonbn']->intro;
-        $session['userlimit'] = intval((int) config::get('userlimit_default'));
-        if ((boolean) config::get('userlimit_editable')) {
+        $session['userlimit'] = intval((int)config::get('userlimit_default'));
+        if ((boolean)config::get('userlimit_editable')) {
             $session['userlimit'] = intval($session['bigbluebuttonbn']->userlimit);
         }
         $session['voicebridge'] = $session['bigbluebuttonbn']->voicebridge;
@@ -219,10 +219,10 @@ 
         if ($session['bigbluebuttonbn']->record) {
             // Check if is enable record all from start.
             if ($session['recordallfromstart']) {
-                $session['welcome'] .= '<br><br>'.get_string('bbbrecordallfromstartwarning',
+                $session['welcome'] .= '<br><br>' . get_string('bbbrecordallfromstartwarning',
                         'bigbluebuttonbn');
             } else {
-                $session['welcome'] .= '<br><br>'.get_string('bbbrecordwarning', 'bigbluebuttonbn');
+                $session['welcome'] .= '<br><br>' . get_string('bbbrecordwarning', 'bigbluebuttonbn');
             }
         }
         $session['openingtime'] = $session['bigbluebuttonbn']->openingtime;
@@ -235,7 +235,7 @@ 
         $session['originServerName'] = $parsedurl['host'];
         $session['originServerUrl'] = $CFG->wwwroot;
         $session['originServerCommonName'] = '';
-        $session['originTag'] = 'moodle-mod_bigbluebuttonbn ('.get_config('mod_bigbluebuttonbn', 'version').')';
+        $session['originTag'] = 'moodle-mod_bigbluebuttonbn (' . get_config('mod_bigbluebuttonbn', 'version') . ')';
         $session['bnserver'] = bigbluebuttonbn_is_bn_server();
         $session['clienttype'] = config::get('clienttype_default');