Adapt security for JSP UI, breaks RIA UI (user natures collection)

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

diff --git a/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/UserNature.java b/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/UserNature.java
index 52d1cd1fa61ab6d7b559d4f1dd08195aeb1f0f77..25710e40529021018fdf94fa1529b7bc42a3aa74 100644 (file)
--- a/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/UserNature.java
+++ b/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/UserNature.java
@@ -18,6 +18,7 @@ package org.argeo.security;
 
 import java.io.Serializable;
 import java.util.List;
+import java.util.Map;
 
 import org.argeo.ArgeoException;
 
@@ -38,7 +39,8 @@ public class UserNature implements Serializable {
        }
 
        public final static void updateUserNaturesWithCheck(
-                       List<UserNature> userNatures, List<UserNature> userNaturesData) {
+                       Map<String, UserNature> userNatures,
+                       Map<String, UserNature> userNaturesData) {
                if (userNatures.size() != userNaturesData.size())
                        throw new ArgeoException(
                                        "It is forbidden to add or remove user natures via this method");
@@ -55,8 +57,8 @@ public class UserNature implements Serializable {
                                                "Could not find a user nature of type " + type);
                }
 
-               for (int i = 0; i < userNatures.size(); i++) {
-                       userNatures.set(i, userNaturesData.get(i));
+               for (String key : userNatures.keySet()) {
+                       userNatures.put(key, userNaturesData.get(key));
                }
        }
 }