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

Add how to ignore namespace in MARC XML Handler #553

Merged
merged 2 commits into from
Jul 16, 2024

Conversation

dr0i
Copy link
Member

@dr0i dr0i commented Jul 15, 2024

Fixes #552.

The @description describes how to set the namespace to null as empty using the flux. The method description describes how to even set it to null thus ignoring the namespace at all using the JAVA-API.

@dr0i dr0i requested a review from fsteeg July 15, 2024 14:58
@dr0i dr0i force-pushed the 552-addHowToIgnoreNamespacesInMarcXmlHandler branch from 2ad7577 to a0cd80c Compare July 15, 2024 15:00
@dr0i dr0i force-pushed the 552-addHowToIgnoreNamespacesInMarcXmlHandler branch from a0cd80c to 5616ae8 Compare July 16, 2024 09:22
The MARC21 namespace is checked by default.
Sometimes there is MARC21 data without the proper namespace or none namespace
at all. To make the handler run in such cases the namespace must be explicitly
set as parameter to match the namespace used in the data.

The @description describes how to set an empty namespace if the namespace is
missing. Ignoring the namepace at all can be done only via JAVA-API by setting
it to null.
@dr0i dr0i force-pushed the 552-addHowToIgnoreNamespacesInMarcXmlHandler branch from 5616ae8 to f81abb6 Compare July 16, 2024 09:30
Copy link
Member

@fsteeg fsteeg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, with one minor suggestion.

@dr0i dr0i merged commit 29722a3 into master Jul 16, 2024
1 check passed
@dr0i dr0i deleted the 552-addHowToIgnoreNamespacesInMarcXmlHandler branch July 16, 2024 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Add how to read MARC XML without namespace definition
2 participants