Django-rest-framework: Los flotadores más grandes pasan incorrectamente la validación de DecimalField

Creado en 5 ago. 2015 · 4Comentarios · Fuente: encode/django-rest-framework

Los flotadores más grandes no fallan en la validación DecimalField .

Por ejemplo:

>>> serializers.DecimalField(max_digits=3, decimal_places=1).run_validation(200000000000.0)
>>> Decimal('2E+11')

Esperaría que se recaude un ValidationError ya que ese número tiene más de 3 dígitos. La validación del formulario de Django arroja el error esperado:

>>> forms.DecimalField(max_digits=3, decimal_places=1).clean(200000000000.0)
ValidationError: [u'Ensure that there are no more than 3 digits in total.']

ValidationError aumenta si elimina un cero de la entrada de muestra anterior:

>>> serializers.DecimalField(max_digits=3, decimal_places=1).run_validation(20000000000.0)
ValidationError: [u'Ensure that there are no more than 3 digits in total.']

De manera similar, la excepción también se genera usando el número original pero como Decimal e int:

>>> serializers.DecimalField(max_digits=3, decimal_places=1).run_validation(200000000000)
ValidationError: [u'Ensure that there are no more than 3 digits in total.']
>>> serializers.DecimalField(max_digits=3, decimal_places=1).run_validation(Decimal('200000000000.0'))
ValidationError: [u'Ensure that there are no more than 3 digits in total.']

Una línea clave cambió recientemente en https://github.com/tomchristie/django-rest-framework/pull/2948 pero no parece correcta y se desvía de la validación forms.DecimalField de Django que, de lo contrario, parece haber sido copiada literal:

decimals = exponent * decimal.Decimal(-1) if exponent < 0 else 0

Realmente no entiendo el problema original abordado en # 2948, así que no sé por qué cambió la línea. Estoy más que feliz de trabajar en el problema si entiendo el problema original.

A continuación se muestra un parche con una prueba fallida:

diff --git a/tests/test_fields.py b/tests/test_fields.py
index 0427873..cf41a5b 100644
--- a/tests/test_fields.py
+++ b/tests/test_fields.py
@@ -773,6 +773,7 @@ class TestDecimalField(FieldValues):
         (Decimal('Nan'), ["A valid number is required."]),
         (Decimal('Inf'), ["A valid number is required."]),
         ('12.345', ["Ensure that there are no more than 3 digits in total."]),
+        (200000000000.0, ["Ensure that there are no more than 3 digits in total."]),
         ('0.01', ["Ensure that there are no more than 1 decimal places."]),
         (123, ["Ensure that there are no more than 2 digits before the decimal point."])
     )

Bug

Fuente

ryankask

Comentario más útil

Ahora resuelto, con una lógica mucho más transparente y obvia.

tomchristie en 6 ago. 2015

😄2

Todos 4 comentarios

Gracias Ryan.

tomchristie en 5 ago. 2015

Ahora resuelto, con una lógica mucho más transparente y obvia.

tomchristie en 6 ago. 2015

😄2

@tomchristie @ryankask - Veo que esto se agregó a un hito de lanzamiento en 2015. Todavía me encuentro con este problema en la versión 3.11.0 . ¿La solución terminó siendo lanzada? Código a continuación:

class RecommendationSerializer(serializers.Serializer):
    total_owed = serializers.DecimalField(decimal_places=2, max_digits=8, min_value='1.00',
                                          rounding=ROUND_DOWN)
    term_length = serializers.IntegerField(min_value=1)

    class Meta:
        fields = ['total_owed', 'term_length']


serializer = RecommendationSerializer(data={'total_owed': '12.333333333333333', 'term_length': 6})
serializer.is_valid(raise_exception=True)

>>> {"total_owed": ["Ensure that there are no more than 8 digits in total."]}

Audace en 24 may. 2020

@Audace obtiene el error de validación esperado. El grupo de discusión es el mejor lugar para abordar esta discusión y otras preguntas de uso. ¡Gracias!