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…