Don't throw a 500 when importing a broken ics reminder file

Signed-off-by: Joas Schilling <coding@schilljs.com>

diff --git a/apps/dav/lib/CalDAV/Reminder/ReminderService.php b/apps/dav/lib/CalDAV/Reminder/ReminderService.php
index 688b4ce6107b4d4d4925da83761e1af34cd0f48a..9c27231cb8a4c8b4d6899c90f0bbdfc23193d9b1 100644 (file)
--- a/apps/dav/lib/CalDAV/Reminder/ReminderService.php
+++ b/apps/dav/lib/CalDAV/Reminder/ReminderService.php
@@ -40,6 +40,7 @@ use OCP\IUserManager;
 use Sabre\VObject;
 use Sabre\VObject\Component\VAlarm;
 use Sabre\VObject\Component\VEvent;
+use Sabre\VObject\InvalidDataException;
 use Sabre\VObject\ParseException;
 use Sabre\VObject\Recur\EventIterator;
 use Sabre\VObject\Recur\NoInstancesException;
@@ -274,7 +275,11 @@ class ReminderService {
                                                continue;
                                        }
 
-                                       $triggerTime = $valarm->getEffectiveTriggerTime();
+                                       try {
+                                               $triggerTime = $valarm->getEffectiveTriggerTime();
+                                       } catch (InvalidDataException $e) {
+                                               continue;
+                                       }
 
                                        // If effective trigger time is in the past
                                        // just skip and generate for next event