jfederico / 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

@@ -30,8 +30,8 @@ 
 use context_module;
 use mod_bigbluebuttonbn\locallib\bigbluebutton;
 
-require_once($CFG->dirroot . '/mod/bigbluebuttonbn/locallib.php');
-require_once($CFG->dirroot . '/lib/grouplib.php');
+require_once($CFG->dirroot.'/mod/bigbluebuttonbn/locallib.php');
+require_once($CFG->dirroot.'/lib/grouplib.php');
 
 /**
  * Mobile output class for 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));
             }
@@ -281,7 +281,7 @@ 
                     'html' => $OUTPUT->render_from_template('mod_bigbluebuttonbn/mobile_view_notification', $data),
                 ),
             ),
-            'javascript' => file_get_contents($CFG->dirroot . '/mod/bigbluebuttonbn/mobile.notification.js'),
+            'javascript' => file_get_contents($CFG->dirroot.'/mod/bigbluebuttonbn/mobile.notification.js'),
             'otherdata' => '',
             'files' => ''
         );

classes/locallib/bigbluebutton.php

Spacing +17 added lines, -17 removed lines

@@ -29,7 +29,7 @@ 
 
 defined('MOODLE_INTERNAL') || die();
 global $CFG;
-require_once($CFG->dirroot . '/mod/bigbluebuttonbn/locallib.php');
+require_once($CFG->dirroot.'/mod/bigbluebuttonbn/locallib.php');
 
 /**
  * Wrapper for executing http requests on a BigBlueButton server.
@@ -48,18 +48,18 @@ 
      * @return string
      */
     public static function action_url($action = '', $data = array(), $metadata = array()) {
-        $baseurl = self::sanitized_url() . $action . '?';
+        $baseurl = self::sanitized_url().$action.'?';
         $metadata = array_combine(
             array_map(
                 function($k) {
-                    return 'meta_' . $k;
+                    return 'meta_'.$k;
                 }
                 , array_keys($metadata)
             ),
             $metadata
         );
         $params = http_build_query($data + $metadata, '', '&');
-        return $baseurl . $params . '&checksum=' . sha1($action . $params . self::sanitized_secret());
+        return $baseurl.$params.'&checksum='.sha1($action.$params.self::sanitized_secret());
     }
 
     /**
@@ -75,7 +75,7 @@ 
         if (substr($serverurl, -4) == '/api') {
             $serverurl = rtrim($serverurl, '/api');
         }
-        return $serverurl . '/api/';
+        return $serverurl.'/api/';
     }
 
     /**
@@ -96,9 +96,9 @@ 
         $pserverurl = parse_url(trim(config::get('server_url')));
         $pserverurlport = "";
         if (isset($pserverurl['port'])) {
-            $pserverurlport = ":" . $pserverurl['port'];
+            $pserverurlport = ":".$pserverurl['port'];
         }
-        return $pserverurl['scheme'] . "://" . $pserverurl['host'] . $pserverurlport . "/";
+        return $pserverurl['scheme']."://".$pserverurl['host'].$pserverurlport."/";
     }
 
     /**
@@ -152,15 +152,15 @@ 
         $bbbsession['serverversion'] = (string) $serverversion;
 
         // Operation URLs.
-        $bbbsession['bigbluebuttonbnURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $cm->id;
-        $bbbsession['logoutURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_view.php?action=logout&id=' . $cm->id .
-            '&bn=' . $bbbsession['bigbluebuttonbn']->id;
-        $bbbsession['recordingReadyURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_broker.php?action=recording_' .
-            'ready&bigbluebuttonbn=' . $bbbsession['bigbluebuttonbn']->id;
-        $bbbsession['meetingEventsURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_broker.php?action=meeting' .
-            '_events&bigbluebuttonbn=' . $bbbsession['bigbluebuttonbn']->id;
-        $bbbsession['joinURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_view.php?action=join&id=' . $cm->id .
-            '&bn=' . $bbbsession['bigbluebuttonbn']->id;
+        $bbbsession['bigbluebuttonbnURL'] = $CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$cm->id;
+        $bbbsession['logoutURL'] = $CFG->wwwroot.'/mod/bigbluebuttonbn/bbb_view.php?action=logout&id='.$cm->id.
+            '&bn='.$bbbsession['bigbluebuttonbn']->id;
+        $bbbsession['recordingReadyURL'] = $CFG->wwwroot.'/mod/bigbluebuttonbn/bbb_broker.php?action=recording_'.
+            'ready&bigbluebuttonbn='.$bbbsession['bigbluebuttonbn']->id;
+        $bbbsession['meetingEventsURL'] = $CFG->wwwroot.'/mod/bigbluebuttonbn/bbb_broker.php?action=meeting'.
+            '_events&bigbluebuttonbn='.$bbbsession['bigbluebuttonbn']->id;
+        $bbbsession['joinURL'] = $CFG->wwwroot.'/mod/bigbluebuttonbn/bbb_view.php?action=join&id='.$cm->id.
+            '&bn='.$bbbsession['bigbluebuttonbn']->id;
 
         return $bbbsession;
     }
@@ -267,7 +267,7 @@ 
         if ($viewinstance) {
             $bbbsession = self::build_bbb_session_fromviewinstance($viewinstance);
             if ($bbbsession) {
-                require_once($CFG->dirroot . "/mod/bigbluebuttonbn/brokerlib.php");
+                require_once($CFG->dirroot."/mod/bigbluebuttonbn/brokerlib.php");
                 $info = bigbluebuttonbn_get_meeting_info($bbbsession['meetingid'], false);
                 $running = false;
                 if ($info['returncode'] == 'SUCCESS') {

classes/external.php

Spacing +3 added lines, -3 removed lines

@@ -61,7 +61,7 @@ 
      */
     public static function view_bigbluebuttonbn($bigbluebuttonbnid) {
         global $DB, $CFG;
-        require_once($CFG->dirroot . "/mod/bigbluebuttonbn/lib.php");
+        require_once($CFG->dirroot."/mod/bigbluebuttonbn/lib.php");
 
         $params = self::validate_parameters(self::view_bigbluebuttonbn_parameters(),
                                             array(
@@ -109,7 +109,7 @@ 
      * @since Moodle 3.3
      */
     public static function get_bigbluebuttonbns_by_courses_parameters() {
-        return new external_function_parameters (
+        return new external_function_parameters(
             array(
                 'courseids' => new external_multiple_structure(
                     new external_value(PARAM_INT, 'Course id'), 'Array of course ids', VALUE_DEFAULT, array()
@@ -225,7 +225,7 @@ 
      */
     public static function can_join($cmid) {
         global $SESSION, $CFG;
-        require_once($CFG->dirroot . "/mod/bigbluebuttonbn/locallib.php");
+        require_once($CFG->dirroot."/mod/bigbluebuttonbn/locallib.php");
 
         $params = self::validate_parameters(self::can_join_parameters(),
             array(

view.php

Spacing +1 added lines, -1 removed lines

@@ -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';

bbb_view.php

Spacing +3 added lines, -3 removed lines

@@ -138,7 +138,7 @@ 
         $lastaccess = json_decode($lastaccess->meta);
         // If the user acceded from Timeline it should be redirected to the Dashboard.
         if (isset($lastaccess->origin) && $lastaccess->origin == BIGBLUEBUTTON_ORIGIN_TIMELINE) {
-            redirect($CFG->wwwroot . '/my/');
+            redirect($CFG->wwwroot.'/my/');
         }
         // Close the tab or window where BBB was opened.
         bigbluebuttonbn_bbb_view_close_window();
@@ -340,7 +340,7 @@ 
  * @return string
  */
 function bigbluebuttonbn_bbb_view_create_meeting_data_record($record) {
-    if ((boolean)\mod_bigbluebuttonbn\locallib\config::recordings_enabled() && $record) {
+    if ((boolean) \mod_bigbluebuttonbn\locallib\config::recordings_enabled() && $record) {
         return 'true';
     }
     return 'false';
@@ -353,7 +353,7 @@ 
  * @return integer
  */
 function bigbluebuttonbn_bbb_view_create_meeting_data_duration($closingtime) {
-    if ((boolean)\mod_bigbluebuttonbn\locallib\config::get('scheduled_duration_enabled')) {
+    if ((boolean) \mod_bigbluebuttonbn\locallib\config::get('scheduled_duration_enabled')) {
         return bigbluebuttonbn_get_duration($closingtime);
     }
     return 0;

classes/locallib/mobileview.php

Spacing +3 added lines, -3 removed lines

@@ -26,7 +26,7 @@ 
 
 defined('MOODLE_INTERNAL') || die();
 global $CFG;
-require_once($CFG->dirroot . '/mod/bigbluebuttonbn/locallib.php');
+require_once($CFG->dirroot.'/mod/bigbluebuttonbn/locallib.php');
 
 /**
  * Methods used to render view BBB in mobile.
@@ -138,7 +138,7 @@ 
      * @return string
      */
     public static function create_meeting_data_record($record) {
-        if ((boolean)\mod_bigbluebuttonbn\locallib\config::recordings_enabled() && $record) {
+        if ((boolean) \mod_bigbluebuttonbn\locallib\config::recordings_enabled() && $record) {
             return 'true';
         }
         return 'false';
@@ -151,7 +151,7 @@ 
      * @return integer
      */
     public static function create_meeting_data_duration($closingtime) {
-        if ((boolean)\mod_bigbluebuttonbn\locallib\config::get('scheduled_duration_enabled')) {
+        if ((boolean) \mod_bigbluebuttonbn\locallib\config::get('scheduled_duration_enabled')) {
             return bigbluebuttonbn_get_duration($closingtime);
         }
         return 0;