Check for null attendees on event.

#1090
phw198 · Feb 28, 2021 · fbaada1 · fbaada1
1 parent 4a1fe3f
commit fbaada1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/OutlookGoogleCalendarSync/GoogleOgcs/GoogleCalendar.cs b/src/OutlookGoogleCalendarSync/GoogleOgcs/GoogleCalendar.cs
@@ -756,7 +756,7 @@ public Event UpdateCalendarEntry(AppointmentItem ai, Event ev, ref int itemModif
                             "more than 200, which Google does not allow.", Console.Markup.warning);
                     }
                 } else if (Settings.Instance.SyncDirection == Sync.Direction.Bidirectional &&
-                        ev.Attendees.Count > Settings.Instance.MaxAttendees && ai.Recipients.Count <= Settings.Instance.MaxAttendees) {
+                        ev.Attendees != null && ev.Attendees.Count > Settings.Instance.MaxAttendees && ai.Recipients.Count <= Settings.Instance.MaxAttendees) {
                     log.Warn("This Google event has " + ev.Attendees.Count + " attendees, more than the user configured maximum. They can't safely be compared.");
                 } else {
                     try {

diff --git a/src/OutlookGoogleCalendarSync/OutlookOgcs/OutlookCalendar.cs b/src/OutlookGoogleCalendarSync/OutlookOgcs/OutlookCalendar.cs
@@ -296,7 +296,7 @@ private void createCalendarEntry(Event ev, ref AppointmentItem ai) {
             ai.Categories = getColour(ev.ColorId, null);
 
             if (Settings.Instance.AddAttendees && ev.Attendees != null) {
-                if (ev.Attendees.Count > Settings.Instance.MaxAttendees) {
+                if (ev.Attendees != null && ev.Attendees.Count > Settings.Instance.MaxAttendees) {
                     log.Warn("This Google event has " + ev.Attendees.Count + " attendees, more than the user configured maximum.");
                 } else {
                     foreach (EventAttendee ea in ev.Attendees) {
@@ -556,10 +556,10 @@ public Boolean UpdateCalendarEntry(ref AppointmentItem ai, Event ev, ref int ite
             }
 
             if (Settings.Instance.AddAttendees) {
-                if (ev.Attendees.Count > Settings.Instance.MaxAttendees) {
+                if (ev.Attendees != null && ev.Attendees.Count > Settings.Instance.MaxAttendees) {
                     log.Warn("This Google event has " + ev.Attendees.Count + " attendees, more than the user configured maximum.");
                 } else if (Settings.Instance.SyncDirection == Sync.Direction.Bidirectional &&
-                        ai.Recipients.Count > Settings.Instance.MaxAttendees && ev.Attendees.Count <= Settings.Instance.MaxAttendees) {
+                        ai.Recipients.Count > Settings.Instance.MaxAttendees && (ev.Attendees == null ? 0 : ev.Attendees.Count) <= Settings.Instance.MaxAttendees) {
                     log.Warn("This Outlook appointment has " + ai.Recipients.Count + " attendees, more than the user configured maximum. They can't safely be compared.");
                 } else {
                     log.Fine("Comparing meeting attendees");