Repository navigation
[QTI] A QTI item with a comment quoting its root tag is published without the node's language #6235
Description
Activity
Plan:
set_qti_item_languagestampsxml:langon the real<qti-assessment-item>root even when a comment (or other prolog markup) quotes that start tag before it.- Match the root start tag after the prolog
- /simplify pass
Assumptions — the issue did not settle these; say so if one is wrong.
- PIs and DOCTYPE are skipped as well as comments; if unwanted, the pattern is wider than the issue asks but no test breaks.
- A
>inside a root attribute value still ends the tag match early ([^>]*); not in scope, and serializers escape it as>. If items carry a raw>,xml:langlands mid-value. - No match after the prolog returns
raw_dataunchanged, as today; if a caller expects an error, it needs a separate change.
@rtibblesbot's comments are generated by an LLM, and should be evaluated accordingly
How was this generated?
- Implemented code following the pre-approved plan using test-driven development (red-green-refactor)
- Ran the project's full test suite and linter before committing
- For UI changes: captured live screenshots to verify rendering
- For backend changes: verified with example API/CLI output
- added a commit that references this issue
on Oct 7, 2026
❌ This issue is not open for contribution. Visit Contributing guidelines to learn about the contributing process and how to find suitable issues.
Target branch: unstable
Observed behavior
set_qti_item_language(utils/assessment/qti/media.py) rewrites the first<qti-assessment-item …>match in the raw text.xml:langis written inside the comment; the real root keeps its own language.validate_qti_item.Steps to reproduce
<!-- <qti-assessment-item foo="1"> -->before the root ofVALID_CHOICE_ITEM(tests/utils/qti/test_validation.py).set_qti_item_language(xml, "fr").xml:lang="en-US".Acceptance Criteria
set_qti_item_languagesets the language on the real root regardless of comments before it.