From 654bcb5b4a4db8b44a24ed4c3e0f10b60709a719 Mon Sep 17 00:00:00 2001
From: cy_altern <cy.altern@gmail.com>
Date: Thu, 5 Sep 2024 20:03:21 +0200
Subject: [PATCH] =?UTF-8?q?Fix=20:=20dans=20le=20mode=20logoon/logooff=20n?=
 =?UTF-8?q?e=20pas=20prendre=20en=20compte=20l'arg=20id=5Fdocument=20pour?=
 =?UTF-8?q?=20ne=20pas=20faire=20planter=20la=20gestion=20du=20logo=20par?=
 =?UTF-8?q?=20SPIP=20qui=20supprime=20puis=20recr=C3=A9e=20un=20doc=20lors?=
 =?UTF-8?q?qu'il=20faut=20chnger=20l'image=20d'un=20logo?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 README.md                | 2 +-
 formulaires/bigform.html | 5 +----
 formulaires/bigform.php  | 3 ++-
 3 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/README.md b/README.md
index bfa4953..131de5b 100644
--- a/README.md
+++ b/README.md
@@ -21,7 +21,7 @@
   - `ajaxReload` le nom d'un bloc ajax à recharger via ajaxReload. dans ce cas, charger_js_titre = non
   - `ajaxCallback` le nom d'une fonction JS qui sera appellée en callback de l'ajaxReload (/!\ le JS de cette fonction doit être dans le bloc rechargé)
   - `boutonSubmit` non par défaut (= bouton masqué) => oui permet d'afficher le bouton submit du FORMULAIRE_BIGFORM
-  - `id_document` l'ID du document pour lequel le fichier chargé va remplacer le fichier existant
+  - `id_document` l'ID du document pour lequel le fichier chargé va remplacer le fichier existant (/!\ non pris en compte avec les modes  `logoon | logooff` puisque dans ce cas le document est supprimé puis recréé)
 
     Exemple pour revenir à la page principale "equipe" avec un bigform appelé dans une popin :
 
diff --git a/formulaires/bigform.html b/formulaires/bigform.html
index 70bbf93..4c4caf3 100644
--- a/formulaires/bigform.html
+++ b/formulaires/bigform.html
@@ -4,19 +4,16 @@
 
 		[(#ENV*{message_ok}|oui)
 			<script>
-				//ajaxReload("#ENV*{message_ok}");
-
 				var aa = '#ENV*{message_ok}';
 				if (aa) {
 					try {
 						aa = JSON.parse(aa);
 					} catch (e) {
-						// utiliser le catch pour la compat descendante = ENV{message_ok} est une simple string avec le nom du bloc à recharger 
+						// utiliser le catch pour la compat descendante = lorsque ENV{message_ok} est une simple string avec le nom du bloc à recharger 
 						ajaxReload(aa);
 					}
 				}
 				//console.log(aa);
-
 				if (aa.bloc_ajax !== undefined) {
 					let bb = '';
 					if (aa.callback_ajax !== undefined) {
diff --git a/formulaires/bigform.php b/formulaires/bigform.php
index 59c3d87..2924759 100644
--- a/formulaires/bigform.php
+++ b/formulaires/bigform.php
@@ -59,7 +59,8 @@ function formulaires_bigform_traiter_dist($objet = 'article', $id_objet = 0, $mo
 	if (array_key_exists('obfusquer', $args)) {
 		$options['obfusquer'] = true;
 	}
-	if (array_key_exists('id_document', $args) && intval($args['id_document'])) {
+	// passer l'id_document à remplacer seulement si ce n'est pas un logo (en cas de logo SPIP supprime puis recrée le doc)
+	if (array_key_exists('id_document', $args) && intval($args['id_document']) && !in_array($mode, ['logoon', 'logooff'])) {
 		$options['id_document'] = intval($args['id_document']);
 	}