Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

using ng-checked="expression" will never update the respective ng-model value. #11106

Closed
wants to merge 1 commit into from

Conversation

pankajparkar
Copy link

I'm adding description so that everyone know that updating ng-checked for checked will never update the binding between ng-model variable.

Plunkr
http://plnkr.co/edit/vdVMmhe0rY1aGzX4koyM?p=preview

I'm adding description so that everyone know that updating ng-checked for checked will never update the binding between ng-model variable.

Plunkr
http://plnkr.co/edit/vdVMmhe0rY1aGzX4koyM?p=preview
@googlebot
Copy link

Thanks for your pull request.

It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA) at https://cla.developers.google.com/.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check the information on your CLA or see this help article on setting the email on your git commits.

Once you've done that, please reply here to let us know. If you signed the CLA as a corporation, please let us know the company's name.

@Narretz
Copy link
Contributor

Narretz commented Feb 20, 2015

I don't understand what you mean. ngChecked will only set the checked attribute, it does not work together with ngModel in any way. Is that what you are trying to say? I#m particularly confused about the ng-changed part of the sentence.

@Narretz Narretz added this to the Ice Box milestone Feb 20, 2015
@Narretz Narretz self-assigned this Feb 20, 2015
@pankajparkar
Copy link
Author

@Narretz Ya, that's exactly i wanted to say..ng-checked with expression only set its checked attribute, it never update respective ng-model variable.

@googlebot
Copy link

CLAs look good, thanks!

@Narretz Narretz closed this in 33f7f26 Jun 22, 2015
netman92 pushed a commit to netman92/angular.js that referenced this pull request Aug 8, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants