News tester update

module/news/news.php

@@ -168,6 +168,11 @@ class news extends common {
 		// Mise à jour des données de module
 		$this->update();
 
+		// Initialisation d'un nouveau module
+		if ($this->getData(['module', $this->getUrl(0)]) === null) {
+			$this->init($this->getUrl(0));
+		}
+
 		// Soumission du formulaire
 		if($this->isPost()) {
 
@@ -179,17 +184,21 @@ class news extends common {
 			}
 
 			$success = file_put_contents(self::DATADIRECTORY . $this->getUrl(0) . '.css' , $style );
+
 			// Fin feuille de style
 
+			$this->setData(['module', $this->getUrl(0), 'theme',[
+				'style' => $success ? self::DATADIRECTORY . $this->getUrl(0) . '.css' : '',
+				'itemsHeight' => $this->getInput('newsConfigItemsHeight',helper::FILTER_STRING_SHORT)
+			]]);
+
 			$this->setData(['module', $this->getUrl(0), 'config',[
 				'feeds' 	 => $this->getInput('newsConfigShowFeeds',helper::FILTER_BOOLEAN),
 				'feedsLabel' => $this->getInput('newsConfigFeedslabel',helper::FILTER_STRING_SHORT),
 				'itemsperPage' => $this->getInput('newsConfigItemsperPage', helper::FILTER_INT,true),
 				'itemsperCol' => $this->getInput('newsConfigItemsperCol', helper::FILTER_INT,true),
-				'itemsHeight' => $this->getInput('newsConfigItemsHeight',helper::FILTER_STRING_SHORT),
+				'versionData' => $this->getData(['module', $this->getUrl(0), 'config', 'versionData'])
-				'versionData' => $this->getData(['module', $this->getUrl(0), 'config', 'versionData']),
+			]]);
-				'style' => $success ? self::DATADIRECTORY . $this->getUrl(0) . '.css' : ''
-				]]);
 			// Valeurs en sortie
 			$this->addOutput([
 				'redirect' => helper::baseUrl() . $this->getUrl(0) . '/config',
@@ -337,6 +346,13 @@ class news extends common {
 
 		// Mise à jour des données de module
 		$this->update();
+
+		// Initialisation d'un nouveau module
+		if ($this->getData(['module', $this->getUrl(0)]) === null) {
+			$this->init($this->getUrl(0));
+		}
+
+
 		// Affichage d'un article
 		if(
 			$this->getUrl(1)
@@ -423,45 +439,37 @@ class news extends common {
 	 */
 	private function update() {
 
-		// Initialisation du thème du nouveau module
-		$this->initCss($this->getUrl(0));
-
 		// Version 3.0
-		if (version_compare($this->getData(['module', $this->getUrl(0), 'config', 'versionData']), '3.0', '<') ) {
+		if ($this->getData(['module', $this->getUrl(0), 'config']) === NULL ) {
-			$this->setData(['module', $this->getUrl(0), 'config', 'itemsperPage', 16]);
+			// Données config et theme absentes du précédent module
-			$this->setData(['module', $this->getUrl(0), 'config', 'itemsperCol', 6]);
+			$this->init($this->getUrl(0));
-			$this->setData(['module', $this->getUrl(0), 'config', 'versionData','3.0']);
-
 		}
 	}
 
 	/**
 	 * Initialisation du thème d'un nouveau module
 	 */
-	private function initCSS($moduleId) {
+	private function init($moduleId) {
 		// Variable commune
 		$fileCSS = self::DATADIRECTORY  . $moduleId . '.css' ;
 
-		// Absence des données CSS
+		// Données du module 
-		if ( $this->getData(['module', $moduleId, 'config', 'itemsHeight']) === null ) {
+		require_once('module/news/ressource/defaultdata.php');
+		$this->setData(['module', $moduleId, 'config',init::$defaultData ]);
+		// Données de thème
+		$this->setData(['module', $moduleId, 'theme',init::$defaultTheme ]);
 
-			$this->setData(['module', $moduleId, 'config', 'itemsHeight', '200px']);
+		// Générer la feuille de CSS
+		$style = '.newsContent {height: ' . $this->getData([ 'module', $moduleId, 'theme', 'itemsHeight' ]) .';}';
+		// Dossier de l'instance
+		if (!is_dir(self::DATADIRECTORY)) {
+			mkdir (self::DATADIRECTORY, 0777, true);
 		}
-		// Absence de la feuille de style
-		if (!file_exists(self::DATADIRECTORY . $moduleId . '.css') ) {
-			// Générer la feuille de CSS
-			$style = '.newsContent {height: 200px;}';
 
-			// Dossier de l'instance
+		// Sauver la feuille de style
-			if (!is_dir(self::DATADIRECTORY)) {
+		file_put_contents(self::DATADIRECTORY .$moduleId . '.css' , $style );
-				mkdir (self::DATADIRECTORY, 0777, true);
-			}
 
-			// Sauver la feuille de style
+		// Stocker le nom de la feuille de style
-			$success = file_put_contents(self::DATADIRECTORY .$moduleId . '.css' , $style );
+		$this->setData(['module', $moduleId, 'theme', 'style', self::DATADIRECTORY . $moduleId . '.css']);
-
-			// Nom de la feuille de style
-			$this->setData(['module', $moduleId, 'config', 'style', self::DATADIRECTORY . $moduleId . '.css']);
-		}
 	}
 }

module/news/ressource/defaultdata.php

@@ -0,0 +1,13 @@
+<?php
+class init extends search {
+    public static $defaultData = [
+        'feeds'              => false,
+        'feedsLabel'         => '',
+        'itemsperPage'       => 16,
+        'itemsperCol'        => 6,
+        'versionData'        => '3.0'
+    ];
+    public static $defaultTheme = [
+        'itemsHeight'        => '200px',
+    ];
+}

module/news/view/config/config.php

@@ -55,7 +55,7 @@
 					<div class="col4">
 						<?php echo template::select('newsConfigItemsHeight', $module::$ItemsHeight, [
 							'label' => 'Hauteur',
-							'selected' => $this->getData(['module', $this->getUrl(0),'config', 'itemsHeight'])
+							'selected' => $this->getData(['module', $this->getUrl(0),'theme', 'itemsHeight'])
 						]); ?>
 					</div>
 				</div>