Projet

Général

Profil

Bug #19929

Ne pas utiliser assert pour vérifier des choses importantes

Ajouté par Frédéric Péters il y a plus de 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Version cible:
-
Début:
07 novembre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Dans le code de retour de validation de paiement il y a des assert qui vont être ignorés en -O ou autre, vraiment pas top.

Fichiers

0001-lingo-do-not-use-assert-to-check-important-things-19.patch (1,81 ko) 0001-lingo-do-not-use-assert-to-check-important-things-19.patch Frédéric Péters, 07 novembre 2017 09:25
0001-lingo-do-not-use-assert-to-check-important-things-19.patch (1,76 ko) 0001-lingo-do-not-use-assert-to-check-important-things-19.patch Frédéric Péters, 07 novembre 2017 10:13

Révisions associées

Révision 9798ff42 (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

lingo: do not use assert to check important things (#19929)

Historique

#1

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

#2

Mis à jour par Thomas Noël il y a plus de 6 ans 

if not payment_response.result == eopayment.CANCELLED:
    # cancellation may not be signed...
    if not payment_response.signed:
        ...

est un peu bizarre à lire à mon goût, mais ack.

(Je lirais plus facilement :

if not payment_response.signed and not (payment_response.result == eopayment.CANCELLED):  # cancellation may not be signed
    ....

)
#4

Mis à jour par Thomas Noël il y a plus de 6 ans

Impec. Ack.

#5

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

  • Statut changé de En cours à Résolu (à déployer)
commit 9798ff4259e2181e1c2a0282e62e1df73b667d39
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Nov 7 09:25:05 2017 +0100

    lingo: do not use assert to check important things (#19929)
#6

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

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

Formats disponibles : Atom PDF