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

Default att values incorrectly processed by Roma web app #1213

Closed
TEITechnicalCouncil opened this issue Sep 15, 2013 · 5 comments
Closed

Default att values incorrectly processed by Roma web app #1213

TEITechnicalCouncil opened this issue Sep 15, 2013 · 5 comments

Comments

@TEITechnicalCouncil
Copy link

When an attribute has a specific default value in the ODD source, and the user attempts to change the value list for it using web roma, the form does not display the existing default value. Consequently, the user has no way of knowing that it exists, and may easily supply a list of attribute values which do not include it, thus generating an invalid DTD or XSD schema. Tested this with <list> which retains the default "simple" even though it does not appear on the screen.

Original comment by: @lb42

@TEITechnicalCouncil
Copy link
Author

does this equally apply when you construct an ODD by hand? that is, if you
provide a list of allowed values, should the defaultVal always be discarded by the ODD processor? I can't fix Roma this side of hell freezing over, but if odd2dtd is wrong, that should be fixed.

Original comment by: @sebastianrahtz

@TEITechnicalCouncil
Copy link
Author

A hand-written ODD in which no new <defaultVal> is specified to replace an existing one should reasonably expect the existing default to continue to apply, if the parent <attDef> is being processed in mode CHANGE. The bug I am reporting here is that web roma does not display an existing defaultVal so the user has no way of telling they are about to shoot themselves in the foot. In the long term of course the correct solution is to deprecate and remove <defaultVal> passim as previously noted.

Original comment by: @lb42

@TEITechnicalCouncil
Copy link
Author

  • Group: AMBER --> ROMA

Original comment by: @sebastianrahtz

@TEITechnicalCouncil
Copy link
Author

This issue was originally assigned to SF user: rahtz
Current user is: sebastianrahtz

@peterstadler
Copy link
Member

moved to TEIC/Roma-Antiqua#21, closing here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants