Skip to content

Commit 771cc13

Browse files
committed
Define default instructions for empty additional instructions of available locales.
Issue: documentacao-e-tarefas/scielo#843 Signed-off-by: Thiago Brasil <thiago@lepidus.com.br>
1 parent 65d6e75 commit 771cc13

1 file changed

Lines changed: 17 additions & 0 deletions

File tree

settings/DataverseSettingsForm.inc.php

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
import('lib.pkp.classes.form.Form');
44
import('plugins.generic.dataverse.classes.dataverseConfiguration.DataverseConfigurationDAO');
55
import('plugins.generic.dataverse.dataverseAPI.actions.DataverseCollectionActions');
6+
import('plugins.generic.dataverse.settings.DefaultAdditionalInstructions');
67

78
class DataverseSettingsForm extends Form
89
{
@@ -90,6 +91,7 @@ public function fetch($request, $template = null, $display = false)
9091

9192
public function execute(...$functionArgs)
9293
{
94+
$this->setDefaultAdditionalInstructions();
9395
foreach (self::CONFIG_VARS as $configVar => $type) {
9496
$this->plugin->updateSetting($this->contextId, $configVar, $this->getData($configVar), $type);
9597
}
@@ -112,4 +114,19 @@ public function validateConfiguration(): bool
112114

113115
return true;
114116
}
117+
118+
private function setDefaultAdditionalInstructions(): void
119+
{
120+
$defaultAdditionalInstructions = new DefaultAdditionalInstructions();
121+
$defaultInstructions = $defaultAdditionalInstructions->getDefaultInstructions();
122+
123+
$additionalInstructions = $this->getData('additionalInstructions');
124+
foreach ($defaultInstructions as $locale => $instructions) {
125+
if (empty($additionalInstructions[$locale])) {
126+
$additionalInstructions[$locale] = $instructions;
127+
}
128+
}
129+
130+
$this->setData('additionalInstructions', $additionalInstructions);
131+
}
115132
}

0 commit comments

Comments
 (0)