Bug 29733 - MARC21: Link 7xx linking fields to marc21_linking_section.pl value builder in sample frameworks
Summary: MARC21: Link 7xx linking fields to marc21_linking_section.pl value builder in...
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: MARC Bibliographic data support (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Janusz Kaczmarek
QA Contact: Testopia
URL:
Keywords: Academy
: 30977 (view as bug list)
Depends on:
Blocks:
 
Reported: 2021-12-19 13:50 UTC by Katrin Fischer
Modified: 2025-04-03 22:01 UTC (History)
2 users (show)

See Also:
GIT URL:
Change sponsored?: ---
Patch complexity: Small patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 29733: MARC21: Link 7xx linking fields to marc21_linking_section.pl value builder (13.74 KB, patch)
2025-04-03 21:04 UTC, Janusz Kaczmarek
Details | Diff | Splinter Review
Bug 29733: MARC21: Link 7xx linking fields to marc21_linking_section.pl value builder (13.79 KB, patch)
2025-04-03 21:45 UTC, Roman Dolny
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Katrin Fischer 2021-12-19 13:50:17 UTC
We do link the authority controlled fields to the authority linker by default, so it would make sense to link the fields used for linking between bibliographic records to the available helper plugin as well. 

At the moment the plugin is not well know, although it provides a great functionality, this could help advertise and improve it.
Comment 1 Katrin Fischer 2022-06-16 12:15:04 UTC
*** Bug 30977 has been marked as a duplicate of this bug. ***
Comment 2 Katrin Fischer 2022-06-16 12:15:35 UTC
Looks like late 2021 me already had the same idea :)
Comment 3 Janusz Kaczmarek 2025-04-03 21:04:25 UTC
Created attachment 180605 [details] [review]
Bug 29733: MARC21: Link 7xx linking fields to marc21_linking_section.pl value builder

When creating an analytical record from scratch and filling the 773
field with marc21_linking_section.pl plugin, the information put
in field 773 is very basic, incomplete, sometimes buggy ($9, $0).

Test plan
=========
1. Configure a MARC bibliographic framework: enable essential
   subfields in 773: 7, a, t, b, d, g, k, x, z, w (check the Editor
   checkbox), put them in this order, and chose plugin
   marc21_linking_section.pl for 773 $t.
2. Start creating a new record and go to 773 $t. Activate the plugin
   to find the a record to become the host record (perform a search
   for 'Linux kernel development'). Note the content of the 773
   field.
3. Apply the patch, restart plack etc.
4. Repeat p. 2.-3. See that the generated 773 field contains now much
   more information, including 773 $7.

Additionally, commented code has been removed and check/variable
initialization added to get rid of warnings in log.
Comment 4 Roman Dolny 2025-04-03 21:45:33 UTC
Created attachment 180607 [details] [review]
Bug 29733: MARC21: Link 7xx linking fields to marc21_linking_section.pl value builder

When creating an analytical record from scratch and filling the 773
field with marc21_linking_section.pl plugin, the information put
in field 773 is very basic, incomplete, sometimes buggy ($9, $0).

Test plan
=========
1. Configure a MARC bibliographic framework: enable essential
   subfields in 773: 7, a, t, b, d, g, k, x, z, w (check the Editor
   checkbox), put them in this order, and chose plugin
   marc21_linking_section.pl for 773 $t.
2. Start creating a new record and go to 773 $t. Activate the plugin
   to find the a record to become the host record (perform a search
   for 'Linux kernel development'). Note the content of the 773
   field.
3. Apply the patch, restart plack etc.
4. Repeat p. 2.-3. See that the generated 773 field contains now much
   more information, including 773 $7.

Additionally, commented code has been removed and check/variable
initialization added to get rid of warnings in log.

Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Comment 5 Roman Dolny 2025-04-03 22:01:56 UTC
It works as described in test plan.
Maybe one thing to improve (similarly like in Bug 39545 ?) -- adding multiple ISBN to subfield $z. Now only last one is included.