Django-rest-framework: बड़ी फ़्लोट गलत तरीके से DecimalField सत्यापन पास करती हैं

को निर्मित 5 अग॰ 2015 · 4टिप्पणियाँ · स्रोत: encode/django-rest-framework

बड़े फ़्लोट DecimalField सत्यापन में विफल नहीं हो रहे हैं।

उदाहरण के लिए:

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

मुझे उम्मीद है कि ValidationError उठाया जाएगा क्योंकि उस संख्या में 3 से अधिक अंक हैं। Django का फॉर्म सत्यापन अपेक्षित त्रुटि फेंकता है:

>>> 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 बढ़ जाता है:

>>> 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.']

इसी तरह, अपवाद भी मूल संख्या का उपयोग करके उठाया जाता है लेकिन Decimal और 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.']

एक प्रमुख पंक्ति हाल ही में https://github.com/tomchristie/django-rest-framework/pull/2948 में बदल गई है, लेकिन यह सही नहीं लगती है और Django के forms.DecimalField सत्यापन से विचलित हो जाती है जो अन्यथा कॉपी की गई प्रतीत होती है शब्दशः:

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

मैं वास्तव में #2948 में संबोधित मूल मुद्दे को नहीं समझता, इसलिए मुझे नहीं पता कि रेखा क्यों बदली। अगर मैं मूल मुद्दे को समझता हूं तो मुझे इस मुद्दे पर काम करने में खुशी होगी।

नीचे एक असफल परीक्षण के साथ एक पैच है:

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

स्रोत

ryankask

सबसे उपयोगी टिप्पणी

अब कुछ और पारदर्शी और स्पष्ट तर्क के साथ हल हो गया है।

tomchristie 6 अग॰ 2015

😄2

सभी 4 टिप्पणियाँ

धन्यवाद रयान।

tomchristie 5 अग॰ 2015

अब कुछ और पारदर्शी और स्पष्ट तर्क के साथ हल हो गया है।

tomchristie 6 अग॰ 2015

😄2

@tomchristie @ryankask - मुझे लगता है कि इसे 2015 में रिलीज़ मील के पत्थर में जोड़ा गया था। मैं अभी भी संस्करण 3.11.0 पर इस मुद्दे पर चल रहा हूं। क्या फिक्स अंत में रिलीज़ हो रहा था? नीचे कोड:

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 24 मई 2020

@Audace आपको अपेक्षित सत्यापन त्रुटि मिलती है। इस चर्चा और अन्य उपयोग प्रश्नों को लेने के लिए चर्चा समूह सबसे अच्छी जगह है। धन्यवाद!

xordoquy 24 मई 2020

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

Django-rest-framework: बड़ी फ़्लोट गलत तरीके से DecimalField सत्यापन पास करती हैं

सबसे उपयोगी टिप्पणी

सभी 4 टिप्पणियाँ

संबंधित मुद्दों