Projet

Général

Profil

Bug #46016

IndexError: list index out of range

Ajouté par Sentry Io il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Lauréline Guérin
Version cible:
-
Début:
24 août 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

https://sentry.entrouvert.org/entrouvert/publik/issues/3933/

IndexError: list index out of range
(3 additional frame(s) were not displayed)
...
  File "django/views/generic/base.py", line 68, in view
    return self.dispatch(request, *args, **kwargs)
  File "django/views/generic/base.py", line 88, in dispatch
    return handler(request, *args, **kwargs)
  File "combo/apps/lingo/views.py", line 489, in post
    return self.handle_payment(request, regie, items, remote_items, next_url, email)
  File "contextlib.py", line 74, in inner
    return func(*args, **kwds)
  File "combo/apps/lingo/views.py", line 374, in handle_payment
    return HttpResponseRedirect(get_payment_status_view(next_url=items[0].source_url))

Fichiers

0001-lingo-fix-payment-view-for-remote-regie-and-min-amou.patch (1,8 ko) 0001-lingo-fix-payment-view-for-remote-regie-and-min-amou.patch Lauréline Guérin, 28 août 2020 15:43

Révisions associées

Révision 1aa02b0f (diff)
Ajouté par Lauréline Guérin il y a plus de 3 ans

lingo: fix payment view for remote regie and min amount (#46016)

Historique

#1

Mis à jour par Lauréline Guérin il y a plus de 3 ans

  • Projet changé de Suivi des traces à Lingo
#2

Mis à jour par Lauréline Guérin il y a plus de 3 ans

  • Assigné à mis à Lauréline Guérin
#3

Mis à jour par Lauréline Guérin il y a plus de 3 ans

#4

Mis à jour par Lauréline Guérin il y a plus de 3 ans

Je ne suis pas bien sûre de cette modif, pour le cas où on a des remote_items:

-            return HttpResponseRedirect(get_payment_status_view(next_url=items[0].source_url))
+            return HttpResponseRedirect(get_payment_status_view(next_url=next_url if remote_items else items[0].source_url))

36588dd35739bee1a9ed90ea9a59136591302342 (#36876) a introduit ce changement:

-            return HttpResponseRedirect(next_url)
+            return HttpResponseRedirect(get_payment_status_view(next_url=items[0].source_url))

#5

Mis à jour par Emmanuel Cazenave il y a plus de 3 ans

  • Statut changé de Solution proposée à Solution validée

Je valide parce que ça évite une erreur 500, ce qui est toujours bon à prendre.

Pour ce qui est du parcours usager, il se retrouvera au final à la racine ou sur ce qui est passé dans next_url, je ne sais pas ce qui passé/ce qui est attendu dans un parcours de paiement avec régie distante.

#6

Mis à jour par Lauréline Guérin il y a plus de 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 1aa02b0fb4b6abc08bcc242552a3400f569c79eb
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Fri Aug 28 15:43:00 2020 +0200

    lingo: fix payment view for remote regie and min amount (#46016)
#7

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

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF