backbone-nested icon indicating copy to clipboard operation
backbone-nested copied to clipboard

change event not triggered when setting an object that contains an array

Open theojapa opened this issue 10 years ago • 4 comments

var model = new Backbone.NestedModel({
    a: {
        b: [
            { c: 1 },
            { d: 2 }
        ]
    }
});

model.bind('change:a.b[0].c', function() { console.log('c changed.'); });

// The following set triggers the change event:
model.set('a.b[0].c', 3);

// The following does not trigger the change event:
model.set({
    a: {
        b: [
            { c: 4 },
            { d: 2 }
        ]
    }
});

But this works as expected sans array:

var model2 = new Backbone.NestedModel({
    a: {
        b: { 
            c: 1
        }
    }
});

model2.bind('change:a.b.c', function() { console.log('c changed.'); });

// The following set triggers the change event:
model2.set({
    a: {
        b: {
            c: 12
        }
    }
});

theojapa avatar Mar 05 '15 15:03 theojapa

Hmm, interesting.

http://jsbin.com/fezuyerire/2/edit?js,console

Seems like a bug, unfortunately :confused: Arrays aren't a problem when they are top-level properties:

http://jsbin.com/fegexe/1/edit?js,console

I can't look into this until this weekend, but would more than welcome a pull request.

afeld avatar Mar 05 '15 15:03 afeld

The issue has happens when arrays are top-level properties:

http://jsbin.com/jixepuzace/1/edit?js,console

theojapa avatar Mar 05 '15 16:03 theojapa

Ohhhh, gotcha, it's the setting with an array. Bug!

afeld avatar Mar 05 '15 16:03 afeld

Have a fix; will create a PR shortly.

theojapa avatar Mar 05 '15 17:03 theojapa