leaguevine / leaguevine-ultistats

MIT License
18 stars 4 forks source link

Remove Leaguevine requirement from modules #37

Open cboulay opened 12 years ago

cboulay commented 12 years ago

There was a period of time where I was trying to figure out some circular dependencies so I tried to flatten the requirements as much as possible. In so doing, I replaced app.api several places with Leaguevine.API. However, this means that Leaguevine must be required everywhere, not just in namespace.js, and that switching to an API not named "Leaguevine" at some point would be a huge headache.

The original goal was to have the Leaguevine-specific definitions in leaguevine.js, and have namespace.js (require | import | include) leaguevine.js and create a namespace.app.api object based on Leaguevine.API. All other modules would include namespace.js and would have access to the api this way. So I need to go back through the code and fix that.