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

case sensitivity for values #30

Closed
berlotti opened this issue Jan 4, 2022 · 3 comments
Closed

case sensitivity for values #30

berlotti opened this issue Jan 4, 2022 · 3 comments
Labels
Agreed documentation Improvements or additions to documentation

Comments

@berlotti
Copy link
Member

berlotti commented Jan 4, 2022

Are the values in the xml nodes case sensitive or not?
Or should we add an attribute to define case sensitivity? Per node or per IDS?

@berlotti berlotti added to vote on To vote on during next meeting January labels Jan 4, 2022
@dro007
Copy link

dro007 commented Jan 10, 2022

This depends on who our target market is. If we are supporting software engineers who are used to typing in camel case then case sensitivity would be ok.
If we want to support "normal" people, then case sensitivity would be extremely frustrating for them. There could be development environments that supported validating case sensitivity for "normal" people, but this adds another layer of complexity.

@aothms
Copy link

aothms commented Jan 11, 2022

My thoughts:

  • entity predefined type are special since they refer to existing definitions. I'd say they need to be case-insensitive, as EXPRESS is also case insensitive and it allows people to copy paste from both the mixed case docs as well as upper case serialization.
  • some pset names are standardized others aren't. Maybe it should be forced case insensitive. I think errors are easily made there and there is no value in differencing case. Same for the property names.
  • for most other cases I'd say it's up to the user, so I'd say an attribute is best
  • we need to consider what to do with the combination of case insensitivity and pattern. So we could either say case-sensitive is an attribute of e.g simpleValue and enum so that pattern is by definition case sensitive, but that would allow case-sensitivity specification on entity. Attribute can also be on facet nodes.
  • Keep in mind that you can't case convert a regex and expect the same behaviour when ran on a case-converted string (because of the character groups such as \S \W).

@berlotti berlotti added Agreed documentation Improvements or additions to documentation and removed January to vote on To vote on during next meeting labels Feb 10, 2022
@berlotti
Copy link
Member Author

Same as for #49 : follow XSD standard of case sensitive

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Agreed documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

3 participants