fix: properly fire custom observables

flarum · Oct 18, 2023 · 5c58265 · 5c58265
1 parent 05324a0
commit 5c58265
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/framework/core/src/Discussion/Discussion.php b/framework/core/src/Discussion/Discussion.php
@@ -85,6 +85,8 @@ class Discussion extends AbstractModel
         'hidden_at' => 'datetime',
     ];
 
+    protected $observables = ['hidden'];
+
     /**
      * The user for which the state relationship should be loaded.
      */
@@ -144,7 +146,9 @@ public function hide(?User $actor = null): static
 
             $this->raise(new Hidden($this));
 
-            $this->fireCustomModelEvent('hidden', false);
+            $this->afterSave(function () {
+                $this->fireCustomModelEvent('hidden', false);
+            });
         }
 
         return $this;
@@ -158,7 +162,9 @@ public function restore(): static
 
             $this->raise(new Restored($this));
 
-            $this->fireCustomModelEvent('restored', false);
+            $this->afterSave(function () {
+                $this->fireCustomModelEvent('restored', false);
+            });
         }
 
         return $this;