Check if the notice can be dismissed not in the command

Cyperghost · Cyperghost · commit 87bde1b0d00d · 2025-08-13T13:52:29.000+02:00
diff --git a/wcfsetup/install/files/lib/command/notice/DismissNotice.class.php b/wcfsetup/install/files/lib/command/notice/DismissNotice.class.php
@@ -24,10 +24,6 @@ public function __construct(
 
     public function __invoke(): void
     {
-        if (!$this->notice->isDismissible) {
-            return;
-        }
-
         if (WCF::getUser()->userID) {
             $this->dismissForUser($this->notice, WCF::getUser()->userID);
         } else {
diff --git a/wcfsetup/install/files/lib/data/notice/NoticeAction.class.php b/wcfsetup/install/files/lib/data/notice/NoticeAction.class.php
@@ -81,7 +81,9 @@ public function dismiss()
     {
         $editor = $this->getSingleObject();
 
-        (new DismissNotice($editor->getDecoratedObject()))();
+        if ($editor->isDismissible) {
+            (new DismissNotice($editor->getDecoratedObject()))();
+        }
 
         return [
             'noticeID' => \reset($this->objectIDs),

Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,9 @@ public function dismiss()`
`81`	`81`	`{`
`82`	`82`	`$editor = $this->getSingleObject();`
`83`	`83`
`84`		`- (new DismissNotice($editor->getDecoratedObject()))();`
	`84`	`+ if ($editor->isDismissible) {`
	`85`	`+ (new DismissNotice($editor->getDecoratedObject()))();`
	`86`	`+ }`
`85`	`87`
`86`	`88`	`return [`
`87`	`89`	`'noticeID' => \reset($this->objectIDs),`