fix(ExAppNotifier): Use empty() instead of count()

marcelklehr · web-flow · commit fdbfa0241c02 · 2025-07-17T10:46:33.000+02:00
more performant

Signed-off-by: Marcel Klehr &lt;mklehr@gmx.net&gt;
diff --git a/lib/Notifications/ExAppNotifier.php b/lib/Notifications/ExAppNotifier.php
@@ -36,7 +36,7 @@ public function getName(): string {
 	}
 
 	public function prepare(INotification $notification, string $languageCode): INotification {
-		if (count($this->service->getExAppsList()) === 0) {
+		if (empty($this->service->getExAppsList())) {
 			throw new UnknownNotificationException();
 		}
 		$exApp = $this->service->getExApp($notification->getApp());

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ public function getName(): string {`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`public function prepare(INotification $notification, string $languageCode): INotification {`
`39`		`- if (count($this->service->getExAppsList()) === 0) {`
	`39`	`+ if (empty($this->service->getExAppsList())) {`
`40`	`40`	`throw new UnknownNotificationException();`
`41`	`41`	`}`
`42`	`42`	`$exApp = $this->service->getExApp($notification->getApp());`