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

fix($rootScope): $watchCollection should handle NaN in objects #7930

Closed
wants to merge 1 commit into from
Closed

fix($rootScope): $watchCollection should handle NaN in objects #7930

wants to merge 1 commit into from

Conversation

shahata
Copy link
Contributor

@shahata shahata commented Jun 21, 2014

This fixes a potential infinite digest in $watchCollection when one of the values is NaN. This was previously fixed for arrays, but needs to be handled for objects as well.

@shahata shahata added cla: yes and removed cla: no labels Jun 21, 2014
@shahata shahata changed the title fix($rootScope): $watchCollection shold handle NaN in objects fix($rootScope): $watchCollection should handle NaN in objects Jun 21, 2014
@rodyhaddad
Copy link
Contributor

LGTM

@rodyhaddad rodyhaddad added this to the 1.3.0 milestone Jun 23, 2014
This fixes a potential infinite digest in $watchCollection when one of the values is NaN. This was previously fixed for arrays, but needs to be handled for objects as well.
@shahata
Copy link
Contributor Author

shahata commented Jul 12, 2014

bump

@rodyhaddad rodyhaddad self-assigned this Jul 14, 2014
rodyhaddad pushed a commit that referenced this pull request Jul 15, 2014
This fixes a potential infinite digest in $watchCollection when one of the values is NaN. This was previously fixed for arrays, but needs to be handled for objects as well.

Closes #7930
ckknight pushed a commit to ckknight/angular.js that referenced this pull request Jul 16, 2014
This fixes a potential infinite digest in $watchCollection when one of the values is NaN. This was previously fixed for arrays, but needs to be handled for objects as well.

Closes angular#7930
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.

2 participants