From 6e9c3733135fa96360c8c76e76b431efa5d7f52d Mon Sep 17 00:00:00 2001 From: 沈斌 <bluelazysb@hotmail.com> Date: Sat, 09 Jun 2018 11:28:32 +0800 Subject: [PATCH] test --- src/main/java/com/moral/service/impl/AccountServiceImpl.java | 37 ++++++++++++++++++++++++------------- 1 files changed, 24 insertions(+), 13 deletions(-) diff --git a/src/main/java/com/moral/service/impl/AccountServiceImpl.java b/src/main/java/com/moral/service/impl/AccountServiceImpl.java index 83484e0..1fbcfa1 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; @@ -38,7 +34,8 @@ @Service public class AccountServiceImpl implements AccountService { - + @Resource + private BCryptPasswordEncoder encoder; @Resource private AccountMapper accountMapper; @@ -47,15 +44,17 @@ @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())) { @@ -75,7 +74,10 @@ 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"); @@ -141,10 +143,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); @@ -168,4 +174,9 @@ return accountMapper.selectCount(account); } + @Override + public Optional<Account> queryAccountByName(String accountName) { + Account account = accountMapper.getByAccountName(accountName); + return Optional.ofNullable(account); + } } -- Gitblit v1.8.0