Avoid mail notifications on superfluous changes

Fixes #3790
2016-11-01 15:26:59 -04:00 · 2016-11-01 15:26:59 -04:00 · 651a1c3910
parent 1ee7693432
commit 651a1c3910
2 changed files with 8 additions and 6 deletions
--- a/1
+++ b/1
@ -26,6 +26,7 @@ Bug fixes
 - [web] added missing options to subscribed calendars (#3863)
 - [web] fixed resource conflict error handling (403 vs 409 HTTP code)
 - [web] restored immediate deletion of messages (without moving them to the trash)
+ - [web] avoid mail notifications on superfluous event changes (#3790)
 - [eas] improve handling of email folders without a parent
 - [eas] never send IMIP reply when the "initiator" is Outlook 2013/2016
 - [core] only consider SMTP addresses for AD's proxyAddresses (#3842)
--- a/SoObjects/Appointments/SOGoAppointmentObject.m
+++ b/SoObjects/Appointments/SOGoAppointmentObject.m
@ -890,12 +890,13 @@ inRecurrenceExceptionsForEvent: (iCalEvent *) theEvent
                            toAttendees: addedAttendees
                               withType: @"calendar:invitation"];
    }
-      
-      [self sendReceiptEmailForObject: newEvent
-                       addedAttendees: addedAttendees
-                     deletedAttendees: deletedAttendees
-                     updatedAttendees: updatedAttendees
-                            operation: EventUpdated];
+
+  if ([changes hasChanges])
+    [self sendReceiptEmailForObject: newEvent
+                     addedAttendees: addedAttendees
+                   deletedAttendees: deletedAttendees
+                   updatedAttendees: updatedAttendees
+                          operation: EventUpdated];

  return nil;
 }