blindsidenetworks / moodle-mod_bigbluebuttonbn

lib.php

Spacing +91 added lines, -91 removed lines

@@ -13,23 +13,23 @@ 
 
 global $BIGBLUEBUTTONBN_CFG, $CFG;
 
-require_once($CFG->dirroot.'/calendar/lib.php');
-require_once($CFG->dirroot.'/message/lib.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($CFG->libdir.'/coursecatlib.php');
-require_once($CFG->libdir.'/enrollib.php');
-require_once($CFG->libdir.'/filelib.php');
-require_once($CFG->libdir.'/formslib.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 . '/mod/lti/OAuth.php');
+require_once($CFG->libdir . '/accesslib.php');
+require_once($CFG->libdir . '/completionlib.php');
+require_once($CFG->libdir . '/datalib.php');
+require_once($CFG->libdir . '/coursecatlib.php');
+require_once($CFG->libdir . '/enrollib.php');
+require_once($CFG->libdir . '/filelib.php');
+require_once($CFG->libdir . '/formslib.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();
@@ -50,7 +50,7 @@ 
 const BIGBLUEBUTTONBN_LOG_EVENT_DELETE = "Delete";
 
 function bigbluebuttonbn_supports($feature) {
-    switch($feature) {
+    switch ($feature) {
         case FEATURE_IDNUMBER:                return true;
         case FEATURE_GROUPS:                  return true;
         case FEATURE_GROUPINGS:               return true;
@@ -79,7 +79,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);
@@ -107,7 +107,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);
@@ -133,7 +133,7 @@ 
 function bigbluebuttonbn_delete_instance($id) {
     global $CFG, $DB, $USER;
 
-    if (! $bigbluebuttonbn = $DB->get_record('bigbluebuttonbn', array('id' => $id))) {
+    if (!$bigbluebuttonbn = $DB->get_record('bigbluebuttonbn', array('id' => $id))) {
         return false;
     }
 
@@ -151,11 +151,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;
     }
 
@@ -171,19 +171,19 @@ 
     $logs = $DB->get_records('bigbluebuttonbn_logs', array('bigbluebuttonbnid' => $bigbluebuttonbn->id, 'log' => BIGBLUEBUTTONBN_LOG_EVENT_CREATE));
     error_log(json_encode($logs));
     $has_recordings = 'false';
-    if (! empty($logs) ) {
+    if (!empty($logs)) {
         error_log("IS not empty");
-        foreach ( $logs as $l ) {
+        foreach ($logs as $l) {
             error_log(json_encode($l));
             $meta = json_decode($l->meta);
-            if ( $meta->record ) {
+            if ($meta->record) {
                 $has_recordings = 'true';
             }
         }
     }
     $log->meta = "{\"has_recordings\":{$has_recordings}}";
 
-    if (! $returnid = $DB->insert_record('bigbluebuttonbn_logs', $log)) {
+    if (!$returnid = $DB->insert_record('bigbluebuttonbn_logs', $log)) {
         $result = false;
     }
 
@@ -221,7 +221,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
 }
 
 /**
@@ -236,7 +236,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) {
 }
 
 /**
@@ -254,7 +254,7 @@ 
  *
  * @return boolean
  **/
-function bigbluebuttonbn_cron () {
+function bigbluebuttonbn_cron() {
     return true;
 }
 
@@ -356,21 +356,21 @@ 
     }
 
     $strbigbluebuttonbn = get_string('modulename', 'bigbluebuttonbn');
-    $strnextsession  = get_string('nextsession', 'bigbluebuttonbn');
+    $strnextsession = get_string('nextsession', 'bigbluebuttonbn');
 
     foreach ($bigbluebuttonbns as $bigbluebuttonbn) {
         $now = time();
-        if ( $bigbluebuttonbn->openingtime and (!$bigbluebuttonbn->closingtime or $bigbluebuttonbn->closingtime > $now)) { // A bigbluebuttonbn is scheduled.
-            $str = '<div class="bigbluebuttonbn overview"><div class="name">'.
-                 $strbigbluebuttonbn.': <a '.($bigbluebuttonbn->visible ? '' : ' class="dimmed"').
-                 ' href="'.$CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$bigbluebuttonbn->coursemodule.'">'.
-                 $bigbluebuttonbn->name.'</a></div>';
-            if ( $bigbluebuttonbn->openingtime > $now ) {
-                $str .= '<div class="info">'.get_string('starts_at', 'bigbluebuttonbn').': '.userdate($bigbluebuttonbn->openingtime).'</div>';
+        if ($bigbluebuttonbn->openingtime and (!$bigbluebuttonbn->closingtime or $bigbluebuttonbn->closingtime > $now)) { // A bigbluebuttonbn is scheduled.
+            $str = '<div class="bigbluebuttonbn overview"><div class="name">' .
+                 $strbigbluebuttonbn . ': <a ' . ($bigbluebuttonbn->visible ? '' : ' class="dimmed"') .
+                 ' href="' . $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $bigbluebuttonbn->coursemodule . '">' .
+                 $bigbluebuttonbn->name . '</a></div>';
+            if ($bigbluebuttonbn->openingtime > $now) {
+                $str .= '<div class="info">' . get_string('starts_at', 'bigbluebuttonbn') . ': ' . userdate($bigbluebuttonbn->openingtime) . '</div>';
             } else {
-                $str .= '<div class="info">'.get_string('started_at', 'bigbluebuttonbn').': '.userdate($bigbluebuttonbn->openingtime).'</div>';
+                $str .= '<div class="info">' . get_string('started_at', 'bigbluebuttonbn') . ': ' . userdate($bigbluebuttonbn->openingtime) . '</div>';
             }
-            $str .= '<div class="info">'.get_string('ends_at', 'bigbluebuttonbn').': '.userdate($bigbluebuttonbn->closingtime).'</div></div>';
+            $str .= '<div class="info">' . get_string('ends_at', 'bigbluebuttonbn') . ': ' . userdate($bigbluebuttonbn->closingtime) . '</div></div>';
 
             if (empty($htmlarray[$bigbluebuttonbn->course]['bigbluebuttonbn'])) {
                 $htmlarray[$bigbluebuttonbn->course]['bigbluebuttonbn'] = $str;
@@ -395,7 +395,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;
     }
 
@@ -421,10 +421,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);
         }
@@ -433,13 +433,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);
@@ -458,8 +458,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 {
@@ -468,7 +468,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;
@@ -478,13 +478,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 {
@@ -495,7 +495,7 @@ 
         $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();
 
@@ -503,35 +503,35 @@ 
         $msg->action = $action;
         $msg->activity_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 = fullname($USER);
 
-        $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);
@@ -582,7 +582,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) {
@@ -598,21 +598,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);
         }
@@ -630,7 +630,7 @@ 
     }
 
     if ($filearea === 'presentation') {
-        $fullpath = "/$context->id/mod_bigbluebuttonbn/$filearea/0/".$filename;
+        $fullpath = "/$context->id/mod_bigbluebuttonbn/$filearea/0/" . $filename;
     } else {
         return false;
     }
@@ -664,10 +664,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());
@@ -686,22 +686,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);
@@ -712,17 +712,17 @@ 
     $msg->user_name = fullname($sender);
     $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 ){
+    foreach ($users as $user) {
+        if ($user->id != $sender->id) {
             $messageid = message_post_message($sender, $user, $message, FORMAT_HTML);
             if (!empty($messageid)) {
-                error_log("Msg to ".$msg->user_name." was sent.");
+                error_log("Msg to " . $msg->user_name . " was sent.");
             } else {
-                error_log("Msg to ".$msg->user_name." was NOT sent.");
+                error_log("Msg to " . $msg->user_name . " was NOT sent.");
             }
         }
     }
@@ -732,7 +732,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 {
@@ -747,7 +747,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 {
@@ -760,10 +760,10 @@ 
 
 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'));
 }