hopefully improve propagation of 'path eq hash' hint into 'in' statem…

…ents Signed-off-by: Robin Appelman <robin@icewind.nl>
nextcloud · Sep 22, 2023 · 63c7d16 · 63c7d16
1 parent 4e08186
commit 63c7d16
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/lib/private/Files/Search/QueryOptimizer/OrEqualsToIn.php b/lib/private/Files/Search/QueryOptimizer/OrEqualsToIn.php
@@ -29,7 +29,10 @@ public function processOperator(ISearchOperator &$operator): bool {
 						return $value;
 					}, $group);
 					$in = new SearchComparison(ISearchComparison::COMPARE_IN, $field, $values);
-					$in->setQueryHint(ISearchComparison::HINT_PATH_EQ_HASH, $group[0]->getQueryHint(ISearchComparison::HINT_PATH_EQ_HASH, true));
+					$pathEqHash = array_reduce($group, function($pathEqHash, ISearchComparison $comparison) {
+						return $comparison->getQueryHint(ISearchComparison::HINT_PATH_EQ_HASH, true) && $pathEqHash;
+					}, true);
+					$in->setQueryHint(ISearchComparison::HINT_PATH_EQ_HASH, $pathEqHash);
 					return $in;
 				} else {
 					return $group[0];
@@ -57,8 +60,7 @@ private function groupEqualsComparisonsByField(array $operators): array {
 		$result = [];
 		foreach ($operators as $operator) {
 			if ($operator instanceof ISearchComparison && $operator->getType() === ISearchComparison::COMPARE_EQUAL) {
-				$key = $operator->getField() . $operator->getQueryHint(ISearchComparison::HINT_PATH_EQ_HASH, true);
-				$result[$key][] = $operator;
+				$result[$operator->getField()][] = $operator;
 			} else {
 				$result[] = [$operator];
 			}