Mercurial > bugzilla-dashboard
changeset 81:e3de1fe32f40
made cache module require 'storage' instead of 'window'
author | Atul Varma <avarma@mozilla.com> |
---|---|
date | Sun, 25 Apr 2010 22:44:00 -0700 |
parents | 677df912e92d |
children | 03bf0a1b85da |
files | js/modules/app.js js/modules/cache.js |
diffstat | 2 files changed, 6 insertions(+), 5 deletions(-) [+] |
line wrap: on
line diff
--- a/js/modules/app.js Sun Apr 25 22:26:14 2010 -0700 +++ b/js/modules/app.js Sun Apr 25 22:44:00 2010 -0700 @@ -3,6 +3,7 @@ var bugzilla = require("app/bugzilla-auth").create(options.Bugzilla); moduleExports.bugzilla = bugzilla; moduleExports.window = options.window; + moduleExports.storage = options.window.localStorage; moduleExports.jQuery = options.jQuery; require("app/ui").init(options.window.document);
--- a/js/modules/cache.js Sun Apr 25 22:26:14 2010 -0700 +++ b/js/modules/cache.js Sun Apr 25 22:44:00 2010 -0700 @@ -1,8 +1,8 @@ Require.modules["cache"] = function(exports, require) { const CACHE_NAME = "cache"; - var window = require("window"); - var cache = window.localStorage.getItem(CACHE_NAME); + var storage = require("storage"); + var cache = storage.getItem(CACHE_NAME); if (cache) cache = JSON.parse(cache); @@ -14,12 +14,12 @@ // Remove the key first, to get around a strange iPad // issue: http://stackoverflow.com/questions/2603682/is-anyone-else-receiving-a-quota-exceeded-err-on-their-ipad-when-accessing-locals - window.localStorage.removeItem(CACHE_NAME); + storage.removeItem(CACHE_NAME); // TODO: We should really catch QUOTA_EXCEEDED_ERR here, // which could be thrown if the user is in private // browsing mode. - window.localStorage.setItem(CACHE_NAME, JSON.stringify(cache)); + storage.setItem(CACHE_NAME, JSON.stringify(cache)); }; exports.get = function get(key) { @@ -27,7 +27,7 @@ }; exports.clear = function clear() { - window.localStorage.removeItem(CACHE_NAME); + storage.removeItem(CACHE_NAME); cache = {}; }; };