Skip to content

Commit

Permalink
fix: post 중복 validator 제거 (#771)
Browse files Browse the repository at this point in the history 
### 🚩 관련사항


### 📢 전달사항
- 게시글 생성에 기존 레거시 코드 validator를 제거했습니다.

### 📸 스크린샷

### 📃 진행사항


### ⚙️ 기타사항

개발기간:
  • Loading branch information
@sanggae4133
sanggae4133 authored Nov 27, 2024
2 parents e519864 + 4913a60 commit 5bad045
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 18 deletions.
29 changes: 15 additions & 14 deletions src/main/java/net/causw/adapter/web/LockerController.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ public class LockerController {
private final LockerService lockerService;

@GetMapping(value = "/{lockerId}")
@Operation(summary = "사물함 조회 Api(완료)", description = "사물함 id를 바탕으로 사물함 정보를 가져오는 Api 입니다.")
@Operation(summary = "사물함 조회 Api", description = "사물함 id를 바탕으로 사물함 정보를 가져오는 Api 입니다.")
@ResponseStatus(value = HttpStatus.OK)
@PreAuthorize("@securityService.isActiveAndNotNoneUserAndAcademicRecordCertified()")
public LockerResponseDto findById(
Expand All @@ -49,7 +49,7 @@ public LockerResponseDto findById(
}

@PostMapping(value = "")
@Operation(summary = "사물함 생성 Api(완료)", description = "사물함을 생성하는 Api입니다.")
@Operation(summary = "사물함 생성 Api(관리자/회장 전용)", description = "사물함을 생성하는 Api입니다.")
@ResponseStatus(value = HttpStatus.CREATED)
@PreAuthorize("@securityService.isActiveAndNotNoneUserAndAcademicRecordCertified() and " +
"hasAnyRole('ADMIN','PERSIDENT', 'VICE_PRESIDENT')")
Expand All @@ -63,7 +63,8 @@ public LockerResponseDto create(
@PutMapping(value = "/{lockerId}")
@ResponseStatus(value = HttpStatus.OK)
@PreAuthorize("@securityService.isActiveAndNotNoneUserAndAcademicRecordCertified()")
@Operation(summary = "사물함 상태 update Api", description = "사물함 상태를 변경하는 Api입니다.")
@Operation(summary = "사물함 상태 update Api", description = "사물함 상태를 변경하는 Api입니다.\n" +
"허용 동작 목록: \"ENABLE(관리자/회장 전용)\", \"DISABLE(관리자/회장 전용)\", \"REGISTER\", \"RETURN\", \"EXTEND\"")
public LockerResponseDto update(
@PathVariable("lockerId") String lockerId,
@Valid @RequestBody LockerUpdateRequestDto lockerUpdateRequestDto,
Expand All @@ -80,7 +81,7 @@ public LockerResponseDto update(
@ResponseStatus(value = HttpStatus.OK)
@PreAuthorize("@securityService.isActiveAndNotNoneUserAndAcademicRecordCertified() and " +
"hasAnyRole('ADMIN','PERSIDENT', 'VICE_PRESIDENT')")
@Operation(summary = "사물함 위치 이동 Api(완료)", description = "사물함의 위치(locker location)를 이동(변경)시키는 Api입니다. ex) 1번 사물함에 있어서 1층 1번 -> 2층 1번, 층만 바뀜")
@Operation(summary = "사물함 위치 이동 Api(관리자/회장 전용)", description = "사물함의 위치(locker location)를 이동(변경)시키는 Api입니다. ex) 1번 사물함에 있어서 1층 1번 -> 2층 1번, 층만 바뀜")
public LockerResponseDto move(
@PathVariable("lockerId") String lockerId,
@Valid @RequestBody LockerMoveRequestDto lockerMoveRequestDto,
Expand All @@ -97,7 +98,7 @@ public LockerResponseDto move(
@ResponseStatus(value = HttpStatus.OK)
@PreAuthorize("@securityService.isActiveAndNotNoneUserAndAcademicRecordCertified() and " +
"hasAnyRole('ADMIN','PERSIDENT', 'VICE_PRESIDENT')")
@Operation(summary = "사물함 삭제 Api(완료)", description = "사물함을 삭제하는 Api입니다.")
@Operation(summary = "사물함 삭제 Api(관리자/회장 전용)", description = "사물함을 삭제하는 Api입니다.")
public LockerResponseDto delete(
@PathVariable("lockerId") String lockerId,
@AuthenticationPrincipal CustomUserDetails userDetails
Expand All @@ -107,15 +108,14 @@ public LockerResponseDto delete(

@GetMapping(value = "/locations")
@ResponseStatus(value = HttpStatus.OK)
@PreAuthorize("@securityService.isActiveAndNotNoneUserAndAcademicRecordCertified() and " +
"hasAnyRole('ADMIN','PERSIDENT', 'VICE_PRESIDENT')")
@Operation(summary = "사물함 층별 사용가능 여부 조회 Api(완료)", description = "사물함 층별 개수 정보와 사용 가능 개수를 제공하는 API입니다.")
@PreAuthorize("@securityService.isActiveAndNotNoneUserAndAcademicRecordCertified()")
@Operation(summary = "사물함 층별 사용가능 여부 조회 Api", description = "사물함 층별 개수 정보와 사용 가능 개수를 제공하는 API입니다.")
public LockerLocationsResponseDto findAllLocation(@AuthenticationPrincipal CustomUserDetails userDetails) {
return this.lockerService.findAllLocation(userDetails.getUser());
}

@GetMapping(value = "/locations/{locationId}")
@Operation(summary = "사물함 특정 층별 사용가능 여부 조회 Api(완료)", description = "사물함 특정 층별 개수 정보와 사용 가능 개수를 제공하는 API입니다.")
@Operation(summary = "사물함 특정 층별 사용가능 여부 조회 Api", description = "사물함 특정 층별 개수 정보와 사용 가능 개수를 제공하는 API입니다.")
@ResponseStatus(value = HttpStatus.OK)
@PreAuthorize("@securityService.isActiveAndNotNoneUserAndAcademicRecordCertified()")
public LockersResponseDto findByLocation(
Expand All @@ -126,7 +126,7 @@ public LockersResponseDto findByLocation(
}

@PostMapping(value = "/locations")
@Operation(summary = "사물함 위치 생성 API(완료)", description = "사물함 특정 층 생성 API 입니다.")
@Operation(summary = "사물함 위치 생성 API(관리자/회장 전용)", description = "사물함 특정 층 생성 API 입니다.")
@ResponseStatus(value = HttpStatus.CREATED)
@PreAuthorize("@securityService.isActiveAndNotNoneUserAndAcademicRecordCertified() and " +
"hasAnyRole('ADMIN','PERSIDENT', 'VICE_PRESIDENT')")
Expand All @@ -138,7 +138,7 @@ public LockerLocationResponseDto createLocation(
}

@PutMapping(value = "/locations/{locationId}")
@Operation(summary = "사물함 위치 업데이트 API(완료)", description = "사물함 특정 층 업데이트 API 입니다.")
@Operation(summary = "사물함 위치 업데이트 API(관리자/회장 전용)", description = "사물함 특정 층 업데이트 API 입니다.")
@ResponseStatus(value = HttpStatus.OK)
@PreAuthorize("@securityService.isActiveAndNotNoneUserAndAcademicRecordCertified() and " +
"hasAnyRole('ADMIN','PERSIDENT', 'VICE_PRESIDENT')")
Expand All @@ -155,7 +155,7 @@ public LockerLocationResponseDto updateLocation(
}

@DeleteMapping(value = "/locations/{locationId}")
@Operation(summary = "사물함 위치 삭제 API(완료)", description = "사물함 특정 층 삭제 API 입니다.")
@Operation(summary = "사물함 위치 삭제 API(관리자/회장 전용)", description = "사물함 특정 층 삭제 API 입니다.")
@ResponseStatus(value = HttpStatus.OK)
@PreAuthorize("@securityService.isActiveAndNotNoneUserAndAcademicRecordCertified() and " +
"hasAnyRole('ADMIN','PERSIDENT', 'VICE_PRESIDENT')")
Expand All @@ -168,6 +168,7 @@ public LockerLocationResponseDto deleteLocation(

@GetMapping(value = "/{lockerId}/log")
@ResponseStatus(value = HttpStatus.OK)
@Operation(summary = "사물함 로그 조회 API(관리자/회장 전용)", description = "사물함 로그를 조회하는 API입니다.")
@PreAuthorize("@securityService.isActiveAndNotNoneUserAndAcademicRecordCertified() and " +
"hasAnyRole('ADMIN','PERSIDENT', 'VICE_PRESIDENT')")
public List<LockerLogResponseDto> findLog(
Expand All @@ -177,7 +178,7 @@ public List<LockerLogResponseDto> findLog(
}

@PostMapping(value = "/expire")
@Operation(summary = "사물함 만료 기한 설정 Api(완료)", description = "사물함 만료 기한을 설정하는 API입니다.(학생회장만 가능)")
@Operation(summary = "사물함 만료 기한 설정 Api(관리자/회장 전용)", description = "사물함 만료 기한을 설정하는 API입니다.(학생회장만 가능)")
@ResponseStatus(value = HttpStatus.OK)
@PreAuthorize("@securityService.isActiveAndNotNoneUserAndAcademicRecordCertified() and " +
"hasAnyRole('ADMIN','PERSIDENT', 'VICE_PRESIDENT')")
Expand All @@ -189,7 +190,7 @@ public void setExpireDate(
}

@PostMapping(value = "/createAll")
@Operation(summary = "사물함 전체 생성 API(관리자)" , description = "현재 존재하는 모든 사물함을 생성하는 API입니다.")
@Operation(summary = "사물함 전체 생성 API(관리자/회장 전용)" , description = "현재 존재하는 모든 사물함을 생성하는 API입니다.")
@ResponseStatus(value = HttpStatus.OK)
@PreAuthorize("@securityService.isActiveAndNotNoneUserAndAcademicRecordCertified() and " +
"hasAnyRole('ADMIN','PERSIDENT', 'VICE_PRESIDENT')")
Expand Down
5 changes: 1 addition & 4 deletions src/main/java/net/causw/application/post/PostService.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -214,9 +214,7 @@ public PostCreateResponseDto createPost(User creator, PostCreateRequestDto post

List<UuidFile> uuidFileList = (attachImageList == null || attachImageList.isEmpty()) ?
new ArrayList<>() :
attachImageList.stream()
.map(multipartFile -> uuidFileService.saveFile(multipartFile, FilePath.POST))
.toList();
uuidFileService.saveFileList(attachImageList, FilePath.POST);

Post post = Post.of(
postCreateRequestDto.getTitle(),
Expand All @@ -232,7 +230,6 @@ public PostCreateResponseDto createPost(User creator, PostCreateRequestDto post
validatorBucket
.consistOf(UserStateValidator.of(creator.getState()))
.consistOf(UserRoleIsNoneValidator.of(roles))
.consistOf(PostNumberOfAttachmentsValidator.of(attachImageList))
.consistOf(TargetIsDeletedValidator.of(board.getIsDeleted(), StaticValue.DOMAIN_BOARD))
.consistOf(UserRoleValidator.of(
roles,
Expand Down

0 comments on commit 5bad045

Please sign in to comment.