fix(util:deepGet): fix losing 0 value (#479)

- close ng-alain/ng-alain#975
ng-alain · Mar 20, 2019 · bf37459 · bf37459
1 parent f491b4d
commit bf37459
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/packages/util/src/other/other.spec.ts b/packages/util/src/other/other.spec.ts
@@ -44,6 +44,10 @@ describe('abc: utils', () => {
     it('should be return default value when paths include null value', () => {
       expect(deepGet({ res: {} }, 'res.address.city')).toBeUndefined();
     });
+    it('should be get zero value', () => {
+      const res = deepGet({ res: { zero: 0 } }, 'res.zero');
+      expect(res).toBe(0);
+    });
   });
 
   it('#deepCopy', () => {

diff --git a/packages/util/src/other/other.ts b/packages/util/src/other/other.ts
@@ -17,7 +17,8 @@ export function deepGet(obj: any, path: string | string[], defaultValue?: any):
     const checkObj = obj[path[0]];
     return typeof checkObj === 'undefined' ? defaultValue : checkObj;
   }
-  return path.reduce((o, k) => (o || {})[k], obj) || defaultValue;
+  const res = path.reduce((o, k) => (o || {})[k], obj);
+  return typeof res === 'undefined' ? defaultValue : res;
 }
 
 export function deepCopy(obj: any): any {