Development #42653
Simplification du filtre add
0%
Description
Report des simplifications faites sur combo,
pour faciliter la maintenance des filtres entre les deux briques.
Fichiers
Demandes liées
Historique
Mis à jour par Nicolas Roche il y a environ 4 ans
- Lié à Development #42062: permettre les manipulation de chaine avec |add ajouté
Mis à jour par Nicolas Roche il y a environ 4 ans
- Lié à Development #41868: Ajouter la conversion décimal sur les fitres mathématiques par défaut ajouté
Mis à jour par Nicolas Roche il y a environ 4 ans
- Lié à Bug #42650: Erreur sur le filtre add ajouté
Mis à jour par Nicolas Roche il y a environ 4 ans
- Fichier 0001-templatetags-simplify-add-filter-42653.patch 0001-templatetags-simplify-add-filter-42653.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a environ 4 ans
Pour commencer, ce patch, il change des comportements qui ne seraient pas couverts par les tests ?
(l'idée étant qu'il vaut mieux les exprimer maintenant, que ça puisse potentiellement être arrêté dès maintenant, plutôt qu'en constatant que ça cache quelque chose après l'avoir déployé en recette).
Mis à jour par Frédéric Péters il y a environ 4 ans
Et tout ça fait référence à #42650 que je ne comprends pas non plus...
Mis à jour par Nicolas Roche il y a environ 4 ans
- Fichier 0001-templatetags-simplify-add-filter-42653.patch 0001-templatetags-simplify-add-filter-42653.patch ajouté
Non, ce patch ne doit pas ne doit pas changer le comportement du filtre add.
J'ai rajouté des tests avec la valeur 0, mais ils ne cassaient rien.
Je ne fais que reporter (sur wcs) les simplifications faites sur #42650 (combo).
Mis à jour par Frédéric Péters il y a environ 4 ans
Non, ce patch ne doit pas ne doit pas changer le comportement du filtre add.
C'est compliqué quand le patch modifie autant les tests; je préférerais éviter. (oui je comprends bien que c'est pour reprendre à l'identique les tests mis dans combo).
Je ne fais que reporter (sur wcs) les simplifications faites sur #42650 (combo).
Il ne faut pas oublier qu'il y a actuellement des conditions/gabarits utilisant le |add mathématique dans w.c.s., qui est une considération qu'il n'y a pas à avoir dans combo (où il y a par contre considération à avoir pour les utilisations actuelles du |add natif); que toutes les situations se trouvent couvertes par le même code, ce serait super, mais ça n'est pas nécessairement possible.
Mis à jour par Thomas Noël il y a presque 4 ans
Le code actuel du add dans wcs fonctionne bien, il n'est pas le même que dans combo, c'est l'histoire qui veut ça, tant pis et pas si grave.
Nicolas, je te propose de rejeter ce ticket plutôt que de prendre le moindre risque sur l'existant.