blindsidenetworks / moodle-mod_bigbluebuttonbn

view.php

Indentation +9 added lines, -9 removed lines

@@ -272,11 +272,11 @@ 
 
     echo $OUTPUT->box_start('generalbox boxaligncenter');
     echo '<br><div class="alert alert-warning">'.get_string('view_groups_selection_warning', 'bigbluebuttonbn').
-      '</div>';
+        '</div>';
     echo $OUTPUT->box_end();
 
     groups_print_activity_menu(
-      $bbbsession['cm'], $CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$bbbsession['cm']->id, false, $onlyexistentgroups);
+        $bbbsession['cm'], $CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$bbbsession['cm']->id, false, $onlyexistentgroups);
     echo '<br><br>';
 }
 
@@ -388,7 +388,7 @@ 
     // JavaScript variables for recordings.
     $jsvars += array(
             'recordings_html' => $bbbsession['bigbluebuttonbn']->recordings_html == '1',
-          );
+            );
 
     // If there are meetings with recordings load the data to the table.
     if ($bbbsession['bigbluebuttonbn']->recordings_html) {
@@ -400,7 +400,7 @@ 
     $jsvars += array(
             'columns' => bigbluebuttonbn_get_recording_columns($bbbsession),
             'data' => bigbluebuttonbn_get_recording_data($bbbsession, $recordings),
-          );
+            );
 
     // Render a YUI table.
     return html_writer::div('', '', array('id' => 'bigbluebuttonbn_yui_table'));
@@ -411,10 +411,10 @@ 
 
     $button = html_writer::tag('input', '',
         array('type' => 'button',
-              'value' => get_string('view_recording_button_import', 'bigbluebuttonbn'),
-              'class' => 'btn btn-secondary',
-              'onclick' => 'window.location=\''.$CFG->wwwroot.'/mod/bigbluebuttonbn/import_view.php?bn='.
-                  $bbbsession['bigbluebuttonbn']->id.'\''));
+                'value' => get_string('view_recording_button_import', 'bigbluebuttonbn'),
+                'class' => 'btn btn-secondary',
+                'onclick' => 'window.location=\''.$CFG->wwwroot.'/mod/bigbluebuttonbn/import_view.php?bn='.
+                    $bbbsession['bigbluebuttonbn']->id.'\''));
     $output = html_writer::start_tag('br');
     $output .= html_writer::tag('span', $button, array('id' => 'import_recording_links_button'));
     $output .= html_writer::tag('span', '', array('id' => 'import_recording_links_table'));
@@ -432,7 +432,7 @@ 
         return '<h4>'.get_string('view_section_title_presentation', 'bigbluebuttonbn').'</h4>'.
                 ''.$OUTPUT->action_icon($bbbsession['presentation']['url'], $icon, null, array(), false).''.
                 ''.$OUTPUT->action_link($bbbsession['presentation']['url'],
-                      $bbbsession['presentation']['name'], null, $attributes).'<br><br>';
+                        $bbbsession['presentation']['name'], null, $attributes).'<br><br>';
     }
 
     return '';

Spacing +34 added lines, -34 removed lines

@@ -23,8 +23,8 @@ 
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v2 or later
  */
 
-require_once(dirname(dirname(dirname(__FILE__))).'/config.php');
-require_once(dirname(__FILE__).'/locallib.php');
+require_once(dirname(dirname(dirname(__FILE__))) . '/config.php');
+require_once(dirname(__FILE__) . '/locallib.php');
 
 $id = required_param('id', PARAM_INT);
 $b = optional_param('n', 0, PARAM_INT);
@@ -59,18 +59,18 @@ 
 if (is_null($serverversion)) {
     if ($bbbsession['administrator']) {
         print_error('view_error_unable_join', 'bigbluebuttonbn',
-            $CFG->wwwroot.'/admin/settings.php?section=modsettingbigbluebuttonbn');
+            $CFG->wwwroot . '/admin/settings.php?section=modsettingbigbluebuttonbn');
         exit;
     }
 
     if ($bbbsession['moderator']) {
         print_error('view_error_unable_join_teacher', 'bigbluebuttonbn',
-            $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course);
+            $CFG->wwwroot . '/course/view.php?id=' . $bigbluebuttonbn->course);
         exit;
     }
 
     print_error('view_error_unable_join_student', 'bigbluebuttonbn',
-        $CFG->wwwroot.'/course/view.php?id='.$bigbluebuttonbn->course);
+        $CFG->wwwroot . '/course/view.php?id=' . $bigbluebuttonbn->course);
     exit;
 }
 
@@ -80,7 +80,7 @@ 
 
 // Print the page header.
 $PAGE->set_context($context);
-$PAGE->set_url($CFG->wwwroot.'/mod/bigbluebuttonbn/view.php', array('id' => $cm->id));
+$PAGE->set_url($CFG->wwwroot . '/mod/bigbluebuttonbn/view.php', array('id' => $cm->id));
 $PAGE->set_title(format_string($bigbluebuttonbn->name));
 $PAGE->set_cacheable(false);
 $PAGE->set_heading($course->fullname);
@@ -90,11 +90,11 @@ 
 if (!has_capability('moodle/category:manage', $context) && !has_capability('mod/bigbluebuttonbn:join', $context)) {
     echo $OUTPUT->header();
     if (isguestuser()) {
-        echo $OUTPUT->confirm('<p>'.get_string('view_noguests', 'bigbluebuttonbn').'</p>'.get_string('liketologin'),
-            get_login_url(), $CFG->wwwroot.'/course/view.php?id='.$course->id);
+        echo $OUTPUT->confirm('<p>' . get_string('view_noguests', 'bigbluebuttonbn') . '</p>' . get_string('liketologin'),
+            get_login_url(), $CFG->wwwroot . '/course/view.php?id=' . $course->id);
     } else {
-        echo $OUTPUT->confirm('<p>'.get_string('view_nojoin', 'bigbluebuttonbn').'</p>'.get_string('liketologin'),
-            get_login_url(), $CFG->wwwroot.'/course/view.php?id='.$course->id);
+        echo $OUTPUT->confirm('<p>' . get_string('view_nojoin', 'bigbluebuttonbn') . '</p>' . get_string('liketologin'),
+            get_login_url(), $CFG->wwwroot . '/course/view.php?id=' . $course->id);
     }
     echo $OUTPUT->footer();
     exit;
@@ -102,7 +102,7 @@ 
 
 // Operation URLs.
 $bbbsession['bigbluebuttonbnURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $bbbsession['cm']->id;
-$bbbsession['logoutURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_view.php?action=logout&id='.$id .
+$bbbsession['logoutURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_view.php?action=logout&id=' . $id .
     '&bn=' . $bbbsession['bigbluebuttonbn']->id;
 $bbbsession['recordingReadyURL'] = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_broker.php?action=recording_' .
     'ready&bigbluebuttonbn=' . $bbbsession['bigbluebuttonbn']->id;
@@ -122,7 +122,7 @@ 
 echo $OUTPUT->footer();
 
 // Shows version as a comment.
-echo '<!-- '.$bbbsession['originTag'].' -->'."\n";
+echo '<!-- ' . $bbbsession['originTag'] . ' -->' . "\n";
 
 // Initialize session variable used across views.
 $SESSION->bigbluebuttonbn_bbbsession = $bbbsession;
@@ -152,7 +152,7 @@ 
     $bbbsession['viewerPW'] = $bigbluebuttonbn->viewerpass;
 
     // Database info related to the activity.
-    $bbbsession['meetingid'] = $bbbsession['bigbluebuttonbn']->meetingid.'-'.$bbbsession['course']->id.'-'.
+    $bbbsession['meetingid'] = $bbbsession['bigbluebuttonbn']->meetingid . '-' . $bbbsession['course']->id . '-' .
         $bbbsession['bigbluebuttonbn']->id;
     $bbbsession['meetingname'] = $bbbsession['bigbluebuttonbn']->name;
     $bbbsession['meetingdescription'] = $bigbluebuttonbn->intro;
@@ -173,7 +173,7 @@ 
         $bbbsession['welcome'] = get_string('mod_form_field_welcome_default', 'bigbluebuttonbn');
     }
     if ($bigbluebuttonbn->record) {
-        $bbbsession['welcome'] .= '<br><br>'.get_string('bbbrecordwarning', 'bigbluebuttonbn');
+        $bbbsession['welcome'] .= '<br><br>' . get_string('bbbrecordwarning', 'bigbluebuttonbn');
     }
 
     $bbbsession['openingtime'] = $bigbluebuttonbn->openingtime;
@@ -189,7 +189,7 @@ 
     $bbbsession['originServerName'] = $parsedurl['host'];
     $bbbsession['originServerUrl'] = $CFG->wwwroot;
     $bbbsession['originServerCommonName'] = '';
-    $bbbsession['originTag'] = 'moodle-mod_bigbluebuttonbn ('.get_config('mod_bigbluebuttonbn', 'version').')';
+    $bbbsession['originTag'] = 'moodle-mod_bigbluebuttonbn (' . get_config('mod_bigbluebuttonbn', 'version') . ')';
 }
 
 function bigbluebuttonbn_view_bbbsession_roles($context, $userid) {
@@ -248,10 +248,10 @@ 
     $bbbsession['group'] = groups_get_activity_group($bbbsession['cm'], true);
 
     // Assign group default values.
-    $bbbsession['meetingid'] = $bbbsession['bigbluebuttonbn']->meetingid.'-'.$bbbsession['course']->id.'-'.
-        $bbbsession['bigbluebuttonbn']->id.'['.$bbbsession['group'].']';
+    $bbbsession['meetingid'] = $bbbsession['bigbluebuttonbn']->meetingid . '-' . $bbbsession['course']->id . '-' .
+        $bbbsession['bigbluebuttonbn']->id . '[' . $bbbsession['group'] . ']';
     $groupname = get_string('allparticipants');
-    $bbbsession['meetingname'] = $bbbsession['bigbluebuttonbn']->name.' ('.$groupname.')';
+    $bbbsession['meetingname'] = $bbbsession['bigbluebuttonbn']->name . ' (' . $groupname . ')';
 
     if (count($groups) == 0) {
         // Only the All participants group exists.
@@ -262,10 +262,10 @@ 
         $bbbsession['group'] = array_values($groups)[0]->id;
     }
 
-    $bbbsession['meetingid'] = $bbbsession['bigbluebuttonbn']->meetingid.'-'.$bbbsession['course']->id.'-'.
-        $bbbsession['bigbluebuttonbn']->id.'['.$bbbsession['group'].']';
+    $bbbsession['meetingid'] = $bbbsession['bigbluebuttonbn']->meetingid . '-' . $bbbsession['course']->id . '-' .
+        $bbbsession['bigbluebuttonbn']->id . '[' . $bbbsession['group'] . ']';
     $groupname = groups_get_group_name($bbbsession['group']);
-    $bbbsession['meetingname'] = $bbbsession['bigbluebuttonbn']->name.' ('.$groupname.')';
+    $bbbsession['meetingname'] = $bbbsession['bigbluebuttonbn']->name . ' (' . $groupname . ')';
 
     if (count($groups) == 1) {
         // There only one group and the user has access to.
@@ -273,12 +273,12 @@ 
     }
 
     echo $OUTPUT->box_start('generalbox boxaligncenter');
-    echo '<br><div class="alert alert-warning">'.get_string('view_groups_selection_warning', 'bigbluebuttonbn').
+    echo '<br><div class="alert alert-warning">' . get_string('view_groups_selection_warning', 'bigbluebuttonbn') .
       '</div>';
     echo $OUTPUT->box_end();
 
     groups_print_activity_menu(
-      $bbbsession['cm'], $CFG->wwwroot.'/mod/bigbluebuttonbn/view.php?id='.$bbbsession['cm']->id, false, $onlyexistentgroups);
+      $bbbsession['cm'], $CFG->wwwroot . '/mod/bigbluebuttonbn/view.php?id=' . $bbbsession['cm']->id, false, $onlyexistentgroups);
     echo '<br><br>';
 }
 
@@ -320,7 +320,7 @@ 
             'M.mod_bigbluebuttonbn.recordings.init', array($jsvars));
     }
 
-    echo $output.html_writer::empty_tag('br').html_writer::empty_tag('br').html_writer::empty_tag('br');
+    echo $output . html_writer::empty_tag('br') . html_writer::empty_tag('br') . html_writer::empty_tag('br');
 
     $PAGE->requires->yui_module('moodle-mod_bigbluebuttonbn-broker', 'M.mod_bigbluebuttonbn.broker.init', array($jsvars));
 }
@@ -331,12 +331,12 @@ 
     // JavaScript variables for room.
     $openingtime = '';
     if ($bbbsession['openingtime']) {
-        $openingtime = get_string('mod_form_field_openingtime', 'bigbluebuttonbn').': '.
+        $openingtime = get_string('mod_form_field_openingtime', 'bigbluebuttonbn') . ': ' .
             userdate($bbbsession['openingtime']);
     }
     $closingtime = '';
     if ($bbbsession['closingtime']) {
-        $closingtime = get_string('mod_form_field_closingtime', 'bigbluebuttonbn').': '.
+        $closingtime = get_string('mod_form_field_closingtime', 'bigbluebuttonbn') . ': ' .
             userdate($bbbsession['closingtime']);
     }
     $jsvars += array(
@@ -353,7 +353,7 @@ 
     $output .= $OUTPUT->box_end();
 
     $output .= $OUTPUT->box_start('generalbox boxaligncenter', 'bigbluebuttonbn_view_action_button_box');
-    $output .= '<br><br><span id="join_button"></span>&nbsp;<span id="end_button"></span>'."\n";
+    $output .= '<br><br><span id="join_button"></span>&nbsp;<span id="end_button"></span>' . "\n";
     $output .= $OUTPUT->box_end();
 
     if ($activity == 'ended') {
@@ -389,7 +389,7 @@ 
     // If there are meetings with recordings load the data to the table.
     if ($bbbsession['bigbluebuttonbn']->recordings_html) {
         // Render a plain html table.
-        return bigbluebutton_output_recording_table($bbbsession, $recordings)."\n";
+        return bigbluebutton_output_recording_table($bbbsession, $recordings) . "\n";
     }
 
     // JavaScript variables for recordings with YUI.
@@ -409,8 +409,8 @@ 
         array('type' => 'button',
               'value' => get_string('view_recording_button_import', 'bigbluebuttonbn'),
               'class' => 'btn btn-secondary',
-              'onclick' => 'window.location=\''.$CFG->wwwroot.'/mod/bigbluebuttonbn/import_view.php?bn='.
-                  $bbbsession['bigbluebuttonbn']->id.'\''));
+              'onclick' => 'window.location=\'' . $CFG->wwwroot . '/mod/bigbluebuttonbn/import_view.php?bn=' .
+                  $bbbsession['bigbluebuttonbn']->id . '\''));
     $output = html_writer::start_tag('br');
     $output .= html_writer::tag('span', $button, array('id' => 'import_recording_links_button'));
     $output .= html_writer::tag('span', '', array('id' => 'import_recording_links_table'));
@@ -425,10 +425,10 @@ 
         $attributes = array('title' => $bbbsession['presentation']['name']);
         $icon = new pix_icon($bbbsession['presentation']['icon'], $bbbsession['presentation']['mimetype_description']);
 
-        return '<h4>'.get_string('view_section_title_presentation', 'bigbluebuttonbn').'</h4>'.
-                ''.$OUTPUT->action_icon($bbbsession['presentation']['url'], $icon, null, array(), false).''.
-                ''.$OUTPUT->action_link($bbbsession['presentation']['url'],
-                      $bbbsession['presentation']['name'], null, $attributes).'<br><br>';
+        return '<h4>' . get_string('view_section_title_presentation', 'bigbluebuttonbn') . '</h4>' .
+                '' . $OUTPUT->action_icon($bbbsession['presentation']['url'], $icon, null, array(), false) . '' .
+                '' . $OUTPUT->action_link($bbbsession['presentation']['url'],
+                      $bbbsession['presentation']['name'], null, $attributes) . '<br><br>';
     }
 
     return '';

lib.php

Doc Comments +14 added lines, -1 removed lines

@@ -227,7 +227,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;
@@ -317,6 +317,9 @@ 
     }
 }
 
+/**
+ * @param integer $now
+ */
 function bigbluebuttonbn_print_overview_element($bigbluebuttonbn, $now) {
     global $CFG;
 
@@ -542,6 +545,10 @@ 
     send_stored_file($file, 0, 0, $forcedownload, $options); // download MUST be forced - security!
 }
 
+/**
+ * @param stdClass $context
+ * @param string $filearea
+ */
 function bigbluebuttonbn_pluginfile_valid($context, $filearea) {
     if ($context->contextlevel != CONTEXT_MODULE) {
         return false;
@@ -558,6 +565,12 @@ 
     return true;
 }
 
+/**
+ * @param stdClass $course
+ * @param stdClass $cm
+ * @param stdClass $context
+ * @param string $filearea
+ */
 function bigbluebuttonbn_pluginfile_file($course, $cm, $context, $filearea, $args) {
     $filename = bigbluebuttonbn_pluginfile_filename($course, $cm, $context, $args);
     if (!$filename) {

Spacing +45 added lines, -45 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;
         }
@@ -83,19 +83,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];
     }
 
@@ -175,7 +175,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)) {
@@ -235,11 +235,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');
     }
 
@@ -258,7 +258,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;
 }
@@ -328,16 +328,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;
 }
@@ -425,7 +425,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.\mod_bigbluebuttonbn\locallib\config::get('shared_secret'));
+        $meetingid = sha1($CFG->wwwroot . $bigbluebuttonbn->id . \mod_bigbluebuttonbn\locallib\config::get('shared_secret'));
         $DB->set_field('bigbluebuttonbn', 'meetingid', $meetingid, array('id' => $bigbluebuttonbn->id));
 
         $action = get_string('mod_form_field_notification_msg_created', 'bigbluebuttonbn');
@@ -507,7 +507,7 @@ 
     if (count($files) == 1) {
         // Get the first (and only) file.
         $file = reset($files);
-        $filesrc = '/'.$file->get_filename();
+        $filesrc = '/' . $file->get_filename();
     }
     // Set the presentation column in the bigbluebuttonbn table.
     $DB->set_field('bigbluebuttonbn', 'presentation', $filesrc, array('id' => $bigbluebuttonbnid));
@@ -564,7 +564,7 @@ 
         return false;
     }
 
-    $fullpath = "/$context->id/mod_bigbluebuttonbn/$filearea/0/".$filename;
+    $fullpath = "/$context->id/mod_bigbluebuttonbn/$filearea/0/" . $filename;
     $fs = get_file_storage();
     $file = $fs->get_file_by_hash(sha1($fullpath));
     if (!$file || $file->is_directory()) {

locallib.php

Doc Comments +12 added lines

@@ -56,6 +56,9 @@ 
 const BIGBLUEBUTTON_EVENT_RECORDING_EDITED = 'recording_edited';
 const BIGBLUEBUTTON_EVENT_RECORDING_VIEWED = 'recording_viewed';
 
+/**
+ * @param string $meta
+ */
 function bigbluebuttonbn_logs(array $bbbsession, $event, array $overrides = [], $meta = null) {
     global $DB;
 
@@ -580,6 +583,12 @@ 
     }
 }
 
+/**
+ * @param string $url
+ * @param string $data
+ *
+ * @return string
+ */
 function bigbluebuttonbn_wrap_xml_load_file_curl_request($url, $method = BIGBLUEBUTTONBN_METHOD_GET,
     $data = null, $contenttype = 'text/xml') {
     $c = new curl();
@@ -1515,6 +1524,9 @@ 
     return html_writer::div(html_writer::table($table), '', array('id' => 'bigbluebuttonbn_html_table'));
 }
 
+/**
+ * @param integer $len
+ */
 function bigbluebuttonbn_html2text($html, $len) {
     $text = strip_tags($html);
     $text = str_replace(' ', ' ', $text);

Indentation +48 added lines, -48 removed lines

@@ -85,9 +85,9 @@ 
 // BigBlueButton API Calls.
 function bigbluebuttonbn_get_join_url($meetingid, $username, $pw, $logouturl, $configtoken = null, $userid = null) {
     $data = ['meetingID' => $meetingid,
-              'fullName' => $username,
-              'password' => $pw,
-              'logoutURL' => $logouturl,
+                'fullName' => $username,
+                'password' => $pw,
+                'logoutURL' => $logouturl,
             ];
 
     if (!is_null($configtoken)) {
@@ -188,24 +188,24 @@ 
 function bigbluebuttonbn_get_meeting_info_array($meetingid) {
     $xml = bigbluebuttonbn_wrap_xml_load_file(
         \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('getMeetingInfo', ['meetingID' => $meetingid])
-      );
+        );
 
     if ($xml && $xml->returncode == 'SUCCESS' && empty($xml->messageKey)) {
         // Meeting info was returned.
         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,
-                   );
+                        '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,
+                    );
     }
 
     if ($xml) {
@@ -269,7 +269,7 @@ 
         // Do getRecordings is executed using a method GET (supported by all versions of BBB).
         $xml = bigbluebuttonbn_wrap_xml_load_file(
             \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('getRecordings', ['meetingID' => implode(',', $mids)])
-          );
+            );
         if ($xml && $xml->returncode == 'SUCCESS' && isset($xml->recordings)) {
             // If there were meetings already created.
             foreach ($xml->recordings->recording as $recording) {
@@ -332,7 +332,7 @@ 
 function bigbluebuttonbn_get_default_config_xml() {
     $xml = bigbluebuttonbn_wrap_xml_load_file(
         \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('getDefaultConfigXML')
-      );
+        );
 
     return $xml;
 }
@@ -406,7 +406,7 @@ 
     foreach ($ids as $id) {
         $xml = bigbluebuttonbn_wrap_xml_load_file(
             \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('deleteRecordings', ['recordID' => $id])
-          );
+            );
         if ($xml && $xml->returncode != 'SUCCESS') {
             return false;
         }
@@ -424,7 +424,7 @@ 
     foreach ($ids as $id) {
         $xml = bigbluebuttonbn_wrap_xml_load_file(
             \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('publishRecordings', ['recordID' => $id, 'publish' => $publish])
-          );
+            );
         if ($xml && $xml->returncode != 'SUCCESS') {
             return false;
         }
@@ -442,7 +442,7 @@ 
     foreach ($ids as $id) {
         $xml = bigbluebuttonbn_wrap_xml_load_file(
             \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('updateRecordings', ['recordID' => $id] + (array) $params)
-          );
+            );
         if ($xml && $xml->returncode != 'SUCCESS') {
             return false;
         }
@@ -501,7 +501,7 @@ 
 function bigbluebuttonbn_end_meeting($meetingid, $modpw) {
     $xml = bigbluebuttonbn_wrap_xml_load_file(
         \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('end', ['meetingID' => $meetingid, 'password' => $modpw])
-      );
+        );
 
     if ($xml) {
         // If the xml packet returned failure it displays the message to the user.
@@ -518,7 +518,7 @@ 
 function bigbluebuttonbn_is_meeting_running($meetingid) {
     $xml = bigbluebuttonbn_wrap_xml_load_file(
         \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('isMeetingRunning', ['meetingID' => $meetingid])
-      );
+        );
 
     if ($xml && $xml->returncode == 'SUCCESS') {
         return ($xml->running == 'true');
@@ -530,7 +530,7 @@ 
 function bigbluebuttonbn_get_server_version() {
     $xml = bigbluebuttonbn_wrap_xml_load_file(
         \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url()
-      );
+        );
 
     if ($xml && $xml->returncode == 'SUCCESS') {
         return $xml->version;
@@ -591,10 +591,10 @@ 
 
         $options = array();
         $options['CURLOPT_HTTPHEADER'] = array(
-                 'Content-Type: '.$contenttype,
-                 'Content-Length: '.strlen($data),
-                 'Content-Language: en-US',
-               );
+                    'Content-Type: '.$contenttype,
+                    'Content-Length: '.strlen($data),
+                    'Content-Language: en-US',
+                );
 
         return $c->post($url, $data, $options);
     }
@@ -673,18 +673,18 @@ 
         'all' => array(
             'name' => get_string('mod_form_field_participant_list_type_all', 'bigbluebuttonbn'),
             'children' => []
-          )
-      );
+            )
+        );
 
     $data['role'] = array(
         'name' => get_string('mod_form_field_participant_list_type_role', 'bigbluebuttonbn'),
         'children' => bigbluebuttonbn_get_roles_select($context)
-      );
+        );
 
     $data['user'] = array(
         'name' => get_string('mod_form_field_participant_list_type_user', 'bigbluebuttonbn'),
         'children' => bigbluebuttonbn_get_users_select($context)
-      );
+        );
 
     return $data;
 }
@@ -731,9 +731,9 @@ 
         }
 
         $participantlistarray[] = array(
-              'selectiontype' => 'role',
-              'selectionid' => $moderatordefault,
-              'role' => BIGBLUEBUTTONBN_ROLE_MODERATOR);
+                'selectiontype' => 'role',
+                'selectionid' => $moderatordefault,
+                'role' => BIGBLUEBUTTONBN_ROLE_MODERATOR);
     }
 
     return $participantlistarray;
@@ -745,11 +745,11 @@ 
             'all' => get_string('mod_form_field_participant_list_type_all', 'bigbluebuttonbn'),
             'role' => get_string('mod_form_field_participant_list_type_role', 'bigbluebuttonbn'),
             'user' => get_string('mod_form_field_participant_list_type_user', 'bigbluebuttonbn'),
-          ],
+            ],
         'type_selected' => 'all',
         'options' => ['all' => '---------------'],
         'selected' => 'all',
-      ];
+        ];
 }
 
 function bigbluebuttonbn_is_moderator($context, $participants, $userid = null, $userroles = null) {
@@ -880,8 +880,8 @@ 
     }
 
     $parray = array('url' => $purl, 'name' => $pname,
-                               'icon' => $picon,
-                               'mimetype_description' => $pmimetypedescrip);
+                                'icon' => $picon,
+                                'mimetype_description' => $pmimetypedescrip);
 
     return $parray;
 }
@@ -961,7 +961,7 @@ 
     }
 
     $event = call_user_func_array('\mod_bigbluebuttonbn\event\bigbluebuttonbn_'.$eventtype.'::create',
-      array($eventproperties));
+        array($eventproperties));
     $event->trigger();
 }
 
@@ -1009,7 +1009,7 @@ 
     // Ping again and refresh the cache.
     $meetinginfo = (array) bigbluebuttonbn_wrap_xml_load_file(
         \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('getMeetingInfo', ['meetingID' => $meetingid])
-      );
+        );
     $cache->set($meetingid, array('creation_time' => time(), 'meeting_info' => json_encode($meetinginfo)));
 
     return $meetinginfo;
@@ -1150,7 +1150,7 @@ 
         $actionbar .= bigbluebuttonbn_actionbar_render_button(
             $recording,
             bigbluebuttonbn_get_recording_data_row_actionbar_payload($recording, $tool)
-          );
+            );
     }
     $head = html_writer::start_tag('div', array(
         'id' => 'recording-actionbar-' . $recording['recordID'],
@@ -1233,8 +1233,8 @@ 
 
     $id = 'playbacks-'.$recording['recordID'];
     $recordingtypes = html_writer::start_tag('div', array('id' => $id, 'data-imported' => $dataimported,
-          'data-recordingid' => $recording['recordID'], 'data-meetingid' => $recording['meetingID'],
-          'title' => $title, $visibility => $visibility));
+            'data-recordingid' => $recording['recordID'], 'data-meetingid' => $recording['meetingID'],
+            'title' => $title, $visibility => $visibility));
     foreach ($recording['playbacks'] as $playback) {
         $onclick = 'M.mod_bigbluebuttonbn.recordings.recording_play(this);';
         $href = $CFG->wwwroot.'/mod/bigbluebuttonbn/bbb_view.php?action=playback&bn='.$bigbluebuttonbnid.
@@ -1340,7 +1340,7 @@ 
             'onclick' => $onclick,
             'data-action' => $data['action'],
             'data-links' => bigbluebuttonbn_get_count_recording_imported_instances($recording['recordID'])
-          );
+            );
         return $OUTPUT->action_icon('#', $icon, null, $linkattributes, false);
     }
 
@@ -1679,8 +1679,8 @@ 
     $activitytime = '';
     if ($time) {
         $activitytime = calendar_day_representation($time).' '.
-          get_string('mod_form_field_notification_msg_at', 'bigbluebuttonbn').' '.
-          calendar_time_representation($time);
+            get_string('mod_form_field_notification_msg_at', 'bigbluebuttonbn').' '.
+            calendar_time_representation($time);
     }
 
     return $activitytime;

Spacing +98 added lines, -98 removed lines

@@ -27,7 +27,7 @@ 
 
 global $CFG;
 
-require_once(dirname(__FILE__).'/lib.php');
+require_once(dirname(__FILE__) . '/lib.php');
 
 const BIGBLUEBUTTONBN_FORCED = true;
 
@@ -76,7 +76,7 @@ 
     if (isset($meta)) {
         $log->meta = $meta;
     } else if ($event == BIGBLUEBUTTONBN_LOG_EVENT_CREATE) {
-        $log->meta = '{"record":'.($bbbsession['record'] ? 'true' : 'false').'}';
+        $log->meta = '{"record":' . ($bbbsession['record'] ? 'true' : 'false') . '}';
     }
 
     $DB->insert_record('bigbluebuttonbn_logs', $log);
@@ -121,8 +121,8 @@ 
 
     if (!is_null($pname) && !is_null($purl)) {
         $method = BIGBLUEBUTTONBN_METHOD_POST;
-        $data = "<?xml version='1.0' encoding='UTF-8'?><modules><module name='" . $pname . "'><document url='".
-            $purl."' /></module></modules>";
+        $data = "<?xml version='1.0' encoding='UTF-8'?><modules><module name='" . $pname . "'><document url='" .
+            $purl . "' /></module></modules>";
     }
 
     $xml = bigbluebuttonbn_wrap_xml_load_file($createmeetingurl, $method, $data);
@@ -167,7 +167,7 @@ 
                                 'moderatorPW' => $meeting->moderatorPW,
                                 'attendeePW' => $meeting->attendeePW,
                                 'hasBeenForciblyEnded' => $meeting->hasBeenForciblyEnded,
-                                'running' => $meeting->running, );
+                                'running' => $meeting->running,);
         }
 
         return $meetings;
@@ -308,12 +308,12 @@ 
 function bigbluebuttonbn_get_recordings_imported_array($courseid, $bigbluebuttonbnid = null, $subset = true) {
     global $DB;
 
-    $select = "courseid = '{$courseid}' AND bigbluebuttonbnid <> '{$bigbluebuttonbnid}' AND log = '".
-        BIGBLUEBUTTONBN_LOG_EVENT_IMPORT."'";
+    $select = "courseid = '{$courseid}' AND bigbluebuttonbnid <> '{$bigbluebuttonbnid}' AND log = '" .
+        BIGBLUEBUTTONBN_LOG_EVENT_IMPORT . "'";
     if ($bigbluebuttonbnid === null) {
-        $select = "courseid = '{$courseid}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_IMPORT."'";
+        $select = "courseid = '{$courseid}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_IMPORT . "'";
     } else if ($subset) {
-        $select = "bigbluebuttonbnid = '{$bigbluebuttonbnid}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_IMPORT."'";
+        $select = "bigbluebuttonbnid = '{$bigbluebuttonbnid}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_IMPORT . "'";
     }
     $recordsimported = $DB->get_records_select('bigbluebuttonbn_logs', $select);
 
@@ -340,37 +340,37 @@ 
 function bigbluebuttonbn_get_default_config_xml_array() {
     $defaultconfigxml = bigbluebuttonbn_getDefaultConfigXML();
 
-    return (array) $defaultconfigxml;
+    return (array)$defaultconfigxml;
 }
 
 function bigbluebuttonbn_get_recording_array_value($recording) {
     // Add formats.
     $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);
+        $playbackarray[(string)$format->type] = array('type' => (string)$format->type,
+            'url' => (string)$format->url, 'length' => (string)$format->length);
         // Add preview per format when existing.
         if ($format->preview) {
             $imagesarray = array();
             foreach ($format->preview->images->image as $image) {
-                $imagearray = array('url' => (string) $image);
+                $imagearray = array('url' => (string)$image);
                 foreach ($image->attributes() as $attkey => $attvalue) {
-                    $imagearray[$attkey] = (string) $attvalue;
+                    $imagearray[$attkey] = (string)$attvalue;
                 }
                 array_push($imagesarray, $imagearray);
             }
-            $playbackarray[(string) $format->type]['preview'] = $imagesarray;
+            $playbackarray[(string)$format->type]['preview'] = $imagesarray;
         }
     }
 
     // Add the metadata to the recordings array.
     $metadataarray = bigbluebuttonbn_get_recording_array_meta(get_object_vars($recording->metadata));
-    $recordingarray = 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);
+    $recordingarray = 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);
     if (isset($recording->protected)) {
-        $recordingarray['protected'] = (string) $recording->protected;
+        $recordingarray['protected'] = (string)$recording->protected;
     }
     // Force protected.
     $recordingarray['protected'] = 'false';
@@ -383,7 +383,7 @@ 
         if (is_object($value)) {
             $value = '';
         }
-        $metadataarray['meta_'.$key] = $value;
+        $metadataarray['meta_' . $key] = $value;
     }
     return $metadataarray;
 }
@@ -441,7 +441,7 @@ 
     $ids = explode(',', $recordids);
     foreach ($ids as $id) {
         $xml = bigbluebuttonbn_wrap_xml_load_file(
-            \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('updateRecordings', ['recordID' => $id] + (array) $params)
+            \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('updateRecordings', ['recordID' => $id] + (array)$params)
           );
         if ($xml && $xml->returncode != 'SUCCESS') {
             return false;
@@ -561,7 +561,7 @@ 
             return $xml;
         } catch (Exception $e) {
             libxml_use_internal_errors($previous);
-            $error = 'Caught exception: '.$e->getMessage();
+            $error = 'Caught exception: ' . $e->getMessage();
             debugging($error, DEBUG_DEVELOPER);
             return null;
         }
@@ -573,7 +573,7 @@ 
         $response = simplexml_load_file($url, 'SimpleXMLElement', LIBXML_NOCDATA | LIBXML_NOBLANKS);
         return $response;
     } catch (Exception $e) {
-        $error = 'Caught exception: '.$e->getMessage();
+        $error = 'Caught exception: ' . $e->getMessage();
         debugging($error, DEBUG_DEVELOPER);
         libxml_use_internal_errors($previous);
         return null;
@@ -591,8 +591,8 @@ 
 
         $options = array();
         $options['CURLOPT_HTTPHEADER'] = array(
-                 'Content-Type: '.$contenttype,
-                 'Content-Length: '.strlen($data),
+                 'Content-Type: ' . $contenttype,
+                 'Content-Length: ' . strlen($data),
                  'Content-Language: en-US',
                );
 
@@ -609,9 +609,9 @@ 
     if ($userroles) {
         $where = '';
         foreach ($userroles as $value) {
-            $where .= (empty($where) ? ' WHERE' : ' AND').' id='.$value->roleid;
+            $where .= (empty($where) ? ' WHERE' : ' AND') . ' id=' . $value->roleid;
         }
-        $userroles = $DB->get_records_sql('SELECT * FROM {role}'.$where);
+        $userroles = $DB->get_records_sql('SELECT * FROM {role}' . $where);
     }
 
     return $userroles;
@@ -624,7 +624,7 @@ 
 }
 
 function bigbluebuttonbn_get_users(context $context = null) {
-    $users = (array) get_enrolled_users($context,'',0,'u.*',null,0,0,true);
+    $users = (array)get_enrolled_users($context, '', 0, 'u.*', null, 0, 0, true);
     foreach ($users as $key => $value) {
         $users[$key] = fullname($value);
     }
@@ -632,7 +632,7 @@ 
 }
 
 function bigbluebuttonbn_get_users_select(context $context = null) {
-    $users = (array) get_enrolled_users($context,'',0,'u.*',null,0,0,true);
+    $users = (array)get_enrolled_users($context, '', 0, 'u.*', null, 0, 0, true);
     foreach ($users as $key => $value) {
         $users[$key] = array('id' => $value->id, 'name' => fullname($value));
     }
@@ -640,7 +640,7 @@ 
 }
 
 function bigbluebuttonbn_get_roles(context $context = null) {
-    $roles = (array) role_get_names($context);
+    $roles = (array)role_get_names($context);
     foreach ($roles as $key => $value) {
         $roles[$key] = $value->localname;
     }
@@ -648,7 +648,7 @@ 
 }
 
 function bigbluebuttonbn_get_roles_select(context $context = null) {
-    $roles = (array) role_get_names($context);
+    $roles = (array)role_get_names($context);
     foreach ($roles as $key => $value) {
         $roles[$key] = array('id' => $value->id, 'name' => $value->localname);
     }
@@ -656,7 +656,7 @@ 
 }
 
 function bigbluebuttonbn_get_role($id) {
-    $roles = (array) role_get_names();
+    $roles = (array)role_get_names();
     if (is_numeric($id)) {
         return $roles[$id];
     }
@@ -820,9 +820,9 @@ 
     $isunique = true;
     if ($voicebridge != 0) {
         $table = 'bigbluebuttonbn';
-        $select = 'voicebridge = '.$voicebridge;
+        $select = 'voicebridge = ' . $voicebridge;
         if ($id) {
-            $select .= ' AND id <> '.$id;
+            $select .= ' AND id <> ' . $id;
         }
         if ($DB->get_records_select($table, $select)) {
             $isunique = false;
@@ -890,7 +890,7 @@ 
     $mt = microtime();
     $rand = mt_rand();
 
-    return md5($mt.$rand);
+    return md5($mt . $rand);
 }
 
 function bigbluebuttonbn_random_password($length = 8) {
@@ -902,41 +902,41 @@ 
 
 function bigbluebuttonbn_events() {
     return array(
-        (string) BIGBLUEBUTTON_EVENT_ACTIVITY_VIEWED,
-        (string) BIGBLUEBUTTON_EVENT_ACTIVITY_MANAGEMENT_VIEWED,
-        (string) BIGBLUEBUTTON_EVENT_LIVE_SESSION,
-        (string) BIGBLUEBUTTON_EVENT_MEETING_CREATED,
-        (string) BIGBLUEBUTTON_EVENT_MEETING_ENDED,
-        (string) BIGBLUEBUTTON_EVENT_MEETING_JOINED,
-        (string) BIGBLUEBUTTON_EVENT_MEETING_LEFT,
-        (string) BIGBLUEBUTTON_EVENT_RECORDING_DELETED,
-        (string) BIGBLUEBUTTON_EVENT_RECORDING_IMPORTED,
-        (string) BIGBLUEBUTTON_EVENT_RECORDING_PROTECTED,
-        (string) BIGBLUEBUTTON_EVENT_RECORDING_PUBLISHED,
-        (string) BIGBLUEBUTTON_EVENT_RECORDING_UNPROTECTED,
-        (string) BIGBLUEBUTTON_EVENT_RECORDING_UNPUBLISHED,
-        (string) BIGBLUEBUTTON_EVENT_RECORDING_EDITED,
-        (string) BIGBLUEBUTTON_EVENT_RECORDING_VIEWED
+        (string)BIGBLUEBUTTON_EVENT_ACTIVITY_VIEWED,
+        (string)BIGBLUEBUTTON_EVENT_ACTIVITY_MANAGEMENT_VIEWED,
+        (string)BIGBLUEBUTTON_EVENT_LIVE_SESSION,
+        (string)BIGBLUEBUTTON_EVENT_MEETING_CREATED,
+        (string)BIGBLUEBUTTON_EVENT_MEETING_ENDED,
+        (string)BIGBLUEBUTTON_EVENT_MEETING_JOINED,
+        (string)BIGBLUEBUTTON_EVENT_MEETING_LEFT,
+        (string)BIGBLUEBUTTON_EVENT_RECORDING_DELETED,
+        (string)BIGBLUEBUTTON_EVENT_RECORDING_IMPORTED,
+        (string)BIGBLUEBUTTON_EVENT_RECORDING_PROTECTED,
+        (string)BIGBLUEBUTTON_EVENT_RECORDING_PUBLISHED,
+        (string)BIGBLUEBUTTON_EVENT_RECORDING_UNPROTECTED,
+        (string)BIGBLUEBUTTON_EVENT_RECORDING_UNPUBLISHED,
+        (string)BIGBLUEBUTTON_EVENT_RECORDING_EDITED,
+        (string)BIGBLUEBUTTON_EVENT_RECORDING_VIEWED
     );
 }
 
 function bigbluebuttonbn_events_action() {
     return array(
-        'view' => (string) BIGBLUEBUTTON_EVENT_ACTIVITY_VIEWED,
-        'view_management' => (string) BIGBLUEBUTTON_EVENT_ACTIVITY_MANAGEMENT_VIEWED,
-        'live_action' => (string) BIGBLUEBUTTON_EVENT_LIVE_SESSION,
-        'meeting_create' => (string) BIGBLUEBUTTON_EVENT_MEETING_CREATED,
-        'meeting_end' => (string) BIGBLUEBUTTON_EVENT_MEETING_ENDED,
-        'meeting_join' => (string) BIGBLUEBUTTON_EVENT_MEETING_JOINED,
-        'meeting_left' => (string) BIGBLUEBUTTON_EVENT_MEETING_LEFT,
-        'recording_delete' => (string) BIGBLUEBUTTON_EVENT_RECORDING_DELETED,
-        'recording_import' => (string) BIGBLUEBUTTON_EVENT_RECORDING_IMPORTED,
-        'recording_protect' => (string) BIGBLUEBUTTON_EVENT_RECORDING_PROTECTED,
-        'recording_publish' => (string) BIGBLUEBUTTON_EVENT_RECORDING_PUBLISHED,
-        'recording_unprotect' => (string) BIGBLUEBUTTON_EVENT_RECORDING_UNPROTECTED,
-        'recording_unpublish' => (string) BIGBLUEBUTTON_EVENT_RECORDING_UNPUBLISHED,
-        'recording_edit' => (string) BIGBLUEBUTTON_EVENT_RECORDING_EDITED,
-        'recording_play' => (string) BIGBLUEBUTTON_EVENT_RECORDING_VIEWED
+        'view' => (string)BIGBLUEBUTTON_EVENT_ACTIVITY_VIEWED,
+        'view_management' => (string)BIGBLUEBUTTON_EVENT_ACTIVITY_MANAGEMENT_VIEWED,
+        'live_action' => (string)BIGBLUEBUTTON_EVENT_LIVE_SESSION,
+        'meeting_create' => (string)BIGBLUEBUTTON_EVENT_MEETING_CREATED,
+        'meeting_end' => (string)BIGBLUEBUTTON_EVENT_MEETING_ENDED,
+        'meeting_join' => (string)BIGBLUEBUTTON_EVENT_MEETING_JOINED,
+        'meeting_left' => (string)BIGBLUEBUTTON_EVENT_MEETING_LEFT,
+        'recording_delete' => (string)BIGBLUEBUTTON_EVENT_RECORDING_DELETED,
+        'recording_import' => (string)BIGBLUEBUTTON_EVENT_RECORDING_IMPORTED,
+        'recording_protect' => (string)BIGBLUEBUTTON_EVENT_RECORDING_PROTECTED,
+        'recording_publish' => (string)BIGBLUEBUTTON_EVENT_RECORDING_PUBLISHED,
+        'recording_unprotect' => (string)BIGBLUEBUTTON_EVENT_RECORDING_UNPROTECTED,
+        'recording_unpublish' => (string)BIGBLUEBUTTON_EVENT_RECORDING_UNPUBLISHED,
+        'recording_edit' => (string)BIGBLUEBUTTON_EVENT_RECORDING_EDITED,
+        'recording_play' => (string)BIGBLUEBUTTON_EVENT_RECORDING_VIEWED
     );
 }
 
@@ -960,7 +960,7 @@ 
         $eventproperties['other'] = $options['other'];
     }
 
-    $event = call_user_func_array('\mod_bigbluebuttonbn\event\bigbluebuttonbn_'.$eventtype.'::create',
+    $event = call_user_func_array('\mod_bigbluebuttonbn\event\bigbluebuttonbn_' . $eventtype . '::create',
       array($eventproperties));
     $event->trigger();
 }
@@ -1003,11 +1003,11 @@ 
     $now = time();
     if (!$forced && isset($result) && $now < ($result['creation_time'] + $cachettl)) {
         // Use the value in the cache.
-        return (array) json_decode($result['meeting_info']);
+        return (array)json_decode($result['meeting_info']);
     }
 
     // Ping again and refresh the cache.
-    $meetinginfo = (array) bigbluebuttonbn_wrap_xml_load_file(
+    $meetinginfo = (array)bigbluebuttonbn_wrap_xml_load_file(
         \mod_bigbluebuttonbn\locallib\bigbluebutton::action_url('getMeetingInfo', ['meetingID' => $meetingid])
       );
     $cache->set($meetingid, array('creation_time' => time(), 'meeting_info' => json_encode($meetinginfo)));
@@ -1065,8 +1065,8 @@ 
  * @param string $configxml
  */
 function bigbluebuttonbn_set_config_xml_params($meetingid, $configxml) {
-    $params = 'configXML='.urlencode($configxml).'&meetingID='.urlencode($meetingid);
-    $configxmlparams = $params.'&checksum='.sha1('setConfigXML'.$params.\mod_bigbluebuttonbn\locallib\config::get('shared_secret'));
+    $params = 'configXML=' . urlencode($configxml) . '&meetingID=' . urlencode($meetingid);
+    $configxmlparams = $params . '&checksum=' . sha1('setConfigXML' . $params . \mod_bigbluebuttonbn\locallib\config::get('shared_secret'));
 
     return $configxmlparams;
 }
@@ -1076,7 +1076,7 @@ 
  * @param string $configxml
  */
 function bigbluebuttonbn_set_config_xml($meetingid, $configxml) {
-    $urldefaultconfig = \mod_bigbluebuttonbn\locallib\config::get('server_url').'api/setConfigXML?';
+    $urldefaultconfig = \mod_bigbluebuttonbn\locallib\config::get('server_url') . 'api/setConfigXML?';
     $configxmlparams = bigbluebuttonbn_set_config_xml_params($meetingid, $configxml);
     $xml = bigbluebuttonbn_wrap_xml_load_file($urldefaultconfig, BIGBLUEBUTTONBN_METHOD_POST,
         $configxmlparams, 'application/x-www-form-urlencoded');
@@ -1090,7 +1090,7 @@ 
  */
 function bigbluebuttonbn_set_config_xml_array($meetingid, $configxml) {
     $configxml = bigbluebuttonbn_setConfigXML($meetingid, $configxml);
-    $configxmlarray = (array) $configxml;
+    $configxmlarray = (array)$configxml;
     if ($configxmlarray['returncode'] != 'SUCCESS') {
         debugging('BigBlueButton was not able to set the custom config.xml file', DEBUG_DEVELOPER);
         return '';
@@ -1127,7 +1127,7 @@ 
         $starttime = $starttime - ($starttime % 1000);
 
         // Set formatted date.
-        $dateformat = get_string('strftimerecentfull', 'langconfig').' %Z';
+        $dateformat = get_string('strftimerecentfull', 'langconfig') . ' %Z';
         $row->date_formatted = userdate($starttime / 1000, $dateformat, usertimezone($USER->timezone));
 
         // Set formatted duration.
@@ -1198,7 +1198,7 @@ 
     }
 
     $recordingpreview = html_writer::start_tag('div',
-        array('id' => 'preview-'.$recording['recordID'], $visibility => $visibility));
+        array('id' => 'preview-' . $recording['recordID'], $visibility => $visibility));
     foreach ($recording['playbacks'] as $playback) {
         if (isset($playback['preview'])) {
             foreach ($playback['preview'] as $image) {
@@ -1231,19 +1231,19 @@ 
         $visibility = 'hidden ';
     }
 
-    $id = 'playbacks-'.$recording['recordID'];
+    $id = 'playbacks-' . $recording['recordID'];
     $recordingtypes = html_writer::start_tag('div', array('id' => $id, 'data-imported' => $dataimported,
           'data-recordingid' => $recording['recordID'], 'data-meetingid' => $recording['meetingID'],
           'title' => $title, $visibility => $visibility));
     foreach ($recording['playbacks'] as $playback) {
         $onclick = 'M.mod_bigbluebuttonbn.recordings.recording_play(this);';
-        $href = $CFG->wwwroot.'/mod/bigbluebuttonbn/bbb_view.php?action=playback&bn='.$bigbluebuttonbnid.
-            '&href='.urlencode($playback['url']).'&rid='.$recording['recordID'];
-        $linkattributes = array('title' => get_string('view_recording_format_'.$playback['type'], 'bigbluebuttonbn'),
+        $href = $CFG->wwwroot . '/mod/bigbluebuttonbn/bbb_view.php?action=playback&bn=' . $bigbluebuttonbnid .
+            '&href=' . urlencode($playback['url']) . '&rid=' . $recording['recordID'];
+        $linkattributes = array('title' => get_string('view_recording_format_' . $playback['type'], 'bigbluebuttonbn'),
             'class' => 'btn btn-sm btn-default', 'onclick' => $onclick,
             'data-action' => 'play', 'data-href' => $href);
-        $recordingtypes .= $OUTPUT->action_link('#', get_string('view_recording_format_'.$playback['type'],
-            'bigbluebuttonbn'), null, $linkattributes).'&#32;';
+        $recordingtypes .= $OUTPUT->action_link('#', get_string('view_recording_format_' . $playback['type'],
+            'bigbluebuttonbn'), null, $linkattributes) . '&#32;';
     }
     $recordingtypes .= html_writer::end_tag('div');
 
@@ -1328,11 +1328,11 @@ 
         $target .= '-' . $data['target'];
     }
     $id = 'recording-' . $target . '-' . $recording['recordID'];
-    $onclick = 'M.mod_bigbluebuttonbn.recordings.recording_'.$data['action'].'(this);';
+    $onclick = 'M.mod_bigbluebuttonbn.recordings.recording_' . $data['action'] . '(this);';
     if ((boolean)\mod_bigbluebuttonbn\locallib\config::get('recording_icons_enabled')) {
         // With icon for $manageaction.
         $iconattributes = array('id' => $id, 'class' => 'iconsmall');
-        $icon = new pix_icon('i/'.$data['tag'],
+        $icon = new pix_icon('i/' . $data['tag'],
             get_string('view_recording_list_actionbar_' . $data['action'], 'bigbluebuttonbn'),
             'moodle', $iconattributes);
         $linkattributes = array(
@@ -1423,7 +1423,7 @@ 
         // There are recordings for this meeting.
         foreach ($recordings as $recording) {
             $row = new html_table_row();
-            $row->id = 'recording-td-'.$recording['recordID'];
+            $row->id = 'recording-td-' . $recording['recordID'];
             $row->attributes['data-imported'] = 'false';
             $texthead = '';
             $texttail = '';
@@ -1437,9 +1437,9 @@ 
             $rowdata = bigbluebuttonbn_get_recording_data_row($bbbsession, $recording, $tools);
             if ($rowdata != null) {
                 $rowdata->date_formatted = str_replace(' ', '&nbsp;', $rowdata->date_formatted);
-                $row->cells = array($texthead.$rowdata->recording.$texttail,
-                    $texthead.$rowdata->activity.$texttail, $texthead.$rowdata->description.$texttail,
-                    $rowdata->preview, $texthead.$rowdata->date_formatted.$texttail,
+                $row->cells = array($texthead . $rowdata->recording . $texttail,
+                    $texthead . $rowdata->activity . $texttail, $texthead . $rowdata->description . $texttail,
+                    $rowdata->preview, $texthead . $rowdata->date_formatted . $texttail,
                     $rowdata->duration_formatted);
                 if ($bbbsession['managerecordings']) {
                     $row->cells[] = $rowdata->actionbar;
@@ -1461,9 +1461,9 @@ 
     // Build the message_body.
     $msg->activity_type = '';
     $msg->activity_title = $bigbluebuttonbn->name;
-    $messagetext = '<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>';
+    $messagetext = '<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, $messagetext);
 }
@@ -1556,15 +1556,15 @@ 
 
     // Gather the bigbluebuttonbnids whose meetingids should be included in the getRecordings request'.
     $select = "id <> '{$bigbluebuttonbnid}' AND course = '{$courseid}'";
-    $selectdeleted = "courseid = '{$courseid}' AND bigbluebuttonbnid <> '{$bigbluebuttonbnid}' AND log = '".
-        BIGBLUEBUTTONBN_LOG_EVENT_DELETE."' AND meta like '%has_recordings%' AND meta like '%true%'";
+    $selectdeleted = "courseid = '{$courseid}' AND bigbluebuttonbnid <> '{$bigbluebuttonbnid}' AND log = '" .
+        BIGBLUEBUTTONBN_LOG_EVENT_DELETE . "' AND meta like '%has_recordings%' AND meta like '%true%'";
     if ($bigbluebuttonbnid === null) {
         $select = "course = '{$courseid}'";
-        $selectdeleted = "courseid = '{$courseid}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_DELETE.
+        $selectdeleted = "courseid = '{$courseid}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_DELETE .
             "' AND meta like '%has_recordings%' AND meta like '%true%'";
     } else if ($subset) {
         $select = "id = '{$bigbluebuttonbnid}'";
-        $selectdeleted = "bigbluebuttonbnid = '{$bigbluebuttonbnid}' AND log = '".BIGBLUEBUTTONBN_LOG_EVENT_DELETE.
+        $selectdeleted = "bigbluebuttonbnid = '{$bigbluebuttonbnid}' AND log = '" . BIGBLUEBUTTONBN_LOG_EVENT_DELETE .
             "' AND meta like '%has_recordings%' AND meta like '%true%'";
     }
     $bigbluebuttonbns = $DB->get_records_select_menu('bigbluebuttonbn', $select, null, 'id', 'id, meetingid');
@@ -1586,7 +1586,7 @@ 
     if (!empty($bigbluebuttonbns)) {
         // Prepare select for loading records based on existent bigbluebuttonbns.
         $sql = 'SELECT DISTINCT meetingid, bigbluebuttonbnid FROM {bigbluebuttonbn_logs} WHERE ';
-        $sql .= '(bigbluebuttonbnid='.implode(' OR bigbluebuttonbnid=', array_keys($bigbluebuttonbns)).')';
+        $sql .= '(bigbluebuttonbnid=' . implode(' OR bigbluebuttonbnid=', array_keys($bigbluebuttonbns)) . ')';
         // Include only Create events and exclude those with record not true.
         $sql .= ' AND log = ? AND meta LIKE ? AND meta LIKE ?';
         // Execute select for loading records based on existent bigbluebuttonbns.
@@ -1678,8 +1678,8 @@ 
 function bigbluebuttonbn_format_activity_time($time) {
     $activitytime = '';
     if ($time) {
-        $activitytime = calendar_day_representation($time).' '.
-          get_string('mod_form_field_notification_msg_at', 'bigbluebuttonbn').' '.
+        $activitytime = calendar_day_representation($time) . ' ' .
+          get_string('mod_form_field_notification_msg_at', 'bigbluebuttonbn') . ' ' .
           calendar_time_representation($time);
     }
 
@@ -1705,7 +1705,7 @@ 
 
 function bigbluebuttonbn_get_moderator_email($context) {
     $moderatoremails = array();
-    $users = (array) get_enrolled_users($context,'',0,'u.*',null,0,0,true);
+    $users = (array)get_enrolled_users($context, '', 0, 'u.*', null, 0, 0, true);
     $counter = 0;
     foreach ($users as $key => $user) {
         if ($counter == 5) {

classes/locallib/notifier.php

Spacing +26 added lines, -26 removed lines

@@ -34,7 +34,7 @@ 
         global $USER;
 
         // Prepare message.
-        $msg = (object) array();
+        $msg = (object)array();
         // Build the message_body.
         $msg->action = $action;
         $msg->activity_type = '';
@@ -54,26 +54,26 @@ 
     }
 
     public static function 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";
         return $messagetext;
     }
 
@@ -83,15 +83,15 @@ 
         $course = $DB->get_record('course', array('id' => $bigbluebuttonbn->course), '*', MUST_EXIST);
 
         // Complete message.
-        $msg = (object) array();
+        $msg = (object)array();
         $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 = (array) get_enrolled_users($context,'',0,'u.*',null,0,0,true);
+        $users = (array)get_enrolled_users($context, '', 0, 'u.*', null, 0, 0, true);
         foreach ($users as $user) {
             if ($user->id != $sender->id) {
                 message_post_message($sender, $user, $message, FORMAT_HTML);

classes/locallib/bigbluebutton.php

Spacing +4 added lines, -4 removed lines

@@ -37,17 +37,17 @@
      * @return string
      */
     public static function action_url($action = '', $data = array(), $metadata = array()) {
-        $baseurl = \mod_bigbluebuttonbn\locallib\config::get('server_url').'api/'.$action.'?';
+        $baseurl = \mod_bigbluebuttonbn\locallib\config::get('server_url') . 'api/' . $action . '?';
         $params = '';
 
         foreach ($data as $key => $value) {
-            $params .= '&'.$key.'='.urlencode($value);
+            $params .= '&' . $key . '=' . urlencode($value);
         }
 
         foreach ($metadata as $key => $value) {
-            $params .= '&'.'meta_'.$key.'='.urlencode($value);
+            $params .= '&' . 'meta_' . $key . '=' . urlencode($value);
         }
 
-        return $baseurl.$params.'&checksum='.sha1($action.$params.\mod_bigbluebuttonbn\locallib\config::get('shared_secret'));
+        return $baseurl . $params . '&checksum=' . sha1($action . $params . \mod_bigbluebuttonbn\locallib\config::get('shared_secret'));
     }
 }

classes/locallib/config.php

Indentation +21 added lines, -21 removed lines

@@ -118,28 +118,28 @@
                  isset($CFG->bigbluebuttonbn['recording_editable']));
     }
 
-     /**
-      * @return array
-      */
+        /**
+         * @return array
+         */
     public static function get_options() {
         return [
-               'version_major' => self::get_moodle_version_major(),
-               'voicebridge_editable' => self::get('voicebridge_editable'),
-               'recording_default' => self::get('recording_default'),
-               'recording_editable' => self::get('recording_editable'),
-               'waitformoderator_default' => self::get('waitformoderator_default'),
-               'waitformoderator_editable' => self::get('waitformoderator_editable'),
-               'userlimit_default' => self::get('userlimit_default'),
-               'userlimit_editable' => self::get('userlimit_editable'),
-               'preuploadpresentation_enabled' => self::get('preuploadpresentation_enabled'),
-               'sendnotifications_enabled' => self::get('sendnotifications_enabled'),
-               'recordings_html_default' => self::get('recordings_html_default'),
-               'recordings_html_editable' => self::get('recordings_html_editable'),
-               'recordings_deleted_activities_default' => self::get('recordings_deleted_activities_default'),
-               'recordings_deleted_activities_editable' => self::get('recordings_deleted_activities_editable'),
-               'recording_icons_enabled' => self::get('recording_icons_enabled'),
-               'instance_type_enabled' => self::recordings_enabled(),
-               'instance_type_default' => BIGBLUEBUTTONBN_TYPE_ALL,
-          ];
+                'version_major' => self::get_moodle_version_major(),
+                'voicebridge_editable' => self::get('voicebridge_editable'),
+                'recording_default' => self::get('recording_default'),
+                'recording_editable' => self::get('recording_editable'),
+                'waitformoderator_default' => self::get('waitformoderator_default'),
+                'waitformoderator_editable' => self::get('waitformoderator_editable'),
+                'userlimit_default' => self::get('userlimit_default'),
+                'userlimit_editable' => self::get('userlimit_editable'),
+                'preuploadpresentation_enabled' => self::get('preuploadpresentation_enabled'),
+                'sendnotifications_enabled' => self::get('sendnotifications_enabled'),
+                'recordings_html_default' => self::get('recordings_html_default'),
+                'recordings_html_editable' => self::get('recordings_html_editable'),
+                'recordings_deleted_activities_default' => self::get('recordings_deleted_activities_default'),
+                'recordings_deleted_activities_editable' => self::get('recordings_deleted_activities_editable'),
+                'recording_icons_enabled' => self::get('recording_icons_enabled'),
+                'instance_type_enabled' => self::recordings_enabled(),
+                'instance_type_default' => BIGBLUEBUTTONBN_TYPE_ALL,
+            ];
     }
 }

Spacing +5 added lines, -5 removed lines

@@ -47,8 +47,8 @@ 
      */
     public static function defaultvalues() {
         return array(
-            'server_url' => (string) BIGBLUEBUTTONBN_DEFAULT_SERVER_URL,
-            'server_url' => (string) BIGBLUEBUTTONBN_DEFAULT_SHARED_SECRET,
+            'server_url' => (string)BIGBLUEBUTTONBN_DEFAULT_SERVER_URL,
+            'server_url' => (string)BIGBLUEBUTTONBN_DEFAULT_SHARED_SECRET,
             'importrecordings_enabled' => 'false',
             'voicebridge_editable' => 'false',
             'recording_default' => 'true',
@@ -98,11 +98,11 @@ 
         global $CFG;
 
         if (isset($CFG->bigbluebuttonbn[$setting])) {
-            return (string) $CFG->bigbluebuttonbn[$setting];
+            return (string)$CFG->bigbluebuttonbn[$setting];
         }
 
-        if (isset($CFG->{'bigbluebuttonbn_'.$setting})) {
-            return (string)$CFG->{'bigbluebuttonbn_'.$setting};
+        if (isset($CFG->{'bigbluebuttonbn_' . $setting})) {
+            return (string)$CFG->{'bigbluebuttonbn_' . $setting};
         }
 
         return  self::defaultvalue($setting);

classes/event/bigbluebuttonbn_live_session_event.php

Spacing +2 added lines, -2 removed lines

@@ -34,8 +34,8 @@
      */
     protected function init() {
         parent::init();
-        $this->description = "The user with id '$this->userid' triggered action $this->other in a ".
-        "bigbluebutton meeting for the bigbluebuttonbn activity with id ".
+        $this->description = "The user with id '$this->userid' triggered action $this->other in a " .
+        "bigbluebutton meeting for the bigbluebuttonbn activity with id " .
         "'$this->objectid' for the course id '$this->contextinstanceid'.";
     }
 

classes/event/bigbluebuttonbn_recording_unprotected.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
      */
     protected function init() {
         parent::init();
-        $this->description = "The user with id '$this->userid' has unprotected a recording with id ".
+        $this->description = "The user with id '$this->userid' has unprotected a recording with id " .
             "'$this->other' in the course id '$this->contextinstanceid'.";
     }
 

classes/event/bigbluebuttonbn_meeting_created.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
      */
     protected function init() {
         parent::init();
-        $this->description = "The user with id '$this->userid' created a bigbluebutton meeting for".
+        $this->description = "The user with id '$this->userid' created a bigbluebutton meeting for" .
             "the bigbluebuttonbn activity with id '$this->objectid' for the course id '$this->contextinstanceid'.";
     }