Changeset 115298 in spip-zone


Ignore:
Timestamp:
May 9, 2019, 3:43:06 PM (13 days ago)
Author:
nicod@…
Message:

Si on met un retour chariot (donc une ligne vide) dans Affichage conditionnel, ça plante tout le javascript généré (en gros ça ferme directement un if ouvrant).
Ajout d'un trim pour éviter ces cas là (et suppression du deuxième isset, il l'est forcément)
Merci Anne-Marie pour le signalement :)

Location:
_plugins_/saisies/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • _plugins_/saisies/trunk/inc/saisies_afficher_si.php

    r115100 r115298  
    4242                // parce que conteneur_class = 'tableau[nom][option]' ne fonctionne evidement pas
    4343                // lorsque le name est un tableau
    44                 if (isset($saisie['options']['afficher_si'])) {
     44                if (isset($saisie['options']['afficher_si']) && trim($saisie['options']['afficher_si'])) {
    4545                        ++$i;
    4646                        // Les [] dans le nom de la saisie sont transformés en _ dans le
     
    6363                                        $class_li = 'editer_'.$nom_underscore;
    6464                        }
    65                         $condition = isset($saisie['options']['afficher_si']) ? $saisie['options']['afficher_si'] : '';
     65                        $condition = trim($saisie['options']['afficher_si']);
    6666                        // retrouver l'identifiant
    6767                        $identifiant = '';
  • _plugins_/saisies/trunk/paquet.xml

    r115296 r115298  
    22        prefix="saisies"
    33        categorie="outil"
    4         version="3.19.4"
     4        version="3.19.5"
    55        etat="stable"
    66        compatibilite="[3.0.0;3.2.*]"
Note: See TracChangeset for help on using the changeset viewer.