61 Commits (1738483aa29cb42cfdab92e575f5b168e5158210)

Author SHA1 Message Date
Maïeul 1738483aa2 Correction de PHPdoc pas besoin de vérifier la sécurité du test afficher_si qu'on passe à `saisies_evaluer_afficher_si()`. 3 months ago
Maïeul 2dddf31a07 2b9babe158 avait supprimé, pour une raison que j'ignore, une negation dans un test... 5 months ago
Maïeul 4ef5c67713 fix warning 5 months ago
Maïeul 2b9babe158 Fix bug signalé ici https://discuter.spip.net/t/affichage-conditionnel-de-saisie-syntaxe-des-tests-spip3-2-13/159418 6 months ago
Maïeul 3bb2935fd3 erreur dans un commentaire, sans doute un copier-coller trop rapide 6 months ago
RastaPopoulos c1625bbd2e Partout où il y a des _request ou des set_request, on doit pouvoir passer un tableau de valeurs qui remplace GET/POST puisque ces fonctions de SPIP savent le faire. Ce qui permet alors de vérifier un jeu de valeurs prédéfinis, qu'on a sous la main, et non pas le POST. 12 months ago
RastaPopoulos db6e3f3d29 Encore un cas où ça n'utilisait pas saisies_request() et ça cherche directement une clé dans le tableau, et donc où ça ne marchait pas pour les names en tableau. J'espère qu'il n'y en a plus, normalement c'était l'objet de #29 mais il restait encore des endroits où ça ne l'utilisait pas. Aussi deuxième bug au passage, même sans les names tableau : pour afficher_si ça ne cherchait la valeur que dans l'env valeurs, alors que depuis le départ dans generer_vue ça cherche dans l'env valeurs et sinon dans l'env directement. Du coup incohérence entre les deux. Par compat dans les deux endroits j'ai rajouté une transformation en chaine vide si c'est null, car saisies_request renvoie null quand ça n'existe pas, alors qu'avant à ces endroits ça mettait chaine vide. Ce n'est peutêtre pas utile pourrait être retiré. 1 year ago
RastaPopoulos 4954ebff20 Perf : il n'y a pas à lancer tout une machinerie de parsing, preg_match_all etc sur 5 fonctions d'affilés, pour une chaine qui est dès le départ vide. 1 year ago
Maïeul 86bb6d4582 un compat historique qu'on se permet de casser 1 year ago
Maïeul 64e8f87546 Corriger la verif des afficher_si côté PHP avec `@config:<>@` et 1 year ago
Maïeul 775a7bdc34 Ne vider les saisies masquées que lors que toutes les vérifications ont 1 year ago
Maïeul a23aff05b9 Excursus, `saisies_afficher_si_get_valeur_champ` : fix warning + utiliser fonction définie par saisies pour savoir si l'on a affaire à une saisie de type fichier. 1 year ago
Maïeul 82a795ebec retablir le fonctionnement de l'option globale afficher_si_avec_post 1 year ago
Maïeul 94f09c16b5 retour sur une fausse bonne idée, qui n'aurait pas du passer si j'avais effectué correctement les tests 1 year ago
Maïeul 8004331a38 retour sur 43b1f32 : ne pas remplacer un warning par un autre bien plus grave + modifier un test pour qu'il soit plus clair à comprendre 1 year ago
Maïeul 58d588fdcc oups 1 year ago
Maïeul 43b1f32784 Fix warning sur l'appel à no `saisies_evaluer_afficher_si` avec un 1 year ago
Maïeul 6f2b0a99c9 inutile de chercher une valeur si le champ est de toute facon inexistant 1 year ago
Maïeul ea1e12e94b fix warning en cas d'absence de valeur dans le pour #AFFICHER_SAISIES 1 year ago
Maïeul 98374a9376 Suite discussion IRC + écrit où @rastapopoulos et moi même nous sommes 1 year ago
Maïeul df59050958 un commentaire pour comprendre un truc obscure sinon 1 year ago
Maïeul e3525809a1 coquille PHP doc 1 year ago
Maïeul 1cb0a6cee0 commentaire qui n'a plus lieu d'être 1 year ago
Maïeul 308a0d83a7 coquille PHPDoc 1 year ago
Maïeul 7d3aed98ba pb #78 étape 4. Une fonction `saisies_afficher_si_liste_masquees($action 1 year ago
Maïeul 8dea1b7d63 `saisies_set_empty_string_recursivement($saisie)` => `saisies_set_recursivement($saisie, $val = '')` 1 year ago
Maïeul 5521ff54b9 `saisies_afficher_si_get_valeur_champ()` se base sur `saisies_request()`, 1 year ago
Maïeul 3a1039cff2 Suivant l'idée de @rastapopoulos 1 year ago
Maïeul 65f104c858 Réécriture de la partie JS des afficher-si, pour pouvoir gérer plus 1 year ago
Maïeul d4d67cb9a8 On fait ce qu'on disait en 1 year ago
Maïeul d2b5c9eb38 Afficher_si en php: si on teste sur un champ inexistant, alors on considère que tout le test est bon 1 year ago
Maïeul 9e91c30a45 Verification des afficher_si côté PHP : il faut prendre l'ensemble des 1 year ago
Maïeul 1f461dea2f a677759 provquait parfois un warning 2 years ago
Maïeul a677759cc6 Lors de la verification PHP des affichages conditionnelles des saisies, 2 years ago
Maïeul 8300b8139e Suite discussion avec @rastapopoulous, unifier le nom des options 2 years ago
Maïeul 1991948140 amélioration du phpdoc pour y voir plus clair sur les options saisies_veirfier_afficher_si() 2 years ago
tcharlss b2d559c812 Également une option globale pour conserver les valeurs de toutes les saisies masquées 2 years ago
tcharlss ca8ed167fa Nouvelle option `afficher_si_avec_post` qui permet de conserver les valeurs des saisies cachées. 2 years ago
RastaPopoulos ed8fab3352 Après des jours de recherche… Gros bug avec un énième (c'est récurent au fil des années) oubli de prendre en compte quand les champs ne sont pas des noms simples mais des sous-tableaux. Là c'est quand on cherche à vider les valeurs de request des champs masqués donc désactivés par afficher_si. Ça faisait un set_request() basique, qui ne faisait donc rien si le nom était tableau[index]. Tout comme il y a saisies_request, on fait donc une fonction saisies_set_request qui fait ça proprement et mutualisé. MAIS comme dit dans le ticket #29 ça n'a strictement rien de propre à Saisies ! On devrait donc prendre en compte ces cas directement dans _request() et set_request() du core, d'après moi. 2 years ago
Matthieu Marcillaud 17d5f5349b Notice PHP -- (imbrication de ternary operators) 2 years ago
RastaPopoulos 14fe3a4a3c Dans les tests de condition *en PHP*, on sait moins facilement si on attend un tableau ou pas, suivant comment ça a été stocké en base (pas pareil en Formidable et Champs Extras pour l'instant). Donc quand on récupère la valeur, on teste si c'était une saisie de type tabulaire ou pas puisqu'on sait ici de quelle saisie il s'agit ! Et dans ce cas on force toujours en tableau. Au passage on renomme des variables pour être plus explicite : saisies_par_nom et valeur au lieu de champ 2 years ago
Maïeul b41d105db7 no_arobase : tests strict en se basant sur null, car on peut vouloir comparer une chaine vide 2 years ago
Maïeul 39f5ed53b9 Nouvelle fonction pour les afficher_si (uniquement en PHP) : pouvoir ne pas passer 2 years ago
maieul@maieul.net 9d441c1608 notice 3 years ago
maieul@maieul.net de9ee7bc76 afficher_si pour la saisie cvt_upload. Permet avec la syntaxe @fichiers_1@:TOTAL > 2 d'afficher quelque chose si on a plus de 2 fichiers envoyé 3 years ago
maieul@maieul.net 9bf3d2a0cd afficher_si : nouvelle syntaxe '@checbox_x@:TOTAL > 3', permet de tester si le nombre total de case cochée est supérieur à 3 3 years ago
maieul@maieul.net 824d3b8493 un static qui ne retournait pas le bon résultat + ne prenait pas en compte une récursivité > un static qui n'en vaut pas la peine 3 years ago
maieul@maieul.net ec802fe811 un code qui s'était retrouvé doublonné avec la fusion de branche 3 years ago
maieul@maieul.net 664c52a0ec tester systématiquement la syntaxe des conditions avant d'essayer de les transformer en js/php + accepter les conditions false / true 3 years ago
maieul@maieul.net b4997c033f evaluer_afficher_si (php). Tenir compte des descriptions de type @tableau[a][b]@ 3 years ago