merge activity settings for calendars, events and todos

Signed-off-by: Robin Appelman <robin@icewind.nl>
nextcloud · Aug 6, 2020 · 5fed97f · 5fed97f
1 parent a4d511d
commit 5fed97f
Show file tree

Hide file tree

Showing 9 changed files with 5 additions and 297 deletions.
diff --git a/apps/dav/appinfo/info.xml b/apps/dav/appinfo/info.xml
@@ -60,13 +60,10 @@
 	<activity>
 		<settings>
 			<setting>OCA\DAV\CalDAV\Activity\Setting\Calendar</setting>
-			<setting>OCA\DAV\CalDAV\Activity\Setting\Event</setting>
-			<setting>OCA\DAV\CalDAV\Activity\Setting\Todo</setting>
 		</settings>
 
 		<filters>
 			<filter>OCA\DAV\CalDAV\Activity\Filter\Calendar</filter>
-			<filter>OCA\DAV\CalDAV\Activity\Filter\Todo</filter>
 		</filters>
 
 		<providers>

diff --git a/apps/dav/lib/CalDAV/Activity/Backend.php b/apps/dav/lib/CalDAV/Activity/Backend.php
@@ -430,7 +430,7 @@ public function onTouchCalendarObject($action, array $calendarData, array $share
 		$event = $this->activityManager->generateEvent();
 		$event->setApp('dav')
 			->setObject('calendar', (int) $calendarData['id'])
-			->setType($object['type'] === 'event' ? 'calendar_event' : 'calendar_todo')
+			->setType('calendar')
 			->setAuthor($currentUser);
 
 		$users = $this->getUsersForShares($shares);

diff --git a/apps/dav/lib/CalDAV/Activity/Filter/Calendar.php b/apps/dav/lib/CalDAV/Activity/Filter/Calendar.php
@@ -81,7 +81,7 @@ public function getIcon() {
 	 * @since 11.0.0
 	 */
 	public function filterTypes(array $types) {
-		return array_intersect(['calendar', 'calendar_event'], $types);
+		return array_intersect(['calendar', 'calendar_event', 'calendar_todo'], $types);
 	}
 
 	/**

diff --git a/apps/dav/lib/CalDAV/Activity/Filter/Todo.php b/apps/dav/lib/CalDAV/Activity/Filter/Todo.php
diff --git a/apps/dav/lib/CalDAV/Activity/Provider/Event.php b/apps/dav/lib/CalDAV/Activity/Provider/Event.php
@@ -119,7 +119,7 @@ protected function generateObjectParameter(array $eventData) {
 	 * @since 11.0.0
 	 */
 	public function parse($language, IEvent $event, IEvent $previousEvent = null) {
-		if ($event->getApp() !== 'dav' || $event->getType() !== 'calendar_event') {
+		if ($event->getApp() !== 'dav' || ($event->getType() !== 'calendar' && $event->getType() !== 'calendar_event')) {
 			throw new \InvalidArgumentException();
 		}
 

diff --git a/apps/dav/lib/CalDAV/Activity/Provider/Todo.php b/apps/dav/lib/CalDAV/Activity/Provider/Todo.php
@@ -37,7 +37,7 @@ class Todo extends Event {
 	 * @since 11.0.0
 	 */
 	public function parse($language, IEvent $event, IEvent $previousEvent = null) {
-		if ($event->getApp() !== 'dav' || $event->getType() !== 'calendar_todo') {
+		if ($event->getApp() !== 'dav' || ($event->getType() !== 'calendar' && $event->getType() !== 'calendar_todo')) {
 			throw new \InvalidArgumentException();
 		}
 

diff --git a/apps/dav/lib/CalDAV/Activity/Setting/Calendar.php b/apps/dav/lib/CalDAV/Activity/Setting/Calendar.php
@@ -51,7 +51,7 @@ public function getIdentifier() {
 	 * @since 11.0.0
 	 */
 	public function getName() {
-		return $this->l->t('A <strong>calendar</strong> was modified');
+		return $this->l->t('A <strong>calendar</strong>, <strong>event</strong> or <strong>todo</strong> was modified');
 	}
 
 	/**

diff --git a/apps/dav/lib/CalDAV/Activity/Setting/Event.php b/apps/dav/lib/CalDAV/Activity/Setting/Event.php
diff --git a/apps/dav/lib/CalDAV/Activity/Setting/Todo.php b/apps/dav/lib/CalDAV/Activity/Setting/Todo.php