pkp/pkp-lib#7595 allow valid special characters in journal acronym us…

…ed in DOI suffix
withanage · Sep 2, 2022 · a26111a · a26111a
1 parent d317b84
commit a26111a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/classes/plugins/PubIdPlugin.inc.php b/classes/plugins/PubIdPlugin.inc.php
@@ -181,7 +181,7 @@ function getPubId($pubObject) {
 				$pubIdSuffix = $this->getSetting($contextId, $suffixPatternsFieldNames[$pubObjectType]);
 
 				// %j - journal initials, remove special characters and uncapitalize
-				$pubIdSuffix = PKPString::regexp_replace('/%j/', PKPString::regexp_replace('/[^A-Za-z0-9]/', '', PKPString::strtolower($context->getAcronym($context->getPrimaryLocale()))), $pubIdSuffix);
+				$pubIdSuffix = PKPString::regexp_replace('/%j/', PKPString::regexp_replace('/[^-._;()\/A-Za-z0-9]/', '', PKPString::strtolower($context->getAcronym($context->getPrimaryLocale()))), $pubIdSuffix);
 
 				// %x - custom identifier
 				if ($pubObject->getStoredPubId('publisher-id')) {
@@ -219,7 +219,7 @@ function getPubId($pubObject) {
 				break;
 
 			default:
-				$pubIdSuffix = PKPString::regexp_replace('/[^A-Za-z0-9]/', '', PKPString::strtolower($context->getAcronym($context->getPrimaryLocale())));
+				$pubIdSuffix = PKPString::regexp_replace('/[^-._;()\/A-Za-z0-9]/', '', PKPString::strtolower($context->getAcronym($context->getPrimaryLocale())));
 
 				if ($issue) {
 					$pubIdSuffix .= '.v' . $issue->getVolume() . 'i' . $issue->getNumber();

diff --git a/plugins/pubIds/doi/DOIPubIdPlugin.inc.php b/plugins/pubIds/doi/DOIPubIdPlugin.inc.php
@@ -405,7 +405,7 @@ public function addPublicationFormFields($hookName, $form) {
 				'value' => $form->publication->getData('pub-id::doi'),
 				'prefix' => $prefix,
 				'pattern' => $pattern,
-				'contextInitials' => PKPString::regexp_replace('/[^A-Za-z0-9]/', '', PKPString::strtolower($form->submissionContext->getData('acronym', $form->submissionContext->getData('primaryLocale')))) ?? '',
+				'contextInitials' => PKPString::regexp_replace('/[^-._;()\/A-Za-z0-9]/', '', PKPString::strtolower($form->submissionContext->getData('acronym', $form->submissionContext->getData('primaryLocale')))) ?? '',
 				'separator' => '/',
 				'submissionId' => $form->publication->getData('submissionId'),
 				'assignIdLabel' => __('plugins.pubIds.doi.editor.doi.assignDoi'),