You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@stratos.apache.org by im...@apache.org on 2015/05/13 07:14:04 UTC

[2/7] stratos git commit: Improved logs as per the comments

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/12552f2f
Tree: http://git-wip-us.apache.org/repos/asf/stratos/tree/12552f2f
Diff: http://git-wip-us.apache.org/repos/asf/stratos/diff/12552f2f

Branch: refs/heads/master
Commit: 12552f2f2a4781831753bcd962614efa96ae8717
Parents: 9d98bed
Author: Dinithi <di...@wso2.com>
Authored: Tue May 12 21:09:08 2015 +0530
Committer: Imesh Gunaratne <im...@apache.org>
Committed: Wed May 13 10:43:56 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/12552f2f/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);