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

Changing ng-options datasource is firing the ng-change automatically #16586

Open
1 of 3 tasks
amiteshhh opened this issue Jun 1, 2018 · 6 comments
Open
1 of 3 tasks

Comments

@amiteshhh
Copy link

I'm submitting a ...

  • bug report
  • feature request
  • other

Current behavior:
Changing ng-options underlying datasource is triggering the ng-change handler when underlying ng-model was changed programmatically previously.

Expected / new behavior:
It should not trigger ng-change handler.

Minimal reproduction of the problem with instructions:

dd1 and dd2 are two independent dropdowns having different ng-options data source containing array of objects.
Step 1: Set ng-model for dd2 programmatically by clicking the button. This is not firing the ng-change handler of dd2 which is correct.

Step 2: Now change the dd1. Inside dd1 change handler assign new array data source to dd2 ng-options. This is automatically triggering the ng-change of dd2.

Demo link: https://plnkr.co/edit/mI44WPHmlPf5B6Sdpbxr?p=preview

AngularJS version: 1.6.9

Issue still exists in the most recent stable version 1.7.0 and in the snapshot AngularJS 1.7.1-build.26050+sha.0b1adbb

Browser: [all | Chrome 66 | Edge 40 | IE 11 ]

@amiteshhh amiteshhh changed the title Changing ng-options is firing the ng-change automatically Changing ng-options datasource is firing the ng-change automatically Jun 1, 2018
@gkalpak
Copy link
Member

gkalpak commented Jun 1, 2018

It seems that whenever one of the values watched by ngOptions is changed, we might end up calling $setViewValue(). This seems incorrect if the changes can be a result of a change on the model side (as is the case in the above example).

@Narretz probably knows more.

@Narretz
Copy link
Contributor

Narretz commented Jun 4, 2018

The behavior is intentional: #7736 I can see how this can be confusing / unwanted, but it's an either-or situation and we've "fixed" this behavior, so we cannot go back. Why exactly is it a problem for your app that the model us updated @amiteshhh ?

@amiteshhh
Copy link
Author

amiteshhh commented Jun 4, 2018

@Narretz In our app we have a drop-down to select country. Based on country selection we display different information for that country. The information contains data for month drop-down(driven by country). By default we are initializing default month to show the default view. Additionally on change of month we fetch data for selected month.
Now user changes country to see information of other country and again we are fetching new set of data including months but at this moment ng-change handler of months automatically triggered by Angular(and this is the whole problem).
By maintaining the same reference of months array its not triggering the ng-change handler but this seems less obvious/awkward solution.

@AndrewMacKay-base
Copy link

Following as this is also an issue for our team at present

@aaronpikkarainen
Copy link

It's my understanding that the purpose of ng-change is to know when the user changes the value. It's not supposed to fire simply when the model is changed programmatically.

I agree that the ng-model value should be updated in this particular case (as fixed in #7736), but I don't think the ng-change should be fired.

If this is an accurate understanding of the purpose of ng-change. could we get a fix for this?

@Narretz
Copy link
Contributor

Narretz commented Aug 6, 2019

@aaronpikkarainen AngularJS is in LTS mode, so we are only going to fix security bugs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants