Fix rendering of attached files on new reply by PushService (#423)

humhub · Oct 12, 2024 · 92364ce · 92364ce
1 parent 0466bad
commit 92364ce
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/docs/CHANGELOG.md b/docs/CHANGELOG.md
@@ -10,6 +10,7 @@ Changelog
 - Enh #417: Replace theme variables with CSS variables
 - Enh #418: Update Active Form for Bootstrap 5
 - Enh #422: Changed deprecated `DOMSubtreeModified` to `MutationObserver`
+- Fix #423: Fix rendering of attached files on new reply by PushService
 
 3.2.2 (July 9, 2024)
 --------------------

diff --git a/models/forms/ReplyForm.php b/models/forms/ReplyForm.php
@@ -80,8 +80,8 @@ public function save()
 
         if ($this->reply->save()) {
             $this->reply->refresh(); // Update created_by date, otherwise db expression is set...
-            $this->reply->notify();
             $this->reply->fileManager->attach(Yii::$app->request->post('fileList'));
+            $this->reply->notify();
 
             // Update last viewed date to avoid marking the conversation as unread
             $userMessage = $this->model->getUserMessage($this->reply->user_id);