ensure outerquery ->where() function doesn't clobber earier ->andWhere()

nextcloud · Aug 7, 2023 · 87bc180 · 87bc180
1 parent 1e9082b
commit 87bc180
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/apps/dav/lib/CalDAV/CalDavBackend.php b/apps/dav/lib/CalDAV/CalDavBackend.php
@@ -1845,6 +1845,10 @@ public function search(array $calendarInfo, $pattern, array $searchProperties,
 			->andWhere($innerQuery->expr()->eq('op.calendartype',
 				$outerQuery->createNamedParameter(self::CALENDAR_TYPE_CALENDAR)));
 
+		$outerQuery->select('c.id', 'c.calendardata', 'c.componenttype', 'c.uid', 'c.uri')
+			->from('calendarobjects', 'c')
+			->where($outerQuery->expr()->isNull('deleted_at'));
+
 		// only return public items for shared calendars for now
 		if (isset($calendarInfo['{http://owncloud.org/ns}owner-principal']) === false || $calendarInfo['principaluri'] !== $calendarInfo['{http://owncloud.org/ns}owner-principal']) {
 			$outerQuery->andWhere($outerQuery->expr()->eq('c.classification',
@@ -1866,10 +1870,6 @@ public function search(array $calendarInfo, $pattern, array $searchProperties,
 					$this->db->escapeLikeParameter($pattern) . '%')));
 		}
 
-		$outerQuery->select('c.id', 'c.calendardata', 'c.componenttype', 'c.uid', 'c.uri')
-			->from('calendarobjects', 'c')
-			->where($outerQuery->expr()->isNull('deleted_at'));
-
 		if (isset($options['timerange'])) {
 			if (isset($options['timerange']['start']) && $options['timerange']['start'] instanceof DateTimeInterface) {
 				$outerQuery->andWhere($outerQuery->expr()->gt('lastoccurence',