Merge pull request #17184 from nextcloud/backport/17174/stable17

[stable17] properly mark birthday calendars as not shareable for now

Author: rullzer

apps/dav/lib/CalDAV/Calendar.php

@@ -363,7 +363,11 @@ public function getPublishStatus() {
 		return $this->caldavBackend->getPublishStatus($this);
 	}
 
-	private function canWrite() {
+	public function canWrite() {
+		if ($this->getName() === BirthdayService::BIRTHDAY_CALENDAR_URI) {
+			return false;
+		}
+
 		if (isset($this->calendarInfo['{http://owncloud.org/ns}read-only'])) {
 			return !$this->calendarInfo['{http://owncloud.org/ns}read-only'];
 		}

apps/dav/lib/CalDAV/Publishing/PublishPlugin.php

@@ -126,7 +126,10 @@ public function propFind(PropFind $propFind, INode $node) {
 			});
 
 			$propFind->handle('{'.self::NS_CALENDARSERVER.'}allowed-sharing-modes', function() use ($node) {
-				return new AllowedSharingModes(!$node->isSubscription(), !$node->isSubscription());
+				$canShare = (!$node->isSubscription() && $node->canWrite());
+				$canPublish = (!$node->isSubscription() && $node->canWrite());
+
+				return new AllowedSharingModes($canShare, $canPublish);
 			});
 		}
 	}