jensarps/IDBWrapper

Polyfill support?

Closed this issue · 6 comments

minj commented

Hey, I love your library.

Does it work with any of the polyfills? I want to use it on Safari.

Sweet, good to know, thanks! That Facebook polyfill has some serious test coverage – but the latest commit is more than a year old. I'll create some integration tests for IDBWrapper to make sure polyfills like this work as expected.

I added some integration tests to check if a polyfill supports the IndexedDB API enough to safely work with IDBWrapper.

I checked for https://github.com/facebook/IndexedDB-polyfill and https://github.com/axemclion/IndexedDBShim, and both seem to work fine. I've uploaded the test pages here:

So, both should be safe to use :)

These tests do not pass for me on Safari Version 7.0.3 (9537.75.14)

Is it something to do with reloading the same store after it has already been created?

Did something change about how indexes are created?

The tests should run fine on both initial and subsequent runs – I need to get my hands on a Safari 7 and check that out.

Thanks for reporting!

Hm, same for me – fails to open the store on the first run but works on second run (where the indexes are then missing). This is pretty sad, as I don't know of any other IndexedDB polyfills.