Browse Source

Merge pull request #2898 from 597365581/patch-6

merge method updateuser if and fix bug#2892
pull/3/MERGE
gabry.wu 4 years ago committed by gaojun2048
parent
commit
5222483471
  1. 10
      dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/UsersService.java

10
dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/UsersService.java

@ -301,13 +301,11 @@ public class UsersService extends BaseService {
user.setEmail(email);
}
if (StringUtils.isNotEmpty(phone)) {
if (!CheckUtils.checkPhone(phone)){
putMsg(result, Status.REQUEST_PARAMS_NOT_VALID_ERROR,phone);
return result;
}
user.setPhone(phone);
if (StringUtils.isNotEmpty(phone) && !CheckUtils.checkPhone(phone)) {
putMsg(result, Status.REQUEST_PARAMS_NOT_VALID_ERROR,phone);
return result;
}
user.setPhone(phone);
user.setQueue(queue);
Date now = new Date();
user.setUpdateTime(now);

Loading…
Cancel
Save