Changeset 109933 in spip-zone


Ignore:
Timestamp:
Apr 17, 2018, 4:17:25 PM (19 months ago)
Author:
rastapopoulos@…
Message:

Gros bug dans Numérotation, donc nettoyage et commentaire pour mieux comprendre. Quand l'objet n'est pas géré et qu'il n'est pas dans une rubrique (bref quand on ne sait pas son parent), c'était prévu que ça ne le gère pas DU TOUT. Or il manquait un return du false. Du coup des objets non rangeables avaient quand même le formulaire pour 'Classer après'. Vu par exemple sur les organisations, où une personne s'était amusé à valider le formulaire, et du coup ça a modifier les dizaines ou centaines d'orgas avec 30. Truc, 290. Machin, etc (et impossible de les virer puisqu'il n'y a pas de parent avec un bouton Dénuméroter). Dans tous les cas à terme il faudrait que ce plugin utilise la nouvelle API de déclaration du parent pour être enfin vraiment générique.

Location:
_plugins_/numerotation/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • _plugins_/numerotation/trunk/inc/numeroter.php

    r107479 r109933  
    4242                // champ parent
    4343                $parent = "";
    44                 if (isset($desc['field']['id_rubrique']))
     44               
     45                // Si l'objet a un champ id_rubrique, on pré-remplit le champ du parent
     46                if (isset($desc['field']['id_rubrique'])) {
    4547                        $parent = "id_rubrique";
    46                 switch($type){
    47                         case 'rubrique': $parent = "id_parent";break;
    48                         case 'mot': $parent = "id_groupe";break;
    49                         case 'groupe_mots': $parent = "";break;
    50                         // type non prevu ici (et pas range dans une rubrique) : on ne fait rien
     48                }
     49               
     50                // Suivant le type d'objet, on gère des cas particuliers
     51                switch($type) {
     52                        case 'rubrique':
     53                                $parent = "id_parent";
     54                                break;
     55                        case 'mot':
     56                                $parent = "id_groupe";
     57                                break;
     58                        case 'groupe_mots':
     59                                $parent = "";
     60                                break;
     61                        // Type non prevu ici (et pas rangé dans une rubrique) : on ne fait rien, on quitte
    5162                        default :
    52                                 if (!$parent) false;
     63                                if (!$parent) {
     64                                        return false;
     65                                }
    5366                                break;
    5467                }
  • _plugins_/numerotation/trunk/paquet.xml

    r107479 r109933  
    22        prefix="numero"
    33        categorie="navigation"
    4         version="2.1.0"
     4        version="2.1.1"
    55        etat="stable"
    66        compatibilite="[3.0.0;3.2.*]"
Note: See TracChangeset for help on using the changeset viewer.