Single select widgets are broken in 0.9.8 #181

Closed
simonk52 opened this Issue Sep 16, 2013 · 1 comment

Comments

Projects
None yet
2 participants
@simonk52

I think commit 4fbdfe2 broke single-select widgets. If the "multiple" widget property is False, the template ends up producing:

<select multiple="">

…which unfortunately is the same as:

<select multiple>

I think the relevant line should look like this instead:

multiple multiple|field.widget.multiple and 'multiple' or None

You can see the problem at http://deformdemo.repoze.org/select/ - it should show a single select field, but instead it's a multiple select field.

It works fine in 0.9.7.

@tseaver tseaver closed this in 1565c53 Sep 16, 2013

@tseaver

This comment has been minimized.

Show comment
Hide comment
@tseaver

tseaver Sep 16, 2013

Member

Thanks for the report!

Member

tseaver commented Sep 16, 2013

Thanks for the report!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment