Refactor Schedule.AddNewAppointment to abide by the CQS principle

teopenna · Sep 27, 2022 · 0adbac0 · 0adbac0
1 parent 39f48a1
commit 0adbac0
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/FrontDesk/src/FrontDesk.Core/ScheduleAggregate/Schedule.cs b/FrontDesk/src/FrontDesk.Core/ScheduleAggregate/Schedule.cs
@@ -31,7 +31,7 @@ private Schedule(Guid id, int clinicId) // used by EF
 
     public DateTimeOffsetRange DateRange { get; private set; }
 
-    public Appointment AddNewAppointment(Appointment appointment)
+    public void AddNewAppointment(Appointment appointment)
     {
       Guard.Against.Null(appointment, nameof(appointment));
       Guard.Against.Default(appointment.Id, nameof(appointment.Id));
@@ -43,8 +43,6 @@ public Appointment AddNewAppointment(Appointment appointment)
 
       var appointmentScheduledEvent = new AppointmentScheduledEvent(appointment);
       Events.Add(appointmentScheduledEvent);
-
-      return appointment;
     }
 
     public void DeleteAppointment(Appointment appointment)