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

Script to check for erroneous links #442

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

hakonhagland
Copy link
Collaborator

@hakonhagland hakonhagland commented Dec 19, 2024

Builds on #441 which should be merged first.

This script checks for erroneously inserted links to keywords in the office:binary-data sections in the keyword files. These links might have been inserted by the keyword linking script, see #435 (comment) for more information.

I ran this script on chapters 4-11 and no erroneous links where detected there (according to this script). I will check chapter 12 next.

Fix link keywords script such that it does not add links inside
office:binary sections. These sections contain image data encoded
as base64, and should not be considered as normal text.
To avoid adding links from Figure captions, we check if we are inside a
draw:frame element
Script to check for erroneous links in the office:binary-data sections
@hakonhagland hakonhagland changed the title Check link bin Script to check for erroneous links Dec 19, 2024
@gdfldm
Copy link
Collaborator

gdfldm commented Dec 19, 2024

no erroneous links where detected

Thanks. I have also had a quick scan through chapters 4.3-11.3 and found very few images and didn't notice any obvious issues.

@hakonhagland
Copy link
Collaborator Author

I have also had a quick scan through chapters 4.3-11.3 and found very few images and didn't notice any obvious issues.

@gdfldm Great :) I ran the script now on Chapter 12.3, and it found two files there: GRUPTREE.fodt and WSEGAICD.fodt. See the two latest commits in #435

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.

2 participants