LDAP support for hierarchy unit. Code clean up.

[lgpl/argeo-commons.git] / org.argeo.util / src / org / argeo / osgi / useradmin / UserDirectoryWorkingCopy.java

diff --git a/org.argeo.util/src/org/argeo/osgi/useradmin/UserDirectoryWorkingCopy.java b/org.argeo.util/src/org/argeo/osgi/useradmin/UserDirectoryWorkingCopy.java
index 0e25bdfa12c441459e4f6bbee26353aa588ec5b2..35a34bdbe4425d61e9cdef09e90007613bc6354b 100644 (file)
--- a/org.argeo.util/src/org/argeo/osgi/useradmin/UserDirectoryWorkingCopy.java
+++ b/org.argeo.util/src/org/argeo/osgi/useradmin/UserDirectoryWorkingCopy.java
@@ -27,8 +27,7 @@ class UserDirectoryWorkingCopy {
        }
 
        public boolean noModifications() {
-               return newUsers.size() == 0 && modifiedUsers.size() == 0
-                               && deletedUsers.size() == 0;
+               return newUsers.size() == 0 && modifiedUsers.size() == 0 && deletedUsers.size() == 0;
        }
 
        public Attributes getAttributes(LdapName dn) {
@@ -40,7 +39,7 @@ class UserDirectoryWorkingCopy {
        public void startEditing(DirectoryUser user) {
                LdapName dn = user.getDn();
                if (modifiedUsers.containsKey(dn))
-                       throw new UserDirectoryException("Already editing " + dn);
+                       throw new IllegalStateException("Already editing " + dn);
                modifiedUsers.put(dn, (Attributes) user.getAttributes().clone());
        }