T&A: Export/Import Refactoring

components/ILIAS/Export/classes/ExportHandler/Info/Export/Component/Handler.php

@@ -58,7 +58,7 @@ protected function init(): void
                 throw new ilExportException('Export class file "' . $export_class_file . '" not found.');
             }
         }
-        $this->sv = $this->getMinimalComponentExporter()->determineSchemaVersion($component, $this->getTarget()->getTargetRelease());
+        $this->sv = $this->getMinimalComponentExporter()->determineSchemaVersion($this->getTarget()->getType(), $this->getTarget()->getTargetRelease());
         $this->sv["uses_dataset"] ??= false;
         $this->sv['xsd_file'] ??= '';
     }

components/ILIAS/Filesystem/src/Stream/ReattachableStream.php

@@ -25,15 +25,23 @@
  */
 class ReattachableStream extends Stream
 {
-    /**
-     * Checks if the stream is attached to the wrapper.
-     * If not, the stream is reattached.
-     */
+    private ?string $reattach_uri = null;
+    private ?string $reattach_mode = null;
+
+    #[\Override]
+    public function detach()
+    {
+        $this->reattach_uri = $this->uri;
+        $this->reattach_mode = $this->_mode;
+
+        return parent::detach();
+    }
+
     #[\Override]
     protected function assertStreamAttached(): void
     {
         if ($this->stream === null) {
-            $this->stream = fopen($this->uri, $this->_mode);
+            $this->stream = fopen($this->reattach_uri, $this->reattach_mode);
         }
     }
 }

components/ILIAS/Test/classes/class.ilObjTest.php

@@ -407,36 +407,6 @@ public function getExportFiles(string $dir = ''): array
         return $files;
     }
 
-    /**
-    * creates data directory for import files
-    * (data_dir/tst_data/tst_<id>/import, depending on data
-    * directory that is set in ILIAS setup/ini)
-    */
-    public static function _createImportDirectory(): string
-    {
-        global $DIC;
-        $ilias = $DIC['ilias'];
-        $tst_data_dir = ilFileUtils::getDataDir() . "/tst_data";
-        ilFileUtils::makeDir($tst_data_dir);
-
-        if (!is_writable($tst_data_dir)) {
-            $ilias->raiseError("Test Data Directory (" . $tst_data_dir
-                . ") not writeable.", $ilias->error_obj->FATAL);
-        }
-
-        // create test directory (data_dir/tst_data/tst_import)
-        $tst_dir = $tst_data_dir . "/tst_import";
-        ilFileUtils::makeDir($tst_dir);
-        if (!@is_dir($tst_dir)) {
-            $ilias->raiseError("Creation of test import directory failed.", $ilias->error_obj->FATAL);
-        }
-
-        // assert that this is empty and does not contain old data
-        ilFileUtils::delDir($tst_dir, true);
-
-        return $tst_dir;
-    }
-
     final public function isComplete(ilTestQuestionSetConfig $test_question_set_config): bool
     {
         if ($this->getMarkSchema() === null
@@ -2798,8 +2768,7 @@ public function getAvailableQuestions($arr_filter, $completeonly = 0): array
     }
 
     /**
-     * Receives parameters from a QTI parser and creates a valid ILIAS test object
-     * @param ilQTIAssessment $assessment
+     * @deprecated 12: Use ILIAS\Test\ExportImport\TestImporter instead
      */
     public function fromXML(ilQTIAssessment $assessment, array $mappings): void
     {