Doesn't work with browserify

[adamscybot]

Backbone associations can not be used in a browserify stack as both the window and exports modules are defined as objects at runtime on the clientside. For this reason, the require('backbone') etc calls are not made and so instead it assigns Backbone and _ to exports._ and exports.Backbone, which are by definition, empty. Presumably that branch of the if logic should only fire when root == window and never root == exports as it does now in a browserify environment.

(typeof exports === "object") instead of (typeof window === "undefined") resolves the issue. Basically the key is to give priority to checking exports over window first.

One way I've seen which seems to cover all the bases is:

  // CommonJS
  if (typeof exports == "object") {
    var _ = require("underscore"),
    Backbone = require("backbone");
    exports = module.exports = Backbone;
  }
  // Browser
  else if (typeof _ !== "undefined" &&
    typeof Backbone !== "undefined") {
    _ = root._;
    Backbone = root.Backbone;
  }

[jdkanani]

Thanks @adamscybot for pointing this out. I have just pushed code to check exports over window as per your code snippet. Let me know if it works for you.

[adamscybot]

Works correctly, thanks. I think the .min version needs updating though. Also a minor version push to NPM would be useful so I can stop pulling this repo directly in my package.json :).

Thanks again!

[dhruvaray]

Wonderful! Sure...In a couple of days at most. Don't forget to show your appreciation by starring us!

[dhruvaray]

Took longer than I thought, but v0.5.3 is now on npm!