Skip to content

Commit 96f2148

Browse files
committed
CLAP-119 Fix: 요청 객체로 받을 시 mainCategory는 받지 않도록 제거
1 parent 5f0a795 commit 96f2148

File tree

6 files changed

+8
-14
lines changed

6 files changed

+8
-14
lines changed

src/main/java/clap/server/adapter/inbound/web/dto/task/ApprovalTaskRequest.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,6 @@
55
import java.time.LocalDateTime;
66

77
public record ApprovalTaskRequest(
8-
@NotNull
9-
Long taskId,
10-
@NotNull
11-
Long mainCategoryId,
128
@NotNull
139
Long categoryId,
1410
@NotNull

src/main/java/clap/server/adapter/inbound/web/task/ManagementTaskController.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,10 +45,11 @@ public ResponseEntity<UpdateTaskResponse> updateTask(
4545

4646
@Operation(summary = "작업 승인")
4747
@Secured({"ROLE_MANAGER"})
48-
@PostMapping("/approval")
48+
@PostMapping("/approval/{taskId}")
4949
public ResponseEntity<ApprovalTaskResponse> approvalTask(
5050
@RequestBody @Valid ApprovalTaskRequest approvalTaskRequest,
51+
@PathVariable Long taskId,
5152
@AuthenticationPrincipal SecurityUserDetails userInfo){
52-
return ResponseEntity.ok(approvalTaskUsecase.approvalTaskByReviewer(userInfo.getUserId(), approvalTaskRequest));
53+
return ResponseEntity.ok(approvalTaskUsecase.approvalTaskByReviewer(userInfo.getUserId(), taskId, approvalTaskRequest));
5354
}
5455
}

src/main/java/clap/server/application/Task/ApprovalTaskService.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,18 +32,17 @@ public class ApprovalTaskService implements ApprovalTaskUsecase {
3232

3333
@Override
3434
@Transactional
35-
public ApprovalTaskResponse approvalTaskByReviewer(Long reviewerId, ApprovalTaskRequest approvalTaskRequest) {
35+
public ApprovalTaskResponse approvalTaskByReviewer(Long reviewerId, Long taskId, ApprovalTaskRequest approvalTaskRequest) {
3636
Member reviewer = memberService.findActiveMember(reviewerId);
3737
if (!reviewer.isReviewer()) {
3838
throw new ApplicationException(MemberErrorCode.NOT_A_REVIEWER);
3939
}
40-
Task task = taskService.findById(approvalTaskRequest.taskId());
40+
Task task = taskService.findById(taskId);
4141
Member processor = memberService.findById(approvalTaskRequest.processorId());
42-
Category mainCategory = categoryService.findById(approvalTaskRequest.mainCategoryId());
4342
Category category = categoryService.findById(approvalTaskRequest.categoryId());
4443
Label label = labelService.findById(approvalTaskRequest.labelId());
4544

46-
task.approveTask(reviewer, processor, approvalTaskRequest.dueDate(),mainCategory, category, label);
45+
task.approveTask(reviewer, processor, approvalTaskRequest.dueDate(), category, label);
4746
return TaskMapper.toApprovalTaskResponse(commandTaskPort.save(task));
4847
}
4948
}

src/main/java/clap/server/application/port/inbound/task/ApprovalTaskUsecase.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,5 @@
44
import clap.server.adapter.inbound.web.dto.task.ApprovalTaskResponse;
55

66
public interface ApprovalTaskUsecase {
7-
ApprovalTaskResponse approvalTaskByReviewer(Long userId, ApprovalTaskRequest approvalTaskRequest);
7+
ApprovalTaskResponse approvalTaskByReviewer(Long userId, Long taskId, ApprovalTaskRequest approvalTaskRequest);
88
}

src/main/java/clap/server/domain/model/task/Label.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
package clap.server.domain.model.task;
22

3-
import clap.server.adapter.outbound.persistense.entity.task.LabelEntity;
43
import clap.server.adapter.outbound.persistense.entity.task.constant.LabelType;
54
import clap.server.domain.model.common.BaseTime;
65
import clap.server.domain.model.member.Member;

src/main/java/clap/server/domain/model/task/Task.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,11 +50,10 @@ public void updateTask(Category category, String title, String description) {
5050
this.taskCode = toTaskCode(category);
5151
}
5252

53-
public void approveTask(Member reviewer, Member processor, LocalDateTime dueDate, Category mainCategory, Category category, Label label) {
53+
public void approveTask(Member reviewer, Member processor, LocalDateTime dueDate, Category category, Label label) {
5454
this.reviewer = reviewer;
5555
this.processor = processor;
5656
this.dueDate = dueDate;
57-
this.category.updateMainCategory(mainCategory);
5857
this.category = category;
5958
this.label = label;
6059
this.taskStatus = TaskStatus.IN_PROGRESS;

0 commit comments

Comments
 (0)