From 3d243e346346a203e729f233f9f5d57f6f024b37 Mon Sep 17 00:00:00 2001
From: fredtempez <ftempez@gmail.com>
Date: Tue, 8 Mar 2022 13:46:55 +0100
Subject: [PATCH] =?UTF-8?q?Bug=20:=20ordre=20des=20pages=20du=20menu=20sta?=
 =?UTF-8?q?ndard=20a=20r=C3=A9soudre?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 core/module/page/page.php              | 22 +++++++------
 core/module/page/view/edit/edit.js.php | 44 ++++++++------------------
 2 files changed, 27 insertions(+), 39 deletions(-)

diff --git a/core/module/page/page.php b/core/module/page/page.php
index 9a672658..e88d259c 100644
--- a/core/module/page/page.php
+++ b/core/module/page/page.php
@@ -389,19 +389,23 @@ class page extends common {
 					$hierarchy = $this->getInput('pageEditParentPageId') ? $this->getHierarchy($this->getInput('pageEditParentPageId')) : array_keys($this->getHierarchy());
 					$position = $this->getInput('pageEditPosition', helper::FILTER_INT);
 					foreach($hierarchy as $hierarchyPageId) {
-						// Ignore la page en cours de modification et les pages dans le menu extra
-						if($hierarchyPageId === $this->getUrl(2) ||
-						   $this->getData(['page', $hierarchyPageId, 'extraPosition']) === false ) {
+
+						// Ignore la page en cours de modification 
+						if($hierarchyPageId === $this->getUrl(2) ) {
 							continue;
 						}
-						// Incrémente de +1 pour laisser la place à la position de la page en cours de modification
-						if($lastPosition === $position) {
+						// Ne traite que les pages du menu standard
+						if ($this->getData(['page', $hierarchyPageId, 'extraPosition']) === false) {
+							// Incrémente de +1 pour laisser la place à la position de la page en cours de modification
+							if($lastPosition === $position) {
+								$lastPosition++;
+							}
+							// Change la position
+							$this->setData(['page', $hierarchyPageId, 'position', $lastPosition]);
+							// Incrémente pour la prochaine position
 							$lastPosition++;
 						}
-						// Change la position
-						$this->setData(['page', $hierarchyPageId, 'position', $lastPosition]);
-						// Incrémente pour la prochaine position
-						$lastPosition++;
+
 					}
 					if ($this->getinput('pageEditBlock') !== 'bar') {
 						$barLeft = $this->getinput('pageEditBarLeft');
diff --git a/core/module/page/view/edit/edit.js.php b/core/module/page/view/edit/edit.js.php
index 95f08152..e344bd80 100644
--- a/core/module/page/view/edit/edit.js.php
+++ b/core/module/page/view/edit/edit.js.php
@@ -240,16 +240,7 @@ $( document ).ready(function() {
 		 if ($("#pageEditExtraPosition").val() == 1 ) {
 			var positionDOM = $("#pageEditPosition");
 			var positionInitial = <?php echo $this->getData(['page',$this->getUrl(2),"position"]); ?>;
-			console.log(positionInitial);
-			// La position précédente était dans le menu standard
-			if (positionInitial >= 2)
-			{
-				positionInitial = 0;
-			}
-			positionDOM.empty().append(
-				$("<option>").val(0).text("Ne pas afficher"),
-				$("<option>").val(1).text("Au début")
-			);
+			buildPagesList(true);
 			$("#pageEditPosition").val(positionInitial);
 		}
 
@@ -531,18 +522,9 @@ pageTypeMenuDOM.on("change", function() {
 
 $("#pageEditExtraPosition").on("change", function() {
 	if ($("#pageEditExtraPosition").val() == 1 ) {
-
-		var positionDOM = $("#pageEditPosition");
-		/*
-		positionDOM.empty().append(
-			$("<option>").val(0).text("Ne pas afficher"),
-			$("<option>").val(1).text("Au début")
-		);
-		$("#pageEditPosition").val(1);
-		*/
-		getPages(true);
+		buildPagesList(true);
 	} else {
-		getPages(false);
+		buildPagesList(false);
 		//$("#pageEditParentPageId").trigger("change");
 	}
 });
@@ -558,14 +540,14 @@ $("#pageEditModuleConfig").on("click", function() {
  * Affiche les pages en fonction de la page parent dans le choix de la position
  */
 $("#pageEditParentPageId").on("change", function() {
-	getPages(false);
+	buildPagesList(false);
 }).trigger("change");
 
 /**
  * Construit un select contenant la liste des pages du site.
  */
 
-function getPages(extra) {
+function buildPagesList(extraPosition) {
 	var hierarchy = <?php echo json_encode($this->getHierarchy()); ?>;
 	var pages = <?php echo json_encode($this->getData(['page'])); ?>;
 	var positionInitial = <?php echo $this->getData(['page',$this->getUrl(2),"position"]); ?>;
@@ -583,8 +565,7 @@ function getPages(extra) {
 	if(parentSelected === "") {
 		// Liste des pages sans parents
 		for(var key in hierarchy) {
-			if(hierarchy.hasOwnProperty(key) &&
-				extra === false ) {
+			if(hierarchy.hasOwnProperty(key) ) {
 				// Sélectionne la page avant s'il s'agit de la page courante
 				if(key === "<?php echo $this->getUrl(2); ?>") {
 					positionSelected = positionPrevious;
@@ -592,11 +573,14 @@ function getPages(extra) {
 				// Sinon ajoute la page à la liste
 				else {
 					// Enregistre la position de cette page afin de la sélectionner si la prochaine page de la liste est la page courante
-					positionPrevious++;
-					// Ajout à la liste
-					positionDOM.append(
-						$("<option>").val(positionPrevious).html("Après \"" + (pages[key].title) + "\"")
-					);
+					if (extraPosition == pages[key].extraPosition ) {
+						positionPrevious++;
+						// Ajout à la liste
+						positionDOM.append(
+							$("<option>").val(positionPrevious).html("Après \"" + (pages[key].title) + "\"")
+						);
+					}
+
 				}
 			}
 		}