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.

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

Add health warning for opaque string comparison #178

Closed
aphillips opened this issue Mar 23, 2017 · 2 comments
Closed

Add health warning for opaque string comparison #178

aphillips opened this issue Mar 23, 2017 · 2 comments
Labels
has-pr i18n-needs-resolution Issue the Internationalization Group has raised and looks for a response on. needs-pr
Milestone

Comments

@aphillips
Copy link

https://www.w3.org/TR/IndexedDB-2/#object-store-construct

Object store names are always compared in a case-sensitive manner, as opaque sequences of 16-bit code units.

The object store name comparison is fine, so long as "comparison" means "for identity/equality". Case-sensitivity is recommended by the I18N WG and Charmod-Norm. However, a health warning may be warranted here, since it is more than just case-sensitive: it is sensitive to the exact encoding sequence. For example, normalization form affects this. Charmod-Norm provides a somewhat exhaustive list of this kind of variation ;-).

A health warning might be:

Object store names are always compared as opaque sequences of 16-bit code units. As a result, name comparison is sensitive to variations in case as well as other minor variations such as normalization form, the inclusion or omission of controls, and other variations in Unicode text. [[Charmod-Norm]]

This is I18N comment w3c/i18n-activity#363. Please add the i18n-comment label.

@inexorabletash inexorabletash added the i18n-needs-resolution Issue the Internationalization Group has raised and looks for a response on. label Mar 23, 2017
@inexorabletash
Copy link
Member

Note that databases, object stores, and indexes all have names which behave the same way. I'll spin up a construct for that.

@inexorabletash
Copy link
Member

PR: #184

@aphillips - can you review?

(and in case I haven't said it yet - thanks for all the awesome feedback!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has-pr i18n-needs-resolution Issue the Internationalization Group has raised and looks for a response on. needs-pr
Projects
None yet
Development

No branches or pull requests

2 participants