From e91ba2b9929a532e6a313a7786a082e76b835aac Mon Sep 17 00:00:00 2001 From: Hypolite Petovan Date: Sun, 3 Oct 2021 11:34:23 -0400 Subject: [PATCH] Check if the string representation of Notify->link is empty in Module\Notifications\Notification - The property is a Url object which will always return true when tested for truthy value --- src/Module/Notifications/Notification.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Module/Notifications/Notification.php b/src/Module/Notifications/Notification.php index 2480e63b0b..19ee410d28 100644 --- a/src/Module/Notifications/Notification.php +++ b/src/Module/Notifications/Notification.php @@ -124,7 +124,7 @@ class Notification extends BaseModule DI::notify()->setAllSeenForRelatedNotify($Notify); } - if ($Notify->link) { + if ((string)$Notify->link) { System::externalRedirect($Notify->link); }