[Bug 32509] New: Embed see-also-from headings into bibliographic records export
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 Bug ID: 32509 Summary: Embed see-also-from headings into bibliographic records export Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Command-line Utilities Assignee: koha-bugs@lists.koha-community.org Reporter: fridolin.somers@biblibre.com QA Contact: testopia@bugs.koha-community.org CC: robin@catalyst.net.nz Like Bug 31286 for see-also-forms 5xx -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |fridolin.somers@biblibre.co |ity.org |m Depends on| |31286 Status|NEW |ASSIGNED --- Comment #1 from Fridolin Somers <fridolin.somers@biblibre.com> --- I set depending on Bug 31286 for easier rebases. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31286 [Bug 31286] Embed see-from headings into bibliographic records export -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 --- Comment #2 from Fridolin Somers <fridolin.somers@biblibre.com> --- Created attachment 144748 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=144748&action=edit Bug 32509: Embed see-also-from headings into bibliographic records export -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 --- Comment #3 from Fridolin Somers <fridolin.somers@biblibre.com> --- This is just the code. If someone is interested, I will provide test plan and UT. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #144748|0 |1 is obsolete| | --- Comment #4 from Fridolin Somers <fridolin.somers@biblibre.com> --- Created attachment 144751 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=144751&action=edit Bug 32509: Embed see-also-from headings into bibliographic records export -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 --- Comment #5 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Should this be 'needs signoff'? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 --- Comment #6 from Fridolin Somers <fridolin.somers@biblibre.com> --- Needs test plan and unit test -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 --- Comment #7 from Fridolin Somers <fridolin.somers@biblibre.com> --- Maybe should rename Koha::RecordProcessor "EmbedSeeFromHeadings" For example "EmbedOtherHeadings" -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=34481 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 Bug 32509 depends on bug 31286, which changed state. Bug 31286 Summary: Embed see-from headings into bibliographic records export https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31286 What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to stable |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Patch doesn't apply Depends on| |34481 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34481 [Bug 34481] Add IncludeSeeAlsoFromInSearches like IncludeSeeFromInSearches -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 --- Comment #8 from Fridolin Somers <fridolin.somers@biblibre.com> --- Ohhh Bug 34481 is in -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 --- Comment #9 from Fridolin Somers <fridolin.somers@biblibre.com> --- Created attachment 190692 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190692&action=edit Bug 32509: Embed see-also-from headings into bibliographic records export Patch from commit c2cd6a9 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #144751|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #190692|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 --- Comment #10 from Fridolin Somers <fridolin.somers@biblibre.com> --- Created attachment 190696 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190696&action=edit Bug 32509: Embed see-also-from headings into bibliographic records export In misc/export_records.pl add an option to add see-also-from headings (from authorities 5xx) into bibliographic records export. Like see-from. Test plan : 1) Choose a biblio record having a field (for example 650) linked to an authority with a see-also-from. 2) Export this record without see-also-from headings : misc/export_records.pl --starting_biblionumber=X --ending_biblionumber=X --filename /tmp/record_without.xml --format xml 3) Export this record with see-also-from headings : misc/export_records.pl --starting_biblionumber=X --ending_biblionumber=X --filename /tmp/record_with.xml --format xml --embed_see_also_from_headings 4) Compare /tmp/record_without.xml and /tmp/record_with.xml => you should see two 650, one with main heading and one with see-also-from heading -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 --- Comment #11 from Fridolin Somers <fridolin.somers@biblibre.com> --- Sent from https://gitlab.com/koha-community/perl-git-bz ;) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #190696|0 |1 is obsolete| | --- Comment #12 from David Nind <david@davidnind.com> --- Created attachment 190743 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=190743&action=edit Bug 32509: Embed see-also-from headings into bibliographic records export In misc/export_records.pl add an option to add see-also-from headings (from authorities 5xx) into bibliographic records export. Like see-from. Test plan : 1) Choose a biblio record having a field (for example 650) linked to an authority with a see-also-from. 2) Export this record without see-also-from headings : misc/export_records.pl --starting_biblionumber=X --ending_biblionumber=X --filename /tmp/record_without.xml --format xml 3) Export this record with see-also-from headings : misc/export_records.pl --starting_biblionumber=X --ending_biblionumber=X --filename /tmp/record_with.xml --format xml --embed_see_also_from_headings 4) Compare /tmp/record_without.xml and /tmp/record_with.xml => you should see two 650, one with main heading and one with see-also-from heading Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com Text to go in the| |Adds a new option, release notes| |`--embed_see_also_from_head | |ings`, to the | |`misc/export_records.pl` | |script. | | | |This will include any | |authority record | |see-also-from headings in | |the export for any | |bibliographic record 6xx | |fields, where the authority | |record has a 5xx. | | | |Example: | |1. A bibliographic record | |has a 650 - SUBJECT ADDED | |ENTRY--TOPICAL TERM: | | . 650$a = Perl (Computer | |program language) | | . 650$9 = 19 | |2. The authority record | |(Perl (Computer program | |language)) has a 550 - SEE | |ALSO FROM TRACING--TOPICAL | |TERM: | | . 550$a = Scripting | |languages (Computer | |science) | | . 550$w = g | |3. With the | |`--embed_see_also_from_head | |ings` option added to | |`misc/export_records.pl` | |you get the authority | |record's 550 information in | |the export: | | | |``` | |<datafield tag="650" | |ind1="z" ind2=" "> | | <subfield | |code="a">Scripting | |languages (Computer | |science)</subfield> | | <subfield | |code="w">g</subfield> | |</datafield> | | | |``` --- Comment #13 from David Nind <david@davidnind.com> --- Testing notes (using KTD): 1. Programming Perl (biblionumber = 262) has a 650 (Perl (Computer program language)) where the authority record (authid = 19) has a 550 (Scripting languages (Computer science)). 2. Attempted a release note with an example. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32509 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lucas@bywatersolutions.com Status|Signed Off |Failed QA --- Comment #14 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Can this get a new or updated unit test to account for the change in Exporter/Record.pm? -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org