Vérification de la réécriture

- Dans la configuration, l'option Apache URL intelligentes ne s'active que si le serveur est Apache et que le module Rewriter est actif. Ce qui exclue les autres serveurs non compatibles comme Nginx, Caddy etc.

.htaccess

@@ -32,5 +32,13 @@ Options -Indexes
     Options -MultiViews
 </IfModule>
 
+ # Enlever le slash final des URL
+ RewriteCond %{REQUEST_FILENAME} !-f
+ RewriteCond %{REQUEST_FILENAME} !-d
+ RewriteCond %{REQUEST_URI} ^(.+)/$
+ RewriteRule ^ %1 [R=301,L]
+
 # ne pas supprimer la ligne URL rewriting !
+
 # URL rewriting
+

README.md

@@ -1,4 +1,4 @@
-# ZwiiCampus 1.10.00
+# ZwiiCampus 1.10.04
 
 ZwiiCampus (Learning Management System) est logiciel auteur destiné à mettre en ligne des tutoriels. Il dispose de plusieurs modalités d'ouverture et d'accès des contenus. Basé sur la version 13 du CMS Zwii, la structure logicielle est solide, le framework de Zwii est éprouvé.
 

core/class/helper.class.php

@@ -338,13 +338,12 @@ class helper
 	{
 		// N'interroge que le serveur Apache
 		if (strpos($_SERVER["SERVER_SOFTWARE"], 'Apache') > 0) {
-			self::$rewriteStatus === false;
-		} elseif (self::$rewriteStatus === null) {
+			self::$rewriteStatus = false;
+		} else {
 			// Ouvre et scinde le fichier .htaccess
 			$htaccess = explode('# URL rewriting', file_get_contents('.htaccess'));
 			// Retourne un boolean en fonction du contenu de la partie réservée à l'URL rewriting
-			//self::$rewriteStatus = (empty($htaccess[1]) === false);
-			self::$rewriteStatus = (strpos($htaccess[1], 'RewriteEngine on') > 0) ? true : false;
+			self::$rewriteStatus = (strpos($htaccess[1], 'RewriteEngine on') !== false);
 		}
 		return self::$rewriteStatus;
 	}

core/class/layout.class.php

@@ -1016,46 +1016,53 @@ class layout extends common
                     )
                     // Sur une page d'accueil
                     or $this->getUrl(0) === ''
-                ) {
-                    // Bouton Editer une page
-                    if ($this->getUser('permission', 'page', 'edit')) {
-                        $leftItems .= '<li>' . template::ico('pencil', [
-                            'href' => helper::baseUrl() . 'page/edit/' . $this->getUrl(0) . '/' . self::$siteContent,
-                            'help' => 'Éditer la page'
-                        ]) . '</li>';
-                    }
-                    // Bouton Editer le module d'une page
-                    if (
-                        $this->getUser('permission', 'page', 'module')
-                        && $this->getData(['page', $this->getUrl(0), 'moduleId'])
                     ) {
-                        $leftItems .= '<li>' . template::ico('gear', [
-                            'href' => helper::baseUrl() . $this->getUrl(0) . '/config',
-                            'help' => 'Module de la page'
-                        ]) . '</li>';
+                        // Bouton Editer une page
+                        if (
+                            $this->getUser('permission', 'page', 'edit')
+                            and $this->geturl(1) !== 'edit'
+                        ) {
+                            $leftItems .= '<li>' . template::ico('pencil', [
+                                'href' => helper::baseUrl() . 'page/edit/' . $this->getUrl(0) . '/' . self::$siteContent,
+                                'help' => 'Éditer la page'
+                            ]) . '</li>';
+                        }
+                        // Bouton Editer le module d'une page
+                        if (
+                            $this->getUser('permission', 'page', 'module')
+                            and $this->geturl(1) !== 'edit'
+                            and $this->getData(['page', $this->getUrl(0), 'moduleId'])
+                        ) {
+                            $leftItems .= '<li>' . template::ico('gear', [
+                                'href' => helper::baseUrl() . $this->getUrl(0) . '/config',
+                                'help' => 'Module de la page'
+                            ]) . '</li>';
+                        }
+                        // Bouton dupliquer une page
+                        if (
+                            $this->getUser('permission', 'page', 'duplicate')
+                            and $this->geturl(1) !== 'edit'
+                        ) {
+                            $leftItems .= '<li>' . template::ico('clone', [
+                                'href' => helper::baseUrl() . 'page/duplicate/' . $this->getUrl(0) . '/' . self::$siteContent,
+                                'help' => 'Dupliquer la page'
+                            ])
+                                . '</li>';
+                        }
+                        // Bouton Effacer une page
+                        if (
+                            $this->getUser('permission', 'page', 'delete')
+                            and $this->geturl(1) !== 'edit'
+    
+                        ) {
+                            $leftItems .= '<li>' . template::ico('trash', [
+                                'href' => helper::baseUrl() . 'page/delete/' . $this->getUrl(0) . '/' . self::$siteContent,
+                                'help' => 'Supprimer la page',
+                                'id' => 'pageDelete'
+                            ])
+                                . '</li>';
+                        }
                     }
-                    // Bouton dupliquer une page
-                    if (
-                        $this->getUser('permission', 'page', 'duplicate')
-                    ) {
-                        $leftItems .= '<li>' . template::ico('clone', [
-                            'href' => helper::baseUrl() . 'page/duplicate/' . $this->getUrl(0) . '/' . self::$siteContent,
-                            'help' => 'Dupliquer la page'
-                        ])
-                            . '</li>';
-                    }
-                    // Bouton Effacer une page
-                    if (
-                        $this->getUser('permission', 'page', 'delete')
-                    ) {
-                        $leftItems .= '<li>' . template::ico('trash', [
-                            'href' => helper::baseUrl() . 'page/delete/' . $this->getUrl(0) . '/' . self::$siteContent,
-                            'help' => 'Supprimer la page',
-                            'id' => 'pageDelete'
-                        ])
-                            . '</li>';
-                    }
-                }
             }
             // Items de droite
             $rightItems = '';

core/class/router.class.php

@@ -428,7 +428,7 @@ class core extends common
 
 			// Stocke le rapport en CSV
 			$file = fopen(common::DATA_DIR . common::$siteContent . '/report.csv', 'a+');
-			fputcsv($file,  [ $this->getUser('id'), $this->getUrl(0) ,time()], ';');
+			fputcsv($file, [$this->getUser('id'), $this->getUrl(0), time()], ';');
 			fclose($file);
 
 		}
@@ -497,6 +497,14 @@ class core extends common
 			) {
 				$access = false;
 			}
+			// Lève une erreur si l'url est celle d'une page avec des éléments surnuméraires  https://www.site.fr/page/truc
+			if (
+				array_key_exists($this->getUrl(0), $this->getData(['page']))
+				and $this->getUrl(1)
+				and $this->getData(['page', $this->getUrl(0), 'moduleId']) === ''
+			) {
+				$access = false;
+			}
 		}
 
 		/**
@@ -798,7 +806,7 @@ class core extends common
 			if ($accessInfo['userName']) {
 				$this->addOutput([
 					'title' => 'Accès verrouillé',
-					'content' => template::speech('<p>'. sprintf(helper::translate('La page %s est ouverte par l\'utilisateur %s</p><p><a style="color:inherit" href="javascript:history.back()">%s</a></p>'), $accessInfo['pageId'], $accessInfo['userName'], helper::translate('Retour')))
+					'content' => template::speech('<p>' . sprintf(helper::translate('La page %s est ouverte par l\'utilisateur %s</p><p><a style="color:inherit" href="javascript:history.back()">%s</a></p>'), $accessInfo['pageId'], $accessInfo['userName'], helper::translate('Retour')))
 				]);
 			} else {
 				if (
@@ -810,7 +818,7 @@ class core extends common
 				} else {
 					$this->addOutput([
 						'title' => 'Accès interdit',
-						'content' => template::speech('<p>' . helper::translate('Vous n\'êtes pas autorisé à consulter cette page (erreur 403)') . '</p><p><a style="color:inherit" href="javascript:history.back()">'. helper::translate('Retour') . '</a></p>')
+						'content' => template::speech('<p>' . helper::translate('Vous n\'êtes pas autorisé à consulter cette page (erreur 403)') . '</p><p><a style="color:inherit" href="javascript:history.back()">' . helper::translate('Retour') . '</a></p>')
 					]);
 				}
 			}
@@ -818,7 +826,8 @@ class core extends common
 
 			// Pour éviter une 404, bascule dans l'espace correct si la page existe dans cet espace.
 			// Parcourir les espaces y compris l'accueil
-			foreach (array_merge(['home'=> []], $this->getData(['course'])) as $courseId => $value) {;
+			foreach (array_merge(['home' => []], $this->getData(['course'])) as $courseId => $value) {
+				;
 				if (
 					// l'espace existe
 					is_dir(common::DATA_DIR . $courseId) &&
@@ -851,7 +860,7 @@ class core extends common
 				// Page par défaut
 				$this->addOutput([
 					'title' => 'Page indisponible',
-					'content' => template::speech('<p>' . helper::translate('La page demandée n\'existe pas ou est introuvable (erreur 404)') . '</p><p><a style="color:inherit" href="javascript:history.back()">'. helper::translate('Retour') . '</a></p>')
+					'content' => template::speech('<p>' . helper::translate('La page demandée n\'existe pas ou est introuvable (erreur 404)') . '</p><p><a style="color:inherit" href="javascript:history.back()">' . helper::translate('Retour') . '</a></p>')
 				]);
 			}
 		}

core/core.js.php

@@ -462,7 +462,7 @@ $(document).ready(function () {
     /**
      * Chargement paresseux des images et des iframes
      */
-    $("img,picture,iframe").attr("loading", "lazy");
+    $("img").attr("loading", "lazy");
 
     /**
      * Effet accordéon

core/core.php

@@ -51,7 +51,7 @@ class common
 	const ACCESS_TIMER = 1800;
 
 	// Numéro de version
-	const ZWII_VERSION = '1.10.00';
+	const ZWII_VERSION = '1.10.04';
 
 	// URL autoupdate
 	const ZWII_UPDATE_URL = 'https://forge.chapril.org/ZwiiCMS-Team/campus-update/raw/branch/master/';

core/module/config/config.php

@@ -490,7 +490,7 @@ class config extends common
 						'cookiesFooterText' => $this->getInput('configLocaleCookiesFooterText', helper::FILTER_STRING_SHORT, $this->getInput('configCookieConsent', helper::FILTER_BOOLEAN)),
 						'buttonValidLabel' => $this->getInput('configLocaleCookiesButtonText', helper::FILTER_STRING_SHORT, $this->getInput('configCookieConsent', helper::FILTER_BOOLEAN)),
 					],
-						'social' => [
+					'social' => [
 						'facebookId' => $this->getInput('socialFacebookId'),
 						'linkedinId' => $this->getInput('socialLinkedinId'),
 						'instagramId' => $this->getInput('socialInstagramId'),
@@ -556,7 +556,7 @@ class config extends common
 				) {
 					// Ajout des lignes dans le .htaccess
 					$fileContent = file_get_contents('.htaccess');
-					$rewriteData = PHP_EOL .
+					$rewriteData =
 						'# URL rewriting' . PHP_EOL .
 						'<IfModule mod_rewrite.c>' . PHP_EOL .
 						"\tRewriteEngine on" . PHP_EOL .
@@ -565,7 +565,7 @@ class config extends common
 						"\tRewriteCond %{REQUEST_FILENAME} !-d" . PHP_EOL .
 						"\tRewriteRule ^(.*)$ index.php?$1 [L]" . PHP_EOL .
 						'</IfModule>' . PHP_EOL .
-						'# URL rewriting' . PHP_EOL;
+						'# URL rewriting';
 					$fileContent = str_replace('# URL rewriting', $rewriteData, $fileContent);
 					file_put_contents(
 						'.htaccess',
@@ -957,4 +957,19 @@ class config extends common
 			]);
 		}
 	}
+
+	/**
+	 * Fonction pour vérifier la présence du module de réécriture
+	 * @return bool
+	 */
+	public function isModRewriteEnabled() {
+		// Check if Apache and mod_rewrite is loaded
+		if (function_exists('apache_get_modules')) {
+			$modules = apache_get_modules();
+			return in_array('mod_rewrite', $modules);
+		} else {
+			// Fallback if not using Apache or unable to detect modules
+			return getenv('HTTP_MOD_REWRITE') == 'On' || getenv('REDIRECT_STATUS') == '200';
+		}
+	}
 }

core/module/config/view/setup/setup.php

@@ -48,7 +48,7 @@
 						<?php echo template::checkbox('configRewrite', true, 'Apache URL intelligentes', [
 							'checked' => helper::checkRewrite(),
 							'help' => 	'Supprime le point d\'interrogation dans les URL, l\'option est indisponible avec les autres serveurs Web',
-							'disabled' =>  stripos($_SERVER["SERVER_SOFTWARE"], 'nginx')
+							'disabled' =>  stripos($_SERVER["SERVER_SOFTWARE"], 'Apache') === false and $module->isModRewriteEnabled()
 						]); ?>
 					</div>
 				</div>

module/blog/blog.php

@@ -16,7 +16,7 @@
 class blog extends common
 {
 
-	const VERSION = '7.6';
+	const VERSION = '7.9';
 	const REALNAME = 'Blog';
 	const DELETE = true;
 	const UPDATE = '0.0';
@@ -196,9 +196,11 @@ class blog extends common
 		$feeds = new \FeedWriter\RSS2();
 
 		// En-tête
-		$feeds->setTitle($this->getData(['page', $this->getUrl(0), 'title']));
+		$feeds->setTitle($this->getData(['page', $this->getUrl(0), 'title']) ? $this->getData(['page', $this->getUrl(0), 'title']): '');
 		$feeds->setLink(helper::baseUrl() . $this->getUrl(0));
-		$feeds->setDescription($this->getData(['page', $this->getUrl(0), 'metaDescription']));
+		if ($this->getData(['page', $this->getUrl(0), 'metaDescription'])) {
+			$feeds->setDescription($this->getData(['page', $this->getUrl(0), 'metaDescription']));
+		}
 		$feeds->setChannelElement('language', 'fr-FR');
 		$feeds->setDate(date('r', time()));
 		$feeds->addGenerator();
@@ -361,7 +363,7 @@ class blog extends common
 				self::$dateFormat = $this->getData(['module', $this->getUrl(0), 'config', 'dateFormat']);
 				self::$timeFormat = $this->getData(['module', $this->getUrl(0), 'config', 'timeFormat']);
 				self::$comments[] = [
-					helper::dateUTF8(self::$dateFormat, $comment['createdOn']) . ' - ' . helper::dateUTF8(self::$timeFormat, $comment['createdOn']),
+					helper::dateUTF8(self::$dateFormat, $comment['createdOn'], self::$i18nUI) . ' - ' . helper::dateUTF8(self::$timeFormat, $comment['createdOn'], self::$i18nUI),
 					$comment['content'],
 					$comment['userId'] ? $this->getData(['user', $comment['userId'], 'firstname']) . ' ' . $this->getData(['user', $comment['userId'], 'lastname']) : $comment['author'],
 					$buttonApproval,
@@ -530,7 +532,7 @@ class blog extends common
 				'<a href="' . helper::baseurl() . $this->getUrl(0) . '/' . $articleIds[$i] . '" target="_blank" >' .
 				$this->getData(['module', $this->getUrl(0), 'posts', $articleIds[$i], 'title']) .
 				'</a>',
-				helper::dateUTF8(self::$dateFormat, $this->getData(['module', $this->getUrl(0), 'posts', $articleIds[$i], 'publishedOn'])) . ' - ' . helper::dateUTF8(self::$timeFormat, $this->getData(['module', $this->getUrl(0), 'posts', $articleIds[$i], 'publishedOn'])),
+				helper::dateUTF8(self::$dateFormat, $this->getData(['module', $this->getUrl(0), 'posts', $articleIds[$i], 'publishedOn']), self::$i18nUI) . ' - ' . helper::dateUTF8(self::$timeFormat, $this->getData(['module', $this->getUrl(0), 'posts', $articleIds[$i], 'publishedOn']), self::$i18nUI),
 				self::$states[$this->getData(['module', $this->getUrl(0), 'posts', $articleIds[$i], 'state'])],
 				// Bouton pour afficher les commentaires de l'article
 				template::button('blogConfigComment' . $articleIds[$i], [

module/blog/changes.md

@@ -1,3 +1,8 @@
+
+# Versions 7.8 - 7.9
+- Le flux RSS ne fonctionne pas si les méta de la page sont vides.
+# Version 7.7
+- Contrôle de la variable de session liée au contenu. Evite des erreurs lorsque plusieurs onglets sont ouverts.
 # Version 7.6
 - Mise à jour RSS Feed
 # Version 7.5

module/form/changes.md

@@ -1,3 +1,5 @@
+# Version 4.3
+- Contrôle de la variable de session liée au contenu. Evite des erreurs lorsque plusieurs onglets sont ouverts.
 # Version 4.2
 - Termes des commandes de profils
 # Version 4.1

module/form/form.php

@@ -17,7 +17,7 @@
 class form extends common
 {
 
-	const VERSION = '4.2';
+	const VERSION = '4.3';
 	const REALNAME = 'Formulaire';
 	const DATADIRECTORY = ''; // Contenu localisé inclus par défaut (page.json et module.json)
 

module/form/view/config/config.php

@@ -53,7 +53,7 @@
     <div class="col1">
         <?php echo template::button('formConfigBack', [
             'class' => 'buttonGrey',
-			'href' => helper::baseUrl() . 'page/edit/' . $this->getUrl(0)  . '/' . self::$siteContent,
+			'href' => helper::baseUrl() . 'page/edit/' . $this->getUrl(0) . '/' . self::$siteContent,
 			'value' => template::ico('left')
         ]); ?>
 	</div>

module/gallery/changes.md

@@ -1,3 +1,5 @@
+# Version 4.2
+- Contrôle de la variable de session liée au contenu. Evite des erreurs lorsque plusieurs onglets sont ouverts.
 # Version 4.1
 - Termes des commandes de profils
 # Version 4.0

module/gallery/gallery.php

@@ -18,7 +18,7 @@ class gallery extends common
 {
 
 
-	const VERSION = '4.1';
+	const VERSION = '4.2';
 	const REALNAME = 'Galerie';
 	const DATADIRECTORY = self::DATA_DIR . 'gallery/';
 

module/gallery/view/config/config.php

@@ -3,7 +3,7 @@
 	<div class="col1">
 		<?php echo template::button('galleryConfigBack', [
 			'class' => 'buttonGrey',
-			'href' => helper::baseUrl() . 'page/edit/' . $this->getUrl(0)  . '/' . self::$siteContent,
+			'href' => helper::baseUrl() . 'page/edit/' . $this->getUrl(0) . '/' . self::$siteContent,
 			'value' => template::ico('left')
 		]); ?>
 	</div>

module/news/changes.md

@@ -1,3 +1,7 @@
+# Versions 5.7 - 5.8
+- Le flux RSS ne fonctionne pas si les méta de la page sont vides.
+# Version 5.6
+- Contrôle de la variable de session liée au contenu. Evite des erreurs lorsque plusieurs onglets sont ouverts.
 # Version 5.5
 - Mise à jour RSS Feed
 # Version 5.4

module/news/news.php

@@ -16,7 +16,7 @@
 class news extends common
 {
 
-	const VERSION = '5.5';
+	const VERSION = '5.8';
 	const REALNAME = 'News';
 	const DATADIRECTORY = self::DATA_DIR . 'news/';
 
@@ -124,9 +124,11 @@ class news extends common
 		$feeds = new \FeedWriter\RSS2();
 
 		// En-tête
-		$feeds->setTitle($this->getData(['page', $this->getUrl(0), 'title']));
+		$feeds->setTitle($this->getData(['page', $this->getUrl(0), 'title']) ? $this->getData(['page', $this->getUrl(0), 'title']): '');
 		$feeds->setLink(helper::baseUrl() . $this->getUrl(0));
-		$feeds->setDescription($this->getData(['page', $this->getUrl(0), 'metaDescription']));
+		if ($this->getData(['page', $this->getUrl(0), 'metaDescription'])) {
+			$feeds->setDescription($this->getData(['page', $this->getUrl(0), 'metaDescription']));
+		};
 		$feeds->setChannelElement('language', 'fr-FR');
 		$feeds->setDate(date('r', time()));
 		$feeds->addGenerator();

module/news/view/config/config.php

@@ -3,7 +3,7 @@
 		<div class="col1">
 			<?php echo template::button('newsConfigBack', [
 				'class' => 'buttonGrey',
-				'href' => helper::baseUrl() . 'page/edit/' . $this->getUrl(0). '/' . self::$siteContent,
+				'href' => helper::baseUrl() . 'page/edit/' . $this->getUrl(0) . '/' . self::$siteContent,
 				'value' => template::ico('left')
 			]); ?>
 		</div>

module/redirection/changes.md

@@ -1,3 +1,5 @@
+# Version 2.2
+- Contrôle de la variable de session liée au contenu. Evite des erreurs lorsque plusieurs onglets sont ouverts.
 # Version 2.1
 - Edition du module ou de la page impossible.
 # Version 2.0

module/redirection/redirection.php

@@ -16,7 +16,7 @@
 class redirection extends common
 {
 
-	const VERSION = '2.1';
+	const VERSION = '2.2';
 	const REALNAME = 'Redirection';
 	const DATADIRECTORY = ''; // Contenu localisé inclus par défaut (page.json et module.json)
 

module/redirection/view/config/config.php

@@ -3,7 +3,7 @@
 	<div class="col1">
 		<?php echo template::button('redirectionConfigBack', [
 			'class' => 'buttonGrey',
-			'href' => helper::baseUrl() . 'page/edit/' . $this->getUrl(0)  . '/' . self::$siteContent,
+			'href' => helper::baseUrl() . 'page/edit/' . $this->getUrl(0) . '/' . self::$siteContent,
 			'value' => template::ico('left')
 		]); ?>
 	</div>

module/redirection/view/index/index.js.php

@@ -19,7 +19,7 @@
   	core.confirm(
   		message,
    		function() {
-  			$(location).attr("href", "<?php echo helper::baseUrl(); ?>page/edit/<?php echo $this->getUrl(0)  . '/' . self::$siteContent; ?>");
+  			$(location).attr("href", "<?php echo helper::baseUrl(); ?>page/edit/<?php echo $this->getUrl(0); ?>");
   		},
   		function() {
   			$(location).attr("href", "<?php echo helper::baseUrl() . $this->getUrl(); ?>/force");

module/search/changes.md

@@ -1,3 +1,5 @@
+# Version 3.2
+- Contrôle de la variable de session liée au contenu. Evite des erreurs lorsque plusieurs onglets sont ouverts.
 # Version 3.1
 - Initialise un module installé dans une page sans avoir ouvert la page de configuration sans lancer de mise à jour.
 # Version 3.0

module/search/search.php

@@ -20,7 +20,7 @@
 class search extends common
 {
 
-	const VERSION = '3.1';
+	const VERSION = '3.2';
 	const REALNAME = 'Recherche';
 	const DATADIRECTORY = self::DATA_DIR . 'search/';
 

module/slider/changes.md

@@ -1,3 +1,5 @@
+# Version 6.5
+- Contrôle de la variable de session liée au contenu. Evite des erreurs lorsque plusieurs onglets sont ouverts.
 # Version 6.4
 - Corrige plusieurs bugs dans les fonctions de tri
 # Version 6.3

module/workshop/view/index/index.php

@@ -1,5 +1,8 @@
 <?php $startRow = 0; ?>
-<?php foreach ($this->getData(['course']) as $courseId => $courseValue): ?>
+<?php $c = $this->getData(['course']);?>
+<?php $titles = array_column($c, 'title'); ?>
+<?php array_multisort($titles, SORT_ASC, $c); ?>
+<?php foreach ($c as $courseId => $courseValue): ?>
     <!-- Filtre de catégorie -->
     <?php if (
         $this->getData(['module', $this->getUrl(0), 'config', 'category']) !== 'all' &&

module/workshop/workshop.php

@@ -153,13 +153,6 @@ class workshop extends common
 			'view' => 'index'
 		]);
 
-
-		// Valeurs en sortie
-		$this->addOutput([
-			'showBarEditButton' => true,
-			'showPageContent' => true,
-			'view' => 'index',
-		]);
 	}
 
 	// Initialise avec des valeurs par défaut