Merge pull request '11000' (#2) from ZwiiCMS-Team/ZwiiCMS:11000 into 11000

Reviewed-on: #2

CHANGES.md

@@ -2,6 +2,7 @@
 
 ## version 10.4.00
 - Modifications :
+    - Filtrage des URL générées par facebook (FBCLID) occasionnant une erreur 404
     - Captcha arithmétique, activation recommandée dans la configuration.
     - Module User
         - Pour les articles de blog et de news, choix de la signature, nom+prenom ; nom+prenom ; id ; pseudo
@@ -15,19 +16,20 @@
         - Approbation des commentaires
     - Gestion des thèmes :
         - Bouton de réinitialisation avec confirmation
-    - Traduction automatique dans la langue du navigateur.
     - Amélioration de la structure du flux RSS.
 
+
 ## Version 10.3.13
 Modifications :
     - Système de données (JsonDB) :
         - Verrouillage des fichiers de données ouverts en écriture.
         - Message d'erreur littéral.
         - Sauvegarde des fichiers de données après un effacement et une écriture.
+        - Identifiant des noms de ressources (id de page , d'utilisateur, etc..) composés de nombres , remplacement du caractère de préfixe "i" par "_".
     - Google Analytics, option d'anonymisation.
-    - Procédure de connexion : les erreurs de captcha sont comptabilisées comme des échecs, allégement des messages d'information.
+    - Procédure de connexion : les erreurs de captcha sont comptabilisées comme des échecs, allégement des messages d'information. Echecs de connexion, informations plus précises dans le journal de connexion.
     - TinyMCE : ajout des scripts possibles.
-Correction :
+Corrections :
     - Notification de commentaire, remplacement du nom de la page par le titre de l'article.
     - Thème : couleur du texte au survol d'un bouton standard.
 

README.md

@@ -1,4 +1,3 @@
-![](https://img.shields.io/github/last-commit/fredtempez/ZwiiCMS/master) ![](https://img.shields.io/github/release-date/fredtempez/ZwiiCMS)
 
 # ZwiiCMS 10.4.00
 
@@ -75,9 +74,9 @@ Pour revenir à la version 8, renommez ce fichier "data.json".
     [R] fr                 Dossier localisé
       [F] page.json        Données des pages
       [F] module.json      Données des modules de pages
-    [F] admin.css          Thème de la partie administration
+    [F] admin.css          Thème des pages d'administration
-    [F] admin.json         Données de la partie administration
+    [F] admin.json         Données de thème des pages d'administration 
-    [F] blacklist.json     Données de connexion des comptes inconnus
+    [F] blacklist.json     Journalisation des tentatives de connexion avec des comptes inconnus
     [F] config.json        Configuration du site
     [F] core.json          Configuration du noyau
     [F] custom.css         Feuille de style de la personnalisation avancée
@@ -85,6 +84,7 @@ Pour revenir à la version 8, renommez ce fichier "data.json".
     [F] theme.css          Thème du site
     [F] theme.json         Données du site
     [F] user.json          Données des utilisateurs
+    [F] .backup            Marqueur de la sauvegarde des fichiers si présent
   [R] file                 Répertoire d'upload du gestionnaire de fichiers
     [R] source             Ressources diverses
     [R] thumb              Miniatures des images

core/class/helper.class.php

@@ -273,7 +273,7 @@ class helper {
 				}
 				// Un ID ne peut pas être un entier, pour éviter les conflits avec le système de pagination
 				if(intval($text) !== 0) {
-					$text = 'i' . $text;
+					$text = '_' . $text;
 				}
 				break;
 			case self::FILTER_INT:

core/class/jsondb/JsonDb.class.php

@@ -132,8 +132,8 @@ class JsonDb extends \Prowebcraft\Dot
             touch($this->db);
         }
         // Backup file
-        if ($this->config['backup']) {
+        if ($this->config['backup'] === true) {
-            copy ($this->db, $this->db . '.back');
+            copy ($this->db, str_replace('json' , 'backup.json', $this->db));
         }
         if ( is_writable($this->db) ) {
             // 3 essais

core/core.js.php

@@ -480,4 +480,10 @@ $(document).ready(function(){
         $(".helpDisplayContent").slideToggle();
 	});
 
+	/**
+	 * Remove ID Facebook from URL
+	 */
+	if(/^\?fbclid=/.test(location.search))
+		location.replace(location.href.replace(/\?fbclid.+/, ""));
+
 });

core/core.php

@@ -154,6 +154,7 @@ class common {
 	];
 	// Langue courante
 	public static $i18nSite;
+	public static $i18nCurrent;
 	public static $timezone;
 	private $url = '';
 	// Données de site
@@ -168,10 +169,10 @@ class common {
 	// Descripteur de données Entrées / Sorties
 	// Liste ici tous les fichiers de données
 	private $dataFiles = [
+		'config' => '',
 		'page' => '',
 		'module' => '',
 		'core' => '',
-		'config' => '',
 		'page' => '',
 		'user' => '',
 		'theme' => '',
@@ -202,13 +203,13 @@ class common {
 		}
 
 		// Instanciation de la classe des entrées / sorties
-		// Récupére les descripteurs
+		// Récupère les descripteurs
 		foreach ($this->dataFiles as $keys => $value) {
 			// Constructeur  JsonDB
 			$this->dataFiles[$keys] = new \Prowebcraft\JsonDb([
 				'name' => $keys . '.json',
 				'dir' => $this->dataPath ($keys,self::$i18nCurrent),
-				'backup' => true
+				'backup' => file_exists('site/data/.backup')
 			]);;
 		}
 
@@ -553,7 +554,16 @@ class common {
 		// Une partie de l'url
 		else {
 			$url = explode('/', $this->url);
-			return array_key_exists($key, $url) ? $url[$key] : null;
+			if (array_key_exists($key, $url) )  {
+				if (strpos($url[$key],'fbclid=')  === false) {
+					$result = $url[$key];
+				} else {
+					$result = $key === 0 ? $this->getData(['config','homePageId']) : '';
+				}
+			} else {
+				$result = null;
+			}
+			return $result;
 		}
 	}
 
@@ -1570,6 +1580,18 @@ class common {
 			$this->setData(['config','translate','nl',false]);
 			$this->setData(['config','translate','pt',false]);
 			
+			// Renommer les fichier de backup
+			if ($this->getInput('configAdvancedFileBackup', helper::FILTER_BOOLEAN) === false) {
+				$path = realpath('site/data');
+				foreach (new RecursiveIteratorIterator(new RecursiveDirectoryIterator($path)) as $filename)
+				{
+					echo "$filename</br>";
+					if (strpos($filename,'back.json')) {
+						rename($filename, str_replace('back.json','backup.json',$filename));
+					}
+				}
+			}
+
 			$this->setData(['core', 'dataVersion', 10400]);
 
 			/**
@@ -1921,7 +1943,7 @@ class core extends common {
 					$access = false;
 				}
 			}
-			// Empêcher l'accès aux page désactivée par URL directe
+			// Empêcher l'accès aux pages désactivées par URL directe
 			if ( ( $this->getData(['page', $this->getUrl(0),'disable']) === true
 					AND $this->getUser('password') !== $this->getInput('ZWII_USER_PASSWORD')
 				) OR (

core/module/config/config.php

@@ -525,7 +525,20 @@ class config extends common {
 					]
 				]
 			]);
-
+			// Efface les fichiers de backup lorsque l'option est désactivée
+			if ($this->getInput('configAdvancedFileBackup', helper::FILTER_BOOLEAN) === false) {
+				$path = realpath('site/data');
+				foreach (new RecursiveIteratorIterator(new RecursiveDirectoryIterator($path)) as $filename)
+				{
+					if (strpos($filename,'backup.json')) {
+						unlink($filename);
+					}
+				}
+				if (file_exists('site/data/.backup')) unlink('site/data/.backup');
+			} else {
+				touch('site/data/.backup');
+			}
+			// Notice
 			if(self::$inputNotices === []) {
 				// Active la réécriture d'URL
 				$rewrite = $this->getInput('rewrite', helper::FILTER_BOOLEAN);
@@ -539,6 +552,8 @@ class config extends common {
 						PHP_EOL .
 						'<ifModule mod_rewrite.c>' . PHP_EOL .
 						"\tRewriteEngine on" . PHP_EOL .
+						"\tRewriteCond %{QUERY_STRING} ^(.*)&?fbclid=[^&]+&?(.*)$ [NC]". PHP_EOL .
+						"\tRewriteRule ^/?(.*)$ /$1?%1%2 [R=301,L]". PHP_EOL .
 						"\tRewriteBase " . helper::baseUrl(false, false) . PHP_EOL .
 						"\tRewriteCond %{REQUEST_FILENAME} !-f" . PHP_EOL .
 						"\tRewriteCond %{REQUEST_FILENAME} !-d" . PHP_EOL .
@@ -572,12 +587,6 @@ class config extends common {
 				'state' => $success
 			]);
 		}
-		// Initialisation du screen - APPEL AUTO DESACTIVE POUR EVITER UN RALENTISSEMENT
-		/*
-		if (!file_exists(self::FILE_DIR.'source/screenshot.jpg')) {
-			$this->configMetaImage();
-		}
-		*/
 		// Valeurs en sortie
 		$this->addOutput([
 			'title' => 'Configuration avancée',

core/module/config/view/advanced/advanced.php

@@ -101,6 +101,12 @@
 							'help' => '<p>Une archive contenant le dossier /site/data est copiée dans le dossier \'site/backup\'. La sauvegarde est conservée pendant 30 jours.</p><p>Les fichiers du site ne sont pas sauvegardés automatiquement.</p>'
 						]); ?>
 				</div>
+				<div class="col4">
+					<?php echo template::checkbox('configAdvancedFileBackup', true, 'Copie de sauvegarde', [
+							'checked' => file_exists('site/data/.backup'),
+							'help' => '<p>Un fichier .backup.json est généré à chaque édition ou effacement d\'une donnée. La désactivation entraîne la suppression de ces fichiers.</p>'
+						]); ?>
+				</div>
 			</div>
 			<div class="row">
 				<div class="col4">

core/module/user/user.php

@@ -365,6 +365,7 @@ class user extends common {
 	 */
 	public function login() {
 		// Soumission du formulaire
+		$logStatus = '';
 		if($this->isPost()) {
 			// Lire Id du compte
 			$userId = $this->getInput('userLoginId', helper::FILTER_ID, true);
@@ -373,29 +374,15 @@ class user extends common {
 				$this->getData(['config','connect','captcha'])
 				AND password_verify($this->getInput('userLoginCaptcha', helper::FILTER_INT), $this->getInput('userLoginCaptchaResult') ) === false )
 			{
-				//self::$inputNotices['userLoginCaptcha'] = 'Incorrect';
+				$captcha = false;
-				$notification = 'Captcha incorrect !';
-				// Cas 1 le nombre de connexions est inférieur aux tentatives autorisées : incrément compteur d'échec
-				if ($this->getData(['user',$userId,'connectFail']) < $this->getData(['config', 'connect', 'attempt'])) {
-					$this->setData(['user',$userId,'connectFail',$this->getdata(['user',$userId,'connectFail']) + 1 ]);
-				}
-				// Cas 2 la limite du nombre de connexion est atteinte : placer le timer
-				if ( $this->getdata(['user',$userId,'connectFail']) == $this->getData(['config', 'connect', 'attempt'])	) {
-						$this->setData(['user',$userId,'connectTimeout', time()]);
-				}
-				// Cas 3 le délai de bloquage court
-				if ($this->getData(['user',$userId,'connectTimeout'])  + $this->getData(['config', 'connect', 'timeout']) > time() ) {
-					$notification = 'Accès bloqué ' . ($this->getData(['config', 'connect', 'timeout']) / 60) . ' minutes.';
-				}
-				// Valeurs en sortie
-				$this->addOutput([
-					'notification' => $notification
-				]);
 			} else {
+				$captcha = true;
+			}
 			/**
 			 * Aucun compte existant
 			 */
 			if ( !$this->getData(['user', $userId])) {
+				$logStatus = 'Compte inconnu';
 				//Stockage de l'IP
 				$this->setData([
 					'blacklist',
@@ -410,6 +397,7 @@ class user extends common {
 				$ipBlackList = helper::arrayCollumn($this->getData(['blacklist']), 'ip');
 				if ( $this->getData(['blacklist',$userId,'connectFail']) >= $this->getData(['config', 'connect', 'attempt'])
 					AND in_array($this->getData(['blacklist',$userId,'ip']),$ipBlackList) ) {
+					$logStatus = 'Compte inconnu verrouillé';
 					// Valeurs en sortie
 					$this->addOutput([
 						'notification' => 'Compte verrouillé',
@@ -419,7 +407,7 @@ class user extends common {
 				} else {
 					// Valeurs en sortie
 					$this->addOutput([
-							'notification' => 'Identifiant ou mot de passe incorrects'
+						'notification' => 'Captcha, identifiant ou mot de passe incorrects'
 					]);
 				}
 			/**
@@ -439,6 +427,7 @@ class user extends common {
 					AND $this->getData(['user',$userId,'connectFail']) < $this->getData(['config', 'connect', 'attempt'])
 					AND password_verify($this->getInput('userLoginPassword', helper::FILTER_STRING_SHORT, true), $this->getData(['user', $userId, 'password']))
 					AND $this->getData(['user', $userId, 'group']) >= self::GROUP_MEMBER
+					AND $captcha === true
 				) {
 					// RAZ
 					$this->setData(['user',$userId,'connectFail',0 ]);
@@ -462,6 +451,7 @@ class user extends common {
 							'state' => false
 						]);
 					} else {
+						$logStatus = 'Connexion réussie';
 						// Valeurs en sortie
 						$this->addOutput([
 							'notification' => 'Bienvenue ' . $this->getData(['user',$userId,'firstname']) . ' ' . $this->getData(['user',$userId,'lastname']) ,
@@ -471,7 +461,8 @@ class user extends common {
 					}
 				// Sinon notification d'échec
 				} else {
-						$notification = 'Identifiant ou mot de passe incorrects';
+					$notification = 'Captcha, identifiant ou mot de passe incorrects';
+					$logStatus = $captcha === true ? 'Erreur de mot de passe' : 'Erreur de captcha';
 					// Cas 1 le nombre de connexions est inférieur aux tentatives autorisées : incrément compteur d'échec
 					if ($this->getData(['user',$userId,'connectFail']) < $this->getData(['config', 'connect', 'attempt'])) {
 						$this->setData(['user',$userId,'connectFail',$this->getdata(['user',$userId,'connectFail']) + 1 ]);
@@ -484,18 +475,7 @@ class user extends common {
 					if ($this->getData(['user',$userId,'connectTimeout'])  + $this->getData(['config', 'connect', 'timeout']) > time() ) {
 						$notification = 'Accès bloqué ' . ($this->getData(['config', 'connect', 'timeout']) / 60) . ' minutes.';
 					}
-						// Journalisation
+
-						$dataLog = mb_detect_encoding(strftime('%d/%m/%y',time()), 'UTF-8', true)
-								? strftime('%d/%m/%y',time()) . ';' . strftime('%R',time()) . ';'
-								: utf8_encode(strftime('%d/%m/%y',time())) . ';' . utf8_encode(strftime('%R',time())) . ';' ;
-						$dataLog .= helper::getIp() . ';';
-						$dataLog .= $userId . ';' ;
-						$dataLog .= $this->getUrl() .';' ;
-						$dataLog .= 'échec de connexion' ;
-						$dataLog .= PHP_EOL;
-						if ($this->getData(['config','connect','log'])) {
-							file_put_contents(self::DATA_DIR . 'journal.log', $dataLog, FILE_APPEND);
-						}
 					// Valeurs en sortie
 					$this->addOutput([
 						'notification' => $notification
@@ -503,7 +483,19 @@ class user extends common {
 				}
 			}
 		}
+		// Journalisation
+		$dataLog = mb_detect_encoding(strftime('%d/%m/%y',time()), 'UTF-8', true)
+				? strftime('%d/%m/%y',time()) . ';' . strftime('%R',time()) . ';'
+				: utf8_encode(strftime('%d/%m/%y',time())) . ';' . utf8_encode(strftime('%R',time())) . ';' ;
+		$dataLog .= helper::getIp() . ';';
+		$dataLog .= $this->getInput('userLoginId', helper::FILTER_ID) . ';' ;
+		$dataLog .= $this->getUrl() .';' ;
+		$dataLog .= $logStatus ;
+		$dataLog .= PHP_EOL;
+		if ($this->getData(['config','connect','log'])) {
+			file_put_contents(self::DATA_DIR . 'journal.log', $dataLog, FILE_APPEND);
 		}
+		// Stockage des cookies
 		if (!empty($_COOKIE['ZWII_USER_ID'])) {
 			self::$userId = $_COOKIE['ZWII_USER_ID'];
 		}

module/blog/blog.php

@@ -81,13 +81,13 @@ class blog extends common {
 	public static $articleConsent = [
 		self::EDIT_ALL 		   => 'Tous les groupes',
 		self::EDIT_GROUP       => 'Groupe du propriétaire',
-		self::EDIT_OWNER       => 'Propiétaire'
+		self::EDIT_OWNER       => 'Propriétaire'
 	];
 
 
 	public static $users = [];
 
-	const BLOG_VERSION = '4.1';
+	const BLOG_VERSION = '4.2';
 
 		/**
 	 * Flux RSS
@@ -627,22 +627,28 @@ class blog extends common {
 					foreach($this->getData(['user']) as $userId => $user) {
 						if ($user['group'] >= $this->getData(['module', $this->getUrl(0),  'posts', $this->getUrl(1), 'commentGroupNotification']) ) {
 							$to[] = $user['mail'];
+							$firstname[] = $user['firstname'];
+							$lastname[] = $user['lastname'];
 						}
 					}
 					// Envoi du mail $sent code d'erreur ou de réussite
 					$notification = $this->getData(['module', $this->getUrl(0),  'posts', $this->getUrl(1), 'commentApproved']) === true ? 'Commentaire déposé en attente d\'approbation': 'Commentaire déposé';
 					if ($this->getData(['module', $this->getUrl(0),  'posts', $this->getUrl(1), 'commentNotification']) === true) {
+						$error = 0;
+						foreach($to as $key => $adress){
 							$sent = $this->sendMail(
-							$to,
+								$adress,
 								'Nouveau commentaire déposé',
-							'Bonjour' . ' <strong>' . $user['firstname'] . ' ' . $user['lastname'] . '</strong>,<br><br>' .
+								'Bonjour' . ' <strong>' . $firstname[$key] . ' ' . $lastname[$key] . '</strong>,<br><br>' .
 								'L\'article <a href="' . helper::baseUrl() . $this->getUrl(0) . '/	' . $this->getUrl(1) . '">' . $this->getData(['module', $this->getUrl(0), 'posts', $this->getUrl(1), 'title']) . '</a> a  reçu un nouveau commentaire.<br><br>',
 								''
 							);
+							if( $sent === false) $error++;
+						}
 						// Valeurs en sortie
 						$this->addOutput([
 							'redirect' => helper::baseUrl() . $this->getUrl() . '#comment',
-							'notification' => ($sent === true ? $notification . '<br/>Une notification a été envoyée.' : $notification . '<br/> Erreur de notification : ' . $sent),
+							'notification' => ($error === 0 ? $notification . '<br/>Une notification a été envoyée.' : $notification . '<br/> Erreur de notification : ' . $sent),
 							'state' => ($sent === true ? true : null)
 						]);
 

site/data/.htaccess

@@ -1,14 +1,5 @@
 # Bloque l'accès aux données
-<Files *.json>
+<FilesMatch ".(htaccess|json|log)$">
 	Order deny,allow
 	Deny from all
-</Files>
+</FilesMatch>
-<Files *.json.back>
-	Order deny,allow
-	Deny from all
-</Files>
-# Bloque l'accès htaccess
-<Files .htaccess>
-	Order deny,allow
-	Deny from all
-</Files>