Wrap Backbone.sync instead of redefining it

codebrew / backbone-rails

Easily use backbone.js with rails 3.1

MIT License

1.62k stars 255 forks source link

Wrap Backbone.sync instead of redefining it #140

Closed gabetax closed 10 years ago

gabetax commented 11 years ago

Fixes #139

Backbone.sync in backbone_rails_sync.js has been effectively an independent fork, which has caused an issue with passed in error handlers in backbone 0.9.9. Instead of continuing to maintain a separate fork of Backbone.sync, this wraps the original vendor Backbone.sync, to implement the following changes:

Sets the X-CSRF-Token HTTP header, still backwards compatible with the
Serializes data inside of paramRoot, if specified for the model

manusajith commented 10 years ago

@gabetax I have added this to the master branch after fixing conflicts. Thanks.!