Projet

Général

Profil

Bug #30770

0 convertit en chaîne vide par le filtre decimal

Ajouté par Nicolas Roche il y a environ 5 ans. Mis à jour il y a environ 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Nicolas Roche
Version cible:
-
Début:
19 février 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Dans un champ commentaire d'un formulaire je demande d'afficher :

0|decimal => {{ 0 | decimal }}

et j'obtient 
0|decimal =>

Fichiers

0001-correct-decimal-filter-output-on-0-30770.patch (1,78 ko) 0001-correct-decimal-filter-output-on-0-30770.patch Nicolas Roche, 21 février 2019 12:32
0001-templatetags-correct-decimal-filter-output-on-0-3077.patch (1,79 ko) 0001-templatetags-correct-decimal-filter-output-on-0-3077.patch avec le domaine dans le message. Nicolas Roche, 21 février 2019 15:22

Révisions associées

Révision 3e763b99 (diff)
Ajouté par Nicolas Roche il y a environ 5 ans

templatetags: correct decimal filter output on 0 (#30770)

Historique

#1

Mis à jour par Nicolas Roche il y a environ 5 ans

  • Description mis à jour (diff)
#2

Mis à jour par Nicolas Roche il y a environ 5 ans

  • Fichier 0001-correct-decimal-filter-output-on-0-30770.patch ajouté
  • Statut changé de En cours à Solution proposée
  • Patch proposed changé de Non à Oui
#3

Mis à jour par Thomas Noël il y a environ 5 ans

Tu peux complètement supprimer le "if value is None: return None" au début de parse_decimal, car ça fera un TypeError lors de la conversion.

Ensuite, un peu de "pythonic", la forme :

    if arg is None:
        return value
    else:
        return defaultfilters.floatformat(value, arg=arg)

on préfère l'écrire ainsi :

    if arg is None:
        return value
    return defaultfilters.floatformat(value, arg=arg)

afin qu'on voit mieux que "il y a toujours un return", ou "quel est le return par défaut".

#4

Mis à jour par Nicolas Roche il y a environ 5 ans

  • Fichier 0002-fix-Thoma-s-advices-thanks.patch ajouté

J'avoue que j'ai pas compris pour le TypeError lors de la conversion,
mais ça marche nickel.

#5

Mis à jour par Thomas Noël il y a environ 5 ans

Peux-tu squasher tes deux patches, et proposer ici (et dans ta branche) le patch final ?

#7

Mis à jour par Nicolas Roche il y a environ 5 ans

  • Fichier 0001-correct-decimal-filter-output-on-0-30770.patch supprimé
#8

Mis à jour par Nicolas Roche il y a environ 5 ans

  • Fichier 0002-fix-Thoma-s-advices-thanks.patch supprimé
#9

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

(on évite de supprimer les patchs attachés, parce que ça fait perdre du contexte aux commentaires qui les concernaient).

#10

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

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

Go, on essaie aussi de préfixer les messages de commit d'un "domaine", ici ça pourrait être "templatetags: correct decimal filter output on 0 (#30770)" (et quand il n'y a rien d'évident, mettre misc: ou general: passe assez).

#12

Mis à jour par Nicolas Roche il y a environ 5 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 3e763b999ae1a136c65336c1eef7f36e89ff07fd
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Tue Feb 19 18:43:33 2019 +0100

    templatetags: correct decimal filter output on 0 (#30770)
#13

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

(c'était ok sans redéposer le fichier ici)

#14

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

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

Formats disponibles : Atom PDF