From 125dfe9b030f8554e47db27005ebfa4ef5fb96c1 Mon Sep 17 00:00:00 2001
From: Robert von Burg <eitch@eitchnet.ch>
Date: Thu, 7 Mar 2024 12:26:51 +0100
Subject: [PATCH] [Fix] Notification only visible to all if forAll

otherwise role, etc. must be set and match
---
 .../policy/notifications/DefaultNotificationsPolicy.java      | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/agent/src/main/java/li/strolch/policy/notifications/DefaultNotificationsPolicy.java b/agent/src/main/java/li/strolch/policy/notifications/DefaultNotificationsPolicy.java
index 2c8dc50ee..e18658c47 100644
--- a/agent/src/main/java/li/strolch/policy/notifications/DefaultNotificationsPolicy.java
+++ b/agent/src/main/java/li/strolch/policy/notifications/DefaultNotificationsPolicy.java
@@ -44,11 +44,11 @@ public class DefaultNotificationsPolicy extends NotificationsPolicy {
 
 	protected boolean isForRole(Resource notification) {
 		List<String> roles = notification.getStringList(BAG_VISIBILITY, PARAM_ROLES);
-		return roles.isEmpty() || roles.stream().anyMatch(r -> tx().getCertificate().hasRole(r));
+		return roles.stream().anyMatch(r -> tx().getCertificate().hasRole(r));
 	}
 
 	protected boolean isForGroup(Resource notification) {
 		List<String> groups = notification.getStringList(BAG_VISIBILITY, PARAM_GROUPS);
-		return groups.isEmpty() || groups.stream().anyMatch(r -> tx().getCertificate().hasGroup(r));
+		return groups.stream().anyMatch(r -> tx().getCertificate().hasGroup(r));
 	}
 }