blindsidenetworks / moodle-mod_bigbluebuttonbn

JWT.php

Doc Comments +1 added lines, -1 removed lines

@@ -137,7 +137,7 @@
      * Sign a string with a given key and algorithm.
      *
      * @param string $msg          The message to sign
-     * @param string|resource $key The secret key
+     * @param string $key The secret key
      * @param string $alg       The signing algorithm. Supported algorithms
      *                               are 'HS256', 'HS384', 'HS512' and 'RS256'
      *

Spacing +4 added lines, -4 removed lines

@@ -150,7 +150,7 @@ 
             throw new DomainException('Algorithm not supported');
         }
         list($function, $algorithm) = self::$supported_algs[$alg];
-        switch($function) {
+        switch ($function) {
             case 'hash_hmac':
                 return hash_hmac($algorithm, $msg, $key, true);
             case 'openssl':
@@ -181,7 +181,7 @@ 
         }
 
         list($function, $algorithm) = self::$supported_algs[$alg];
-        switch($function) {
+        switch ($function) {
             case 'openssl':
                 $success = openssl_verify($msg, $signature, $key, $algorithm);
                 if (!$success) {
@@ -228,8 +228,8 @@ 
              * manually detect large ints in the JSON string and quote them (thus converting
              *them to strings) before decoding, hence the preg_replace() call.
              */
-            $max_int_length = strlen((string) PHP_INT_MAX) - 1;
-            $json_without_bigints = preg_replace('/:\s*(-?\d{'.$max_int_length.',})/', ': "$1"', $input);
+            $max_int_length = strlen((string)PHP_INT_MAX) - 1;
+            $json_without_bigints = preg_replace('/:\s*(-?\d{' . $max_int_length . ',})/', ': "$1"', $input);
             $obj = json_decode($json_without_bigints);
         }
 

lib.php

Doc Comments +8 added lines, -10 removed lines

@@ -66,7 +66,6 @@ 
  * will create a new instance and return the id number
  * of the new instance.
  *
- * @param object $bigbluebuttonbn An object from the form in mod_form.php
  * @return int The id of the newly inserted bigbluebuttonbn record
  */
 function bigbluebuttonbn_add_instance($data, $mform) {
@@ -93,7 +92,6 @@ 
  * (defined by the form in mod_form.php) this function
  * will update an existing instance with new data.
  *
- * @param object $bigbluebuttonbn An object from the form in mod_form.php
  * @return boolean Success/Fail
  */
 function bigbluebuttonbn_update_instance($data, $mform) {
@@ -162,7 +160,7 @@ 
  * $return->time = the time they did it
  * $return->info = a short text description
  *
- * @return null
+ * @return boolean
  */
 function bigbluebuttonbn_user_outline($course, $user, $mod, $bigbluebuttonbn) {
     return true;
@@ -230,7 +228,7 @@ 
  * See other modules as example.
  *
  * @param int $bigbluebuttonbnid ID of an instance of this module
- * @return mixed boolean/array of students
+ * @return boolean boolean/array of students
  */
 function bigbluebuttonbn_get_participants($bigbluebuttonbnid) {
     return false;
@@ -238,7 +236,7 @@ 
 
 /**
  * Returns all other caps used in module
- * @return array
+ * @return string[]
  */
 function bigbluebuttonbn_get_extra_capabilities() {
     return array('moodle/site:accessallgroups');
@@ -255,7 +253,7 @@ 
  * as reference.
  *
  * @param int $bigbluebuttonbnid ID of an instance of this module
- * @return mixed
+ * @return boolean
  */
 function bigbluebuttonbn_scale_used($bigbluebuttonbnid, $scaleid) {
     $return = false;
@@ -288,7 +286,7 @@ 
 
 /**
  * List of view style log actions
- * @return array
+ * @return string[]
  */
 function bigbluebuttonbn_get_view_actions() {
     return array('view', 'view all');
@@ -296,7 +294,7 @@ 
 
 /**
  * List of update style log actions
- * @return array
+ * @return string[]
  */
 function bigbluebuttonbn_get_post_actions() {
     return array('update', 'add', 'create', 'join', 'end', 'left', 'publish', 'unpublish', 'delete');
@@ -310,7 +308,7 @@ 
  *
  * @global object
  * @param object $coursemodule
- * @return object|null
+ * @return null|cached_cm_info
  */
 function bigbluebuttonbn_get_coursemodule_info($coursemodule) {
     global $CFG, $DB;
@@ -502,7 +500,7 @@ 
  * @param array $args extra arguments
  * @param bool $forcedownload whether or not force download
  * @param array $options additional options affecting the file serving
- * @return bool false if file not found, does not return if found - justsend the file
+ * @return false|null false if file not found, does not return if found - justsend the file
  */
 function bigbluebuttonbn_pluginfile($course, $cm, $context, $filearea, $args, $forcedownload, array $options=array()) {
     global $CFG, $DB;

Indentation -1 removed lines

@@ -206,7 +206,6 @@
 
 /**
  * Prints single activity item prepared by {@see recordingsbn_get_recent_mod_activity()}
-
  * @return void
  */
 function bigbluebuttonbn_print_recent_mod_activity($activity, $courseid, $detail, $modnames, $viewfullnames) {

Spacing +79 added lines, -79 removed lines

@@ -13,21 +13,21 @@ 
 
 global $BIGBLUEBUTTONBN_CFG, $CFG;
 
-require_once($CFG->dirroot.'/calendar/lib.php');
-require_once($CFG->dirroot.'/message/lib.php');
-require_once($CFG->dirroot.'/lib/accesslib.php');
-require_once($CFG->dirroot.'/lib/filelib.php');
-require_once($CFG->dirroot.'/lib/formslib.php');
-require_once($CFG->dirroot.'/mod/lti/OAuth.php');
-require_once($CFG->libdir.'/accesslib.php');
-require_once($CFG->libdir.'/completionlib.php');
-require_once($CFG->libdir.'/datalib.php');
-require_once(dirname(__FILE__).'/JWT.php');
-
-if( file_exists(dirname(__FILE__).'/config.php') ) {
-    require_once(dirname(__FILE__).'/config.php');
-    if( isset($BIGBLUEBUTTONBN_CFG) ) {
-        $CFG = (object) array_merge((array)$CFG, (array)$BIGBLUEBUTTONBN_CFG);
+require_once($CFG->dirroot . '/calendar/lib.php');
+require_once($CFG->dirroot . '/message/lib.php');
+require_once($CFG->dirroot . '/lib/accesslib.php');
+require_once($CFG->dirroot . '/lib/filelib.php');
+require_once($CFG->dirroot . '/lib/formslib.php');
+require_once($CFG->dirroot . '/mod/lti/OAuth.php');
+require_once($CFG->libdir . '/accesslib.php');
+require_once($CFG->libdir . '/completionlib.php');
+require_once($CFG->libdir . '/datalib.php');
+require_once(dirname(__FILE__) . '/JWT.php');
+
+if (file_exists(dirname(__FILE__) . '/config.php')) {
+    require_once(dirname(__FILE__) . '/config.php');
+    if (isset($BIGBLUEBUTTONBN_CFG)) {
+        $CFG = (object)array_merge((array)$CFG, (array)$BIGBLUEBUTTONBN_CFG);
     }
 } else {
     $BIGBLUEBUTTONBN_CFG = new stdClass();
@@ -43,7 +43,7 @@ 
 
 
 function bigbluebuttonbn_supports($feature) {
-    switch($feature) {
+    switch ($feature) {
         case FEATURE_IDNUMBER:                return true;
         case FEATURE_GROUPS:                  return true;
         case FEATURE_GROUPINGS:               return true;
@@ -72,7 +72,7 @@ 
 function bigbluebuttonbn_add_instance($data, $mform) {
     global $DB, $CFG;
 
-    $draftitemid = isset($data->presentation)? $data->presentation: null;
+    $draftitemid = isset($data->presentation) ? $data->presentation : null;
     $context = bigbluebuttonbn_get_context_module($data->coursemodule);
 
     bigbluebuttonbn_process_pre_save($data);
@@ -100,7 +100,7 @@ 
     global $DB, $CFG;
 
     $data->id = $data->instance;
-    $draftitemid = isset($data->presentation)? $data->presentation: null;
+    $draftitemid = isset($data->presentation) ? $data->presentation : null;
     $context = bigbluebuttonbn_get_context_module($data->coursemodule);
 
     bigbluebuttonbn_process_pre_save($data);
@@ -126,7 +126,7 @@ 
 function bigbluebuttonbn_delete_instance($id) {
     global $CFG, $DB;
 
-    if (! $bigbluebuttonbn = $DB->get_record('bigbluebuttonbn', array('id' => $id))) {
+    if (!$bigbluebuttonbn = $DB->get_record('bigbluebuttonbn', array('id' => $id))) {
         return false;
     }
 
@@ -135,7 +135,7 @@ 
     //
     // End the session associated with this instance (if it's running)
     //
-    $meetingID = $bigbluebuttonbn->meetingid.'-'.$bigbluebuttonbn->course.'-'.$bigbluebuttonbn->id;
+    $meetingID = $bigbluebuttonbn->meetingid . '-' . $bigbluebuttonbn->course . '-' . $bigbluebuttonbn->id;
     
     $modPW = $bigbluebuttonbn->moderatorpass;
     $url = bigbluebuttonbn_get_cfg_server_url();
@@ -144,11 +144,11 @@ 
     //if( bigbluebuttonbn_isMeetingRunning($meetingID, $url, $shared_secret) )
     //    $getArray = bigbluebuttonbn_doEndMeeting( $meetingID, $modPW, $url, $shared_secret );
 
-    if (! $DB->delete_records('bigbluebuttonbn', array('id' => $bigbluebuttonbn->id))) {
+    if (!$DB->delete_records('bigbluebuttonbn', array('id' => $bigbluebuttonbn->id))) {
         $result = false;
     }
 
-    if (! $DB->delete_records('event', array('modulename'=>'bigbluebuttonbn', 'instance'=>$bigbluebuttonbn->id))) {
+    if (!$DB->delete_records('event', array('modulename'=>'bigbluebuttonbn', 'instance'=>$bigbluebuttonbn->id))) {
         $result = false;
     }
 
@@ -186,7 +186,7 @@ 
  * @return boolean
  */
 function bigbluebuttonbn_print_recent_activity($course, $isteacher, $timestart) {
-    return false;  //  True if anything was printed, otherwise false
+    return false; //  True if anything was printed, otherwise false
 }
 
 /**
@@ -201,7 +201,7 @@ 
  * @param int $groupid defaults to 0
  * @return void adds items into $activities and increases $index
  */
-function bigbluebuttonbn_get_recent_mod_activity(&$activities, &$index, $timestart, $courseid, $cmid, $userid=0, $groupid=0) {
+function bigbluebuttonbn_get_recent_mod_activity(&$activities, &$index, $timestart, $courseid, $cmid, $userid = 0, $groupid = 0) {
 }
 
 /**
@@ -219,7 +219,7 @@ 
  *
  * @return boolean
  **/
-function bigbluebuttonbn_cron () {
+function bigbluebuttonbn_cron() {
     return true;
 }
 
@@ -315,7 +315,7 @@ 
 function bigbluebuttonbn_get_coursemodule_info($coursemodule) {
     global $CFG, $DB;
 
-    if ( !$bigbluebuttonbn = $DB->get_record('bigbluebuttonbn', array('id'=>$coursemodule->instance), 'id, name, intro, introformat, newwindow')) {
+    if (!$bigbluebuttonbn = $DB->get_record('bigbluebuttonbn', array('id'=>$coursemodule->instance), 'id, name, intro, introformat, newwindow')) {
         return NULL;
     }
 
@@ -341,10 +341,10 @@ 
 function bigbluebuttonbn_process_pre_save(&$bigbluebuttonbn) {
     global $DB, $CFG;
 
-    if ( !isset($bigbluebuttonbn->timecreated) || !$bigbluebuttonbn->timecreated ) {
+    if (!isset($bigbluebuttonbn->timecreated) || !$bigbluebuttonbn->timecreated) {
         $bigbluebuttonbn->timecreated = time();
         //Assign password only if it is a new activity
-        if( isset($bigbluebuttonbn->add) && !empty($bigbluebuttonbn->add) ) {
+        if (isset($bigbluebuttonbn->add) && !empty($bigbluebuttonbn->add)) {
             $bigbluebuttonbn->moderatorpass = bigbluebuttonbn_random_password(12);
             $bigbluebuttonbn->viewerpass = bigbluebuttonbn_random_password(12);
         }
@@ -353,13 +353,13 @@ 
         $bigbluebuttonbn->timemodified = time();
     }
 
-    if (! isset($bigbluebuttonbn->newwindow))
+    if (!isset($bigbluebuttonbn->newwindow))
         $bigbluebuttonbn->newwindow = 0;
-    if (! isset($bigbluebuttonbn->wait))
+    if (!isset($bigbluebuttonbn->wait))
         $bigbluebuttonbn->wait = 0;
-    if (! isset($bigbluebuttonbn->record))
+    if (!isset($bigbluebuttonbn->record))
         $bigbluebuttonbn->record = 0;
-    if (! isset($bigbluebuttonbn->tagging))
+    if (!isset($bigbluebuttonbn->tagging))
         $bigbluebuttonbn->tagging = 0;
 
     $bigbluebuttonbn->participants = htmlspecialchars_decode($bigbluebuttonbn->participants);
@@ -378,8 +378,8 @@ 
 
     // Now that an id was assigned, generate and set the meetingid property based on 
     // [Moodle Instance + Activity ID + BBB Secret] (but only for new activities)
-    if( isset($bigbluebuttonbn->add) && !empty($bigbluebuttonbn->add) ) {
-        $bigbluebuttonbn_meetingid = sha1($CFG->wwwroot.$bigbluebuttonbn->id.bigbluebuttonbn_get_cfg_shared_secret());
+    if (isset($bigbluebuttonbn->add) && !empty($bigbluebuttonbn->add)) {
+        $bigbluebuttonbn_meetingid = sha1($CFG->wwwroot . $bigbluebuttonbn->id . bigbluebuttonbn_get_cfg_shared_secret());
         $DB->set_field('bigbluebuttonbn', 'meetingid', $bigbluebuttonbn_meetingid, array('id' => $bigbluebuttonbn->id));
         $action = get_string('mod_form_field_notification_msg_created', 'bigbluebuttonbn');
     } else {
@@ -388,7 +388,7 @@ 
     $at = get_string('mod_form_field_notification_msg_at', 'bigbluebuttonbn');
 
     // Add evento to the calendar when if openingtime is set
-    if ( isset($bigbluebuttonbn->openingtime) && $bigbluebuttonbn->openingtime ){
+    if (isset($bigbluebuttonbn->openingtime) && $bigbluebuttonbn->openingtime) {
         $event = new stdClass();
         $event->name        = $bigbluebuttonbn->name;
         $event->courseid    = $bigbluebuttonbn->course;
@@ -398,13 +398,13 @@ 
         $event->instance    = $bigbluebuttonbn->id;
         $event->timestart   = $bigbluebuttonbn->openingtime;
 
-        if ( $bigbluebuttonbn->closingtime ){
+        if ($bigbluebuttonbn->closingtime) {
             $event->durationtime = $bigbluebuttonbn->closingtime - $bigbluebuttonbn->openingtime;
         } else {
             $event->durationtime = 0;
         }
 
-        if ( $event->id = $DB->get_field('event', 'id', array('modulename'=>'bigbluebuttonbn', 'instance'=>$bigbluebuttonbn->id)) ) {
+        if ($event->id = $DB->get_field('event', 'id', array('modulename'=>'bigbluebuttonbn', 'instance'=>$bigbluebuttonbn->id))) {
             $calendarevent = calendar_event::load($event->id);
             $calendarevent->update($event);
         } else {
@@ -415,45 +415,45 @@ 
         $DB->delete_records('event', array('modulename'=>'bigbluebuttonbn', 'instance'=>$bigbluebuttonbn->id));
     }
 
-    if( isset($bigbluebuttonbn->notification) && $bigbluebuttonbn->notification ) {
+    if (isset($bigbluebuttonbn->notification) && $bigbluebuttonbn->notification) {
         // Prepare message
         $msg = new stdClass();
 
         /// Build the message_body
         $msg->action = $action;
         $msg->activity_type = "";
-        if( isset($bigbluebuttonbn->type) && $bigbluebuttonbn->type != 0 )
+        if (isset($bigbluebuttonbn->type) && $bigbluebuttonbn->type != 0)
             $msg->activity_type = bigbluebuttonbn_get_predefinedprofile_name($bigbluebuttonbn->type);
         $msg->activity_title = $bigbluebuttonbn->name;
-        $message_text = '<p>'.$msg->activity_type.' &quot;'.$msg->activity_title.'&quot; '.get_string('email_body_notification_meeting_has_been', 'bigbluebuttonbn').' '.$msg->action.'.</p>';
+        $message_text = '<p>' . $msg->activity_type . ' &quot;' . $msg->activity_title . '&quot; ' . get_string('email_body_notification_meeting_has_been', 'bigbluebuttonbn') . ' ' . $msg->action . '.</p>';
 
         /// Add the meeting details to the message_body
         $msg->action = ucfirst($action);
         $msg->activity_description = "";
-        if( !empty($bigbluebuttonbn->intro) )
+        if (!empty($bigbluebuttonbn->intro))
             $msg->activity_description = trim($bigbluebuttonbn->intro);
         $msg->activity_openingtime = "";
         if ($bigbluebuttonbn->openingtime) {
-            $msg->activity_openingtime = calendar_day_representation($bigbluebuttonbn->openingtime).' '.$at.' '.calendar_time_representation($bigbluebuttonbn->openingtime);
+            $msg->activity_openingtime = calendar_day_representation($bigbluebuttonbn->openingtime) . ' ' . $at . ' ' . calendar_time_representation($bigbluebuttonbn->openingtime);
         }
         $msg->activity_closingtime = "";
-        if ($bigbluebuttonbn->closingtime ) {
-            $msg->activity_closingtime = calendar_day_representation($bigbluebuttonbn->closingtime).' '.$at.' '.calendar_time_representation($bigbluebuttonbn->closingtime);
+        if ($bigbluebuttonbn->closingtime) {
+            $msg->activity_closingtime = calendar_day_representation($bigbluebuttonbn->closingtime) . ' ' . $at . ' ' . calendar_time_representation($bigbluebuttonbn->closingtime);
         }
-        $msg->activity_owner = $USER->firstname.' '.$USER->lastname;
+        $msg->activity_owner = $USER->firstname . ' ' . $USER->lastname;
 
-        $message_text .= '<p><b>'.$msg->activity_title.'</b> '.get_string('email_body_notification_meeting_details', 'bigbluebuttonbn').':';
+        $message_text .= '<p><b>' . $msg->activity_title . '</b> ' . get_string('email_body_notification_meeting_details', 'bigbluebuttonbn') . ':';
         $message_text .= '<table border="0" style="margin: 5px 0 0 20px"><tbody>';
-        $message_text .= '<tr><td style="font-weight:bold;color:#555;">'.get_string('email_body_notification_meeting_title', 'bigbluebuttonbn').': </td><td>';
-        $message_text .= $msg->activity_title.'</td></tr>';
-        $message_text .= '<tr><td style="font-weight:bold;color:#555;">'.get_string('email_body_notification_meeting_description', 'bigbluebuttonbn').': </td><td>';
-        $message_text .= $msg->activity_description.'</td></tr>';
-        $message_text .= '<tr><td style="font-weight:bold;color:#555;">'.get_string('email_body_notification_meeting_start_date', 'bigbluebuttonbn').': </td><td>';
-        $message_text .= $msg->activity_openingtime.'</td></tr>';
-        $message_text .= '<tr><td style="font-weight:bold;color:#555;">'.get_string('email_body_notification_meeting_end_date', 'bigbluebuttonbn').': </td><td>';
-        $message_text .= $msg->activity_closingtime.'</td></tr>';
-        $message_text .= '<tr><td style="font-weight:bold;color:#555;">'.$msg->action.' '.get_string('email_body_notification_meeting_by', 'bigbluebuttonbn').': </td><td>';
-        $message_text .= $msg->activity_owner.'</td></tr></tbody></table></p>';
+        $message_text .= '<tr><td style="font-weight:bold;color:#555;">' . get_string('email_body_notification_meeting_title', 'bigbluebuttonbn') . ': </td><td>';
+        $message_text .= $msg->activity_title . '</td></tr>';
+        $message_text .= '<tr><td style="font-weight:bold;color:#555;">' . get_string('email_body_notification_meeting_description', 'bigbluebuttonbn') . ': </td><td>';
+        $message_text .= $msg->activity_description . '</td></tr>';
+        $message_text .= '<tr><td style="font-weight:bold;color:#555;">' . get_string('email_body_notification_meeting_start_date', 'bigbluebuttonbn') . ': </td><td>';
+        $message_text .= $msg->activity_openingtime . '</td></tr>';
+        $message_text .= '<tr><td style="font-weight:bold;color:#555;">' . get_string('email_body_notification_meeting_end_date', 'bigbluebuttonbn') . ': </td><td>';
+        $message_text .= $msg->activity_closingtime . '</td></tr>';
+        $message_text .= '<tr><td style="font-weight:bold;color:#555;">' . $msg->action . ' ' . get_string('email_body_notification_meeting_by', 'bigbluebuttonbn') . ': </td><td>';
+        $message_text .= $msg->activity_owner . '</td></tr></tbody></table></p>';
 
         // Send notification to all users enrolled
         bigbluebuttonbn_send_notification($USER, $bigbluebuttonbn, $message_text);
@@ -504,7 +504,7 @@ 
  * @param array $options additional options affecting the file serving
  * @return bool false if file not found, does not return if found - justsend the file
  */
-function bigbluebuttonbn_pluginfile($course, $cm, $context, $filearea, $args, $forcedownload, array $options=array()) {
+function bigbluebuttonbn_pluginfile($course, $cm, $context, $filearea, $args, $forcedownload, array $options = array()) {
     global $CFG, $DB;
 
     if ($context->contextlevel != CONTEXT_MODULE) {
@@ -520,21 +520,21 @@ 
         return false;
     }
 
-    if( sizeof($args) > 1 ) {
+    if (sizeof($args) > 1) {
         $cache = cache::make_from_params(cache_store::MODE_APPLICATION, 'mod_bigbluebuttonbn', 'presentation_cache');
         $presentation_nonce_key = sha1($bigbluebuttonbn->id);
         $presentation_nonce = $cache->get($presentation_nonce_key);
         $presentation_nonce_value = $presentation_nonce['value'];
         $presentation_nonce_counter = $presentation_nonce['counter'];
 
-        if( $args["0"] != $presentation_nonce_value ) {
+        if ($args["0"] != $presentation_nonce_value) {
             return false;
         }
 
         //The nonce value is actually used twice because BigBlueButton reads the file two times
         $presentation_nonce_counter += 1;
-        if( $presentation_nonce_counter < 2 ) {
-            $cache->set($presentation_nonce_key, array( "value" => $presentation_nonce_value, "counter" => $presentation_nonce_counter ));
+        if ($presentation_nonce_counter < 2) {
+            $cache->set($presentation_nonce_key, array("value" => $presentation_nonce_value, "counter" => $presentation_nonce_counter));
         } else {
             $cache->delete($presentation_nonce_key);
         }
@@ -552,7 +552,7 @@ 
     }
 
     if ($filearea === 'presentation') {
-        $fullpath = "/$context->id/mod_bigbluebuttonbn/$filearea/0/".$filename;
+        $fullpath = "/$context->id/mod_bigbluebuttonbn/$filearea/0/" . $filename;
     } else {
         return false;
     }
@@ -586,10 +586,10 @@ 
  * @package  mod_bigbluebuttonbn
  * @return array a list of available roles
  */
-function bigbluebuttonbn_get_db_moodle_roles($rolename='all') {
+function bigbluebuttonbn_get_db_moodle_roles($rolename = 'all') {
     global $DB;
 
-    if( $rolename != 'all')
+    if ($rolename != 'all')
         $roles = $DB->get_record('role', array('shortname' => $rolename));
     else
         $roles = $DB->get_records('role', array());
@@ -608,22 +608,22 @@ 
 
     $roles = bigbluebuttonbn_get_db_moodle_roles();
     $sqluserids = array();
-    foreach($roles as $role){
+    foreach ($roles as $role) {
         $users = get_role_users($role->id, $context);
-        foreach($users as $user) {
+        foreach ($users as $user) {
             array_push($sqluserids, $user->id);
         }
     }
 
     $users_array = array();
-    if( !empty($sqluserids) ) {
+    if (!empty($sqluserids)) {
         $users_array = $DB->get_records_select("user", "id IN (" . implode(', ', $sqluserids) . ") AND deleted = 0");
     }
 
     return $users_array;
 }
 
-function bigbluebuttonbn_send_notification($sender, $bigbluebuttonbn, $message="") {
+function bigbluebuttonbn_send_notification($sender, $bigbluebuttonbn, $message = "") {
     global $CFG, $DB;
 
     $context = bigbluebuttonbn_get_context_course($bigbluebuttonbn->course);
@@ -631,19 +631,19 @@ 
 
     //Complete message
     $msg = new stdClass();
-    $msg->user_name = $sender->firstname.' '.$sender->lastname;
+    $msg->user_name = $sender->firstname . ' ' . $sender->lastname;
     $msg->user_email = $sender->email;
     $msg->course_name = "$course->fullname";
-    $message .= '<p><hr/><br/>'.get_string('email_footer_sent_by', 'bigbluebuttonbn').' '.$msg->user_name.'('.$msg->user_email.') ';
-    $message .= get_string('email_footer_sent_from', 'bigbluebuttonbn').' '.$msg->course_name.'.</p>';
+    $message .= '<p><hr/><br/>' . get_string('email_footer_sent_by', 'bigbluebuttonbn') . ' ' . $msg->user_name . '(' . $msg->user_email . ') ';
+    $message .= get_string('email_footer_sent_from', 'bigbluebuttonbn') . ' ' . $msg->course_name . '.</p>';
     
     $users = bigbluebuttonbn_get_users($context);
-    foreach( $users as $user ) {
-        if( $user->id != $sender->id ){
-            error_log("Sending msg to ".$user->firstname." ".$user->lastname.".");
+    foreach ($users as $user) {
+        if ($user->id != $sender->id) {
+            error_log("Sending msg to " . $user->firstname . " " . $user->lastname . ".");
             $messageid = message_post_message($sender, $user, $message, FORMAT_HTML);
             if (!empty($messageid)) {
-                error_log("Msg sent to ".$user->firstname." ".$user->lastname.".");
+                error_log("Msg sent to " . $user->firstname . " " . $user->lastname . ".");
             } else {
                 error_log("Msg was NOT sent.");
             }
@@ -655,7 +655,7 @@ 
     global $CFG;
 
     $version_major = bigbluebuttonbn_get_moodle_version_major();
-    if ( $version_major < '2013111800' ) {
+    if ($version_major < '2013111800') {
         //This is valid before v2.6
         $context = get_context_instance(CONTEXT_MODULE, $id);
     } else {
@@ -670,7 +670,7 @@ 
     global $CFG;
 
     $version_major = bigbluebuttonbn_get_moodle_version_major();
-    if ( $version_major < '2013111800' ) {
+    if ($version_major < '2013111800') {
         //This is valid before v2.6
         $context = get_context_instance(CONTEXT_COURSE, $id);
     } else {
@@ -683,11 +683,11 @@ 
 
 function bigbluebuttonbn_get_cfg_server_url() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_server_url)? trim(trim($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_server_url),'/').'/': (isset($CFG->bigbluebuttonbn_server_url)? trim(trim($CFG->bigbluebuttonbn_server_url),'/').'/': 'http://test-install.blindsidenetworks.com/bigbluebutton/'));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_server_url) ? trim(trim($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_server_url), '/') . '/' : (isset($CFG->bigbluebuttonbn_server_url) ? trim(trim($CFG->bigbluebuttonbn_server_url), '/') . '/' : 'http://test-install.blindsidenetworks.com/bigbluebutton/'));
 }
 
 function bigbluebuttonbn_get_cfg_shared_secret() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_shared_secret)? trim($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_shared_secret): (isset($CFG->bigbluebuttonbn_shared_secret)? trim($CFG->bigbluebuttonbn_shared_secret): '8cd8ef52e8e101574e400365b55e11a6'));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_shared_secret) ? trim($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_shared_secret) : (isset($CFG->bigbluebuttonbn_shared_secret) ? trim($CFG->bigbluebuttonbn_shared_secret) : '8cd8ef52e8e101574e400365b55e11a6'));
 }
 

Braces +23 added lines, -16 removed lines

@@ -353,14 +353,18 @@ 
         $bigbluebuttonbn->timemodified = time();
     }
 
-    if (! isset($bigbluebuttonbn->newwindow))
-        $bigbluebuttonbn->newwindow = 0;
-    if (! isset($bigbluebuttonbn->wait))
-        $bigbluebuttonbn->wait = 0;
-    if (! isset($bigbluebuttonbn->record))
-        $bigbluebuttonbn->record = 0;
-    if (! isset($bigbluebuttonbn->tagging))
-        $bigbluebuttonbn->tagging = 0;
+    if (! isset($bigbluebuttonbn->newwindow)) {
+            $bigbluebuttonbn->newwindow = 0;
+    }
+    if (! isset($bigbluebuttonbn->wait)) {
+            $bigbluebuttonbn->wait = 0;
+    }
+    if (! isset($bigbluebuttonbn->record)) {
+            $bigbluebuttonbn->record = 0;
+    }
+    if (! isset($bigbluebuttonbn->tagging)) {
+            $bigbluebuttonbn->tagging = 0;
+    }
 
     $bigbluebuttonbn->participants = htmlspecialchars_decode($bigbluebuttonbn->participants);
 }
@@ -422,16 +426,18 @@ 
         /// Build the message_body
         $msg->action = $action;
         $msg->activity_type = "";
-        if( isset($bigbluebuttonbn->type) && $bigbluebuttonbn->type != 0 )
-            $msg->activity_type = bigbluebuttonbn_get_predefinedprofile_name($bigbluebuttonbn->type);
+        if( isset($bigbluebuttonbn->type) && $bigbluebuttonbn->type != 0 ) {
+                    $msg->activity_type = bigbluebuttonbn_get_predefinedprofile_name($bigbluebuttonbn->type);
+        }
         $msg->activity_title = $bigbluebuttonbn->name;
         $message_text = '<p>'.$msg->activity_type.' &quot;'.$msg->activity_title.'&quot; '.get_string('email_body_notification_meeting_has_been', 'bigbluebuttonbn').' '.$msg->action.'.</p>';
 
         /// Add the meeting details to the message_body
         $msg->action = ucfirst($action);
         $msg->activity_description = "";
-        if( !empty($bigbluebuttonbn->intro) )
-            $msg->activity_description = trim($bigbluebuttonbn->intro);
+        if( !empty($bigbluebuttonbn->intro) ) {
+                    $msg->activity_description = trim($bigbluebuttonbn->intro);
+        }
         $msg->activity_openingtime = "";
         if ($bigbluebuttonbn->openingtime) {
             $msg->activity_openingtime = calendar_day_representation($bigbluebuttonbn->openingtime).' '.$at.' '.calendar_time_representation($bigbluebuttonbn->openingtime);
@@ -589,10 +595,11 @@ 
 function bigbluebuttonbn_get_db_moodle_roles($rolename='all') {
     global $DB;
 
-    if( $rolename != 'all')
-        $roles = $DB->get_record('role', array('shortname' => $rolename));
-    else
-        $roles = $DB->get_records('role', array());
+    if( $rolename != 'all') {
+            $roles = $DB->get_record('role', array('shortname' => $rolename));
+    } else {
+            $roles = $DB->get_records('role', array());
+    }
 
     return $roles;
 }

locallib.php

Doc Comments +31 added lines

@@ -36,6 +36,9 @@ 
 const BIGBLUEBUTTONBN_LOG_EVENT_JOIN = "Join";
 const BIGBLUEBUTTONBN_LOG_EVENT_LOGOUT = "Logout";
 
+/**
+ * @param string $event
+ */
 function bigbluebuttonbn_log(array $bbbsession, $event) {
     global $DB;
 
@@ -181,6 +184,9 @@ 
     return $xml;
 }
 
+/**
+ * @param string $meetingID
+ */
 function bigbluebuttonbn_getMeetingInfoArray( $meetingID, $modPW, $URL, $SALT ) {
     $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getMeetingInfoURL( $meetingID, $modPW, $URL, $SALT ) );
 
@@ -285,6 +291,10 @@ 
     else return 1;
 }
 
+/**
+ * @param string $URL
+ * @param string $SALT
+ */
 function bigbluebuttonbn_doDeleteRecordings( $recordIDs, $URL, $SALT ) {
     $ids = 	explode(",", $recordIDs);
     foreach( $ids as $id){
@@ -295,6 +305,11 @@ 
     return true;
 }
 
+/**
+ * @param string $set
+ * @param string $URL
+ * @param string $SALT
+ */
 function bigbluebuttonbn_doPublishRecordings( $recordIDs, $set, $URL, $SALT ) {
     $ids = 	explode(",", $recordIDs);
     foreach( $ids as $id){
@@ -305,6 +320,10 @@ 
     return true;
 }
 
+/**
+ * @param string $URL
+ * @param string $SALT
+ */
 function bigbluebuttonbn_doEndMeeting( $meetingID, $modPW, $URL, $SALT ) {
     $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getEndMeetingURL( $meetingID, $modPW, $URL, $SALT ) );
 
@@ -326,6 +345,9 @@ 
 }
 
 
+/**
+ * @param string $URL
+ */
 function bigbluebuttonbn_getServerVersion( $URL ){
     $xml = bigbluebuttonbn_wrap_xml_load_file( $URL."api" );
     if ( $xml && $xml->returncode == 'SUCCESS' ) {
@@ -344,6 +366,9 @@ 
     }
 }
 
+/**
+ * @param string $url
+ */
 function bigbluebuttonbn_wrap_xml_load_file($url, $method=BIGBLUEBUTTONBN_METHOD_GET, $data=null) {
     if (extension_loaded('curl')) {
         $c = new curl();
@@ -776,6 +801,9 @@ 
     $cache = cache::make_from_params(cache_store::MODE_APPLICATION, 'mod_bigbluebuttonbn', 'meetings_cache');
 }
 
+/**
+ * @param boolean $is_moderator
+ */
 function bigbluebuttonbn_bbb_broker_participant_joined($meetingid, $is_moderator) {
     $cache = cache::make_from_params(cache_store::MODE_APPLICATION, 'mod_bigbluebuttonbn', 'meetings_cache');
     $result = $cache->get($meetingid);
@@ -1256,6 +1284,9 @@ 
     return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recordingready_enabled)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recordingready_enabled: (isset($CFG->bigbluebuttonbn_recordingready_enabled)? $CFG->bigbluebuttonbn_recordingready_enabled: false));
 }
 
+/**
+ * @return string
+ */
 function bigbluebuttonbn_get_cfg_moderator_default() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
     return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_moderator_default)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_moderator_default: (isset($CFG->bigbluebuttonbn_moderator_default)? $CFG->bigbluebuttonbn_moderator_default: 'owner'));

Indentation +2 added lines, -2 removed lines

@@ -53,9 +53,9 @@
     $returnid = $DB->insert_record('bigbluebuttonbn_log', $log);
 }
 
- ////////////////////////////
+    ////////////////////////////
 //  BigBlueButton API Calls  //
- ////////////////////////////
+    ////////////////////////////
 function bigbluebuttonbn_getJoinURL( $meetingID, $userName, $PW, $SALT, $URL ) {
     $url_join = $URL."api/join?";
     $params = 'meetingID='.urlencode($meetingID).'&fullName='.urlencode($userName).'&password='.urlencode($PW);

Spacing +277 added lines, -277 removed lines

@@ -14,7 +14,7 @@ 
 
 global $BIGBLUEBUTTONBN_CFG, $CFG;
 
-require_once(dirname(__FILE__).'/lib.php');
+require_once(dirname(__FILE__) . '/lib.php');
 
 const BIGBLUEBUTTONBN_FORCED = true;
 
@@ -47,8 +47,8 @@ 
     $log->userid = $bbbsession['userID'];
     $log->timecreated = time();
     $log->event = $event;
-    if( $event == BIGBLUEBUTTONBN_LOG_EVENT_CREATE)
-        $log->meta = '{"record":'.($bbbsession['record']? 'true': 'false').'}';
+    if ($event == BIGBLUEBUTTONBN_LOG_EVENT_CREATE)
+        $log->meta = '{"record":' . ($bbbsession['record'] ? 'true' : 'false') . '}';
 
     $returnid = $DB->insert_record('bigbluebuttonbn_log', $log);
 }
@@ -56,119 +56,119 @@ 
  ////////////////////////////
 //  BigBlueButton API Calls  //
  ////////////////////////////
-function bigbluebuttonbn_getJoinURL( $meetingID, $userName, $PW, $SALT, $URL ) {
-    $url_join = $URL."api/join?";
-    $params = 'meetingID='.urlencode($meetingID).'&fullName='.urlencode($userName).'&password='.urlencode($PW);
-    return ($url_join.$params.'&checksum='.sha1("join".$params.$SALT) );
+function bigbluebuttonbn_getJoinURL($meetingID, $userName, $PW, $SALT, $URL) {
+    $url_join = $URL . "api/join?";
+    $params = 'meetingID=' . urlencode($meetingID) . '&fullName=' . urlencode($userName) . '&password=' . urlencode($PW);
+    return ($url_join . $params . '&checksum=' . sha1("join" . $params . $SALT));
 }
 
-function bigbluebuttonbn_getCreateMeetingURL($name, $meetingID, $attendeePW, $moderatorPW, $welcome, $logoutURL, $SALT, $URL, $record = 'false', $duration=0, $voiceBridge=0, $metadata = array() ) {
-    $url_create = $URL."api/create?";
+function bigbluebuttonbn_getCreateMeetingURL($name, $meetingID, $attendeePW, $moderatorPW, $welcome, $logoutURL, $SALT, $URL, $record = 'false', $duration = 0, $voiceBridge = 0, $metadata = array()) {
+    $url_create = $URL . "api/create?";
 
-    $params = 'name='.urlencode($name).'&meetingID='.urlencode($meetingID).'&attendeePW='.urlencode($attendeePW).'&moderatorPW='.urlencode($moderatorPW).'&logoutURL='.urlencode($logoutURL).'&record='.$record;
+    $params = 'name=' . urlencode($name) . '&meetingID=' . urlencode($meetingID) . '&attendeePW=' . urlencode($attendeePW) . '&moderatorPW=' . urlencode($moderatorPW) . '&logoutURL=' . urlencode($logoutURL) . '&record=' . $record;
 
     $voiceBridge = intval($voiceBridge);
-    if ( $voiceBridge > 0 && $voiceBridge < 79999)
-        $params .= '&voiceBridge='.$voiceBridge;
+    if ($voiceBridge > 0 && $voiceBridge < 79999)
+        $params .= '&voiceBridge=' . $voiceBridge;
 
     $duration = intval($duration);
-    if( $duration > 0 )
-        $params .= '&duration='.$duration;
+    if ($duration > 0)
+        $params .= '&duration=' . $duration;
 
-    if( trim( $welcome ) )
-        $params .= '&welcome='.urlencode($welcome);
+    if (trim($welcome))
+        $params .= '&welcome=' . urlencode($welcome);
 
     foreach ($metadata as $key => $value) {
-        $params .= '&'.$key.'='.urlencode($value);
+        $params .= '&' . $key . '=' . urlencode($value);
     }
 
-    return ( $url_create.$params.'&checksum='.sha1("create".$params.$SALT) );
+    return ($url_create . $params . '&checksum=' . sha1("create" . $params . $SALT));
 }
 
-function bigbluebuttonbn_getIsMeetingRunningURL( $meetingID, $URL, $SALT ) {
-    $base_url = $URL."api/isMeetingRunning?";
-    $params = 'meetingID='.urlencode($meetingID);
-    return ($base_url.$params.'&checksum='.sha1("isMeetingRunning".$params.$SALT) );
+function bigbluebuttonbn_getIsMeetingRunningURL($meetingID, $URL, $SALT) {
+    $base_url = $URL . "api/isMeetingRunning?";
+    $params = 'meetingID=' . urlencode($meetingID);
+    return ($base_url . $params . '&checksum=' . sha1("isMeetingRunning" . $params . $SALT));
 }
 
-function bigbluebuttonbn_getMeetingInfoURL( $meetingID, $modPW, $URL, $SALT ) {
-    $base_url = $URL."api/getMeetingInfo?";
-    $params = 'meetingID='.urlencode($meetingID).'&password='.urlencode($modPW);
-    return ( $base_url.$params.'&checksum='.sha1("getMeetingInfo".$params.$SALT));
+function bigbluebuttonbn_getMeetingInfoURL($meetingID, $modPW, $URL, $SALT) {
+    $base_url = $URL . "api/getMeetingInfo?";
+    $params = 'meetingID=' . urlencode($meetingID) . '&password=' . urlencode($modPW);
+    return ($base_url . $params . '&checksum=' . sha1("getMeetingInfo" . $params . $SALT));
 }
 
 function bigbluebuttonbn_getMeetingsURL($URL, $SALT) {
-    $base_url = $URL."api/getMeetings?";
+    $base_url = $URL . "api/getMeetings?";
     $params = '';
-    return ( $base_url.$params.'&checksum='.sha1("getMeetings".$params.$SALT));
+    return ($base_url . $params . '&checksum=' . sha1("getMeetings" . $params . $SALT));
 }
 
-function bigbluebuttonbn_getEndMeetingURL( $meetingID, $modPW, $URL, $SALT ) {
-    $base_url = $URL."api/end?";
-    $params = 'meetingID='.urlencode($meetingID).'&password='.urlencode($modPW);
-    return ( $base_url.$params.'&checksum='.sha1("end".$params.$SALT) );
+function bigbluebuttonbn_getEndMeetingURL($meetingID, $modPW, $URL, $SALT) {
+    $base_url = $URL . "api/end?";
+    $params = 'meetingID=' . urlencode($meetingID) . '&password=' . urlencode($modPW);
+    return ($base_url . $params . '&checksum=' . sha1("end" . $params . $SALT));
 }
 
-function bigbluebuttonbn_getRecordingsURL( $URL, $SALT, $meetingID=null ) {
-    $base_url_record = $URL."api/getRecordings?";
-    if( $meetingID == null ) {
+function bigbluebuttonbn_getRecordingsURL($URL, $SALT, $meetingID = null) {
+    $base_url_record = $URL . "api/getRecordings?";
+    if ($meetingID == null) {
         $params = "";
     } else {
-        $params = "meetingID=".urlencode($meetingID);
+        $params = "meetingID=" . urlencode($meetingID);
     }
-    return ($base_url_record.$params."&checksum=".sha1("getRecordings".$params.$SALT) );
+    return ($base_url_record . $params . "&checksum=" . sha1("getRecordings" . $params . $SALT));
 }
 
-function bigbluebuttonbn_getDeleteRecordingsURL( $recordID, $URL, $SALT ) {
-    $url_delete = $URL."api/deleteRecordings?";
-    $params = 'recordID='.urlencode($recordID);
-    return ($url_delete.$params.'&checksum='.sha1("deleteRecordings".$params.$SALT) );
+function bigbluebuttonbn_getDeleteRecordingsURL($recordID, $URL, $SALT) {
+    $url_delete = $URL . "api/deleteRecordings?";
+    $params = 'recordID=' . urlencode($recordID);
+    return ($url_delete . $params . '&checksum=' . sha1("deleteRecordings" . $params . $SALT));
 }
 
-function bigbluebuttonbn_getPublishRecordingsURL( $recordID, $set, $URL, $SALT ) {
-    $url_publish = $URL."api/publishRecordings?";
-    $params = 'recordID='.$recordID."&publish=".$set;
-    return ($url_publish.$params.'&checksum='.sha1("publishRecordings".$params.$SALT) );
+function bigbluebuttonbn_getPublishRecordingsURL($recordID, $set, $URL, $SALT) {
+    $url_publish = $URL . "api/publishRecordings?";
+    $params = 'recordID=' . $recordID . "&publish=" . $set;
+    return ($url_publish . $params . '&checksum=' . sha1("publishRecordings" . $params . $SALT));
 }
 
 function bigbluebuttonbn_getCapabilitiesURL($URL, $SALT) {
-    $base_url = $URL."api/getCapabilities?";
+    $base_url = $URL . "api/getCapabilities?";
     $params = '';
-    return ( $base_url.$params.'&checksum='.sha1("getCapabilities".$params.$SALT));
+    return ($base_url . $params . '&checksum=' . sha1("getCapabilities" . $params . $SALT));
 }
 
-function bigbluebuttonbn_getCreateMeetingArray( $username, $meetingID, $welcomeString, $mPW, $aPW, $SALT, $URL, $logoutURL, $record='false', $duration=0, $voiceBridge=0, $metadata=array(), $presentation_name=null, $presentation_url=null ) {
-    if( !is_null($presentation_name) && !is_null($presentation_url) ) {
-        $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getCreateMeetingURL($username, $meetingID, $aPW, $mPW, $welcomeString, $logoutURL, $SALT, $URL, $record, $duration, $voiceBridge, $metadata),
+function bigbluebuttonbn_getCreateMeetingArray($username, $meetingID, $welcomeString, $mPW, $aPW, $SALT, $URL, $logoutURL, $record = 'false', $duration = 0, $voiceBridge = 0, $metadata = array(), $presentation_name = null, $presentation_url = null) {
+    if (!is_null($presentation_name) && !is_null($presentation_url)) {
+        $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getCreateMeetingURL($username, $meetingID, $aPW, $mPW, $welcomeString, $logoutURL, $SALT, $URL, $record, $duration, $voiceBridge, $metadata),
                 BIGBLUEBUTTONBN_METHOD_POST,
-                "<?xml version='1.0' encoding='UTF-8'?><modules><module name='presentation'><document url='".$presentation_url."' /></module></modules>"
+                "<?xml version='1.0' encoding='UTF-8'?><modules><module name='presentation'><document url='" . $presentation_url . "' /></module></modules>"
                 );
     } else {
-        $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getCreateMeetingURL($username, $meetingID, $aPW, $mPW, $welcomeString, $logoutURL, $SALT, $URL, $record, $duration, $voiceBridge, $metadata) );
+        $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getCreateMeetingURL($username, $meetingID, $aPW, $mPW, $welcomeString, $logoutURL, $SALT, $URL, $record, $duration, $voiceBridge, $metadata));
     }
 
-    if( $xml ) {
-        if($xml->meetingID) return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey, 'meetingID' => $xml->meetingID, 'attendeePW' => $xml->attendeePW, 'moderatorPW' => $xml->moderatorPW, 'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded );
-        else return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey );
+    if ($xml) {
+        if ($xml->meetingID) return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey, 'meetingID' => $xml->meetingID, 'attendeePW' => $xml->attendeePW, 'moderatorPW' => $xml->moderatorPW, 'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded);
+        else return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey);
     }
     else {
         return null;
     }
 }
 
-function bigbluebuttonbn_getMeetingsArray( $URL, $SALT ) {
-    $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getMeetingsURL( $URL, $SALT ) );
+function bigbluebuttonbn_getMeetingsArray($URL, $SALT) {
+    $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getMeetingsURL($URL, $SALT));
 
-    if( $xml && $xml->returncode == 'SUCCESS' && $xml->messageKey ) {    //The meetings were returned
+    if ($xml && $xml->returncode == 'SUCCESS' && $xml->messageKey) {    //The meetings were returned
         return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey);
 
-    } else if($xml && $xml->returncode == 'SUCCESS'){                    //If there were meetings already created
+    } else if ($xml && $xml->returncode == 'SUCCESS') {                    //If there were meetings already created
         foreach ($xml->meetings->meeting as $meeting) {
-            $meetings[] = array( 'meetingID' => $meeting->meetingID, 'moderatorPW' => $meeting->moderatorPW, 'attendeePW' => $meeting->attendeePW, 'hasBeenForciblyEnded' => $meeting->hasBeenForciblyEnded, 'running' => $meeting->running );
+            $meetings[] = array('meetingID' => $meeting->meetingID, 'moderatorPW' => $meeting->moderatorPW, 'attendeePW' => $meeting->attendeePW, 'hasBeenForciblyEnded' => $meeting->hasBeenForciblyEnded, 'running' => $meeting->running);
         }
         return $meetings;
 
-    } else if( $xml ) { //If the xml packet returned failure it displays the message to the user
+    } else if ($xml) { //If the xml packet returned failure it displays the message to the user
         return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey);
 
     } else { //If the server is unreachable, then prompts the user of the necessary action
@@ -176,21 +176,21 @@ 
     }
 }
 
-function bigbluebuttonbn_getMeetingInfo( $meetingID, $modPW, $URL, $SALT ) {
-    $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getMeetingInfoURL( $meetingID, $modPW, $URL, $SALT ) );
+function bigbluebuttonbn_getMeetingInfo($meetingID, $modPW, $URL, $SALT) {
+    $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getMeetingInfoURL($meetingID, $modPW, $URL, $SALT));
     return $xml;
 }
 
-function bigbluebuttonbn_getMeetingInfoArray( $meetingID, $modPW, $URL, $SALT ) {
-    $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getMeetingInfoURL( $meetingID, $modPW, $URL, $SALT ) );
+function bigbluebuttonbn_getMeetingInfoArray($meetingID, $modPW, $URL, $SALT) {
+    $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getMeetingInfoURL($meetingID, $modPW, $URL, $SALT));
 
-    if( $xml && $xml->returncode == 'SUCCESS' && $xml->messageKey == null){//The meeting info was returned
-        return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey );
+    if ($xml && $xml->returncode == 'SUCCESS' && $xml->messageKey == null) {//The meeting info was returned
+        return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey);
 
-    } else if($xml && $xml->returncode == 'SUCCESS'){ //If there were meetings already created
-        return array('returncode' => $xml->returncode, 'meetingID' => $xml->meetingID, 'moderatorPW' => $xml->moderatorPW, 'attendeePW' => $xml->attendeePW, 'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded, 'running' => $xml->running, 'recording' => $xml->recording, 'startTime' => $xml->startTime, 'endTime' => $xml->endTime, 'participantCount' => $xml->participantCount, 'moderatorCount' => $xml->moderatorCount, 'attendees' => $xml->attendees, 'metadata' => $xml->metadata );
+    } else if ($xml && $xml->returncode == 'SUCCESS') { //If there were meetings already created
+        return array('returncode' => $xml->returncode, 'meetingID' => $xml->meetingID, 'moderatorPW' => $xml->moderatorPW, 'attendeePW' => $xml->attendeePW, 'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded, 'running' => $xml->running, 'recording' => $xml->recording, 'startTime' => $xml->startTime, 'endTime' => $xml->endTime, 'participantCount' => $xml->participantCount, 'moderatorCount' => $xml->moderatorCount, 'attendees' => $xml->attendees, 'metadata' => $xml->metadata);
 
-    } else if( ($xml && $xml->returncode == 'FAILED') || $xml) { //If the xml packet returned failure it displays the message to the user
+    } else if (($xml && $xml->returncode == 'FAILED') || $xml) { //If the xml packet returned failure it displays the message to the user
         return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey);
 
     } else { //If the server is unreachable, then prompts the user of the necessary action
@@ -198,42 +198,42 @@ 
     }
 }
 
-function bigbluebuttonbn_getRecordingsArray( $meetingIDs, $URL, $SALT ) {
+function bigbluebuttonbn_getRecordingsArray($meetingIDs, $URL, $SALT) {
     $recordings = array();
 
-    if( is_array($meetingIDs) ) {
+    if (is_array($meetingIDs)) {
         // getRecordings is executes using a method POST (supported only on BBB 1.0 and later)
-        $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getRecordingsURL( $URL, $SALT ), BIGBLUEBUTTONBN_METHOD_POST, $meetingIDs );
+        $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getRecordingsURL($URL, $SALT), BIGBLUEBUTTONBN_METHOD_POST, $meetingIDs);
     } else {
         // getRecordings is executes using a method GET supported by any version of BBB
-        $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getRecordingsURL( $URL, $SALT, $meetingIDs ) );
+        $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getRecordingsURL($URL, $SALT, $meetingIDs));
     }
 
-    if( $xml && $xml->returncode == 'SUCCESS' && $xml->messageKey ) {//The meetings were returned
-        $recordings = array('returncode' => (string) $xml->returncode, 'message' => (string) $xml->message, 'messageKey' => (string) $xml->messageKey);
+    if ($xml && $xml->returncode == 'SUCCESS' && $xml->messageKey) {//The meetings were returned
+        $recordings = array('returncode' => (string)$xml->returncode, 'message' => (string)$xml->message, 'messageKey' => (string)$xml->messageKey);
 
-    } else if($xml && $xml->returncode == 'SUCCESS' && isset($xml->recordings)){ //If there were meetings already created
+    } else if ($xml && $xml->returncode == 'SUCCESS' && isset($xml->recordings)) { //If there were meetings already created
         foreach ($xml->recordings->recording as $recording) {
             $playbackArray = array();
-            foreach ( $recording->playback->format as $format ){
-                $playbackArray[(string) $format->type] = array( 'type' => (string) $format->type, 'url' => (string) $format->url );
+            foreach ($recording->playback->format as $format) {
+                $playbackArray[(string)$format->type] = array('type' => (string)$format->type, 'url' => (string)$format->url);
             }
 
             //Add the metadata to the recordings array
             $metadataArray = array();
             $metadata = get_object_vars($recording->metadata);
             foreach ($metadata as $key => $value) {
-                if(is_object($value)) $value = '';
-                $metadataArray['meta_'.$key] = $value;
+                if (is_object($value)) $value = '';
+                $metadataArray['meta_' . $key] = $value;
             }
 
-            $recordings[] = array( 'recordID' => (string) $recording->recordID, 'meetingID' => (string) $recording->meetingID, 'meetingName' => (string) $recording->name, 'published' => (string) $recording->published, 'startTime' => (string) $recording->startTime, 'endTime' => (string) $recording->endTime, 'playbacks' => $playbackArray ) + $metadataArray;
+            $recordings[] = array('recordID' => (string)$recording->recordID, 'meetingID' => (string)$recording->meetingID, 'meetingName' => (string)$recording->name, 'published' => (string)$recording->published, 'startTime' => (string)$recording->startTime, 'endTime' => (string)$recording->endTime, 'playbacks' => $playbackArray) + $metadataArray;
         }
 
         usort($recordings, 'bigbluebuttonbn_recordingBuildSorter');
 
-    } else if( $xml ) { //If the xml packet returned failure it displays the message to the user
-        $recordings = array('returncode' => (string) $xml->returncode, 'message' => (string) $xml->message, 'messageKey' => (string) $xml->messageKey);
+    } else if ($xml) { //If the xml packet returned failure it displays the message to the user
+        $recordings = array('returncode' => (string)$xml->returncode, 'message' => (string)$xml->message, 'messageKey' => (string)$xml->messageKey);
 
     } else { //If the server is unreachable, then prompts the user of the necessary action
     }
@@ -241,37 +241,37 @@ 
     return $recordings;
 }
 
-function bigbluebuttonbn_getRecordingArray( $recordingID, $meetingID, $URL, $SALT ) {
+function bigbluebuttonbn_getRecordingArray($recordingID, $meetingID, $URL, $SALT) {
     $recording = array();
 
-    $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getRecordingsURL( $URL, $SALT, $meetingID ) );
+    $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getRecordingsURL($URL, $SALT, $meetingID));
 
-    if( $xml && $xml->returncode == 'SUCCESS' && $xml->messageKey ) {//The meetings were returned
-        $recordings = array('returncode' => (string) $xml->returncode, 'message' => (string) $xml->message, 'messageKey' => (string) $xml->messageKey);
+    if ($xml && $xml->returncode == 'SUCCESS' && $xml->messageKey) {//The meetings were returned
+        $recordings = array('returncode' => (string)$xml->returncode, 'message' => (string)$xml->message, 'messageKey' => (string)$xml->messageKey);
 
-    } else if($xml && $xml->returncode == 'SUCCESS' && isset($xml->recordings)){ //If there were meetings already created
+    } else if ($xml && $xml->returncode == 'SUCCESS' && isset($xml->recordings)) { //If there were meetings already created
         foreach ($xml->recordings->recording as $recording) {
-            if( $recording->recordID == $recordingID ) {
+            if ($recording->recordID == $recordingID) {
                 $playbackArray = array();
-                foreach ( $recording->playback->format as $format ){
-                    $playbackArray[(string) $format->type] = array( 'type' => (string) $format->type, 'url' => (string) $format->url );
+                foreach ($recording->playback->format as $format) {
+                    $playbackArray[(string)$format->type] = array('type' => (string)$format->type, 'url' => (string)$format->url);
                 }
 
                 //Add the metadata to the recordings array
                 $metadataArray = array();
                 $metadata = get_object_vars($recording->metadata);
                 foreach ($metadata as $key => $value) {
-                    if(is_object($value)) $value = '';
-                    $metadataArray['meta_'.$key] = $value;
+                    if (is_object($value)) $value = '';
+                    $metadataArray['meta_' . $key] = $value;
                 }
 
-                $recording = array( 'recordID' => (string) $recording->recordID, 'meetingID' => (string) $recording->meetingID, 'meetingName' => (string) $recording->name, 'published' => (string) $recording->published, 'startTime' => (string) $recording->startTime, 'endTime' => (string) $recording->endTime, 'playbacks' => $playbackArray ) + $metadataArray;
+                $recording = array('recordID' => (string)$recording->recordID, 'meetingID' => (string)$recording->meetingID, 'meetingName' => (string)$recording->name, 'published' => (string)$recording->published, 'startTime' => (string)$recording->startTime, 'endTime' => (string)$recording->endTime, 'playbacks' => $playbackArray) + $metadataArray;
                 break;
             }
         }
 
-    } else if( $xml ) { //If the xml packet returned failure it displays the message to the user
-        $recording = array('returncode' => (string) $xml->returncode, 'message' => (string) $xml->message, 'messageKey' => (string) $xml->messageKey);
+    } else if ($xml) { //If the xml packet returned failure it displays the message to the user
+        $recording = array('returncode' => (string)$xml->returncode, 'message' => (string)$xml->message, 'messageKey' => (string)$xml->messageKey);
 
     } else { //If the server is unreachable, then prompts the user of the necessary action
     }
@@ -279,36 +279,36 @@ 
     return $recording;
 }
 
-function bigbluebuttonbn_recordingBuildSorter($a, $b){
-    if( $a['startTime'] < $b['startTime']) return -1;
-    else if( $a['startTime'] == $b['startTime']) return 0;
+function bigbluebuttonbn_recordingBuildSorter($a, $b) {
+    if ($a['startTime'] < $b['startTime']) return -1;
+    else if ($a['startTime'] == $b['startTime']) return 0;
     else return 1;
 }
 
-function bigbluebuttonbn_doDeleteRecordings( $recordIDs, $URL, $SALT ) {
-    $ids = 	explode(",", $recordIDs);
-    foreach( $ids as $id){
-        $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getDeleteRecordingsURL($id, $URL, $SALT) );
-        if( $xml && $xml->returncode != 'SUCCESS' )
+function bigbluebuttonbn_doDeleteRecordings($recordIDs, $URL, $SALT) {
+    $ids = explode(",", $recordIDs);
+    foreach ($ids as $id) {
+        $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getDeleteRecordingsURL($id, $URL, $SALT));
+        if ($xml && $xml->returncode != 'SUCCESS')
             return false;
     }
     return true;
 }
 
-function bigbluebuttonbn_doPublishRecordings( $recordIDs, $set, $URL, $SALT ) {
-    $ids = 	explode(",", $recordIDs);
-    foreach( $ids as $id){
-        $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getPublishRecordingsURL($id, $set, $URL, $SALT) );
-        if( $xml && $xml->returncode != 'SUCCESS' )
+function bigbluebuttonbn_doPublishRecordings($recordIDs, $set, $URL, $SALT) {
+    $ids = explode(",", $recordIDs);
+    foreach ($ids as $id) {
+        $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getPublishRecordingsURL($id, $set, $URL, $SALT));
+        if ($xml && $xml->returncode != 'SUCCESS')
             return false;
     }
     return true;
 }
 
-function bigbluebuttonbn_doEndMeeting( $meetingID, $modPW, $URL, $SALT ) {
-    $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getEndMeetingURL( $meetingID, $modPW, $URL, $SALT ) );
+function bigbluebuttonbn_doEndMeeting($meetingID, $modPW, $URL, $SALT) {
+    $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getEndMeetingURL($meetingID, $modPW, $URL, $SALT));
 
-    if( $xml ) { //If the xml packet returned failure it displays the message to the user
+    if ($xml) { //If the xml packet returned failure it displays the message to the user
         return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey);
     }
     else { //If the server is unreachable, then prompts the user of the necessary action
@@ -316,44 +316,44 @@ 
     }
 }
 
-function bigbluebuttonbn_isMeetingRunning( $meetingID, $URL, $SALT ) {
-    $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getIsMeetingRunningURL( $meetingID, $URL, $SALT ) );
-    if ( $xml && $xml->returncode == 'SUCCESS' ) {
-        return ( ( $xml->running == 'true' ) ? true : false);
+function bigbluebuttonbn_isMeetingRunning($meetingID, $URL, $SALT) {
+    $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getIsMeetingRunningURL($meetingID, $URL, $SALT));
+    if ($xml && $xml->returncode == 'SUCCESS') {
+        return (($xml->running == 'true') ? true : false);
     } else {
-        return ( false );
+        return (false);
     }
 }
 
 
-function bigbluebuttonbn_getServerVersion( $URL ){
-    $xml = bigbluebuttonbn_wrap_xml_load_file( $URL."api" );
-    if ( $xml && $xml->returncode == 'SUCCESS' ) {
+function bigbluebuttonbn_getServerVersion($URL) {
+    $xml = bigbluebuttonbn_wrap_xml_load_file($URL . "api");
+    if ($xml && $xml->returncode == 'SUCCESS') {
         return $xml->version;
     } else {
         return NULL;
     }
 }
 
-function bigbluebuttonbn_getMeetingXML( $meetingID, $URL, $SALT ) {
-    $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getIsMeetingRunningURL( $meetingID, $URL, $SALT ) );
-    if ( $xml && $xml->returncode == 'SUCCESS') {
-        return ( str_replace('</response>', '', str_replace("<?xml version=\"1.0\"?>\n<response>", '', $xml->asXML())));
+function bigbluebuttonbn_getMeetingXML($meetingID, $URL, $SALT) {
+    $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getIsMeetingRunningURL($meetingID, $URL, $SALT));
+    if ($xml && $xml->returncode == 'SUCCESS') {
+        return (str_replace('</response>', '', str_replace("<?xml version=\"1.0\"?>\n<response>", '', $xml->asXML())));
     } else {
         return 'false';
     }
 }
 
-function bigbluebuttonbn_wrap_xml_load_file($url, $method=BIGBLUEBUTTONBN_METHOD_GET, $data=null) {
+function bigbluebuttonbn_wrap_xml_load_file($url, $method = BIGBLUEBUTTONBN_METHOD_GET, $data = null) {
     if (extension_loaded('curl')) {
         $c = new curl();
-        $c->setopt( Array( "SSL_VERIFYPEER" => true));
-        if( $method == BIGBLUEBUTTONBN_METHOD_POST ) {
-            if( !is_null($data) ) {
-                if( !is_array($data) ) {
+        $c->setopt(Array("SSL_VERIFYPEER" => true));
+        if ($method == BIGBLUEBUTTONBN_METHOD_POST) {
+            if (!is_null($data)) {
+                if (!is_array($data)) {
                     $options['CURLOPT_HTTPHEADER'] = array(
                             'Content-Type: text/xml',
-                            'Content-Length: '.strlen($data),
+                            'Content-Length: ' . strlen($data),
                             'Content-Language: en-US'
                         );
                     $response = $c->post($url, $data, $options);
@@ -375,7 +375,7 @@ 
             try {
                 $xml = new SimpleXMLElement($response, LIBXML_NOCDATA);
                 return $xml;
-            } catch (Exception $e){
+            } catch (Exception $e) {
                 libxml_use_internal_errors($previous);
                 error_log("The XML response is not correct on wrap_simplexml_load_file");
                 error_log($response);
@@ -389,18 +389,18 @@ 
     } else {
         $previous = libxml_use_internal_errors(true);
         try {
-            $xml = simplexml_load_file($url,'SimpleXMLElement', LIBXML_NOCDATA);
+            $xml = simplexml_load_file($url, 'SimpleXMLElement', LIBXML_NOCDATA);
             return $xml;
-        } catch  (Exception $e){
+        } catch (Exception $e) {
             libxml_use_internal_errors($previous);
             return NULL;
         }
     }
 }
 
-function bigbluebuttonbn_get_role_name($role_shortname){
+function bigbluebuttonbn_get_role_name($role_shortname) {
     $role = bigbluebuttonbn_get_db_moodle_roles($role_shortname);
-    if( $role != null && $role->name != "") {
+    if ($role != null && $role->name != "") {
         $role_name = $role->name;
     } else {
         switch ($role_shortname) {
@@ -420,13 +420,13 @@ 
     return $role_name;
 }
 
-function bigbluebuttonbn_get_roles($rolename='all', $format='json'){
+function bigbluebuttonbn_get_roles($rolename = 'all', $format = 'json') {
     $roles = bigbluebuttonbn_get_db_moodle_roles($rolename);
     $roles_array = array();
-    foreach($roles as $role){
-        if( $format=='json' ) {
+    foreach ($roles as $role) {
+        if ($format == 'json') {
             array_push($roles_array,
-                    array( "id" => $role->shortname,
+                    array("id" => $role->shortname,
                         "name" => bigbluebuttonbn_get_role_name($role->shortname)
                     )
             );
@@ -437,19 +437,19 @@ 
     return $roles_array;
 }
 
-function bigbluebuttonbn_get_roles_json($rolename='all'){
+function bigbluebuttonbn_get_roles_json($rolename = 'all') {
     return json_encode(bigbluebuttonbn_get_roles($rolename));
 }
 
-function bigbluebuttonbn_get_users_json($users, $full=false) {
-    if( $full ) {
+function bigbluebuttonbn_get_users_json($users, $full = false) {
+    if ($full) {
         return json_encode($users);
     } else {
         $users_array = array();
-        foreach($users as $user){
+        foreach ($users as $user) {
             array_push($users_array,
-                    array( "id" => $user->id,
-                            "name" => $user->firstname.' '.$user->lastname
+                    array("id" => $user->id,
+                            "name" => $user->firstname . ' ' . $user->lastname
                     )
             );
         }
@@ -457,15 +457,15 @@ 
     }
 }
 
-function bigbluebuttonbn_get_participant_list($bigbluebuttonbn=null, $context=null){
+function bigbluebuttonbn_get_participant_list($bigbluebuttonbn = null, $context = null) {
     global $CFG, $USER;
 
     $participant_list_array = array();
 
-    if( $bigbluebuttonbn != null ) {
+    if ($bigbluebuttonbn != null) {
         $participant_list = json_decode($bigbluebuttonbn->participants);
         if (is_array($participant_list)) {
-            foreach($participant_list as $participant){
+            foreach ($participant_list as $participant) {
                 array_push($participant_list_array,
                         array(
                             "selectiontype" => $participant->selectiontype,
@@ -485,16 +485,16 @@ 
         );
 
         $moderator_defaults = bigbluebuttonbn_get_cfg_moderator_default();
-        if ( !isset($moderator_defaults) ) {
+        if (!isset($moderator_defaults)) {
             $moderator_defaults = array('owner');
         } else {
             $moderator_defaults = explode(',', $moderator_defaults);
         }
-        foreach( $moderator_defaults as $moderator_default ) {
-            if( $moderator_default == 'owner' ) {
+        foreach ($moderator_defaults as $moderator_default) {
+            if ($moderator_default == 'owner') {
                 $users = bigbluebuttonbn_get_users($context);
-                foreach( $users as $user ){
-                    if( $user->id == $USER->id ){
+                foreach ($users as $user) {
+                    if ($user->id == $USER->id) {
                         array_push($participant_list_array,
                                 array(
                                         "selectiontype" => "user",
@@ -520,7 +520,7 @@ 
     return $participant_list_array;
 }
 
-function bigbluebuttonbn_get_participant_list_json($bigbluebuttonbnid=null){
+function bigbluebuttonbn_get_participant_list_json($bigbluebuttonbnid = null) {
     return json_encode(bigbluebuttonbn_get_participant_list($bigbluebuttonbnid));
 }
 
@@ -529,21 +529,21 @@ 
 
     if (is_array($participant_list)) {
         // Iterate looking for all configuration
-        foreach($participant_list as $participant){
-            if( $participant->selectiontype == 'all' ) {
-                if ( $participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR )
+        foreach ($participant_list as $participant) {
+            if ($participant->selectiontype == 'all') {
+                if ($participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR)
                     return true;
             }
         }
 
         //Iterate looking for roles
         $db_moodle_roles = bigbluebuttonbn_get_db_moodle_roles();
-        foreach($participant_list as $participant){
-            if( $participant->selectiontype == 'role' ) {
-                foreach( $roles as $role ) {
+        foreach ($participant_list as $participant) {
+            if ($participant->selectiontype == 'role') {
+                foreach ($roles as $role) {
                     $db_moodle_role = bigbluebuttonbn_moodle_db_role_lookup($db_moodle_roles, $role->roleid);
-                    if( $participant->selectionid == $db_moodle_role->shortname ) {
-                        if ( $participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR )
+                    if ($participant->selectionid == $db_moodle_role->shortname) {
+                        if ($participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR)
                             return true;
                     }
                 }
@@ -551,10 +551,10 @@ 
         }
 
         //Iterate looking for users
-        foreach($participant_list as $participant){
-            if( $participant->selectiontype == 'user' ) {
-                if( $participant->selectionid == $user ) {
-                    if ( $participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR )
+        foreach ($participant_list as $participant) {
+            if ($participant->selectiontype == 'user') {
+                if ($participant->selectionid == $user) {
+                    if ($participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR)
                         return true;
                 }
             }
@@ -565,8 +565,8 @@ 
 }
 
 function bigbluebuttonbn_moodle_db_role_lookup($db_moodle_roles, $role_id) {
-    foreach( $db_moodle_roles as $db_moodle_role ){
-        if( $role_id ==  $db_moodle_role->id ) {
+    foreach ($db_moodle_roles as $db_moodle_role) {
+        if ($role_id == $db_moodle_role->id) {
             return $db_moodle_role;
         }
     }
@@ -574,22 +574,22 @@ 
 
 function bigbluebuttonbn_get_error_key($messageKey, $defaultKey = null) {
     $key = $defaultKey;
-    if ( $messageKey == "checksumError" ){
+    if ($messageKey == "checksumError") {
         $key = 'index_error_checksum';
-    } else if ( $messageKey == 'maxConcurrent' ) {
+    } else if ($messageKey == 'maxConcurrent') {
         $key = 'view_error_max_concurrent';
     }
     return $key;
 }
 
-function bigbluebuttonbn_voicebridge_unique($voicebridge, $id=null) {
+function bigbluebuttonbn_voicebridge_unique($voicebridge, $id = null) {
     global $DB;
     $is_unique = true;
-    if( $voicebridge != 0 ) {
+    if ($voicebridge != 0) {
         $table = "bigbluebuttonbn";
-        $select = "voicebridge = ".$voicebridge;
-        if( $id ) $select .= " AND id <> ".$id;
-        if ( $rooms = $DB->get_records_select($table, $select)  ) {
+        $select = "voicebridge = " . $voicebridge;
+        if ($id) $select .= " AND id <> " . $id;
+        if ($rooms = $DB->get_records_select($table, $select)) {
             $is_unique = false;
         }
     }
@@ -602,8 +602,8 @@ 
 
     $duration = 0;
     $now = time();
-    if( $closingtime > 0 && $now < $closingtime ) {
-        $duration = ceil(($closingtime - $now)/60);
+    if ($closingtime > 0 && $now < $closingtime) {
+        $duration = ceil(($closingtime - $now) / 60);
         $compensation_time = intval(bigbluebuttonbn_get_cfg_scheduled_duration_compensation());
         $duration = intval($duration) + $compensation_time;
     }
@@ -611,13 +611,13 @@ 
     return $duration;
 }
 
-function bigbluebuttonbn_get_presentation_array($context, $presentation, $id=null) {
+function bigbluebuttonbn_get_presentation_array($context, $presentation, $id = null) {
     $presentation_name = null;
     $presentation_url = null;
     $presentation_icon = null;
     $presentation_mimetype_description = null;
 
-    if( !empty($presentation) ) {
+    if (!empty($presentation)) {
         $fs = get_file_storage();
         $files = $fs->get_area_files($context->id, 'mod_bigbluebuttonbn', 'presentation', 0, 'itemid, filepath, filename', false);
         if (count($files) < 1) {
@@ -631,12 +631,12 @@ 
             $presentation_icon = file_file_icon($file, 24);
             $presentation_mimetype_description = get_mimetype_description($file);
 
-            if( !is_null($id) ) {
+            if (!is_null($id)) {
                 //Create the nonce component for granting a temporary public access
                 $cache = cache::make_from_params(cache_store::MODE_APPLICATION, 'mod_bigbluebuttonbn', 'presentation_cache');
                 $presentation_nonce_key = sha1($id);
                 $presentation_nonce_value = bigbluebuttonbn_generate_nonce(); 
-                $cache->set($presentation_nonce_key, array( "value" => $presentation_nonce_value, "counter" => 0 ));
+                $cache->set($presentation_nonce_key, array("value" => $presentation_nonce_value, "counter" => 0));
 
                 //The item id was adapted for granting public access to the presentation once in order to allow BigBlueButton to gather the file
                 $url = moodle_url::make_pluginfile_url($file->get_contextid(), $file->get_component(), $file->get_filearea(), $presentation_nonce_value, $file->get_filepath(), $file->get_filename());
@@ -647,7 +647,7 @@ 
         }
     }
 
-    $presentation_array = array( "url" => $presentation_url, "name" => $presentation_name, "icon" => $presentation_icon, "mimetype_description" => $presentation_mimetype_description);
+    $presentation_array = array("url" => $presentation_url, "name" => $presentation_name, "icon" => $presentation_icon, "mimetype_description" => $presentation_mimetype_description);
 
     return $presentation_array;
 }
@@ -657,13 +657,13 @@ 
     $mt = microtime();
     $rand = mt_rand();
 
-    return md5($mt.$rand);
+    return md5($mt . $rand);
 }
 
-function bigbluebuttonbn_random_password( $length = 8 ) {
+function bigbluebuttonbn_random_password($length = 8) {
 
     $chars = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789!@#$%^&*()_-=+;:,.?";
-    $password = substr( str_shuffle( $chars ), 0, $length );
+    $password = substr(str_shuffle($chars), 0, $length);
 
     return $password;
 }
@@ -755,7 +755,7 @@ 
     global $CFG;
 
     $version_major = bigbluebuttonbn_get_moodle_version_major();
-    if ( $version_major < '2014051200' ) {
+    if ($version_major < '2014051200') {
         //This is valid before v2.7
         bigbluebuttonbn_event_log_legacy($event_type, $bigbluebuttonbn, $context, $cm);
 
@@ -765,7 +765,7 @@ 
     }
 }
 
-function bigbluebuttonbn_bbb_broker_get_recordings($meetingid, $password, $forced=false) {
+function bigbluebuttonbn_bbb_broker_get_recordings($meetingid, $password, $forced = false) {
     global $CFG;
 
     $recordings = array();
@@ -781,19 +781,19 @@ 
     $result = $cache->get($meetingid);
     $meeting_info = json_decode($result['meeting_info']);
     $meeting_info->participantCount += 1;
-    if( $is_moderator ) {
+    if ($is_moderator) {
         $meeting_info->moderatorCount += 1;
     }
-    $cache->set($meetingid, array('creation_time' => $result['creation_time'], 'meeting_info' => json_encode($meeting_info) ));
+    $cache->set($meetingid, array('creation_time' => $result['creation_time'], 'meeting_info' => json_encode($meeting_info)));
 }
 
 function bigbluebuttonbn_bbb_broker_is_meeting_running($meeting_info) {
-    $meeting_running = ( isset($meeting_info) && isset($meeting_info->returncode) && $meeting_info->returncode == 'SUCCESS' );
+    $meeting_running = (isset($meeting_info) && isset($meeting_info->returncode) && $meeting_info->returncode == 'SUCCESS');
 
     return $meeting_running;
 }
 
-function bigbluebuttonbn_bbb_broker_get_meeting_info($meetingid, $password, $forced=false) {
+function bigbluebuttonbn_bbb_broker_get_meeting_info($meetingid, $password, $forced = false) {
     global $CFG;
 
     $meeting_info = array();
@@ -804,19 +804,19 @@ 
     $cache = cache::make_from_params(cache_store::MODE_APPLICATION, 'mod_bigbluebuttonbn', 'meetings_cache');
     $result = $cache->get($meetingid);
     $now = time();
-    if( isset($result) && $now < ($result['creation_time'] + $cache_ttl) && !$forced ) {
+    if (isset($result) && $now < ($result['creation_time'] + $cache_ttl) && !$forced) {
         //Use the value in the cache
         $meeting_info = json_decode($result['meeting_info']);
     } else {
         //Ping again and refresh the cache
-        $meeting_info = (array) bigbluebuttonbn_getMeetingInfo( $meetingid, $password, $endpoint, $shared_secret );
-        $cache->set($meetingid, array('creation_time' => time(), 'meeting_info' => json_encode($meeting_info) ));
+        $meeting_info = (array)bigbluebuttonbn_getMeetingInfo($meetingid, $password, $endpoint, $shared_secret);
+        $cache->set($meetingid, array('creation_time' => time(), 'meeting_info' => json_encode($meeting_info)));
     }
 
     return $meeting_info;
 }
 
-function bigbluebuttonbn_bbb_broker_do_end_meeting($meetingid, $password){
+function bigbluebuttonbn_bbb_broker_do_end_meeting($meetingid, $password) {
     global $CFG;
 
     $endpoint = bigbluebuttonbn_get_cfg_server_url();
@@ -825,16 +825,16 @@ 
     bigbluebuttonbn_doEndMeeting($meetingid, $password, $endpoint, $shared_secret);
 }
 
-function bigbluebuttonbn_bbb_broker_do_publish_recording($recordingid, $publish=true){
+function bigbluebuttonbn_bbb_broker_do_publish_recording($recordingid, $publish = true) {
     global $CFG;
 
     $endpoint = bigbluebuttonbn_get_cfg_server_url();
     $shared_secret = bigbluebuttonbn_get_cfg_shared_secret();
 
-    bigbluebuttonbn_doPublishRecordings($recordingid, ($publish)? 'true': 'false', $endpoint, $shared_secret);
+    bigbluebuttonbn_doPublishRecordings($recordingid, ($publish) ? 'true' : 'false', $endpoint, $shared_secret);
 }
 
-function bigbluebuttonbn_bbb_broker_do_delete_recording($recordingid){
+function bigbluebuttonbn_bbb_broker_do_delete_recording($recordingid) {
     global $CFG;
 
     $endpoint = bigbluebuttonbn_get_cfg_server_url();
@@ -846,18 +846,18 @@ 
 function bigbluebuttonbn_bbb_broker_validate_parameters($params) {
     $error = '';
 
-    if ( !isset($params['callback']) ) {
+    if (!isset($params['callback'])) {
         $error = $bigbluebuttonbn_bbb_broker_add_error($error, 'This call must include a javascript callback.');
     }
 
-    if ( !isset($params['action']) ) {
+    if (!isset($params['action'])) {
         $error = $bigbluebuttonbn_bbb_broker_add_error($error, 'Action parameter must be included.');
     } else {
-        switch ( strtolower($params['action']) ){
+        switch (strtolower($params['action'])) {
             case 'server_ping':
             case 'meeting_info':
             case 'meeting_end':
-                if ( !isset($params['id']) ) {
+                if (!isset($params['id'])) {
                     $error = $bigbluebuttonbn_bbb_broker_add_error($error, 'The meetingID must be specified.');
                 }
                 break;
@@ -866,28 +866,28 @@ 
             case 'recording_publish':
             case 'recording_unpublish':
             case 'recording_delete':
-                if ( !isset($params['id']) ) {
+                if (!isset($params['id'])) {
                     $error = bigbluebuttonbn_bbb_broker_add_error($error, 'The recordingID must be specified.');
                 }
                 break;
             case 'recording_ready':
-                if( empty($params['signed_parameters']) ) {
+                if (empty($params['signed_parameters'])) {
                     $error = bigbluebuttonbn_bbb_broker_add_error($error, 'A JWT encoded string must be included as [signed_parameters].');
                 }
                 break;
             default:
-                $error = bigbluebuttonbn_bbb_broker_add_error($error, 'Action '.$params['action'].' can not be performed.');
+                $error = bigbluebuttonbn_bbb_broker_add_error($error, 'Action ' . $params['action'] . ' can not be performed.');
         }
     }
 
     return $error;
 }
 
-function bigbluebuttonbn_bbb_broker_add_error($org_msg, $new_msg='') {
+function bigbluebuttonbn_bbb_broker_add_error($org_msg, $new_msg = '') {
     $error = $org_msg;
 
-    if( !empty($new_msg) ) {
-        if( !empty($error) ) $error .= ' ';
+    if (!empty($new_msg)) {
+        if (!empty($error)) $error .= ' ';
         $error .= $new_msg;
     }
 
@@ -899,27 +899,27 @@ 
 
     $row = null;
 
-    if ( $bbbsession['managerecordings'] || $recording['published'] == 'true' ) {
+    if ($bbbsession['managerecordings'] || $recording['published'] == 'true') {
         $length = 0;
-        $endTime = isset($recording['endTime'])? floatval($recording['endTime']):0;
+        $endTime = isset($recording['endTime']) ? floatval($recording['endTime']) : 0;
         $endTime = $endTime - ($endTime % 1000);
-        $startTime = isset($recording['startTime'])? floatval($recording['startTime']):0;
+        $startTime = isset($recording['startTime']) ? floatval($recording['startTime']) : 0;
         $startTime = $startTime - ($startTime % 1000);
         $duration = intval(($endTime - $startTime) / 60000);
 
         //$meta_course = isset($recording['meta_context'])?str_replace('"', '\"', $recording['meta_context']):'';
         //For backward compatibility
-        if( isset($recording['meta_contextactivity']) ) {
+        if (isset($recording['meta_contextactivity'])) {
             $meta_activity = str_replace('"', '\"', $recording['meta_contextactivity']);
-        } if( isset($recording['meta_bbb-recording-name']) ) {
+        } if (isset($recording['meta_bbb-recording-name'])) {
             $meta_activity = str_replace('"', '\"', $recording['meta_bbb-recording-name']);
         } else {
             $meta_activity = str_replace('"', '\"', $recording['meetingName']);
         }
 
-        if( isset($recording['meta_contextactivitydescription']) ) {
+        if (isset($recording['meta_contextactivitydescription'])) {
             $meta_description = str_replace('"', '\"', $recording['meta_contextactivitydescription']);
-        } else if( isset($recording['meta_bbb-recording-description']) ) {
+        } else if (isset($recording['meta_bbb-recording-description'])) {
             $meta_description = str_replace('"', '\"', $recording['meta_bbb-recording-description']);
         } else {
             $meta_description = '';
@@ -928,12 +928,12 @@ 
         $actionbar = '';
         $params['id'] = $bbbsession['cm']->id;
         $params['recordingid'] = $recording['recordID'];
-        if ( $bbbsession['managerecordings'] ) {
+        if ($bbbsession['managerecordings']) {
             $url = '#';
             $action = null;
 
             ///Set action [show|hide]
-            if ( $recording['published'] == 'true' ){
+            if ($recording['published'] == 'true') {
                 $manage_tag = 'hide';
                 $manage_action = 'unpublish';
             } else {
@@ -941,36 +941,36 @@ 
                 $manage_action = 'publish';
             }
 
-            if ( bigbluebuttonbn_get_cfg_recording_icons_enabled() ) {
+            if (bigbluebuttonbn_get_cfg_recording_icons_enabled()) {
                 //With icon for publish/unpublish
-                $icon_attributes = array('id' => 'recording-btn-'.$manage_action.'-'.$recording['recordID']);
-                $icon = new pix_icon('t/'.$manage_tag, get_string($manage_tag), 'moodle', $icon_attributes);
-                $link_attributes = array('id' => 'recording-link-'.$manage_action.'-'.$recording['recordID'], 'onclick' => 'M.mod_bigbluebuttonbn.broker_manageRecording("'.$manage_action.'", "'.$recording['recordID'].'", "'.$recording['meetingID'].'");');
+                $icon_attributes = array('id' => 'recording-btn-' . $manage_action . '-' . $recording['recordID']);
+                $icon = new pix_icon('t/' . $manage_tag, get_string($manage_tag), 'moodle', $icon_attributes);
+                $link_attributes = array('id' => 'recording-link-' . $manage_action . '-' . $recording['recordID'], 'onclick' => 'M.mod_bigbluebuttonbn.broker_manageRecording("' . $manage_action . '", "' . $recording['recordID'] . '", "' . $recording['meetingID'] . '");');
                 $actionbar .= $OUTPUT->action_icon($url, $icon, $action, $link_attributes, false);
 
                 //With icon for delete
-                $icon_attributes = array('id' => 'recording-btn-delete-'.$recording['recordID']);
+                $icon_attributes = array('id' => 'recording-btn-delete-' . $recording['recordID']);
                 $icon = new pix_icon('t/delete', get_string('delete'), 'moodle', $icon_attributes);
-                $link_attributes = array('id' => 'recording-link-delete-'.$recording['recordID'], 'onclick' => 'if(confirm("'.get_string('view_recording_delete_confirmation', 'bigbluebuttonbn').'?")) M.mod_bigbluebuttonbn.broker_manageRecording("delete", "'.$recording['recordID'].'", "'.$recording['meetingID'].'");');
+                $link_attributes = array('id' => 'recording-link-delete-' . $recording['recordID'], 'onclick' => 'if(confirm("' . get_string('view_recording_delete_confirmation', 'bigbluebuttonbn') . '?")) M.mod_bigbluebuttonbn.broker_manageRecording("delete", "' . $recording['recordID'] . '", "' . $recording['meetingID'] . '");');
                 $actionbar .= $OUTPUT->action_icon($url, $icon, $action, $link_attributes, false);
 
             } else {
                 //With text for publish/unpublish
-                $actionbar .= $OUTPUT->action_link($url, get_string($manage_tag), $action, array('title' => get_string($manage_tag), 'class' => 'btn btn-xs', 'onclick' => 'M.mod_bigbluebuttonbn.broker_manageRecording("'.$manage_action.'", "'.$recording['recordID'].'", "'.$recording['meetingID'].'");') );
+                $actionbar .= $OUTPUT->action_link($url, get_string($manage_tag), $action, array('title' => get_string($manage_tag), 'class' => 'btn btn-xs', 'onclick' => 'M.mod_bigbluebuttonbn.broker_manageRecording("' . $manage_action . '", "' . $recording['recordID'] . '", "' . $recording['meetingID'] . '");'));
                 $actionbar .= "&nbsp;";
 
                 //With text for delete
-                $actionbar .= $OUTPUT->action_link($url, get_string('delete'), $action, array('title' => get_string('delete'), 'class' => 'btn btn-xs btn-danger', 'onclick' => 'if(confirm("Are you sure to delete?")) M.mod_bigbluebuttonbn.broker_manageRecording("delete", "'.$recording['recordID'].'", "'.$recording['meetingID'].'");') );
+                $actionbar .= $OUTPUT->action_link($url, get_string('delete'), $action, array('title' => get_string('delete'), 'class' => 'btn btn-xs btn-danger', 'onclick' => 'if(confirm("Are you sure to delete?")) M.mod_bigbluebuttonbn.broker_manageRecording("delete", "' . $recording['recordID'] . '", "' . $recording['meetingID'] . '");'));
             }
         }
 
         if ($recording['published'] == 'true') {
-            $recording_types = '<div id="playbacks-'.$recording['recordID'].'">';
+            $recording_types = '<div id="playbacks-' . $recording['recordID'] . '">';
         } else {
-            $recording_types = '<div id="playbacks-'.$recording['recordID'].'" hidden>';
+            $recording_types = '<div id="playbacks-' . $recording['recordID'] . '" hidden>';
         }
-        foreach ( $recording['playbacks'] as $playback ) {
-            $recording_types .= $OUTPUT->action_link($playback['url'], get_string('view_recording_format_'.$playback['type'], 'bigbluebuttonbn'), null, array('title' => get_string('view_recording_format_'.$playback['type'], 'bigbluebuttonbn'), 'target' => '_new') ).'&#32;';
+        foreach ($recording['playbacks'] as $playback) {
+            $recording_types .= $OUTPUT->action_link($playback['url'], get_string('view_recording_format_' . $playback['type'], 'bigbluebuttonbn'), null, array('title' => get_string('view_recording_format_' . $playback['type'], 'bigbluebuttonbn'), 'target' => '_new')) . '&#32;';
         }
         $recording_types .= '</div>';
 
@@ -985,7 +985,7 @@ 
         $row->date = floatval($recording['startTime']);
         $row->date_formatted = $formattedStartDate;
         $row->duration = $duration;
-        if ( $bbbsession['managerecordings'] ) {
+        if ($bbbsession['managerecordings']) {
             $row->actionbar = $actionbar;
         }
     }
@@ -1011,7 +1011,7 @@ 
         array("key" =>"duration", "label" => $view_recording_duration, "width" => "50px")
         );
 
-    if ( $bbbsession['managerecordings'] ) {
+    if ($bbbsession['managerecordings']) {
         array_push($recordingsbn_columns, array("key" =>"actionbar", "label" => $view_recording_actionbar, "width" => "75px", "allowHTML" => true));
     }
 
@@ -1022,10 +1022,10 @@ 
     $table_data = array();
 
     ///Build table content
-    if ( isset($recordings) && !array_key_exists('messageKey', $recordings)) {  // There are recordings for this meeting
-        foreach ( $recordings as $recording ) {
+    if (isset($recordings) && !array_key_exists('messageKey', $recordings)) {  // There are recordings for this meeting
+        foreach ($recordings as $recording) {
             $row = bigbluebuttonbn_get_recording_data_row($bbbsession, $recording);
-            if( $row != null ) {
+            if ($row != null) {
                 array_push($table_data, $row);
             }
         }
@@ -1053,27 +1053,27 @@ 
     $table->data = array();
 
     ///Initialize table headers
-    if ( $bbbsession['managerecordings'] ) {
-        $table->head  = array ($view_recording_recording, $view_recording_activity, $view_recording_description, $view_recording_date, $view_recording_duration, $view_recording_actionbar);
-        $table->align = array ('left', 'left', 'left', 'left', 'center', 'left');
+    if ($bbbsession['managerecordings']) {
+        $table->head  = array($view_recording_recording, $view_recording_activity, $view_recording_description, $view_recording_date, $view_recording_duration, $view_recording_actionbar);
+        $table->align = array('left', 'left', 'left', 'left', 'center', 'left');
     } else {
-        $table->head  = array ($view_recording_recording, $view_recording_activity, $view_recording_description, $view_recording_date, $view_recording_duration);
-        $table->align = array ('left', 'left', 'left', 'left', 'center');
+        $table->head  = array($view_recording_recording, $view_recording_activity, $view_recording_description, $view_recording_date, $view_recording_duration);
+        $table->align = array('left', 'left', 'left', 'left', 'center');
     }
 
     ///Build table content
-    if ( isset($recordings) && !array_key_exists('messageKey', $recordings)) {  // There are recordings for this meeting
-        foreach ( $recordings as $recording ){
+    if (isset($recordings) && !array_key_exists('messageKey', $recordings)) {  // There are recordings for this meeting
+        foreach ($recordings as $recording) {
             $row = new html_table_row();
-            $row->id = 'recording-td-'.$recording['recordID'];
+            $row->id = 'recording-td-' . $recording['recordID'];
 
             $row_data = bigbluebuttonbn_get_recording_data_row($bbbsession, $recording);
-            if( $row_data != null ) {
+            if ($row_data != null) {
                 $row_data->date_formatted = str_replace(" ", "&nbsp;", $row_data->date_formatted);
-                if ( $bbbsession['managerecordings'] ) {
-                    $row->cells = array ($row_data->recording, $row_data->activity, $row_data->description, $row_data->date_formatted, $row_data->duration, $row_data->actionbar );
+                if ($bbbsession['managerecordings']) {
+                    $row->cells = array($row_data->recording, $row_data->activity, $row_data->description, $row_data->date_formatted, $row_data->duration, $row_data->actionbar);
                 } else {
-                    $row->cells = array ($row_data->recording, $row_data->activity, $row_data->description, $row_data->date_formatted, $row_data->duration );
+                    $row->cells = array($row_data->recording, $row_data->activity, $row_data->description, $row_data->date_formatted, $row_data->duration);
                 }
 
                 array_push($table->data, $row);
@@ -1092,15 +1092,15 @@ 
 
     /// Build the message_body
     $msg->activity_type = "";
-    if( isset($bigbluebuttonbn->type) && $bigbluebuttonbn->type != 0 )
+    if (isset($bigbluebuttonbn->type) && $bigbluebuttonbn->type != 0)
         $msg->activity_type = bigbluebuttonbn_get_predefinedprofile_name($bigbluebuttonbn->type);
     $msg->activity_title = $bigbluebuttonbn->name;
-    $message_text = '<p>'.get_string('email_body_recording_ready_for', 'bigbluebuttonbn').' '.$msg->activity_type.' &quot;'.$msg->activity_title.'&quot; '.get_string('email_body_recording_ready_is_ready', 'bigbluebuttonbn').'.</p>';
+    $message_text = '<p>' . get_string('email_body_recording_ready_for', 'bigbluebuttonbn') . ' ' . $msg->activity_type . ' &quot;' . $msg->activity_title . '&quot; ' . get_string('email_body_recording_ready_is_ready', 'bigbluebuttonbn') . '.</p>';
     
     bigbluebuttonbn_send_notification($sender, $bigbluebuttonbn, $message_text);
 }
 
-function bigbluebuttonbn_server_offers($capability_name){
+function bigbluebuttonbn_server_offers($capability_name) {
     global $CFG;
 
     $capability_offered = null;
@@ -1114,12 +1114,12 @@ 
     $host_ends = explode(".", $host);
     $host_ends_length = count($host_ends);
 
-    if( $host_ends_length > 0 && $host_ends[$host_ends_length -1] == 'com' &&  $host_ends[$host_ends_length -2] == 'blindsidenetworks' ) {
+    if ($host_ends_length > 0 && $host_ends[$host_ends_length - 1] == 'com' && $host_ends[$host_ends_length - 2] == 'blindsidenetworks') {
         //Validate the capabilities offered
-        $capabilities = bigbluebuttonbn_getCapabilitiesArray( $endpoint, $shared_secret );
-        if( $capabilities ) {
+        $capabilities = bigbluebuttonbn_getCapabilitiesArray($endpoint, $shared_secret);
+        if ($capabilities) {
             foreach ($capabilities as $capability) {
-                if( $capability["name"] == $capability_name)
+                if ($capability["name"] == $capability_name)
                     $capability_offered = $capability;
             }
         }
@@ -1128,14 +1128,14 @@ 
     return $capability_offered;
 }
 
-function bigbluebuttonbn_server_offers_bn_capabilities(){
+function bigbluebuttonbn_server_offers_bn_capabilities() {
     //Validates if the server may have extended capabilities
     $parsed_url = parse_url(bigbluebuttonbn_get_cfg_server_url());
     $host = isset($parsed_url['host']) ? $parsed_url['host'] : '';
     $host_ends = explode(".", $host);
     $host_ends_length = count($host_ends);
 
-    return ( $host_ends_length > 0 && $host_ends[$host_ends_length -1] == 'com' && $host_ends[$host_ends_length -2] == 'blindsidenetworks' );
+    return ($host_ends_length > 0 && $host_ends[$host_ends_length - 1] == 'com' && $host_ends[$host_ends_length - 2] == 'blindsidenetworks');
 }
 
 function bigbluebuttonbn_get_locales_for_ui() {
@@ -1168,79 +1168,79 @@ 
 
 function bigbluebuttonbn_get_cfg_server_url_default() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_server_url)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_server_url: (isset($CFG->bigbluebuttonbn_server_url)? $CFG->bigbluebuttonbn_server_url: (isset($CFG->BigBlueButtonBNServerURL)? $CFG->BigBlueButtonBNServerURL: 'http://test-install.blindsidenetworks.com/bigbluebutton/')));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_server_url) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_server_url : (isset($CFG->bigbluebuttonbn_server_url) ? $CFG->bigbluebuttonbn_server_url : (isset($CFG->BigBlueButtonBNServerURL) ? $CFG->BigBlueButtonBNServerURL : 'http://test-install.blindsidenetworks.com/bigbluebutton/')));
 }
 
 function bigbluebuttonbn_get_cfg_shared_secret_default() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_shared_secret)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_shared_secret: (isset($CFG->bigbluebuttonbn_shared_secret)? $CFG->bigbluebuttonbn_shared_secret: (isset($CFG->BigBlueButtonBNSecuritySalt)? $CFG->BigBlueButtonBNSecuritySalt: '8cd8ef52e8e101574e400365b55e11a6')));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_shared_secret) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_shared_secret : (isset($CFG->bigbluebuttonbn_shared_secret) ? $CFG->bigbluebuttonbn_shared_secret : (isset($CFG->BigBlueButtonBNSecuritySalt) ? $CFG->BigBlueButtonBNSecuritySalt : '8cd8ef52e8e101574e400365b55e11a6')));
 }
 
 function bigbluebuttonbn_get_cfg_voicebridge_editable() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_voicebridge_editable)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_voicebridge_editable: (isset($CFG->bigbluebuttonbn_voicebridge_editable)? $CFG->bigbluebuttonbn_voicebridge_editable: false));    
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_voicebridge_editable) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_voicebridge_editable : (isset($CFG->bigbluebuttonbn_voicebridge_editable) ? $CFG->bigbluebuttonbn_voicebridge_editable : false));    
 }
 
 function bigbluebuttonbn_get_cfg_recording_default() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recording_default)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recording_default: (isset($CFG->bigbluebuttonbn_recording_default)? $CFG->bigbluebuttonbn_recording_default: true));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recording_default) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recording_default : (isset($CFG->bigbluebuttonbn_recording_default) ? $CFG->bigbluebuttonbn_recording_default : true));
 }
 
 function bigbluebuttonbn_get_cfg_recording_editable() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recording_editable)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recording_editable: (isset($CFG->bigbluebuttonbn_recording_editable)? $CFG->bigbluebuttonbn_recording_editable: true));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recording_editable) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recording_editable : (isset($CFG->bigbluebuttonbn_recording_editable) ? $CFG->bigbluebuttonbn_recording_editable : true));
 }
 
 function bigbluebuttonbn_get_cfg_recording_tagging_default() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recordingtagging_default)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recordingtagging_default: (isset($CFG->bigbluebuttonbn_recordingtagging_default)? $CFG->bigbluebuttonbn_recordingtagging_default: false));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recordingtagging_default) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recordingtagging_default : (isset($CFG->bigbluebuttonbn_recordingtagging_default) ? $CFG->bigbluebuttonbn_recordingtagging_default : false));
 }
 
 function bigbluebuttonbn_get_cfg_recording_tagging_editable() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recordingtagging_editable)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recordingtagging_editable: (isset($CFG->bigbluebuttonbn_recordingtagging_editable)? $CFG->bigbluebuttonbn_recordingtagging_editable: false));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recordingtagging_editable) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recordingtagging_editable : (isset($CFG->bigbluebuttonbn_recordingtagging_editable) ? $CFG->bigbluebuttonbn_recordingtagging_editable : false));
 }
 
 function bigbluebuttonbn_get_cfg_recording_icons_enabled() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recording_icons_enabled)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recording_icons_enabled: (isset($CFG->bigbluebuttonbn_recording_icons_enabled)? $CFG->bigbluebuttonbn_recording_icons_enabled: true));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recording_icons_enabled) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recording_icons_enabled : (isset($CFG->bigbluebuttonbn_recording_icons_enabled) ? $CFG->bigbluebuttonbn_recording_icons_enabled : true));
 }
 
 function bigbluebuttonbn_get_cfg_waitformoderator_default() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_waitformoderator_default)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_waitformoderator_default: (isset($CFG->bigbluebuttonbn_waitformoderator_default)? $CFG->bigbluebuttonbn_waitformoderator_default: false));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_waitformoderator_default) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_waitformoderator_default : (isset($CFG->bigbluebuttonbn_waitformoderator_default) ? $CFG->bigbluebuttonbn_waitformoderator_default : false));
 }
 
 function bigbluebuttonbn_get_cfg_waitformoderator_editable() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_waitformoderator_editable)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_waitformoderator_editable: (isset($CFG->bigbluebuttonbn_waitformoderator_editable)? $CFG->bigbluebuttonbn_waitformoderator_editable: true));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_waitformoderator_editable) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_waitformoderator_editable : (isset($CFG->bigbluebuttonbn_waitformoderator_editable) ? $CFG->bigbluebuttonbn_waitformoderator_editable : true));
 }
 
 function bigbluebuttonbn_get_cfg_waitformoderator_ping_interval() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_waitformoderator_ping_interval)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_waitformoderator_ping_interval: (isset($CFG->bigbluebuttonbn_waitformoderator_ping_interval)? $CFG->bigbluebuttonbn_waitformoderator_ping_interval: 15));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_waitformoderator_ping_interval) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_waitformoderator_ping_interval : (isset($CFG->bigbluebuttonbn_waitformoderator_ping_interval) ? $CFG->bigbluebuttonbn_waitformoderator_ping_interval : 15));
 }
 
 function bigbluebuttonbn_get_cfg_waitformoderator_cache_ttl() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_waitformoderator_cache_ttl)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_waitformoderator_cache_ttl: (isset($CFG->bigbluebuttonbn_waitformoderator_cache_ttl)? $CFG->bigbluebuttonbn_waitformoderator_cache_ttl: 60));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_waitformoderator_cache_ttl) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_waitformoderator_cache_ttl : (isset($CFG->bigbluebuttonbn_waitformoderator_cache_ttl) ? $CFG->bigbluebuttonbn_waitformoderator_cache_ttl : 60));
 }
 
 function bigbluebuttonbn_get_cfg_userlimit_default() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_userlimit_default)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_userlimit_default: (isset($CFG->bigbluebuttonbn_userlimit_default)? $CFG->bigbluebuttonbn_userlimit_default: 0));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_userlimit_default) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_userlimit_default : (isset($CFG->bigbluebuttonbn_userlimit_default) ? $CFG->bigbluebuttonbn_userlimit_default : 0));
 }
 
 function bigbluebuttonbn_get_cfg_userlimit_editable() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_userlimit_editable)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_userlimit_editable: (isset($CFG->bigbluebuttonbn_userlimit_editable)? $CFG->bigbluebuttonbn_userlimit_editable: false));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_userlimit_editable) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_userlimit_editable : (isset($CFG->bigbluebuttonbn_userlimit_editable) ? $CFG->bigbluebuttonbn_userlimit_editable : false));
 }
 
 function bigbluebuttonbn_get_cfg_preuploadpresentation_enabled() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
     if (extension_loaded('curl')) {
         // This feature only works if curl is installed
-        return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_preuploadpresentation_enabled)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_preuploadpresentation_enabled: (isset($CFG->bigbluebuttonbn_preuploadpresentation_enabled)? $CFG->bigbluebuttonbn_preuploadpresentation_enabled: false));
+        return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_preuploadpresentation_enabled) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_preuploadpresentation_enabled : (isset($CFG->bigbluebuttonbn_preuploadpresentation_enabled) ? $CFG->bigbluebuttonbn_preuploadpresentation_enabled : false));
     } else {
         return false;
     }
@@ -1248,30 +1248,30 @@ 
 
 function bigbluebuttonbn_get_cfg_sendnotifications_enabled() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_sendnotifications_enabled)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_sendnotifications_enabled: (isset($CFG->bigbluebuttonbn_sendnotifications_enabled)? $CFG->bigbluebuttonbn_sendnotifications_enabled: false));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_sendnotifications_enabled) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_sendnotifications_enabled : (isset($CFG->bigbluebuttonbn_sendnotifications_enabled) ? $CFG->bigbluebuttonbn_sendnotifications_enabled : false));
 }
 
 function bigbluebuttonbn_get_cfg_recordingready_enabled() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recordingready_enabled)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recordingready_enabled: (isset($CFG->bigbluebuttonbn_recordingready_enabled)? $CFG->bigbluebuttonbn_recordingready_enabled: false));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recordingready_enabled) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_recordingready_enabled : (isset($CFG->bigbluebuttonbn_recordingready_enabled) ? $CFG->bigbluebuttonbn_recordingready_enabled : false));
 }
 
 function bigbluebuttonbn_get_cfg_moderator_default() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_moderator_default)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_moderator_default: (isset($CFG->bigbluebuttonbn_moderator_default)? $CFG->bigbluebuttonbn_moderator_default: 'owner'));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_moderator_default) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_moderator_default : (isset($CFG->bigbluebuttonbn_moderator_default) ? $CFG->bigbluebuttonbn_moderator_default : 'owner'));
 }
 
 function bigbluebuttonbn_get_cfg_scheduled_duration_enabled() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_scheduled_duration_enabled)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_scheduled_duration_enabled: (isset($CFG->bigbluebuttonbn_scheduled_duration_enabled)? $CFG->bigbluebuttonbn_scheduled_duration_enabled: false));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_scheduled_duration_enabled) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_scheduled_duration_enabled : (isset($CFG->bigbluebuttonbn_scheduled_duration_enabled) ? $CFG->bigbluebuttonbn_scheduled_duration_enabled : false));
 }
 
 function bigbluebuttonbn_get_cfg_scheduled_duration_compensation() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_scheduled_duration_compensation)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_scheduled_duration_compensation: (isset($CFG->bigbluebuttonbn_scheduled_duration_compensation)? $CFG->bigbluebuttonbn_scheduled_duration_compensation: 10));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_scheduled_duration_compensation) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_scheduled_duration_compensation : (isset($CFG->bigbluebuttonbn_scheduled_duration_compensation) ? $CFG->bigbluebuttonbn_scheduled_duration_compensation : 10));
 }
 
 function bigbluebuttonbn_get_cfg_scheduled_pre_opening() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_scheduled_pre_opening)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_scheduled_pre_opening: (isset($CFG->bigbluebuttonbn_scheduled_pre_opening)? $CFG->bigbluebuttonbn_scheduled_pre_opening: 10));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_scheduled_pre_opening) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_scheduled_pre_opening : (isset($CFG->bigbluebuttonbn_scheduled_pre_opening) ? $CFG->bigbluebuttonbn_scheduled_pre_opening : 10));
 }

Braces +60 added lines, -36 removed lines

@@ -47,8 +47,9 @@ 
     $log->userid = $bbbsession['userID'];
     $log->timecreated = time();
     $log->event = $event;
-    if( $event == BIGBLUEBUTTONBN_LOG_EVENT_CREATE)
-        $log->meta = '{"record":'.($bbbsession['record']? 'true': 'false').'}';
+    if( $event == BIGBLUEBUTTONBN_LOG_EVENT_CREATE) {
+            $log->meta = '{"record":'.($bbbsession['record']? 'true': 'false').'}';
+    }
 
     $returnid = $DB->insert_record('bigbluebuttonbn_log', $log);
 }
@@ -68,15 +69,18 @@ 
     $params = 'name='.urlencode($name).'&meetingID='.urlencode($meetingID).'&attendeePW='.urlencode($attendeePW).'&moderatorPW='.urlencode($moderatorPW).'&logoutURL='.urlencode($logoutURL).'&record='.$record;
 
     $voiceBridge = intval($voiceBridge);
-    if ( $voiceBridge > 0 && $voiceBridge < 79999)
-        $params .= '&voiceBridge='.$voiceBridge;
+    if ( $voiceBridge > 0 && $voiceBridge < 79999) {
+            $params .= '&voiceBridge='.$voiceBridge;
+    }
 
     $duration = intval($duration);
-    if( $duration > 0 )
-        $params .= '&duration='.$duration;
+    if( $duration > 0 ) {
+            $params .= '&duration='.$duration;
+    }
 
-    if( trim( $welcome ) )
-        $params .= '&welcome='.urlencode($welcome);
+    if( trim( $welcome ) ) {
+            $params .= '&welcome='.urlencode($welcome);
+    }
 
     foreach ($metadata as $key => $value) {
         $params .= '&'.$key.'='.urlencode($value);
@@ -148,10 +152,12 @@ 
     }
 
     if( $xml ) {
-        if($xml->meetingID) return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey, 'meetingID' => $xml->meetingID, 'attendeePW' => $xml->attendeePW, 'moderatorPW' => $xml->moderatorPW, 'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded );
-        else return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey );
-    }
-    else {
+        if($xml->meetingID) {
+            return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey, 'meetingID' => $xml->meetingID, 'attendeePW' => $xml->attendeePW, 'moderatorPW' => $xml->moderatorPW, 'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded );
+        } else {
+            return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey );
+        }
+    } else {
         return null;
     }
 }
@@ -223,7 +229,9 @@ 
             $metadataArray = array();
             $metadata = get_object_vars($recording->metadata);
             foreach ($metadata as $key => $value) {
-                if(is_object($value)) $value = '';
+                if(is_object($value)) {
+                    $value = '';
+                }
                 $metadataArray['meta_'.$key] = $value;
             }
 
@@ -261,7 +269,9 @@ 
                 $metadataArray = array();
                 $metadata = get_object_vars($recording->metadata);
                 foreach ($metadata as $key => $value) {
-                    if(is_object($value)) $value = '';
+                    if(is_object($value)) {
+                        $value = '';
+                    }
                     $metadataArray['meta_'.$key] = $value;
                 }
 
@@ -280,17 +290,22 @@ 
 }
 
 function bigbluebuttonbn_recordingBuildSorter($a, $b){
-    if( $a['startTime'] < $b['startTime']) return -1;
-    else if( $a['startTime'] == $b['startTime']) return 0;
-    else return 1;
-}
+    if( $a['startTime'] < $b['startTime']) {
+        return -1;
+    } else if( $a['startTime'] == $b['startTime']) {
+        return 0;
+    } else {
+        return 1;
+    }
+    }
 
 function bigbluebuttonbn_doDeleteRecordings( $recordIDs, $URL, $SALT ) {
     $ids = 	explode(",", $recordIDs);
     foreach( $ids as $id){
         $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getDeleteRecordingsURL($id, $URL, $SALT) );
-        if( $xml && $xml->returncode != 'SUCCESS' )
-            return false;
+        if( $xml && $xml->returncode != 'SUCCESS' ) {
+                    return false;
+        }
     }
     return true;
 }
@@ -299,8 +314,9 @@ 
     $ids = 	explode(",", $recordIDs);
     foreach( $ids as $id){
         $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getPublishRecordingsURL($id, $set, $URL, $SALT) );
-        if( $xml && $xml->returncode != 'SUCCESS' )
-            return false;
+        if( $xml && $xml->returncode != 'SUCCESS' ) {
+                    return false;
+        }
     }
     return true;
 }
@@ -310,8 +326,7 @@ 
 
     if( $xml ) { //If the xml packet returned failure it displays the message to the user
         return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey);
-    }
-    else { //If the server is unreachable, then prompts the user of the necessary action
+    } else { //If the server is unreachable, then prompts the user of the necessary action
         return null;
     }
 }
@@ -531,8 +546,9 @@ 
         // Iterate looking for all configuration
         foreach($participant_list as $participant){
             if( $participant->selectiontype == 'all' ) {
-                if ( $participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR )
-                    return true;
+                if ( $participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR ) {
+                                    return true;
+                }
             }
         }
 
@@ -543,8 +559,9 @@ 
                 foreach( $roles as $role ) {
                     $db_moodle_role = bigbluebuttonbn_moodle_db_role_lookup($db_moodle_roles, $role->roleid);
                     if( $participant->selectionid == $db_moodle_role->shortname ) {
-                        if ( $participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR )
-                            return true;
+                        if ( $participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR ) {
+                                                    return true;
+                        }
                     }
                 }
             }
@@ -554,8 +571,9 @@ 
         foreach($participant_list as $participant){
             if( $participant->selectiontype == 'user' ) {
                 if( $participant->selectionid == $user ) {
-                    if ( $participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR )
-                        return true;
+                    if ( $participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR ) {
+                                            return true;
+                    }
                 }
             }
         }
@@ -588,7 +606,9 @@ 
     if( $voicebridge != 0 ) {
         $table = "bigbluebuttonbn";
         $select = "voicebridge = ".$voicebridge;
-        if( $id ) $select .= " AND id <> ".$id;
+        if( $id ) {
+            $select .= " AND id <> ".$id;
+        }
         if ( $rooms = $DB->get_records_select($table, $select)  ) {
             $is_unique = false;
         }
@@ -887,7 +907,9 @@ 
     $error = $org_msg;
 
     if( !empty($new_msg) ) {
-        if( !empty($error) ) $error .= ' ';
+        if( !empty($error) ) {
+            $error .= ' ';
+        }
         $error .= $new_msg;
     }
 
@@ -1092,8 +1114,9 @@ 
 
     /// Build the message_body
     $msg->activity_type = "";
-    if( isset($bigbluebuttonbn->type) && $bigbluebuttonbn->type != 0 )
-        $msg->activity_type = bigbluebuttonbn_get_predefinedprofile_name($bigbluebuttonbn->type);
+    if( isset($bigbluebuttonbn->type) && $bigbluebuttonbn->type != 0 ) {
+            $msg->activity_type = bigbluebuttonbn_get_predefinedprofile_name($bigbluebuttonbn->type);
+    }
     $msg->activity_title = $bigbluebuttonbn->name;
     $message_text = '<p>'.get_string('email_body_recording_ready_for', 'bigbluebuttonbn').' '.$msg->activity_type.' &quot;'.$msg->activity_title.'&quot; '.get_string('email_body_recording_ready_is_ready', 'bigbluebuttonbn').'.</p>';
     
@@ -1119,8 +1142,9 @@ 
         $capabilities = bigbluebuttonbn_getCapabilitiesArray( $endpoint, $shared_secret );
         if( $capabilities ) {
             foreach ($capabilities as $capability) {
-                if( $capability["name"] == $capability_name)
-                    $capability_offered = $capability;
+                if( $capability["name"] == $capability_name) {
+                                    $capability_offered = $capability;
+                }
             }
         }
     }

view.php

Indentation +24 added lines, -24 removed lines

@@ -280,28 +280,28 @@ 
     echo $OUTPUT->box_end();
     if( $bbbsession['tagging'] && ($bbbsession['administrator'] || $bbbsession['moderator']) ){
         echo ''.
-          '<div id="panelContent" class="hidden">'.
-          '  <div class="yui3-widget-bd">'.
-          '    <form>'.
-          '      <fieldset>'.
-          '        <input type="hidden" name="join" id="meeting_join_url" value="">'.
-          '        <input type="hidden" name="message" id="meeting_message" value="">'.
-          '        <div>'.
-          '          <label for="name">'.get_string('view_recording_name', 'bigbluebuttonbn').'</label><br/>'.
-          '          <input type="text" name="name" id="recording_name" placeholder="">'.
-          '        </div><br>'.
-          '        <div>'.
-          '          <label for="description">'.get_string('view_recording_description', 'bigbluebuttonbn').'</label><br/>'.
-          '          <input type="text" name="description" id="recording_description" value="" placeholder="">'.
-          '        </div><br>'.
-          '        <div>'.
-          '          <label for="tags">'.get_string('view_recording_tags', 'bigbluebuttonbn').'</label><br/>'.
-          '          <input type="text" name="tags" id="recording_tags" value="" placeholder="">'.
-          '        </div>'.
-          '      </fieldset>'.
-          '    </form>'.
-          '  </div>'.
-          '</div>';
+            '<div id="panelContent" class="hidden">'.
+            '  <div class="yui3-widget-bd">'.
+            '    <form>'.
+            '      <fieldset>'.
+            '        <input type="hidden" name="join" id="meeting_join_url" value="">'.
+            '        <input type="hidden" name="message" id="meeting_message" value="">'.
+            '        <div>'.
+            '          <label for="name">'.get_string('view_recording_name', 'bigbluebuttonbn').'</label><br/>'.
+            '          <input type="text" name="name" id="recording_name" placeholder="">'.
+            '        </div><br>'.
+            '        <div>'.
+            '          <label for="description">'.get_string('view_recording_description', 'bigbluebuttonbn').'</label><br/>'.
+            '          <input type="text" name="description" id="recording_description" value="" placeholder="">'.
+            '        </div><br>'.
+            '        <div>'.
+            '          <label for="tags">'.get_string('view_recording_tags', 'bigbluebuttonbn').'</label><br/>'.
+            '          <input type="text" name="tags" id="recording_tags" value="" placeholder="">'.
+            '        </div>'.
+            '      </fieldset>'.
+            '    </form>'.
+            '  </div>'.
+            '</div>';
     }
 
     echo $OUTPUT->box_start('generalbox boxaligncenter', 'bigbluebuttonbn_view_action_button_box');
@@ -338,8 +338,8 @@ 
         $icon = new pix_icon($bbbsession['presentation']['icon'], $bbbsession['presentation']['mimetype_description']);
 
         echo '<h4>'.get_string('view_section_title_presentation', 'bigbluebuttonbn').'</h4>'.
-             ''.$OUTPUT->action_icon($bbbsession['presentation']['url'], $icon, null, array(), false).''.
-             ''.$OUTPUT->action_link($bbbsession['presentation']['url'], $bbbsession['presentation']['name'], null, $attributes).'<br><br>';
+                ''.$OUTPUT->action_icon($bbbsession['presentation']['url'], $icon, null, array(), false).''.
+                ''.$OUTPUT->action_link($bbbsession['presentation']['url'], $bbbsession['presentation']['name'], null, $attributes).'<br><br>';
     }
 
     bigbluebuttonbn_view_recordings($bbbsession);

Spacing +89 added lines, -89 removed lines

@@ -9,12 +9,12 @@ 
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v2 or later
  */
 
-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');
 
-$id = required_param('id', PARAM_INT);              // Course Module ID, or
-$b  = optional_param('n', 0, PARAM_INT);            // bigbluebuttonbn instance ID
-$group  = optional_param('group', 0, PARAM_INT);    // group instance ID
+$id = required_param('id', PARAM_INT); // Course Module ID, or
+$b  = optional_param('n', 0, PARAM_INT); // bigbluebuttonbn instance ID
+$group  = optional_param('group', 0, PARAM_INT); // group instance ID
 
 if ($id) {
     $cm = get_coursemodule_from_id('bigbluebuttonbn', $id, 0, false, MUST_EXIST);
@@ -31,7 +31,7 @@ 
 require_login($course, true, $cm);
 
 $version_major = bigbluebuttonbn_get_moodle_version_major();
-if ( $version_major < '2013111800' ) {
+if ($version_major < '2013111800') {
     //This is valid before v2.6
     $module = $DB->get_record('modules', array('name' => 'bigbluebuttonbn'));
     $module_version = $module->version;
@@ -57,7 +57,7 @@ 
 $bbbsession['roles'] = get_user_roles($context, $USER->id, true);
 
 // User roles
-if( $bigbluebuttonbn->participants == null || $bigbluebuttonbn->participants == "" || $bigbluebuttonbn->participants == "[]" ){
+if ($bigbluebuttonbn->participants == null || $bigbluebuttonbn->participants == "" || $bigbluebuttonbn->participants == "[]") {
     //The room that is being used comes from a previous version
     $bbbsession['moderator'] = has_capability('mod/bigbluebuttonbn:moderate', $context);
 } else {
@@ -77,16 +77,16 @@ 
 // Database info related to the activity
 $bbbsession['meetingdescription'] = $bigbluebuttonbn->intro;
 $bbbsession['welcome'] = $bigbluebuttonbn->welcome;
-if( !isset($bbbsession['welcome']) || $bbbsession['welcome'] == '') {
+if (!isset($bbbsession['welcome']) || $bbbsession['welcome'] == '') {
     $bbbsession['welcome'] = get_string('mod_form_field_welcome_default', 'bigbluebuttonbn'); 
 }
 
 $bbbsession['userlimit'] = intval($bigbluebuttonbn->userlimit);
-$bbbsession['voicebridge'] = ($bigbluebuttonbn->voicebridge > 0)? 70000 + $bigbluebuttonbn->voicebridge: $bigbluebuttonbn->voicebridge;
+$bbbsession['voicebridge'] = ($bigbluebuttonbn->voicebridge > 0) ? 70000 + $bigbluebuttonbn->voicebridge : $bigbluebuttonbn->voicebridge;
 $bbbsession['wait'] = $bigbluebuttonbn->wait;
 $bbbsession['record'] = $bigbluebuttonbn->record;
-if( $bigbluebuttonbn->record )
-    $bbbsession['welcome'] .= '<br><br>'.get_string('bbbrecordwarning', 'bigbluebuttonbn');
+if ($bigbluebuttonbn->record)
+    $bbbsession['welcome'] .= '<br><br>' . get_string('bbbrecordwarning', 'bigbluebuttonbn');
 $bbbsession['tagging'] = $bigbluebuttonbn->tagging;
 
 $bbbsession['openingtime'] = $bigbluebuttonbn->openingtime;
@@ -106,7 +106,7 @@ 
 $bbbsession['originServerName'] = $parsedUrl['host'];
 $bbbsession['originServerUrl'] = $CFG->wwwroot;
 $bbbsession['originServerCommonName'] = '';
-$bbbsession['originTag'] = 'moodle-mod_bigbluebuttonbn ('.$module_version.')';
+$bbbsession['originTag'] = 'moodle-mod_bigbluebuttonbn (' . $module_version . ')';
 // Metadata (context)
 $bbbsession['contextActivityName'] = "";
 $bbbsession['contextActivityDescription'] = "";
@@ -117,22 +117,22 @@ 
 
 // Validates if the BigBlueButton server is running
 $serverVersion = bigbluebuttonbn_getServerVersion($bbbsession['endpoint']);
-if ( !isset($serverVersion) ) { //Server is not working
-    if ( $bbbsession['administrator'] )
-        print_error( 'view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
-    else if ( $bbbsession['moderator'] )
-        print_error( 'view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course );
+if (!isset($serverVersion)) { //Server is not working
+    if ($bbbsession['administrator'])
+        print_error('view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
+    else if ($bbbsession['moderator'])
+        print_error('view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot . '/course/view.php?id=' . $bigbluebuttonbn->course);
     else
-        print_error( 'view_error_unable_join_student', 'bigbluebuttonbn', $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course );
+        print_error('view_error_unable_join_student', 'bigbluebuttonbn', $CFG->wwwroot . '/course/view.php?id=' . $bigbluebuttonbn->course);
 } else {
-    $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getMeetingsURL( $bbbsession['endpoint'], $bbbsession['shared_secret'] ) );
-    if ( !isset($xml) || !isset($xml->returncode) || $xml->returncode == 'FAILED' ){ // The shared secret is wrong
-        if ( $bbbsession['administrator'] )
-            print_error( 'view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
-        else if ( $bbbsession['moderator'] )
-            print_error( 'view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course );
+    $xml = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getMeetingsURL($bbbsession['endpoint'], $bbbsession['shared_secret']));
+    if (!isset($xml) || !isset($xml->returncode) || $xml->returncode == 'FAILED') { // The shared secret is wrong
+        if ($bbbsession['administrator'])
+            print_error('view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
+        else if ($bbbsession['moderator'])
+            print_error('view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot . '/course/view.php?id=' . $bigbluebuttonbn->course);
         else
-            print_error( 'view_error_unable_join_student', 'bigbluebuttonbn', $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course );
+            print_error('view_error_unable_join_student', 'bigbluebuttonbn', $CFG->wwwroot . '/course/view.php?id=' . $bigbluebuttonbn->course);
     }
 }
 
@@ -142,12 +142,12 @@ 
 
 // Print the page header
 $PAGE->set_context($context);
-$PAGE->set_url($CFG->wwwroot.'/mod/bigbluebuttonbn/view.php', array('id' => $cm->id));
+$PAGE->set_url($CFG->wwwroot . '/mod/bigbluebuttonbn/view.php', array('id' => $cm->id));
 $PAGE->set_title(format_string($bigbluebuttonbn->name));
 $PAGE->set_cacheable(false);
 $PAGE->set_heading($course->shortname);
 
-if( $bigbluebuttonbn->newwindow == 1 ) {
+if ($bigbluebuttonbn->newwindow == 1) {
     $PAGE->blocks->show_only_fake_blocks();
 
 } else {
@@ -155,14 +155,14 @@ 
 }
 
 // Validate if the user is in a role allowed to join
-if ( !has_capability('moodle/category:manage', $context) && !has_capability('mod/bigbluebuttonbn:join', $context) ) {
+if (!has_capability('moodle/category:manage', $context) && !has_capability('mod/bigbluebuttonbn:join', $context)) {
     echo $OUTPUT->header();
     if (isguestuser()) {
-        echo $OUTPUT->confirm('<p>'.get_string('view_noguests', 'bigbluebuttonbn').'</p>'.get_string('liketologin'),
-            get_login_url(), $CFG->wwwroot.'/course/view.php?id='.$course->id);
+        echo $OUTPUT->confirm('<p>' . get_string('view_noguests', 'bigbluebuttonbn') . '</p>' . get_string('liketologin'),
+            get_login_url(), $CFG->wwwroot . '/course/view.php?id=' . $course->id);
     } else { 
-        echo $OUTPUT->confirm('<p>'.get_string('view_nojoin', 'bigbluebuttonbn').'</p>'.get_string('liketologin'),
-            get_login_url(), $CFG->wwwroot.'/course/view.php?id='.$course->id);
+        echo $OUTPUT->confirm('<p>' . get_string('view_nojoin', 'bigbluebuttonbn') . '</p>' . get_string('liketologin'),
+            get_login_url(), $CFG->wwwroot . '/course/view.php?id=' . $course->id);
     }
 
     echo $OUTPUT->footer();
@@ -170,10 +170,10 @@ 
 }
 
 // Operation URLs
-$bbbsession['courseURL'] = $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course;
-$bbbsession['logoutURL'] = $CFG->wwwroot.'/mod/bigbluebuttonbn/bbb_view.php?action=logout&id='.$id.'&bn='.$bbbsession['bigbluebuttonbnid'];
-$bbbsession['recordingReadyURL'] = $CFG->wwwroot.'/mod/bigbluebuttonbn/bbb_broker.php?action=recording_ready';
-$bbbsession['joinURL'] = $CFG->wwwroot.'/mod/bigbluebuttonbn/bbb_view.php?action=join&id='.$id.'&bigbluebuttonbn='.$bbbsession['bigbluebuttonbnid'];
+$bbbsession['courseURL'] = $CFG->wwwroot . '/course/view.php?id=' . $bigbluebuttonbn->course;
+$bbbsession['logoutURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_view.php?action=logout&id=' . $id . '&bn=' . $bbbsession['bigbluebuttonbnid'];
+$bbbsession['recordingReadyURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_broker.php?action=recording_ready';
+$bbbsession['joinURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_view.php?action=join&id=' . $id . '&bigbluebuttonbn=' . $bbbsession['bigbluebuttonbnid'];
 
 $bigbluebuttonbn_view = '';
 
@@ -182,20 +182,20 @@ 
 
 /// find out current groups mode
 $groupmode = groups_get_activity_groupmode($bbbsession['cm']);
-if ($groupmode == NOGROUPS ) {  //No groups mode
-    $bbbsession['meetingid'] = $bigbluebuttonbn->meetingid.'-'.$bbbsession['course']->id.'-'.$bbbsession['bigbluebuttonbnid'];
+if ($groupmode == NOGROUPS) {  //No groups mode
+    $bbbsession['meetingid'] = $bigbluebuttonbn->meetingid . '-' . $bbbsession['course']->id . '-' . $bbbsession['bigbluebuttonbnid'];
     $bbbsession['meetingname'] = $bigbluebuttonbn->name;
 
 } else {                                        // Separate or visible groups mode
     echo $OUTPUT->box_start('generalbox boxaligncenter');
-    echo '<br><div class="alert alert-warning">'.get_string('view_groups_selection_warning', 'bigbluebuttonbn').'</div>';
+    echo '<br><div class="alert alert-warning">' . get_string('view_groups_selection_warning', 'bigbluebuttonbn') . '</div>';
     echo $OUTPUT->box_end();
 
     $bbbsession['group'] = groups_get_activity_group($bbbsession['cm'], true);
-    if ($groupmode == SEPARATEGROUPS ) {
-        groups_print_activity_menu($cm, $CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$bbbsession['cm']->id, false, true);
-        if( $bbbsession['group'] == 0 ) {
-            if ( $bbbsession['administrator'] ) {
+    if ($groupmode == SEPARATEGROUPS) {
+        groups_print_activity_menu($cm, $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $bbbsession['cm']->id, false, true);
+        if ($bbbsession['group'] == 0) {
+            if ($bbbsession['administrator']) {
                 $my_groups = groups_get_all_groups($bbbsession['course']->id);
             } else {
                 $my_groups = groups_get_activity_allowed_groups($bbbsession['cm']);
@@ -205,21 +205,21 @@ 
         }
 
     } else {
-        groups_print_activity_menu($cm, $CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$bbbsession['cm']->id);
+        groups_print_activity_menu($cm, $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $bbbsession['cm']->id);
     }
 
-    $bbbsession['meetingid'] = $bigbluebuttonbn->meetingid.'-'.$bbbsession['course']->id.'-'.$bbbsession['bigbluebuttonbnid'].'['.$bbbsession['group'].']';
-    if( $bbbsession['group'] > 0 )
+    $bbbsession['meetingid'] = $bigbluebuttonbn->meetingid . '-' . $bbbsession['course']->id . '-' . $bbbsession['bigbluebuttonbnid'] . '[' . $bbbsession['group'] . ']';
+    if ($bbbsession['group'] > 0)
         $group_name = groups_get_group_name($bbbsession['group']);
     else
         $group_name = get_string('allparticipants');
-    $bbbsession['meetingname'] = $bigbluebuttonbn->name.' ('.$group_name.')';
+    $bbbsession['meetingname'] = $bigbluebuttonbn->name . ' (' . $group_name . ')';
 }
 // Metadata (contextActivityName updated after meeting name is assigned)
 $bbbsession['contextActivityName'] = $bbbsession['meetingname'];
 
 $now = time();
-if (!empty($bigbluebuttonbn->openingtime) && $now < $bigbluebuttonbn->openingtime ) {
+if (!empty($bigbluebuttonbn->openingtime) && $now < $bigbluebuttonbn->openingtime) {
     //CALLING BEFORE
     $bigbluebuttonbn_view = 'before';
 
@@ -252,7 +252,7 @@ 
         'meetingid' => $bbbsession['meetingid'],
         'bigbluebuttonbnid' => $bbbsession['bigbluebuttonbnid'],
         'bigbluebuttonbntype' => $bbbsession['bigbluebuttonbntype'],
-        'ping_interval' => ($waitformoderator_ping_interval > 0? $waitformoderator_ping_interval * 1000: 15000),
+        'ping_interval' => ($waitformoderator_ping_interval > 0 ? $waitformoderator_ping_interval * 1000 : 15000),
         'userlimit' => $bbbsession['userlimit'],
         'locales' => bigbluebuttonbn_get_locales_for_ui()
     );
@@ -269,7 +269,7 @@ 
 // Finish the page
 echo $OUTPUT->footer();
 
-function bigbluebuttonbn_view_joining($bbbsession){
+function bigbluebuttonbn_view_joining($bbbsession) {
     global $CFG, $DB, $OUTPUT;
 
     echo $OUTPUT->heading($bbbsession['meetingname'], 3);
@@ -278,29 +278,29 @@ 
     echo '<br><span id="status_bar"></span><br>';
     echo '<span id="control_panel"></span>';
     echo $OUTPUT->box_end();
-    if( $bbbsession['tagging'] && ($bbbsession['administrator'] || $bbbsession['moderator']) ){
-        echo ''.
-          '<div id="panelContent" class="hidden">'.
-          '  <div class="yui3-widget-bd">'.
-          '    <form>'.
-          '      <fieldset>'.
-          '        <input type="hidden" name="join" id="meeting_join_url" value="">'.
-          '        <input type="hidden" name="message" id="meeting_message" value="">'.
-          '        <div>'.
-          '          <label for="name">'.get_string('view_recording_name', 'bigbluebuttonbn').'</label><br/>'.
-          '          <input type="text" name="name" id="recording_name" placeholder="">'.
-          '        </div><br>'.
-          '        <div>'.
-          '          <label for="description">'.get_string('view_recording_description', 'bigbluebuttonbn').'</label><br/>'.
-          '          <input type="text" name="description" id="recording_description" value="" placeholder="">'.
-          '        </div><br>'.
-          '        <div>'.
-          '          <label for="tags">'.get_string('view_recording_tags', 'bigbluebuttonbn').'</label><br/>'.
-          '          <input type="text" name="tags" id="recording_tags" value="" placeholder="">'.
-          '        </div>'.
-          '      </fieldset>'.
-          '    </form>'.
-          '  </div>'.
+    if ($bbbsession['tagging'] && ($bbbsession['administrator'] || $bbbsession['moderator'])) {
+        echo '' .
+          '<div id="panelContent" class="hidden">' .
+          '  <div class="yui3-widget-bd">' .
+          '    <form>' .
+          '      <fieldset>' .
+          '        <input type="hidden" name="join" id="meeting_join_url" value="">' .
+          '        <input type="hidden" name="message" id="meeting_message" value="">' .
+          '        <div>' .
+          '          <label for="name">' . get_string('view_recording_name', 'bigbluebuttonbn') . '</label><br/>' .
+          '          <input type="text" name="name" id="recording_name" placeholder="">' .
+          '        </div><br>' .
+          '        <div>' .
+          '          <label for="description">' . get_string('view_recording_description', 'bigbluebuttonbn') . '</label><br/>' .
+          '          <input type="text" name="description" id="recording_description" value="" placeholder="">' .
+          '        </div><br>' .
+          '        <div>' .
+          '          <label for="tags">' . get_string('view_recording_tags', 'bigbluebuttonbn') . '</label><br/>' .
+          '          <input type="text" name="tags" id="recording_tags" value="" placeholder="">' .
+          '        </div>' .
+          '      </fieldset>' .
+          '    </form>' .
+          '  </div>' .
           '</div>';
     }
 
@@ -311,19 +311,19 @@ 
     bigbluebuttonbn_view_recordings($bbbsession);
 }
 
-function bigbluebuttonbn_view_before( $bbbsession ){
+function bigbluebuttonbn_view_before($bbbsession) {
     global $CFG, $DB, $OUTPUT;
 
     echo $OUTPUT->heading(get_string('view_message_conference_not_started', 'bigbluebuttonbn'), 3);
 
     echo '<table>';
     if ($bbbsession['openingtime']) {
-        echo '<tr><td class="c0">'.get_string('mod_form_field_openingtime','bigbluebuttonbn').':</td>';
-        echo '    <td class="c1">'.userdate($bbbsession['openingtime']).'</td></tr>';
+        echo '<tr><td class="c0">' . get_string('mod_form_field_openingtime', 'bigbluebuttonbn') . ':</td>';
+        echo '    <td class="c1">' . userdate($bbbsession['openingtime']) . '</td></tr>';
     }
     if ($bbbsession['closingtime']) {
-        echo '<tr><td class="c0">'.get_string('mod_form_field_closingtime','bigbluebuttonbn').':</td>';
-        echo '    <td class="c1">'.userdate($bbbsession['closingtime']).'</td></tr>';
+        echo '<tr><td class="c0">' . get_string('mod_form_field_closingtime', 'bigbluebuttonbn') . ':</td>';
+        echo '    <td class="c1">' . userdate($bbbsession['closingtime']) . '</td></tr>';
     }
     echo '</table>';
 }
@@ -333,32 +333,32 @@ 
 
     echo $OUTPUT->heading(get_string('view_message_conference_has_ended', 'bigbluebuttonbn'), 3);
 
-    if( !is_null($bbbsession['presentation']['url']) ) {
+    if (!is_null($bbbsession['presentation']['url'])) {
         $attributes = array('title' => $bbbsession['presentation']['name']);
         $icon = new pix_icon($bbbsession['presentation']['icon'], $bbbsession['presentation']['mimetype_description']);
 
-        echo '<h4>'.get_string('view_section_title_presentation', 'bigbluebuttonbn').'</h4>'.
-             ''.$OUTPUT->action_icon($bbbsession['presentation']['url'], $icon, null, array(), false).''.
-             ''.$OUTPUT->action_link($bbbsession['presentation']['url'], $bbbsession['presentation']['name'], null, $attributes).'<br><br>';
+        echo '<h4>' . get_string('view_section_title_presentation', 'bigbluebuttonbn') . '</h4>' .
+             '' . $OUTPUT->action_icon($bbbsession['presentation']['url'], $icon, null, array(), false) . '' .
+             '' . $OUTPUT->action_link($bbbsession['presentation']['url'], $bbbsession['presentation']['name'], null, $attributes) . '<br><br>';
     }
 
     bigbluebuttonbn_view_recordings($bbbsession);
 }
 
 function bigbluebuttonbn_view_recordings($bbbsession) {
-    if( isset($bbbsession['record']) && $bbbsession['record'] ) {
-        echo '<h4>'.get_string('view_section_title_recordings', 'bigbluebuttonbn').'</h4>';
+    if (isset($bbbsession['record']) && $bbbsession['record']) {
+        echo '<h4>' . get_string('view_section_title_recordings', 'bigbluebuttonbn') . '</h4>';
 
         $recordings = bigbluebuttonbn_getRecordingsArray($bbbsession['meetingid'], $bbbsession['endpoint'], $bbbsession['shared_secret']);
-        if ( isset($recordings) && !array_key_exists('messageKey', $recordings)) {  // There are recordings for this meeting
+        if (isset($recordings) && !array_key_exists('messageKey', $recordings)) {  // There are recordings for this meeting
             $table = bigbluebuttonbn_get_recording_table($bbbsession, $recordings);
         }
 
-        if( isset($table->data) ) {
+        if (isset($table->data)) {
             //Print the table
-            echo '<div id="bigbluebuttonbn_html_table">'."\n";
-            echo html_writer::table($table)."\n";
-            echo '</div>'."\n";
+            echo '<div id="bigbluebuttonbn_html_table">' . "\n";
+            echo html_writer::table($table) . "\n";
+            echo '</div>' . "\n";
 
         } else {
             print_string('view_message_norecordings', 'bigbluebuttonbn');

Braces +21 added lines, -17 removed lines

@@ -85,8 +85,9 @@ 
 $bbbsession['voicebridge'] = ($bigbluebuttonbn->voicebridge > 0)? 70000 + $bigbluebuttonbn->voicebridge: $bigbluebuttonbn->voicebridge;
 $bbbsession['wait'] = $bigbluebuttonbn->wait;
 $bbbsession['record'] = $bigbluebuttonbn->record;
-if( $bigbluebuttonbn->record )
+if( $bigbluebuttonbn->record ) {
     $bbbsession['welcome'] .= '<br><br>'.get_string('bbbrecordwarning', 'bigbluebuttonbn');
+}
 $bbbsession['tagging'] = $bigbluebuttonbn->tagging;
 
 $bbbsession['openingtime'] = $bigbluebuttonbn->openingtime;
@@ -118,22 +119,24 @@ 
 // Validates if the BigBlueButton server is running
 $serverVersion = bigbluebuttonbn_getServerVersion($bbbsession['endpoint']);
 if ( !isset($serverVersion) ) { //Server is not working
-    if ( $bbbsession['administrator'] )
-        print_error( 'view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
-    else if ( $bbbsession['moderator'] )
-        print_error( 'view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course );
-    else
-        print_error( 'view_error_unable_join_student', 'bigbluebuttonbn', $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course );
-} else {
-    $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getMeetingsURL( $bbbsession['endpoint'], $bbbsession['shared_secret'] ) );
-    if ( !isset($xml) || !isset($xml->returncode) || $xml->returncode == 'FAILED' ){ // The shared secret is wrong
-        if ( $bbbsession['administrator'] )
+    if ( $bbbsession['administrator'] ) {
             print_error( 'view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
-        else if ( $bbbsession['moderator'] )
+    } else if ( $bbbsession['moderator'] ) {
             print_error( 'view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course );
-        else
+    } else {
             print_error( 'view_error_unable_join_student', 'bigbluebuttonbn', $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course );
     }
+    } else {
+    $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getMeetingsURL( $bbbsession['endpoint'], $bbbsession['shared_secret'] ) );
+    if ( !isset($xml) || !isset($xml->returncode) || $xml->returncode == 'FAILED' ){ // The shared secret is wrong
+        if ( $bbbsession['administrator'] ) {
+                    print_error( 'view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
+        } else if ( $bbbsession['moderator'] ) {
+                    print_error( 'view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course );
+        } else {
+                    print_error( 'view_error_unable_join_student', 'bigbluebuttonbn', $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course );
+        }
+    }
 }
 
 // Mark viewed by user (if required)
@@ -209,10 +212,11 @@ 
     }
 
     $bbbsession['meetingid'] = $bigbluebuttonbn->meetingid.'-'.$bbbsession['course']->id.'-'.$bbbsession['bigbluebuttonbnid'].'['.$bbbsession['group'].']';
-    if( $bbbsession['group'] > 0 )
-        $group_name = groups_get_group_name($bbbsession['group']);
-    else
-        $group_name = get_string('allparticipants');
+    if( $bbbsession['group'] > 0 ) {
+            $group_name = groups_get_group_name($bbbsession['group']);
+    } else {
+            $group_name = get_string('allparticipants');
+    }
     $bbbsession['meetingname'] = $bigbluebuttonbn->name.' ('.$group_name.')';
 }
 // Metadata (contextActivityName updated after meeting name is assigned)

bbb_view.php

Switch Indentation +96 added lines, -96 removed lines

@@ -44,115 +44,115 @@
     $bbbsession = $SESSION->bigbluebuttonbn_bbbsession;
 }
 switch (strtolower($action)) {
-    case 'logout':
-        if ( isset($bbbsession) && !is_null($bbbsession) ) {
-            /// Moodle event logger: Create an event for meeting left
-            bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_LEFT, $bigbluebuttonbn, $context, $cm);
+        case 'logout':
+            if ( isset($bbbsession) && !is_null($bbbsession) ) {
+                /// Moodle event logger: Create an event for meeting left
+                bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_LEFT, $bigbluebuttonbn, $context, $cm);
 
-            /// Update the cache
-            $meeting_info = bigbluebuttonbn_bbb_broker_get_meeting_info($bbbsession['meetingid'], $bbbsession['modPW'], BIGBLUEBUTTONBN_FORCED);
+                /// Update the cache
+                $meeting_info = bigbluebuttonbn_bbb_broker_get_meeting_info($bbbsession['meetingid'], $bbbsession['modPW'], BIGBLUEBUTTONBN_FORCED);
 
-            /// Close the tab or window where BBB was opened
-            bigbluebutton_bbb_view_close_window();
-
-        } else {
-            bigbluebutton_bbb_view_close_window_manually();
-        }
-        break;
-    case 'join':
-        if ( isset($bbbsession) && !is_null($bbbsession) ) {
-            //See if the session is in progress
-            if( bigbluebuttonbn_isMeetingRunning( $bbbsession['meetingid'], $bbbsession['endpoint'], $bbbsession['shared_secret'] ) ) {
-                /// Since the meeting is already running, we just join the session
-                bigbluebutton_bbb_view_execute_join($bbbsession, $cm, $context, $bigbluebuttonbn);
+                /// Close the tab or window where BBB was opened
+                bigbluebutton_bbb_view_close_window();
 
             } else {
-                // If user is administrator, moderator or if is viewer and no waiting is required
-                if( $bbbsession['administrator'] || $bbbsession['moderator'] || !$bbbsession['wait'] ) {
-                    /// Prepare the metadata
-                    $metadata = array("meta_bn-origin" => $bbbsession['origin'],
-                            "meta_bbb-origin-version" => $bbbsession['originVersion'],
-                            "meta_bbb-origin-server-name" => $bbbsession['originServerName'],
-                            "meta_bbb-origin-server-common-name" => $bbbsession['originServerCommonName'],
-                            "meta_bbb-origin-tag" => $bbbsession['originTag'],
-                            "meta_bbb-context" => $bbbsession['course']->fullname,
-                            "meta_bbb-recording-name" => (isset($name) && $name != '')? $name: $bbbsession['contextActivityName'],
-                            "meta_bbb-recording-description" => (isset($description) && $description != '')? $description: $bbbsession['contextActivityDescription'],
-                            "meta_bbb-recording-tags" => (isset($tags) && $tags != '')? $tags: $bbbsession['contextActivityTags'],
-                    );
-
-                    if ( bigbluebuttonbn_server_offers_bn_capabilities() && bigbluebuttonbn_get_cfg_recordingready_enabled() ) {
-                        $metadata["meta_bn-recording-ready-url"] = $bbbsession['recordingReadyURL'];
-                    }
+                bigbluebutton_bbb_view_close_window_manually();
+            }
+            break;
+        case 'join':
+            if ( isset($bbbsession) && !is_null($bbbsession) ) {
+                //See if the session is in progress
+                if( bigbluebuttonbn_isMeetingRunning( $bbbsession['meetingid'], $bbbsession['endpoint'], $bbbsession['shared_secret'] ) ) {
+                    /// Since the meeting is already running, we just join the session
+                    bigbluebutton_bbb_view_execute_join($bbbsession, $cm, $context, $bigbluebuttonbn);
 
-                    /// Set the duration for the meeting
-                    if ( bigbluebuttonbn_get_cfg_scheduled_duration_enabled() ) {
-                        $durationtime = bigbluebuttonbn_get_duration($bigbluebuttonbn->openingtime, $bigbluebuttonbn->closingtime);
-                        if( $durationtime > 0 )
-                            $bbbsession['welcome'] .= '<br><br>'.str_replace("%duration%", ''.$durationtime, get_string('bbbdurationwarning', 'bigbluebuttonbn'));
-                    } else {
-                        $durationtime = 0;
-                    }
-                    /// Execute the create command
-                    $response = bigbluebuttonbn_getCreateMeetingArray(
-                            $bbbsession['meetingname'],
-                            $bbbsession['meetingid'],
-                            $bbbsession['welcome'],
-                            $bbbsession['modPW'],
-                            $bbbsession['viewerPW'],
-                            $bbbsession['shared_secret'],
-                            $bbbsession['endpoint'],
-                            $bbbsession['logoutURL'],
-                            $bbbsession['record']? 'true': 'false',
-                            $durationtime,
-                            $bbbsession['voicebridge'],
-                            $metadata,
-                            $bbbsession['presentation']['name'],
-                            $bbbsession['presentation']['url']
-                    );
-
-                    if (!$response) {
-                        // If the server is unreachable, then prompts the user of the necessary action
-                        if ( $bbbsession['administrator'] ) {
-                            print_error( 'view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
-                        } else if ( $bbbsession['moderator'] ) {
-                            print_error( 'view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
-                        } else {
-                            print_error( 'view_error_unable_join_student', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
+                } else {
+                    // If user is administrator, moderator or if is viewer and no waiting is required
+                    if( $bbbsession['administrator'] || $bbbsession['moderator'] || !$bbbsession['wait'] ) {
+                        /// Prepare the metadata
+                        $metadata = array("meta_bn-origin" => $bbbsession['origin'],
+                                "meta_bbb-origin-version" => $bbbsession['originVersion'],
+                                "meta_bbb-origin-server-name" => $bbbsession['originServerName'],
+                                "meta_bbb-origin-server-common-name" => $bbbsession['originServerCommonName'],
+                                "meta_bbb-origin-tag" => $bbbsession['originTag'],
+                                "meta_bbb-context" => $bbbsession['course']->fullname,
+                                "meta_bbb-recording-name" => (isset($name) && $name != '')? $name: $bbbsession['contextActivityName'],
+                                "meta_bbb-recording-description" => (isset($description) && $description != '')? $description: $bbbsession['contextActivityDescription'],
+                                "meta_bbb-recording-tags" => (isset($tags) && $tags != '')? $tags: $bbbsession['contextActivityTags'],
+                        );
+
+                        if ( bigbluebuttonbn_server_offers_bn_capabilities() && bigbluebuttonbn_get_cfg_recordingready_enabled() ) {
+                            $metadata["meta_bn-recording-ready-url"] = $bbbsession['recordingReadyURL'];
                         }
 
-                    } else if( $response['returncode'] == "FAILED" ) {
-                        // The meeting was not created
-                        $error_key = bigbluebuttonbn_get_error_key( $response['messageKey'], 'view_error_create' );
-                        if( !$error_key ) {
-                            print_error( $response['message'], 'bigbluebuttonbn' );
+                        /// Set the duration for the meeting
+                        if ( bigbluebuttonbn_get_cfg_scheduled_duration_enabled() ) {
+                            $durationtime = bigbluebuttonbn_get_duration($bigbluebuttonbn->openingtime, $bigbluebuttonbn->closingtime);
+                            if( $durationtime > 0 )
+                                $bbbsession['welcome'] .= '<br><br>'.str_replace("%duration%", ''.$durationtime, get_string('bbbdurationwarning', 'bigbluebuttonbn'));
                         } else {
-                            print_error( $error_key, 'bigbluebuttonbn' );
+                            $durationtime = 0;
+                        }
+                        /// Execute the create command
+                        $response = bigbluebuttonbn_getCreateMeetingArray(
+                                $bbbsession['meetingname'],
+                                $bbbsession['meetingid'],
+                                $bbbsession['welcome'],
+                                $bbbsession['modPW'],
+                                $bbbsession['viewerPW'],
+                                $bbbsession['shared_secret'],
+                                $bbbsession['endpoint'],
+                                $bbbsession['logoutURL'],
+                                $bbbsession['record']? 'true': 'false',
+                                $durationtime,
+                                $bbbsession['voicebridge'],
+                                $metadata,
+                                $bbbsession['presentation']['name'],
+                                $bbbsession['presentation']['url']
+                        );
+
+                        if (!$response) {
+                            // If the server is unreachable, then prompts the user of the necessary action
+                            if ( $bbbsession['administrator'] ) {
+                                print_error( 'view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
+                            } else if ( $bbbsession['moderator'] ) {
+                                print_error( 'view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
+                            } else {
+                                print_error( 'view_error_unable_join_student', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
+                            }
+
+                        } else if( $response['returncode'] == "FAILED" ) {
+                            // The meeting was not created
+                            $error_key = bigbluebuttonbn_get_error_key( $response['messageKey'], 'view_error_create' );
+                            if( !$error_key ) {
+                                print_error( $response['message'], 'bigbluebuttonbn' );
+                            } else {
+                                print_error( $error_key, 'bigbluebuttonbn' );
+                            }
+
+                        } else if ($response['hasBeenForciblyEnded'] == "true"){
+                            print_error( get_string( 'index_error_forciblyended', 'bigbluebuttonbn' ));
+
+                        } else { ///////////////Everything is ok /////////////////////
+                            /// Moodle event logger: Create an event for meeting created
+                            bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_CREATED, $bigbluebuttonbn, $context, $cm);
+                            /// Internal logger: Instert a record with the meeting created
+                            bigbluebuttonbn_log($bbbsession, 'Create');
+                            /// Since the meeting is already running, we just join the session
+                            bigbluebutton_bbb_view_execute_join($bbbsession, $cm, $context, $bigbluebuttonbn);
                         }
 
-                    } else if ($response['hasBeenForciblyEnded'] == "true"){
-                        print_error( get_string( 'index_error_forciblyended', 'bigbluebuttonbn' ));
-
-                    } else { ///////////////Everything is ok /////////////////////
-                        /// Moodle event logger: Create an event for meeting created
-                        bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_CREATED, $bigbluebuttonbn, $context, $cm);
-                        /// Internal logger: Instert a record with the meeting created
-                        bigbluebuttonbn_log($bbbsession, 'Create');
-                        /// Since the meeting is already running, we just join the session
-                        bigbluebutton_bbb_view_execute_join($bbbsession, $cm, $context, $bigbluebuttonbn);
+                    } else {
+                        header('Location: '.$bbbsession['logoutURL'] );
                     }
-
-                } else {
-                    header('Location: '.$bbbsession['logoutURL'] );
                 }
-            }
 
-        } else {
-            print_error( 'view_error_unable_join', 'bigbluebuttonbn' );
-        }
-        break;
-    default:
-        bigbluebutton_bbb_view_close_window();
+            } else {
+                print_error( 'view_error_unable_join', 'bigbluebuttonbn' );
+            }
+            break;
+        default:
+            bigbluebutton_bbb_view_close_window();
 }
 
 

Spacing +36 added lines, -36 removed lines

@@ -8,11 +8,11 @@ 
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v2 or later
  */
 
-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');
 
-$id = optional_param('id', 0, PARAM_INT);  // course_module ID, or
-$bn = optional_param('bn', 0, PARAM_INT);  // bigbluebuttonbn instance ID
+$id = optional_param('id', 0, PARAM_INT); // course_module ID, or
+$bn = optional_param('bn', 0, PARAM_INT); // bigbluebuttonbn instance ID
 $action = required_param('action', PARAM_TEXT);
 $name = optional_param('name', '', PARAM_TEXT);
 $description = optional_param('description', '', PARAM_TEXT);
@@ -39,13 +39,13 @@ 
 $PAGE->set_cacheable(false);
 $PAGE->blocks->show_only_fake_blocks();
 
-if ( isset($SESSION) && isset($SESSION->bigbluebuttonbn_bbbsession)) {
+if (isset($SESSION) && isset($SESSION->bigbluebuttonbn_bbbsession)) {
     require_login($course, true, $cm);
     $bbbsession = $SESSION->bigbluebuttonbn_bbbsession;
 }
 switch (strtolower($action)) {
     case 'logout':
-        if ( isset($bbbsession) && !is_null($bbbsession) ) {
+        if (isset($bbbsession) && !is_null($bbbsession)) {
             /// Moodle event logger: Create an event for meeting left
             bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_LEFT, $bigbluebuttonbn, $context, $cm);
 
@@ -60,15 +60,15 @@ 
         }
         break;
     case 'join':
-        if ( isset($bbbsession) && !is_null($bbbsession) ) {
+        if (isset($bbbsession) && !is_null($bbbsession)) {
             //See if the session is in progress
-            if( bigbluebuttonbn_isMeetingRunning( $bbbsession['meetingid'], $bbbsession['endpoint'], $bbbsession['shared_secret'] ) ) {
+            if (bigbluebuttonbn_isMeetingRunning($bbbsession['meetingid'], $bbbsession['endpoint'], $bbbsession['shared_secret'])) {
                 /// Since the meeting is already running, we just join the session
                 bigbluebutton_bbb_view_execute_join($bbbsession, $cm, $context, $bigbluebuttonbn);
 
             } else {
                 // If user is administrator, moderator or if is viewer and no waiting is required
-                if( $bbbsession['administrator'] || $bbbsession['moderator'] || !$bbbsession['wait'] ) {
+                if ($bbbsession['administrator'] || $bbbsession['moderator'] || !$bbbsession['wait']) {
                     /// Prepare the metadata
                     $metadata = array("meta_bn-origin" => $bbbsession['origin'],
                             "meta_bbb-origin-version" => $bbbsession['originVersion'],
@@ -76,20 +76,20 @@ 
                             "meta_bbb-origin-server-common-name" => $bbbsession['originServerCommonName'],
                             "meta_bbb-origin-tag" => $bbbsession['originTag'],
                             "meta_bbb-context" => $bbbsession['course']->fullname,
-                            "meta_bbb-recording-name" => (isset($name) && $name != '')? $name: $bbbsession['contextActivityName'],
-                            "meta_bbb-recording-description" => (isset($description) && $description != '')? $description: $bbbsession['contextActivityDescription'],
-                            "meta_bbb-recording-tags" => (isset($tags) && $tags != '')? $tags: $bbbsession['contextActivityTags'],
+                            "meta_bbb-recording-name" => (isset($name) && $name != '') ? $name : $bbbsession['contextActivityName'],
+                            "meta_bbb-recording-description" => (isset($description) && $description != '') ? $description : $bbbsession['contextActivityDescription'],
+                            "meta_bbb-recording-tags" => (isset($tags) && $tags != '') ? $tags : $bbbsession['contextActivityTags'],
                     );
 
-                    if ( bigbluebuttonbn_server_offers_bn_capabilities() && bigbluebuttonbn_get_cfg_recordingready_enabled() ) {
+                    if (bigbluebuttonbn_server_offers_bn_capabilities() && bigbluebuttonbn_get_cfg_recordingready_enabled()) {
                         $metadata["meta_bn-recording-ready-url"] = $bbbsession['recordingReadyURL'];
                     }
 
                     /// Set the duration for the meeting
-                    if ( bigbluebuttonbn_get_cfg_scheduled_duration_enabled() ) {
+                    if (bigbluebuttonbn_get_cfg_scheduled_duration_enabled()) {
                         $durationtime = bigbluebuttonbn_get_duration($bigbluebuttonbn->openingtime, $bigbluebuttonbn->closingtime);
-                        if( $durationtime > 0 )
-                            $bbbsession['welcome'] .= '<br><br>'.str_replace("%duration%", ''.$durationtime, get_string('bbbdurationwarning', 'bigbluebuttonbn'));
+                        if ($durationtime > 0)
+                            $bbbsession['welcome'] .= '<br><br>' . str_replace("%duration%", '' . $durationtime, get_string('bbbdurationwarning', 'bigbluebuttonbn'));
                     } else {
                         $durationtime = 0;
                     }
@@ -103,7 +103,7 @@ 
                             $bbbsession['shared_secret'],
                             $bbbsession['endpoint'],
                             $bbbsession['logoutURL'],
-                            $bbbsession['record']? 'true': 'false',
+                            $bbbsession['record'] ? 'true' : 'false',
                             $durationtime,
                             $bbbsession['voicebridge'],
                             $metadata,
@@ -113,25 +113,25 @@ 
 
                     if (!$response) {
                         // If the server is unreachable, then prompts the user of the necessary action
-                        if ( $bbbsession['administrator'] ) {
-                            print_error( 'view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
-                        } else if ( $bbbsession['moderator'] ) {
-                            print_error( 'view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
+                        if ($bbbsession['administrator']) {
+                            print_error('view_error_unable_join', 'bigbluebuttonbn', $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
+                        } else if ($bbbsession['moderator']) {
+                            print_error('view_error_unable_join_teacher', 'bigbluebuttonbn', $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
                         } else {
-                            print_error( 'view_error_unable_join_student', 'bigbluebuttonbn', $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn' );
+                            print_error('view_error_unable_join_student', 'bigbluebuttonbn', $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
                         }
 
-                    } else if( $response['returncode'] == "FAILED" ) {
+                    } else if ($response['returncode'] == "FAILED") {
                         // The meeting was not created
-                        $error_key = bigbluebuttonbn_get_error_key( $response['messageKey'], 'view_error_create' );
-                        if( !$error_key ) {
-                            print_error( $response['message'], 'bigbluebuttonbn' );
+                        $error_key = bigbluebuttonbn_get_error_key($response['messageKey'], 'view_error_create');
+                        if (!$error_key) {
+                            print_error($response['message'], 'bigbluebuttonbn');
                         } else {
-                            print_error( $error_key, 'bigbluebuttonbn' );
+                            print_error($error_key, 'bigbluebuttonbn');
                         }
 
-                    } else if ($response['hasBeenForciblyEnded'] == "true"){
-                        print_error( get_string( 'index_error_forciblyended', 'bigbluebuttonbn' ));
+                    } else if ($response['hasBeenForciblyEnded'] == "true") {
+                        print_error(get_string('index_error_forciblyended', 'bigbluebuttonbn'));
 
                     } else { ///////////////Everything is ok /////////////////////
                         /// Moodle event logger: Create an event for meeting created
@@ -143,12 +143,12 @@ 
                     }
 
                 } else {
-                    header('Location: '.$bbbsession['logoutURL'] );
+                    header('Location: ' . $bbbsession['logoutURL']);
                 }
             }
 
         } else {
-            print_error( 'view_error_unable_join', 'bigbluebuttonbn' );
+            print_error('view_error_unable_join', 'bigbluebuttonbn');
         }
         break;
     default:
@@ -172,9 +172,9 @@ 
 function bigbluebutton_bbb_view_execute_join($bbbsession, $cm, $context, $bigbluebuttonbn) {
     //// Update the cache
     $meeting_info = bigbluebuttonbn_bbb_broker_get_meeting_info($bbbsession['meetingid'], $bbbsession['modPW'], true);
-    if( $bbbsession['userlimit'] == 0 || intval($meeting_info['participantCount']) < $bbbsession['userlimit']  ) {
+    if ($bbbsession['userlimit'] == 0 || intval($meeting_info['participantCount']) < $bbbsession['userlimit']) {
         //// Build the URL
-        if( $bbbsession['administrator'] || $bbbsession['moderator'] ) {
+        if ($bbbsession['administrator'] || $bbbsession['moderator']) {
             $password = $bbbsession['modPW'];
         } else {
             $password = $bbbsession['viewerPW'];
@@ -185,11 +185,11 @@ 
         /// Internal logger: Instert a record with the meeting created
         bigbluebuttonbn_log($bbbsession, 'Join');
         //// Before executing the redirect, increment the number of participants
-        bigbluebuttonbn_bbb_broker_participant_joined($bbbsession['meetingid'], ($bbbsession['administrator'] || $bbbsession['moderator']) );
+        bigbluebuttonbn_bbb_broker_participant_joined($bbbsession['meetingid'], ($bbbsession['administrator'] || $bbbsession['moderator']));
         //// Execute the redirect
-        header('Location: '.$join_url );
+        header('Location: ' . $join_url);
 
     } else {
-        header('Location: '.$bbbsession['logoutURL'] );
+        header('Location: ' . $bbbsession['logoutURL']);
     }
 }
\ No newline at end of file

Braces +3 added lines, -2 removed lines

@@ -88,8 +88,9 @@
                     /// Set the duration for the meeting
                     if ( bigbluebuttonbn_get_cfg_scheduled_duration_enabled() ) {
                         $durationtime = bigbluebuttonbn_get_duration($bigbluebuttonbn->openingtime, $bigbluebuttonbn->closingtime);
-                        if( $durationtime > 0 )
-                            $bbbsession['welcome'] .= '<br><br>'.str_replace("%duration%", ''.$durationtime, get_string('bbbdurationwarning', 'bigbluebuttonbn'));
+                        if( $durationtime > 0 ) {
+                                                    $bbbsession['welcome'] .= '<br><br>'.str_replace("%duration%", ''.$durationtime, get_string('bbbdurationwarning', 'bigbluebuttonbn'));
+                        }
                     } else {
                         $durationtime = 0;
                     }

db/upgrade.php

Spacing +41 added lines, -41 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
@@ -238,8 +238,8 @@ 
         $field = new xmldb_field('userid');
         //$field->set_attributes(XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, 0, 'bigbluebuttonbnid');
         $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');

index.php

Indentation +1 added lines, -1 removed lines

@@ -93,7 +93,7 @@
         } else {
             $getArray = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getEndMeetingURL( $meetingID, $modPW, $endpoint, $shared_secret ));
         }
-	   redirect('index.php?id='.$id);
+        redirect('index.php?id='.$id);
     }
 }
 

Spacing +50 added lines, -50 removed lines

@@ -8,12 +8,12 @@ 
  * @copyright 2010-2015 Blindside Networks Inc.
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v2 or later
  */
-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');
 
-$id = required_param('id', PARAM_INT);      // Course Module ID, or
-$a  = optional_param('a', 0, PARAM_INT);    // bigbluebuttonbn instance ID
-$g  = optional_param('g', 0, PARAM_INT);    // group instance ID
+$id = required_param('id', PARAM_INT); // Course Module ID, or
+$a  = optional_param('a', 0, PARAM_INT); // bigbluebuttonbn instance ID
+$g  = optional_param('g', 0, PARAM_INT); // group instance ID
 
 if ($id) {
     $course = $DB->get_record('course', array('id'=>$id), '*', MUST_EXIST);
@@ -36,7 +36,7 @@ 
 $PAGE->navbar->add(get_string('modulename', 'bigbluebuttonbn'), "index.php?id=$course->id");
 
 /// Get all the appropriate data
-if (! $bigbluebuttonbns = get_all_instances_in_course('bigbluebuttonbn', $course)) {
+if (!$bigbluebuttonbns = get_all_instances_in_course('bigbluebuttonbn', $course)) {
     notice('There are no instances of bigbluebuttonbn', "../../course/view.php?id=$course->id");
 }
 
@@ -44,18 +44,18 @@ 
 $timenow            = time();
 $strweek            = get_string('week');
 $strtopic           = get_string('topic');
-$heading_name       = get_string('index_heading_name', 'bigbluebuttonbn' );
-$heading_group      = get_string('index_heading_group', 'bigbluebuttonbn' );
+$heading_name       = get_string('index_heading_name', 'bigbluebuttonbn');
+$heading_group      = get_string('index_heading_group', 'bigbluebuttonbn');
 $heading_users      = get_string('index_heading_users', 'bigbluebuttonbn');
 $heading_viewer     = get_string('index_heading_viewer', 'bigbluebuttonbn');
-$heading_moderator  = get_string('index_heading_moderator', 'bigbluebuttonbn' );
-$heading_actions    = get_string('index_heading_actions', 'bigbluebuttonbn' );
-$heading_recording  = get_string('index_heading_recording', 'bigbluebuttonbn' );
+$heading_moderator  = get_string('index_heading_moderator', 'bigbluebuttonbn');
+$heading_actions    = get_string('index_heading_actions', 'bigbluebuttonbn');
+$heading_recording  = get_string('index_heading_recording', 'bigbluebuttonbn');
 
 $table = new html_table();
 
-$table->head  = array ($strweek, $heading_name, $heading_group, $heading_users, $heading_viewer, $heading_moderator, $heading_recording, $heading_actions );
-$table->align = array ('center', 'left', 'center', 'center', 'center',  'center', 'center' );
+$table->head  = array($strweek, $heading_name, $heading_group, $heading_users, $heading_viewer, $heading_moderator, $heading_recording, $heading_actions);
+$table->align = array('center', 'left', 'center', 'center', 'center', 'center', 'center');
 
 $endpoint = bigbluebuttonbn_get_cfg_server_url();
 $shared_secret = bigbluebuttonbn_get_cfg_shared_secret();
@@ -66,14 +66,14 @@ 
     //
     // A request to end the meeting
     //
-    if (! $bigbluebuttonbn = $DB->get_record('bigbluebuttonbn', array('id' => $a), '*', MUST_EXIST) ) {
+    if (!$bigbluebuttonbn = $DB->get_record('bigbluebuttonbn', array('id' => $a), '*', MUST_EXIST)) {
         print_error("BigBlueButton ID $a is incorrect");
     }
     $course = $DB->get_record('course', array('id' => $bigbluebuttonbn->course), '*', MUST_EXIST);
     $cm = get_coursemodule_from_instance('bigbluebuttonbn', $bigbluebuttonbn->id, $course->id, false, MUST_EXIST);
 
     //User roles
-    if( $bigbluebuttonbn->participants == null || $bigbluebuttonbn->participants == "" || $bigbluebuttonbn->participants == "[]" ){
+    if ($bigbluebuttonbn->participants == null || $bigbluebuttonbn->participants == "" || $bigbluebuttonbn->participants == "[]") {
         //The room that is being used comes from a previous version
         $moderator = has_capability('mod/bigbluebuttonbn:moderate', $context);
     } else {
@@ -81,19 +81,19 @@ 
     }
     $administrator = has_capability('moodle/category:manage', $context);
 
-    if( $moderator || $administrator ) {
+    if ($moderator || $administrator) {
         bigbluebuttonbn_event_log(BIGBLUEBUTTON_EVENT_MEETING_ENDED, $bigbluebuttonbn, $context, $cm);
 
         echo get_string('index_ending', 'bigbluebuttonbn');
 
-        $meetingID = $bigbluebuttonbn->meetingid.'-'.$course->id.'-'.$bigbluebuttonbn->id;
+        $meetingID = $bigbluebuttonbn->meetingid . '-' . $course->id . '-' . $bigbluebuttonbn->id;
         $modPW = $bigbluebuttonbn->moderatorpass;
-        if( $g != '0'  ) {
-            $getArray = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getEndMeetingURL( $meetingID.'['.$g.']', $modPW, $endpoint, $shared_secret ) );
+        if ($g != '0') {
+            $getArray = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getEndMeetingURL($meetingID . '[' . $g . ']', $modPW, $endpoint, $shared_secret));
         } else {
-            $getArray = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getEndMeetingURL( $meetingID, $modPW, $endpoint, $shared_secret ));
+            $getArray = bigbluebuttonbn_wrap_xml_load_file(bigbluebuttonbn_getEndMeetingURL($meetingID, $modPW, $endpoint, $shared_secret));
         }
-	   redirect('index.php?id='.$id);
+	   redirect('index.php?id=' . $id);
     }
 }
 
@@ -101,7 +101,7 @@ 
     $cm = get_coursemodule_from_id('bigbluebuttonbn', $bigbluebuttonbn->coursemodule, 0, false, MUST_EXIST);
 
     //User roles
-    if( $bigbluebuttonbn->participants == null || $bigbluebuttonbn->participants == "" || $bigbluebuttonbn->participants == "[]" ){
+    if ($bigbluebuttonbn->participants == null || $bigbluebuttonbn->participants == "" || $bigbluebuttonbn->participants == "[]") {
         //The room that is being used comes from a previous version
         $moderator = has_capability('mod/bigbluebuttonbn:moderate', $context);
     } else {
@@ -109,11 +109,11 @@ 
     }
     $administrator = has_capability('moodle/category:manage', $context);
 
-    if ( groups_get_activity_groupmode($cm) > 0 ){
-        $table->data[] = displayBigBlueButtonRooms($endpoint, $shared_secret, ($administrator || $moderator), $course, $bigbluebuttonbn, (object) array('id'=>0, 'name'=>get_string('allparticipants')));
+    if (groups_get_activity_groupmode($cm) > 0) {
+        $table->data[] = displayBigBlueButtonRooms($endpoint, $shared_secret, ($administrator || $moderator), $course, $bigbluebuttonbn, (object)array('id'=>0, 'name'=>get_string('allparticipants')));
         $groups = groups_get_activity_allowed_groups($cm);
-        if( isset($groups)) {
-            foreach( $groups as $group){
+        if (isset($groups)) {
+            foreach ($groups as $group) {
                 $table->data[] = displayBigBlueButtonRooms($endpoint, $shared_secret, ($administrator || $moderator), $course, $bigbluebuttonbn, $group);
             }
         }
@@ -130,7 +130,7 @@ 
 echo $OUTPUT->footer();
 
 /// Functions
-function displayBigBlueButtonRooms($endpoint, $shared_secret, $moderator, $course, $bigbluebuttonbn, $groupObj = null ){
+function displayBigBlueButtonRooms($endpoint, $shared_secret, $moderator, $course, $bigbluebuttonbn, $groupObj = null) {
     $joinURL = null;
     $group = "-";
     $users = "-";
@@ -140,22 +140,22 @@ 
     $moderatorList = "-";
     $recording = "-";
 
-    if ( !$bigbluebuttonbn->visible ) {
+    if (!$bigbluebuttonbn->visible) {
         // Nothing to do
     } else {
         $modPW = $bigbluebuttonbn->moderatorpass;
         $attPW = $bigbluebuttonbn->viewerpass;
 
-        $meetingID = $bigbluebuttonbn->meetingid.'-'.$course->id.'-'.$bigbluebuttonbn->id;
+        $meetingID = $bigbluebuttonbn->meetingid . '-' . $course->id . '-' . $bigbluebuttonbn->id;
         //
         // Output Users in the meeting
         //
-        if( $groupObj == null ){
-            $meetingInfo = bigbluebuttonbn_getMeetingInfoArray( $meetingID, $modPW, $endpoint, $shared_secret );
-            $joinURL = '<a href="view.php?id='.$bigbluebuttonbn->coursemodule.'">'.format_string($bigbluebuttonbn->name).'</a>';
+        if ($groupObj == null) {
+            $meetingInfo = bigbluebuttonbn_getMeetingInfoArray($meetingID, $modPW, $endpoint, $shared_secret);
+            $joinURL = '<a href="view.php?id=' . $bigbluebuttonbn->coursemodule . '">' . format_string($bigbluebuttonbn->name) . '</a>';
         } else {
-            $meetingInfo = bigbluebuttonbn_getMeetingInfoArray( $meetingID.'['.$groupObj->id.']', $modPW, $endpoint, $shared_secret );
-            $joinURL = '<a href="view.php?id='.$bigbluebuttonbn->coursemodule.'&group='.$groupObj->id.'">'.format_string($bigbluebuttonbn->name).'</a>';
+            $meetingInfo = bigbluebuttonbn_getMeetingInfoArray($meetingID . '[' . $groupObj->id . ']', $modPW, $endpoint, $shared_secret);
+            $joinURL = '<a href="view.php?id=' . $bigbluebuttonbn->coursemodule . '&group=' . $groupObj->id . '">' . format_string($bigbluebuttonbn->name) . '</a>';
             $group = $groupObj->name;
         }
         
@@ -163,7 +163,7 @@ 
             //
             // The server was unreachable
             //
-            print_error( get_string( 'index_error_unable_display', 'bigbluebuttonbn' ));
+            print_error(get_string('index_error_unable_display', 'bigbluebuttonbn'));
             return;
         }
 
@@ -172,7 +172,7 @@ 
             // There was an error returned
             //
             if ($meetingInfo['messageKey'] == "checksumError") {
-                print_error( get_string( 'index_error_checksum', 'bigbluebuttonbn' ));
+                print_error(get_string('index_error_checksum', 'bigbluebuttonbn'));
                 return;
             }
 
@@ -184,39 +184,39 @@ 
                 //
                 // There was an error
                 //
-                $users = $meetingInfo['messageKey'].": ".$meetingInfo['message'];
+                $users = $meetingInfo['messageKey'] . ": " . $meetingInfo['message'];
             }
         } else {
             //
             // The meeting info was returned
             //
             if ($meetingInfo['running'] == 'true') {
-                if ( $moderator ) {
-                    if( $groupObj == null ) {
-                        $actions = '<form name="form1" method="post" action=""><INPUT type="hidden" name="id" value="'.$course->id.'"><INPUT type="hidden" name="a" value="'.$bigbluebuttonbn->id.'"><INPUT type="submit" name="submit" value="end" onclick="return confirm(\''. get_string('index_confirm_end', 'bigbluebuttonbn' ).'\')"></form>';
+                if ($moderator) {
+                    if ($groupObj == null) {
+                        $actions = '<form name="form1" method="post" action=""><INPUT type="hidden" name="id" value="' . $course->id . '"><INPUT type="hidden" name="a" value="' . $bigbluebuttonbn->id . '"><INPUT type="submit" name="submit" value="end" onclick="return confirm(\'' . get_string('index_confirm_end', 'bigbluebuttonbn') . '\')"></form>';
                     } else {
-                        $actions = '<form name="form1" method="post" action=""><INPUT type="hidden" name="id" value="'.$course->id.'"><INPUT type="hidden" name="a" value="'.$bigbluebuttonbn->id.'"><INPUT type="hidden" name="g" value="'.$groupObj->id.'"><INPUT type="submit" name="submit" value="end" onclick="return confirm(\''. get_string('index_confirm_end', 'bigbluebuttonbn' ).'\')"></form>';
+                        $actions = '<form name="form1" method="post" action=""><INPUT type="hidden" name="id" value="' . $course->id . '"><INPUT type="hidden" name="a" value="' . $bigbluebuttonbn->id . '"><INPUT type="hidden" name="g" value="' . $groupObj->id . '"><INPUT type="submit" name="submit" value="end" onclick="return confirm(\'' . get_string('index_confirm_end', 'bigbluebuttonbn') . '\')"></form>';
                     }
                 }
-                if ( isset($meetingInfo['recording']) && $meetingInfo['recording'] == 'true' ){ // if it has been set when meeting created, set the variable on/off
-                    $recording = get_string('index_enabled', 'bigbluebuttonbn' );
+                if (isset($meetingInfo['recording']) && $meetingInfo['recording'] == 'true') { // if it has been set when meeting created, set the variable on/off
+                    $recording = get_string('index_enabled', 'bigbluebuttonbn');
                 }
                  
                 $xml = $meetingInfo['attendees'];
-                if (count( $xml ) && count( $xml->attendee ) ) {
-                    $users = count( $xml->attendee );
+                if (count($xml) && count($xml->attendee)) {
+                    $users = count($xml->attendee);
                     $viewer_count = 0;
                     $moderator_count = 0;
-                    foreach ( $xml->attendee as $attendee ) {
-                        if ($attendee->role == "MODERATOR" ) {
-                            if ( $viewer_count++ > 0 ) {
+                    foreach ($xml->attendee as $attendee) {
+                        if ($attendee->role == "MODERATOR") {
+                            if ($viewer_count++ > 0) {
                                 $moderatorList .= ", ";
                             } else {
                                 $moderatorList = "";
                             }
                             $moderatorList .= $attendee->fullName;
                         } else {
-                            if ( $moderator_count++ > 0 ) {
+                            if ($moderator_count++ > 0) {
                                 $viewerList .= ", ";
                             } else {
                                 $viewerList = "";
@@ -228,6 +228,6 @@ 
             }
         }
 
-        return array ($bigbluebuttonbn->section, $joinURL, $group, $users, $viewerList, $moderatorList, $recording, $actions );
+        return array($bigbluebuttonbn->section, $joinURL, $group, $users, $viewerList, $moderatorList, $recording, $actions);
     }
 }
\ No newline at end of file

version.php

Spacing +1 added lines, -1 removed lines

@@ -15,5 +15,5 @@
 $plugin->requires = 2013111800;
 $plugin->cron     = 0;
 $plugin->component = 'mod_bigbluebuttonbn';
-$plugin->maturity = MATURITY_STABLE;    // [MATURITY_STABLE | MATURITY_RC | MATURITY_BETA | MATURITY_ALPHA]
+$plugin->maturity = MATURITY_STABLE; // [MATURITY_STABLE | MATURITY_RC | MATURITY_BETA | MATURITY_ALPHA]
 $plugin->release  = '2.0.4';

classes/event/bigbluebuttonbn_recording_deleted.php

Spacing +1 added lines, -1 removed lines

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