Hi all,

Going through the process of scoping upgrading from 1.8 to 1.9 and after 
having hundreds of test cases failing with "TypeError: get_context_data() 
missing 1 required positional argument: 'form'" I would just like to make 
an observation.

It seems like it is a good optimization - as discussed in 
https://code.djangoproject.com/ticket/24643 to move form init from 
get()/post() to it's own get_context_data() method for consistency. However 
this change was not made with any depreciation schedule. The release notes 
discuss that it may be backward incompatible if methods don't call super(), 
but it is also incompatible if user derived get_context_data() methods are 
written presuming that the form has already been instantiated and has 
form=form as a keyword argument. I'll need to work out the best way of 
dealing with this, but it would appear that I am going to have to go from 
1.8 to 1.9 hard without being able to have code pass tests in both at the 
same time - unless I introspect the Django version. It would have been 
better if this change had have gone through a deprecation cycle.

Cheers,
Wayne Merry


-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/django-developers.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/f436dd35-f2e9-4e38-83e8-35c9c0e122f3%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to