Populate cacheTimestamps and cacheHashes #25

Merged
merged 3 commits into from Jan 17, 2012

Projects

None yet

2 participants

@thatmarvin
  1. I made settings a module variable so that I can init var assetManager = require('connect-assetmanager'); assetManager(assets) in app.js, and just call assetManager() elsewhere without having to pass in assets again.
  2. cacheTimestamps and cacheHashes are also made module variables. They were otherwise returning empty objects no matter where/when I invoked assetManager().

Both changes assume that the asset groups only needs to be initialized once.

Marvin Tam added some commits Dec 1, 2011
Marvin Tam - make settings variable local to the module so that assetManager() (…
…no args) would just use whatever that was previously init

- make cacheHashes and cacheTimestamps local to the module so that they are properly populated and accessible
c7cda1b
Marvin Tam throw exception if no asset groups exists adf7cff
Marvin Tam remove extra spaces 45a3198
@mape mape merged commit 8c57637 into mape:master Jan 17, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment