Selected not rendered for SelectMultiple when prefix is present

[Adam Gomaa]

See the testcase with the problem here. I have attached the diff that fixes the problem for me, but it breaks other widgets (regressiontests.forms.tests.__test__.form_tests fails), and it's not really clear to me how to fix it or really what the root cause is. Either way, hopefully it's a starting point for someone who understands the internals of newforms better than myself...

[Adam Gomaa]

The patch that solves this particular problem (lets the test case not fail).

[Adam Gomaa]

Testcase that fails on current trunk, passes with patch applied.

[Jacob]

Looks good. A proper patch with the testcase and the fix all in a single diff would be extremely helpful to whoever applies this.

[Ivan Giuliani]

This ticket is invalid. You need to specify the prefix when passing data to the form. The provided test case (slightly modified version):

CHOICES=[(1, 'one'), (2, 'two')]

class TestForm(forms.Form):
    number = forms.MultipleChoiceField(choices=CHOICES, required=False)

data = { 'number': 1 }
foo = TestForm(data, prefix='mypre')

should be:

CHOICES=[(1, 'one'), (2, 'two')]

class TestForm(forms.Form):
    number = forms.MultipleChoiceField(choices=CHOICES, required=False)

data = { 'mypre-number': 1 }
foo = TestForm(data, prefix='mypre')

[Ivan Giuliani]

(the attribute's value in data in the examples should be [1], not 1)

[Jacob]

Milestone 1.0 deleted