diff --git a/lib/Notification/NotificationHelper.php b/lib/Notification/NotificationHelper.php index 9715da7fc..299e1332d 100644 --- a/lib/Notification/NotificationHelper.php +++ b/lib/Notification/NotificationHelper.php @@ -107,7 +107,7 @@ class NotificationHelper { if ($user->getUID() === $board->getOwner() && count($board->getAcl()) === 0) { // Notify if all or assigned is configured for unshared boards $shouldNotify = true; - } else if ($notificationSetting === ConfigService::SETTING_BOARD_NOTIFICATION_DUE_ASSIGNED && $this->assignedUsersMapper->isUserAssigned($card->getId(), $user->getUID())) { + } elseif ($notificationSetting === ConfigService::SETTING_BOARD_NOTIFICATION_DUE_ASSIGNED && $this->assignedUsersMapper->isUserAssigned($card->getId(), $user->getUID())) { // Notify if the user is assigned and has the assigned setting selected $shouldNotify = true; } diff --git a/tests/unit/Notification/NotificationHelperTest.php b/tests/unit/Notification/NotificationHelperTest.php index 2cd8fcc66..80d7997e6 100644 --- a/tests/unit/Notification/NotificationHelperTest.php +++ b/tests/unit/Notification/NotificationHelperTest.php @@ -42,7 +42,6 @@ use OCP\Notification\INotification; use PHPUnit\Framework\MockObject\MockObject; class DummyUser extends \OC\User\User { - private $uid; public function __construct($uid) {