Rename angular.foreach to angular.forEach to make the api consistent.

camelcase is used for other angular functions and forEach is also
used by EcmaScript standard.

- rename the internal as well as the external function name
- tweak the implementation of the function so that it doesn't
  clober it self when we extend the angular object with an
  object that has a forEach property equal to this forEach function

Closes #85
This commit is contained in:
Igor Minar
2011-01-07 22:02:23 -08:00
parent c79aba92f6
commit 0a6cf70deb
32 changed files with 101 additions and 96 deletions

View File

@@ -53,12 +53,12 @@ function fromJson(json, useNative) {
throw e;
}
// TODO make foreach optionally recursive and remove this function
// TODO make forEach optionally recursive and remove this function
function transformDates(obj) {
if (isString(obj) && obj.length === DATE_ISOSTRING_LN) {
return angularString.toDate(obj);
} else if (isArray(obj) || isObject(obj)) {
foreach(obj, function(val, name) {
forEach(obj, function(val, name) {
obj[name] = transformDates(val);
});
}