Merge branch 'add-issn-data-checking' into 'main'

Check for ISSN before using.

See merge request softwares-pkp/plugins_ojs/OASwitchboard!35

Author: YvesLepidus

classes/messages/P1Pio.php

@@ -206,16 +206,22 @@ public function getJournalData(): array
 
         $journalData = [
             'name' => (string) $journal->getLocalizedName(),
-            'id' => (string) $this->chooseIssn($journal),
-            'eissn' => (string) $journal->getData('onlineIssn'),
-            'issn' => (string) $journal->getData('printIssn')
+            'id' => (string) $this->chooseIssn($journal)
         ];
+
+        if (!empty($journal->getData('onlineIssn'))) {
+            $journalData['eissn'] = (string) $journal->getData('onlineIssn');
+        }
+        if (!empty($journal->getData('printIssn'))) {
+            $journalData['issn'] = (string) $journal->getData('printIssn');
+        }
+
         return $journalData;
     }
 
-    private function chooseIssn($journal)
+    private function chooseIssn($journal): string
     {
-        return $journal->getData('onlineIssn') ?: $journal->getData('printIssn') ?: null;
+        return $journal->getData('onlineIssn') ?: $journal->getData('printIssn') ?: "";
     }
 
     public function getContent(): array

version.xml

@@ -12,6 +12,6 @@
 <version>
 	<application>OASwitchboard</application>
 	<type>plugins.generic</type>
-	<release>2.0.1.5</release>
-	<date>2024-08-01</date>
+	<release>2.0.1.6</release>
+	<date>2024-08-12</date>
 </version>