a8m / a8m/obj-parse

Enhancement Suggestion for obj-parse setterFn

Offen
#1 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
JavaScript
Sterne
3
Forks
1
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Hi Ariel, first off thank you for obj-parse and deepkeys. Both are wonderful utilities that I am using in my project(s). I wanted to send you this change to setterFn, as it might be of benefit to others who would like to be able to handle key values which are functions:

``` javascript
var setterFn = function(scope, path, value) {
var s = scope;
for(var i = 0; i < path.length - 1; i++) { //check the path
s = s[path[i]]; // s is the current path segment
if(angular.isUndefined(s)) { //make sure the path segment is good/defined
return undefined;
}
}
//ok, we have a good path, if it is a vanilla value return that, or, if a function call it
//probably we could be more sophisticated and do something like value.call(scope,path,value)
//just use value() for now
s[path[i]] = (angular.isFunction(value) ? value() : value);
return scope; //return the updated scope
};
```

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.