Merge pull request #1467 from tfranzel/decimal_l18n

Add l18n handling for Decimal field #1466

Author: tfranzel

drf_spectacular/openapi.py

@@ -9,6 +9,7 @@
 from django.core import exceptions as django_exceptions
 from django.core import validators
 from django.db import models
+from django.utils.formats import get_format
 from django.utils.translation import gettext_lazy as _
 from rest_framework import permissions, renderers, serializers
 from rest_framework.fields import _UnvalidatedField, empty
@@ -821,7 +822,8 @@ def _map_serializer_field(self, field, direction, bypass_extensions=False):
                     content['pattern'] = (
                         r'^-?0?' if field.max_whole_digits == 0 else fr'^-?\d{{0,{field.max_whole_digits}}}'
                     )
-                    content['pattern'] += fr'(?:\.\d{{0,{field.decimal_places}}})?$'
+                    sep = get_format("DECIMAL_SEPARATOR") if field.localize else "."
+                    content['pattern'] += fr'(?:\{sep}\d{{0,{field.decimal_places}}})?$'
             else:
                 content = build_basic_type(OpenApiTypes.DECIMAL)
                 if field.max_whole_digits is not None:

tests/test_i18n.py

@@ -24,6 +24,7 @@
 class I18nModel(models.Model):
     field_str = models.TextField()
     field_choice = models.CharField(max_length=10, choices=TRANSPORT_CHOICES)
+    field_decimal = models.DecimalField(max_digits=4, decimal_places=2)
 
     class Meta:
         verbose_name = _("Internationalization")
@@ -34,6 +35,7 @@ class XSerializer(serializers.ModelSerializer):
     class Meta:
         model = I18nModel
         fields = '__all__'
+        extra_kwargs = {'field_decimal': {'localize': True}}
 
 
 class XViewset(mixins.ListModelMixin, mixins.RetrieveModelMixin, viewsets.GenericViewSet):

tests/test_i18n.yml

@@ -141,8 +141,13 @@ components:
           type: string
         field_choice:
           $ref: '#/components/schemas/FieldChoiceEnum'
+        field_decimal:
+          type: string
+          format: decimal
+          pattern: ^-?\d{0,2}(?:\,\d{0,2})?$
       required:
       - field_choice
+      - field_decimal
       - field_str
       - id
   securitySchemes: