Skip to content

Commit d58a2b1

Browse files
authored
Merge pull request #284 from CSE-Shaco/develop
fix(recruit): normalize phone numbers by removing hyphens before DB c…
2 parents 1cb5cfb + e347e75 commit d58a2b1

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

src/main/java/inha/gdgoc/domain/recruit/member/dto/request/RecruitMemberRequest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,12 +31,13 @@ public class RecruitMemberRequest {
3131
private Boolean isPayed;
3232

3333
public RecruitMember toEntity(AdmissionSemester admissionSemester) {
34+
String cleanPhone = phoneNumber.replaceAll("[^0-9]", "");
3435
return RecruitMember.builder()
3536
.name(name)
3637
.grade(grade)
3738
.studentId(studentId)
3839
.enrolledClassification(EnrolledClassification.fromStatus(enrolledClassification))
39-
.phoneNumber(phoneNumber)
40+
.phoneNumber(cleanPhone)
4041
.nationality(nationality)
4142
.email(email)
4243
.gender(Gender.fromType(gender))

src/main/java/inha/gdgoc/domain/recruit/member/service/RecruitMemberService.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,8 @@ public CheckStudentIdResponse isRegisteredStudentId(String studentId) {
6060
}
6161

6262
public CheckPhoneNumberResponse isRegisteredPhoneNumber(String phoneNumber) {
63-
boolean exists = recruitMemberRepository.existsByPhoneNumber(phoneNumber);
63+
String cleanPhone = phoneNumber.replaceAll("[^0-9]", "");
64+
boolean exists = recruitMemberRepository.existsByPhoneNumber(cleanPhone);
6465

6566
return new CheckPhoneNumberResponse(exists);
6667
}

0 commit comments

Comments
 (0)