RE: [IndexedDB] Sync API for setVersion Changes

On Thu, Apr 21, 2011 at 4:26 PM, Jonas Sicking wrote:
> On Thu, Apr 21, 2011 at 2:50 PM, Israel Hilerio <israelh@microsoft.com>
> wrote:
> > On Mon, Apr 20, 2011 at 3:12 PM, Jonas Sicking wrote:
> >>
> >> Looks great to me. Remember that we need to make both .transaction
> >> and .setVersion throw if called within the callback from either of them.
> >>
> >> / Jonas
> >>
> >> On Mon, Apr 18, 2011 at 11:22 AM, Israel Hilerio
> >> <israelh@microsoft.com>
> >> wrote:
> >> > I've created a bug to outline a potential change to the Sync API to
> >> > handle
> >> the context for setVersion transaction inside a callback method.  I
> >> wanted to follow up with the group to see what other people's thought
> >> about this change.  The Bug has all the details for the proposed
> >> change.  The bug# is
> >> 12438 [1].
> >> >
> >> > Thanks,
> >> >
> >> > Israel
> >> >
> >> > [1] http://www.w3.org/Bugs/Public/show_bug.cgi?id=12438
> >> >
> >> >
> >> >
> > Good catch.  What about adding the following to those two methods:
> >
> > New setVersion Exception:
> > * The setVersion method needs to throw IDBDatabaseException with a
> NOT_ALLOWED_ERR description when the setVersion method is called from
> within the IDBTransactionCallback of a transaction method.
> >
> > New transaction Exception:
> > * The transaction method needs throw IDBDatabaseException  with a
> NOT_ALLOWED_ERR description when the transaction method is called from
> within the IDBTransactionCallback of a setVersion method.
> 
> Both should say "within the IDBTransactionCallback of a transaction or
> setVersion method", right? Or simply "within the IDBTransactionCallback".
> 
> / Jonas

Makes sense.  I will add these descriptions to the bug and will coordinate with Eliot to make these changes/updates to the spec.

israel

Received on Thursday, 21 April 2011 23:37:35 UTC