X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;f=security%2Fruntime%2Forg.argeo.security.core%2Fsrc%2Fmain%2Fjava%2Forg%2Fargeo%2Fsecurity%2FSimpleArgeoUser.java;h=ca7229c2b27cef7770295f84500ba54d98ccd2b8;hb=ab4dcfd6237c3f2064e63b3e6bb1a740f0a56e2c;hp=19c45dfbad0e0aeb7087272bb90cc28823894bc8;hpb=79d37fb873ba8ebaafe64c924f53bc9c76ea6d8c;p=lgpl%2Fargeo-commons.git

diff --git a/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/SimpleArgeoUser.java b/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/SimpleArgeoUser.java
index 19c45dfba..ca7229c2b 100644
--- a/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/SimpleArgeoUser.java
+++ b/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/SimpleArgeoUser.java
@@ -22,6 +22,12 @@ import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 
+import org.argeo.ArgeoException;
+
+/**
+ * Read-write implementation of an Argeo user. Typically initialized with a
+ * generic instance (read-only9 in order to modify a user.
+ */
 public class SimpleArgeoUser implements ArgeoUser, Serializable {
 	private static final long serialVersionUID = 1L;
 
@@ -47,7 +53,44 @@ public class SimpleArgeoUser implements ArgeoUser, Serializable {
 	}
 
 	public void updateUserNatures(Map<String, UserNature> userNaturesData) {
-		UserNature.updateUserNaturesWithCheck(userNatures, userNaturesData);
+		updateUserNaturesWithCheck(userNatures, userNaturesData);
+	}
+
+	public static void updateUserNaturesWithCheck(
+			Map<String, UserNature> userNatures,
+			Map<String, UserNature> userNaturesData) {
+		// checks consistency
+		if (userNatures.size() != userNaturesData.size())
+			throw new ArgeoException(
+					"It is forbidden to add or remove user natures via this method");
+
+		for (String type : userNatures.keySet()) {
+			if (!userNaturesData.containsKey(type))
+				throw new ArgeoException(
+						"Could not find a user nature of type " + type);
+		}
+
+		// for (int i = 0; i < userNatures.size(); i++) {
+		// String type = userNatures.get(i).getType();
+		// boolean found = false;
+		// for (int j = 0; j < userNatures.size(); j++) {
+		// String newType = userNaturesData.get(j).getType();
+		// if (type.equals(newType))
+		// found = true;
+		// }
+		// if (!found)
+		// throw new ArgeoException(
+		// "Could not find a user nature of type " + type);
+		// }
+
+		for (String key : userNatures.keySet()) {
+			userNatures.put(key, userNaturesData.get(key));
+		}
+	}
+
+	@Override
+	public String toString() {
+		return username;
 	}
 
 	public List<String> getRoles() {