From a332264920c9cd9226d566c19e36e5d5b5549c1b Mon Sep 17 00:00:00 2001
From: ZhuDongming <773644075@qq.com>
Date: Mon, 12 Aug 2019 15:11:31 +0800
Subject: [PATCH] Merge branch 'master' of http://blit.7drlb.com:8888/r/screen_api_v2

---
 src/main/java/com/moral/service/impl/AccountServiceImpl.java |   79 ++++++++++++++++++++++++++++-----------
 1 files changed, 56 insertions(+), 23 deletions(-)

diff --git a/src/main/java/com/moral/service/impl/AccountServiceImpl.java b/src/main/java/com/moral/service/impl/AccountServiceImpl.java
index 1680f0c..243ac98 100644
--- a/src/main/java/com/moral/service/impl/AccountServiceImpl.java
+++ b/src/main/java/com/moral/service/impl/AccountServiceImpl.java
@@ -2,20 +2,16 @@
 
 import static com.moral.common.bean.Constants.IS_DELETE_FALSE;
 import static com.moral.common.bean.Constants.IS_DELETE_TRUE;
-import static com.moral.common.util.Crypto.md5;
 import static com.moral.common.util.ResourceUtil.getValue;
 import static org.apache.commons.lang3.StringUtils.isNumeric;
 import static org.springframework.util.ObjectUtils.isEmpty;
 
-import java.util.Date;
-import java.util.HashMap;
-import java.util.HashSet;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
+import java.util.*;
 
 import javax.annotation.Resource;
 
+import org.apache.commons.lang3.StringUtils;
+import org.springframework.security.crypto.bcrypt.BCryptPasswordEncoder;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 import org.springframework.util.ObjectUtils;
@@ -26,6 +22,7 @@
 import com.moral.common.exception.BusinessException;
 import com.moral.common.util.Crypto;
 import com.moral.common.util.ResourceUtil;
+import com.moral.common.util.ValidateUtil;
 import com.moral.entity.Account;
 import com.moral.entity.Organization;
 import com.moral.mapper.AccountMapper;
@@ -38,7 +35,8 @@
 
 @Service
 public class AccountServiceImpl implements AccountService {
-
+    @Resource
+	private  BCryptPasswordEncoder encoder;
 	@Resource
 	private AccountMapper accountMapper;
 
@@ -47,21 +45,24 @@
 
 	@Resource
 	private OrganizationMapper organizationMapper;
-	
+
 	@Override
 	public Map<String, Object> screenLogin(Map<String, Object> parameters) {
 		Map<String, Object> result = new HashMap<String, Object>();
 		Account account = new Account();
 		account.setAccountName((String) parameters.get("account"));
-		account.setPassword(md5((String) parameters.get("password")));
+		String rawPassword = (String) parameters.get("password");
+//		account.setPassword(encoder.encode((String) parameters.get("password")));
 		account = accountMapper.selectOne(account);
-		if (isEmpty(account)) {
+		boolean isValid = account == null ? false:encoder.matches(rawPassword,account.getPassword());
+		if (!isValid) {
 			result.put("msg", "���������������������������������");
 		} else {
 			if (IS_DELETE_FALSE.equals(account.getIsDelete())) {
 				result.put("msg", "���������������");
 				result.put("accountId", account.getId());
 				result.put("orgId", account.getOrganizationId());
+				setOrgIdsByAccount(result);
 			} else {
 				result.put("msg","���������������������������������������������");
 			}
@@ -75,12 +76,15 @@
 		account.setAccountName(accountName);
 		return accountMapper.selectOne(account);
 	}
-
+	@Override
+	public Account getAccountById(Integer id){
+		return accountMapper.selectByPrimaryKey(id);
+	}
 	@Override
 	public void setOrgIdsByAccount(Map<String, Object> parameters) {
-		String accountId = (String) parameters.get("accountId");
+		String accountId = parameters.get("accountId").toString();
 		accountId = accountId.replaceFirst("-", "");
-		if (!isNumeric((String) parameters.get("accountId"))) {
+		if (!isNumeric(accountId)) {
 			throw new BusinessException("accountId ������������������");
 		}
 
@@ -121,14 +125,16 @@
 				organizationIds.add(account.getOrganizationId());
 			}
 		}
-		example = new Example(Organization.class);
-		example.or().andIn("id", organizationIds);
-		List<Organization> organizations = organizationMapper.selectByExample(example);
-		for (Account account : accounts) {
-			for (Organization organization : organizations) {
-				if (account.getOrganizationId() == organization.getId()) {
-					account.setOrganization(organization);
-					break;
+		if(!ObjectUtils.isEmpty(organizationIds)){
+			example = new Example(Organization.class);
+			example.or().andIn("id", organizationIds);
+			List<Organization> organizations = organizationMapper.selectByExample(example);
+			for (Account account : accounts) {
+				for (Organization organization : organizations) {
+					if (account.getOrganizationId() == organization.getId()) {
+						account.setOrganization(organization);
+						break;
+					}
 				}
 			}
 		}
@@ -139,10 +145,14 @@
 	@Override
 	@Transactional
 	public Integer saveOrUpdateAccount(Account account) {
+		// ������������
+		if(!StringUtils.isBlank(account.getPassword())){
+			account.setPassword(encoder.encode(account.getPassword()));
+		}
 		if (ObjectUtils.isEmpty(account.getId())) {
 			account.setIsDelete(Constants.IS_DELETE_FALSE);
 			account.setCreateTime(new Date());
-			account.setPassword(Crypto.md5(ResourceUtil.getValue("password")));
+			account.setPassword(encoder.encode(ResourceUtil.getValue("password")));
 			return accountMapper.insertSelective(account);
 		} else {
 			return accountMapper.updateByPrimaryKeySelective(account);
@@ -166,4 +176,27 @@
 		return accountMapper.selectCount(account);
 	}
 
+	@Override
+	public Optional<Account> queryAccountByName(String accountName) {
+		Account account = accountMapper.getByAccountName(accountName);
+		return Optional.ofNullable(account);
+	}
+
+	@Override
+	public Account companyLogin(Map<String, Object> parameters) {
+		ValidateUtil.notNull(parameters.get("account"), "param.is.null");
+		ValidateUtil.notNull(parameters.get("password"), "param.is.null");
+		Account account = new Account();
+		account.setIsDelete(Constants.IS_DELETE_FALSE);
+		account.setAccountName(parameters.get("account").toString());
+		account = accountMapper.selectOne(account);
+		if (ObjectUtils.isEmpty(account)) {
+			throw new BusinessException("���������������������������������������");
+		}
+		if (!encoder.matches(parameters.get("password").toString(), account.getPassword())) {
+			throw new BusinessException("���������������������������������");
+		}
+		account.setPassword(parameters.get("password").toString());
+		return account;
+	}
 }

--
Gitblit v1.8.0