Valider 604915a9 rédigé par marcimat@rezo.net's avatar marcimat@rezo.net
Parcourir les fichiers

J'ai trouvé la source du bug : ne pas croire que array_values modifie le...

J'ai trouvé la source du bug : ne pas croire que array_values modifie le tableau d'entrée… il le retourne !
Corrige les problèmes relatifs à l'insertion ou suppression de logos après qu'on ait supprimé un objet éditorial
et d'autres bugs aussi après cette action (notamment, l'action 'pre construire' qui pouvait s'appliquer sur un autre objet que
celui qu'on éditait !)
parent 34979623
Chargement en cours
Chargement en cours
Chargement en cours
Chargement en cours
+9 −0
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -3,6 +3,15 @@ Ce plugin génère d'autres plugins !
-----------------------------------


2.0.16 le 30 04 2015
--------------------

* Correction du traitement des images des objets éditoriaux lors de la
  suppression d'un objet éditorial dans la fabrique (et certainement
  d'autres problèmes au passage lors de la suppession d'un objet).
  Les logos ne devraient plus se mélanger les pinceaux lorsqu'on en ajoute
  ou supprime.

2.0.12 le 10 02 2015
--------------------

+6 −4
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -93,14 +93,16 @@ function fabrique_action_modification_formulaire($f_action, $data) {
			// on supprime l'element i de l'objet et des images.
			$images = session_get(FABRIQUE_ID_IMAGES);
			unset($data['objets'][$i]);
			// supprimer les images des objets devenus inexistants
			unset($images['objets'][$i]);
			// + supprimer les images des objets devenus inexistants
			//   si cela n'avait pas été fait (ancien bug).
			foreach ($images['objets'] as $i => $im) {
				if (!isset($data['objets'][$i])) {
					unset($images['objets'][$i]);
				}
			}
			array_values($data['objets']);
			array_values($images['objets']);
			$data['objets']   = array_values($data['objets']);
			$images['objets'] = array_values($images['objets']);
			session_set(FABRIQUE_ID, $data);
			session_set(FABRIQUE_ID_IMAGES, $images);
			// on supprime l'accordion ouvert (sinon il se reouvre sur l'onglet suppression).
@@ -137,7 +139,7 @@ function fabrique_action_modification_formulaire($f_action, $data) {
				if (count($i) == 2) {
					unset ($images[$type][ $i[0] ][ $i[1] ]); // paquet/logo/0
				} elseif (count($i) == 3) {
					unset ($images[$type][ $i[0] ][ $i[1] ][ $i[2] ]); // obje/x/logo/0
					unset ($images[$type][ $i[0] ][ $i[1] ][ $i[2] ]); // objet/x/logo/0
				}
			}
			session_set(FABRIQUE_ID_IMAGES, $images);
+1 −1
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
<paquet
	prefix="fabrique"
	categorie="outil"
	version="2.0.15"
	version="2.0.16"
	etat="test"
	compatibilite="[3.1.0-dev;3.1.*]"
	logo="prive/themes/spip/images/fabrique-64.png"
+1 −1

Fichier modifié.

Contient uniquement des modifications d'espaces.