Merge pull request #364 from DDD-Community/bug/363/todo

fix: 기존 루틴 관련 ToDo 삭제 로직 수정 및 ToDo 업데이트 방식 개선 (bug/363/todo)

Author: dyno-jun

app/src/main/java/com/growit/app/todo/domain/service/RoutineServiceImpl.java

@@ -205,8 +205,7 @@ private ToDoResult updateFromDate(ToDo existingToDo, UpdateToDoCommand command)
   }
 
   private ToDoResult updateAllRoutineToDos(ToDo existingToDo, UpdateToDoCommand command) {
-    deleteOtherRoutineToDos(
-        existingToDo.getRoutine().getId(), existingToDo.getId(), command.userId());
+    deleteAllRoutineToDos(existingToDo.getRoutine().getId(), command.userId());
 
     if (command.routine() != null && command.routine().isValid()) {
       CreateToDoCommand createCommand =
@@ -219,10 +218,17 @@ private ToDoResult updateAllRoutineToDos(ToDo existingToDo, UpdateToDoCommand co
               command.routine());
       return createRoutineToDos(createCommand);
     } else {
-      existingToDo.updateBy(command);
-      existingToDo.removeRoutine();
-      toDoRepository.saveToDo(existingToDo);
-      return new ToDoResult(existingToDo.getId());
+      CreateToDoCommand createCommand =
+          new CreateToDoCommand(
+              command.userId(),
+              command.goalId(),
+              command.content(),
+              command.date(),
+              command.isImportant(),
+              null);
+      ToDo newToDo = ToDo.from(createCommand);
+      toDoRepository.saveToDo(newToDo);
+      return new ToDoResult(newToDo.getId());
     }
   }