AngularJs ng-checked using a function

Question

I have a form and I am trying to use a function in input ng-checked="someFunction()" and I don't know if this is possible or I am doing something wrong. I have function in my controller and I am calling it in view. As far as the function I think it's definitely working and the ng-checked is firing it but returning true or false doesn't change anything. So the question would be is there a way to use function in 'ng-checked' ?

$scope.multiAnswers = function (answers, optionId) {
  angular.forEach(answers, function (answer, key) {
    if (answer.option_choice_id == optionId) {
      return true;
    }
  });
  return false;
};

Afaik ng-checked is used only with on/off value with a model. So you could exploit the model instead. — Whisher
– Whisher, Commented Feb 11, 2014 at 12:22

Community · Accepted Answer · 2017-05-23 11:54:06Z

23

ng-checked does work with functions. Here is a demo:

$scope.getCheckedFalse = function(){
    return false;
};

$scope.getCheckedTrue = function(){
    return true;
};

Html:

<input type="checkbox" ng-checked="getCheckedFalse()"/>
<input type="checkbox" ng-checked="getCheckedTrue()"/>

DEMO

Your problem is that you never return true at the end of the function. return true; inside angular.forEach does not help.

Try:

$scope.multiAnswers = function (answers, optionId) {
     var returnValue = false;
     angular.forEach(answers, function (answer, key) {
         if (answer.option_choice_id == optionId) {
              returnValue = true;
              return;
         }
     });

     return returnValue;
};

It looks like that we cannot break from angular.forEach: Angular JS break ForEach

To improve performance to break immediately when answer.option_choice_id == optionId is true. You could try jQuery $.each or using vanilar javascript (for loop).

edited May 23, 2017 at 11:54

CommunityBot

11 silver badge

answered Feb 11, 2014 at 13:06

Khanh TO

49k13 gold badges101 silver badges117 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Achyuth23 Over a year ago

that's awesome. .

Matt Way · Accepted Answer · 2014-02-11 12:22:30Z

3

What you need to do is use a state variable as the ng-checked expression, eg: ng-checked="someValue". Then it is up to you elsewhere in your code to update that $scope value. The problem is that by using a function, it is unaware when in fact it should update its value.

edited Feb 11, 2014 at 12:22

answered Feb 11, 2014 at 12:15

Matt Way

33.3k10 gold badges85 silver badges86 bronze badges

4 Comments

sarunast Over a year ago

It's in the scope. Sorry I pass 'this' object later to the scope. It wasn't obvious from the code.

sarunast Over a year ago

Thanks for explaining. I was having similar ideas, the problem is that I can't use the variable because the forms are dynamic. Is there some other way ? Using a filter maybe ?

Matt Way Over a year ago

You will need to elaborate. Are you able to create a fiddle?

sarunast Over a year ago

It's quite hard to make a fiddle. Basically I generate different types forms from database using angularjs. And I want to show which options of check-box were selected in the past by the user.

Collectives™ on Stack Overflow

AngularJs ng-checked using a function

2 Answers 2

1 Comment

4 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

4 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related