Projet

Général

Profil

Development #58916

Création d'un élément de panier: Il n'est pas possible d'utiliser "envoyer les données du formulaire" et de surcharger les options de création en même temps

Ajouté par Benjamin Dauvergne il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Rejeté
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
23 novembre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Il me semblait que c'était possible avant, qu'on faisait request_body.update(extra) mais je ne retrouve pas de code correspondant dans l'historique de lingo, peut-être que ça n'a toujours été supporté que pour 'amount'.

Je propose de systématiser cela en faisant if extra: request_body.update(extra), histoire de simplifier tout ça, il faudrait faire le tour des appels à add-basket-item un peu partout pour s'assurer de ne rien casser (surcharge de display_name qui n'a jamais marché et qui se mettrait à marcher par exemple changeant le titre habituel des items). Ça pourrait aussi casser des appels qui supposent la somme de plusieurs sources pour amount : le code actuel fait la somme de request.GET.getlist('amount'), puis request_body['amount'] et enfin request_body['extra']['amount'].

C'est un peu compliqué, l'API est nulle tant pis, recherchons simplement display_name et reference_id aussi dans extra.

Historique

#2

Mis à jour par Frédéric Péters il y a plus de 2 ans

Je serais plutôt pour considérer le "envoyer les données du formulaire" comme déprécié, et de là ne pas ajouter de prise en charge supplémentaire.

#3

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

  • Tags accessible supprimé
  • Statut changé de Nouveau à Rejeté

Ça me va, c'était mon premier élan mais j'ai quand même voulu vérifier.

Formats disponibles : Atom PDF