Fixed array syntax issues.

Author: Juan-Pablo-Castillo-Velasquez

backup/moodle2/backup_hsuforum_activity_task.class.php

@@ -70,7 +70,7 @@ public function get_comment_file_annotation_info() {
     static public function encode_content_links($content) {
         global $CFG;
 
-        $base = preg_quote($CFG->wwwroot,"/");
+        $base = preg_quote($CFG->wwwroot,"/",);
 
         // Link to the list of forums
         $search="/(".$base."\/mod\/hsuforum\/index.php\?id\=)([0-9]+)/";

backup/moodle2/restore_hsuforum_stepslib.php

@@ -275,7 +275,7 @@ protected function after_restore() {
             $sd->reveal = 0;
             $sdid = hsuforum_add_discussion($sd, null, null, $this->task->get_userid());
             // Mark the post as mailed
-            $DB->set_field ('hsuforum_posts','mailed', '1', array('discussion' => $sdid));
+            $DB->set_field ('hsuforum_posts','mailed', '1', array('discussion' => $sdid,));
             // Copy all the files from mod_foum/intro to mod_hsuforum/post
             $fs = get_file_storage();
             $files = $fs->get_area_files($this->task->get_contextid(), 'mod_hsuforum', 'intro');

classes/controller/edit_controller.php

@@ -119,7 +119,7 @@ public function reply_action() {
                 'message'       => $message,
                 'messageformat' => $messageformat,
                 'reveal'        => $reveal,
-                'mailnow'       => $mailnow
+                'mailnow'       => $mailnow,
             );
             if (!empty($subject)) {
                 $data['subject'] = $subject;
@@ -175,7 +175,7 @@ public function add_discussion_action() {
                 'reveal'        => $reveal,
                 'timestart'     => $timestart,
                 'timeend'       => $timeend,
-                'mailnow'       => $mailnow
+                'mailnow'       => $mailnow,
             );
             return $this->discussionservice->handle_add_discussion($course, $cm, $forum, $context, $options, $posttomygroups);
         } catch (\Exception $e) {
@@ -238,7 +238,7 @@ public function update_post_action() {
                 'reveal'        => $reveal,
                 'privatereply'  => $privatereply,
                 'timestart'     => $timestart,
-                'timeend'       => $timeend
+                'timeend'       => $timeend,
             ));
         } catch (\Exception $e) {
             return new json_response($e);
@@ -281,7 +281,7 @@ public function edit_post_form_action() {
                 'isdiscussion' => true,
                 'timestart'    => $discussion->timestart,
                 'timeend'      => $discussion->timeend,
-                'offset'       => $offset
+                'offset'       => $offset,
             ]);
         }
     }

classes/message/inbound/reply_handler.php

@@ -253,7 +253,7 @@ public function process_message(\stdClass $record, \stdClass $messagedata) {
                 'discussionid'  => $discussion->id,
                 'forumid'       => $forum->id,
                 'forumtype'     => $forum->type,
-            )
+            ),
         );
         $event = \mod_hsuforum\event\post_created::create($params);
         $event->add_record_snapshot('hsuforum_posts', $addpost);

classes/output/big_search_form.php

@@ -73,7 +73,7 @@ public function __construct($course) {
         $this->forumoptions = array_map(function($option) use ($forumoptions) {
             return [
                 'value' => $option,
-                'name' => $forumoptions[$option]
+                'name' => $forumoptions[$option],
             ];
         }, array_keys($forumoptions));
     }
@@ -200,7 +200,7 @@ public function export_for_template(renderer_base $output) {
         foreach ($tags as $tag) {
             $data->tagoptions[] = ['value'    => $tag->rawname,
                                    'text'     => $tag->fieldname,
-                                   'selected' => in_array($tag->rawname, $this->tags)
+                                   'selected' => in_array($tag->rawname, $this->tags),
             ];
         }
 

classes/post_form.php

@@ -52,7 +52,7 @@ public static function attachment_options($forum) {
             'maxbytes' => $maxbytes,
             'maxfiles' => $forum->maxattachments,
             'accepted_types' => '*',
-            'return_types' => FILE_INTERNAL | FILE_CONTROLLED_LINK
+            'return_types' => FILE_INTERNAL | FILE_CONTROLLED_LINK,
         );
     }
 
@@ -72,7 +72,7 @@ public static function editor_options(context_module $context, $postid) {
             'maxbytes' => $maxbytes,
             'trusttext'=> true,
             'return_types'=> FILE_INTERNAL | FILE_EXTERNAL,
-            'subdirs' => file_area_contains_subdirs($context, 'mod_hsuforum', 'post', $postid)
+            'subdirs' => file_area_contains_subdirs($context, 'mod_hsuforum', 'post', $postid),
         );
     }
 

classes/privacy/provider.php

@@ -98,7 +98,7 @@ public static function get_metadata(collection $items) : collection {
             'userid' => 'privacy:metadata:hsuforum_queue:userid',
             'discussionid' => 'privacy:metadata:hsuforum_queue:discussionid',
             'postid' => 'privacy:metadata:hsuforum_queue:postid',
-            'timemodified' => 'privacy:metadata:hsuforum_queue:timemodified'
+            'timemodified' => 'privacy:metadata:hsuforum_queue:timemodified',
         ], 'privacy:metadata:hsuforum_queue');
 
         // The 'hsuforum_read' table stores data about which forum posts have been read by each user.
@@ -561,7 +561,7 @@ protected static function export_discussion_data(int $userid, array $mappings) {
                 'pinned' => transform::yesno((bool)$discussion->pinned),
                 'timemodified' => transform::datetime($discussion->timemodified),
                 'usermodified' => transform::datetime($discussion->usermodified),
-                'creator_was_you' => transform::yesno($discussion->userid == $userid)
+                'creator_was_you' => transform::yesno($discussion->userid == $userid),
             ];
 
             $discussionarea = static::get_discussion_area($discussion);

classes/renderables/advanced_editor.php

@@ -110,7 +110,7 @@ public function get_data($draftid = 0){
             'editor' => $editor,
             'options' => $options,
             'fpoptions' => $fpoptions,
-            'draftitemid' => $draftitemid
+            'draftitemid' => $draftitemid,
         ];
     }
 }

classes/search/post.php

@@ -144,7 +144,7 @@ public function uses_file_indexing() {
     public function get_search_fileareas() {
         $fileareas = array(
             'attachment',
-            'post'
+            'post',
         );
 
         return $fileareas;
@@ -315,7 +315,7 @@ protected function get_discussion($discussionid) {
     protected function get_contexts_to_reindex_extra_sql() {
         return [
             'JOIN {hsuforum_discussions} fd ON fd.course = cm.course AND fd.forum = cm.instance',
-            'MAX(fd.timemodified) DESC'
+            'MAX(fd.timemodified) DESC',
         ];
     }
 

classes/service/discussion_service.php

@@ -242,7 +242,7 @@ public function trigger_discussion_created($course, \context_module $context, $c
             'objectid' => $discussion->id,
             'other'    => array(
                 'forumid' => $forum->id,
-            )
+            ),
         );
         $event = discussion_created::create($params);
         $event->add_record_snapshot('hsuforum_discussions', $discussion);