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.

Sign up for GitHub

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

Jump to bottom

Problem extracting checkbox value #172

Closed
psotelo opened this issue Nov 10, 2016 · 5 comments
Closed

Problem extracting checkbox value #172

psotelo opened this issue Nov 10, 2016 · 5 comments
Labels
bug core help wanted

Comments

@psotelo
Copy link

psotelo commented Nov 10, 2016 •
edited
Loading

When I click over the generated checkboxes its internal value is always "false". This is de HTML code generated

<input type="checkbox" ng-reflect-name="checkbox_54245" ng-reflect-dynamic-id="checkbox_54245" id="checkbox_54245" name="checkbox_54245" value="false" class="ng-valid ng-dirty ng-touched">

why is happening this?

thank you
@udos86 udos86 added the bug label Nov 10, 2016
@udos86
Copy link
Owner

udos86 commented Nov 10, 2016 •
edited
Loading

@psotelo You're absolutely right. This seems to be a bug in DynamicCheckboxGroupModel while everything is working fine with single DynamicCheckboxModels. I'll have to verify why the value update is not reflected in the markup. Thanks!

@udos86
Copy link
Owner

udos86 commented Nov 10, 2016 •
edited
Loading

To avoid this for the moment use a DynamicFormGroupModel instead of a DynamicCheckboxGroupModel. Nevertheless under the hood everything seems to be fine. Only the reflection in markup is not updated. Maybe somebody can help?

@udos86 udos86 removed the bug label Nov 10, 2016
@JBusch
Copy link

JBusch commented Nov 11, 2016

I would have a look!

@udos86
Copy link
Owner

udos86 commented Nov 11, 2016

@JBusch That would be really cool!

@udos86 udos86 added the bug label Nov 26, 2016
@udos86 udos86 closed this as completed in f11cdd9 Nov 27, 2016
@psotelo
Copy link
Author

psotelo commented Dec 20, 2016

Thank you! I'm going to test it!

@udos86 udos86 added the core label Jan 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug core help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@JBusch @udos86 @psotelo