spip-contrib-extensions/saisies | 3 commits
Par RastaPopoulos, le 10 juin 2021 à 14h59min :
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.
(cherry picked from commit 4954ebff2039b93d397e1c9e2561479c6226b575)
Modifié
inc/saisies_afficher_si_php.php
==============================
Par tcharlss, le 10 juin 2021 à 15h13min :
Test plus robuste pour activer l’onglet par défaut (sinon erreur JS dans certains cas)
(cherry picked from commit 0603db45570b46812ec582023904ddcf10d02eda)
Modifié
javascript/saisies.js
==============================
Par RastaPopoulos, le 10 juin 2021 à 15h34min :
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é.
(cherry picked from commit db6e3f3d29c8ca4a833d82b1f4057075a7e8a78a)
Modifié
inc/saisies_afficher_si_php.php