#12285: Confusing error message when ModelForm does not have model set
------------------------------------+---------------------------------------
          Reporter:  RaceCondition  |         Owner:  kmtracey
            Status:  new            |     Milestone:  1.2     
         Component:  Forms          |       Version:  1.1     
        Resolution:                 |      Keywords:          
             Stage:  Accepted       |     Has_patch:  1       
        Needs_docs:  0              |   Needs_tests:  0       
Needs_better_patch:  0              |  
------------------------------------+---------------------------------------
Comment (by peterbe):

 I think 12285.diff is good. Applied it and it works fine.
 The module already raises {{{Exception}}} errors on a bunch of other
 things and the tests are already a doctest.

 Raising Exception is IMHO a bad practice since it's not very explicit. A
 custom defined error would be much better.

-- 
Ticket URL: <http://code.djangoproject.com/ticket/12285#comment:6>
Django <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