Author: jonathan
Date: Thu Aug  7 15:49:01 2008
New Revision: 20559
URL: http://wso2.org/svn/browse/wso2?view=rev&revision=20559

Log:
Copying 20556 to branch.  With this fix MASHUP-1082 is fixed.

Modified:
   
branches/mashup/java/1.5/java/modules/core/src/org/wso2/mashup/realm/wsas/WSASUserStoreAdmin.java

Modified: 
branches/mashup/java/1.5/java/modules/core/src/org/wso2/mashup/realm/wsas/WSASUserStoreAdmin.java
URL: 
http://wso2.org/svn/browse/wso2/branches/mashup/java/1.5/java/modules/core/src/org/wso2/mashup/realm/wsas/WSASUserStoreAdmin.java?rev=20559&r1=20558&r2=20559&view=diff
==============================================================================
--- 
branches/mashup/java/1.5/java/modules/core/src/org/wso2/mashup/realm/wsas/WSASUserStoreAdmin.java
   (original)
+++ 
branches/mashup/java/1.5/java/modules/core/src/org/wso2/mashup/realm/wsas/WSASUserStoreAdmin.java
   Thu Aug  7 15:49:01 2008
@@ -212,13 +212,19 @@
 
     public void updateUser(String userName, Object newCredential, Object 
oldCredential)
             throws UserStoreException {
+        boolean success;
         try {
             UserAdmin admin = new UserAdmin();
-            admin.editUserPassword((String) oldCredential, userName, (String) 
newCredential);
+            success = admin.editUserPassword((String) oldCredential, userName, 
(String)
+                    newCredential);
+
+            // Since API is fixed, we have to throw an exception if edit 
operation returns false.
+            if (!success) {
+                throw new UserStoreException("Password Change Failed");
+            }
         } catch (Exception e) {
             throw new UserStoreException(e.getMessage());
         }
-
     }
 
     public void removeUserFromRole(String userName, String roleName) throws 
UserStoreException {

_______________________________________________
Mashup-dev mailing list
[email protected]
http://mailman.wso2.org/cgi-bin/mailman/listinfo/mashup-dev

Reply via email to