Summary: | IDBIndex.openCursor() matches indices on multiple object stores | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | David Fahlander <david.fahlander> | ||||
Component: | WebCore Misc. | Assignee: | Brady Eidson <beidson> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Major | CC: | achristensen, alecflett, beidson, commit-queue, david.fahlander, jsbell, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | HTML5, InRadar | ||||
Version: | Safari Technology Preview | ||||||
Hardware: | Mac | ||||||
OS: | OS X 10.11 | ||||||
URL: | https://bl.ocks.org/dfahlander/a7a0608513d527742c5a09e3c95dd343 | ||||||
Attachments: |
|
Description
David Fahlander
2016-06-16 02:59:55 PDT
Any news on this? Any news on this one? I adore the try to really support the full indexedDB spec, but as you are about to release Safari 10, please view this severe indexedDB bug! You have the URL (https://cdn.rawgit.com/dfahlander/Dexie.js/v1.4.0-rc.1/test/run-unit-tests.html) that reproduces the error. And you have the description on what's happening (in the main description of this bug). Please contact me if you need to get more explanations about it. Kind Regards, David Fahlander, author of dexie.js (http://dexie.org) Added another simpler repro of this bug (simple gist) that is not Dexie-dependant: https://bl.ocks.org/dfahlander/a7a0608513d527742c5a09e3c95dd343 Tested again with Safari 10.0 (12602.1.38.2) on browserstack.com. I've nailed it down further and it seems this behaviour occurs only when the different object stores primary keys collide. In my repro, I use options {autoIncrement: true} when creating the ObjectStore, which will for sure make the primary keys on 'schools' and 'students' collide. Please keep me in the loop when testing this out. I'll be happy to help out. Feel free to run the dexie unit tests that pass on Chrome and Firefox: https://cdn.rawgit.com/dfahlander/Dexie.js/v1.4.2/test/run-unit-tests.html I've been in contact with Brady Eidson about this issue over twitter, so he is informed about it. Please make sure this bug is taken care of and assigned to the right person. The simple reproduction of the bug is available here: https://bl.ocks.org/dfahlander/a7a0608513d527742c5a09e3c95dd343 and it also contains text files with expected output versus Safari 10 output. Please fix this bug. This was a problem in the query for index cursors. Easy fix, patch coming soon. Wish I saw this when it was filed :( With my fix, with the Dexie test suite, I see: 1122 assertions of 1122 passed, 0 failed. Created attachment 289607 [details]
Patch
Wow :-) This is great news! |