Branch: refs/heads/master
  Home:   https://github.com/django/django
  Commit: 9fb0f5dddc4cf7f2d294af1bcde2c359cffd90a5
      
https://github.com/django/django/commit/9fb0f5dddc4cf7f2d294af1bcde2c359cffd90a5
  Author: Marc Tamlyn <marc.tam...@gmail.com>
  Date:   2014-05-24 (Sat, 24 May 2014)

  Changed paths:
    M django/forms/forms.py
    M docs/ref/forms/api.txt
    M docs/releases/1.7.txt
    M docs/topics/forms/modelforms.txt
    M tests/model_forms/tests.py

  Log Message:
  -----------
  Fixed #22510 -- Harden field removal to only None.

Refs #8620.

If we allow any value to remove form fields then we get name clashes
with method names, media classes etc. There was a backwards
incompatibility introduced meaning ModelForm subclasses with declared
fields called media or clean would lose those fields.

Field removal is now only permitted by using the sentinel value None.
The docs have been slightly reworded to refer to removal of fields
rather than shadowing.

Thanks to gcbirzan for the report and initial patch, and several of the
core team for opinions.


-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/53808cbac81b8_4008637d38139a0%40hookshot-fe1-cp1-prd.iad.github.net.mail.
For more options, visit https://groups.google.com/d/optout.

Reply via email to