- From: Joshua Bell <notifications@github.com>
- Date: Mon, 05 May 2025 14:30:24 -0700
- To: w3c/IndexedDB <IndexedDB@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
- Message-ID: <w3c/IndexedDB/pull/455/review/2816148928@github.com>
@inexorabletash requested changes on this pull request. Overall looks good, with some suggestions. > +When [=cursor/source handle=] is an [=index handle=], set [=cursor/source=] to [=index-handle/index|the index handle's associated index=]. +Otherwise, set [=cursor/source=] to [=object-store-handle/object store|the object store handle's associated object store=]. These lines read like imperative algorithm steps, but they don't have a trigger. How about this: ```suggestion If the cursor's [=cursor/source handle=] is an [=index handle=], then the cursor's [=cursor/source=] is [=index-handle/index|the index handle's associated index=]. Otherwise, cursor's [=cursor/source=] is [=object-store-handle/object store|the object store handle's associated object store=]. ``` > @@ -6721,6 +6719,7 @@ For the revision history of the second edition, see [that document's Revision Hi * Clarify that only [=transaction/inactive=] [=/transactions=] should attempt to auto-commit. (<#436>) * Correct [=/upgrade a database=] steps to handle aborted transactions. (<#436>) * Update [=/iterate a cursor=] value serialization to use [=/value=] for [=/object stores=] instead of [=index/referenced values=]. (<#452>) +* Add [=cursor/source handle=] to [=/cursor=] to avoid exposing internal indexes and object stores to script. ```suggestion * Add [=cursor/source handle=] to [=/cursor=] to avoid exposing internal indexes and object stores to script. (<#445>) ``` -- Reply to this email directly or view it on GitHub: https://github.com/w3c/IndexedDB/pull/455#pullrequestreview-2816148928 You are receiving this because you are subscribed to this thread. Message ID: <w3c/IndexedDB/pull/455/review/2816148928@github.com>
Received on Monday, 5 May 2025 21:30:28 UTC