Improved logs as per the comments

Project: http://git-wip-us.apache.org/repos/asf/stratos/repo
Commit: http://git-wip-us.apache.org/repos/asf/stratos/commit/083180ff
Tree: http://git-wip-us.apache.org/repos/asf/stratos/tree/083180ff
Diff: http://git-wip-us.apache.org/repos/asf/stratos/diff/083180ff

Branch: refs/heads/master
Commit: 083180ff701897a809f57d5c933286b4e11aca2a
Parents: 87ae08d
Author: Dinithi <[email protected]>
Authored: Tue May 12 21:09:08 2015 +0530
Committer: Dinithi <[email protected]>
Committed: Tue May 12 21:09:08 2015 +0530

----------------------------------------------------------------------
 .../account/mgt/services/AccountMgtService.java | 20 ++++++++------------
 1 file changed, 8 insertions(+), 12 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/stratos/blob/083180ff/components/org.apache.stratos.account.mgt/src/main/java/org/apache/stratos/account/mgt/services/AccountMgtService.java
----------------------------------------------------------------------
diff --git 
a/components/org.apache.stratos.account.mgt/src/main/java/org/apache/stratos/account/mgt/services/AccountMgtService.java
 
b/components/org.apache.stratos.account.mgt/src/main/java/org/apache/stratos/account/mgt/services/AccountMgtService.java
index 11d5a91..22835cc 100644
--- 
a/components/org.apache.stratos.account.mgt/src/main/java/org/apache/stratos/account/mgt/services/AccountMgtService.java
+++ 
b/components/org.apache.stratos.account.mgt/src/main/java/org/apache/stratos/account/mgt/services/AccountMgtService.java
@@ -170,19 +170,15 @@ public class AccountMgtService extends AbstractAdmin {
         RealmService realmService = Util.getRealmService();
         try {
             Map<String, String> claimsMap = new HashMap<String, String>();
-            claimsMap.put(UserCoreConstants.ClaimTypeURIs.GIVEN_NAME,
-                    accountInfoBean.getFirstName());
+            claimsMap.put(UserCoreConstants.ClaimTypeURIs.GIVEN_NAME, 
accountInfoBean.getFirstName());
             claimsMap.put(UserCoreConstants.ClaimTypeURIs.SURNAME, 
accountInfoBean.getLastName());
-            UserStoreManager userStoreManager =
-                    (UserStoreManager) 
realmService.getTenantUserRealm(tenantId)
-                            .getUserStoreManager();
-            userStoreManager.setUserClaimValues(
-                    ClaimsMgtUtil.getAdminUserNameFromTenantId(realmService, 
tenantId),
+            UserStoreManager userStoreManager = (UserStoreManager) 
realmService.getTenantUserRealm(tenantId)
+                    .getUserStoreManager();
+            
userStoreManager.setUserClaimValues(ClaimsMgtUtil.getAdminUserNameFromTenantId(realmService,
 tenantId),
                     claimsMap, UserCoreConstants.DEFAULT_PROFILE);
-            log.info("FirstName: " + accountInfoBean.getFirstName() +
-                    " has been updated to the tenant admin " +
-                    ClaimsMgtUtil.getAdminUserNameFromTenantId(realmService, 
tenantId) + " of " +
-                    tenant.getDomain());
+            log.info("FirstName: " + accountInfoBean.getFirstName() + " of the 
tenant " + tenant.getDomain() + " has " +
+                    "been updated to the tenant admin " + 
ClaimsMgtUtil.getAdminUserNameFromTenantId(realmService,
+                    tenantId) + " of " + tenant.getDomain());
 
             //Notify tenant update to all listeners
             TenantInfoBean tenantInfoBean = new TenantInfoBean();
@@ -196,7 +192,7 @@ public class AccountMgtService extends AbstractAdmin {
             // this is expected, as many users haven't given their fullnames
             // during their registration.
             String msg =
-                    "Error in updating the firstname: " + 
accountInfoBean.getFirstName() +
+                    "Error in updating first name: " + 
accountInfoBean.getFirstName() +
                             " for the tenant admin: " +
                             
ClaimsMgtUtil.getAdminUserNameFromTenantId(realmService, tenantId);
             log.info(msg);

Reply via email to