yathit / ydn-db

Javascript database module for Indexeddb, Web SQL and localStorage storage mechanisms supporting version migration, advanced query, SQL and transaction.
Apache License 2.0
503 stars 41 forks source link

Bugfix/90 #112

Closed mehdirande closed 8 years ago

mehdirande commented 8 years ago

This PR is related to the PR in ydn-base (from the branch with the same name). It's mean to support call of onSuccess in the simple_cursor with a null parameter. Another fix was necessary to make indexCursor work with simple storage : The index cache was trying to add entries for storage entries without value for the given index which was throwing a ""Failed to execute 'cmp' on 'IDBFactory': The parameter is not a valid key.". The second commit skip storage entries without values for the index .


This change is Reviewable