Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Why is delete() disallowed for key cursors? #62

Open
inexorabletash opened this issue Jan 21, 2016 · 7 comments
Open

Why is delete() disallowed for key cursors? #62

inexorabletash opened this issue Jan 21, 2016 · 7 comments
Labels
blink +1 cursors easy feature request gecko +1 needs-pr needs-tests
Milestone
V3

Comments

@inexorabletash
Copy link
Member

The delete() method has:

If this cursor's key only flag is set, throw an InvalidStateError exception.

I can't think of a good reason for this limitation. Anyone know why it's in there?
@inexorabletash inexorabletash changed the title Why is delete() restricted to IDBCursorWithValue? Why is delete() disallowed for key cursors? Jan 21, 2016
@sicking
Copy link
Contributor

sicking commented Jan 21, 2016

No idea. I think we should remove that restriction.

@inexorabletash
Copy link
Member Author

Same for update() ?

@sicking
Copy link
Contributor

sicking commented Jan 22, 2016

Yep

@inexorabletash
Copy link
Member Author

@beidson @aliams - opinions?

@aliams
Copy link

aliams commented Jun 9, 2016

This limitation could be for the same reason that I mentioned here: #68 (comment)

@beidson
Copy link

beidson commented Jun 9, 2016

I don't buy the "IDBIndex is only supposed to be a window into an IDBObjectStore" argument.

A record is a record is a record.

I'm okay with this working.

@beidson beidson mentioned this issue Jun 9, 2016
Closed
@inexorabletash
Copy link
Member Author

TPAC 2019 Web Apps Indexed DB triage notes:

Possible V3 fodder, unsure if there is demand. This was probably just an observation about a special case in the spec.

@inexorabletash inexorabletash added this to the V3 milestone Sep 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
blink +1 cursors easy feature request gecko +1 needs-pr needs-tests
Projects
None yet
Milestone
V3
Development

No branches or pull requests
4 participants
@inexorabletash @sicking @aliams @beidson