MT-97_8 : SNS :: Update_RoleTable_While_Account_CRUD_operations (#85)

src/main/java/com/nisum/mytime/service/AccountServiceImpl.java

@@ -16,6 +16,7 @@ import org.springframework.stereotype.Service;
 
 import com.nisum.mytime.exception.handler.MyTimeException;
 import com.nisum.mytime.model.Account;
+import com.nisum.mytime.model.Domains;
 import com.nisum.mytime.model.EmployeeRoles;
 import com.nisum.mytime.repository.AccountRepo;
 import com.nisum.mytime.utils.CommomUtil;
@@ -79,86 +80,60 @@ public class AccountServiceImpl implements AccountService {
 			}else {
 				response = "Error occured while account creating";
 			}
-		}else if(action != null && action.equals("U")){//U means updating existing Account
+		}else if(action != null && action.equals("U")){
+			List<String> employeeIds = null;
+			List<String> dmsAddedByUser = null;
+	        List<String> dmsDeletedByUser = null;
+	        
+	        Map<String, Integer> dmsCount = new HashMap<String,Integer>();
+	        
+			boolean dupAccChkFlag = duplicateAccountCheck(account);
 			
-			boolean flag = false;
-			boolean nameFlag = false;
-			boolean accountExistFlag  = false;
-			
-			List<Account> accounts = accountRepo.findAll();
-			
-			Account accountBeforeUpdate = accountRepo.findByAccountId(account.getAccountId());
-			
-			nameFlag = accountBeforeUpdate.getAccountName().equalsIgnoreCase(account.getAccountName());
-			
-			if(accounts!= null && !accounts.isEmpty() && accounts.size()>0 ){
-				for(Account accObj : accounts){
-					accountExistFlag = accObj.getAccountName().trim().equalsIgnoreCase(account.getAccountName().trim());
-					if(accountExistFlag){
-						break;
-					}
-				}
-			}
-			
-			if( nameFlag && accountExistFlag) {
+			if(dupAccChkFlag) {
 				response ="Account already exist";
 				return response;
 			}
+
+	        
+	        Account accountBeforeUpdate = accountRepo.findByAccountId(account.getAccountId());
+	        
+			List<String> updatedAccDms = account.getDeliveryManagers();
+			List<String> beforeAccUpdateDms = accountBeforeUpdate.getDeliveryManagers();
 			
-			//if(accountBeforeUpdate.getAccountName().equalsIgnoreCase(account.getAccountName())) {
-				List<String> employeeIds = null;
-				List<Account> accountList = null;
-				List<String> dmsAddedByUser = null;
-		        List<String> dmsDeletedByUser = null;
-		        
-		        Map<String, Integer> dmsCount = new HashMap<String,Integer>();
-		        
-		        
-		        
-				List<String> updatedAccDms = account.getDeliveryManagers();
-				List<String> beforeAccUpdateDms = accountBeforeUpdate.getDeliveryManagers();
-				
-				
-				dmsAddedByUser = CommomUtil.getAddedManagersList(beforeAccUpdateDms, updatedAccDms);
-				dmsDeletedByUser = CommomUtil.getDeletedManagersList(beforeAccUpdateDms, updatedAccDms);
-				
-				accountList = accountRepo.findAll();
-				
-				if(accountList != null && !accountList.isEmpty() && accountList.size() >0) {
-			        for (Account acc : accountList) {
-			            employeeIds = acc.getDeliveryManagers();
-			            if(employeeIds != null && !employeeIds.isEmpty() && employeeIds.size() >0) {
-				             for(String eId:employeeIds) {
-				            	  if(dmsCount.get(eId) != null) 
-				            		  dmsCount.put(eId, dmsCount.get(eId)+1);
-				                  else
-				                	  dmsCount.put(eId, 1);
-				             	employeeIds = null;
-				             }
-			            }
-			         }
-				}
-				
-		        for (String empId : dmsDeletedByUser) {
-		        	if(dmsCount.get(empId)==1) {   		
-		                //Service call for RoleMapping 
-		                roleMappingService.deleteRole(empId,roleId);
-		        	}
-		       } 
-		        
-		       account.setStatus(accountBeforeUpdate.getStatus());
-		       account.setAccountName(account.getAccountName().trim());
-		       accountResult = accountRepo.save(account);
-			   roleMappingService.saveUniqueEmployeeAndRole(dmsAddedByUser, roleId);
-			   response="Updated succesfully";
-			   return response;
-			}
+			dmsAddedByUser = CommomUtil.getAddedManagersList(beforeAccUpdateDms, updatedAccDms);
+			dmsDeletedByUser = CommomUtil.getDeletedManagersList(beforeAccUpdateDms, updatedAccDms);
 			
-
-
+			List<Account> accountList = accountRepo.findAll();
 			
-
-		//}
+			if(accountList != null && !accountList.isEmpty() && accountList.size() >0) {
+		        for (Account acc : accountList) {
+		            employeeIds = acc.getDeliveryManagers();
+		            if(employeeIds != null && !employeeIds.isEmpty() && employeeIds.size() >0) {
+			             for(String eId:employeeIds) {
+			            	  if(dmsCount.get(eId) != null) 
+			            		  dmsCount.put(eId, dmsCount.get(eId)+1);
+			                  else
+			                	  dmsCount.put(eId, 1);
+			             	employeeIds = null;
+			             }
+		            }
+		         }
+			}
+			
+	       for (String empId : dmsDeletedByUser) {
+	        	if(dmsCount.get(empId)==1) {   		
+	                //Service call for RoleMapping 
+	                roleMappingService.deleteRole(empId,roleId);
+	        	}
+	       } 
+	       
+	       account.setStatus(accountBeforeUpdate.getStatus());
+	       account.setAccountName(account.getAccountName().trim());
+	       accountResult = accountRepo.save(account);
+		   roleMappingService.saveUniqueEmployeeAndRole(dmsAddedByUser, roleId);
+		   response="Updated succesfully";
+		   return response;
+		}
 		return response;
 	}
 
@@ -261,12 +236,24 @@ public class AccountServiceImpl implements AccountService {
 		return (MyTimeUtils.ACC + MyTimeUtils.ZERO_) + (getAccounts().size() + MyTimeUtils.ONE);
 	}
 	
-   private boolean duplicateCheck(String accountName,String accountId){
-    	boolean check=false;
-    	List<Account> accountList=accountRepo.findByaccountNameAndAccountId(accountName, accountId);
-        if(accountList.size()>0) 
-        	check= true;
-        return check;
+    private boolean duplicateAccountCheck(Account UpdatedAccount ) {
+    	boolean flag = false;
+    	
+       	Account accbeforeUpdate = accountRepo.findByAccountId(UpdatedAccount.getAccountId());
+       	
+       	boolean nameFlag = accbeforeUpdate.getAccountName().equals(UpdatedAccount.getAccountName());
+       	boolean addrFlag = accbeforeUpdate.getClientAddress().equals(UpdatedAccount.getClientAddress());
+       	boolean industryFlag = accbeforeUpdate.getIndustryType().equals(UpdatedAccount.getIndustryType());
+       	boolean dmsFlag=accbeforeUpdate.getDeliveryManagers().toString().contentEquals(UpdatedAccount.getDeliveryManagers().toString());
+       	
+       	boolean ignoreFlag = accbeforeUpdate.getAccountName().equalsIgnoreCase(UpdatedAccount.getAccountName());
+       	
+       	if( nameFlag || !addrFlag || !industryFlag || !dmsFlag ) {
+       		flag = false;
+       	}else if(ignoreFlag){
+       		flag = true;
+       	}       	
+    	return flag;
     }
 
 }