#32 Set null values if full-text isn't selected for a locale

Vitaliy-1 · Dec 15, 2020 · 7659619 · 7659619
1 parent e11a0c3
commit 7659619
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/JatsParserPlugin.inc.php b/JatsParserPlugin.inc.php
@@ -400,6 +400,7 @@ function editPublicationFullText(string $hookname, array $args) {
 		foreach ($localePare as $localeKey => $fileId) {
 			if (empty($fileId)) {
 				$newPublication->setData('jatsParser::fullText', null, $localeKey);
+				$newPublication->setData('jatsParser::fullTextFileId', null, $localeKey);
 				continue;
 			}
 			$submissionFile = $submissionFileDao->getLatestRevision($fileId, SUBMISSION_FILE_PRODUCTION_READY);

diff --git a/classes/components/forms/PublicationJATSUploadForm.inc.php b/classes/components/forms/PublicationJATSUploadForm.inc.php
@@ -62,14 +62,19 @@ public function __construct($action, $locales, $publication, $submissionFiles, $
 			);
 		}
 
+		// Update the values so the proper option is selected on thr form initiation if full-text isn't selected for the specific locale
+		$values = $publication->getData('jatsParser::fullTextFileId');
+		$emptyValues = array_fill_keys(array_keys($options), null);
+		empty($values) ? $values = $emptyValues : $values = array_merge($emptyValues, $values);
+
 		if (!empty($options)) {
 			$this->addField(new FieldOptions('jatsParser::fullTextFileId', [
 				'label' => __('plugins.generic.jatsParser.publication.jats.label'),
 				'description' => $msg,
 				'isMultilingual' => true,
 				'type' => 'radio',
 				'options' => $options,
-				'value' => $publication->getData('jatsParser::fullTextFileId'),
+				'value' => $values,
 			]))->addField(new FieldOptions('jatsParser::pdfGalley', [
 				'label' => __('plugins.generic.jatsParser.publication.jats.pdf.label'),
 				'type' => 'checkbox',