switch-change="onChange()" gets called immediately when loading about angular-bootstrap-switch HOT 7 CLOSED

The problem is most likely that the ng-model value is evaluated as undefined when the template is rendered and when the controller sets available to false the directive interprets it as a model change, which it strictly speaking is even though one might argue that a ng-model change from undefined->true/false should not trigger a model change since it's almost always just an init case.

The problem can be solved by adding an ng if like this

ng-if="model.available !== undefined"

to the bs-switch input or a wrapping element, ng-show will not work since that wont stop the template from rendering the input element.

from angular-bootstrap-switch.

I tried to debug this and noticed that the switchChange() is being called upon a model change. I'm wondering why this would need to call the switchChange function if a model is changed ?

          controller.$render = function () {
            initMaybe();
            var newValue = controller.$modelValue;
            if (newValue !== undefined && newValue !== null) {
              element.bootstrapSwitch('state', newValue === getTrueValue(), true);
            } else {
              element.bootstrapSwitch('indeterminate', true, true);
              controller.$setViewValue(undefined);
            }
            switchChange();
          };

from angular-bootstrap-switch.

What do you mean? switch-change is called every time the model changes. If you want to listen to view changes, please use ng-change.

from angular-bootstrap-switch.

ok I'll try that, what's switch-change for ?

from angular-bootstrap-switch.

It's for model changes.

from angular-bootstrap-switch.

I also ran into this and I find it unexpected behaviour. In my app an undefined settings item gets initialized and this is triggering switch-change. Using ng-change now. It works as I wanted it to work.

from angular-bootstrap-switch.

Please re-open if you still have this issue.

from angular-bootstrap-switch.