Commit 2e4eb418 authored by suhuiguang's avatar suhuiguang

1.人员证件号码验证bug

parent e3fb26de
...@@ -347,9 +347,10 @@ public interface IOrgUsrService { ...@@ -347,9 +347,10 @@ public interface IOrgUsrService {
/** /**
* 查询身份证号码是否重复 * 查询身份证号码是否重复
* @param certificatesNumber 身份证号码 * @param certificatesNumber 身份证号码
* @param orgUserId
* @return 标识 * @return 标识
*/ */
boolean checkCertificatesNumber(String certificatesNumber); boolean checkCertificatesNumber(String certificatesNumber, Long orgUserId);
/** /**
* 根据id删除消防人员 * 根据id删除消防人员
......
...@@ -378,8 +378,9 @@ public class OrgPersonController extends BaseController { ...@@ -378,8 +378,9 @@ public class OrgPersonController extends BaseController {
@TycloudOperation(ApiLevel = UserType.AGENCY) @TycloudOperation(ApiLevel = UserType.AGENCY)
@RequestMapping(value = "/people/checkCertificates/{certificatesNumber}", method = RequestMethod.GET) @RequestMapping(value = "/people/checkCertificates/{certificatesNumber}", method = RequestMethod.GET)
@ApiOperation(httpMethod = "GET", value = "查询身份证号码是否重复", notes = "查询身份证号码是否重复") @ApiOperation(httpMethod = "GET", value = "查询身份证号码是否重复", notes = "查询身份证号码是否重复")
public ResponseModel<Boolean> checkCertificatesNumber(@PathVariable String certificatesNumber) { public ResponseModel<Boolean> checkCertificatesNumber(@PathVariable String certificatesNumber,
return ResponseHelper.buildResponse(iOrgUsrService.checkCertificatesNumber(certificatesNumber)); @RequestParam(required = false) Long orgUserId) {
return ResponseHelper.buildResponse(iOrgUsrService.checkCertificatesNumber(certificatesNumber, orgUserId));
} }
/** /**
......
...@@ -3,7 +3,6 @@ package com.yeejoin.amos.boot.module.common.biz.service.impl; ...@@ -3,7 +3,6 @@ package com.yeejoin.amos.boot.module.common.biz.service.impl;
import java.io.Serializable; import java.io.Serializable;
import java.lang.reflect.Field; import java.lang.reflect.Field;
import java.lang.reflect.Method; import java.lang.reflect.Method;
import java.lang.reflect.Type;
import java.text.SimpleDateFormat; import java.text.SimpleDateFormat;
import java.util.*; import java.util.*;
import java.util.stream.Collectors; import java.util.stream.Collectors;
...@@ -18,10 +17,8 @@ import com.yeejoin.amos.boot.biz.common.utils.RedisUtils; ...@@ -18,10 +17,8 @@ import com.yeejoin.amos.boot.biz.common.utils.RedisUtils;
import com.yeejoin.amos.boot.module.common.api.dto.*; import com.yeejoin.amos.boot.module.common.api.dto.*;
import com.yeejoin.amos.boot.module.common.api.entity.*; import com.yeejoin.amos.boot.module.common.api.entity.*;
import com.yeejoin.amos.boot.module.common.api.mapper.*; import com.yeejoin.amos.boot.module.common.api.mapper.*;
import com.yeejoin.amos.boot.module.common.api.service.IFireCompanyInfoService;
import org.apache.commons.collections4.CollectionUtils; import org.apache.commons.collections4.CollectionUtils;
import org.apache.commons.lang3.StringUtils; import org.apache.commons.lang3.StringUtils;
import org.checkerframework.checker.units.qual.C;
import org.slf4j.Logger; import org.slf4j.Logger;
import org.slf4j.LoggerFactory; import org.slf4j.LoggerFactory;
import org.springframework.beans.BeanUtils; import org.springframework.beans.BeanUtils;
...@@ -3133,12 +3130,13 @@ public class OrgUsrServiceImpl extends BaseService<OrgUsrDto, OrgUsr, OrgUsrMapp ...@@ -3133,12 +3130,13 @@ public class OrgUsrServiceImpl extends BaseService<OrgUsrDto, OrgUsr, OrgUsrMapp
} }
@Override @Override
public boolean checkCertificatesNumber(String certificatesNumber) { public boolean checkCertificatesNumber(String certificatesNumber, Long orgUserId) {
QueryWrapper<DynamicFormInstance> queryWrapper = new QueryWrapper<>(); LambdaQueryWrapper<DynamicFormInstance> queryWrapper = new LambdaQueryWrapper<>();
queryWrapper.eq("field_code", "certificatesNumber"); queryWrapper.eq(DynamicFormInstance::getFieldCode, "certificatesNumber");
queryWrapper.eq("field_value", certificatesNumber); queryWrapper.eq(DynamicFormInstance::getFieldValue, certificatesNumber);
queryWrapper.ne(orgUserId !=null, DynamicFormInstance::getInstanceId, orgUserId);
List<DynamicFormInstance> list = dynamicFormInstanceService.list(queryWrapper); List<DynamicFormInstance> list = dynamicFormInstanceService.list(queryWrapper);
return !ObjectUtils.isEmpty(list) && list.size() > 1; return !list.isEmpty();
} }
@Override @Override
......
...@@ -780,7 +780,7 @@ public class ExcelServiceImpl { ...@@ -780,7 +780,7 @@ public class ExcelServiceImpl {
if (!set.add(orgUsrExcelDto.getCertificatesNumber())) { if (!set.add(orgUsrExcelDto.getCertificatesNumber())) {
throw new RuntimeException("身份证号码重复!"); throw new RuntimeException("身份证号码重复!");
} else { } else {
boolean b = orgUsrService.checkCertificatesNumber(orgUsrExcelDto.getCertificatesNumber()); boolean b = orgUsrService.checkCertificatesNumber(orgUsrExcelDto.getCertificatesNumber(), null);
if (b) { if (b) {
throw new RuntimeException("身份证号码重复!"); throw new RuntimeException("身份证号码重复!");
} }
...@@ -1385,7 +1385,7 @@ public class ExcelServiceImpl { ...@@ -1385,7 +1385,7 @@ public class ExcelServiceImpl {
fireExpertsServiceImpl.saveBatch(excelEntityList); fireExpertsServiceImpl.saveBatch(excelEntityList);
} }
@Async @Async
private void excelImportDutyPerson(MultipartFile multipartFile, String dutyType,String uuidString) throws Exception { private void excelImportDutyPerson(MultipartFile multipartFile, String dutyType,String uuidString) {
try { try {
ImportDutyPerson(multipartFile,dutyType); ImportDutyPerson(multipartFile,dutyType);
if(redisUtils.hasKey(uuidString)){ if(redisUtils.hasKey(uuidString)){
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment