django / django-localflavor

Country-specific Django helpers, formerly of contrib fame
https://django-localflavor.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
828 stars 290 forks source link

Use the value returned by field's clean() #401

Closed emesik closed 4 years ago

emesik commented 4 years ago

I've noticed that for many flavors the super().clean() of fields is called but the result is ignored. Such behavior voids base class validation, for example the strip=True setting is always ignored.

This change unifies all flavors by using the results of base class validations.

benkonrath commented 4 years ago

Looks good, thanks! Could you add a changelog entry for this? Thanks.