warning when settings are missing

FriendsOfREDAXO · Oct 27, 2021 · 8a184f5 · 8a184f5
1 parent 56ccbf3
commit 8a184f5
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 3 deletions.
diff --git a/lang/de_de.lang b/lang/de_de.lang
@@ -140,6 +140,7 @@ multinewsletter_config_lang_subscribe = Anmelden Button
 multinewsletter_config_lang_unsubscribe = Abmelden Button
 multinewsletter_config_lang_nogroup_selected = Kein Newsletter ausgewählt
 multinewsletter_config_submit = Einstellungen speichern
+multinewsletter_config_warning = Einstellungen noch nicht festgelegt. Diese müssen zur Verwendung des Addons zuerst festgelegt werden. Dies betrifft auch die Übersetzungen.
 
 multinewsletter_config_title_mailchimp = Mailchimp Einstellungen
 multinewsletter_expl_config_mailchimp = Es ist möglich die Benutzer mit Mailchimp zu synchronisieren. Dazu muss man zuerst auf Mailchimp die jeweiligen Listen erstellen und anschließend im Addon der Gruppe die entsprechende List zuweisen.<br>DSGVO Hinweis: der Abonnent muss darauf hingewiesen werden, dass seine Daten an Dritte (den Betreiber von Mailchimp) weitergegeben werden.

diff --git a/lang/en_gb.lang b/lang/en_gb.lang
@@ -140,6 +140,7 @@ multinewsletter_config_lang_subscribe = Subscribe button
 multinewsletter_config_lang_unsubscribe = Unsubscribe button
 multinewsletter_config_lang_nogroup_selected = No newsletter selected
 multinewsletter_config_submit = Save settings
+multinewsletter_config_warning = Settings are missing. Settings need to be saved once to use the addon. This includes translation settings.
 
 multinewsletter_config_title_mailchimp = Mailchimp settings
 multinewsletter_expl_config_mailchimp = It is possible to sync users with Mailchimp. To do this, first create the lists in Mailchimp and then assign list in group settings.<br>GDPR hint: the user must be informed, that his data is transferred to a third party (MailChimp).

diff --git a/modules/80/4/output.php b/modules/80/4/output.php
@@ -119,7 +119,7 @@ function sendActivationMail($yform) {
 			html||<p>* '. $addon->getConfig("lang_". rex_clang::getCurrentId() ."_compulsory", "") .'<br><br></p>
 			html||<p> '. $addon->getConfig("lang_". rex_clang::getCurrentId() ."_safety", "") .'<br><br></p>
 
-			submit|submit|'. ($addon->getConfig("lang_". rex_clang::getCurrentId() ."_subscribe", "") ?: 'submit') .'|no_db'. PHP_EOL;
+			submit|submit|'. $addon->getConfig("lang_". rex_clang::getCurrentId() ."_subscribe", "Send") .'|no_db'. PHP_EOL;
 	if($ask_name) {
 		$form_data .= 'validate|empty|firstname|'. $addon->getConfig("lang_". rex_clang::getCurrentId() ."_invalid_firstname", "") .'
 			validate|empty|lastname|'. $addon->getConfig("lang_". rex_clang::getCurrentId() ."_invalid_name", "") . PHP_EOL;

diff --git a/pages/help.changelog.php b/pages/help.changelog.php
@@ -8,6 +8,7 @@
 		<li>Whoops beim Zurücksetzen des Versands unter MySQL 8.</li>
 		<li>Bugfix: Versandeinstellungen für eigenen SMTP Server wurden bei Aktivierungsmail ignoriert.</li>
 		<li>Der Versand für fehlgeschlagene Empfänger kann aus dem Archiv heraus wiederholt werden.</li>
+		<li>Warnmeldung eingebaut, wenn wichtige Eintellungen noch nicht festegelegt sind, die für die Verwendung des Addons Voraussetzung sind.</li>
 		<li>Modul 80-1, 80-2 und 80-3 mit Eingabefelder im Redaxo Stil.</li>
 		<li>Modul 80-1 "MultiNewsletter Anmeldung mit Name und Anrede": Doppleter Versand bei von Anmeldemails bei Seitenreload behoben.</li>
 		<li>Modul 80-4 "MultiNewsletter YForm Anmeldung": Aktivierungsschlüssel war nicht mit anderen Modulen kompatibel.</li>

diff --git a/pages/index.php b/pages/index.php
@@ -15,8 +15,15 @@ function raw_field($label, $content) {
 
 echo rex_view::title($this->i18n('multinewsletter_addon_short_title'));
 
-if (rex_config::get('d2u_helper', 'article_id_privacy_policy', 0) == 0 || rex_config::get('d2u_helper', 'article_id_impress', 0) == 0) {
-	print rex_view::warning(rex_i18n::msg('d2u_helper_gdpr_warning'));
+if (rex_be_controller::getCurrentPage() != 'multinewsletter/settings' &&
+		(!rex_config::get('multinewsletter', 'link', 0) || !rex_config::get('multinewsletter', 'sender', 0) || !rex_config::get('multinewsletter', 'link_abmeldung', 0) || !rex_config::get('multinewsletter', 'lang_'. rex_clang::getStartId() .'_subscribe', 0))) {
+	print rex_view::error(rex_i18n::msg('multinewsletter_config_warning'));
+	rex_be_controller::setCurrentPage('multinewsletter/settings');
+}
+else {
+	if (rex_config::get('d2u_helper', 'article_id_privacy_policy', 0) == 0 || rex_config::get('d2u_helper', 'article_id_impress', 0) == 0) {
+		print rex_view::warning(rex_i18n::msg('d2u_helper_gdpr_warning'));
+	}
 }
 
 rex_be_controller::includeCurrentPageSubPath();