From 06ad3c346781f61ff64646ad2c61c9ba89998973 Mon Sep 17 00:00:00 2001
From: fredtempez <ftempez@gmail.com>
Date: Fri, 1 Oct 2021 10:42:21 +0200
Subject: [PATCH] =?UTF-8?q?searchpage=20et=20legal=20:=20la=20d=C3=A9sacti?=
 =?UTF-8?q?vation=20de=20la=20page=20dans=20la=20config=20entra=C3=AEne=20?=
 =?UTF-8?q?la=20d=C3=A9sactivation=20du=20lien=20dans=20le=20footer.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 core/module/config/config.php            | 27 ++++++++++--------------
 core/module/theme/view/footer/footer.php |  4 ++--
 2 files changed, 13 insertions(+), 18 deletions(-)

diff --git a/core/module/config/config.php b/core/module/config/config.php
index edf8547d..d5bee95d 100644
--- a/core/module/config/config.php
+++ b/core/module/config/config.php
@@ -377,27 +377,19 @@ class config extends common {
 	public function index() {
 		// Soumission du formulaire
 		if($this->isPost()) {
-			$success = true;
-			// Empêcher la modification si défini dans footer
+
+			// Répercuter la suppression de la page dans la configuration du footer
 			if ( $this->getData(['theme','footer','displaySearch']) === true
 				AND $this->getInput('configSearchPageId') === 'none'
 				){
-					$searchPageId = $this->getData(['locale','searchPageId']);
-					self::$inputNotices['configSearchPageId'] = 'Désactiver l\'option dans le pied de page';
-					$success = false;
-			} else {
-					$searchPageId = $this->getInput('configSearchPageId');
+					$this->setData(['theme', 'footer', 'displaySearch', false]);
 			}
-			// Empêcher la modification si défini dans footer
 			if ( $this->getData(['theme','footer','displayLegal']) === true
 				AND $this->getInput('configLegalPageId') === 'none'
 				){
-					$legalPageId = $this->getData(['locale','legalPageId']);
-					self::$inputNotices['configLegalPageId'] = 'Désactiver l\'option dans le pied de page';
-					$success = false;
-			} else {
-					$legalPageId = $this->getInput('configLegalPageId');
+					$this->setData(['theme', 'footer', 'displayLegal', false]);
 			}
+
 			// Sauvegarder
 			$this->setData([
 				'locale',
@@ -406,20 +398,23 @@ class config extends common {
 					'page404' => $this->getInput('configPage404'),
 					'page403' => $this->getInput('configPage403'),
 					'page302' => $this->getInput('configPage302'),
-					'legalPageId' => $legalPageId,
-					'searchPageId' => $searchPageId,
+					'legalPageId' => $this->getInput('configLegalPageId'),
+					'searchPageId' => $this->getInput('configSearchPageId'),
 					'metaDescription' => $this->getInput('configMetaDescription', helper::FILTER_STRING_LONG, true),
 					'title' => $this->getInput('configTitle', helper::FILTER_STRING_SHORT, true)
 				]
 			]);
+
 			$this->setData(['config', 'i18n', 'enable', $this->getInput('configI18n',helper::FILTER_BOOLEAN) ]);
+
 			// Générer robots.txt et sitemap
 			$this->generateFiles();
+
 			// Valeurs en sortie
 			$this->addOutput([
 				'redirect' => helper::baseUrl() . $this->getUrl(),
 				'notification' => 'Modifications enregistrées',
-				'state' => $success
+				'state' => true
 			]);
 		}
 
diff --git a/core/module/theme/view/footer/footer.php b/core/module/theme/view/footer/footer.php
index fb5d1b7d..8e3aa1d2 100644
--- a/core/module/theme/view/footer/footer.php
+++ b/core/module/theme/view/footer/footer.php
@@ -80,14 +80,14 @@
                     <?php echo template::checkbox('themeFooterDisplayLegal', true, 'Mentions légales', [
                             'checked' => $this->getData(['locale', 'legalPageId']) === 'none' ? false : $this->getData(['theme', 'footer', 'displayLegal']),
                             'disabled' => $this->getData(['locale', 'legalPageId']) === 'none' ? true : false,
-                            'help' => 'Sélectionnez la page contenant les mentions légales dans Configurer le site / Pages spéciales'
+                            'help' => $this->getData(['locale', 'legalPageId']) === 'none' ? 'Une page contenant les mentions légales n\'est pas définie dans la configuration du site / pages spéciales.' : ''
                     ]); ?>
                 </div>
                 <div class="col6">
                     <?php echo template::checkbox('themeFooterDisplaySearch', true, 'Rechercher dans le site', [
                             'checked' => $this->getData(['locale', 'searchPageId']) === 'none' ? false : $this->getData(['theme', 'footer', 'displaySearch']),
                             'disabled' => $this->getData(['locale', 'searchPageId']) === 'none' ? true : false,
-                            'help' => 'Sélectionnez la page contenant un module de recherche dans Configurer le site / Pages spéciales'
+                            'help' => $this->getData(['locale', 'searchPageId']) === 'none' ? 'Une page contenant un module de recherche n\'est pas définie dans la configuration du site / pages spéciales.' : ''
                         ]); ?>
                 </div>
             </div>