Merge pull request ardalis#47 from snowfrogdev/main

Refactor Schedule.AddNewAppointment to abide by the CQS principle
teopenna · Sep 27, 2022 · 4c9ae69 · 4c9ae69
2 parents 39f48a1 + 0adbac0
commit 4c9ae69
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)