diff options
| author | Brenton | 2013-06-01 23:29:18 -0700 | 
|---|---|---|
| committer | Jeff Cross | 2013-07-24 10:58:56 -0700 | 
| commit | 1dcafd18afed4465ee13db91cedc8fecc3aa2c96 (patch) | |
| tree | 5f6cb7c53c4606072637a529df44046c4369821a /src | |
| parent | 4a7b6a4555a76b19dd217171fc0ddce6707bca95 (diff) | |
| download | angular.js-1dcafd18afed4465ee13db91cedc8fecc3aa2c96.tar.bz2 | |
fix(equals): {} and [] should not be considered equivalent
angular.equals was returning inconsistent values for the comparison between
{} and []:
    angular.equals({}, []) // true
    angular.equals([], {}]) // false
Since these object are not of the same type, they should not be considered
equivalent.
Diffstat (limited to 'src')
| -rw-r--r-- | src/Angular.js | 3 | 
1 files changed, 2 insertions, 1 deletions
| diff --git a/src/Angular.js b/src/Angular.js index a41ef4e8..4e050a0c 100644 --- a/src/Angular.js +++ b/src/Angular.js @@ -687,6 +687,7 @@ function equals(o1, o2) {    if (t1 == t2) {      if (t1 == 'object') {        if (isArray(o1)) { +        if (!isArray(o2)) return false;          if ((length = o1.length) == o2.length) {            for(key=0; key<length; key++) {              if (!equals(o1[key], o2[key])) return false; @@ -698,7 +699,7 @@ function equals(o1, o2) {        } else if (isRegExp(o1) && isRegExp(o2)) {          return o1.toString() == o2.toString();        } else { -        if (isScope(o1) || isScope(o2) || isWindow(o1) || isWindow(o2)) return false; +        if (isScope(o1) || isScope(o2) || isWindow(o1) || isWindow(o2) || isArray(o2)) return false;          keySet = {};          for(key in o1) {            if (key.charAt(0) === '$' || isFunction(o1[key])) continue; | 
