Skip prototype properties when looping over objects in objToPaths()

README.md

@@ -46,6 +46,10 @@ Example code:
         console.log(val);
     });
 
+    //... and all parent attributes as well
+    model.bind('change:user.name', function(model, val) { console.log(val); });
+    model.bind('change:user', function(model, val) { console.log(val); });
+
     //Wildcards are supported
     model.bind('change:user.*', function() {});
 

distribution/deep-model.js

@@ -145,13 +145,17 @@
             separator = DeepModel.keyPathSeparator;
 
         for (var key in obj) {
+            if (!obj.hasOwnProperty(key)) continue;
+
             var val = obj[key];
 
             if (val && val.constructor === Object && !_.isEmpty(val)) {
                 //Recursion for embedded objects
                 var obj2 = objToPaths(val);
 
                 for (var key2 in obj2) {
+                    if (!obj2.hasOwnProperty(key2)) continue;
+
                     var val2 = obj2[key2];
 
                     ret[key + separator + key2] = val2;
@@ -346,6 +350,25 @@
                 }
                 //</custom code>
               }
+              //<custom code>
+              // Flag triggered events so that they are not triggered more then once.
+              var triggered = {};
+              _.each(changes, function(key) {
+
+                var fields = key.split(separator);
+
+                //Trigger change events for parent keys without wildcard (*) notation.
+
+                for(var n = fields.length - 1; n > 0; n--) {
+                  var parentKey = _.first(fields, n).join(separator);
+
+                    if (!triggered[parentKey]) {
+                        this.trigger('change:' + parentKey, this, getNested(current, parentKey), options);
+                    }
+                    triggered[parentKey] = true;
+                }
+              }, this);
+              //</custom code>                
             }
 
             if (changing) return this;

src/deep-model.js

@@ -24,13 +24,17 @@
             separator = DeepModel.keyPathSeparator;
 
         for (var key in obj) {
+            if (!obj.hasOwnProperty(key)) continue;
+
             var val = obj[key];
 
             if (val && val.constructor === Object && !_.isEmpty(val)) {
                 //Recursion for embedded objects
                 var obj2 = objToPaths(val);
 
                 for (var key2 in obj2) {
+                    if (!obj2.hasOwnProperty(key2)) continue;
+
                     var val2 = obj2[key2];
 
                     ret[key + separator + key2] = val2;
@@ -225,6 +229,25 @@
                 }
                 //</custom code>
               }
+              //<custom code>
+              // Flag triggered events so that they are not triggered more then once.
+              var triggered = {};
+              _.each(changes, function(key) {
+
+                var fields = key.split(separator);
+
+                //Trigger change events for parent keys without wildcard (*) notation.
+
+                for(var n = fields.length - 1; n > 0; n--) {
+                  var parentKey = _.first(fields, n).join(separator);
+
+                    if (!triggered[parentKey]) {
+                        this.trigger('change:' + parentKey, this, getNested(current, parentKey), options);
+                    }
+                    triggered[parentKey] = true;
+                }
+              }, this);
+              //</custom code>                
             }
 
             if (changing) return this;

test/deep-model.test.js

@@ -262,6 +262,8 @@ test("set: Triggers model change:[attribute] events", function() {
             'change:user.name.first',
             'change:user.name.*',
             'change:user.*',
+            'change:user.name',
+            'change:user',
             'change'
         ]);
     })();
@@ -503,6 +505,8 @@ test("unset: Triggers model change:[attribute] events", function() {
             'change:user.name.first',
             'change:user.name.*',
             'change:user.*',
+            'change:user.name',
+            'change:user',
             'change'
         ]);
     })();