jfederico / moodle-mod_bigbluebuttonbn

config-dist.php

Indentation +7 added lines, -7 removed lines

@@ -103,7 +103,7 @@ 
  * recordings from a different activity even from a different course.
  **/
 
- /*
+    /*
  * When the value is set to 1 (checked) the bigbluebuttonbn rooms or
  * activities will have the 'import recordings' capability enabled.
  * $CFG->bigbluebuttonbn['importrecordings_enabled'] = 0;
@@ -124,7 +124,7 @@ 
  * 'Join session' button enabled
  **/
 
- /*
+    /*
  * When the value is set to 1 (checked) the bigbluebuttonbn rooms or
  * activities will have the 'wait for moderator' capability enabled by
  * default.
@@ -276,7 +276,7 @@ 
  * $CFG->bigbluebuttonbn['recordings_preview_editable'] = 0;
  */
 
- /* When the value is set to 1 (checked) the playback URLs will be validated
+    /* When the value is set to 1 (checked) the playback URLs will be validated
   * before the user access it.
   * $CFG->bigbluebuttonbn['recordings_validate_url'] = 1;
   */
@@ -366,27 +366,27 @@ 
  * $CFG->bigbluebuttonbn['general_warning_message'] = "Would you like to record your BigBlueButton sessions for later viewing? ";
  */
 
- /*
+    /*
  * The warning box is always shown to administrators, but it is also possible to define other roles
  * to whom the it will be shown. The roles are based on the shortnames defined by Moodle:
  *     'manager,coursecreator,editingteacher,teacher,student,guest,user,frontpage'
  * $CFG->bigbluebuttonbn['general_warning_roles'] = 'editingteacher,teacher';
  */
 
- /*
+    /*
  * As the general_warning_message is shown in a box, its type can be defined with general_warning_type
  * The default type is 'info' which is normaly rendered in blue when using a bootstrap theme.
  * All the modifiers for boxed in bootstrap can be used [info|success|warning|danger].
  * $CFG->bigbluebuttonbn['general_warning_box_type'] = 'info';
  */
 
- /*
+    /*
  * Additionally, when general_warning_button_href value is different than "", a button
  * can also be shown right after the message.
  * $CFG->bigbluebuttonbn['general_warning_button_href'] = "http://blindsidenetworks.com/";
  */
 
- /*
+    /*
  * Finally, the text and class for the button can be modified
  * $CFG->bigbluebuttonbn['general_warning_button_text'] = "Upgrade your site";
  * $CFG->bigbluebuttonbn['general_warning_button_class'] = "btn btn-primary";

classes/settings/renderer.php

Spacing +24 added lines, -24 removed lines

@@ -27,7 +27,7 @@ 
 
 defined('MOODLE_INTERNAL') || die();
 
-require_once($CFG->dirroot . '/mod/bigbluebuttonbn/locallib.php');
+require_once($CFG->dirroot.'/mod/bigbluebuttonbn/locallib.php');
 require_once($CFG->libdir.'/adminlib.php');
 
 /**
@@ -63,12 +63,12 @@ 
      */
     public function render_group_header($name, $itemname = null, $itemdescription = null) {
         if ($itemname === null) {
-            $itemname = get_string('config_' . $name, 'bigbluebuttonbn');
+            $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);
+        $item = new \admin_setting_heading('bigbluebuttonbn_config_'.$name, $itemname, $itemdescription);
         $this->settings->add($item);
     }
 
@@ -97,9 +97,9 @@ 
      * @return Object
      */
     public function render_group_element_text($name, $default = null, $type = PARAM_RAW) {
-        $item = new \admin_setting_configtext('bigbluebuttonbn_' . $name,
-                get_string('config_' . $name, 'bigbluebuttonbn'),
-                get_string('config_' . $name . '_description', 'bigbluebuttonbn'),
+        $item = new \admin_setting_configtext('bigbluebuttonbn_'.$name,
+                get_string('config_'.$name, 'bigbluebuttonbn'),
+                get_string('config_'.$name.'_description', 'bigbluebuttonbn'),
                 $default, $type);
         return $item;
     }
@@ -113,9 +113,9 @@ 
      * @return Object
      */
     public function render_group_element_checkbox($name, $default = null) {
-        $item = new \admin_setting_configcheckbox('bigbluebuttonbn_' . $name,
-                get_string('config_' . $name, 'bigbluebuttonbn'),
-                get_string('config_' . $name . '_description', 'bigbluebuttonbn'),
+        $item = new \admin_setting_configcheckbox('bigbluebuttonbn_'.$name,
+                get_string('config_'.$name, 'bigbluebuttonbn'),
+                get_string('config_'.$name.'_description', 'bigbluebuttonbn'),
                 $default);
         return $item;
     }
@@ -130,9 +130,9 @@ 
      * @return Object
      */
     public function render_group_element_configmultiselect($name, $defaultsetting, $choices) {
-        $item = new \admin_setting_configmultiselect('bigbluebuttonbn_' . $name,
-                get_string('config_' . $name, 'bigbluebuttonbn'),
-                get_string('config_' . $name . '_description', 'bigbluebuttonbn'),
+        $item = new \admin_setting_configmultiselect('bigbluebuttonbn_'.$name,
+                get_string('config_'.$name, 'bigbluebuttonbn'),
+                get_string('config_'.$name.'_description', 'bigbluebuttonbn'),
                 $defaultsetting, $choices);
         return $item;
     }
@@ -147,9 +147,9 @@ 
      * @return Object
      */
     public function render_group_element_configselect($name, $defaultsetting, $choices) {
-        $item = new \admin_setting_configselect('bigbluebuttonbn_' . $name,
-                get_string('config_' . $name, 'bigbluebuttonbn'),
-                get_string('config_' . $name . '_description', 'bigbluebuttonbn'),
+        $item = new \admin_setting_configselect('bigbluebuttonbn_'.$name,
+                get_string('config_'.$name, 'bigbluebuttonbn'),
+                get_string('config_'.$name.'_description', 'bigbluebuttonbn'),
                 $defaultsetting, $choices);
         return $item;
     }
@@ -165,14 +165,14 @@ 
      * @return Object
      */
     public function render_warning_message($name, $message, $type = 'warning', $closable = true) {
-        $output = $this->output->box_start('box boxalignleft adminerror alert alert-' . $type . ' alert-block fade in',
-            'bigbluebuttonbn_' . $name)."\n";
+        $output = $this->output->box_start('box boxalignleft adminerror alert alert-'.$type.' alert-block fade in',
+            'bigbluebuttonbn_'.$name)."\n";
         if ($closable) {
-            $output .= '  <button type="button" class="close" data-dismiss="alert">&times;</button>' . "\n";
+            $output .= '  <button type="button" class="close" data-dismiss="alert">&times;</button>'."\n";
         }
-        $output .= '  ' . $message . "\n";
-        $output .= $this->output->box_end() . "\n";
-        $item = new \admin_setting_heading('bigbluebuttonbn_' . $name, '', $output);
+        $output .= '  '.$message."\n";
+        $output .= $this->output->box_end()."\n";
+        $item = new \admin_setting_heading('bigbluebuttonbn_'.$name, '', $output);
         $this->settings->add($item);
         return $item;
     }
@@ -194,8 +194,8 @@ 
         $filemanageroptions['mainfile'] = true;
 
         $filemanager = new \admin_setting_configstoredfile('mod_bigbluebuttonbn/presentationdefault',
-            get_string('config_' . $name, 'bigbluebuttonbn'),
-            get_string('config_' . $name . '_description', 'bigbluebuttonbn'),
+            get_string('config_'.$name, 'bigbluebuttonbn'),
+            get_string('config_'.$name.'_description', 'bigbluebuttonbn'),
             'presentationdefault',
             0,
             $filemanageroptions);

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/locallib/mobileview.php

Spacing +7 added lines, -7 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.
@@ -95,18 +95,18 @@ 
             'bbb-recording-tags' => bigbluebuttonbn_get_tags($bbbsession['cm']->id), // Same as $id.
         ];
         // Check recording status.
-        if ((boolean)\mod_bigbluebuttonbn\locallib\config::get('recordingstatus_enabled')) {
+        if ((boolean) \mod_bigbluebuttonbn\locallib\config::get('recordingstatus_enabled')) {
             $metadatabbb["bn-recording-status"] = json_encode(
                 array(
-                    'email' => array('"' . fullname($USER) . '" <' . $USER->email . '>'),
+                    'email' => array('"'.fullname($USER).'" <'.$USER->email.'>'),
                     'context' => $bbbsession['bigbluebuttonbnURL']
                 )
             );
         }
-        if ((boolean)\mod_bigbluebuttonbn\locallib\config::get('recordingready_enabled')) {
+        if ((boolean) \mod_bigbluebuttonbn\locallib\config::get('recordingready_enabled')) {
             $metadatabbb['bn-recording-ready-url'] = $bbbsession['recordingReadyURL'];
         }
-        if ((boolean)\mod_bigbluebuttonbn\locallib\config::get('meetingevents_enabled')) {
+        if ((boolean) \mod_bigbluebuttonbn\locallib\config::get('meetingevents_enabled')) {
             $metadatabbb['bn-meeting-events-url'] = $bbbsession['meetingEventsURL'];
         }
         return $metadatabbb;
@@ -167,7 +167,7 @@ 
      * @return string
      */
     public static function bigbluebutton_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';
@@ -180,7 +180,7 @@ 
      * @return integer
      */
     public static function bigbluebutton_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/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';