Skip to content

Commit

Permalink
Merge pull request #2526 from oat-sa/fix/AUT-3784/use-only-MappedMeta…
Browse files Browse the repository at this point in the history
…dataInjector

Remove Legacy Metadata Import
  • Loading branch information
bartlomiejmarszal authored Nov 13, 2024
2 parents cf4b996 + a286c24 commit 4c534b6
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion models/classes/class.QtiTestService.php
Original file line number Diff line number Diff line change
Expand Up @@ -813,7 +813,6 @@ protected function importTest(

// 4. Import metadata for the resource (use same mechanics as item resources).
// Metadata will be set as property values.
$this->getMetadataImporter()->inject($qtiTestResource->getIdentifier(), $testResource);
//todo: fix taoSetup to be aware of containers. This is only workaround.
if (
$this->getServiceManager()->getContainer()->has(MappedMetadataInjector::class)
Expand Down
2 changes: 1 addition & 1 deletion models/classes/export/AbstractQtiTestExporter.php
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,7 @@ public function export(array $options = []): Report
$this->getMetadataExporter()->export($this->getItem(), $this->getManifest());

$this->genericLomOntologyExtractor()->extract(
array_merge([$this->getItem()], $this->getItems()),
[$this->getItem()],
$this->getManifest()
);

Expand Down

0 comments on commit 4c534b6

Please sign in to comment.