From de0749b51b2e03bafa35f8d683e2f8ef4e99b0c2 Mon Sep 17 00:00:00 2001
From: Fred Tempez <frederic.tempez@outlook.com>
Date: Sat, 24 Aug 2024 17:29:17 +0200
Subject: [PATCH] 13.3.06 Login, connexion persistante

---
 CHANGES.md                  |  1 +
 core/class/helper.class.php |  4 ++--
 core/module/user/user.php   | 33 +++++++++++++++++++++++----------
 3 files changed, 26 insertions(+), 12 deletions(-)

diff --git a/CHANGES.md b/CHANGES.md
index 78fb425d..206c1635 100755
--- a/CHANGES.md
+++ b/CHANGES.md
@@ -5,6 +5,7 @@
 - Répare le bouton d'effacement en mode édition d'une page. 
 - Corrige la feuille de style du slider les balises H1, H3 et A.
 - Corrige l'absence de contenu de page lorsque le module est en position libre et que le mot clé [MODULE] n'a pas été inséré.
+- Corrige l'option "REster connecter sur ce navigateur' dont la connexion est désormais réellement persistente.
 
 ## Versions 13.3.05
 ** Corrections : **
diff --git a/core/class/helper.class.php b/core/class/helper.class.php
index da8c3a57..1753918e 100644
--- a/core/class/helper.class.php
+++ b/core/class/helper.class.php
@@ -407,9 +407,9 @@ class helper
 	 * @param string $cookieKey Clé du cookie à supprimer
 	 */
 	public static function deleteCookie($cookieKey)
-	{
-		unset($_COOKIE[$cookieKey]);
+	{		
 		setcookie($cookieKey, '', time() - 3600, helper::baseUrl(false, false), '', false, true);
+		unset($_COOKIE[$cookieKey]);
 	}
 
 	/**
diff --git a/core/module/user/user.php b/core/module/user/user.php
index 48a212c6..4e5ebe26 100644
--- a/core/module/user/user.php
+++ b/core/module/user/user.php
@@ -574,7 +574,7 @@ class user extends common
 		// Stoppe si le profil est affecté
 		foreach ($groups as $userId) {
 			if ((string) $this->getData(['user', $userId, 'profil']) === $this->getUrl(3)) {
-				$profilUsed= false;
+				$profilUsed = false;
 			}
 		}
 		foreach ($this->getData(['profil']) as $groupId => $groupData) {
@@ -935,11 +935,11 @@ class user extends common
 		// recherche les membres du groupe 
 		$groups = helper::arrayColumn($this->getData(['user']), 'group');
 		$groups = array_keys($groups, $this->getUrl(2));
-		$flag= true;
+		$flag = true;
 		// Stoppe si le profil est affecté
 		foreach ($groups as $userId) {
 			if ((string) $this->getData(['user', $userId, 'profil']) === $this->getUrl(3)) {
-				$flag= false;
+				$flag = false;
 			}
 		}
 		if (
@@ -956,7 +956,7 @@ class user extends common
 			if ($flag) {
 				$this->deleteData(['profil', $this->getUrl(2), $this->getUrl(3)]);
 			}
-			
+
 			// Valeurs en sortie
 			$this->addOutput([
 				'redirect' => helper::baseUrl() . $this->getUrl(0) . '/profil',
@@ -1045,10 +1045,22 @@ class user extends common
 					// RAZ
 					$this->setData(['user', $userId, 'connectFail', 0]);
 					$this->setData(['user', $userId, 'connectTimeout', 0]);
-					// Expiration
+					
+					// Validité du cookie
 					$expire = $this->getInput('userLoginLongTime', helper::FILTER_BOOLEAN) === true ? strtotime("+1 year") : 0;
-					setcookie('ZWII_USER_ID', $userId, $expire, helper::baseUrl(false, false), '', helper::isHttps(), true);
-					setcookie('ZWII_USER_PASSWORD', $this->getData(['user', $userId, 'password']), $expire, helper::baseUrl(false, false), '', helper::isHttps(), true);
+					switch ($this->getInput('userLoginLongTime', helper::FILTER_BOOLEAN)) {
+						case false:
+							// Cookie de session
+							setcookie('ZWII_USER_ID', $userId, $expire, helper::baseUrl(false, false), '', helper::isHttps(), true);
+							setcookie('ZWII_USER_PASSWORD', $this->getData(['user', $userId, 'password']), $expire, helper::baseUrl(false, false), '', helper::isHttps(), true);
+							break;
+						default:
+							// Cookie persistant
+							setcookie('ZWII_USER_ID', $userId, $expire, helper::baseUrl(false, false));
+							setcookie('ZWII_USER_PASSWORD', $this->getData(['user', $userId, 'password']), $expire, helper::baseUrl(false, false));
+							break;
+					}
+
 					// Accès multiples avec le même compte
 					$this->setData(['user', $userId, 'accessCsrf', $_SESSION['csrf']]);
 					// Valeurs en sortie lorsque le site est en maintenance et que l'utilisateur n'est pas administrateur
@@ -1064,8 +1076,9 @@ class user extends common
 					} else {
 						$logStatus = 'Connexion réussie';
 						$pageId = $this->getUrl(2);
-						if ($this->getData(['config', 'page404']) === $pageId
-						 || $this->getData(['config', 'page403']) === $pageId
+						if (
+							$this->getData(['config', 'page404']) === $pageId
+							|| $this->getData(['config', 'page403']) === $pageId
 						) {
 							$pageId = '';
 						}
@@ -1391,7 +1404,7 @@ class user extends common
 			// Ignorer les entrées de répertoire parent et actuel
 			if ($file == '.' || $file == '..') {
 				continue;
-			} 
+			}
 			// Construisez le chemin complet du fichier ou du répertoire
 			$path = $dir . '/' . $file;
 			// Vérifiez si c'est un répertoire