[Bug 8071] New: link beween bib and authorities with the authid
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8071 Priority: P5 - low Change sponsored?: --- Bug ID: 8071 Assignee: gmcharlt@gmail.com Summary: link beween bib and authorities with the authid Severity: blocker Classification: Unclassified OS: All Reporter: stephane.delaye@biblibre.com Hardware: All Status: NEW Version: rel_3_8 Component: Cataloging Product: Koha I have in a database authorities (marcxml.auth_header) $9<space>0<space> in the tag 2XX So when I want link a bib record with a auth record, koha create the link with the $9<space>0<space> in the tag 2XX and not with the authid from auth_header. Result in the thesaurus all the link are bad -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8071 delaye <stephane.delaye@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P5 - low |P1 - high -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8071 laurenthdl <henridamien.laurent@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Assignee|gmcharlt@gmail.com |henridamien.laurent@biblibr | |e.com --- Comment #1 from laurenthdl <henridamien.laurent@biblibre.com> --- Created attachment 9513 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=9513&action=edit patch proposed This is a quick fix. and fixes the erroneous data. What could be done, though it would add some access to database would be to check the presence of the authority before sending the authid to detail pages. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8071 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de --- Comment #2 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi, this is marked as blocker, but I am not sure how to test it. Can you perhaps provide a test plan? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8071 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #9513|0 |1 is obsolete| | --- Comment #3 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 11134 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11134&action=edit Proposed patch (2) I propose a patch with in addition to first patch the behavior : when a thesaurus is applied to a field, first set fields values from authority and then set $9 field with authid. (see js in blinddetail-biblio-search.tt). -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8071 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fridolyn.somers@biblibre.co | |m --- Comment #4 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Test plan : ----------- Test thesaurus in cataloging : Edit an authority type to allow $9 edition in the field that will be copied into biblio. Create a new authority of this type using and set '0' in the subfield $9 defined above. Save the authority. Wait for incremental indexing (or perform rebuild_zebra.pl -a -z). Create a new record. Edit a field using as thesaurus the above authority type. Click on thesaurus link (usually at $a). Search for the authority created above. Choose it. Without patch => $9 in biblio field will have '0'. With patch => $9 in biblio field will have the authid number. + Test authority with $9='0' in MARC view. + Test authorities merge. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8071 Fridolyn SOMERS <fridolyn.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #11134|0 |1 is obsolete| | Assignee|henridamien.laurent@biblibr |fridolyn.somers@biblibre.co |e.com |m --- Comment #5 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Created attachment 11157 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11157&action=edit Proposed patch (3) Patch revision : After a discussion, we agreed that the special behavior for $9=' 0 ' is too specific to be pushed into community version. Still the presence of $9 in authority should not break the thesaurus in cataloging nor the merge, that is what is still in the patch. I added some comments to specify why $9 is special. In Biblio.pm : I corrected GetMarcSeries, it was using MARC21 fields even in a UNIMARC config. In GetMarcSubjects, GetMarcAuthors and GetMarcSeries, I modified them so that the $9 subfield is selected before looping on other subfields. Without that, the @link_loop was correctly set only if $9 is first subfield. Also, I tried to unify there code, like using $record->field('X..') to avoid looping on all fields. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8071 --- Comment #6 from Fridolyn SOMERS <fridolyn.somers@biblibre.com> --- Remove "Test authority with $9='0' in MARC view" form test plan. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8071 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8071 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #11157|0 |1 is obsolete| | --- Comment #7 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- Created attachment 11554 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11554&action=edit Bug 8071: link between bib and authorities with the authid Do not automatically populate $9 in bibliographic headings when the $9 is set in the authorized heading field of the authority record. Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8071 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |mtj@kohaaloha.com Version|rel_3_8 |master QA Contact| |mtj@kohaaloha.com --- Comment #8 from Mason James <mtj@kohaaloha.com> --- patch looks good, applied to MASTER, passing QA... $ koha-qa.pl testing 1 commit(s) (applied to commit 60790d0) * 955eee1 Bug 8071: link between bib and authorities with the authid C4/AuthoritiesMarc.pm C4/Biblio.pm authorities/blinddetail-biblio-search.pl koha-tmpl/intranet-tmpl/prog/en/modules/authorities/blinddetail-biblio-search.tt * C4/AuthoritiesMarc.pm OK * C4/Biblio.pm OK * authorities/blinddetail-biblio-search.pl OK * koha-tmpl/intranet-tmpl/prog/en/modules/authorities/blinddetail-biblio-seOK -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8071 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |paul.poulain@biblibre.com Version|master |rel_3_10 --- Comment #9 from Paul Poulain <paul.poulain@biblibre.com> --- Patch pushed to master -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8071 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable CC| |chris@bigballofwax.co.nz Version|rel_3_10 |rel_3_8 --- Comment #10 from Chris Cormack <chris@bigballofwax.co.nz> --- Pushed to 3.8.x will be in 3.8.5. Template change but no string changes. But being a blocker I would push anyway. (Blockers trump freezes at least in my book) -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org