From 8a184f5cdd4a617fbb753fa599520435f30a9f6c Mon Sep 17 00:00:00 2001 From: Tobias Krais Date: Wed, 27 Oct 2021 16:56:23 +0200 Subject: [PATCH] warning when settings are missing --- lang/de_de.lang | 1 + lang/en_gb.lang | 1 + modules/80/4/output.php | 2 +- pages/help.changelog.php | 1 + pages/index.php | 11 +++++++++-- 5 files changed, 13 insertions(+), 3 deletions(-) diff --git a/lang/de_de.lang b/lang/de_de.lang index 7329829..ca3c934 100644 --- 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.
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 index 6912c75..0952f5c 100644 --- 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.
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 index ee22a31..aaf8b09 100644 --- a/modules/80/4/output.php +++ b/modules/80/4/output.php @@ -119,7 +119,7 @@ function sendActivationMail($yform) { html||

* '. $addon->getConfig("lang_". rex_clang::getCurrentId() ."_compulsory", "") .'

html||

'. $addon->getConfig("lang_". rex_clang::getCurrentId() ."_safety", "") .'

- 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 index eaa0d59..0228ace 100644 --- a/pages/help.changelog.php +++ b/pages/help.changelog.php @@ -8,6 +8,7 @@
  • Whoops beim Zurücksetzen des Versands unter MySQL 8.
  • Bugfix: Versandeinstellungen für eigenen SMTP Server wurden bei Aktivierungsmail ignoriert.
  • Der Versand für fehlgeschlagene Empfänger kann aus dem Archiv heraus wiederholt werden.
    • +
  • Warnmeldung eingebaut, wenn wichtige Eintellungen noch nicht festegelegt sind, die für die Verwendung des Addons Voraussetzung sind.
  • Modul 80-1, 80-2 und 80-3 mit Eingabefelder im Redaxo Stil.
  • Modul 80-1 "MultiNewsletter Anmeldung mit Name und Anrede": Doppleter Versand bei von Anmeldemails bei Seitenreload behoben.
  • Modul 80-4 "MultiNewsletter YForm Anmeldung": Aktivierungsschlüssel war nicht mit anderen Modulen kompatibel.
    • diff --git a/pages/index.php b/pages/index.php index 56c0138..aff6cf8 100644 --- 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(); \ No newline at end of file