Projet

Général

Profil

Bug #42621

IntegrityError: null value in column "subject" violates not-null constraint

Ajouté par Sentry Io il y a environ 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Lauréline Guérin
Version cible:
-
Début:
07 mai 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

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

IntegrityError: null value in column "subject" violates not-null constraint
DETAIL:  Failing row contains (92, null, https://beaulieu.test-eservices.montpellier3m.fr/reservation-can..., , 11.70, 2020-05-07 09:12:38.047597+00, null, null, 1, null, null, t, f, {"url": "https://beaulieu.test-eservices.montpellier3m.fr/reserv..., null, null).

  File "django/db/backends/utils.py", line 64, in execute
    return self.cursor.execute(sql, params)

IntegrityError: null value in column "subject" violates not-null constraint
DETAIL:  Failing row contains (92, null, https://beaulieu.test-eservices.montpellier3m.fr/reservation-can..., , 11.70, 2020-05-07 09:12:38.047597+00, null, null, 1, null, null, t, f, {"url": "https://beaulieu.test-eservices.montpellier3m.fr/reserv..., null, null).

(15 additional frame(s) were not displayed)
...
  File "django/db/backends/utils.py", line 79, in execute
    return super(CursorDebugWrapper, self).execute(sql, params)
  File "django/db/backends/utils.py", line 64, in execute
    return self.cursor.execute(sql, params)
  File "django/db/utils.py", line 94, in __exit__
    six.reraise(dj_exc_type, dj_exc_value, traceback)
  File "django/utils/six.py", line 685, in reraise
    raise value.with_traceback(tb)
  File "django/db/backends/utils.py", line 64, in execute
    return self.cursor.execute(sql, params)

Internal Server Error: /api/lingo/add-basket-item

Fichiers

0001-lingo-fix-add-basket-item-view-when-display_name-is-.patch (1,46 ko) 0001-lingo-fix-add-basket-item-view-when-display_name-is-.patch Lauréline Guérin, 11 mai 2020 10:05
0001-lingo-fix-add-basket-item-view-when-display_name-is-.patch (2,24 ko) 0001-lingo-fix-add-basket-item-view-when-display_name-is-.patch Lauréline Guérin, 13 mai 2020 18:04

Révisions associées

Révision ee1b833e (diff)
Ajouté par Lauréline Guérin il y a presque 4 ans

lingo: fix add basket item view when display_name is missing (#42621)

Historique

#1

Mis à jour par Lauréline Guérin il y a environ 4 ans

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

Mis à jour par Lauréline Guérin il y a presque 4 ans

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

Mis à jour par Lauréline Guérin il y a presque 4 ans

#4

Mis à jour par Thomas Noël il y a presque 4 ans

Je pense qu'il faut éviter un subject vide, parce qu'on l'utilise dans l'affichage, genre 

templates/lingo/combo/basket.html:  <li><a {% if item.source_url %}href="{{ item.source_url }}{% endif %}">{{ item.subject }}</a>: {{ item.amount }} €

Peut-être poser un bête _('No subject') ? Ou faire un 400 qui refuse de créer le paiement ?...

#5

Mis à jour par Lauréline Guérin il y a presque 4 ans

Je n'ai pas assez connaissance du métier pour décider s'il faut mettre un default ou refuser le paiement.
Tu peux trancher Thomas stp ? :)

#6

Mis à jour par Thomas Noël il y a presque 4 ans

Lauréline Guerin a écrit :

Je n'ai pas assez connaissance du métier pour décider s'il faut mettre un default ou refuser le paiement.
Tu peux trancher Thomas stp ? :)

Alors alors... je regarde un peu plus...

Actuellement ça ne marche pas, et créer des paiement sans objet ça me semble pas bien.

Dans la doc https://doc-publik.entrouvert.com/admin-fonctionnel/les-tutos/paiement/

Dans la zone « envoyer les données » vous devez préciser le montant du panier qui va être créé
ainsi que le libellé de la transaction et l'url du formulaire. Ces valeurs sont envoyées
respectivement dans via les noms de données :

    amount
    display_name, qui sera typiquement {{form_display_name}}
    url, qui sera typiquement {{form_url}}

Comme c'est écrit "vous devez", je propose de répondre avec une erreur 400 si display_name est absent.

#8

Mis à jour par Thomas Noël il y a presque 4 ans

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

Mis à jour par Lauréline Guérin il y a presque 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit ee1b833ef4a138703015cd71d1515a3fdd22796a
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Mon May 11 10:04:59 2020 +0200

    lingo: fix add basket item view when display_name is missing (#42621)
#10

Mis à jour par Frédéric Péters il y a presque 4 ans

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

Formats disponibles : Atom PDF