blindsidenetworks / moodle-mod_bigbluebuttonbn

backup/moodle2/backup_bigbluebuttonbn_activity_task.class.php

Spacing +4 added lines, -4 removed lines

@@ -25,7 +25,7 @@ 
 
 defined('MOODLE_INTERNAL') || die;
 
-require_once($CFG->dirroot.'/mod/bigbluebuttonbn/backup/moodle2/backup_bigbluebuttonbn_stepslib.php');
+require_once($CFG->dirroot . '/mod/bigbluebuttonbn/backup/moodle2/backup_bigbluebuttonbn_stepslib.php');
 
 /**
  * Backup task that provides all the settings and steps to perform one complete backup of the activity.
@@ -62,14 +62,14 @@ 
     public static function encode_content_links($content) {
         global $CFG;
 
-        $base = preg_quote($CFG->wwwroot.'/mod/bigbluebuttonbn', '#');
+        $base = preg_quote($CFG->wwwroot . '/mod/bigbluebuttonbn', '#');
 
         // Link to the list of bigbluebuttonbns.
-        $pattern = '#('.$base."\/index.php\?id\=)([0-9]+)#";
+        $pattern = '#(' . $base . "\/index.php\?id\=)([0-9]+)#";
         $content = preg_replace($pattern, '$@BIGBLUEBUTTONBNINDEX*$2@$', $content);
 
         // Link to bigbluebuttonbn view by moduleid.
-        $pattern = '#('.$base."\/view.php\?id\=)([0-9]+)#";
+        $pattern = '#(' . $base . "\/view.php\?id\=)([0-9]+)#";
         $content = preg_replace($pattern, '$@BIGBLUEBUTTONBNVIEWBYID*$2@$', $content);
 
         return $content;

backup/moodle2/backup_bigbluebuttonbn_stepslib.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
         $logs = new backup_nested_element('logs');
 
         $log = new backup_nested_element('log', array('id'), array(
-                'courseid', 'bigbluebuttonbnid', 'userid', 'timecreated', 'meetingid', 'log', 'meta', ));
+                'courseid', 'bigbluebuttonbnid', 'userid', 'timecreated', 'meetingid', 'log', 'meta',));
 
         // Build the tree.
         $bigbluebuttonbn->add_child($logs);

backup/moodle2/restore_bigbluebuttonbn_stepslib.php

Spacing +2 added lines, -2 removed lines

@@ -49,7 +49,7 @@ 
      */
     protected function process_bigbluebuttonbn($data) {
         global $DB;
-        $data = (object) $data;
+        $data = (object)$data;
         $data->course = $this->get_courseid();
         $data->timemodified = $this->apply_date_offset($data->timemodified);
         // Insert the bigbluebuttonbn record.
@@ -65,7 +65,7 @@ 
      */
     protected function process_bigbluebuttonbn_logs($data) {
         global $DB;
-        $data = (object) $data;
+        $data = (object)$data;
         // Apply modifications.
         $data->courseid = $this->get_mappingid('course', $data->courseid);
         $data->bigbluebuttonbnid = $this->get_new_parentid('bigbluebuttonbn');

backup/moodle2/restore_bigbluebuttonbn_activity_task.class.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
 
 defined('MOODLE_INTERNAL') || die();
 
-require_once($CFG->dirroot.'/mod/bigbluebuttonbn/backup/moodle2/restore_bigbluebuttonbn_stepslib.php');
+require_once($CFG->dirroot . '/mod/bigbluebuttonbn/backup/moodle2/restore_bigbluebuttonbn_stepslib.php');
 
 /**
  * Restore task that provides all the settings and steps to perform one complete restore of the activity.

classes/settings/renderer.php

Spacing +3 added lines, -3 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);
@@ -148,7 +148,7 @@ 
     public function render_warning_message($name, $message, $type = 'warning', $closable = true) {
         global $OUTPUT;
         $output = $OUTPUT->box_start('box boxalignleft adminerror alert alert-' . $type . ' alert-block fade in',
-            'bigbluebuttonbn_' . $name)."\n";
+            'bigbluebuttonbn_' . $name) . "\n";
         if ($closable) {
             $output .= '  <button type="button" class="close" data-dismiss="alert">&times;</button>' . "\n";
         }

classes/event/base.php

Indentation +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
             'objectid' => $this->objectid,
             'contextinstanceid' => $this->contextinstanceid,
             'other' => $this->other
-          );
+            );
         $string = $this->description;
         foreach ($vars as $key => $value) {
             $string = str_replace("##" . $key, $value, $string);

db/upgrade.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 
 defined('MOODLE_INTERNAL') || die();
 
-require_once(dirname(dirname(__FILE__)).'/locallib.php');
+require_once(dirname(dirname(__FILE__)) . '/locallib.php');
 
 /**
  * Performs data migrations and updates on upgrade.

classes/privacy/provider.php

Spacing +1 added lines, -1 removed lines

@@ -212,7 +212,7 @@
             function($instanceid, $data) use ($user, $instanceidstocmids) {
                 $context = \context_module::instance($instanceidstocmids[$instanceid]);
                 $contextdata = helper::get_context_data($context, $user);
-                $finaldata = (object) array_merge((array) $contextdata, ['logs' => $data]);
+                $finaldata = (object)array_merge((array)$contextdata, ['logs' => $data]);
                 helper::export_context_files($context, $user);
                 writer::with_context($context)->export_data([], $finaldata);
             }

Indentation +6 added lines, -6 removed lines

@@ -61,11 +61,11 @@ 
      */
     public static function _get_metadata(collection $collection) {
 
-         // The table bigbluebuttonbn stores only the room properties.
-         // However, there is a chance that some personal information is stored as metadata.
-         // This would be done in the column 'participants' where rules can be set to define BBB roles.
-         // It is fair to say that only the userid is stored, which is useless if user is removed.
-         // But if this is a concern a refactoring on the way the rules are stored will be required.
+            // The table bigbluebuttonbn stores only the room properties.
+            // However, there is a chance that some personal information is stored as metadata.
+            // This would be done in the column 'participants' where rules can be set to define BBB roles.
+            // It is fair to say that only the userid is stored, which is useless if user is removed.
+            // But if this is a concern a refactoring on the way the rules are stored will be required.
         $collection->add_database_table('bigbluebuttonbn', [
             'participants' => 'privacy:metadata:bigbluebuttonbn:participants',
         ], 'privacy:metadata:bigbluebuttonbn');
@@ -208,7 +208,7 @@ 
                     'meetingid' => $record->meetingid,
                     'log' => $record->log,
                     'meta' => $record->meta,
-                  ];
+                    ];
                 return $carry;
             },
             function($instanceid, $data) use ($user, $instanceidstocmids) {

Doc Comments +2 added lines

@@ -54,6 +54,7 @@ 
          * Get the list of users who have data within a context.
          *
          * @param   userlist $userlist The userlist containing the list of users who have data in this context/plugin combination.
+         * @return void
          */
         public static function get_users_in_context(userlist $userlist);
 
@@ -61,6 +62,7 @@ 
          * Delete multiple users within a single context.
          *
          * @param   approved_userlist       $userlist The approved context and user information to delete information for.
+         * @return void
          */
         public static function delete_data_for_users(approved_userlist $userlist);
     }

classes/external.php

Spacing +1 added lines, -1 removed lines

@@ -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()