Authentication working for workbench, web and data.

[lgpl/argeo-commons.git] / org.argeo.security.ui / src / org / argeo / security / ui / commands / OpenChangePasswordDialog.java

diff --git a/org.argeo.security.ui/src/org/argeo/security/ui/commands/OpenChangePasswordDialog.java b/org.argeo.security.ui/src/org/argeo/security/ui/commands/OpenChangePasswordDialog.java
index ed3e2c13c7e8c1eaeebbaa791ff20fad5f31835e..560cc8bc9f3de093a340288e192dc5ad7ffabdc5 100644 (file)
--- a/org.argeo.security.ui/src/org/argeo/security/ui/commands/OpenChangePasswordDialog.java
+++ b/org.argeo.security.ui/src/org/argeo/security/ui/commands/OpenChangePasswordDialog.java
@@ -63,6 +63,7 @@ public class OpenChangePasswordDialog extends AbstractHandler {
                return null;
        }
 
+       @SuppressWarnings("unchecked")
        protected void changePassword(char[] oldPassword, char[] newPassword) {
                Subject subject = Subject.getSubject(AccessController.getContext());
                String name = subject.getPrincipals(X500Principal.class).iterator()
@@ -76,9 +77,8 @@ public class OpenChangePasswordDialog extends AbstractHandler {
                try {
                        userTransaction.begin();
                        User user = (User) userAdmin.getRole(dn.toString());
-                       if (user.hasCredential(null, oldPassword)) {
+                       if (user.hasCredential(null, oldPassword))
                                user.getCredentials().put(null, newPassword);
-                       }
                        userTransaction.commit();
                } catch (Exception e) {
                        try {