fixed another required paramter following an optional one

Author: Matthias Opitz

classes/controller/edit_controller.php

@@ -228,7 +228,7 @@ public function update_post_action() {
             $timestart = $timestart === 0 ? $discussion->timestart : 0;
             $timeend = $timeend === 0 ? $discussion->timeend : 0;
 
-            return $this->postservice->handle_update_post($course, $cm, $forum, $context, $discussion, $post, $files,  array(
+            return $this->postservice->handle_update_post($course, $cm, $forum, $context, $discussion, $post,  array(
                 'subject'       => $subject,
                 'name'          => $subject,
                 'groupid'       => $groupid,
@@ -239,7 +239,7 @@ public function update_post_action() {
                 'privatereply'  => $privatereply,
                 'timestart'     => $timestart,
                 'timeend'       => $timeend
-            ));
+            ), $files);
         } catch (\Exception $e) {
             return new json_response($e);
         }

classes/service/post_service.php

@@ -132,7 +132,7 @@ public function handle_user_autosubscription($forum, $post) {
      * @param array $options These override default post values, EG: set the post message with this
      * @return json_response
      */
-    public function handle_update_post($course, $cm, $forum, $context, $discussion, $post, array $deletefiles, array $options) {
+    public function handle_update_post($course, $cm, $forum, $context, $discussion, $post, array $options, array $deletefiles = array()) {
 
         $this->require_can_edit_post($forum, $context, $discussion, $post);