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=29166472146f8bd3e6e39aa30597da0c2b6f835d;hb=a8233e9378854fc9ed1f4186095d06866cbea9f8;hp=e736da8f4edfdb430e43117a03bede536aaa143b;hpb=dc6f911c680574a9b57c34dcf515ac8f9aa808da;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 e736da8f4..291664721 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
@@ -24,7 +24,13 @@ import java.util.Map;
 
 import org.argeo.ArgeoException;
 
-public class SimpleArgeoUser implements ArgeoUser, Serializable {
+/**
+ * Read-write implementation of an Argeo user. Typically initialized with a
+ * generic instance (read-only9 in order to modify a user.
+ */
+@Deprecated
+public class SimpleArgeoUser implements ArgeoUser, Serializable,
+		Comparable<ArgeoUser> {
 	private static final long serialVersionUID = 1L;
 
 	private String username;
@@ -48,10 +54,12 @@ public class SimpleArgeoUser implements ArgeoUser, Serializable {
 		return userNatures;
 	}
 
+	@Deprecated
 	public void updateUserNatures(Map<String, UserNature> userNaturesData) {
 		updateUserNaturesWithCheck(userNatures, userNaturesData);
 	}
 
+	@Deprecated
 	public static void updateUserNaturesWithCheck(
 			Map<String, UserNature> userNatures,
 			Map<String, UserNature> userNaturesData) {
@@ -66,24 +74,27 @@ public class SimpleArgeoUser implements ArgeoUser, Serializable {
 						"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 boolean equals(Object obj) {
+		if (!(obj instanceof ArgeoUser))
+			return false;
+		return ((ArgeoUser) obj).getUsername().equals(username);
+	}
+
+	public int compareTo(ArgeoUser o) {
+		return username.compareTo(o.getUsername());
+	}
+
+	@Override
+	public int hashCode() {
+		return username.hashCode();
+	}
+
 	@Override
 	public String toString() {
 		return username;
@@ -101,6 +112,7 @@ public class SimpleArgeoUser implements ArgeoUser, Serializable {
 		this.username = username;
 	}
 
+	@Deprecated
 	public void setUserNatures(Map<String, UserNature> userNatures) {
 		this.userNatures = userNatures;
 	}