Description
1. There are some duplicate codes in method mapXXPortalUserToVXPortalUserForDefaultAccount.
userProfile.setLoginId(user.getLoginId()); userProfile.setEmailAddress(user.getEmailAddress()); userProfile.setStatus(user.getStatus()); userProfile.setUserRoleList(new ArrayList<String>()); userProfile.setId(user.getId()); userProfile.setFirstName(user.getFirstName()); userProfile.setLastName(user.getLastName()); userProfile.setPublicScreenName(user.getPublicScreenName()); userProfile.setEmailAddress(user.getEmailAddress());
We set the email address twice.
2. We have already checked if sess is null, so don't need to check it in below logic.
UserSessionBase sess = ContextUtil.getCurrentUserSession(); if (sess != null) { if (sess != null && sess.isUserAdmin() || sess.isKeyAdmin()) { return; } if (sess.getXXPortalUser().getId().equals(gjUser.getId())) { return; } }