vidigami / backbone-orm

A polystore ORM for Node.js and the browser
http://vidigami.github.io/backbone-orm
MIT License
237 stars 15 forks source link

Error redefining sync method #2

Closed pshangov closed 10 years ago

pshangov commented 10 years ago

Hi, I am attempting to get started with backbone-orm. When I run the following bit of code:

var Backbone = require("backbone");

var Project = Backbone.Model.extend({
    urlRoot : "xxx"
});

Project.prototype.sync = require("backbone-orm").sync(Project);

I get this error:

 ~\Development\backbone-fs> node .\bug.js

C:\Users\pshangov\Development\backbone-fs\node_modules\backbone-orm\lib\extensions\model_stream.js:5
unction ctor() { this.constructor = child; } ctor.prototype = parent.prototype
                                                                ^
TypeError: Cannot read property 'prototype' of undefined
    at __extends (C:\Users\pshangov\Development\backbone-fs\node_modules\backbone-orm\lib\extensions\model_stream.js:5:199)
    at C:\Users\pshangov\Development\backbone-fs\node_modules\backbone-orm\lib\extensions\model_stream.js:19:5
    at Object.<anonymous> (C:\Users\pshangov\Development\backbone-fs\node_modules\backbone-orm\lib\extensions\model_stream.js:51:5)
    at Object.<anonymous> (C:\Users\pshangov\Development\backbone-fs\node_modules\backbone-orm\lib\extensions\model_stream.js:53:4)
    at Module._compile (module.js:449:26)
    at Object.Module._extensions..js (module.js:467:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:362:17)
    at require (module.js:378:17)

OS: Windows 7 64-bit node version 0.8.21 backbone-orm version 0.5.1

The same happens on latest version from git (eb085e5400f65a0d7b775b2de07f7e785ea2afab)

There is no error if I comment out the "Project.prototype.sync = ..." line.

Cheers, Peter

gwilymhumphreys commented 10 years ago

Thanks for letting us know mate (and soz for the delay in responding).

The error here is from us assuming Node 0.10+ streams are available - are you able to upgrade to Node 0.10+?

I've pushed a check that should allow it to work on older versions, but I'd recommend upgrading Node if possible.

Cheers again!

pshangov commented 10 years ago

Works now, thanks!