fix(ngRepeat): add support to iterate an object's properties even if it does not inherit from Object

HeberLZ · petebacondarwin · commit 7ea2c7f36ef8 · 2015-09-07T22:06:41.000+01:00
Closes angular#9964
diff --git a/src/ng/directive/ngRepeat.js b/src/ng/directive/ngRepeat.js
@@ -427,7 +427,7 @@ var ngRepeatDirective = ['$parse', '$animate', function($parse, $animate) {
             // if object, extract keys, in enumeration order, unsorted
             collectionKeys = [];
             for (var itemKey in collection) {
-              if (collection.hasOwnProperty(itemKey) && itemKey.charAt(0) !== '$') {
+              if (hasOwnProperty.call(collection, itemKey) && itemKey.charAt(0) !== '$') {
                 collectionKeys.push(itemKey);
               }
             }
diff --git a/test/ng/directive/ngRepeatSpec.js b/test/ng/directive/ngRepeatSpec.js
@@ -166,6 +166,26 @@ describe('ngRepeat', function() {
     expect(element.text()).toEqual('age:20|wealth:20|prodname:Bingo|dogname:Bingo|codename:20|');
   });
 
+
+  it('should iterate over on object created using `Object.create(null)`', function() {
+    element = $compile(
+      '<ul>' +
+        '<li ng-repeat="(key, value) in items">{{key}}:{{value}}|</li>' +
+      '</ul>')(scope);
+
+    var items = Object.create(null);
+    items.misko = 'swe';
+    items.shyam = 'set';
+
+    scope.items = items;
+    scope.$digest();
+    expect(element.text()).toEqual('misko:swe|shyam:set|');
+
+    delete items.shyam;
+    scope.$digest();
+    expect(element.text()).toEqual('misko:swe|');
+  });
+
   describe('track by', function() {
     it('should track using expression function', function() {
       element = $compile(

Original file line number	Diff line number	Diff line change
`@@ -427,7 +427,7 @@ var ngRepeatDirective = ['$parse', '$animate', function($parse, $animate) {`
`427`	`427`	`// if object, extract keys, in enumeration order, unsorted`
`428`	`428`	`collectionKeys = [];`
`429`	`429`	`for (var itemKey in collection) {`
`430`		`- if (collection.hasOwnProperty(itemKey) && itemKey.charAt(0) !== '$') {`
	`430`	`+ if (hasOwnProperty.call(collection, itemKey) && itemKey.charAt(0) !== '$') {`
`431`	`431`	`collectionKeys.push(itemKey);`
`432`	`432`	`}`
`433`	`433`	`}`