[paniers] Dans le pipeline panier2commande_prix, on ajoute la cle (...)

Salut Rasta,

avec ton mail, je ne peux pas faire repondre à tous, car tu dois 
répondre au groupe : gmane.comp.web.spip.devel.

Pour moi, cette fonctionnalité ne pose aucun problème de compatibilité 
avec le plugin et n'apporte pas de problème de sécurité.

Il ne m'a pas semblé indispensable de faire un PR, sinon il faut 3 fois 
plus de temps pour coder une fonctionnalité. Il me semble que tu ne fais 
pas des PR pour tous tes commits.

L'objet de ce commit :

j'ai besoin d'avoir l'information que le prix a été modifié dans le 
detail d'une commande. Pour éviter de refaire le calcul, j'ai simplement 
ajouté cette fonctionnalité.

1. On récupère cette info dans le pipeline panier2commande_prix

2. On le ré injecte dans le pipeline post_edition : 
remplir_commande_details (OK, je vais enlever le s à detail)

J'ai opté pour faire passer dans ce nouveau pipeline le plus 
d'information, ca se discute, mais cela peut eviter de remodifier le 
plugin par la suite.

Ok, c'est une erreur l'ajout dans le pipeline remplir_commande






Le 18/02/2021 à 09:28, RastaPopoulos a écrit :
> Le 18/02/2021 à 08:45, tofulm a écrit :
>> Dans le pipeline panier2commande_prix, on ajoute la cle : prix_modifie.
>> On ajoute aussi un pipeline post_edition : remplir_commande_details, qui
>> se declenche apres le modification de la table spip_commandes_details,
>> en lui faisant passer en data la cle : prix_modifie.
>> Cela peut permettre d'indiquer dans la table spip_commandes_liens si 
>> le prix a été modifié.
> Yo,
> dis, quand il s'agit d'ajout fonctionnel non bénin, est-ce que tu 
> pourrais créer un ticket et faire une PR pour expliquer la 
> problématique ? Parce que là on sait pas ce que c'est censé résoudre, 
> on sait pas ce que ça fait, ni comment ça s'utilise, ce qu'on doit y 
> mettre dedans et pourquoi… et ça se trouve on aurait défini une autre 
> solution si plusieurs personnes avaient relu. Ça vaut pour le plugin 
> Prix aussi. Les plugins de commerce c'est quand même sensible, donc on 
> devrait toujours avoir de la relecture (c'est d'ailleurs pour ça que 
> Bank est pas dans le commun).
>
> Là par exemple :
> - c'est quoi "prix_modifie", on sait pas pourquoi cet ajout et avec 
> quoi ça doit se remplir, dans quel but
> - l'objet c'est commandes_detail et c'est pour un unique détail donc 
> l'action devrait être "remplir_commandes_detail" et non pas 
> "remplir_commande_details"
> - tu fusionnes avec tout $prix_pipeline, ce qui fait que les champs 
> contiennent aussi "prix" et "prix_ht" et non pas juste "prix_modifie", 
> alors même que commandes_detail a déjà un champ "prix_unitaire_ht", et 
> bref on sait jamais ça pourrait rendre confus et causer des problèmes, 
> vu qu'on n'est pas censé avoir ces champs là
> - tu remets ce 'prix_modifie' dans le pipeline qui concerne la 
> commande entière, alors que cette valeur est remplie dans une boucle 
> uniquement *détail par détail* du panier, donc en fait ça y mets juste 
> la dernière valeur de la boucle, ça n'a il me semble aucun sens, aucun 
> rapport avec la commande entière
>
> Bref, en l'état, ça me parait pas super à commiter ça comme ça dans le 
> plugin direct…
>
_______________________________________________
liste: https://listes.rezo.net/mailman/listinfo/spip-dev
doc: https://www.spip.net/
dev: https://core.spip.net/
irc://irc.freenode.net/spip