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

Assemble without xmlbase #733

Merged
merged 4 commits into from
Jul 1, 2024
Merged

Conversation

tomschr
Copy link
Collaborator

@tomschr tomschr commented Jun 27, 2024

We had problems image paths pointing to invalid parts. That was mainly due to having xml:base attributes inside the assembled bigfile. These attributes were taken into account when creating the image path.

For an assembled bigfile, this isn't needed. This customization layer discards these xml:base attributes.

tomschr added 2 commits June 27, 2024 14:50
* Allow to preserve or discard xml:base attributes
  (XSLT parameter preserve-xmlbase: 1 = preserve xml:base, 0 = discard them)
  This addition was needed to avoid wrong image paths as the xml:base
  attribute is taken into account to construct the path.
Add $preserve-xmlbase XSLT parameter to preserve or discard
xml:base attributes (default is to discard them)
@tomschr tomschr added cat-bug topic-assembly Everything related to assembly processing labels Jun 27, 2024
@tomschr tomschr self-assigned this Jun 27, 2024
@fsundermeyer fsundermeyer changed the base branch from main to 4.0.beta2 June 27, 2024 17:14
@fsundermeyer fsundermeyer marked this pull request as ready for review July 1, 2024 12:47
@fsundermeyer fsundermeyer merged commit 52ce55e into 4.0.beta2 Jul 1, 2024
2 checks passed
fsundermeyer added a commit that referenced this pull request Aug 5, 2024
* Allow to preserve or discard xml:base attributes
  (XSLT parameter preserve-xmlbase: 1 = preserve xml:base, 0 = discard them)
  This addition was needed to avoid wrong image paths as the xml:base
  attribute is taken into account to construct the path.

---------

Co-authored-by: Frank Sundermeyer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cat-bug topic-assembly Everything related to assembly processing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants