Inspection of "v2.1.0: Bumped version and updated maturity" - blindsidenetworks/moodle-mod_bigbluebuttonbn - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 16d8be...175c16 )

by Jesus

created 2016-08-03 14:20 UTC

Status

Spacing +339 added lines, -339 removed lines patch added patch discarded remove patch

@@ -14,7 +14,7 @@  discard block
 block discarded – undo
 
 global $BIGBLUEBUTTONBN_CFG, $CFG;
 
-require_once(dirname(__FILE__).'/lib.php');
+require_once(dirname(__FILE__) . '/lib.php');
 
 const BIGBLUEBUTTONBN_FORCED = true;
 
@@ -33,21 +33,21 @@  discard block
 block discarded – undo
 const BIGBLUEBUTTON_EVENT_RECORDING_PUBLISHED = 'recording_published';
 const BIGBLUEBUTTON_EVENT_RECORDING_UNPUBLISHED = 'recording_unpublished';
 
-function bigbluebuttonbn_logs(array $bbbsession, $event, array $overrides = [], $meta = NULL ) {
+function bigbluebuttonbn_logs(array $bbbsession, $event, array $overrides = [], $meta = NULL) {
     global $DB;
 
     $log = new stdClass();
 
-    $log->courseid = isset($overrides['courseid'])? $overrides['courseid']: $bbbsession['course']->id;
-    $log->bigbluebuttonbnid = isset($overrides['bigbluebuttonbnid'])? $overrides['bigbluebuttonbnid']: $bbbsession['bigbluebuttonbn']->id;
-    $log->userid = isset($overrides['userid'])? $overrides['userid']: $bbbsession['userID'];
-    $log->meetingid = isset($overrides['meetingid'])? $overrides['meetingid']: $bbbsession['meetingid'];
-    $log->timecreated = isset($overrides['timecreated'])? $overrides['timecreated']: time();
+    $log->courseid = isset($overrides['courseid']) ? $overrides['courseid'] : $bbbsession['course']->id;
+    $log->bigbluebuttonbnid = isset($overrides['bigbluebuttonbnid']) ? $overrides['bigbluebuttonbnid'] : $bbbsession['bigbluebuttonbn']->id;
+    $log->userid = isset($overrides['userid']) ? $overrides['userid'] : $bbbsession['userID'];
+    $log->meetingid = isset($overrides['meetingid']) ? $overrides['meetingid'] : $bbbsession['meetingid'];
+    $log->timecreated = isset($overrides['timecreated']) ? $overrides['timecreated'] : time();
     $log->log = $event;
-    if ( isset($meta) ) {
+    if (isset($meta)) {
         $log->meta = $meta;
-    } else if( $event == BIGBLUEBUTTONBN_LOG_EVENT_CREATE) {
-        $log->meta = '{"record":'.($bbbsession['record']? 'true': 'false').'}';
+    } else if ($event == BIGBLUEBUTTONBN_LOG_EVENT_CREATE) {
+        $log->meta = '{"record":' . ($bbbsession['record'] ? 'true' : 'false') . '}';
     }
 
     $returnid = $DB->insert_record('bigbluebuttonbn_logs', $log);
@@ -56,127 +56,127 @@  discard block
 block discarded – undo
  ////////////////////////////
 //  BigBlueButton API Calls  //
  ////////////////////////////
-function bigbluebuttonbn_getJoinURL( $meetingID, $userName, $PW, $SALT, $URL, $logoutURL ) {
-    $url_join = $URL."api/join?";
-    $params = 'meetingID='.urlencode($meetingID).'&fullName='.urlencode($userName).'&password='.urlencode($PW).'&logoutURL='.urlencode($logoutURL);
-    $url = $url_join.$params.'&checksum='.sha1("join".$params.$SALT);
+function bigbluebuttonbn_getJoinURL($meetingID, $userName, $PW, $SALT, $URL, $logoutURL) {
+    $url_join = $URL . "api/join?";
+    $params = 'meetingID=' . urlencode($meetingID) . '&fullName=' . urlencode($userName) . '&password=' . urlencode($PW) . '&logoutURL=' . urlencode($logoutURL);
+    $url = $url_join . $params . '&checksum=' . sha1("join" . $params . $SALT);
     return $url;
 }
 
-function bigbluebuttonbn_getCreateMeetingURL($name, $meetingID, $attendeePW, $moderatorPW, $welcome, $logoutURL, $SALT, $URL, $record = 'false', $duration=0, $voiceBridge=0, $maxParticipants=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, $maxParticipants = 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;
 
     $maxParticipants = intval($maxParticipants);
-    if( $maxParticipants > 0 )
-        $params .= '&maxParticipants='.$maxParticipants;
+    if ($maxParticipants > 0)
+        $params .= '&maxParticipants=' . $maxParticipants;
 
-    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);
     }
 
-    $url = $url_create.$params.'&checksum='.sha1("create".$params.$SALT);
+    $url = $url_create . $params . '&checksum=' . sha1("create" . $params . $SALT);
     return $url;
 }
 
-function bigbluebuttonbn_getIsMeetingRunningURL( $meetingID, $URL, $SALT ) {
-    $base_url = $URL."api/isMeetingRunning?";
-    $params = 'meetingID='.urlencode($meetingID);
-    $url = $base_url.$params.'&checksum='.sha1("isMeetingRunning".$params.$SALT);
+function bigbluebuttonbn_getIsMeetingRunningURL($meetingID, $URL, $SALT) {
+    $base_url = $URL . "api/isMeetingRunning?";
+    $params = 'meetingID=' . urlencode($meetingID);
+    $url = $base_url . $params . '&checksum=' . sha1("isMeetingRunning" . $params . $SALT);
     return $url;
 }
 
-function bigbluebuttonbn_getMeetingInfoURL( $meetingID, $modPW, $URL, $SALT ) {
-    $base_url = $URL."api/getMeetingInfo?";
-    $params = 'meetingID='.urlencode($meetingID).'&password='.urlencode($modPW);
-    $url = $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);
+    $url = $base_url . $params . '&checksum=' . sha1("getMeetingInfo" . $params . $SALT);
     return $url;
 }
 
-function bigbluebuttonbn_getMeetingsURL( $URL, $SALT ) {
-    $base_url = $URL."api/getMeetings?";
-    $url = $base_url.'&checksum='.sha1("getMeetings".$SALT);
+function bigbluebuttonbn_getMeetingsURL($URL, $SALT) {
+    $base_url = $URL . "api/getMeetings?";
+    $url = $base_url . '&checksum=' . sha1("getMeetings" . $SALT);
     return $url;
 }
 
-function bigbluebuttonbn_getEndMeetingURL( $meetingID, $modPW, $URL, $SALT ) {
-    $base_url = $URL."api/end?";
-    $params = 'meetingID='.urlencode($meetingID).'&password='.urlencode($modPW);
-    $url = $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);
+    $url = $base_url . $params . '&checksum=' . sha1("end" . $params . $SALT);
     return $url;
 }
 
-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);
     }
-    $url = $base_url_record.$params."&checksum=".sha1("getRecordings".$params.$SALT);
+    $url = $base_url_record . $params . "&checksum=" . sha1("getRecordings" . $params . $SALT);
     return $url;
 }
 
-function bigbluebuttonbn_getDeleteRecordingsURL( $recordID, $URL, $SALT ) {
-    $url_delete = $URL."api/deleteRecordings?";
-    $params = 'recordID='.urlencode($recordID);
-    $url = $url_delete.$params.'&checksum='.sha1("deleteRecordings".$params.$SALT);
+function bigbluebuttonbn_getDeleteRecordingsURL($recordID, $URL, $SALT) {
+    $url_delete = $URL . "api/deleteRecordings?";
+    $params = 'recordID=' . urlencode($recordID);
+    $url = $url_delete . $params . '&checksum=' . sha1("deleteRecordings" . $params . $SALT);
     return $url;
 }
 
-function bigbluebuttonbn_getPublishRecordingsURL( $recordID, $set, $URL, $SALT ) {
-    $url_publish = $URL."api/publishRecordings?";
-    $params = 'recordID='.$recordID."&publish=".$set;
-    $url = $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;
+    $url = $url_publish . $params . '&checksum=' . sha1("publishRecordings" . $params . $SALT);
     return $url;
 }
 
-function bigbluebuttonbn_getCreateMeetingArray( $username, $meetingID, $welcomeString, $mPW, $aPW, $SALT, $URL, $logoutURL, $record='false', $duration=0, $voiceBridge=0, $maxParticipants=0, $metadata=array(), $presentation_name=null, $presentation_url=null ) {
+function bigbluebuttonbn_getCreateMeetingArray($username, $meetingID, $welcomeString, $mPW, $aPW, $SALT, $URL, $logoutURL, $record = 'false', $duration = 0, $voiceBridge = 0, $maxParticipants = 0, $metadata = array(), $presentation_name = null, $presentation_url = null) {
     $create_meeting_url = bigbluebuttonbn_getCreateMeetingURL($username, $meetingID, $aPW, $mPW, $welcomeString, $logoutURL, $SALT, $URL, $record, $duration, $voiceBridge, $maxParticipants, $metadata);
-    if( !is_null($presentation_name) && !is_null($presentation_url) ) {
-        $xml = bigbluebuttonbn_wrap_xml_load_file( $create_meeting_url,
+    if (!is_null($presentation_name) && !is_null($presentation_url)) {
+        $xml = bigbluebuttonbn_wrap_xml_load_file($create_meeting_url,
                 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( $create_meeting_url );
+        $xml = bigbluebuttonbn_wrap_xml_load_file($create_meeting_url);
     }
 
-    if ( $xml ) {
+    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 );
+            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 );
+            return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey);
     } else {
         return null;
     }
 }
 
-function bigbluebuttonbn_getMeetingsArray($meetingID, $URL, $SALT ) {
-    $xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getMeetingsURL($URL, $SALT) );
+function bigbluebuttonbn_getMeetingsArray($meetingID, $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
@@ -184,21 +184,21 @@  discard block
 block discarded – undo
     }
 }
 
-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
@@ -206,19 +206,19 @@  discard block
 block discarded – undo
     }
 }
 
-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' && isset($xml->recordings) ) { //If there were meetings already created
-        foreach ( $xml->recordings->recording as $recording ) {
+    if ($xml && $xml->returncode == 'SUCCESS' && isset($xml->recordings)) { //If there were meetings already created
+        foreach ($xml->recordings->recording as $recording) {
             $recordings[] = bigbluebuttonbn_getRecordingArrayRow($recording);
         }
 
@@ -228,7 +228,7 @@  discard block
 block discarded – undo
     return $recordings;
 }
 
-function bigbluebuttonbn_index_recordings($recordings, $index_key='recordID') {
+function bigbluebuttonbn_index_recordings($recordings, $index_key = 'recordID') {
     $indexed_recordings = array();
 
     foreach ($recordings as $recording) {
@@ -238,14 +238,14 @@  discard block
 block discarded – undo
     return $indexed_recordings;
 }
 
-function bigbluebuttonbn_getRecordingArray( $recordingID, $meetingID, $URL, $SALT ) {
+function bigbluebuttonbn_getRecordingArray($recordingID, $meetingID, $URL, $SALT) {
     $recordingArray = 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' && isset($xml->recordings) ) { //If there were meetings already created
+    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) {
                 $recordingArray = bigbluebuttonbn_getRecordingArrayRow($recording);
                 break;
             }
@@ -255,57 +255,57 @@  discard block
 block discarded – undo
     return $recordingArray;
 }
 
-function bigbluebuttonbn_getRecordingArrayRow( $recording ) {
+function bigbluebuttonbn_getRecordingArrayRow($recording) {
     $recordingArrayRow = array();
 
     $playbackArray = array();
-    foreach ( $recording->playback->format as $format ) {
-        $playbackArray[(string) $format->type] = array( 'type' => (string) $format->type, 'url' => (string) $format->url, 'length' => (string) $format->length );
+    foreach ($recording->playback->format as $format) {
+        $playbackArray[(string)$format->type] = array('type' => (string)$format->type, 'url' => (string)$format->url, 'length' => (string)$format->length);
     }
 
     //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;
+    foreach ($metadata as $key => $value) {
+        if (is_object($value)) $value = '';
+        $metadataArray['meta_' . $key] = $value;
     }
 
-    $recordingArrayRow = 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;
+    $recordingArrayRow = 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;
 
     return $recordingArrayRow;
 }
 
-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
@@ -313,46 +313,46 @@  discard block
 block discarded – undo
     }
 }
 
-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) {
-    if ( bigbluebuttonbn_debugdisplay() ) error_log("Request to: ".$url);
+function bigbluebuttonbn_wrap_xml_load_file($url, $method = BIGBLUEBUTTONBN_METHOD_GET, $data = null) {
+    if (bigbluebuttonbn_debugdisplay()) error_log("Request to: " . $url);
 
     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);
@@ -374,9 +374,9 @@  discard block
 block discarded – undo
             try {
                 $xml = new SimpleXMLElement($response, LIBXML_NOCDATA);
                 return $xml;
-            } catch (Exception $e){
+            } catch (Exception $e) {
                 libxml_use_internal_errors($previous);
-                $error = 'Caught exception: '.$e->getMessage();
+                $error = 'Caught exception: ' . $e->getMessage();
                 error_log($error);
                 return NULL;
             }
@@ -388,18 +388,18 @@  discard block
 block discarded – undo
     } 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) {
@@ -419,13 +419,13 @@  discard block
 block discarded – undo
     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)
                     )
             );
@@ -436,19 +436,19 @@  discard block
 block discarded – undo
     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
                     )
             );
         }
@@ -456,15 +456,15 @@  discard block
 block discarded – undo
     }
 }
 
-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,
@@ -484,16 +484,16 @@  discard block
 block discarded – undo
         );
 
         $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",
@@ -519,7 +519,7 @@  discard block
 block discarded – undo
     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));
 }
 
@@ -528,21 +528,21 @@  discard block
 block discarded – undo
 
     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;
                     }
                 }
@@ -550,10 +550,10 @@  discard block
 block discarded – undo
         }
 
         //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;
                 }
             }
@@ -564,8 +564,8 @@  discard block
 block discarded – undo
 }
 
 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;
         }
     }
@@ -573,23 +573,23 @@  discard block
 block discarded – undo
 
 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 @@  discard block
 block discarded – undo
 
     $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 @@  discard block
 block discarded – undo
     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 @@  discard block
 block discarded – undo
             $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 @@  discard block
 block discarded – undo
         }
     }
 
-    $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 @@  discard block
 block discarded – undo
     $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 @@  discard block
 block discarded – undo
     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 @@  discard block
 block discarded – undo
     }
 }
 
-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 @@  discard block
 block discarded – undo
     $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 @@  discard block
 block discarded – undo
     $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 @@  discard block
 block discarded – undo
     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_publish_recording_imported($recordingid, $courseID, $bigbluebuttonbnID, $publish=true){
+function bigbluebuttonbn_bbb_broker_do_publish_recording_imported($recordingid, $courseID, $bigbluebuttonbnID, $publish = true) {
     global $DB;
 
     //Locate the record to be updated
@@ -843,9 +843,9 @@  discard block
 block discarded – undo
     $recordings_imported = array();
     foreach ($records as $key => $record) {
         $meta = json_decode($record->meta, true);
-        if( $recordingid == $meta['recording']['recordID'] ) {
+        if ($recordingid == $meta['recording']['recordID']) {
             // Found, prepare data for the update
-            $meta['recording']['published'] = ($publish)? 'true': 'false';
+            $meta['recording']['published'] = ($publish) ? 'true' : 'false';
             $records[$key]->meta = json_encode($meta);
 
             // Proceed with the update
@@ -854,7 +854,7 @@  discard block
 block discarded – undo
     }
 }
 
-function bigbluebuttonbn_bbb_broker_do_delete_recording($recordingid){
+function bigbluebuttonbn_bbb_broker_do_delete_recording($recordingid) {
     global $CFG;
 
     $endpoint = bigbluebuttonbn_get_cfg_server_url();
@@ -863,7 +863,7 @@  discard block
 block discarded – undo
     bigbluebuttonbn_doDeleteRecordings($recordingid, $endpoint, $shared_secret);
 }
 
-function bigbluebuttonbn_bbb_broker_do_delete_recording_imported($recordingid, $courseID, $bigbluebuttonbnID){
+function bigbluebuttonbn_bbb_broker_do_delete_recording_imported($recordingid, $courseID, $bigbluebuttonbnID) {
     global $DB;
 
     //Locate the record to be updated
@@ -872,7 +872,7 @@  discard block
 block discarded – undo
     $recordings_imported = array();
     foreach ($records as $key => $record) {
         $meta = json_decode($record->meta, true);
-        if( $recordingid == $meta['recording']['recordID'] ) {
+        if ($recordingid == $meta['recording']['recordID']) {
             // Execute delete
             $DB->delete_records("bigbluebuttonbn_logs", array('id' => $key));
         }
@@ -882,18 +882,18 @@  discard block
 block discarded – undo
 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;
@@ -903,79 +903,79 @@  discard block
 block discarded – undo
             case 'recording_unpublish':
             case 'recording_delete':
             case 'recording_import':
-                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;
     }
 
     return $error;
 }
 
-function bigbluebuttonbn_get_recording_data_row($bbbsession, $recording, $tools=["publishing", "deleting"]) {
+function bigbluebuttonbn_get_recording_data_row($bbbsession, $recording, $tools = ["publishing", "deleting"]) {
     global $OUTPUT, $CFG, $USER;
 
     $row = null;
 
-    if ( $bbbsession['managerecordings'] || $recording['published'] == 'true' ) {
+    if ($bbbsession['managerecordings'] || $recording['published'] == 'true') {
         $length = 0;
-        $startTime = isset($recording['startTime'])? floatval($recording['startTime']):0;
+        $startTime = isset($recording['startTime']) ? floatval($recording['startTime']) : 0;
         $startTime = $startTime - ($startTime % 1000);
-        $endTime = isset($recording['endTime'])? floatval($recording['endTime']):0;
+        $endTime = isset($recording['endTime']) ? floatval($recording['endTime']) : 0;
         $endTime = $endTime - ($endTime % 1000);
         $duration = intval(array_values($recording['playbacks'])[0]['length']);
 
         //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 = '';
         }
 
         //Set recording_types
-        if ( isset($recording['imported']) ) {
-            $attributes = 'data-imported="true" title='.get_string('view_recording_link_warning', 'bigbluebuttonbn');
+        if (isset($recording['imported'])) {
+            $attributes = 'data-imported="true" title=' . get_string('view_recording_link_warning', 'bigbluebuttonbn');
         } else {
             $attributes = 'data-imported="false"';
         }
 
         $recording_types = '';
         if ($recording['published'] == 'true') {
-            $recording_types .= '<div id="playbacks-'.$recording['recordID'].'" '.$attributes.'>';
+            $recording_types .= '<div id="playbacks-' . $recording['recordID'] . '" ' . $attributes . '>';
         } else {
-            $recording_types .= '<div id="playbacks-'.$recording['recordID'].'" '.$attributes.'" hidden>';
+            $recording_types .= '<div id="playbacks-' . $recording['recordID'] . '" ' . $attributes . '" 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>';
 
@@ -984,11 +984,11 @@  discard block
 block discarded – undo
 
         //Set actionbar, if user is allowed to manage recordings
         $actionbar = '';
-        if ( $bbbsession['managerecordings'] ) {
+        if ($bbbsession['managerecordings']) {
             // Set style for imported links
-            if( isset($recording['imported']) ) {
+            if (isset($recording['imported'])) {
                 $recordings_imported_count = 0;
-                $tag_tail = ' '.get_string('view_recording_link', 'bigbluebuttonbn');
+                $tag_tail = ' ' . get_string('view_recording_link', 'bigbluebuttonbn');
                 $head = '<i>';
                 $tail = '</i>';
             } else {
@@ -1002,53 +1002,53 @@  discard block
 block discarded – undo
 
             if (in_array("publishing", $tools)) {
                 ///Set action [show|hide]
-                if ( $recording['published'] == 'true' ){
+                if ($recording['published'] == 'true') {
                     $manage_tag = 'hide';
                     $manage_action = 'unpublish';
                 } else {
                     $manage_tag = 'show';
                     $manage_action = 'publish';
                 }
-                $onclick = 'M.mod_bigbluebuttonbn.broker_manageRecording("'.$manage_action.'", "'.$recording['recordID'].'", "'.$recording['meetingID'].'");';
+                $onclick = 'M.mod_bigbluebuttonbn.broker_manageRecording("' . $manage_action . '", "' . $recording['recordID'] . '", "' . $recording['meetingID'] . '");';
 
-                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).$tag_tail, 'moodle', $icon_attributes);
-                    $link_attributes = array('id' => 'recording-link-'.$manage_action.'-'.$recording['recordID'], 'onclick' => $onclick, 'data-links' => $recordings_imported_count);
+                    $icon_attributes = array('id' => 'recording-btn-' . $manage_action . '-' . $recording['recordID']);
+                    $icon = new pix_icon('t/' . $manage_tag, get_string($manage_tag) . $tag_tail, 'moodle', $icon_attributes);
+                    $link_attributes = array('id' => 'recording-link-' . $manage_action . '-' . $recording['recordID'], 'onclick' => $onclick, 'data-links' => $recordings_imported_count);
                     $actionbar .= $OUTPUT->action_icon($url, $icon, $action, $link_attributes, false);
                 } else {
                     //With text for publish/unpublish
-                    $link_attributes = array('title' => get_string($manage_tag).$tag_tail, 'class' => 'btn btn-xs', 'onclick' => $onclick, 'data-links' => $recordings_imported_count);
-                    $actionbar .= $OUTPUT->action_link($url, get_string($manage_tag).$tag_tail, $action, $link_attributes);
+                    $link_attributes = array('title' => get_string($manage_tag) . $tag_tail, 'class' => 'btn btn-xs', 'onclick' => $onclick, 'data-links' => $recordings_imported_count);
+                    $actionbar .= $OUTPUT->action_link($url, get_string($manage_tag) . $tag_tail, $action, $link_attributes);
                     $actionbar .= "&nbsp;";
                 }
             }
 
             if (in_array("deleting", $tools)) {
-                $onclick = 'M.mod_bigbluebuttonbn.broker_manageRecording("delete", "'.$recording['recordID'].'", "'.$recording['meetingID'].'");';
+                $onclick = 'M.mod_bigbluebuttonbn.broker_manageRecording("delete", "' . $recording['recordID'] . '", "' . $recording['meetingID'] . '");';
 
-                if ( bigbluebuttonbn_get_cfg_recording_icons_enabled() ) {
+                if (bigbluebuttonbn_get_cfg_recording_icons_enabled()) {
                     //With icon for delete
-                    $icon_attributes = array('id' => 'recording-btn-delete-'.$recording['recordID']);
-                    $icon = new pix_icon('t/delete', get_string('delete').$tag_tail, 'moodle', $icon_attributes);
-                    $link_attributes = array('id' => 'recording-link-delete-'.$recording['recordID'], 'onclick' => $onclick, 'data-links' => $recordings_imported_count);
+                    $icon_attributes = array('id' => 'recording-btn-delete-' . $recording['recordID']);
+                    $icon = new pix_icon('t/delete', get_string('delete') . $tag_tail, 'moodle', $icon_attributes);
+                    $link_attributes = array('id' => 'recording-link-delete-' . $recording['recordID'], 'onclick' => $onclick, 'data-links' => $recordings_imported_count);
                     $actionbar .= $OUTPUT->action_icon($url, $icon, $action, $link_attributes, false);
                 } else {
                     //With text for delete
-                    $link_attributes = array('title' => get_string('delete').$tag_tail, 'class' => 'btn btn-xs btn-danger', 'onclick' => $onclick, 'data-links' => $recordings_imported_count);
-                    $actionbar .= $OUTPUT->action_link($url, get_string('delete').$tag_tail, $action, $link_attributes);
+                    $link_attributes = array('title' => get_string('delete') . $tag_tail, 'class' => 'btn btn-xs btn-danger', 'onclick' => $onclick, 'data-links' => $recordings_imported_count);
+                    $actionbar .= $OUTPUT->action_link($url, get_string('delete') . $tag_tail, $action, $link_attributes);
                 }
             }
 
             if (in_array("importing", $tools)) {
-                $onclick = 'M.mod_bigbluebuttonbn.broker_manageRecording("import", "'.$recording['recordID'].'", "'.$recording['meetingID'].'");';
+                $onclick = 'M.mod_bigbluebuttonbn.broker_manageRecording("import", "' . $recording['recordID'] . '", "' . $recording['meetingID'] . '");';
 
-                if ( bigbluebuttonbn_get_cfg_recording_icons_enabled() ) {
+                if (bigbluebuttonbn_get_cfg_recording_icons_enabled()) {
                     //With icon for import
-                    $icon_attributes = array('id' => 'recording-btn-import-'.$recording['recordID']);
+                    $icon_attributes = array('id' => 'recording-btn-import-' . $recording['recordID']);
                     $icon = new pix_icon('i/import', get_string('import'), 'moodle', $icon_attributes);
-                    $link_attributes = array('id' => 'recording-link-import-'.$recording['recordID'], 'onclick' => $onclick);
+                    $link_attributes = array('id' => 'recording-link-import-' . $recording['recordID'], 'onclick' => $onclick);
                     $actionbar .= $OUTPUT->action_icon($url, $icon, $action, $link_attributes, false);
                 } else {
                     //With text for import
@@ -1070,7 +1070,7 @@  discard block
 block discarded – undo
         $row->date_formatted = "{$head}{$formattedStartDate}{$tail}";
         $row->duration = "{$duration}";
         $row->duration_formatted = "{$head}{$duration}{$tail}";
-        if ( $bbbsession['managerecordings'] ) {
+        if ($bbbsession['managerecordings']) {
             $row->actionbar = $actionbar;
         }
     }
@@ -1096,21 +1096,21 @@  discard block
 block discarded – undo
         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));
     }
 
     return $recordingsbn_columns;
 }
 
-function bigbluebuttonbn_get_recording_data($bbbsession, $recordings, $tools=["publishing", "deleting"]) {
+function bigbluebuttonbn_get_recording_data($bbbsession, $recordings, $tools = ["publishing", "deleting"]) {
     $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, $tools);
-            if( $row != null ) {
+            if ($row != null) {
                 array_push($table_data, $row);
             }
         }
@@ -1119,7 +1119,7 @@  discard block
 block discarded – undo
     return $table_data;
 }
 
-function bigbluebuttonbn_get_recording_table($bbbsession, $recordings, $tools=['publishing','deleting']) {
+function bigbluebuttonbn_get_recording_table($bbbsession, $recordings, $tools = ['publishing', 'deleting']) {
     global $OUTPUT, $CFG;
 
     ///Set strings to show
@@ -1138,20 +1138,20 @@  discard block
 block discarded – undo
     $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'];
-            if ( isset($recording['imported']) ) {
+            $row->id = 'recording-td-' . $recording['recordID'];
+            if (isset($recording['imported'])) {
                 $row->attributes['data-imported'] = 'true';
                 $row->attributes['title'] = get_string('view_recording_link_warning', 'bigbluebuttonbn');
             } else {
@@ -1159,12 +1159,12 @@  discard block
 block discarded – undo
             }
 
             $row_data = bigbluebuttonbn_get_recording_data_row($bbbsession, $recording, $tools);
-            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_formatted, $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_formatted, $row_data->actionbar);
                 } else {
-                    $row->cells = array ($row_data->recording, $row_data->activity, $row_data->description, $row_data->date_formatted, $row_data->duration_formatted );
+                    $row->cells = array($row_data->recording, $row_data->activity, $row_data->description, $row_data->date_formatted, $row_data->duration_formatted);
                 }
 
                 array_push($table->data, $row);
@@ -1184,12 +1184,12 @@  discard block
 block discarded – undo
     /// Build the message_body
     $msg->activity_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;
@@ -1203,12 +1203,12 @@  discard block
 block discarded – undo
     $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;
             }
         }
@@ -1217,14 +1217,14 @@  discard block
 block discarded – undo
     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() {
@@ -1267,89 +1267,89 @@  discard block
 block discarded – undo
 
 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_importrecordings_enabled() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_importrecordings_enabled)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_importrecordings_enabled: (isset($CFG->bigbluebuttonbn_importrecordings_enabled)? $CFG->bigbluebuttonbn_importrecordings_enabled: false));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_importrecordings_enabled) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_importrecordings_enabled : (isset($CFG->bigbluebuttonbn_importrecordings_enabled) ? $CFG->bigbluebuttonbn_importrecordings_enabled : false));
 }
 
 function bigbluebuttonbn_get_cfg_importrecordings_from_deleted_activities_enabled() {
     global $BIGBLUEBUTTONBN_CFG, $CFG;
-    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_importrecordings_from_deleted_activities_enabled)? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_importrecordings_from_deleted_activities_enabled: (isset($CFG->bigbluebuttonbn_importrecordings_from_deleted_activities_enabled)? $CFG->bigbluebuttonbn_importrecordings_from_deleted_activities_enabled: false));
+    return (isset($BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_importrecordings_from_deleted_activities_enabled) ? $BIGBLUEBUTTONBN_CFG->bigbluebuttonbn_importrecordings_from_deleted_activities_enabled : (isset($CFG->bigbluebuttonbn_importrecordings_from_deleted_activities_enabled) ? $CFG->bigbluebuttonbn_importrecordings_from_deleted_activities_enabled : false));
 }
 
 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;
     }
@@ -1357,37 +1357,37 @@  discard block
 block discarded – undo
 
 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));
 }
 
 function bigbluebuttonbn_import_get_courses_for_select(array $bbbsession) {
 
-    if( $bbbsession['administrator'] ) {
+    if ($bbbsession['administrator']) {
         $courses = get_courses('all', 'c.id ASC', 'c.id,c.shortname,c.fullname');
         //It includes the name of the site as a course (category 0), so remove the first one
         unset($courses["1"]);
@@ -1396,21 +1396,21 @@  discard block
 block discarded – undo
     }
 
     $courses_for_select = [];
-    foreach($courses as $course) {
-        if( $course->id != $bbbsession['course']->id ) {
+    foreach ($courses as $course) {
+        if ($course->id != $bbbsession['course']->id) {
             $courses_for_select[$course->id] = $course->fullname;
         }
     }
     return $courses_for_select;
 }
 
-function bigbluebuttonbn_getRecordedMeetingsDeleted($courseID, $bigbluebuttonbnID=NULL) {
+function bigbluebuttonbn_getRecordedMeetingsDeleted($courseID, $bigbluebuttonbnID = NULL) {
     global $DB;
 
     $records_deleted = array();
 
-    $filter = array('courseid' => $courseID, 'log' => BIGBLUEBUTTONBN_LOG_EVENT_DELETE );
-    if ( $bigbluebuttonbnID != NULL ) {
+    $filter = array('courseid' => $courseID, 'log' => BIGBLUEBUTTONBN_LOG_EVENT_DELETE);
+    if ($bigbluebuttonbnID != NULL) {
         $filter['id'] = $bigbluebuttonbnID;
     }
 
@@ -1419,17 +1419,17 @@  discard block
 block discarded – undo
     foreach ($bigbluebuttonbns_deleted as $key => $bigbluebuttonbn_deleted) {
         $records = $DB->get_records('bigbluebuttonbn_logs', array('courseid' => $courseID, 'log' => BIGBLUEBUTTONBN_LOG_EVENT_CREATE));
 
-        if( !empty($records) ) {
+        if (!empty($records)) {
             //Remove duplicates
             $unique_records = array();
             foreach ($records as $key => $record) {
-                if (array_key_exists($record->meetingid, $unique_records) ) {
+                if (array_key_exists($record->meetingid, $unique_records)) {
                     unset($records[$key]);
                 } else {
                     $meta = json_decode($record->meta);
-                    if ( !$meta->record ) {
+                    if (!$meta->record) {
                         unset($records[$key]);
-                    } else if ( $bigbluebuttonbn_deleted->meetingid != substr($record->meetingid, 0, strlen($bigbluebuttonbn_deleted->meetingid))) {
+                    } else if ($bigbluebuttonbn_deleted->meetingid != substr($record->meetingid, 0, strlen($bigbluebuttonbn_deleted->meetingid))) {
                         unset($records[$key]);
                     } else {
                         array_push($unique_records, $record->meetingid);
@@ -1444,27 +1444,27 @@  discard block
 block discarded – undo
     return $records_deleted;
 }
 
-function bigbluebuttonbn_getRecordedMeetings($courseID, $bigbluebuttonbnID=NULL) {
+function bigbluebuttonbn_getRecordedMeetings($courseID, $bigbluebuttonbnID = NULL) {
     global $DB;
 
     $records = Array();
 
     $filter = array('course' => $courseID);
-    if ( $bigbluebuttonbnID != NULL ) {
+    if ($bigbluebuttonbnID != NULL) {
         $filter['id'] = $bigbluebuttonbnID;
     }
     $bigbluebuttonbns = $DB->get_records('bigbluebuttonbn', $filter);
 
-    if ( !empty($bigbluebuttonbns) ) {
+    if (!empty($bigbluebuttonbns)) {
         $table = 'bigbluebuttonbn_logs';
 
         //Prepare select for loading records based on existent bigbluebuttonbns
         $select = "";
         foreach ($bigbluebuttonbns as $key => $bigbluebuttonbn) {
-            $select .= strlen($select) == 0? "(": " OR ";
-            $select .= "bigbluebuttonbnid=".$bigbluebuttonbn->id;
+            $select .= strlen($select) == 0 ? "(" : " OR ";
+            $select .= "bigbluebuttonbnid=" . $bigbluebuttonbn->id;
         }
-        $select .= ") AND log='".BIGBLUEBUTTONBN_LOG_EVENT_CREATE."'";
+        $select .= ") AND log='" . BIGBLUEBUTTONBN_LOG_EVENT_CREATE . "'";
 
         //Execute select for loading records based on existent bigbluebuttonbns
         $records = $DB->get_records_select($table, $select);
@@ -1472,8 +1472,8 @@  discard block
 block discarded – undo
         //Remove duplicates
         $unique_records = array();
         foreach ($records as $key => $record) {
-            $record_key = $record->meetingid.','.$record->bigbluebuttonbnid.','.$record->meta;
-            if( array_search($record_key, $unique_records) === false ) {
+            $record_key = $record->meetingid . ',' . $record->bigbluebuttonbnid . ',' . $record->meta;
+            if (array_search($record_key, $unique_records) === false) {
                 array_push($unique_records, $record_key);
             } else {
                 unset($records[$key]);
@@ -1483,7 +1483,7 @@  discard block
 block discarded – undo
         //Remove the ones with record=false
         foreach ($records as $key => $record) {
             $meta = json_decode($record->meta);
-            if ( !$meta || !$meta->record ) {
+            if (!$meta || !$meta->record) {
                 unset($records[$key]);
             }
         }
@@ -1497,15 +1497,15 @@  discard block
 block discarded – undo
 
     // Load the meetingIDs to be used in the getRecordings request
     $meetingID = '';
-    if ( is_numeric($courseID) ) {
+    if (is_numeric($courseID)) {
         $results = bigbluebuttonbn_getRecordedMeetings($courseID);
 
-        if( bigbluebuttonbn_get_cfg_importrecordings_from_deleted_activities_enabled() ) {
+        if (bigbluebuttonbn_get_cfg_importrecordings_from_deleted_activities_enabled()) {
             $results_deleted = bigbluebuttonbn_getRecordedMeetingsDeleted($courseID);
             $results = array_merge($results, $results_deleted);
         }
 
-        if( $results ) {
+        if ($results) {
             //Eliminates duplicates
             $mIDs = array();
             foreach ($results as $result) {
@@ -1520,7 +1520,7 @@  discard block
 block discarded – undo
     }
 
     // If there were meetingIDs excecute the getRecordings request
-    if ( $meetingID != '' ) {
+    if ($meetingID != '') {
         $recordings = bigbluebuttonbn_getRecordingsArray($meetingID, $URL, $SALT);
     }
 
@@ -1543,37 +1543,37 @@  discard block
 block discarded – undo
     $recordings_already_imported_indexed = bigbluebuttonbn_index_recordings($recordings_already_imported);
 
     foreach ($recordings as $key => $recording) {
-        if( isset($recordings_already_imported_indexed[$recording['recordID']]) ) {
+        if (isset($recordings_already_imported_indexed[$recording['recordID']])) {
             unset($recordings[$key]);
         }
     }
     return $recordings;
 }
 
-function bigbluebutton_output_recording_table($bbbsession, $recordings, $tools=['publishing','deleting']) {
+function bigbluebutton_output_recording_table($bbbsession, $recordings, $tools = ['publishing', 'deleting']) {
 
-    if ( isset($recordings) && !empty($recordings) ) {  // There are recordings for this meeting
+    if (isset($recordings) && !empty($recordings)) {  // There are recordings for this meeting
         $table = bigbluebuttonbn_get_recording_table($bbbsession, $recordings, $tools);
     }
 
     $output = '';
-    if( isset($table->data) ) {
+    if (isset($table->data)) {
         //Print the table
-        $output .= '<div id="bigbluebuttonbn_html_table">'."\n";
-        $output .= html_writer::table($table)."\n";
-        $output .= '</div>'."\n";
+        $output .= '<div id="bigbluebuttonbn_html_table">' . "\n";
+        $output .= html_writer::table($table) . "\n";
+        $output .= '</div>' . "\n";
 
     } else {
-        $output .= get_string('view_message_norecordings', 'bigbluebuttonbn').'<br>'."\n";
+        $output .= get_string('view_message_norecordings', 'bigbluebuttonbn') . '<br>' . "\n";
     }
 
     return $output;
 }
 
-function bigbluebuttonbn_getRecordingsImported($courseID, $bigbluebuttonbnID=NULL) {
+function bigbluebuttonbn_getRecordingsImported($courseID, $bigbluebuttonbnID = NULL) {
     global $DB;
 
-    if ( $bigbluebuttonbnID != NULL ) {
+    if ($bigbluebuttonbnID != NULL) {
         // Fetch only those related to the $courseID and $bigbluebuttonbnID requested
         $recordings_imported = $DB->get_records('bigbluebuttonbn_logs', array('courseid' => $courseID, 'bigbluebuttonbnid' => $bigbluebuttonbnID, 'log' => BIGBLUEBUTTONBN_LOG_EVENT_IMPORT));
     } else {
@@ -1583,7 +1583,7 @@  discard block
 block discarded – undo
     return $recordings_imported;
 }
 
-function bigbluebuttonbn_getRecordingsImportedArray($courseID, $bigbluebuttonbnID=NULL) {
+function bigbluebuttonbn_getRecordingsImportedArray($courseID, $bigbluebuttonbnID = NULL) {
     $recordings_imported = bigbluebuttonbn_getRecordingsImported($courseID, $bigbluebuttonbnID);
     $recordings_imported_array = bigbluebuttonbn_import_get_recordings_imported($recordings_imported);
     return $recordings_imported_array;
@@ -1592,7 +1592,7 @@  discard block
 block discarded – undo
 function bigbluebuttonbn_getRecordingsImportedAllInstances($recordID) {
     global $DB, $CFG;
 
-    $recordings_imported = $DB->get_records_sql('SELECT * FROM '.$CFG->prefix.'bigbluebuttonbn_logs WHERE log=? AND '.$DB->sql_like('meta', '?'), array( BIGBLUEBUTTONBN_LOG_EVENT_IMPORT, '%'.$recordID.'%' ));
+    $recordings_imported = $DB->get_records_sql('SELECT * FROM ' . $CFG->prefix . 'bigbluebuttonbn_logs WHERE log=? AND ' . $DB->sql_like('meta', '?'), array(BIGBLUEBUTTONBN_LOG_EVENT_IMPORT, '%' . $recordID . '%'));
     return $recordings_imported;
 }
 
@@ -1611,8 +1611,8 @@  discard block
 block discarded – undo
 function bigbluebuttonbn_html2text($html, $len) {
     $text = strip_tags($html);
     $text = str_replace("&nbsp;", ' ', $text);
-    if( strlen($text) > $len )
-        $text = substr($text, 0, $len)."...";
+    if (strlen($text) > $len)
+        $text = substr($text, 0, $len) . "...";
     else
         $text = substr($text, 0, $len);
     return $text;

Please login to merge, or discard this patch.

Braces +64 added lines, -39 removed lines patch added patch discarded remove patch

@@ -69,19 +69,23 @@  discard block
 block discarded – undo
     $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;
+    }
 
     $maxParticipants = intval($maxParticipants);
-    if( $maxParticipants > 0 )
-        $params .= '&maxParticipants='.$maxParticipants;
+    if( $maxParticipants > 0 ) {
+            $params .= '&maxParticipants='.$maxParticipants;
+    }
 
-    if( trim( $welcome ) )
-        $params .= '&welcome='.urlencode($welcome);
+    if( trim( $welcome ) ) {
+            $params .= '&welcome='.urlencode($welcome);
+    }
 
     foreach ($metadata as $key => $value) {
         $params .= '&'.$key.'='.urlencode($value);
@@ -155,10 +159,11 @@  discard block
 block discarded – undo
     }
 
     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->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;
     }
@@ -267,7 +272,9 @@  discard block
 block discarded – undo
     $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;
     }
 
@@ -277,17 +284,22 @@  discard block
 block discarded – undo
 }
 
 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;
 }
@@ -296,8 +308,9 @@  discard block
 block discarded – undo
     $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;
 }
@@ -307,8 +320,7 @@  discard block
 block discarded – undo
 
     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;
     }
 }
@@ -342,7 +354,9 @@  discard block
 block discarded – undo
 }
 
 function bigbluebuttonbn_wrap_xml_load_file($url, $method=BIGBLUEBUTTONBN_METHOD_GET, $data=null) {
-    if ( bigbluebuttonbn_debugdisplay() ) error_log("Request to: ".$url);
+    if ( bigbluebuttonbn_debugdisplay() ) {
+        error_log("Request to: ".$url);
+    }
 
     if (extension_loaded('curl')) {
         $c = new curl();
@@ -530,8 +544,9 @@  discard block
 block discarded – undo
         // 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;
+                }
             }
         }
 
@@ -542,8 +557,9 @@  discard block
 block discarded – undo
                 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;
+                        }
                     }
                 }
             }
@@ -553,8 +569,9 @@  discard block
 block discarded – undo
         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 +605,9 @@  discard block
 block discarded – undo
     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;
         }
@@ -924,7 +943,9 @@  discard block
 block discarded – undo
     $error = $org_msg;
 
     if( !empty($new_msg) ) {
-        if( !empty($error) ) $error .= ' ';
+        if( !empty($error) ) {
+            $error .= ' ';
+        }
         $error .= $new_msg;
     }
 
@@ -1208,8 +1229,9 @@  discard block
 block discarded – undo
         $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;
+                }
             }
         }
     }
@@ -1513,7 +1535,9 @@  discard block
 block discarded – undo
             }
             //Generates the meetingID string
             foreach ($mIDs as $mID) {
-                if (strlen($meetingID) > 0) $meetingID .= ',';
+                if (strlen($meetingID) > 0) {
+                    $meetingID .= ',';
+                }
                 $meetingID .= $mID;
             }
         }
@@ -1611,9 +1635,10 @@  discard block
 block discarded – undo
 function bigbluebuttonbn_html2text($html, $len) {
     $text = strip_tags($html);
     $text = str_replace("&nbsp;", ' ', $text);
-    if( strlen($text) > $len )
-        $text = substr($text, 0, $len)."...";
-    else
-        $text = substr($text, 0, $len);
+    if( strlen($text) > $len ) {
+            $text = substr($text, 0, $len)."...";
+    } else {
+            $text = substr($text, 0, $len);
+    }
     return $text;
 }
\ No newline at end of file

Please login to merge, or discard this patch.

		@@ -69,19 +69,23 @@ discard block
		block discarded – undo
69	69	$params = 'name='.urlencode($name).'&meetingID='.urlencode($meetingID).'&attendeePW='.urlencode($attendeePW).'&moderatorPW='.urlencode($moderatorPW).'&logoutURL='.urlencode($logoutURL).'&record='.$record;
70	70
71	71	$voiceBridge = intval($voiceBridge);
72		- if ( $voiceBridge > 0 && $voiceBridge < 79999)
73		- $params .= '&voiceBridge='.$voiceBridge;
	72	+ if ( $voiceBridge > 0 && $voiceBridge < 79999) {
	73	+ $params .= '&voiceBridge='.$voiceBridge;
	74	+ }
74	75
75	76	$duration = intval($duration);
76		- if( $duration > 0 )
77		- $params .= '&duration='.$duration;
	77	+ if( $duration > 0 ) {
	78	+ $params .= '&duration='.$duration;
	79	+ }
78	80
79	81	$maxParticipants = intval($maxParticipants);
80		- if( $maxParticipants > 0 )
81		- $params .= '&maxParticipants='.$maxParticipants;
	82	+ if( $maxParticipants > 0 ) {
	83	+ $params .= '&maxParticipants='.$maxParticipants;
	84	+ }
82	85
83		- if( trim( $welcome ) )
84		- $params .= '&welcome='.urlencode($welcome);
	86	+ if( trim( $welcome ) ) {
	87	+ $params .= '&welcome='.urlencode($welcome);
	88	+ }
85	89
86	90	foreach ($metadata as $key => $value) {
87	91	$params .= '&'.$key.'='.urlencode($value);
		@@ -155,10 +159,11 @@ discard block
		block discarded – undo
155	159	}
156	160
157	161	if ( $xml ) {
158		- if ($xml->meetingID)
159		- return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey, 'meetingID' => $xml->meetingID, 'attendeePW' => $xml->attendeePW, 'moderatorPW' => $xml->moderatorPW, 'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded );
160		- else
161		- return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey );
	162	+ if ($xml->meetingID) {
	163	+ return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey, 'meetingID' => $xml->meetingID, 'attendeePW' => $xml->attendeePW, 'moderatorPW' => $xml->moderatorPW, 'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded );
	164	+ } else {
	165	+ return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey );
	166	+ }
162	167	} else {
163	168	return null;
164	169	}
		@@ -267,7 +272,9 @@ discard block
		block discarded – undo
267	272	$metadataArray = array();
268	273	$metadata = get_object_vars($recording->metadata);
269	274	foreach ( $metadata as $key => $value ) {
270		- if ( is_object($value) ) $value = '';
	275	+ if ( is_object($value) ) {
	276	+ $value = '';
	277	+ }
271	278	$metadataArray['meta_'.$key] = $value;
272	279	}
273	280
		@@ -277,17 +284,22 @@ discard block
		block discarded – undo
277	284	}
278	285
279	286	function bigbluebuttonbn_recordingBuildSorter($a, $b){
280		- if( $a['startTime'] < $b['startTime']) return -1;
281		- else if( $a['startTime'] == $b['startTime']) return 0;
282		- else return 1;
283		-}
	287	+ if( $a['startTime'] < $b['startTime']) {
	288	+ return -1;
	289	+ } else if( $a['startTime'] == $b['startTime']) {
	290	+ return 0;
	291	+ } else {
	292	+ return 1;
	293	+ }
	294	+ }
284	295
285	296	function bigbluebuttonbn_doDeleteRecordings( $recordIDs, $URL, $SALT ) {
286	297	$ids = explode(",", $recordIDs);
287	298	foreach( $ids as $id){
288	299	$xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getDeleteRecordingsURL($id, $URL, $SALT) );
289		- if( $xml && $xml->returncode != 'SUCCESS' )
290		- return false;
	300	+ if( $xml && $xml->returncode != 'SUCCESS' ) {
	301	+ return false;
	302	+ }
291	303	}
292	304	return true;
293	305	}
		@@ -296,8 +308,9 @@ discard block
		block discarded – undo
296	308	$ids = explode(",", $recordIDs);
297	309	foreach( $ids as $id){
298	310	$xml = bigbluebuttonbn_wrap_xml_load_file( bigbluebuttonbn_getPublishRecordingsURL($id, $set, $URL, $SALT) );
299		- if( $xml && $xml->returncode != 'SUCCESS' )
300		- return false;
	311	+ if( $xml && $xml->returncode != 'SUCCESS' ) {
	312	+ return false;
	313	+ }
301	314	}
302	315	return true;
303	316	}
		@@ -307,8 +320,7 @@ discard block
		block discarded – undo
307	320
308	321	if( $xml ) { //If the xml packet returned failure it displays the message to the user
309	322	return array('returncode' => $xml->returncode, 'message' => $xml->message, 'messageKey' => $xml->messageKey);
310		- }
311		- else { //If the server is unreachable, then prompts the user of the necessary action
	323	+ } else { //If the server is unreachable, then prompts the user of the necessary action
312	324	return null;
313	325	}
314	326	}
		@@ -342,7 +354,9 @@ discard block
		block discarded – undo
342	354	}
343	355
344	356	function bigbluebuttonbn_wrap_xml_load_file($url, $method=BIGBLUEBUTTONBN_METHOD_GET, $data=null) {
345		- if ( bigbluebuttonbn_debugdisplay() ) error_log("Request to: ".$url);
	357	+ if ( bigbluebuttonbn_debugdisplay() ) {
	358	+ error_log("Request to: ".$url);
	359	+ }
346	360
347	361	if (extension_loaded('curl')) {
348	362	$c = new curl();
		@@ -530,8 +544,9 @@ discard block
		block discarded – undo
530	544	// Iterate looking for all configuration
531	545	foreach($participant_list as $participant){
532	546	if( $participant->selectiontype == 'all' ) {
533		- if ( $participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR )
534		- return true;
	547	+ if ( $participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR ) {
	548	+ return true;
	549	+ }
535	550	}
536	551	}
537	552
		@@ -542,8 +557,9 @@ discard block
		block discarded – undo
542	557	foreach( $roles as $role ) {
543	558	$db_moodle_role = bigbluebuttonbn_moodle_db_role_lookup($db_moodle_roles, $role->roleid);
544	559	if( $participant->selectionid == $db_moodle_role->shortname ) {
545		- if ( $participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR )
546		- return true;
	560	+ if ( $participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR ) {
	561	+ return true;
	562	+ }
547	563	}
548	564	}
549	565	}
		@@ -553,8 +569,9 @@ discard block
		block discarded – undo
553	569	foreach($participant_list as $participant){
554	570	if( $participant->selectiontype == 'user' ) {
555	571	if( $participant->selectionid == $user ) {
556		- if ( $participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR )
557		- return true;
	572	+ if ( $participant->role == BIGBLUEBUTTONBN_ROLE_MODERATOR ) {
	573	+ return true;
	574	+ }
558	575	}
559	576	}
560	577	}
		@@ -588,7 +605,9 @@ discard block
		block discarded – undo
588	605	if( $voicebridge != 0 ) {
589	606	$table = "bigbluebuttonbn";
590	607	$select = "voicebridge = ".$voicebridge;
591		- if( $id ) $select .= " AND id <> ".$id;
	608	+ if( $id ) {
	609	+ $select .= " AND id <> ".$id;
	610	+ }
592	611	if ( $rooms = $DB->get_records_select($table, $select) ) {
593	612	$is_unique = false;
594	613	}
		@@ -924,7 +943,9 @@ discard block
		block discarded – undo
924	943	$error = $org_msg;
925	944
926	945	if( !empty($new_msg) ) {
927		- if( !empty($error) ) $error .= ' ';
	946	+ if( !empty($error) ) {
	947	+ $error .= ' ';
	948	+ }
928	949	$error .= $new_msg;
929	950	}
930	951
		@@ -1208,8 +1229,9 @@ discard block
		block discarded – undo
1208	1229	$capabilities = bigbluebuttonbn_getCapabilitiesArray( $endpoint, $shared_secret );
1209	1230	if( $capabilities ) {
1210	1231	foreach ($capabilities as $capability) {
1211		- if( $capability["name"] == $capability_name)
1212		- $capability_offered = $capability;
	1232	+ if( $capability["name"] == $capability_name) {
	1233	+ $capability_offered = $capability;
	1234	+ }
1213	1235	}
1214	1236	}
1215	1237	}
		@@ -1513,7 +1535,9 @@ discard block
		block discarded – undo
1513	1535	}
1514	1536	//Generates the meetingID string
1515	1537	foreach ($mIDs as $mID) {
1516		- if (strlen($meetingID) > 0) $meetingID .= ',';
	1538	+ if (strlen($meetingID) > 0) {
	1539	+ $meetingID .= ',';
	1540	+ }
1517	1541	$meetingID .= $mID;
1518	1542	}
1519	1543	}
		@@ -1611,9 +1635,10 @@ discard block
		block discarded – undo
1611	1635	function bigbluebuttonbn_html2text($html, $len) {
1612	1636	$text = strip_tags($html);
1613	1637	$text = str_replace(" ", ' ', $text);
1614		- if( strlen($text) > $len )
1615		- $text = substr($text, 0, $len)."...";
1616		- else
1617		- $text = substr($text, 0, $len);
	1638	+ if( strlen($text) > $len ) {
	1639	+ $text = substr($text, 0, $len)."...";
	1640	+ } else {
	1641	+ $text = substr($text, 0, $len);
	1642	+ }
1618	1643	return $text;
1619	1644	}
1620	1645	\ No newline at end of file

blindsidenetworks / moodle-mod_bigbluebuttonbn

Push — master ( 16d8be...175c16 )

Status

Category

Spacing +339 added lines, -339 removed lines patch added patch discarded remove patch

Braces +64 added lines, -39 removed lines patch added patch discarded remove patch