Skip to content

fix(examples): make example documents schema-valid and resolvable - #163

Draft
gronke wants to merge 2 commits into
mainfrom
fix/example-documents
Draft

fix(examples): make example documents schema-valid and resolvable#163
gronke wants to merge 2 commits into
mainfrom
fix/example-documents

Conversation

@gronke

@gronke gronke commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Resolve the examples' xi:includes via symlinks to source/, and fix the remaining schema violations:

  • exampleinvoice: denomination usd
  • examplereport: drop stale <duration> and unused pieHeight; add required @number
  • examplequickscope: drop redundant <days>; give <planning> start/end

blocked by #162

gronke added 2 commits June 10, 2026 10:30
The XSD schemas were stricter and less complete than the tooling and
templates, so valid PenText documents failed XSD validation. Align the
schemas with what the XSLT actually supports:

- common.xsd: remove the duplicate global 'location' element.
- pentestreport.xsd: make finding @Prefix optional, make <impact> optional,
  and allow <code> in technicaldescription, section and appendix.
- offerte.xsd: add the placeholder elements the stylesheet resolves
  (p_draftdue, p_duration, p_location, p_participants) and allow
  participants, location and duration inside activityinfo.

Schema-only change; every edit is a relaxation or addition, so documents
that validated before still validate.
Resolve the examples' xi:includes via symlinks to source/, and fix the
remaining schema violations:
- exampleinvoice: denomination usd
- examplereport: drop stale `<duration>` and unused `pieHeight`; add required `@number`
- examplequickscope: drop redundant `<days>`; give `<planning>` start/end
@gronke
gronke force-pushed the fix/example-documents branch from 11acfda to 159ec77 Compare June 10, 2026 17:08
@gronke
gronke marked this pull request as draft June 10, 2026 17:08
Base automatically changed from fix/dtd-schema-completeness to main June 30, 2026 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant