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

Distinguish between dependent and independent styles when validating #39

Closed
rmzelle opened this issue Mar 31, 2011 · 6 comments
Closed
Labels

Comments

@rmzelle
Copy link
Member

rmzelle commented Mar 31, 2011

The CSL schema should probably be more strict for dependent styles (e.g. it shouldn't allow for stuff like style-options (punctuation-in-quote) on cs:style).

@bdarcus
Copy link
Member

bdarcus commented Mar 31, 2011

So that would probably suggest something like:

start = base | dependent
dependent = element cs:style { info }

@rmzelle
Copy link
Member Author

rmzelle commented Apr 26, 2011

See 36b2d4d

@rmzelle
Copy link
Member Author

rmzelle commented Apr 27, 2011

And 957a32f

@rmzelle rmzelle closed this as completed Apr 27, 2011
@rmzelle
Copy link
Member Author

rmzelle commented Oct 20, 2011

See also

0a47df6
56cc0c9
874daed

@rmzelle
Copy link
Member Author

rmzelle commented Oct 22, 2011

With this ticket, I split up the schema to have different requirements for dependent and independent styles, and I disallowed a few attributes and attribute values for each. A while back, I cleaned up the style repository so that all styles would upgrade correctly to CSL 1.0.1 and still be valid.

However, I've become worried that older CSL 1.0 styles in the wild (e.g. those installed by Zotero users) might become invalid CSL 1.0.1 styles, so I've relaxed the validation requirements again: ac98d78

The affected attributes/values are labelled in the schema as "legacy" or "obsolete". They can be removed in the CSL 1.1 schema, and stripped from the styles through the CSL 1.1 style upgrade XSLT.

@rmzelle rmzelle reopened this Oct 22, 2011
@rmzelle
Copy link
Member Author

rmzelle commented Oct 26, 2011

For removing obsolete attributes/values, see #58.

@rmzelle rmzelle closed this as completed Oct 26, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants