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

XML: change doctype values #44

Closed
Intelligent2013 opened this issue Aug 7, 2020 · 2 comments
Closed

XML: change doctype values #44

Intelligent2013 opened this issue Aug 7, 2020 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@Intelligent2013
Copy link

Using of XSLTs modules depends on content of doctype XML element. To prevent of further issues in command line calls due names with space chars, upper cases, etc. need to change doctype to more short values.

Non-correct values found here:

  • mpfd-bpn, mpfd-compliance:
<doctype>Supervision of MPF Intermediaries</doctype>
  • mpfd-guidelines:
<doctype>Guidelines</doctype>
  • mpfd-standards:
<doctype>Compliance Standards for MPF Trustees</doctype>

Source: metanorma/mn-native-pdf#162 (comment)

@Intelligent2013 Intelligent2013 added the bug Something isn't working label Aug 7, 2020
@Intelligent2013 Intelligent2013 assigned opoudjis and unassigned CAMOBAP Aug 14, 2020
@opoudjis opoudjis added this to Needs triage in Nick Nicholas via automation Aug 14, 2020
@opoudjis opoudjis moved this from Needs triage to Current in Nick Nicholas Aug 14, 2020
@opoudjis
Copy link
Contributor

Doctype was entered into MPF samples as free text. Given that this is a proof of concept gem with zero analysis, there is no controlled vocabulary for MPFA documents. It is also clear that uppercase characters and spaces need to be excluded from doctype, if they haven't been already; ruling them as invalid is not going to be enough, given the downstream implications identified here. So doctype needs to be normalised before it even gets validated.

@opoudjis
Copy link
Contributor

Need to use doctype() as an accessor function throughout Metanorma.

opoudjis added a commit to metanorma/metanorma-standoc that referenced this issue Aug 15, 2020
opoudjis added a commit to metanorma/metanorma-generic that referenced this issue Aug 15, 2020
opoudjis added a commit to metanorma/metanorma-csa that referenced this issue Aug 15, 2020
opoudjis added a commit to metanorma/metanorma-cc that referenced this issue Aug 15, 2020
opoudjis added a commit to metanorma/metanorma-gb that referenced this issue Aug 15, 2020
opoudjis added a commit to metanorma/metanorma-ietf that referenced this issue Aug 15, 2020
opoudjis added a commit to metanorma/metanorma-iso that referenced this issue Aug 15, 2020
opoudjis added a commit to metanorma/metanorma-standoc that referenced this issue Aug 15, 2020
opoudjis added a commit to metanorma/metanorma-itu that referenced this issue Aug 15, 2020
opoudjis added a commit to metanorma/metanorma-m3aawg that referenced this issue Aug 15, 2020
opoudjis added a commit to metanorma/metanorma-ogc that referenced this issue Aug 15, 2020
opoudjis added a commit to metanorma/metanorma-un that referenced this issue Aug 15, 2020
Nick Nicholas automation moved this from Current to Closed Aug 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Nick Nicholas
  
Closed
Development

No branches or pull requests

3 participants