Merge pull request #256 from martin-dimitrov13/master

 Fixed object/get throwing exception for null or undefined input objects

Author: roboshoes

CHANGELOG.md

@@ -1,5 +1,8 @@
 mout changelog
 ==============
+v1.1.1 (2019/04/18)
+--------------------
+ - fixed `object/get` throwing an exception for null or undefined input objects
 
 v1.1.0 (2017/10/04)
 --------------------
@@ -204,4 +207,3 @@ v0.1.0 (2013/01/09)
 -------------------
 
 - Rename project from "amd-utils" to "mout"
-

bower.json

@@ -40,5 +40,5 @@
     "type": "git",
     "url": "git://github.com/mout/mout.git"
   },
-  "version": "1.1.0"
+  "version": "1.1.1"
 }

doc/object.md

@@ -363,7 +363,7 @@ functions(obj); // ['foo']
 ## get(obj, propName):*
 
 Returns nested property value. Will return `undefined` if property doesn't
-exist.
+exist or if the object is null or undefined.
 
 See: [`set()`](#set), [`namespace()`](#namespace), [`has()`](#has)
 
@@ -378,6 +378,7 @@ var lorem = {
 
 get(lorem, 'ipsum.dolor.sit'); // "amet"
 get(lorem, 'foo.bar');         // undefined
+get(undefined, 'foo.bar');     // undefined
 ```
 
 

package.json

@@ -1,7 +1,7 @@
 {
   "name": "mout",
   "description": "Modular Utilities",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "homepage": "http://moutjs.com/",
   "author": "Miller Medeiros <contact@millermedeiros.com> (http://blog.millermedeiros.com)",
   "contributors": [

src/index.js

@@ -1,12 +1,12 @@
 /**@license
- * mout v1.1.0 | http://moutjs.com | MIT license
+ * mout v1.1.1 | http://moutjs.com | MIT license
  */
 define(function(require){
 
 //automatically generated, do not edit!
 //run `node build` instead
 return {
-    'VERSION' : '1.1.0',
+    'VERSION' : '1.1.1',
     'array' : require('./array'),
     'collection' : require('./collection'),
     'date' : require('./date'),

src/object/get.js

@@ -4,9 +4,9 @@ define(['../lang/isPrimitive'], function (isPrimitive) {
      * get "nested" object property
      */
     function get(obj, prop){
+        if (!obj) return;
         var parts = prop.split('.'),
             last = parts.pop();
-
         while (prop = parts.shift()) {
             obj = obj[prop];
             if (obj == null) return;

tests/spec/object/spec-get.js

@@ -23,7 +23,7 @@ define(
                         lorem : function(){}
                     }
                 };
-                
+
                 foo.bar.lorem.ipsum = 'dolor'
 
                 expect( get(foo, 'bar.lorem.ipsum') ).toBe( 'dolor' );
@@ -58,8 +58,20 @@ define(
                 expect( get(foo, 'foo.bar.baz') ).toBe(undef);
             });
 
+            it('should return undefined for undefined input objects', function() {
+              var foo = undefined;
+
+              var undef;
+              expect( get(foo, 'bar.baz') ).toBe(undef);
+            });
+
+            it('should return undefined for null input objects', function() {
+              var foo = null;
+
+              var undef;
+              expect( get(foo, 'bar.baz') ).toBe(undef);
+            });
         });
 
     }
 );
-