Wrap roles so that it is writable

[lgpl/argeo-commons.git] / security / runtime / org.argeo.security.core / src / main / java / org / argeo / security / core / DefaultSecurityService.java

diff --git a/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/core/DefaultSecurityService.java b/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/core/DefaultSecurityService.java
index 68f97d4a069e3eaa73257d943da72913ea5bf24d..62ce6c759cc8506a0f523850749007acd34b0161 100644 (file)
--- a/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/core/DefaultSecurityService.java
+++ b/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/core/DefaultSecurityService.java
@@ -16,6 +16,7 @@
 
 package org.argeo.security.core;
 
+import java.util.HashSet;
 import java.util.Iterator;
 import java.util.Set;
 
@@ -134,7 +135,8 @@ public class DefaultSecurityService extends DefaultCurrentUserService implements
        }
 
        public Set<ArgeoUser> listUsersInRole(String role) {
-               Set<ArgeoUser> lst = securityDao.listUsersInRole(role);
+               Set<ArgeoUser> lst = new HashSet<ArgeoUser>(
+                               securityDao.listUsersInRole(role));
                Iterator<ArgeoUser> it = lst.iterator();
                while (it.hasNext()) {
                        if (it.next().getUsername()