diff options
Diffstat (limited to 'src')
| -rw-r--r-- | src/Angular.js | 11 | ||||
| -rw-r--r-- | src/widgets.js | 11 |
2 files changed, 8 insertions, 14 deletions
diff --git a/src/Angular.js b/src/Angular.js index c815d5fc..f4cb9b51 100644 --- a/src/Angular.js +++ b/src/Angular.js @@ -461,12 +461,14 @@ function map(obj, iterator, context) { * @function * * @description - * Determines the number of elements in an array or number of properties of an object. + * Determines the number of elements in an array, number of properties of an object or string + * length. * * Note: this function is used to augment the Object type in angular expressions. See * {@link angular.Object} for more info. * - * @param {Object|Array} obj Object or array to inspect. + * @param {Object|Array|string} obj Object, array or string to inspect. + * @param {boolean} [ownPropsOnly=false] Count only "own" properties in an object * @returns {number} The size of `obj` or `0` if `obj` is neither an object or an array. * * @example @@ -483,14 +485,15 @@ function map(obj, iterator, context) { * </doc:scenario> * </doc:example> */ -function size(obj) { +function size(obj, ownPropsOnly) { var size = 0, key; if (obj) { if (isNumber(obj.length)) { return obj.length; } else if (isObject(obj)){ for (key in obj) - size++; + if (!ownPropsOnly || obj.hasOwnProperty(key)) + size++; } } return size; diff --git a/src/widgets.js b/src/widgets.js index a5a8ee79..a7d59289 100644 --- a/src/widgets.js +++ b/src/widgets.js @@ -913,19 +913,10 @@ angularWidget('@ng:repeat', function(expression, element){ childCount = children.length, lastIterElement = iterStartElement, collection = this.$tryEval(rhs, iterStartElement), - is_array = isArray(collection), - collectionLength = 0, + collectionLength = size(collection, true), childScope, key; - if (is_array) { - collectionLength = collection.length; - } else { - for (key in collection) - if (collection.hasOwnProperty(key)) - collectionLength++; - } - for (key in collection) { if (collection.hasOwnProperty(key)) { if (index < childCount) { |
