Degrade gracefully in IE on local files

[JeroenVu]

localStorage is not available in Internet Explorer when loading files directly from the filesystem.
This "fix" prevents your whole app to crash. Everything works, but nothing gets saved.

I found a project, called jStorage (http://www.jstorage.info/) that tries to fix this issue.

[gsklee]

I am not sure if this actually fixes anything. I mean, yeah, it no longer crashes, but you still have a malfunctioning Web Storage. In such cases, you might wanna consider using other data persistence solution.

[JeroenVu]

Too bad you don't see the added value of not crashing an entire application... I'm using it in my project anyway.

Btw, thanks for your great work on this module!

[gsklee]

Alright - can you do a test for me? Since the fake Web Storage API you provided didn't even include a getItem method, I suspect you only need an empty object to cheat IE9.

Can you modify L43 to var webStorage = $window[storageType] || {}, and see if this prevents the crash?

If not, make it var webStorage = $window[storageType] || {setItem: function() {}, removeItem: function() {}},

Many thanks!

[JeroenVu]

Using an empty object prevents the crash in IE7, IE8, IE9 and IE10 (tested with browser mode in IE10), but it shows an error in the console: SCRIPT438: Object doesn't support property or method 'setItem'.

That's why I added the empty functions. I could have added getItem also, but it will never be called, because i < webStorage.length will always evaluate to false since webStorage.length is undefined.

It might not be the best solution for every project to just silently ignore this problem though. What about logging something meaningful to the console?

[gsklee]

It would be great if you can try catch the error and log a message, yep.

[gsklee]

Just did a test on a real IE8 and the app didn't crash at all; instead there is a TypeError: 'length' is null or not an objectundefined error in the console.