Repo::getResourceByIdsAsync(): remove technical search metadata befor…

…e passing the meta to the resource
acdh-oeaw · Oct 1, 2024 · b4d8d49 · b4d8d49
1 parent 19ea4aa
commit b4d8d49
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 1 deletion.
diff --git a/src/acdhOeaw/arche/lib/Repo.php b/src/acdhOeaw/arche/lib/Repo.php
@@ -46,6 +46,7 @@
 use quickRdfIo\NQuadsSerializer;
 use quickRdfIo\Util as RdfIoUtil;
 use termTemplates\QuadTemplate as QT;
+use termTemplates\PredicateTemplate as PT;
 use acdhOeaw\arche\lib\exception\Conflict;
 use acdhOeaw\arche\lib\exception\Deleted;
 use acdhOeaw\arche\lib\exception\NotFound;
@@ -489,7 +490,10 @@ public function getResourceByIdsAsync(array $ids, ?string $class = null): RepoRe
                     return new RejectedPromise(new NotFound());
                 case 1;
                     $class = $class ?? self::$resourceClass;
-                    $res = new $class($matches[0], $this);
+                    $res   = new $class($matches[0], $this);
+                    $graph->delete(new PT($this->schema->searchMatch));
+                    $graph->delete(new PT($this->schema->searchOrder));
+                    $graph->delete(new PT($this->schema->searchOrderValue));
                     $res->setGraph($graph);
                     return $res;
                 default:

diff --git a/tests/RepoTest.php b/tests/RepoTest.php
@@ -131,6 +131,24 @@ public function testSearchById(): void {
 
         $res2 = self::$repo->getResourceById($id);
         $this->assertEquals($res1->getUri(), $res2->getUri());
+        $meta = [
+            'getGraph()'    => $res2->getGraph(),
+            'getMetadata()' => $res2->getMetadata(),
+        ];
+        foreach ($meta as $k => $v) {
+            $this->assertTrue($v->none(new PT(self::$schema->searchMatch)), $k);
+            $this->assertTrue($v->none(new PT(self::$schema->searchOrder)), $k);
+            $this->assertTrue($v->none(new PT(self::$schema->searchOrderValue)), $k);
+            $this->assertTrue($v->none(new PT(self::$schema->searchFts)), $k);
+            $this->assertTrue($v->none(new PT(self::$schema->searchFtsProperty)), $k);
+            $this->assertTrue($v->none(new PT(self::$schema->searchFtsQuery)), $k);
+            $this->assertTrue($v->none(new PT(self::$schema->searchMatch)), $k);
+            $this->assertTrue($v->none(new PT(self::$schema->searchOrder)), $k);
+            $this->assertTrue($v->none(new PT(self::$schema->searchOrderValue)), $k);
+            $this->assertTrue($v->none(new PT(self::$schema->searchFts)), $k);
+            $this->assertTrue($v->none(new PT(self::$schema->searchFtsProperty)), $k);
+            $this->assertTrue($v->none(new PT(self::$schema->searchFtsQuery)), $k);
+        }
     }
 
     public function testSearchByIdNotFound(): void {