blindsidenetworks / moodle-mod_bigbluebuttonbn

vendor/firebase/php-jwt/src/JWT.php

Spacing +5 added lines, -5 removed lines

@@ -156,7 +156,7 @@ 
         if ($keyId !== null) {
             $header['kid'] = $keyId;
         }
-        if ( isset($head) && is_array($head) ) {
+        if (isset($head) && is_array($head)) {
             $header = array_merge($head, $header);
         }
         $segments = array();
@@ -188,7 +188,7 @@ 
             throw new DomainException('Algorithm not supported');
         }
         list($function, $algorithm) = static::$supported_algs[$alg];
-        switch($function) {
+        switch ($function) {
             case 'hash_hmac':
                 return hash_hmac($algorithm, $msg, $key, true);
             case 'openssl':
@@ -222,7 +222,7 @@ 
         }
 
         list($function, $algorithm) = static::$supported_algs[$alg];
-        switch($function) {
+        switch ($function) {
             case 'openssl':
                 $success = openssl_verify($msg, $signature, $key, $algorithm);
                 if (!$success) {
@@ -270,8 +270,8 @@ 
              * manually detect large ints in the JSON string and quote them (thus converting
              *them to strings) before decoding, hence the preg_replace() call.
              */
-            $max_int_length = strlen((string) PHP_INT_MAX) - 1;
-            $json_without_bigints = preg_replace('/:\s*(-?\d{'.$max_int_length.',})/', ': "$1"', $input);
+            $max_int_length = strlen((string)PHP_INT_MAX) - 1;
+            $json_without_bigints = preg_replace('/:\s*(-?\d{' . $max_int_length . ',})/', ': "$1"', $input);
             $obj = json_decode($json_without_bigints);
         }
 

Doc Comments +3 added lines, -3 removed lines

@@ -48,9 +48,9 @@ 
      * Decodes a JWT string into a PHP object.
      *
      * @param string        $jwt            The JWT
-     * @param string|array  $key            The key, or map of keys.
+     * @param string  $key            The key, or map of keys.
      *                                      If the algorithm used is asymmetric, this is the public key
-     * @param array         $allowed_algs   List of supported verification algorithms
+     * @param string[]         $allowed_algs   List of supported verification algorithms
      *                                      Supported algorithms are 'HS256', 'HS384', 'HS512' and 'RS256'
      *
      * @return object The JWT's payload as a PHP object
@@ -174,7 +174,7 @@ 
      * Sign a string with a given key and algorithm.
      *
      * @param string            $msg    The message to sign
-     * @param string|resource   $key    The secret key
+     * @param string   $key    The secret key
      * @param string            $alg    The signing algorithm.
      *                                  Supported algorithms are 'HS256', 'HS384', 'HS512' and 'RS256'
      *

lib.php

Doc Comments +12 added lines, -1 removed lines

@@ -226,7 +226,7 @@ 
  * $return->time = the time they did it
  * $return->info = a short text description.
  *
- * @return bool
+ * @return string
  */
 function bigbluebuttonbn_user_outline($course, $user, $mod, $bigbluebuttonbn) {
     global $DB;
@@ -315,6 +315,9 @@ 
     }
 }
 
+/**
+ * @param integer $now
+ */
 function bigbluebuttonbn_print_overview_element($bigbluebuttonbn, $now) {
     global $CFG;
 
@@ -550,6 +553,11 @@ 
     send_stored_file($file, 0, 0, $forcedownload, $options); // download MUST be forced - security!
 }
 
+/**
+ * @param stdClass $course
+ * @param stdClass $cm
+ * @param stdClass $context
+ */
 function bigbluebuttonbn_pluginfile_filename($course, $cm, $context, $args) {
     global $DB;
 
@@ -644,6 +652,9 @@ 
     bigbluebuttonbn_notification_send($USER, $bigbluebuttonbn, bigbluebuttonbn_notification_msg_html($msg));
 }
 
+/**
+ * @param stdClass $msg
+ */
 function bigbluebuttonbn_notification_msg_html($msg) {
     $messagetext = '<p>'.$msg->activity_type.' &quot;'.$msg->activity_title.'&quot; '.
         get_string('email_body_notification_meeting_has_been', 'bigbluebuttonbn').' '.$msg->action.'.</p>'."\n";

Indentation +8 added lines, -8 removed lines

@@ -232,9 +232,9 @@ 
     global $DB;
 
     $completed = $DB->count_records('bigbluebuttonbn_logs', array('courseid' => $course->id,
-                                                              'bigbluebuttonbnid' => $bigbluebuttonbn->id,
-                                                              'userid' => $user->id,
-                                                              'log' => 'Join', ), '*');
+                                                                'bigbluebuttonbnid' => $bigbluebuttonbn->id,
+                                                                'userid' => $user->id,
+                                                                'log' => 'Join', ), '*');
 
     if ($completed > 0) {
         return fullname($user).' '.get_string('view_message_has_joined', 'bigbluebuttonbn').' '.
@@ -254,9 +254,9 @@ 
 function bigbluebuttonbn_user_complete($course, $user, $mod, $bigbluebuttonbn) {
     global $DB;
     $completed = $DB->count_recorda('bigbluebuttonbn_logs', array('courseid' => $course->id,
-                                                              'bigbluebuttonbnid' => $bigbluebuttonbn->id,
-                                                              'userid' => $user->id,
-                                                              'log' => 'Join', ), '*', IGNORE_MULTIPLE);
+                                                                'bigbluebuttonbnid' => $bigbluebuttonbn->id,
+                                                                'userid' => $user->id,
+                                                                'log' => 'Join', ), '*', IGNORE_MULTIPLE);
 
     return $completed > 0;
 }
@@ -329,12 +329,12 @@ 
     $str = '<div class="bigbluebuttonbn overview">'."\n";
     $str .= '  <div class="name">'.get_string('modulename', 'bigbluebuttonbn').':&nbsp;'."\n";
     $str .= '    <a '.$classes.'href="'.$CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$bigbluebuttonbn->coursemodule.
-      '">'.$bigbluebuttonbn->name.'</a>'."\n";
+        '">'.$bigbluebuttonbn->name.'</a>'."\n";
     $str .= '  </div>'."\n";
     $str .= '  <div class="info">'.get_string($start, 'bigbluebuttonbn').': '.userdate($bigbluebuttonbn->openingtime).
         '</div>'."\n";
     $str .= '  <div class="info">'.get_string('ends_at', 'bigbluebuttonbn').': '.userdate($bigbluebuttonbn->closingtime)
-      .'</div>'."\n";
+        .'</div>'."\n";
     $str .= '</div>'."\n";
 
     return $str;

Spacing +70 added lines, -70 removed lines

@@ -27,32 +27,32 @@ 
 
 global $CFG;
 
-require_once($CFG->dirroot.'/calendar/lib.php');
-require_once($CFG->dirroot.'/message/lib.php');
-require_once($CFG->dirroot.'/mod/lti/OAuth.php');
-require_once($CFG->libdir.'/accesslib.php');
-require_once($CFG->libdir.'/completionlib.php');
-require_once($CFG->libdir.'/datalib.php');
-require_once($CFG->libdir.'/coursecatlib.php');
-require_once($CFG->libdir.'/enrollib.php');
-require_once($CFG->libdir.'/filelib.php');
-require_once($CFG->libdir.'/formslib.php');
-
-if (file_exists(dirname(__FILE__).'/vendor/firebase/php-jwt/src/JWT.php')) {
-    require_once(dirname(__FILE__).'/vendor/firebase/php-jwt/src/JWT.php');
+require_once($CFG->dirroot . '/calendar/lib.php');
+require_once($CFG->dirroot . '/message/lib.php');
+require_once($CFG->dirroot . '/mod/lti/OAuth.php');
+require_once($CFG->libdir . '/accesslib.php');
+require_once($CFG->libdir . '/completionlib.php');
+require_once($CFG->libdir . '/datalib.php');
+require_once($CFG->libdir . '/coursecatlib.php');
+require_once($CFG->libdir . '/enrollib.php');
+require_once($CFG->libdir . '/filelib.php');
+require_once($CFG->libdir . '/formslib.php');
+
+if (file_exists(dirname(__FILE__) . '/vendor/firebase/php-jwt/src/JWT.php')) {
+    require_once(dirname(__FILE__) . '/vendor/firebase/php-jwt/src/JWT.php');
 }
 
 if (!isset($CFG->bigbluebuttonbn)) {
     $CFG->bigbluebuttonbn = array();
 }
 
-if (file_exists(dirname(__FILE__).'/config.php')) {
-    require_once(dirname(__FILE__).'/config.php');
+if (file_exists(dirname(__FILE__) . '/config.php')) {
+    require_once(dirname(__FILE__) . '/config.php');
     // Old BigBlueButtonBN cfg schema. For backward compatibility.
     global $BIGBLUEBUTTONBN_CFG;
 
     if (isset($BIGBLUEBUTTONBN_CFG)) {
-        foreach ((array) $BIGBLUEBUTTONBN_CFG as $key => $value) {
+        foreach ((array)$BIGBLUEBUTTONBN_CFG as $key => $value) {
             $cfgkey = str_replace("bigbluebuttonbn_", "", $key);
             $CFG->bigbluebuttonbn[$cfgkey] = $value;
         }
@@ -82,19 +82,19 @@ 
     }
 
     $features = array(
-        (string) FEATURE_IDNUMBER => true,
-        (string) FEATURE_GROUPS => true,
-        (string) FEATURE_GROUPINGS => true,
-        (string) FEATURE_GROUPMEMBERSONLY => true,
-        (string) FEATURE_MOD_INTRO => true,
-        (string) FEATURE_BACKUP_MOODLE2 => true,
-        (string) FEATURE_COMPLETION_TRACKS_VIEWS => true,
-        (string) FEATURE_GRADE_HAS_GRADE => false,
-        (string) FEATURE_GRADE_OUTCOMES => false,
-        (string) FEATURE_SHOW_DESCRIPTION => true,
+        (string)FEATURE_IDNUMBER => true,
+        (string)FEATURE_GROUPS => true,
+        (string)FEATURE_GROUPINGS => true,
+        (string)FEATURE_GROUPMEMBERSONLY => true,
+        (string)FEATURE_MOD_INTRO => true,
+        (string)FEATURE_BACKUP_MOODLE2 => true,
+        (string)FEATURE_COMPLETION_TRACKS_VIEWS => true,
+        (string)FEATURE_GRADE_HAS_GRADE => false,
+        (string)FEATURE_GRADE_OUTCOMES => false,
+        (string)FEATURE_SHOW_DESCRIPTION => true,
     );
 
-    if (isset($features[(string) $feature])) {
+    if (isset($features[(string)$feature])) {
         return $features[$feature];
     }
 
@@ -174,7 +174,7 @@ 
     }
 
     // End the session associated with this instance (if it's running).
-    $meetingid = $bigbluebuttonbn->meetingid.'-'.$bigbluebuttonbn->course.'-'.$bigbluebuttonbn->id;
+    $meetingid = $bigbluebuttonbn->meetingid . '-' . $bigbluebuttonbn->course . '-' . $bigbluebuttonbn->id;
     $modpw = $bigbluebuttonbn->moderatorpass;
 
     if (bigbluebuttonbn_is_meeting_running($meetingid)) {
@@ -234,11 +234,11 @@ 
     $completed = $DB->count_records('bigbluebuttonbn_logs', array('courseid' => $course->id,
                                                               'bigbluebuttonbnid' => $bigbluebuttonbn->id,
                                                               'userid' => $user->id,
-                                                              'log' => 'Join', ), '*');
+                                                              'log' => 'Join',), '*');
 
     if ($completed > 0) {
-        return fullname($user).' '.get_string('view_message_has_joined', 'bigbluebuttonbn').' '.
-            get_string('view_message_session_for', 'bigbluebuttonbn').' '.(string) $completed.' '.
+        return fullname($user) . ' ' . get_string('view_message_has_joined', 'bigbluebuttonbn') . ' ' .
+            get_string('view_message_session_for', 'bigbluebuttonbn') . ' ' . (string)$completed . ' ' .
             get_string('view_message_times', 'bigbluebuttonbn');
     }
 
@@ -256,7 +256,7 @@ 
     $completed = $DB->count_recorda('bigbluebuttonbn_logs', array('courseid' => $course->id,
                                                               'bigbluebuttonbnid' => $bigbluebuttonbn->id,
                                                               'userid' => $user->id,
-                                                              'log' => 'Join', ), '*', IGNORE_MULTIPLE);
+                                                              'log' => 'Join',), '*', IGNORE_MULTIPLE);
 
     return $completed > 0;
 }
@@ -326,16 +326,16 @@ 
     if ($bigbluebuttonbn->visible) {
         $classes = 'class="dimmed" ';
     }
-    $str = '<div class="bigbluebuttonbn overview">'."\n";
-    $str .= '  <div class="name">'.get_string('modulename', 'bigbluebuttonbn').':&nbsp;'."\n";
-    $str .= '    <a '.$classes.'href="'.$CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$bigbluebuttonbn->coursemodule.
-      '">'.$bigbluebuttonbn->name.'</a>'."\n";
-    $str .= '  </div>'."\n";
-    $str .= '  <div class="info">'.get_string($start, 'bigbluebuttonbn').': '.userdate($bigbluebuttonbn->openingtime).
-        '</div>'."\n";
-    $str .= '  <div class="info">'.get_string('ends_at', 'bigbluebuttonbn').': '.userdate($bigbluebuttonbn->closingtime)
-      .'</div>'."\n";
-    $str .= '</div>'."\n";
+    $str = '<div class="bigbluebuttonbn overview">' . "\n";
+    $str .= '  <div class="name">' . get_string('modulename', 'bigbluebuttonbn') . ':&nbsp;' . "\n";
+    $str .= '    <a ' . $classes . 'href="' . $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $bigbluebuttonbn->coursemodule .
+      '">' . $bigbluebuttonbn->name . '</a>' . "\n";
+    $str .= '  </div>' . "\n";
+    $str .= '  <div class="info">' . get_string($start, 'bigbluebuttonbn') . ': ' . userdate($bigbluebuttonbn->openingtime) .
+        '</div>' . "\n";
+    $str .= '  <div class="info">' . get_string('ends_at', 'bigbluebuttonbn') . ': ' . userdate($bigbluebuttonbn->closingtime)
+      .'</div>' . "\n";
+    $str .= '</div>' . "\n";
 
     return $str;
 }
@@ -426,7 +426,7 @@ 
     /* Now that an id was assigned, generate and set the meetingid property based on
      * [Moodle Instance + Activity ID + BBB Secret] (but only for new activities) */
     if (isset($bigbluebuttonbn->add) && !empty($bigbluebuttonbn->add)) {
-        $meetingid = sha1($CFG->wwwroot.$bigbluebuttonbn->id.bigbluebuttonbn_get_cfg_shared_secret());
+        $meetingid = sha1($CFG->wwwroot . $bigbluebuttonbn->id . bigbluebuttonbn_get_cfg_shared_secret());
         $DB->set_field('bigbluebuttonbn', 'meetingid', $meetingid, array('id' => $bigbluebuttonbn->id));
 
         $action = get_string('mod_form_field_notification_msg_created', 'bigbluebuttonbn');
@@ -500,7 +500,7 @@ 
         // Get the first (and only) file.
         $file = reset($files);
         // Set the presentation column in the bigbluebuttonbn table.
-        $DB->set_field('bigbluebuttonbn', 'presentation', '/'.$file->get_filename(), array('id' => $bigbluebuttonbnid));
+        $DB->set_field('bigbluebuttonbn', 'presentation', '/' . $file->get_filename(), array('id' => $bigbluebuttonbnid));
     } else {
         // Set the presentation column in the bigbluebuttonbn table.
         $DB->set_field('bigbluebuttonbn', 'presentation', '', array('id' => $bigbluebuttonbnid));
@@ -540,7 +540,7 @@ 
         return false;
     }
 
-    $fullpath = "/$context->id/mod_bigbluebuttonbn/$filearea/0/".$filename;
+    $fullpath = "/$context->id/mod_bigbluebuttonbn/$filearea/0/" . $filename;
     $fs = get_file_storage();
     if (!$file = $fs->get_file_by_hash(sha1($fullpath)) or $file->is_directory()) {
         return false;
@@ -645,26 +645,26 @@ 
 }
 
 function bigbluebuttonbn_notification_msg_html($msg) {
-    $messagetext = '<p>'.$msg->activity_type.' &quot;'.$msg->activity_title.'&quot; '.
-        get_string('email_body_notification_meeting_has_been', 'bigbluebuttonbn').' '.$msg->action.'.</p>'."\n";
-    $messagetext .= '<p><b>'.$msg->activity_title.'</b> '.
-        get_string('email_body_notification_meeting_details', 'bigbluebuttonbn').':'."\n";
-    $messagetext .= '<table border="0" style="margin: 5px 0 0 20px"><tbody>'."\n";
-    $messagetext .= '<tr><td style="font-weight:bold;color:#555;">'.
-        get_string('email_body_notification_meeting_title', 'bigbluebuttonbn').': </td><td>'."\n";
-    $messagetext .= $msg->activity_title.'</td></tr>'."\n";
-    $messagetext .= '<tr><td style="font-weight:bold;color:#555;">'.
-        get_string('email_body_notification_meeting_description', 'bigbluebuttonbn').': </td><td>'."\n";
-    $messagetext .= $msg->activity_description.'</td></tr>'."\n";
-    $messagetext .= '<tr><td style="font-weight:bold;color:#555;">'.
-        get_string('email_body_notification_meeting_start_date', 'bigbluebuttonbn').': </td><td>'."\n";
-    $messagetext .= $msg->activity_openingtime.'</td></tr>'."\n";
-    $messagetext .= '<tr><td style="font-weight:bold;color:#555;">'.
-        get_string('email_body_notification_meeting_end_date', 'bigbluebuttonbn').': </td><td>'."\n";
-    $messagetext .= $msg->activity_closingtime.'</td></tr>'."\n";
-    $messagetext .= '<tr><td style="font-weight:bold;color:#555;">'.$msg->action.' '.
-        get_string('email_body_notification_meeting_by', 'bigbluebuttonbn').': </td><td>'."\n";
-    $messagetext .= $msg->activity_owner.'</td></tr></tbody></table></p>'."\n";
+    $messagetext = '<p>' . $msg->activity_type . ' &quot;' . $msg->activity_title . '&quot; ' .
+        get_string('email_body_notification_meeting_has_been', 'bigbluebuttonbn') . ' ' . $msg->action . '.</p>' . "\n";
+    $messagetext .= '<p><b>' . $msg->activity_title . '</b> ' .
+        get_string('email_body_notification_meeting_details', 'bigbluebuttonbn') . ':' . "\n";
+    $messagetext .= '<table border="0" style="margin: 5px 0 0 20px"><tbody>' . "\n";
+    $messagetext .= '<tr><td style="font-weight:bold;color:#555;">' .
+        get_string('email_body_notification_meeting_title', 'bigbluebuttonbn') . ': </td><td>' . "\n";
+    $messagetext .= $msg->activity_title . '</td></tr>' . "\n";
+    $messagetext .= '<tr><td style="font-weight:bold;color:#555;">' .
+        get_string('email_body_notification_meeting_description', 'bigbluebuttonbn') . ': </td><td>' . "\n";
+    $messagetext .= $msg->activity_description . '</td></tr>' . "\n";
+    $messagetext .= '<tr><td style="font-weight:bold;color:#555;">' .
+        get_string('email_body_notification_meeting_start_date', 'bigbluebuttonbn') . ': </td><td>' . "\n";
+    $messagetext .= $msg->activity_openingtime . '</td></tr>' . "\n";
+    $messagetext .= '<tr><td style="font-weight:bold;color:#555;">' .
+        get_string('email_body_notification_meeting_end_date', 'bigbluebuttonbn') . ': </td><td>' . "\n";
+    $messagetext .= $msg->activity_closingtime . '</td></tr>' . "\n";
+    $messagetext .= '<tr><td style="font-weight:bold;color:#555;">' . $msg->action . ' ' .
+        get_string('email_body_notification_meeting_by', 'bigbluebuttonbn') . ': </td><td>' . "\n";
+    $messagetext .= $msg->activity_owner . '</td></tr></tbody></table></p>' . "\n";
 }
 
 function bigbluebuttonbn_notification_send($sender, $bigbluebuttonbn, $message = '') {
@@ -678,18 +678,18 @@ 
     $msg->user_name = fullname($sender);
     $msg->user_email = $sender->email;
     $msg->course_name = "$course->fullname";
-    $message .= '<p><hr/><br/>'.get_string('email_footer_sent_by', 'bigbluebuttonbn').' '.
-        $msg->user_name.'('.$msg->user_email.') ';
-    $message .= get_string('email_footer_sent_from', 'bigbluebuttonbn').' '.$msg->course_name.'.</p>';
+    $message .= '<p><hr/><br/>' . get_string('email_footer_sent_by', 'bigbluebuttonbn') . ' ' .
+        $msg->user_name . '(' . $msg->user_email . ') ';
+    $message .= get_string('email_footer_sent_from', 'bigbluebuttonbn') . ' ' . $msg->course_name . '.</p>';
 
     $users = get_enrolled_users($context);
     foreach ($users as $user) {
         if ($user->id != $sender->id) {
             $messageid = message_post_message($sender, $user, $message, FORMAT_HTML);
             if (!empty($messageid)) {
-                debugging('Msg to '.$msg->user_name.' was sent.', DEBUG_DEVELOPER);
+                debugging('Msg to ' . $msg->user_name . ' was sent.', DEBUG_DEVELOPER);
             } else {
-                debugging('Msg to '.$msg->user_name.' was NOT sent.', DEBUG_DEVELOPER);
+                debugging('Msg to ' . $msg->user_name . ' was NOT sent.', DEBUG_DEVELOPER);
             }
         }
     }

db/upgrade.php

Indentation +36 added lines, -36 removed lines

@@ -36,23 +36,23 @@ 
 
         // Change welcome, allow null.
         $fielddefinition = array('type' => XMLDB_TYPE_TEXT,
-                                  'precision' => null,
-                                  'unsigned' => null,
-                                  'notnull' => XMLDB_NOTNULL,
-                                  'sequence' => null,
-                                  'default' => null,
-                                  'previous' => 'type');
+                                    'precision' => null,
+                                    'unsigned' => null,
+                                    'notnull' => XMLDB_NOTNULL,
+                                    'sequence' => null,
+                                    'default' => null,
+                                    'previous' => 'type');
         xmldb_bigbluebuttonbn_add_change_field($dbman, 'bigbluebuttonbn', 'welcome',
             $fielddefinition);
 
         // Change userid definition in bigbluebuttonbn_log.
         $fielddefinition = array('type' => XMLDB_TYPE_INTEGER,
-                                  'precision' => '10',
-                                  'unsigned' => XMLDB_UNSIGNED,
-                                  'notnull' => XMLDB_NOTNULL,
-                                  'sequence' => null,
-                                  'default' => null,
-                                  'previous' => 'bigbluebuttonbnid');
+                                    'precision' => '10',
+                                    'unsigned' => XMLDB_UNSIGNED,
+                                    'notnull' => XMLDB_NOTNULL,
+                                    'sequence' => null,
+                                    'default' => null,
+                                    'previous' => 'bigbluebuttonbnid');
         xmldb_bigbluebuttonbn_add_change_field($dbman, 'bigbluebuttonbn_log', 'userid',
             $fielddefinition);
 
@@ -78,34 +78,34 @@ 
 
         // Add field type.
         $fielddefinition = array('type' => XMLDB_TYPE_INTEGER,
-                                  'precision' => '2',
-                                  'unsigned' => XMLDB_UNSIGNED,
-                                  'notnull' => XMLDB_NOTNULL,
-                                  'sequence' => null,
-                                  'default' => 0,
-                                  'previous' => 'id');
+                                    'precision' => '2',
+                                    'unsigned' => XMLDB_UNSIGNED,
+                                    'notnull' => XMLDB_NOTNULL,
+                                    'sequence' => null,
+                                    'default' => 0,
+                                    'previous' => 'id');
         xmldb_bigbluebuttonbn_add_change_field($dbman, 'bigbluebuttonbn', 'type',
             $fielddefinition);
 
         // Add field recordings_html.
         $fielddefinition = array('type' => XMLDB_TYPE_INTEGER,
-                                  'precision' => '1',
-                                  'unsigned' => XMLDB_UNSIGNED,
-                                  'notnull' => XMLDB_NOTNULL,
-                                  'sequence' => null,
-                                  'default' => 0,
-                                  'previous' => null);
+                                    'precision' => '1',
+                                    'unsigned' => XMLDB_UNSIGNED,
+                                    'notnull' => XMLDB_NOTNULL,
+                                    'sequence' => null,
+                                    'default' => 0,
+                                    'previous' => null);
         xmldb_bigbluebuttonbn_add_change_field($dbman, 'bigbluebuttonbn', 'recordings_html',
             $fielddefinition);
 
         // Add field recordings_deleted_activities.
         $fielddefinition = array('type' => XMLDB_TYPE_INTEGER,
-                                  'precision' => '1',
-                                  'unsigned' => XMLDB_UNSIGNED,
-                                  'notnull' => XMLDB_NOTNULL,
-                                  'sequence' => null,
-                                  'default' => 1,
-                                  'previous' => null);
+                                    'precision' => '1',
+                                    'unsigned' => XMLDB_UNSIGNED,
+                                    'notnull' => XMLDB_NOTNULL,
+                                    'sequence' => null,
+                                    'default' => 1,
+                                    'previous' => null);
         xmldb_bigbluebuttonbn_add_change_field($dbman, 'bigbluebuttonbn', 'recordings_deleted_activities',
             $fielddefinition);
 
@@ -119,12 +119,12 @@ 
     $table = new xmldb_table($tablename);
     $field = new xmldb_field($fieldname);
     $field->set_attributes($fielddefinition['type'],
-                           $fielddefinition['precision'],
-                           $fielddefinition['unsigned'],
-                           $fielddefinition['notnull'],
-                           $fielddefinition['sequence'],
-                           $fielddefinition['default'],
-                           $fielddefinition['previous']);
+                            $fielddefinition['precision'],
+                            $fielddefinition['unsigned'],
+                            $fielddefinition['notnull'],
+                            $fielddefinition['sequence'],
+                            $fielddefinition['default'],
+                            $fielddefinition['previous']);
     if ($dbman->field_exists($table, $field)) {
         $dbman->change_field($table, $field, true, true);
 

bbb_broker.php

Indentation +9 added lines, -9 removed lines

@@ -167,14 +167,14 @@ 
     }
     $statuscanjoin = '"can_join": '.($canjoin ? 'true' : 'false');
     echo $params['callback'].'({"running": '.($meetingrunning ? 'true' : 'false').
-                              ',"info": '.json_encode($meetinginfo).
-                              ',"status": {'.'"join_url": "'.$bbbsession['joinURL'].'", '.
-                                             '"joinbuttontext": "'.$joinbuttontext.'", '.
-                                             '"message": "'.$initialmessage.'", '.
-                                             $statuscanjoin.', '.
-                                             $statuscanend.', '.
-                                             $statuscantag.', '.
-                                          '}});';
+                                ',"info": '.json_encode($meetinginfo).
+                                ',"status": {'.'"join_url": "'.$bbbsession['joinURL'].'", '.
+                                                '"joinbuttontext": "'.$joinbuttontext.'", '.
+                                                '"message": "'.$initialmessage.'", '.
+                                                $statuscanjoin.', '.
+                                                $statuscanend.', '.
+                                                $statuscantag.', '.
+                                            '}});';
 }
 
 function bigbluebuttonbn_broker_meeting_end($bbbsession, $params, $bigbluebuttonbn, $cm) {
@@ -297,7 +297,7 @@ 
     if (isset($recordings[$params['id']]) && isset($recordings[$params['id']]['imported'])) {
         // Execute publish on imported recording link, if the real recording is published.
         $realrecordings = bigbluebuttonbn_get_recordings_array($recordings[$params['id']]['meetingID'],
-                                                             $recordings[$params['id']]['recordID']);
+                                                                $recordings[$params['id']]['recordID']);
         $status = $realrecordings[$params['id']]['published'];
         if ($status === 'true') {
             // Only if the physical recording is published, execute publish on imported recording link.

Spacing +36 added lines, -36 removed lines

@@ -24,8 +24,8 @@ 
 
 defined('MOODLE_INTERNAL') || die();
 
-require_once(dirname(dirname(dirname(__FILE__))).'/config.php');
-require_once(dirname(__FILE__).'/locallib.php');
+require_once(dirname(dirname(dirname(__FILE__))) . '/config.php');
+require_once(dirname(__FILE__) . '/locallib.php');
 
 use \Firebase\JWT\JWT;
 
@@ -118,12 +118,12 @@ 
                     break;
             }
         } catch (Exception $e) {
-            header('HTTP/1.0 500 Internal Server Error. '.$e->getMessage());
+            header('HTTP/1.0 500 Internal Server Error. ' . $e->getMessage());
             return;
         }
     }
 } else {
-    header('HTTP/1.0 400 Bad Request. '.$error);
+    header('HTTP/1.0 400 Bad Request. ' . $error);
 }
 
 function bigbluebuttonbn_broker_meeting_info($bbbsession, $params) {
@@ -144,7 +144,7 @@ 
 
         if ($bbbsession['administrator'] || $bbbsession['moderator']) {
             $endbuttontext = get_string('view_conference_action_end', 'bigbluebuttonbn');
-            $statuscanend = '"can_end": true, "endbuttontext": "'.$endbuttontext.'"';
+            $statuscanend = '"can_end": true, "endbuttontext": "' . $endbuttontext . '"';
         }
     } else {
         // If user is administrator, moderator or if is viewer and no waiting is required.
@@ -154,7 +154,7 @@ 
         if (!$bbbsession['administrator'] && !$bbbsession['moderator'] && $bbbsession['wait']) {
             $initialmessage = get_string('view_message_conference_not_started', 'bigbluebuttonbn');
             if ($bbbsession['wait']) {
-                $initialmessage .= ' '.get_string('view_message_conference_wait_for_moderator', 'bigbluebuttonbn');
+                $initialmessage .= ' ' . get_string('view_message_conference_wait_for_moderator', 'bigbluebuttonbn');
             }
             $canjoin = false;
         }
@@ -163,17 +163,17 @@ 
         if ($bbbsession['tagging'] && ($bbbsession['administrator'] || $bbbsession['moderator'])) {
             $cantag = true;
         }
-        $statuscanend = '"can_tag": '.($cantag ? 'true' : 'false');
+        $statuscanend = '"can_tag": ' . ($cantag ? 'true' : 'false');
     }
-    $statuscanjoin = '"can_join": '.($canjoin ? 'true' : 'false');
-    echo $params['callback'].'({"running": '.($meetingrunning ? 'true' : 'false').
-                              ',"info": '.json_encode($meetinginfo).
-                              ',"status": {'.'"join_url": "'.$bbbsession['joinURL'].'", '.
-                                             '"joinbuttontext": "'.$joinbuttontext.'", '.
-                                             '"message": "'.$initialmessage.'", '.
-                                             $statuscanjoin.', '.
-                                             $statuscanend.', '.
-                                             $statuscantag.', '.
+    $statuscanjoin = '"can_join": ' . ($canjoin ? 'true' : 'false');
+    echo $params['callback'] . '({"running": ' . ($meetingrunning ? 'true' : 'false') .
+                              ',"info": ' . json_encode($meetinginfo) .
+                              ',"status": {' . '"join_url": "' . $bbbsession['joinURL'] . '", ' .
+                                             '"joinbuttontext": "' . $joinbuttontext . '", ' .
+                                             '"message": "' . $initialmessage . '", ' .
+                                             $statuscanjoin . ', ' .
+                                             $statuscanend . ', ' .
+                                             $statuscantag . ', ' .
                                           '}});';
 }
 
@@ -193,7 +193,7 @@ 
     // Update the cache.
     bigbluebuttonbn_get_meetinginfo($params['id'], BIGBLUEBUTTONBN_FORCED);
 
-    echo $params['callback'].'({ "status": true });';
+    echo $params['callback'] . '({ "status": true });';
 }
 
 function bigbluebuttonbn_broker_recording_links($bbbsession, $params) {
@@ -203,10 +203,10 @@ 
         return;
     }
 
-    $out = $params['callback'].'({"status": "false"});';
+    $out = $params['callback'] . '({"status": "false"});';
     if (isset($params['id']) && $params['id'] != '') {
         $importedall = bigbluebuttonbn_get_recording_imported_instances($params['id']);
-        $out = $params['callback'].'({ "status": "true", "links": '.count($importedall).'});';
+        $out = $params['callback'] . '({ "status": "true", "links": ' . count($importedall) . '});';
     }
     echo $out;
 }
@@ -228,10 +228,10 @@ 
     if (isset($recordings[$params['id']])) {
         // Look up for an update on the imported recording.
         $recording = $recordings[$params['id']];
-        $out = $params['callback'].'({ "status": "false" });';
+        $out = $params['callback'] . '({ "status": "false" });';
         if (isset($recording) && !empty($recording) && !array_key_exists('messageKey', $recording)) {
             // The recording was found.
-            $out = $params['callback'].'({ "status": "true", "published": "'.$recording['published'].'"});';
+            $out = $params['callback'] . '({ "status": "true", "published": "' . $recording['published'] . '"});';
         }
         echo $out;
         return;
@@ -239,10 +239,10 @@ 
 
     // As the recordingid was not identified as imported recording link, look up for a real recording.
     $recording = bigbluebuttonbn_get_recordings_array($params['idx'], $params['id']);
-    $out = $params['callback'].'({"status": "false"});';
+    $out = $params['callback'] . '({"status": "false"});';
     if (isset($recording) && !empty($recording) && array_key_exists($params['id'], $recording)) {
         // The recording was found.
-        $out = $params['callback'].'({ "status": "true", "published": "'.$recording[$params['id']]['published'].'"});';
+        $out = $params['callback'] . '({ "status": "true", "published": "' . $recording[$params['id']]['published'] . '"});';
     }
     echo $out;
 }
@@ -273,7 +273,7 @@ 
             $eventlog = BIGBLUEBUTTON_EVENT_RECORDING_PUBLISHED;
             break;
         case 'recording_unpublish':
-            $callbackresponse = bigbluebuttonbn_broker_recording_action_unpublish($bbbsession, $params, $recordings);;
+            $callbackresponse = bigbluebuttonbn_broker_recording_action_unpublish($bbbsession, $params, $recordings); ;
             $eventlog = BIGBLUEBUTTON_EVENT_RECORDING_UNPUBLISHED;
             break;
         case 'recording_delete':
@@ -376,8 +376,8 @@ 
         $decodedparameters = JWT::decode($params['signed_parameters'], bigbluebuttonbn_get_cfg_shared_secret(),
             array('HS256'));
     } catch (Exception $e) {
-        $error = 'Caught exception: '.$e->getMessage();
-        header('HTTP/1.0 400 Bad Request. '.$error);
+        $error = 'Caught exception: ' . $e->getMessage();
+        header('HTTP/1.0 400 Bad Request. ' . $error);
         return;
     }
 
@@ -386,8 +386,8 @@ 
     $meetingidelements = explode('-', $meetingidelements[0]);
 
     if (!isset($bigbluebuttonbn) || $bigbluebuttonbn->meetingid != $meetingidelements[0]) {
-        $error = 'Caught exception: '.$e->getMessage();
-        header('HTTP/1.0 410 Gone. '.$error);
+        $error = 'Caught exception: ' . $e->getMessage();
+        header('HTTP/1.0 410 Gone. ' . $error);
         return;
     }
 
@@ -396,8 +396,8 @@ 
         bigbluebuttonbn_send_notification_recording_ready($bigbluebuttonbn);
         header('HTTP/1.0 202 Accepted');
     } catch (Exception $e) {
-        $error = 'Caught exception: '.$e->getMessage();
-        header('HTTP/1.0 503 Service Unavailable. '.$error);
+        $error = 'Caught exception: ' . $e->getMessage();
+        header('HTTP/1.0 503 Service Unavailable. ' . $error);
     }
 }
 
@@ -412,13 +412,13 @@ 
     $importrecordings = $SESSION->bigbluebuttonbn_importrecordings;
     if (!isset($importrecordings[$params['id']])) {
         $error = "Recording {$params['id']} could not be found. It can not be imported";
-        header('HTTP/1.0 404 Not found. '.$error);
+        header('HTTP/1.0 404 Not found. ' . $error);
         return;
     }
 
     $importrecordings[$params['id']]['imported'] = true;
     $overrides['meetingid'] = $importrecordings[$params['id']]['meetingID'];
-    $meta = '{"recording":'.json_encode($importrecordings[$params['id']]).'}';
+    $meta = '{"recording":' . json_encode($importrecordings[$params['id']]) . '}';
     bigbluebuttonbn_logs($bbbsession, BIGBLUEBUTTONBN_LOG_EVENT_IMPORT, $overrides, $meta);
     // Moodle event logger: Create an event for recording imported.
     if (isset($bigbluebuttonbn)) {
@@ -436,8 +436,8 @@ 
         $decodedparameters = JWT::decode($params['signed_parameters'], bigbluebuttonbn_get_cfg_shared_secret(),
             array('HS256'));
     } catch (Exception $e) {
-        $error = 'Caught exception: '.$e->getMessage();
-        header('HTTP/1.0 400 Bad Request. '.$error);
+        $error = 'Caught exception: ' . $e->getMessage();
+        header('HTTP/1.0 400 Bad Request. ' . $error);
         return;
     }
 
@@ -446,8 +446,8 @@ 
     $meetingidelements = explode('-', $meetingidelements[0]);
 
     if (!isset($bigbluebuttonbn) || $bigbluebuttonbn->meetingid != $meetingidelements[0]) {
-        $error = 'Caught exception: '.$e->getMessage();
-        header('HTTP/1.0 410 Gone. '.$error);
+        $error = 'Caught exception: ' . $e->getMessage();
+        header('HTTP/1.0 410 Gone. ' . $error);
         return;
     }
 

classes/event/bigbluebuttonbn_activity_viewed.php

Spacing +3 added lines, -3 removed lines

@@ -52,11 +52,11 @@ 
      * @return string
      */
     public function get_description() {
-        $a = (object) array('userid' => $this->userid,
+        $a = (object)array('userid' => $this->userid,
                             'bigbluebuttonbnid' => $this->objectid,
                             'courseid' => $this->contextinstanceid);
 
-        return "The user with id '$a->userid' viewed the bigbluebuttonbn activity ".
+        return "The user with id '$a->userid' viewed the bigbluebuttonbn activity " .
         "with id '$a->bigbluebuttonbnid' for the course id '$a->courseid'.";
     }
 
@@ -67,7 +67,7 @@ 
      */
     protected function get_legacy_logdata() {
         return array($this->courseid, 'bigbluebuttonbn', 'activity viewed',
-                'view.php?pageid='.$this->objectid, 'BigBlueButtonBN activity viewed',
+                'view.php?pageid=' . $this->objectid, 'BigBlueButtonBN activity viewed',
                 $this->contextinstanceid);
     }
 

classes/event/bigbluebuttonbn_recording_imported.php

Spacing +3 added lines, -3 removed lines

@@ -53,11 +53,11 @@ 
      */
     public function get_description() {
         $rid = isset($this->other['rid']) ? $this->other['rid'] : '';
-        $a = (object) array('userid' => $this->userid,
+        $a = (object)array('userid' => $this->userid,
                             'recordingid' => $rid,
                             'courseid' => $this->contextinstanceid);
 
-        return "The user with id '$a->userid' has imported a recording with id ".
+        return "The user with id '$a->userid' has imported a recording with id " .
         "'$a->recordingid' into the course id '$a->courseid'.";
     }
 
@@ -68,7 +68,7 @@ 
      */
     protected function get_legacy_logdata() {
         return array($this->courseid, 'bigbluebuttonbn', 'recording imported',
-                'view.php?pageid='.$this->objectid, 'Recording imported',
+                'view.php?pageid=' . $this->objectid, 'Recording imported',
                 $this->contextinstanceid);
     }
 

classes/event/bigbluebuttonbn_recording_published.php

Spacing +3 added lines, -3 removed lines

@@ -53,11 +53,11 @@ 
      */
     public function get_description() {
         $rid = isset($this->other['rid']) ? $this->other['rid'] : '';
-        $a = (object) array('userid' => $this->userid,
+        $a = (object)array('userid' => $this->userid,
                             'recordingid' => $rid,
                             'courseid' => $this->contextinstanceid);
 
-        return "The user with id '$a->userid' has published a recording with id ".
+        return "The user with id '$a->userid' has published a recording with id " .
         "'$a->recordingid' in the course id '$a->courseid'.";
     }
 
@@ -68,7 +68,7 @@ 
      */
     protected function get_legacy_logdata() {
         return array($this->courseid, 'bigbluebuttonbn', 'recording published',
-                'view.php?pageid='.$this->objectid, 'Recording published',
+                'view.php?pageid=' . $this->objectid, 'Recording published',
                 $this->contextinstanceid);
     }
 

classes/event/bigbluebuttonbn_recording_unpublished.php

Spacing +3 added lines, -3 removed lines

@@ -53,11 +53,11 @@ 
      */
     public function get_description() {
         $rid = isset($this->other['rid']) ? $this->other['rid'] : '';
-        $a = (object) array('userid' => $this->userid,
+        $a = (object)array('userid' => $this->userid,
                             'recordingid' => $rid,
                             'courseid' => $this->contextinstanceid);
 
-        return "The user with id '$a->userid' has unpublished a recording with id ".
+        return "The user with id '$a->userid' has unpublished a recording with id " .
         "'$a->recordingid' in the course id '$a->courseid'.";
     }
 
@@ -68,7 +68,7 @@ 
      */
     protected function get_legacy_logdata() {
         return array($this->courseid, 'bigbluebuttonbn', 'recording unpublished',
-                'view.php?pageid='.$this->objectid, 'Recording unpublished',
+                'view.php?pageid=' . $this->objectid, 'Recording unpublished',
                 $this->contextinstanceid);
     }
 

classes/event/bigbluebuttonbn_meeting_event.php

Spacing +4 added lines, -4 removed lines

@@ -52,13 +52,13 @@ 
      * @return string
      */
     public function get_description() {
-        $a = (object) array('userid' => $this->userid,
+        $a = (object)array('userid' => $this->userid,
                             'bigbluebuttonbnid' => $this->objectid,
                             'courseid' => $this->contextinstanceid,
                             'action' => $this->other);
 
-        return "The user with id '$a->userid' triggered action $a->action in a ".
-        "bigbluebutton meeting for the bigbluebuttonbn activity with id ".
+        return "The user with id '$a->userid' triggered action $a->action in a " .
+        "bigbluebutton meeting for the bigbluebuttonbn activity with id " .
         "'$a->bigbluebuttonbnid' for the course id '$a->courseid'.";
     }
 
@@ -69,7 +69,7 @@ 
      */
     protected function get_legacy_logdata() {
         return array($this->courseid, 'bigbluebuttonbn', 'meeting left',
-                'view.php?pageid='.$this->objectid, 'BigBlueButtonBN meeting live session event',
+                'view.php?pageid=' . $this->objectid, 'BigBlueButtonBN meeting live session event',
                 $this->contextinstanceid);
     }