#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
-~----------~----~----~----~------~----~------~--~---

Reply via email to