Merge pull request #71 from elektronik2k5/bugfix/dont-fail-on-object-with-null-proto

Bugfix: don't fail on object with null proto

Author: mariocasciaro

index.js

@@ -79,7 +79,7 @@
     };
 
     function getShallowProperty(obj, prop) {
-      if(options.includeInheritedProps || (typeof prop === 'number' && Array.isArray(obj)) || obj.hasOwnProperty(prop)) {
+      if (options.includeInheritedProps || (typeof prop === 'number' && Array.isArray(obj)) || _hasOwnProperty.call(obj, prop)) {
         return obj[prop];
       }
     }

package.json

@@ -1,7 +1,7 @@
 {
   "name": "object-path",
   "description": "Access deep object properties using a path",
-  "version": "0.11.0",
+  "version": "0.11.1",
   "author": {
     "name": "Mario Casciaro"
   },

test.js

@@ -102,6 +102,16 @@ describe('get', function() {
     expect(objectPath.get(undefined, 'test', 'a')).to.be.deep.equal('a');
   });
 
+  it(
+    'should not fail on an object with a null prototype',
+    function assertSuccessForObjWithNullProto(){
+      var foo = 'FOO';
+      var objWithNullProto = Object.create(null);
+      objWithNullProto.foo = foo;
+      expect(objectPath.get(objWithNullProto, 'foo')).to.equal(foo);
+    }
+  );
+
   it('should skip non own properties', function() {
     var Base = function(enabled){ };
     Base.prototype = {
@@ -772,7 +782,7 @@ describe('bind object', function () {
   });
 });
 
-describe('Don\' access not own properties [default]', function () {
+describe('Don\'t access not own properties [default]', function () {
   it('should not get a not own property', function() {
     var Obj = function() {};
     Obj.prototype.notOwn = {a: 'a'};