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

Write valid leader without counting #527 #547

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

TobiasNx
Copy link
Contributor

@TobiasNx TobiasNx commented Jun 26, 2024

Instead of just appending the incoming leader elements by encode-marcmxl, this adds the static values for Pos: 00-04, 10-16, 20-23.

By using 0as default value for Pos: 00-04 and 12-16.

#527 (comment)

Instead of just appending the incoming leader elements, this adds the static values for Pos: 00-04, 10-16, 20-23.

By using `0`as default value for Pos: 00-04 and 12-16.

#527 (comment)
@TobiasNx TobiasNx force-pushed the 527-output-valid-leader branch from ea05347 to a0f2736 Compare June 26, 2024 07:57
@blackwinter
Copy link
Member

What's the point of doing the "easy" solution now that we already have the "hard" solution?

@TobiasNx
Copy link
Contributor Author

The leader is always valid?

Copy link
Member

@blackwinter blackwinter left a comment

Choose a reason for hiding this comment

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

I guess I'm not too excited about this, but in the end I don't really care.

It should be tied to ensureCorrectMarc21Xml="false", though.

Checkstyle violations should be fixed, tests should be added/fixed.

@blackwinter blackwinter assigned TobiasNx and unassigned blackwinter Jun 26, 2024
@TobiasNx
Copy link
Contributor Author

I will put this on hold, the failing test hints that it is not as easy: issue336_createRecordWithTopLevelLeader_defaultMarc21Xml

dr0i and others added 4 commits November 19, 2024 09:12
If the incoming leader is 8 Metafacture is now able to create correct leaders for marc xml with static default values.
@TobiasNx
Copy link
Contributor Author

It should be tied to ensureCorrectMarc21Xml="false", though.

@blackwinter how would I tie my adjustments to ensureCorrectMarc21Xml="false",

@TobiasNx TobiasNx assigned blackwinter and unassigned TobiasNx Nov 19, 2024
@blackwinter
Copy link
Member

What's the actual use case here? When would this change apply in practice?

Because the implementation gets complicated by the fact that the setting for ensureCorrectMarc21Xml needs to be propagated down into the encoder instance.

@blackwinter blackwinter assigned TobiasNx and unassigned blackwinter Nov 19, 2024
@TobiasNx
Copy link
Contributor Author

Always create valid marc xml records from marc binary besides the option to create correct calculate records.
When helping Verbundgruppe with Metafacture at least in my opinion it becomes more handy to output always a valid leader.

@blackwinter
Copy link
Member

So binary MARC always has a leader of length 8, is that what you're saying? Or only in their case?

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.

3 participants