Bug 32509 - Embed see-also-from headings into bibliographic records export
Summary: Embed see-also-from headings into bibliographic records export
Status: ASSIGNED
Alias: None
Product: Koha
Classification: Unclassified
Component: Command-line Utilities (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Fridolin Somers
QA Contact: Testopia
URL:
Keywords:
Depends on: 31286
Blocks:
  Show dependency tree / graph
 
Reported: 2022-12-20 19:05 UTC by Fridolin Somers
Modified: 2023-08-04 19:33 UTC (History)
1 user (show)

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

Attachments
Bug 32509: Embed see-also-from headings into bibliographic records export (4.11 KB, patch)
2022-12-20 22:12 UTC, Fridolin Somers 		Details | Diff | Splinter Review
Bug 32509: Embed see-also-from headings into bibliographic records export (4.12 KB, patch)
2022-12-20 22:44 UTC, Fridolin Somers 		Details | Diff | Splinter Review
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Fridolin Somers 2022-12-20 19:05:31 UTC 
Like Bug 31286 for see-also-forms 5xx
Comment 1 Fridolin Somers 2022-12-20 19:06:13 UTC 
I set depending on Bug 31286 for easier rebases.
Comment 2 Fridolin Somers 2022-12-20 22:12:02 UTC 
Created attachment 144748 [details] [review]
Bug 32509: Embed see-also-from headings into bibliographic records export
Comment 3 Fridolin Somers 2022-12-20 22:12:54 UTC 
This is just the code.

If someone is interested, I will provide test plan and UT.
Comment 4 Fridolin Somers 2022-12-20 22:44:42 UTC 
Created attachment 144751 [details] [review]
Bug 32509: Embed see-also-from headings into bibliographic records export
Comment 5 Katrin Fischer 2023-06-10 21:40:00 UTC 
Should this be 'needs signoff'?
Comment 6 Fridolin Somers 2023-08-02 22:03:23 UTC 
Needs test plan and unit test
Comment 7 Fridolin Somers 2023-08-04 19:24:08 UTC 
Maybe should rename Koha::RecordProcessor "EmbedSeeFromHeadings"
For example "EmbedOtherHeadings"