Browse Source

Merge pull request #42 from keen/FIX-deepExtend-undefined-bug

Drop properties with undefined values
master
Eric Anderson 9 years ago
parent
commit
a07ef9c22b
  1. 2
      .travis.yml
  2. 2
      dist/keen-tracking.js
  3. 2
      dist/keen-tracking.min.js
  4. 2
      lib/utils/deepExtend.js

2
.travis.yml

@ -1,6 +1,6 @@
language: node_js
node_js:
- "0.10"
- "5.1"
branches:
only:
- master

2
dist/keen-tracking.js

@ -787,7 +787,7 @@ function deepExtend(target){
if ('undefined' !== typeof target[prop] && 'object' === typeof arguments[i][prop] && arguments[i][prop] !== null) {
deepExtend(target[prop], clone(arguments[i][prop]));
}
else {
else if (arguments[i][prop] !== undefined) {
target[prop] = clone(arguments[i][prop]);
}
}

2
dist/keen-tracking.min.js

File diff suppressed because one or more lines are too long

2
lib/utils/deepExtend.js

@ -18,7 +18,7 @@ function deepExtend(target){
deepExtend(target[prop], clone(arguments[i][prop]));
}
// Otherwise just copy it over...
else {
else if (arguments[i][prop] !== undefined) {
target[prop] = clone(arguments[i][prop]);
}
}

Loading…
Cancel
Save