Bug page null dans les modules

CHANGES.md

@@ -1,6 +1,9 @@
 # Changelog
 
 ## version 10.4.00
+Corrections :
+    - Bug de pages non accessibles dans le menu suite à la création d'entrées fantômes dans le fichier de données de modules.
+    - Nettoyage du fichiers des données de modules des entrées fantômes.
 - Modifications :
     - Filtrage des URL générées par facebook (FBCLID) occasionnant une erreur 404
     - Captcha arithmétique, activation recommandée dans la configuration.

core/core.php

@@ -44,7 +44,7 @@ class common {
 	const ACCESS_TIMER = 1800;
 
 	// Numéro de version
-	const ZWII_VERSION = '10.4.00.013';
+	const ZWII_VERSION = '10.4.00.014';
 	const ZWII_UPDATE_CHANNEL = "v10";
 
 	public static $actions = [];
@@ -1499,12 +1499,8 @@ class common {
 			foreach ($pageList as $parentKey => $parent) {
 				//La page est un blog
 				if ($this->getData(['page',$parent,'moduleId']) === 'blog' ) {
-					//echo "<pre>".$parent;
 					$articleIds = array_keys(helper::arrayCollumn($this->getData(['module', $parent, 'posts']), 'publishedOn', 'SORT_DESC'));
-					//var_dump($articleIds);
 					foreach ($articleIds as $key => $article) {
-						//echo $article;
-						//echo "<p>";
 						// Droits les deux groupes
 						$this->setData(['module',  $parent, 'posts', $article,'editConsent', 3]);
 						// Limite de taille 500
@@ -1542,7 +1538,6 @@ class common {
 				$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));
 					}
@@ -1556,6 +1551,16 @@ class common {
 				unlink('module/search/ressource/vartheme.css');
 			$this->deleteData(['theme','search','keywordColor']);
 
+			// Nettoyer les modules avec des données null
+
+			$modules = $this->getData(['module']);
+			foreach($modules as $key => $value) {
+				if (is_null($value) ) {
+					unset($modules[$key]);
+				}
+			}
+			$this->setData (['module',$modules]);
+
 			$this->setData(['core', 'dataVersion', 10400]);
 
 			/**

core/module/install/ressource/defaultdata.php

@@ -781,17 +781,7 @@ class init extends common {
 						'values' => ''
 					]
 				]
-			],
-			'locale' => [
-				'homePageId' => 'accueil',
-				'page302' => 'erreur302',
-				'page403' => 'erreur403',
-				'page404' => 'erreur404',
-				'legalPageId' => 'mentions-legales',
-				'searchPageId' => 'recherche',
-				'metaDescription' => 'Zwii est un CMS sans base de données qui permet de créer et gérer facilement un site web sans aucune connaissance en programmation.',
-				'title' => 'Votre site en quelques clics !'
-			],
+			]
 		]
     ];
 }

core/module/page/page.php

@@ -317,8 +317,10 @@ class page extends common {
 						$this->setData(['page', $childrenPageId, 'parentPageId', $pageId]);
 					}
 					// Change l'id de page dans les données des modules
-					$this->setData(['module', $pageId, $this->getData(['module', $this->getUrl(2)])]);
-					$this->deleteData(['module', $this->getUrl(2)]);
+					if ($this->getData(['module', $this->getUrl(2)]) !== null ) {
+						$this->setData(['module', $pageId, $this->getData(['module', $this->getUrl(2)])]);
+						$this->deleteData(['module', $this->getUrl(2)]);
+					}
 					// Si la page correspond à la page d'accueil, change l'id dans la configuration du site
 					if($this->getData(['locale', 'homePageId']) === $this->getUrl(2)) {
 						$this->setData(['locale', 'homePageId', $pageId]);