Produits Entr'ouvert » Combo

Voilà c'est juste pour savoir si l'idée c'est ça (ou pas du tout).
Question aussi: cette fonction a l'air d'être appelée en POST pour on récupère les paramètres dans request.GET ?
Il y a peut-être une subtilité que je n'ai pas saisie.

Un id ça doit forcément être un entier ou convertible en entier, donc ça risque de foirer si on y met un slug, je dirai qu'il faudrait plutôt prévoir un paramètre 'regie_slug'. Aussi il faudrait vérifier que regie_id est bien un entier avant de s'en servir, là on a un traceback potentiel sur une erreur.

raise Exception ça va faire un traceback aussi, faut renvoyer une HttpResponseBadRequest au pire.

Je mettrai item.regie = Regie.objects.get(id=regie_id) après le except, c'est plus clair que ça ne concerne que le int() et sait-on jamais si Regies.objects.get() se mettait à lever des ValueError un jour on le verrait pas.

Perso de plus en plus je n'ai pas envie d'expliquer "slug" et traduit/présente ça comme étant l'identifiant; et en terme d'API, pour prendre ce qui a été fait côté Chrono :

        try:
            agenda = Agenda.objects.get(slug=agenda_identifier)
        except Agenda.DoesNotExist:
            try:
                # legacy access by agenda id
                agenda = Agenda.objects.get(id=int(agenda_identifier))
            except ValueError:
                # ...

Ici donc, avoir le paramètre nommé regie_id, mais pouvoir prendre un slug, et si jamais ça échoue avoir un fallback legacy, identifié comme tel, qu'on pourra supprimer à l'occasion, ça me va très bien.

Et si ça ne match ni slug ni id, alors comme écrit Benjamin, ça ne doit pas lever une erreur 500.

Je réunirais les deux exceptions dans le même except, avec comme unique message "Unknown regie". Ne pas oublier d'actualiser le message de commit.

ok juste pour le message de commit je ne vois pas en quoi il n'est plus actuel ? On permet bien la prise en charge du slug non?

"regie_slug" ça fait penser à un nom de paramètre; quelque chose comme "accept regie slug in add basket item regie_id parameter" plutôt.

https://dev.entrouvert.org/projects/publik/wiki/Paiement contient la documentation. → l'appel est en POST, mais NameId et regie_id sont des paramètres passés dans l'URL.

Pour les tests, ce serait mieux d'étendre test_add_amount_to_basket, qui a déjà sur sa fin un test de regie_id, à dupliquer pour qu'il soit appelé avec le slug de la régie.

Il faut utiliser un autre montant, pour être sûr de concerner uniquement la nouvelle entrée dans le panier.