[patch] modify as_hidden() to handle MultipleChoiceField correctly

[Honza Král <Honza.Kral@…>]

It is a quick and dirty patch (works though ;) ), but since the information on what field requires list as value is gone, I had to use the Field class to identify the correct value. I don't like the idea of checking for Field;s class every time I render a hidden field, but it was the best I could do without going further into the code.

The added widget class renders multiple <input type="hidden" .. > tags (one for every checked option).

I will be glad for any feedback and/or comments.

[Honza Král <Honza.Kral@…>]

implementation of MultipleHiddenInput

[Adrian Holovaty]

Hmmm, yeah, doing an isinstance there seems a bit inelegant. I'll play around with some other implementations... Maybe it's as easy as giving the Field class a hidden_field attribute, which MultipleChoiceField would override.

[Honza Král <Honza.Kral@…>]

I originally thought of using the requires_list field, but it was no longer there so I decided not to add another field with roughly the same meaning...

btw. it seems more natural to me for as_hidden, as_text etc to be handled by Field and not BoundField - the latter knows nothing about the data (as demonstrated in this problem) and therefore cannot adapt its behaviour, the Field seems better suited for this.

[Adrian Holovaty]

(In [4298]) Fixed #3193 -- newforms: Modified as_hidden() to handle MultipleChoiceField correctly. Thanks for the report, Honza Kral