blindsidenetworks / moodle-mod_bigbluebuttonbn

classes/event/bigbluebuttonbn_meeting_joined.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
      * @return string
      */
     public function get_description() {
-        $a = (object) array('userid' => $this->userid, 'bigbluebuttonbnid' => $this->objectid, 'courseid' => $this->contextinstanceid);
+        $a = (object)array('userid' => $this->userid, 'bigbluebuttonbnid' => $this->objectid, 'courseid' => $this->contextinstanceid);
         return "The user with id '$a->userid' has joined a bigbluebutton meeting for the bigbluebuttonbn activity with id '$a->bigbluebuttonbnid' for the course id '$a->courseid'.";
     }
 

classes/event/bigbluebuttonbn_recording_imported.php

Spacing +2 added lines, -2 removed lines

@@ -38,8 +38,8 @@
      * @return string
      */
     public function get_description() {
-        $rid = isset($this->other['rid'])? $this->other['rid']: '';
-        $a = (object) array('userid' => $this->userid, 'recordingid' => $rid, 'courseid' => $this->contextinstanceid);
+        $rid = isset($this->other['rid']) ? $this->other['rid'] : '';
+        $a = (object)array('userid' => $this->userid, 'recordingid' => $rid, 'courseid' => $this->contextinstanceid);
         return "The user with id '$a->userid' has imported a recording with id '$a->recordingid' into the course id '$a->courseid'.";
     }
 

classes/event/bigbluebuttonbn_meeting_created.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
      * @return string
      */
     public function get_description() {
-        $a = (object) array('userid' => $this->userid, 'bigbluebuttonbnid' => $this->objectid, 'courseid' => $this->contextinstanceid);
+        $a = (object)array('userid' => $this->userid, 'bigbluebuttonbnid' => $this->objectid, 'courseid' => $this->contextinstanceid);
         return "The user with id '$a->userid' created a bigbluebutton meeting for the bigbluebuttonbn activity with id '$a->bigbluebuttonbnid' for the course id '$a->courseid'.";
     }
 

classes/event/bigbluebuttonbn_recording_published.php

Spacing +2 added lines, -2 removed lines

@@ -38,8 +38,8 @@
      * @return string
      */
     public function get_description() {
-        $rid = isset($this->other['rid'])? $this->other['rid']: '';
-        $a = (object) array('userid' => $this->userid, 'recordingid' => $rid, 'courseid' => $this->contextinstanceid);
+        $rid = isset($this->other['rid']) ? $this->other['rid'] : '';
+        $a = (object)array('userid' => $this->userid, 'recordingid' => $rid, 'courseid' => $this->contextinstanceid);
         return "The user with id '$a->userid' has published a recording with id '$a->recordingid' in the course id '$a->courseid'.";
     }
 

classes/event/bigbluebuttonbn_meeting_ended.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
      * @return string
      */
     public function get_description() {
-        $a = (object) array('userid' => $this->userid, 'bigbluebuttonbnid' => $this->objectid, 'courseid' => $this->contextinstanceid);
+        $a = (object)array('userid' => $this->userid, 'bigbluebuttonbnid' => $this->objectid, 'courseid' => $this->contextinstanceid);
         return "A bigbluebutton meeting for the bigbluebuttonbn activity with id '$a->bigbluebuttonbnid' for the course id '$a->courseid' has been forcibly ended by the user with id '$a->userid'.";
     }
 

classes/event/bigbluebuttonbn_meeting_left.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
      * @return string
      */
     public function get_description() {
-        $a = (object) array('userid' => $this->userid, 'bigbluebuttonbnid' => $this->objectid, 'courseid' => $this->contextinstanceid);
+        $a = (object)array('userid' => $this->userid, 'bigbluebuttonbnid' => $this->objectid, 'courseid' => $this->contextinstanceid);
         return "The user with id '$a->userid' has left a bigbluebutton meeting for the bigbluebuttonbn activity with id '$a->bigbluebuttonbnid' for the course id '$a->courseid'.";
     }
 

db/upgrade.php

Spacing +50 added lines, -50 removed lines

@@ -10,7 +10,7 @@ 
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v2 or later
  */
 
-function xmldb_bigbluebuttonbn_upgrade($oldversion=0) {
+function xmldb_bigbluebuttonbn_upgrade($oldversion = 0) {
 
     global $CFG, $THEME, $DB;
     $dbman = $DB->get_manager(); // loads ddl manager and xmldb classes
@@ -20,11 +20,11 @@ 
     if ($result && $oldversion < 2012040200) {
         // Define field intro to be droped from bigbluebuttonbn
         $table = new xmldb_table('bigbluebuttonbn');
-        $field = new xmldb_field('intro', XMLDB_TYPE_TEXT, 'medium', null, null, null, null,'name');
+        $field = new xmldb_field('intro', XMLDB_TYPE_TEXT, 'medium', null, null, null, null, 'name');
 
         // Drop field intro
         if ($dbman->field_exists($table, $field)) {
-            $dbman->drop_field($table, $field, $continue=true, $feedback=true);
+            $dbman->drop_field($table, $field, $continue = true, $feedback = true);
         }
 
         // Define field introformat to be droped from bigbluebuttonbn
@@ -33,7 +33,7 @@ 
 
         // Drop field introformat
         if ($dbman->field_exists($table, $field)) {
-            $dbman->drop_field($table, $field, $continue=true, $feedback=true);
+            $dbman->drop_field($table, $field, $continue = true, $feedback = true);
         }
 
         // Once we reach this point, we can store the new version and consider the module
@@ -73,7 +73,7 @@ 
         $field = new xmldb_field('welcome');
         $field->set_attributes(XMLDB_TYPE_TEXT, null, null, XMLDB_NOTNULL, null, null, null, null, 'type');
 
-        $dbman->change_field_type($table, $field, $continue=true, $feedback=true);
+        $dbman->change_field_type($table, $field, $continue = true, $feedback = true);
 
         upgrade_mod_savepoint(true, 2012100100, 'bigbluebuttonbn');
     }
@@ -85,7 +85,7 @@ 
         $field->set_attributes(XMLDB_TYPE_INTEGER, '1', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, 0);
 
         if (!$dbman->field_exists($table, $field)) {
-            $dbman->add_field($table, $field, $continue=true, $feedback=true);
+            $dbman->add_field($table, $field, $continue = true, $feedback = true);
         }
 
         upgrade_mod_savepoint(true, 2014050100, 'bigbluebuttonbn');
@@ -97,7 +97,7 @@ 
         $field = new xmldb_field('participants', XMLDB_TYPE_TEXT, 'medium', null, null, null, null);
 
         if (!$dbman->field_exists($table, $field)) {
-            $dbman->add_field($table, $field, $continue=true, $feedback=true);
+            $dbman->add_field($table, $field, $continue = true, $feedback = true);
         }
 
         upgrade_mod_savepoint(true, 2014070420, 'bigbluebuttonbn');
@@ -109,7 +109,7 @@ 
         $field = new xmldb_field('participants');
         $field->set_attributes(XMLDB_TYPE_TEXT, 'medium', null, null, null, null);
 
-        $dbman->change_field_type($table, $field, $continue=true, $feedback=true);
+        $dbman->change_field_type($table, $field, $continue = true, $feedback = true);
 
         upgrade_mod_savepoint(true, 2014101004, 'bigbluebuttonbn');
     }
@@ -119,67 +119,67 @@ 
         $table = new xmldb_table('bigbluebuttonbn');
         //// Drop field timeduration
         $field = new xmldb_field('timeduration');
-        if( $dbman->field_exists($table, $field) ) {
-            $dbman->drop_field($table, $field, $continue=true, $feedback=true);
+        if ($dbman->field_exists($table, $field)) {
+            $dbman->drop_field($table, $field, $continue = true, $feedback = true);
         }
         //// Drop field allmoderators
         $field = new xmldb_field('allmoderators');
-        if( $dbman->field_exists($table, $field) ) {
-            $dbman->drop_field($table, $field, $continue=true, $feedback=true);
+        if ($dbman->field_exists($table, $field)) {
+            $dbman->drop_field($table, $field, $continue = true, $feedback = true);
         }
         //// Add field intro
         $field = new xmldb_field('intro');
         $field->set_attributes(XMLDB_TYPE_TEXT, 'medium', null, null, null, null, 'name');
-        if( !$dbman->field_exists($table, $field) ) {
-            $dbman->add_field($table, $field, $continue=true, $feedback=true);
+        if (!$dbman->field_exists($table, $field)) {
+            $dbman->add_field($table, $field, $continue = true, $feedback = true);
         }
         //// Add field introformat
         $field = new xmldb_field('introformat');
         $field->set_attributes(XMLDB_TYPE_INTEGER, '4', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, 1, 'intro');
-        if( !$dbman->field_exists($table, $field) ) {
-            $dbman->add_field($table, $field, $continue=true, $feedback=true);
+        if (!$dbman->field_exists($table, $field)) {
+            $dbman->add_field($table, $field, $continue = true, $feedback = true);
         }
         //// Add field tagging
         $field = new xmldb_field('tagging');
         $field->set_attributes(XMLDB_TYPE_INTEGER, '1', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, 0, 'record');
-        if( !$dbman->field_exists($table, $field) ) {
-            $dbman->add_field($table, $field, $continue=true, $feedback=true);
+        if (!$dbman->field_exists($table, $field)) {
+            $dbman->add_field($table, $field, $continue = true, $feedback = true);
         }
         //// Add field presentation
         $field = new xmldb_field('presentation');
         $field->set_attributes(XMLDB_TYPE_TEXT, 'medium', null, null, null, null, 'timemodified');
-        if( !$dbman->field_exists($table, $field) ) {
-            $dbman->add_field($table, $field, $continue=true, $feedback=true);
+        if (!$dbman->field_exists($table, $field)) {
+            $dbman->add_field($table, $field, $continue = true, $feedback = true);
         }
         //// Add field type
         $field = new xmldb_field('type');
         $field->set_attributes(XMLDB_TYPE_INTEGER, '2', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, 0, 'course');
-        if( !$dbman->field_exists($table, $field) ) {
-            $dbman->add_field($table, $field, $continue=true, $feedback=true);
+        if (!$dbman->field_exists($table, $field)) {
+            $dbman->add_field($table, $field, $continue = true, $feedback = true);
         }
         //// Rename field timeavailable
         $field = new xmldb_field('timeavailable');
         $field->set_attributes(XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, 0);
-        if( $dbman->field_exists($table, $field) ) {
-            $dbman->rename_field($table, $field, 'openingtime', $continue=true, $feedback=true);
+        if ($dbman->field_exists($table, $field)) {
+            $dbman->rename_field($table, $field, 'openingtime', $continue = true, $feedback = true);
         }
         //// Rename field timedue
         $field = new xmldb_field('timedue');
         $field->set_attributes(XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, 0);
-        if( $dbman->field_exists($table, $field) ) {
-            $dbman->rename_field($table, $field, 'closingtime', $continue=true, $feedback=true);
+        if ($dbman->field_exists($table, $field)) {
+            $dbman->rename_field($table, $field, 'closingtime', $continue = true, $feedback = true);
         }
         //// Add field timecreated
         $field = new xmldb_field('timecreated');
         $field->set_attributes(XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, 0, 'closingtime');
-        if( !$dbman->field_exists($table, $field) ) {
-            $dbman->add_field($table, $field, $continue=true, $feedback=true);
+        if (!$dbman->field_exists($table, $field)) {
+            $dbman->add_field($table, $field, $continue = true, $feedback = true);
         }
         //// Add field userlimit
         $field = new xmldb_field('userlimit');
         $field->set_attributes(XMLDB_TYPE_INTEGER, '3', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, 0);
         if (!$dbman->field_exists($table, $field)) {
-            $dbman->add_field($table, $field, $continue=true, $feedback=true);
+            $dbman->add_field($table, $field, $continue = true, $feedback = true);
         }
 
         // Update the bigbluebuttonbn_log table
@@ -187,18 +187,18 @@ 
         //// Add field userid
         $field = new xmldb_field('userid');
         $field->set_attributes(XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, 0, 'bigbluebuttonbnid');
-        if( !$dbman->field_exists($table, $field) ) {
-            $dbman->add_field($table, $field, $continue=true, $feedback=true);
+        if (!$dbman->field_exists($table, $field)) {
+            $dbman->add_field($table, $field, $continue = true, $feedback = true);
         }
         //// Add field meta
         $field = new xmldb_field('meta');
         $field->set_attributes(XMLDB_TYPE_TEXT, 'medium', null, null, null, null, 'event');
-        if( !$dbman->field_exists($table, $field) ) {
-            $dbman->add_field($table, $field, $continue=true, $feedback=true);
+        if (!$dbman->field_exists($table, $field)) {
+            $dbman->add_field($table, $field, $continue = true, $feedback = true);
         }
         //// Drop field recording
         $field = new xmldb_field('record');
-        if( $dbman->field_exists($table, $field) ) {
+        if ($dbman->field_exists($table, $field)) {
             //// Migrate data in field recording to new format in meta
             $meta = new \stdClass();
 
@@ -211,7 +211,7 @@ 
             $DB->set_field('bigbluebuttonbn_log', 'meta', json_encode($meta), array('event' => 'Create', 'record' => 0));
 
             // Drop field recording
-            $dbman->drop_field($table, $field, $continue=true, $feedback=true);
+            $dbman->drop_field($table, $field, $continue = true, $feedback = true);
         }
 
         upgrade_mod_savepoint(true, 2015063000, 'bigbluebuttonbn');
@@ -222,14 +222,14 @@ 
         $table = new xmldb_table('bigbluebuttonbn');
         //// Drop field description
         $field = new xmldb_field('description');
-        if( $dbman->field_exists($table, $field) ) {
-            $dbman->drop_field($table, $field, $continue=true, $feedback=true);
+        if ($dbman->field_exists($table, $field)) {
+            $dbman->drop_field($table, $field, $continue = true, $feedback = true);
         }
         //// Change welcome, allow null
         $field = new xmldb_field('welcome');
         $field->set_attributes(XMLDB_TYPE_TEXT, null, null, null, null, null, null, null, 'type');
-        if( $dbman->field_exists($table, $field) ) {
-            $dbman->change_field_notnull($table, $field, $continue=true, $feedback=true);
+        if ($dbman->field_exists($table, $field)) {
+            $dbman->change_field_notnull($table, $field, $continue = true, $feedback = true);
         }
 
         // Update the bigbluebuttonbn_log table
@@ -237,22 +237,22 @@ 
         //// Change welcome, allow null
         $field = new xmldb_field('userid');
         $field->set_attributes(XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, null, null, null, null, null, 'bigbluebuttonbnid');
-        if( $dbman->field_exists($table, $field) ) {
-            $dbman->change_field_notnull($table, $field, $continue=true, $feedback=true);
+        if ($dbman->field_exists($table, $field)) {
+            $dbman->change_field_notnull($table, $field, $continue = true, $feedback = true);
         }
 
         upgrade_mod_savepoint(true, 2015080605, 'bigbluebuttonbn');
     }
 
-    if ( $result && $oldversion < 2016011305 ) {
+    if ($result && $oldversion < 2016011305) {
         // Update the bigbluebuttonbn table
         $table = new xmldb_table('bigbluebuttonbn');
 
         // Define field type to be droped from bigbluebuttonbn
         $field = new xmldb_field('type');
         $field->set_attributes(XMLDB_TYPE_INTEGER, '2', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, 0, 'course');
-        if ( $dbman->field_exists($table, $field) ) {
-            $dbman->drop_field($table, $field, $continue=true, $feedback=true);
+        if ($dbman->field_exists($table, $field)) {
+            $dbman->drop_field($table, $field, $continue = true, $feedback = true);
         }
 
         // Define field 'event' to be renamed
@@ -264,17 +264,17 @@ 
         // Conditionally rename the table
         if ($dbman->table_exists($table)) {
             // Update the bigbluebuttonbn_log table
-            if ( $dbman->field_exists($table, $field) ) {
-                $dbman->rename_field($table, $field, 'log', $continue=true, $feedback=true);
+            if ($dbman->field_exists($table, $field)) {
+                $dbman->rename_field($table, $field, 'log', $continue = true, $feedback = true);
             }
             // Rename bigbluebuttonbn_log table to bigbluebuttonbn_logs
-            $dbman->rename_table($table, 'bigbluebuttonbn_logs', $continue=true, $feedback=true);
+            $dbman->rename_table($table, 'bigbluebuttonbn_logs', $continue = true, $feedback = true);
         } else {
             // It was already renamed, select it only
             $table = new xmldb_table('bigbluebuttonbn_logs');
             // Update the bigbluebuttonbn_logs table
-            if ( $dbman->field_exists($table, $field) ) {
-                $dbman->rename_field($table, $field, 'log', $continue=true, $feedback=true);
+            if ($dbman->field_exists($table, $field)) {
+                $dbman->rename_field($table, $field, 'log', $continue = true, $feedback = true);
             }
         }
 
@@ -287,7 +287,7 @@ 
         //// Drop field newwindow
         $field = new xmldb_field('newwindow');
         if ($dbman->field_exists($table, $field)) {
-            $dbman->drop_field($table, $field, $continue=true, $feedback=true);
+            $dbman->drop_field($table, $field, $continue = true, $feedback = true);
         }
 
         upgrade_mod_savepoint(true, 2016051910, 'bigbluebuttonbn');

bbb_broker.php

Braces +3 added lines, -2 removed lines

@@ -118,8 +118,9 @@
                         //Execute the end command
                         $meeting_info = bigbluebuttonbn_bbb_broker_do_end_meeting($params['id'], $bbbsession['modPW']);
                         // Moodle event logger: Create an event for meeting ended
-                        if( isset($bigbluebuttonbn) )
-                            bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_ENDED, $bigbluebuttonbn, $context, $cm);
+                        if( isset($bigbluebuttonbn) ) {
+                                                    bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_ENDED, $bigbluebuttonbn, $context, $cm);
+                        }
                         // Update the cache
                         $meeting_info = bigbluebuttonbn_bbb_broker_get_meeting_info($params['id'], $bbbsession['modPW'], true);
 

Spacing +11 added lines, -11 removed lines

@@ -81,49 +81,49 @@
                             $can_join = false;
                         }
 
-                        if( $bbbsession['administrator'] || $bbbsession['moderator'] ) {
+                        if ($bbbsession['administrator'] || $bbbsession['moderator']) {
                             $end_button_text = get_string('view_conference_action_end', 'bigbluebuttonbn');
                             $can_end = true;
-                            $status_can_end = '"can_end": true, "end_button_text": "'.$end_button_text.'", ';
+                            $status_can_end = '"can_end": true, "end_button_text": "' . $end_button_text . '", ';
                         }
 
                     } else {
                         // If user is administrator, moderator or if is viewer and no waiting is required
                         $join_button_text = get_string('view_conference_action_join', 'bigbluebuttonbn');
-                        if ( $bbbsession['administrator'] || $bbbsession['moderator'] || !$bbbsession['wait'] ) {
+                        if ($bbbsession['administrator'] || $bbbsession['moderator'] || !$bbbsession['wait']) {
                             $initial_message = get_string('view_message_conference_room_ready', 'bigbluebuttonbn');
                             $can_join = true;
 
                         } else {
                             $initial_message = get_string('view_message_conference_not_started', 'bigbluebuttonbn');
-                            if ( $bbbsession['wait'] ) {
-                                $initial_message .= ' '.get_string('view_message_conference_wait_for_moderator', 'bigbluebuttonbn');
+                            if ($bbbsession['wait']) {
+                                $initial_message .= ' ' . get_string('view_message_conference_wait_for_moderator', 'bigbluebuttonbn');
                             }
                             $can_join = false;
                         }
 
-                        if( $bbbsession['tagging'] && ($bbbsession['administrator'] || $bbbsession['moderator']) ) {
+                        if ($bbbsession['tagging'] && ($bbbsession['administrator'] || $bbbsession['moderator'])) {
                             $can_tag = true;
 
                         } else {
                             $can_tag = false;
                         }
-                        $status_can_end = '"can_tag": '.($can_tag? 'true': 'false').', ';
+                        $status_can_end = '"can_tag": ' . ($can_tag ? 'true' : 'false') . ', ';
                     }
 
-                    echo $params['callback'].'({ "running": '.($meeting_running? 'true':'false').', "info": '.json_encode($meeting_info).', "status": {"can_join": '.($can_join? 'true':'false').',"join_url": "'.$bbbsession['joinURL'].'","join_button_text": "'.$join_button_text.'", '.$status_can_end.$status_can_tag.'"message": "'.$initial_message.'"} });';
+                    echo $params['callback'] . '({ "running": ' . ($meeting_running ? 'true' : 'false') . ', "info": ' . json_encode($meeting_info) . ', "status": {"can_join": ' . ($can_join ? 'true' : 'false') . ',"join_url": "' . $bbbsession['joinURL'] . '","join_button_text": "' . $join_button_text . '", ' . $status_can_end . $status_can_tag . '"message": "' . $initial_message . '"} });';
                     break;
                 case 'meeting_end':
-                    if( $bbbsession['administrator'] || $bbbsession['moderator'] ) {
+                    if ($bbbsession['administrator'] || $bbbsession['moderator']) {
                         //Execute the end command
                         $meeting_info = bigbluebuttonbn_bbb_broker_do_end_meeting($params['id'], $bbbsession['modPW']);
                         // Moodle event logger: Create an event for meeting ended
-                        if( isset($bigbluebuttonbn) )
+                        if (isset($bigbluebuttonbn))
                             bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_ENDED, $bigbluebuttonbn, $context, $cm);
                         // Update the cache
                         $meeting_info = bigbluebuttonbn_bbb_broker_get_meeting_info($params['id'], $bbbsession['modPW'], true);
 
-                        echo $params['callback'].'({ "status": true });';
+                        echo $params['callback'] . '({ "status": true });';
                     } else {
                         // ERROR: User not authorized to execute end command.
                         header("HTTP/1.0 401 Unauthorized. User not authorized to execute end command");

import_view.php

Indentation +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
         'class' => 'btn btn-default',
         'value' => get_string('view_recording_button_return', 'bigbluebuttonbn'),
         'onclick' => 'window.location=\'' . $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $cm->id . '\''
-      );
+        );
     $output .= html_writer::tag('input', '', $buttonoptions);
 
     $jsvars = array(