Merge pull request #2504 from oat-sa/fix/AUT-3814/properties-without-…

…range Prevent Metadata exporter gather list from property without range
oat-sa · Sep 23, 2024 · 44abc70 · 44abc70
2 parents eb97437 + cef7e63
commit 44abc70
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/models/classes/metadata/metaMetadata/PropertyMapper.php b/models/classes/metadata/metaMetadata/PropertyMapper.php
@@ -64,7 +64,7 @@ public function getMetadataProperties(Property $property): array
 
     private function isIgnoredForCollectionGathering(Property $property): bool
     {
-        return in_array($property->getUri(), $this->getIgnoredProperties());
+        return in_array($property->getUri(), $this->getIgnoredProperties()) || $property->getRange() === null;
     }
 
     private function getIgnoredProperties(): array

diff --git a/test/unit/models/classes/metadata/metaMetadata/PropertyMapperTest.php b/test/unit/models/classes/metadata/metaMetadata/PropertyMapperTest.php
@@ -50,6 +50,7 @@ public function testGetMetadataProperties(): void
         $property = $this->createMock(Property::class);
         $resourceMock = $this->createMock(Resource::class);
         $property->method('getUri')->willReturn('uri');
+        $property->method('getRange')->willReturn('range');
         $resourceMock->method('getUri')->willReturn('resource_uri');
 
         $property