disable iCal default alarms
authorCornelius Weiß <mail@corneliusweiss.de>
Wed, 19 Feb 2014 15:30:12 +0000 (16:30 +0100)
committerPhilipp Schüle <p.schuele@metaways.de>
Thu, 4 Sep 2014 09:25:28 +0000 (11:25 +0200)
Change-Id: I03d620a00958f7e5280a3b0330e3af417c0e9f97
Reviewed-on: http://gerrit.tine20.com/customers/351
Tested-by: Jenkins CI (http://ci.tine20.com/)
Reviewed-by: Philipp Schüle <p.schuele@metaways.de>
tests/tine20/Calendar/Frontend/CalDAV/PluginDefaultAlarmsTest.php
tine20/Calendar/Frontend/CalDAV/PluginDefaultAlarms.php

index 4b0c1a2..8ad34e8 100644 (file)
@@ -12,7 +12,7 @@
 /**
  * Test helper
  */
-require_once 'vendor/sabre/dav/tests/Sabre/HTTP/ResponseMock.php';
+require_once __DIR__ . '/../../../../../tine20/vendor/sabre/dav/tests/Sabre/HTTP/ResponseMock.php';
 
 /**
  * Test class for Tinebase_WebDav_Plugin_OwnCloud
index ade5682..ecccccc 100644 (file)
@@ -7,6 +7,15 @@
  * 
  * see: http://tools.ietf.org/html/draft-daboo-valarm-extensions-04
  *
+ * NOTE: At the moment we disable all default alarms as iCal shows alarms
+ *       for events having no alarm. Acknowliging this alarms may lead to problems.
+ *       
+ * NOTE: iCal Montain Lion & Mavericks sets default alarms for the whole account, 
+ *       but respects when we set default alarms per calendar. 
+ *       
+ *       So in future we might disable default alarms for shared cals and
+ *       use the default alarms configured for each personal cal.
+ *       
  * @package    Sabre
  * @subpackage CalDAV
  * @copyright  Copyright (c) 2014-2014 Metaways Infosystems GmbH (http://www.metaways.de)
@@ -96,9 +105,11 @@ class Calendar_Frontend_CalDAV_PluginDefaultAlarms extends \Sabre\DAV\ServerPlug
                 unset($requestedProperties[array_search($property, $requestedProperties)]);
                 
                 $valarm = $vcalendar->create('VALARM');
-                $valarm->add('ACTION',  'AUDIO');
-                $valarm->add('ATTACH',  'Basso', array('VALUE' => 'URI'));
-                $valarm->add('TRIGGER', '-PT15H');
+//                 $valarm->add('ACTION',  'AUDIO');
+//                 $valarm->add('ATTACH',  'Basso', array('VALUE' => 'URI'));
+//                 $valarm->add('TRIGGER', '-PT15H');
+                $valarm->add('ACTION',  'NONE');
+                $valarm->add('TRIGGER', '19760401T005545Z', array('VALUE' => 'DATE-TIME'));
                 $valarm->add('UID',     '17DC9682-230E-47D6-A035-EEAB602B1229');
                 
                 // Taking out \r to not screw up the xml output
@@ -123,9 +134,11 @@ class Calendar_Frontend_CalDAV_PluginDefaultAlarms extends \Sabre\DAV\ServerPlug
                 unset($requestedProperties[array_search($property, $requestedProperties)]);
                 
                 $valarm = $vcalendar->create('VALARM');
-                $valarm->add('ACTION',  'AUDIO');
-                $valarm->add('ATTACH',  'Basso', array('VALUE' => 'URI'));
-                $valarm->add('TRIGGER', '-PT15H');
+//                 $valarm->add('ACTION',  'AUDIO');
+//                 $valarm->add('ATTACH',  'Basso', array('VALUE' => 'URI'));
+//                 $valarm->add('TRIGGER', '-PT15H');
+                $valarm->add('ACTION',  'NONE');
+                $valarm->add('TRIGGER', '19760401T005545Z', array('VALUE' => 'DATE-TIME'));
                 $valarm->add('UID',     '27DC9682-230E-47D6-A035-EEAB602B1229');
                 
                 // Taking out \r to not screw up the xml output