derbyjs / racer

Realtime model synchronization engine for Node.js
1.19k stars 118 forks source link

model.pass is broken #104

Closed adrianheine closed 11 years ago

adrianheine commented 11 years ago

Since ddbbbfe1ec819f257ca592f8dbf819ec35712ccc, the following test script does not show the expected behavior:

var racer = require('./lib/racer');
var store = racer.createStore();
var model = store.createModel();

model.on('set', function () {
  console.log(arguments);
});

model.pass('pass').set('root', 'value');

Instead, nothing is passed.

lefnire commented 11 years ago

i think you need the second argument:

model.on('set', '*', function(){
  console.log(arguments);
})
adrianheine commented 11 years ago

Thanks for your hint, but it doesn't change anything. Furthermore, I git bisect'ed the thing, so I'm pretty sure the commit mentioned above did break this.

nateps commented 11 years ago

This does appear to be a bug.

On Mon, Mar 18, 2013 at 7:27 AM, Adrian Lang notifications@github.comwrote:

Thanks for your hint, but it doesn't change anything. Furthermore, I git bisect'ed the thing, so I'm pretty sure the commit mentioned above did break this.

— Reply to this email directly or view it on GitHubhttps://github.com/codeparty/racer/issues/104#issuecomment-15057472 .

nateps commented 11 years ago

fixed in 0.5