Changeset 88917 in spip-zone


Ignore:
Timestamp:
Apr 30, 2015, 8:24:40 AM (4 years ago)
Author:
marcimat@…
Message:

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 !)

Location:
_plugins_/fabrique/trunk
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • _plugins_/fabrique/trunk/changelog.txt

    r87398 r88917  
    33-----------------------------------
    44
     5
     62.0.16 le 30 04 2015
     7--------------------
     8
     9* Correction du traitement des images des objets éditoriaux lors de la
     10  suppression d'un objet éditorial dans la fabrique (et certainement
     11  d'autres problèmes au passage lors de la suppession d'un objet).
     12  Les logos ne devraient plus se mélanger les pinceaux lorsqu'on en ajoute
     13  ou supprime.
    514
    6152.0.12 le 10 02 2015
  • _plugins_/fabrique/trunk/formulaires/fabriquer_plugin_actions.php

    r88916 r88917  
    9494                        $images = session_get(FABRIQUE_ID_IMAGES);
    9595                        unset($data['objets'][$i]);
    96                         // supprimer les images des objets devenus inexistants
     96                        unset($images['objets'][$i]);
     97                        // + supprimer les images des objets devenus inexistants
     98                        //   si cela n'avait pas été fait (ancien bug).
    9799                        foreach ($images['objets'] as $i => $im) {
    98100                                if (!isset($data['objets'][$i])) {
     
    100102                                }
    101103                        }
    102                         array_values($data['objets']);
    103                         array_values($images['objets']);
     104                        $data['objets']   = array_values($data['objets']);
     105                        $images['objets'] = array_values($images['objets']);
    104106                        session_set(FABRIQUE_ID, $data);
    105107                        session_set(FABRIQUE_ID_IMAGES, $images);
     
    138140                                        unset ($images[$type][ $i[0] ][ $i[1] ]); // paquet/logo/0
    139141                                } elseif (count($i) == 3) {
    140                                         unset ($images[$type][ $i[0] ][ $i[1] ][ $i[2] ]); // obje/x/logo/0
     142                                        unset ($images[$type][ $i[0] ][ $i[1] ][ $i[2] ]); // objet/x/logo/0
    141143                                }
    142144                        }
  • _plugins_/fabrique/trunk/formulaires/inc-fabriquer_plugin_objet.html

    r84673 r88917  
    5555        [(#ENV{objets/#CLE/logo/0/fichier}|oui)
    5656                <div class='center'>
    57                 [(#ENV{objets/#CLE/logo/0/fichier}|fabrique_miniature_image{128})]
     57                        [(#ENV{objets/#CLE/logo/0/fichier}|fabrique_miniature_image{128})]
    5858                        <input type='submit' name='f_action\[supprimer_logo\]\[objets/#CLE/logo/0\]' class='submit del right action' value='<:fabrique:bouton_supprimer_logo:>' />
    5959                </div>
  • _plugins_/fabrique/trunk/paquet.xml

    r88916 r88917  
    22        prefix="fabrique"
    33        categorie="outil"
    4         version="2.0.15"
     4        version="2.0.16"
    55        etat="test"
    66        compatibilite="[3.1.0-dev;3.1.*]"
Note: See TracChangeset for help on using the changeset viewer.