Merge pull request #39 from lepidus/main

Feature/format ISBN with hyphens (OMP 3.4.0)

Author: thiagolepidus

classes/factories/ThothPublicationFactory.inc.php

@@ -72,7 +72,13 @@ private function getIsbnByPublicationFormat($publicationFormat)
         $identificationCodes = $publicationFormat->getIdentificationCodes()->toArray();
         foreach ($identificationCodes as $identificationCode) {
             if ($identificationCode->getCode() == '15' || $identificationCode->getCode() == '24') {
-                return $identificationCode->getValue();
+                $isbn = $identificationCode->getValue();
+                try {
+                    $isbn13 = \Biblys\Isbn\Isbn::convertToIsbn13($isbn);
+                    return str_replace('-', '', $isbn13) === $isbn ? $isbn13 : $isbn;
+                } catch (Exception $e) {
+                    return $isbn;
+                }
             }
         }
 

classes/services/ThothBookService.inc.php

@@ -90,15 +90,12 @@ public function validate($publication)
         }
 
         $publicationFormats = DAORegistry::getDAO('PublicationFormatDAO')
-            ->getApprovedByPublicationId($publication->getId())
-            ->toArray();
+            ->getByPublicationId($publication->getId());
         foreach ($publicationFormats as $publicationFormat) {
-            if ($publicationFormat->getIsAvailable()) {
-                $errors = array_merge(
-                    $errors,
-                    ThothService::publication()->validate($publicationFormat)
-                );
-            }
+            $errors = array_merge(
+                $errors,
+                ThothService::publication()->validate($publicationFormat)
+            );
         }
 
         return $errors;

composer.lock

@@ -85,7 +85,7 @@ msgid "plugins.generic.thoth.register.warning"
 msgstr "The submission metadata can not be submitted to Thoth because of the following errors:"
 
 msgid "plugins.generic.thoth.validation.isbn"
-msgstr "\"{$isbn}\" of \"{$formatName}\" publication format is not a valid ISBN-13. A valid ISBN-13 must be exactly 17 characters (numbers and hyphens)."
+msgstr "\"{$isbn}\" of \"{$formatName}\" publication format is not a valid ISBN-13."
 
 msgid "plugins.generic.thoth.validation.doiExists"
 msgstr "A work with DOI \"{$doi}\" already exists."

locale/en/locale.po

@@ -85,7 +85,7 @@ msgid "plugins.generic.thoth.register.warning"
 msgstr "Los metadatos de envio no pueden ser enviados a Thoth debido a los siguientes errores:"
 
 msgid "plugins.generic.thoth.validation.isbn"
-msgstr "\"{$isbn}\" del formato de publicación \"{$formatName}\" no es un ISBN-13 válido. Un ISBN-13 válido debe tener exactamente 17 caracteres (números y guiones)."
+msgstr "\"{$isbn}\" del formato de publicación \"{$formatName}\" no es un ISBN-13 válido."
 
 msgid "plugins.generic.thoth.validation.doiExists"
 msgstr "Un trabajo con el DOI \"{$doi}\" ya existe."

locale/es/locale.po

@@ -85,7 +85,7 @@ msgid "plugins.generic.thoth.register.warning"
 msgstr "Os metadados da submissão não podem ser enviados para Thoth devido aos seguintes erros:"
 
 msgid "plugins.generic.thoth.validation.isbn"
-msgstr "\"{$isbn}\" do formato de publicação \"{$formatName}\" não é um ISBN-13 válido. Um ISBN-13 válido deve ter exatamente 17 caracteres (números e hífens)."
+msgstr "\"{$isbn}\" do formato de publicação \"{$formatName}\" não é um ISBN-13 válido."
 
 msgid "plugins.generic.thoth.validation.doiExists"
 msgstr "Um trabalho com o DOI \"{$doi}\" já existe."

locale/pt_BR/locale.po

@@ -3,8 +3,8 @@
 <version>
 	<application>thoth</application>
 	<type>plugins.generic</type>
-	<release>0.2.8.0</release>
-	<date>2025-05-01</date>
+	<release>0.2.8.1</release>
+	<date>2025-05-16</date>
 	<lazy-load>1</lazy-load>
 	<class>ThothPlugin</class>
 </version>