fix: Metadata field search

Signed-off-by: Louis Chemineau <louis@chmn.me>

[skip ci]

Author: artonge

lib/private/Files/Cache/SearchBuilder.php

@@ -33,6 +33,7 @@
 use OCP\Files\Search\ISearchComparison;
 use OCP\Files\Search\ISearchOperator;
 use OCP\Files\Search\ISearchOrder;
+use OCP\FilesMetadata\IFilesMetadataManager;
 use OCP\FilesMetadata\IMetadataQuery;
 
 /**
@@ -101,13 +102,9 @@ class SearchBuilder {
 
 	public const TAG_FAVORITE = '_$!<Favorite>!$_';
 
-	/** @var IMimeTypeLoader */
-	private $mimetypeLoader;
-
 	public function __construct(
 		IMimeTypeLoader $mimetypeLoader
 	) {
-		$this->mimetypeLoader = $mimetypeLoader;
 	}
 
 	/**
@@ -306,7 +303,6 @@ private function validateComparison(ISearchComparison $operator) {
 
 
 	private function getExtraOperatorField(ISearchComparison $operator, IMetadataQuery $metadataQuery): array {
-		$paramType = self::$fieldTypes[$operator->getField()];
 		$field = $operator->getField();
 		$value = $operator->getValue();
 		$type = $operator->getType();

tests/lib/Files/Cache/SearchBuilderTest.php

@@ -30,6 +30,7 @@
 use OCP\Files\Search\ISearchBinaryOperator;
 use OCP\Files\Search\ISearchComparison;
 use OCP\Files\Search\ISearchOperator;
+use OCP\FilesMetadata\IFilesMetadataManager;
 use Test\TestCase;
 
 /**
@@ -39,9 +40,12 @@ class SearchBuilderTest extends TestCase {
 	/** @var IQueryBuilder */
 	private $builder;
 
-	/** @var IMimeTypeLoader|\PHPUnit\Framework\MockObject\MockObject */
+	/** @var IMimeTypeLoader&\PHPUnit\Framework\MockObject\MockObject */
 	private $mimetypeLoader;
 
+	/** @var IFilesMetadataManager&\PHPUnit\Framework\MockObject\MockObject */
+	private $filesMetadataManager;
+
 	/** @var SearchBuilder */
 	private $searchBuilder;
 
@@ -52,6 +56,7 @@ protected function setUp(): void {
 		parent::setUp();
 		$this->builder = \OC::$server->getDatabaseConnection()->getQueryBuilder();
 		$this->mimetypeLoader = $this->createMock(IMimeTypeLoader::class);
+		$this->filesMetadataManager = $this->createMock(IFilesMetadataManager::class);
 
 		$this->mimetypeLoader->expects($this->any())
 			->method('getId')
@@ -75,7 +80,7 @@ protected function setUp(): void {
 				[6, 'image']
 			]);
 
-		$this->searchBuilder = new SearchBuilder($this->mimetypeLoader);
+		$this->searchBuilder = new SearchBuilder($this->mimetypeLoader, $this->filesMetadataManager);
 		$this->numericStorageId = 10000;
 
 		$this->builder->select(['fileid'])