15234 2011-12-16 18:18:46 +0000 Throw if IDBTransaction.objectStore() or IDBObjectStore.index() called after transaction finished 2011-12-27 22:46:25 +0000 1 1 1 Unclassified WebAppsWG Indexed Database API unspecified All All RESOLVED FIXED P2 normal --- 1 jsbell dave.null eliotgra israelh mike public-webapps public-webapps-bugzilla oldest_to_newest 61681 0 jsbell 2011-12-16 18:18:46 +0000 The spec currently requires that IDBTransaction.objectStore() and IDBObjectStore.index() are callable after the containing transaction has finished. Since the "child" objects (IDBObjectStore and IDBIndex, respectively) contain references to the parent object, this complicates lifecycle management of these objects for implementations. The implementation requirements could be simplified by specifying that these methods throw InvalidStateError once the transaction is finished. Alternately, the spec could be to throw TransactionInactiveError if the transaction is not active, much like the get() methods. 61715 1 israelh 2011-12-17 00:26:32 +0000 Per email thread discussion, we should throw InvalidStateError for both of the APIs below. 62064 2 eliotgra 2011-12-27 21:35:52 +0000 Each method already currently has the following exception and description: **InvalidStateError **Occurs if a request is made on a source object that has been deleted or removed. Please let me know if I am misreading this and you're asking for something else. Thanks, Eliot 62074 3 jsbell 2011-12-27 22:12:26 +0000 It's not clear that clause covers the case this issue is discussing. Here's a snippet: var trans = db.transaction("store"); var store = trans.objectStore("store"); // do something useful with store trans.oncomplete = function() { var store = trans.objectStore("store"); // this issue is requesting that the above line should throw }; In this case, it does not seem that the phrase "a source object that has been deleted or removed" applies. 62075 4 eliotgra 2011-12-27 22:16:06 +0000 How _would_ you phrase it to cover this? I am happy to make the change in the spec. 62076 5 jsbell 2011-12-27 22:28:12 +0000 The trivial change would be: For IDBTransaction.objectStore(): "Occurs if a request is made on a source object that has been deleted or removed, or if the transaction has finished." For IDBObjectStore.index(): "Occurs if a request is made on a source object that has been deleted or removed, or if the transaction the object store belongs to has finished." I assume the "deleted or removed" phrasing - used globally in the spec - applies to cases where the IDBDatabase has been deleted or the IDBObjectStore has been deleted within a VERSION_CHANGE transaction. 62077 6 eliotgra 2011-12-27 22:46:25 +0000 Changes from comment 5 are in the Editor's draft. Thanks for your help, Joshua. Have a happy new year. Eliot