#25 immediatelly set _exportPlugin during plugin initialization so th…

…at it is available in a job
pkp · Apr 8, 2024 · 4677601 · 4677601
1 parent 8110f5e
commit 4677601
Showing 1 changed file with 10 additions and 31 deletions.
diff --git a/MedraPlugin.php b/MedraPlugin.php
@@ -106,11 +106,10 @@ public function setEnabled($enabled)
     public function exportSubmissions(array $submissions, Context $context): array
     {
         // Get filter and set objectsFileNamePart (see: PubObjectsExportPlugin::prepareAndExportPubObjects)
-        $exportPlugin = $this->_getExportPlugin();
-        $filterName = $exportPlugin->getSubmissionFilter();
+        $filterName = $this->_exportPlugin->getSubmissionFilter();
         $xmlErrors = [];
 
-        $temporaryFileId = $exportPlugin->exportAsDownload($context, $submissions, $filterName, 'articles', null, $xmlErrors);
+        $temporaryFileId = $this->_exportPlugin->exportAsDownload($context, $submissions, $filterName, 'articles', null, $xmlErrors);
         return ['temporaryFileId' => $temporaryFileId, 'xmlErrors' => $xmlErrors];
     }
 
@@ -120,10 +119,9 @@ public function exportSubmissions(array $submissions, Context $context): array
      */
     public function depositSubmissions(array $submissions, Context $context): array
     {
-        $exportPlugin = $this->_getExportPlugin();
-        $filterName = $exportPlugin->getSubmissionFilter();
+        $filterName = $this->_exportPlugin->getSubmissionFilter();
         $responseMessage = '';
-        $status = $exportPlugin->exportAndDeposit($context, $submissions, $filterName, $responseMessage);
+        $status = $this->_exportPlugin->exportAndDeposit($context, $submissions, $filterName, $responseMessage);
 
         return [
             'hasErrors' => !$status,
@@ -138,11 +136,10 @@ public function depositSubmissions(array $submissions, Context $context): array
     public function exportIssues(array $issues, Context $context): array
     {
         // Get filter and set objectsFileNamePart (see: PubObjectsExportPlugin::prepareAndExportPubObjects)
-        $exportPlugin = $this->_getExportPlugin();
-        $filterName = $exportPlugin->getIssueFilter();
+        $filterName = $this->_exportPlugin->getIssueFilter();
         $xmlErrors = [];
 
-        $temporaryFileId = $exportPlugin->exportAsDownload($context, $issues, $filterName, 'issues', null, $xmlErrors);
+        $temporaryFileId = $this->_exportPlugin->exportAsDownload($context, $issues, $filterName, 'issues', null, $xmlErrors);
         return ['temporaryFileId' => $temporaryFileId, 'xmlErrors' => $xmlErrors];
     }
 
@@ -152,10 +149,9 @@ public function exportIssues(array $issues, Context $context): array
      */
     public function depositIssues(array $issues, Context $context): array
     {
-        $exportPlugin = $this->_getExportPlugin();
-        $filterName = $exportPlugin->getIssueFilter();
+        $filterName = $this->_exportPlugin->getIssueFilter();
         $responseMessage = '';
-        $status = $exportPlugin->exportAndDeposit($context, $issues, $filterName, $responseMessage);
+        $status = $this->_exportPlugin->exportAndDeposit($context, $issues, $filterName, $responseMessage);
 
         return [
             'hasErrors' => !$status,
@@ -222,7 +218,7 @@ public function addAsRegistrationAgencyOption(string $hookName, array $args)
     public function addRegistrationAgencyName(string $hookName, array $args): bool
     {
         $config = &$args[0];
-        $config['registrationAgencyNames'][$this->_getExportPlugin()->getName()] = $this->getRegistrationAgencyName();
+        $config['registrationAgencyNames'][$this->_exportPlugin->getName()] = $this->getRegistrationAgencyName();
         return HOOK::CONTINUE;
     }
 
@@ -352,6 +348,7 @@ public function getFailedMsgSettingName(): string
     private function _pluginInitialization()
     {
         PluginRegistry::register('importexport', new MedraExportPlugin($this), $this->getPluginPath());
+        $this->_exportPlugin = PluginRegistry::getPlugin('importexport', 'MedraExportPlugin');
 
         Hook::add('DoiSettingsForm::setEnabledRegistrationAgencies', [$this, 'addAsRegistrationAgencyOption']);
         Hook::add('DoiSetupSettingsForm::getObjectTypes', [$this, 'addAllowedObjectTypes']);
@@ -360,24 +357,6 @@ private function _pluginInitialization()
         Hook::add('Doi::markRegistered', [$this, 'editMarkRegisteredParams']);
     }
 
-    /**
-     * Get mEDRA export plugin
-     */
-    private function _getExportPlugin(): MedraExportPlugin
-    {
-        if (empty($this->_exportPlugin)) {
-            $pluginCategory = 'importexport';
-            $pluginPathName = 'MedraExportPlugin';
-            $this->_exportPlugin = PluginRegistry::getPlugin($pluginCategory, $pluginPathName);
-            // If being run from CLI, there is no context, so plugin initialization would not have been fired
-            if ($this->_exportPlugin === null && !isset($_SERVER['SERVER_NAME'])) {
-                $this->_pluginInitialization();
-                $this->_exportPlugin = PluginRegistry::getPlugin($pluginCategory, $pluginPathName);
-            }
-        }
-        return $this->_exportPlugin;
-    }
-
     /**
      * Call the migration script before the plugin installation
      *