#231: [patch] Make appropriate model fields honor max_length param
----------------------------------------------------------------+-----------
Reporter: nesh <nesh [at] studioquattro [dot] co [dot] yu> |
Owner: donspaulding
Status: assigned |
Component: Metasystem
Version: |
Resolution:
Keywords: FileFields sprintsept14 |
Stage: Accepted
Has_patch: 1 |
Needs_docs: 0
Needs_tests: 0 |
Needs_better_patch: 1
----------------------------------------------------------------+-----------
Changes (by PhiR):
* needs_better_patch: 0 => 1
Comment:
The patch looks nice. I've checked that all backends are covered. One
small nitpick: the doc says max_length only exist in the dev version,
maybe it would be more generic to say that it is only supported in version
> .96. I've had to touch tests/regressiontests/max_lengths/__init__.py to
make the tests works though, it should be added in the patch (if possible,
i know svn is tricky with empty files).
--
Ticket URL: <http://code.djangoproject.com/ticket/231#comment:13>
Django Code <http://code.djangoproject.com/>
The web framework for perfectionists with deadlines
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---