Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix BitFieldCheckboxSelectMultiple for new form #35

Merged
merged 2 commits into from
May 11, 2015

Conversation

habfast
Copy link

@habfast habfast commented Feb 24, 2014

In the admin, when creating a new form, the default is the integer version of the value, which cannot be looped over in CheckboxSelectMultiple.render() (raises an Error). This converts the integer value to Array. Works with default=X, X being an integer or an array of flags.

habfast added 2 commits February 24, 2014 22:33
In the admin, when creating a new form, the default is the integer version of the value, which cannot be looped over in CheckboxSelectMultiple.render() (raises an Error). This converts the integer value to Array. Works with default=X, X being an integer or an array of flags.
@johnboiles
Copy link

Looks like this replaces #32 and fixes #28.

Thanks for writing this fix. Breaking the ability to add new items via admin seems like a very serious bug.

@decaz
Copy link

decaz commented Sep 12, 2014

Does anyone merge this fix?

@LukeSkyw
Copy link

Hi, can a maintainer merge this request. I have a similar problem. Impossible to add en entry which has a bitfield.

dcramer added a commit that referenced this pull request May 11, 2015
fix BitFieldCheckboxSelectMultiple for new form
@dcramer dcramer merged commit 24126a0 into disqus:master May 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants