From 6a239668430f902f81548bf08fec5c16124756cb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Tempez?= <frederic.tempez@outlook.com>
Date: Sun, 28 Nov 2021 12:07:07 +0100
Subject: [PATCH] Cookie OKay !

---
 core/core.js.php                              | 36 ++++++++++--
 core/core.php                                 | 55 +++++++++---------
 core/include/update.inc.php                   |  2 +-
 core/layout/common.css                        | 11 ++--
 core/module/config/config.php                 |  3 +-
 core/module/config/view/locale/locale.php     | 57 +++++++++++--------
 core/module/install/ressource/defaultdata.php |  2 +-
 core/module/theme/view/footer/footer.php      |  2 +-
 8 files changed, 105 insertions(+), 63 deletions(-)

diff --git a/core/core.js.php b/core/core.js.php
index ac429c36..c8ce0f0a 100644
--- a/core/core.js.php
+++ b/core/core.js.php
@@ -224,18 +224,27 @@ core.start = function() {
 
 		// Crée le cookie d'acceptation Google Analytics si l'ID a été saisie
 		var analytics = "<?php echo $this->getData(['config', 'seo', 'analyticsId']);?>";
+		// l'Id GA est défini dans la configuration, afficher la checkbox d'acceptation
 		if( analytics.length > 0){
-			document.cookie = "ZWII_COOKIE_GA_CONSENT=" + $("#googleAnalytics").prop("checked") + "<?php echo $_SERVER['PHP_SELF']; ?>" +";" + domain + ";" + path + ";" + samesite + ";" + expires;
+			// Traitement du retour de la checkbox
+			if ($("#googleAnalytics").is(":checked")) {
+				// L'URL du serveur faut TRUE
+				document.cookie = "ZWII_COOKIE_GA_CONSENT=" + "<?php echo $_SERVER['PHP_SELF']; ?>" + ";" + domain + ";" + path + ";" + samesite + ";" + expires;
+			} else {
+				document.cookie = "ZWII_COOKIE_GA_CONSENT=false;" + domain + ";" + path + ";" + samesite + ";" + expires;
+			}
+
 		}
 
-		// Stocke lz cookie d'acceptation
+		// Stocke le cookie d'acceptation
 		document.cookie = "ZWII_COOKIE_CONSENT=<?php echo $_SERVER['PHP_SELF']; ?>;" + domain + ";" + path + ";" + samesite + ";" + expires;
 	});
 
 	/**
 	* Suppression du cookie de consentement
-	*/
-	$('#footerCookies').bind('click', function(event) {
+	
+	$('#cookieConsentRefused').on('click', function() {
+	
 		var samesite = "samesite=lax";
 		var getUrl   = window.location;
 		var domain   = "domain=" + getUrl.host;
@@ -244,8 +253,17 @@ core.start = function() {
 		var e = new Date();
 		e.setFullYear(e.getFullYear() - 1);
 		var expires = "expires=" + e.toUTCString();
-		document.cookie = "ZWII_COOKIE_CONSENT=<?php echo $_SERVER['PHP_SELF']; ?>;" + domain + ";" + path + ";" + samesite + ";" + expires;
+		document.cookie = "ZWII_COOKIE_CONSENT=false;" + domain + ";" + path + ";" + samesite + ";" + expires;
+
+		// Désactiver le cookie GA
+		document.cookie = "ZWII_COOKIE_GA_CONSENT=false;" + domain + ";" + path + ";" + samesite + ";" + expires;
+		
+
+		
+		// fermer la fenêtre
+		$("#cookieConsent").fadeOut();
 	});
+	*/
 
 	/**
 	 * Fermeture de la popup des cookies
@@ -254,6 +272,14 @@ core.start = function() {
 		$(this).parents("#cookieConsent").fadeOut();
 	});
 
+	/**
+	 * Commande de gestion des cookies dans le footer
+	 */
+	 
+	 $("footer #footerCookies").on("click", function() {
+		$("#cookieConsent").removeClass("displayNone");
+	});
+
 	/**
 	 * Affiche / Cache le menu en mode responsive
 	 */
diff --git a/core/core.php b/core/core.php
index e217235d..bbe2c319 100644
--- a/core/core.php
+++ b/core/core.php
@@ -1174,7 +1174,7 @@ class common {
 	 */
 	 public function showAnalytics() {
  		if( !empty($code = $this->getData(['config', 'seo', 'analyticsId'])) &&
- 		    $this->getInput('ZWII_COOKIE_GA_CONSENT') === 'true'.$_SERVER['PHP_SELF'])  {
+ 		    $this->getInput('ZWII_COOKIE_GA_CONSENT') === $_SERVER['PHP_SELF'] )  {
  			echo '<!-- Global site tag (gtag.js) - Google Analytics -->
  				<script async src="https://www.googletagmanager.com/gtag/js?id='. $code .'"></script>
  				<script>
@@ -1191,33 +1191,36 @@ class common {
 	 */
 	public function showCookies() {
 
-		if( $this->getInput('ZWII_COOKIE_CONSENT') !== $_SERVER['PHP_SELF'] AND
-		   $this->getData(['config', 'cookieConsent']) === true
-	   ){
-
-		   $analytics = $this->getData(['locale', 'seo', 'analyticsId']);
-		   $legalPage = $this->getData(['locale', 'legalPageId']);
-		   $item  = '<div id="cookieConsent">';
-		   $item .= '<div class="cookieClose">';
-		   $item .= template::ico('cancel');
-		   $item .= '</div>';
-		   $item .= '<h3>'. $this->getData(['locale', 'cookies', 'cookiesTitleText']) . '</h3>';
-		   $item .= '<p>' . $this->getData(['locale', 'cookies', 'cookiesZwiiText']) . '</p>';
-		   if ($legalPage !== 'none')  {
+		// Gestion des cookies intégrée
+		if ($this->getData(['config', 'cookieConsent']) === true ) 
+			{
+			// Détermine si le bloc doit être affiché selon la validité du cookie
+			// L'URL du serveur faut TRUE
+			$enable = $this->getInput('ZWII_COOKIE_CONSENT') !==  $_SERVER['PHP_SELF'] ? '' : 'displayNone';
+			// Construction de la division contenant un forulaire
+			$item  = '<div id="cookieConsent" class="' . $enable . '">';
+			$item .= '<div class="cookieClose">';
+			$item .= template::ico('cancel');
+			$item .= '</div>';
+			$item .= '<h3>'. $this->getData(['locale', 'cookies', 'cookiesTitleText']) . '</h3>';
+			$item .= '<p>' . $this->getData(['locale', 'cookies', 'cookiesZwiiText']) . '</p>';
+			$legalPage = $this->getData(['locale', 'legalPageId']);
+			if ($legalPage !== 'none')  {
 				$item .= '<p><a href="' . helper::baseUrl() . $legalPage . '">' . $this->getData(['locale', 'cookies', 'cookiesLinkMlText']) . '</a></p>';
 			}
-		   if( $analytics !== null AND $analytics !=='' ){				
+			$item .= '<form method="POST" action="" id="cookieForm">';
+			$analytics = $this->getData(['config', 'seo', 'analyticsId']);
+			$stateCookieGA = $this->getInput('ZWII_COOKIE_GA_CONSENT') === $_SERVER['PHP_SELF'] ? 'checked="checked"' : '';
+			var_dump($stateCookieGA);
+			if( $analytics !== null AND $analytics !== '' ) {
 				$item .= '<p>' . $this->getData(['locale', 'cookies', 'cookiesGaText']) . '</p>';
-		   }
-		   $item .= '<form method="POST" action="" id="cookieForm">';
-		   if( $analytics !== null AND $analytics !=='' ) {
-			   $item .= '<input type="checkbox" id="googleAnalytics" name="googleAnalytics" value="GA">';
-			   $item .= '<label for="googleAnalytics">' . $this->getData(['locale', 'cookies', 'cookiesCheckboxGaText']) . '</label>';
-		   }
-		   $item .= '<br><br>';
-		   $item .= '<input type="submit" id="cookieConsentConfirm" value="Valider">';
-		   $item .= '</form></div>';
-		   echo $item;
+				$item .= '<input type="checkbox" id="googleAnalytics" name="googleAnalytics" value="GA" ' . $stateCookieGA . '>';
+				$item .= '<label for="googleAnalytics">' . $this->getData(['locale', 'cookies', 'cookiesCheckboxGaText']) . '</label>';
+			}
+			$item .= '<br><br>';
+			$item .= '<input type="submit" id="cookieConsentConfirm" value="' . $this->getData(['locale', 'cookies', 'cookiesButtonText']) . '">';
+			$item .= '</form></div>';
+			echo $item;
 		}
 
 	}
@@ -1484,7 +1487,7 @@ class common {
 		$items .= '</span>';
 		// Affichage de la gestion des cookies
 		$label = empty($this->getData(['locale', 'cookies', 'cookiesFooterText'])) ? 'Confidentialité' : $this->getData(['locale', 'cookies', 'cookiesFooterText']) ;
-		$items .= ($this->getData(['config', 'cookieConsent']) === true && $this->getData(['theme', 'footer', 'displayCookie'])) ? '':'<span id="footerCookies"><wbr>&nbsp;|&nbsp;<a href="#">'. $label .'</a></span>';
+		$items .= ($this->getData(['config', 'cookieConsent']) === true && $this->getData(['theme', 'footer', 'displayCookie'])) === true ? '<span id="footerCookies"><wbr>&nbsp;|&nbsp;<a href="javascript:void(0)">'. $label .'</a></span>' : '';
 		// Affichage du lien de connexion
 		if(
             (
diff --git a/core/include/update.inc.php b/core/include/update.inc.php
index afdc8e94..fa8d71fd 100644
--- a/core/include/update.inc.php
+++ b/core/include/update.inc.php
@@ -707,8 +707,8 @@ if ($this->getData(['core', 'dataVersion']) < 11200) {
 	$this->setData(['locale', 'cookies', 'cookieTitleText', 'Gérer les cookies']);
 	$this->setData(['locale', 'cookies', 'cookieLinkMlText', 'Voir les mentions légales']);
 	$this->setData(['locale', 'cookies', 'cookieCheckboxGaText', 'Autorisation des cookies Google Analytics']);
+	$this->setData(['locale', 'cookies', 'cookieButtonText', 'J\'ai compris']);
 	
-
 	// Mise à jour
 	$this->setData(['core', 'dataVersion', 11200]);
 }
diff --git a/core/layout/common.css b/core/layout/common.css
index 292ba574..88be61b0 100755
--- a/core/layout/common.css
+++ b/core/layout/common.css
@@ -986,9 +986,9 @@ footer #footerSocials .zwiico-github:hover {
 
 /* Message sur les cookies */
 #cookieConsent {
-	width: 80%;
+	width: 60%;
 	margin: auto;
-	opacity: .9;
+	opacity: .95;
 	background: #212223;
 	position: fixed;
 	right: 0;
@@ -1001,17 +1001,20 @@ footer #footerSocials .zwiico-github:hover {
 	font-size: 1em;
 }
 
+#cookieConsentConfirm {
+	background-color: green;
+}
+
 #cookieConsentConfirm {
 	cursor: pointer;
 	margin-left: 10px;
-	background: #666;
 	padding: 4px 8px;
 	display: inline-block;
 	transition: background .3s ease-out;
 }
 
 #cookieConsentConfirm:hover {
-	background: #777;
+	filter: grayscale(50%);
 }
 
 #cookieConsent .cookieClose {
diff --git a/core/module/config/config.php b/core/module/config/config.php
index 98f7645d..9e5f58a6 100644
--- a/core/module/config/config.php
+++ b/core/module/config/config.php
@@ -445,7 +445,8 @@ class config extends common {
 						'cookiesTitleText'	=> $this->getInput('localeCookiesTitleText', helper::FILTER_STRING_SHORT, $this->getData(['config', 'cookieConsent'])),
 						'cookiesLinkMlText'	=> $this->getInput('localeCookiesLinkMlText', helper::FILTER_STRING_SHORT, $this->getData(['config', 'cookieConsent'])),
 						'cookiesCheckboxGaText'	=> $this->getInput('localeCookiesCheckboxGaText', helper::FILTER_STRING_SHORT, $this->getData(['config', 'cookieConsent'])),
-						'cookiesFooterText' => $this->getInput('localeCookiesFooterText', helper::FILTER_STRING_SHORT, $this->getData(['config', 'cookieConsent']))
+						'cookiesFooterText' =>  $this->getInput('localeCookiesFooterText', helper::FILTER_STRING_SHORT, $this->getData(['config', 'cookieConsent'])),
+						'cookiesButtonText' =>$this->getInput('localeCookiesButtonText', helper::FILTER_STRING_SHORT, $this->getData(['config', 'cookieConsent']))
 					]
 				]
 			]);
diff --git a/core/module/config/view/locale/locale.php b/core/module/config/view/locale/locale.php
index 88e596a8..18681cc2 100644
--- a/core/module/config/view/locale/locale.php
+++ b/core/module/config/view/locale/locale.php
@@ -167,26 +167,6 @@
 						</a>
 					</span>
 				</h4>
-				<div class="row">
-					<div class="col12">
-						<?php echo template::textarea('localeCookiesZwiiText', [
-							'help' => 'Saisissez le message pour les cookies déposés par ZwiiCMS, nécessaires au fonctionnement et qui ne nécessitent pas de consentement.',
-							'label' => 'Cookies Zwii',
-							'value' => $this->getData(['locale', 'cookies', 'cookiesZwiiText']),
-							'placeHolder' => 'Ce site utilise des cookies nécessaires à son fonctionnement, ils permettent de fluidifier son fonctionnement par exemple en mémorisant les données de connexion, la langue que vous avez choisie ou la validation de ce message.'
-						]); ?>
-					</div>
-				</div>
-				<div class="row">
-					<div class="col12">
-						<?php echo template::textarea('localeCookiesGaText', [
-							'help' => 'Saisissez le message pour les cookies déposés par Google Analytics, le consentement est requis.',
-							'label' => 'Cookies Google Analytics',
-							'value' => $this->getData(['locale', 'cookies', 'cookiesGaText']),
-							'placeHolder' => 'Il utilise également des cookies permettant de réaliser des statistiques de visites pour améliorer votre expérience utilisateur, ces cookies déposés par Google Analytics ont besoin de votre consentement.'
-						]); ?>
-					</div>
-				</div>
 				<div class="row">
 					<div class="col12">
 						<?php echo template::text('localeCookiesTitleText', [
@@ -198,15 +178,35 @@
 					</div>
 				</div>
 				<div class="row">
-					<div class="col6">
+					<div class="col8">
+						<?php echo template::textarea('localeCookiesZwiiText', [
+							'help' => 'Saisissez le message pour les cookies déposés par ZwiiCMS, nécessaires au fonctionnement et qui ne nécessitent pas de consentement.',
+							'label' => 'Cookies Zwii',
+							'value' => $this->getData(['locale', 'cookies', 'cookiesZwiiText']),
+							'placeHolder' => 'Ce site utilise des cookies nécessaires à son fonctionnement, ils permettent de fluidifier son fonctionnement par exemple en mémorisant les données de connexion, la langue que vous avez choisie ou la validation de ce message.'
+						]); ?>
+					</div>
+
+					<div class="col4">
 						<?php echo template::text('localeCookiesLinkMlText', [
-							'help' => 'Saisissez le texte du lien vers les mentions légales.',
-							'label' => 'Lien vers mentions légales',
+							'help' => 'Saisissez le texte du lien vers les mentions légales,la page doit être définie dans la configuration du site.',
+							'label' => 'Lien page des mentions légales.',
 							'value' => $this->getData(['locale', 'cookies', 'cookiesLinkMlText']),
 							'placeHolder' => 'Voir les mentions légales'
 						]); ?>
 					</div>
-					<div class="col6">
+				</div>
+				<div class="row">
+					<div class="col8">
+						<?php echo template::textarea('localeCookiesGaText', [
+							'help' => 'Saisissez le message pour les cookies déposés par Google Analytics, le consentement est requis.',
+							'label' => 'Cookies Google Analytics',
+							'value' => $this->getData(['locale', 'cookies', 'cookiesGaText']),
+							'placeHolder' => 'Il utilise également des cookies permettant de réaliser des statistiques de visites pour améliorer votre expérience utilisateur, ces cookies déposés par Google Analytics ont besoin de votre consentement.'
+						]); ?>
+					</div>
+
+					<div class="col4">
 						<?php echo template::text('localeCookiesCheckboxGaText', [
 							'help' => 'Saisissez le texte de la case à cocher Google Analytics.',
 							'label' => 'Checkbox Google Analytics',
@@ -215,6 +215,15 @@
 						]); ?>
 					</div>
 				</div>
+				<div class="row">
+					<div class="col4 offset4">
+						<?php echo template::text('localeCookiesButtonText', [
+							'label' => 'Bouton de validation',
+							'value' => $this->getData(['locale', 'cookies', 'cookiesButtonText']),
+							'placeHolder' => 'J\'ai compris'
+						]); ?>	
+					</div>
+				</div>
 			</div>
 		</div>
 	</div>
diff --git a/core/module/install/ressource/defaultdata.php b/core/module/install/ressource/defaultdata.php
index 176352b7..5840ff64 100644
--- a/core/module/install/ressource/defaultdata.php
+++ b/core/module/install/ressource/defaultdata.php
@@ -81,8 +81,8 @@ class init extends common {
 				'cookiesLinkMlText' => 'Voir les mentions légales',
 				'cookiesCheckboxGaText' => 'Autorisation des cookies Google Analytics',
 				'cookiesFooterText' => 'Confidentialité'
+				'cookieButtonText' => 'J\'ai compris'
 			]
-
 		],
 		'page' => [
 			'accueil' => [
diff --git a/core/module/theme/view/footer/footer.php b/core/module/theme/view/footer/footer.php
index 9c1df67e..c336bc21 100644
--- a/core/module/theme/view/footer/footer.php
+++ b/core/module/theme/view/footer/footer.php
@@ -90,7 +90,7 @@
                 </div>
                 <div class="col3">
                     <?php echo template::checkbox('themefooterDisplayCookie', true, 'Confidentialité', [
-                            'checked' => $this->getData(['config', 'cookieConsent']) ? false : $this->getData(['theme', 'footer', 'displayCookie']),
+                            'checked' => $this->getData(['config', 'cookieConsent']) === true ? $this->getData(['theme', 'footer', 'displayCookie']) : false,
                             'help' => 'Affiche le message relatif au cookie, disponible si le message d\'acceptation des cookies est activé.',
                             'disabled' => !$this->getData(['config', 'cookieConsent'])
                         ]); ?>