[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Bug 33277 depends on bug 33159, which changed state. Bug 33159 Summary: Thesaurus is not defined by second indicator for controlled fields outside of 6XX https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33159 What|Removed |Added Status|Pushed to stable|RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Lucas Gass changed: What|Removed |Added CC||lu...@bywatersolutions.com Resolution|--- |FIXED Status|Pushed to stable|RESOLVED --- Comment #73 from Lucas Gass --- Missing 22.05.x dependencies, no backport. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Jonathan Druart changed: What|Removed |Added Keywords|rel_23_05_candidate | -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #72 from Matt Blenkinsop --- Nice work everyone! Pushed to stable for 22.11.x -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Matt Blenkinsop changed: What|Removed |Added Status|Pushed to master|Pushed to stable Version(s)|23.05.00|23.05.00,22.11.06 released in|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #71 from Marcel de Rooy --- Opened bug 33718 for issues with tests when you have the prefs set like in the test plan here. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Marcel de Rooy changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=33718 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #70 from Tomás Cohen Arazi --- Pushed to master for 23.05. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Tomás Cohen Arazi changed: What|Removed |Added Version(s)||23.05.00 released in|| Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Tomás Cohen Arazi changed: What|Removed |Added Keywords||rel_23_05_candidate -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Marcel de Rooy changed: What|Removed |Added Attachment #150891|0 |1 is obsolete|| --- Comment #69 from Marcel de Rooy --- Created attachment 150983 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150983=edit Bug 33277: (follow-up) Don't use regex, tidy Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Marcel de Rooy changed: What|Removed |Added Attachment #149842|0 |1 is obsolete|| --- Comment #68 from Marcel de Rooy --- Created attachment 150982 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150982=edit Bug 33277: Adjust source file for zebra mappings The previous patches adjusted the mappings directly - moving this change to the correct build file Not needed for sign off, but QA can test that nothing changes when rebuilding the files: xsltproc etc/zebradb/xsl/koha-indexdefs-to-zebra.xsl etc/zebradb/marc_defs/marc21/authorities/authority-koha-indexdefs.xml > etc/zebradb/marc_defs/marc21/authorities/authority-zebra-indexdefs.xsl Signed-off-by: Frank Hansen Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Marcel de Rooy changed: What|Removed |Added Attachment #149841|0 |1 is obsolete|| --- Comment #67 from Marcel de Rooy --- Created attachment 150981 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150981=edit Bug 33277: (follow-up) Fix notdefined search for Zebra and ES Signed-off-by: Frank Hansen Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Marcel de Rooy changed: What|Removed |Added Attachment #149840|0 |1 is obsolete|| --- Comment #66 from Marcel de Rooy --- Created attachment 150980 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150980=edit Bug 33277: Search thesaurus for 'notspecified' Signed-off-by: Frank Hansen Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Marcel de Rooy changed: What|Removed |Added Attachment #149839|0 |1 is obsolete|| --- Comment #65 from Marcel de Rooy --- Created attachment 150979 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150979=edit Bug 33277: Add comments and missing thesauri Signed-off-by: Frank Hansen Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Marcel de Rooy changed: What|Removed |Added Attachment #149838|0 |1 is obsolete|| --- Comment #64 from Marcel de Rooy --- Created attachment 150978 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150978=edit Bug 33277: Support custom thesaurus authority with no 040 specified Signed-off-by: Frank Hansen Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Marcel de Rooy changed: What|Removed |Added Attachment #149837|0 |1 is obsolete|| --- Comment #63 from Marcel de Rooy --- Created attachment 150977 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150977=edit Bug 33277: Do not search using thesaurus when linking non 6XX fields This patch sets thesaurus as undefined when linking any field except 6XX This fixes the case where authrotiy records don't have the thesaurus defined Consequently - this means that Koha does not support multiple thesaurus records for authorities outside of subjects i.e. Using the default linker, and having both an LCSH and Sears record for 'Shakespeare,William' A 100 entry will find two results and the heading won't be linked. Previously we always linked to the LCSH To test: 1 - Import the attached auths and biblio (from bug 33159 comment 24) 2 - Set system preferences: RequireChoosingExistingAuthority - don't require AutoCreateAuthorities - don't generate CatalogModuleRelink - Do LinkerKeepStale - Don't LinkerModule - default LinkerRelink - do 3 - Edit the imported bib 4 - Save it 5 - Headings are not linked except 600 6 - Apply patch 7 - Restart all 8 - Edit and save record 9 - Headings are successfully linked Signed-off-by: Frank Hansen Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Marcel de Rooy changed: What|Removed |Added Patch complexity|--- |Small patch Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #62 from Marcel de Rooy --- Note: there was also a space in the bib side: Hamlet -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277
--- Comment #61 from Marcel de Rooy ---
Update: Most auth fields are linked now! The 600 Hamlet is not linked since it
looks for:
'value' => [
"Hamlet posta\x{107} fikcyjna",
'notdefined'
],
And the $c does not occur in the authority data. So thats explained too.
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #60 from Marcel de Rooy --- (In reply to Nick Clemens from comment #59) > (In reply to Marcel de Rooy from comment #55) > None of them? Or some of them? > None. > Restart zebra (restart_all) then reindex Will try that again. > Make sure AutoLinkBiblios is set to 'Do' Yes, I verified all prefs with the test plan. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277
--- Comment #59 from Nick Clemens ---
(In reply to Marcel de Rooy from comment #55)
> Tested this with Zebra. Copied
> cp etc/zebradb/marc_defs/marc21/authorities/authority-zebra-indexdefs.xsl
> /etc/koha/zebradb/marc_defs/marc21/authorities/authority-zebra-indexdefs.xsl
>
> Rebuilt index, restarted Zebra.
> Conform test plan, opened record and saved it again. Checked sysprefs. The
> links are not relinked to the correct authids.
None of them? Or some of them?
Restart zebra (restart_all) then reindex
Make sure AutoLinkBiblios is set to 'Do'(In reply to Marcel de Rooy from
comment #56)
> > ! grep
> > /$thesaurus/,('lcsh','lcac','mesh','nal','notspecified','cash','rvm','sears','aat')
> This regex might not always do what you want.
> Probably we could better do something like: none { $_ eq $thesaurus } etc
Updated
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #58 from Nick Clemens --- Created attachment 150891 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150891=edit Bug 33277: (follow-up) Don't use regex, tidy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Nick Clemens changed: What|Removed |Added Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Nick Clemens changed: What|Removed |Added Attachment #148957|0 |1 is obsolete|| --- Comment #57 from Nick Clemens --- Created attachment 150890 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150890=edit Updated authority records for testing -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Nick Clemens changed: What|Removed |Added Attachment #148734|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277
--- Comment #56 from Marcel de Rooy ---
> ! grep
> /$thesaurus/,('lcsh','lcac','mesh','nal','notspecified','cash','rvm','sears','aat')
This regex might not always do what you want.
Probably we could better do something like: none { $_ eq $thesaurus } etc
etc/zebradb/xsl/koha-indexdefs-to-zebra.xsl
notdefined
This could be done much simpler. Funny construction here. Out of scope.
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Marcel de Rooy changed: What|Removed |Added Status|BLOCKED |Failed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #55 from Marcel de Rooy --- Tested this with Zebra. Copied cp etc/zebradb/marc_defs/marc21/authorities/authority-zebra-indexdefs.xsl /etc/koha/zebradb/marc_defs/marc21/authorities/authority-zebra-indexdefs.xsl Rebuilt index, restarted Zebra. Conform test plan, opened record and saved it again. Checked sysprefs. The links are not relinked to the correct authids. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Marcel de Rooy changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |m.de.r...@rijksmuseum.nl |y.org | -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Marcel de Rooy changed: What|Removed |Added Status|Signed Off |BLOCKED --- Comment #54 from Marcel de Rooy --- Looking here now -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #53 from Nick Clemens --- *** Bug 32916 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Jonathan Druart changed: What|Removed |Added CC||jonathan.druart+koha@gmail. ||com --- Comment #52 from Jonathan Druart --- Janusz, do you think you could add your stamp to this (bug 33277) and bug 33557? If you do we can consider they Passed QA I think. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #51 from Nick Clemens --- (In reply to Caroline Cyr La Rose from comment #50) > I think this doesn't apply anymore. There was an old patch the needed obseleted, should apply now -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Nick Clemens changed: What|Removed |Added Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Nick Clemens changed: What|Removed |Added Attachment #148530|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Caroline Cyr La Rose changed: What|Removed |Added Status|Signed Off |Patch doesn't apply CC||caroline.cyr-la-rose@inlibr ||o.com --- Comment #50 from Caroline Cyr La Rose --- I think this doesn't apply anymore. Bug 33277 - Correctly handle linking subfields with no defined thesaurus 148530 - Bug 33277: Search thesaurus for 'notspecified' 149837 - Bug 33277: Do not search using thesaurus when linking non 6XX fields 149838 - Bug 33277: Support custom thesaurus authority with no 040 specified 149839 - Bug 33277: Add comments and missing thesauri 149840 - Bug 33277: Search thesaurus for 'notspecified' 149841 - Bug 33277: (follow-up) Fix notdefined search for Zebra and ES 149842 - Bug 33277: Adjust source file for zebra mappings Apply? [(y)es, (n)o, (i)nteractive] y Applying: Bug 33277: Search thesaurus for 'notspecified' error: sha1 information is lacking or useless (Koha/SearchEngine/Elasticsearch/QueryBuilder.pm). error: could not build fake ancestor Patch failed at 0001 Bug 33277: Search thesaurus for 'notspecified' hint: Use 'git am --show-current-patch=diff' to see the failed patch When you have resolved this problem run "git bz apply --continue". If you would prefer to skip this patch, instead run "git bz apply --skip". To restore the original branch and stop patching run "git bz apply --abort". Patch left in /tmp/Bug-33277-Search-thesaurus-for-notspecified-1Gg31H.patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Caroline Cyr La Rose changed: What|Removed |Added Depends on||30280 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30280 [Bug 30280] Support authority records with common subject headings from different thesaurus -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Nick Clemens changed: What|Removed |Added Severity|critical|blocker CC||m.de.r...@rijksmuseum.nl -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Frank Hansen changed: What|Removed |Added Attachment #148735|0 |1 is obsolete|| --- Comment #49 from Frank Hansen --- Created attachment 149842 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=149842=edit Bug 33277: Adjust source file for zebra mappings The previous patches adjusted the mappings directly - moving this change to the correct build file Not needed for sign off, but QA can test that nothing changes when rebuilding the files: xsltproc etc/zebradb/xsl/koha-indexdefs-to-zebra.xsl etc/zebradb/marc_defs/marc21/authorities/authority-koha-indexdefs.xml > etc/zebradb/marc_defs/marc21/authorities/authority-zebra-indexdefs.xsl Signed-off-by: Frank Hansen -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Frank Hansen changed: What|Removed |Added Attachment #148531|0 |1 is obsolete|| --- Comment #48 from Frank Hansen --- Created attachment 149841 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=149841=edit Bug 33277: (follow-up) Fix notdefined search for Zebra and ES Signed-off-by: Frank Hansen -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #47 from Frank Hansen --- Created attachment 149840 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=149840=edit Bug 33277: Search thesaurus for 'notspecified' Signed-off-by: Frank Hansen -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Frank Hansen changed: What|Removed |Added Attachment #148529|0 |1 is obsolete|| --- Comment #46 from Frank Hansen --- Created attachment 149839 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=149839=edit Bug 33277: Add comments and missing thesauri Signed-off-by: Frank Hansen -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Frank Hansen changed: What|Removed |Added Attachment #148528|0 |1 is obsolete|| --- Comment #45 from Frank Hansen --- Created attachment 149838 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=149838=edit Bug 33277: Support custom thesaurus authority with no 040 specified Signed-off-by: Frank Hansen -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Frank Hansen changed: What|Removed |Added Attachment #148527|0 |1 is obsolete|| --- Comment #44 from Frank Hansen --- Created attachment 149837 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=149837=edit Bug 33277: Do not search using thesaurus when linking non 6XX fields This patch sets thesaurus as undefined when linking any field except 6XX This fixes the case where authrotiy records don't have the thesaurus defined Consequently - this means that Koha does not support multiple thesaurus records for authorities outside of subjects i.e. Using the default linker, and having both an LCSH and Sears record for 'Shakespeare,William' A 100 entry will find two results and the heading won't be linked. Previously we always linked to the LCSH To test: 1 - Import the attached auths and biblio (from bug 33159 comment 24) 2 - Set system preferences: RequireChoosingExistingAuthority - don't require AutoCreateAuthorities - don't generate CatalogModuleRelink - Do LinkerKeepStale - Don't LinkerModule - default LinkerRelink - do 3 - Edit the imported bib 4 - Save it 5 - Headings are not linked except 600 6 - Apply patch 7 - Restart all 8 - Edit and save record 9 - Headings are successfully linked Signed-off-by: Frank Hansen Current status: Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #43 from Katrin Fischer --- Thank you Frank. Note for QA: Please add sign-off lines! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Frank Hansen changed: What|Removed |Added Status|Needs Signoff |Signed Off --- Comment #42 from Frank Hansen --- (In reply to Nick Clemens from comment #41) > (In reply to Frank Hansen from comment #40) > > (In reply to Katrin Fischer from comment #39) > > > Can someone help to get this tested? > > > > I have now tested the patch on my testing docker. I get different results > > depending on whether I choose the ES or the Zebra engine. I am following the > > test plan in comment #17 > > > > ES works flawlessly. All headings are linked after the patch is applied. > > > > But when doing the same using Zebra, without the patch installed only field > > 100 is linked. After applying the patch and restarted not all headings are > > linked. Only > > field 100, 655 (Literature English), 700 and 830 fields are linked after > > saving the record again. > > > > Have I missed something. > > The test plan is missing the steps for Zebra as the index files need to be > copied and then a full reindex: > sudo cp > /kohadevbox/koha/etc/zebradb/marc_defs/marc21/authorities/authority-zebra- > indexdefs.xsl > /etc/koha/zebradb/marc_defs/marc21/authorities/authority-zebra-indexdefs.xsl > restart_all > sudo koha-rebuild-zebra -v -f kohadev Yes! It works for me. I'm signing off. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Nick Clemens changed: What|Removed |Added Blocks||33557 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33557 [Bug 33557] Add a system preference to disable/enable thesaurus checking during authority linking -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #41 from Nick Clemens --- (In reply to Frank Hansen from comment #40) > (In reply to Katrin Fischer from comment #39) > > Can someone help to get this tested? > > I have now tested the patch on my testing docker. I get different results > depending on whether I choose the ES or the Zebra engine. I am following the > test plan in comment #17 > > ES works flawlessly. All headings are linked after the patch is applied. > > But when doing the same using Zebra, without the patch installed only field > 100 is linked. After applying the patch and restarted not all headings are > linked. Only > field 100, 655 (Literature English), 700 and 830 fields are linked after > saving the record again. > > Have I missed something. The test plan is missing the steps for Zebra as the index files need to be copied and then a full reindex: sudo cp /kohadevbox/koha/etc/zebradb/marc_defs/marc21/authorities/authority-zebra-indexdefs.xsl /etc/koha/zebradb/marc_defs/marc21/authorities/authority-zebra-indexdefs.xsl restart_all sudo koha-rebuild-zebra -v -f kohadev -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #40 from Frank Hansen --- (In reply to Katrin Fischer from comment #39) > Can someone help to get this tested? I have now tested the patch on my testing docker. I get different results depending on whether I choose the ES or the Zebra engine. I am following the test plan in comment #17 ES works flawlessly. All headings are linked after the patch is applied. But when doing the same using Zebra, without the patch installed only field 100 is linked. After applying the patch and restarted not all headings are linked. Only field 100, 655 (Literature English), 700 and 830 fields are linked after saving the record again. Have I missed something. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Nick Clemens changed: What|Removed |Added Severity|major |critical -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #39 from Katrin Fischer --- Can someone help to get this tested? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #38 from Nick Clemens --- (In reply to Phil Ringnalda from comment #36) > Finally got my hands on a working sandbox again, and with the updated test > authorities and the current patches, while the Literatura angielska‡2DBN > genre now links, the ind2=4 Literatura światowa one now does not link (with > either '|' or going back to 'n' in 008/11). This one actually cannot be tested on a sandbox - it requires copying the updated zebra files for indexing -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #37 from Nick Clemens --- Created attachment 148957 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148957=edit Updated authority records for testing Hamlet auth was marked as LOC, changed to 'z' -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #36 from Phil Ringnalda --- Finally got my hands on a working sandbox again, and with the updated test authorities and the current patches, while the Literatura angielska‡2DBN genre now links, the ind2=4 Literatura światowa one now does not link (with either '|' or going back to 'n' in 008/11). -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #35 from Nick Clemens --- Created attachment 148735 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148735=edit Bug 33277: Adjust source file for zebra mappings The previous patches adjusted the mappings directly - moving this change to the correct build file Not needed for sign off, but QA can test that nothing changes when rebuilding the files: xsltproc etc/zebradb/xsl/koha-indexdefs-to-zebra.xsl etc/zebradb/marc_defs/marc21/authorities/authority-koha-indexdefs.xml > etc/zebradb/marc_defs/marc21/authorities/authority-zebra-indexdefs.xsl -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #34 from Nick Clemens --- (In reply to Janusz Kaczmarek from comment #31) > (In reply to Nick Clemens from comment #26) > I understand your explanation about hardcoding Subject-heading-thesaurus > but, to be sincere, I do not share this argument. Firstly, in Zebra also we > have the definitions in the configuration file--if somebody changes there > something he/she shouldn't, its his/her fault. And secondly, which is IMHO > more important, shall we than, in the next step, hardcode also > Heading-use-main-or-added-entry for 008/14, Heading-use-series-added-entry > for 008/16, Heading-use-subject-added-entry for 008/15 when we will be > wanting to narrow the search for linking even more specifically (which > should definitely be done at some point)? So, I am still not convinced if > it is the right idea... I would suggest we hardcode the new fields if needed for linking. Linking is an internal process that shouldn't be adjusted by the user. Rather than allow them to break the system, let us save them :-) Zebra can technically be adjusted by the end user, however, ES allows direct updating via the interface. The intention of allowing the user access, IMHO, is to allow them to adjust searching for their needs. Adjusting internal processes isn't something we need to provide access to. The same MARC fields can be indexed into another field if the user needs them for searching. Authority searching doesn't use or provide these fields either, so allowing to be user adjusted only allows linking to break. Happy to continue that discussion, but I think we can address it wholesale on another bug. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #33 from Nick Clemens --- (In reply to Phil Ringnalda from comment #27) > I just followed the test plan in a sandbox, first with Zebra and then with > ElasticSearch, and in neither case was the Literatura angielska genre > heading linked. I had the wrong marker in the authority record, I believe I fixed it. The 008 thesaurus was set as 'n' - not applicable - rather then '|' - no attempt to code. All headings should link with the new file, or you can make that edit yourself -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Nick Clemens changed: What|Removed |Added Attachment #148418|0 |1 is obsolete|| --- Comment #32 from Nick Clemens --- Created attachment 148734 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148734=edit Updated authority records for testing Replaces an 'n' in the 008 for '|' -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #31 from Janusz Kaczmarek --- (In reply to Nick Clemens from comment #26) > (In reply to Janusz Kaczmarek from comment #25) > > Nick, I have looked into the corrected version. I see one problem: with > > Zebra, for: > > > > 655 7 $a Literatura angielska $2 DBN > > > > you search first for: "Literatura angielska + DBN" and, since there is no > > 040 $f, there is the second query for: "Literatura angielska + undefined' > > The second query passes "notdefined" - not undef [...] Nick, thank you very much for you comment. My eyes misses the difference between 'notdefined' for 'z' without 040 $f and 'notspecified' for '|'. I am sorry for unnecessarily bothering you. I understand your explanation about hardcoding Subject-heading-thesaurus but, to be sincere, I do not share this argument. Firstly, in Zebra also we have the definitions in the configuration file--if somebody changes there something he/she shouldn't, its his/her fault. And secondly, which is IMHO more important, shall we than, in the next step, hardcode also Heading-use-main-or-added-entry for 008/14, Heading-use-series-added-entry for 008/16, Heading-use-subject-added-entry for 008/15 when we will be wanting to narrow the search for linking even more specifically (which should definitely be done at some point)? So, I am still not convinced if it is the right idea... -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #30 from Phil Ringnalda --- On https://sandboxes.biblibre.eu/ if you want to poke at it, see where I might have gone wrong. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #29 from Phil Ringnalda --- I didn't for the Zebra case, since I applied the patches on sandbox creation so I didn't think about needing to. Just switched it back to Zebra, did a reindex, and it still didn't link. For ElasticSearch I did reindex before trying anything, since nothing works post-switch until you do. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #28 from Nick Clemens --- (In reply to Phil Ringnalda from comment #27) > I just followed the test plan in a sandbox, first with Zebra and then with > ElasticSearch, and in neither case was the Literatura angielska genre > heading linked. Did you reindex after applying the patches? While I tried to avoid it, it is required -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #27 from Phil Ringnalda --- I just followed the test plan in a sandbox, first with Zebra and then with ElasticSearch, and in neither case was the Literatura angielska genre heading linked. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #26 from Nick Clemens --- (In reply to Janusz Kaczmarek from comment #25) > Nick, I have looked into the corrected version. I see one problem: with > Zebra, for: > > 655 7 $a Literatura angielska $2 DBN > > you search first for: "Literatura angielska + DBN" and, since there is no > 040 $f, there is the second query for: "Literatura angielska + undefined' The second query passes "notdefined" - not undef > But having two auth records: > > 00116nz a2200049n 4500 > 008 230210 n anznnbabn a ana d > 155$a Literatura angielska This is stored as thesaurus "notdefined" > > 00116nz a2200049n 4500 > 008 230210 n an|nnbabn a ana d > 155$a Literatura angielska This is stored as thesaurus "notspecified" > > you will get both as response, while only one of them is appropriate (the > first one) [note 008/11]. Here is what the searches look like in Zebra 11:09:06-23/03 zebrasrv(1) [request] Search authorities OK 0 1 1+0 RPN @attrset Bib-1 @or @attr 1=Local-Number @attr 4=109 @attr 7=1 0 @and @and @attr 5=100 @attr 1=authtype GENRE/FORM @attr 6=3 @attr 5=100 @attr 4=1 @attr 1=Match-heading "Literatura angielska" @attr 5=100 @attr 4=1 @attr 1=Subject-heading-thesaurus DBN 11:09:06-23/03 zebrasrv(1) [request] Search authorities OK 1 1 1+0 RPN @attrset Bib-1 @or @attr 1=Local-Number @attr 4=109 @attr 7=1 0 @and @and @attr 5=100 @attr 1=authtype GENRE/FORM @attr 6=3 @attr 5=100 @attr 4=1 @attr 1=Match-heading "Literatura angielska" @attr 5=100 @attr 4=1 @attr 1=Subject-heading-thesaurus notdefined It should only return the second > > Why not to index, in Zebra, directly 008/11 and 040 $f into the same index, > without the 'index_subject_thesaurus' help-template which inserts > 'undefined' for lacking 040 $f? > Note that in ES in this case you are adding subject-heading-thesaurus == 'z' > to the query, not '|'. > In ES we are storing both the code and the 040f, so one with 040f will match on that - then when searching without the z will match > > Also, it is not clear for me why do you hardcoded subject-heading-thesaurus > in Koha/SearchEngine/Elasticsearch.pm, instead of putting it in mapping.yaml > as: The thesaurus search is used only internally for linking - this should not be customised or updated by the user. We do the same for 'Match-heading' to ensure consistent linking -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #25 from Janusz Kaczmarek --- Nick, I have looked into the corrected version. I see one problem: with Zebra, for: 655 7 $a Literatura angielska $2 DBN you search first for: "Literatura angielska + DBN" and, since there is no 040 $f, there is the second query for: "Literatura angielska + undefined' But having two auth records: 00116nz a2200049n 4500 008 230210 n anznnbabn a ana d 155$a Literatura angielska 00116nz a2200049n 4500 008 230210 n an|nnbabn a ana d 155$a Literatura angielska you will get both as response, while only one of them is appropriate (the first one) [note 008/11]. Why not to index, in Zebra, directly 008/11 and 040 $f into the same index, without the 'index_subject_thesaurus' help-template which inserts 'undefined' for lacking 040 $f? Note that in ES in this case you are adding subject-heading-thesaurus == 'z' to the query, not '|'. Also, it is not clear for me why do you hardcoded subject-heading-thesaurus in Koha/SearchEngine/Elasticsearch.pm, instead of putting it in mapping.yaml as: Subject-heading-thesaurus: label: Subject-heading-thesaurus mappings: - facet: '' marc_field: 008_/11 marc_type: marc21 sort: ~ suggestible: '' - facet: '' marc_field: 040f marc_type: marc21 sort: ~ suggestible: '' type: '' which seems to me more general and therefor natural... -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #24 from Frank Hansen --- (In reply to Nick Clemens from comment #23) > (In reply to Frank Hansen from comment #22) > > > I would still like to comment on ind2=4. As for us, we treat them as local > > subject headings only and they are not included in any thesaurus. Therefore, > > I would prefer not to see any authority records for either 008/11=| (or > > 008/11=n) in our catalog, even if the format allows it. > > Are you saying you won't import those records - or you want those records to > be ignored when linking? The former sounds easy enough, the second seems it > would require a syspref to not link those > Sorry if I was unclear. Records with ind2=4 is not exported from National Library catalogue at first. So they will not be imported to our local catalogue. We are not planning to create any either manually or automatically. > > I also want to comment on 040$f. According to the Swedish cataloguing > > guidelines here > ... > > So I question about allowing to omit 040$f when 008/11=z. > > This sounds like a comment on cataloging standards, and maybe something that > can be set in frameworks. But I don't think we need code for this? > Maybe not. We will use 040$f to specify system/thesaurus conventions in any case. I just want to make sure that it not have a negative impact on filtering on thesauruses if we omit 040$f. > > What causes the most trouble for me is, as mentioned earlier, that non-6XX > > fields are hardcoded to LCSH. This creates problems for libraries outside > > the US, as we typically do not use LCSH names. I wish we could skip > > thesaurus search for those fields. > > These patches do now skip searching thesaurus when linking non-subject fields I'm testing bug33277 and what I have seen so far looks good. (both ES and Zebra) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #23 from Nick Clemens --- (In reply to Frank Hansen from comment #22) > I would still like to comment on ind2=4. As for us, we treat them as local > subject headings only and they are not included in any thesaurus. Therefore, > I would prefer not to see any authority records for either 008/11=| (or > 008/11=n) in our catalog, even if the format allows it. Are you saying you won't import those records - or you want those records to be ignored when linking? The former sounds easy enough, the second seems it would require a syspref to not link those > I also want to comment on 040$f. According to the Swedish cataloguing > guidelines here ... > So I question about allowing to omit 040$f when 008/11=z. This sounds like a comment on cataloging standards, and maybe something that can be set in frameworks. But I don't think we need code for this? > What causes the most trouble for me is, as mentioned earlier, that non-6XX > fields are hardcoded to LCSH. This creates problems for libraries outside > the US, as we typically do not use LCSH names. I wish we could skip > thesaurus search for those fields. These patches do now skip searching thesaurus when linking non-subject fields -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #22 from Frank Hansen --- (In reply to Phil Ringnalda from comment #16) > I've never seen an authority record for a heading intended to be coded with > ind2=4, so I don't have a practical opinion. If someone ordered me to create > one, I would have used 008/11=z and an empty 040$f to say "this is an Other > thesaurus, but I cannot speak its name." But if Janusz has experience with > ones using 008/11=| I'm quite happy to defer to experience, and wait to see > if someone using z objects later. > > (Because I think horrible thoughts, a WhatSubjectThesaurusCodeIsUnknown > preference came to mind to solve disagreements, but anyone who feels we need > that really ought to either be forced to write it, or be forced to pay > someone handsomely for writing it.) I understand that different countries may have different interpretations of Library of Congress MARC specifications. And the Library of Congress allows free interpretation. I'm pretty sure that in Sweden we have our interpretation, which is also available here: https://katalogverk.kb.se/katalogisering/Formathandboken/index.html (only in Swedish). And Sweden are not to decide the format of course. And different countries may also have different needs. I would still like to comment on ind2=4. As for us, we treat them as local subject headings only and they are not included in any thesaurus. Therefore, I would prefer not to see any authority records for either 008/11=| (or 008/11=n) in our catalog, even if the format allows it. Here in Sweden, a kind of transition is made from field 653 for uncontrolled subject headings to using 650 with ind2=4 instead. That's another reason why I don't want to see any authority records with 008/11=| in our catalog. We are don't autocreate any authority records in our catalog. Almost all is imported from Netional Library (Libris). I also want to comment on 040$f. According to the Swedish cataloguing guidelines here https://katalogverk.kb.se/katalogisering/Formathandboken/kallorsystem/index.html (only in Swedish) 040$f must be filled in if 008/11=z. We in Sweden use many different thesauruses. In Library of Congress list here, https://www.loc.gov/standards/sourcelist/subject.html both the Swedish and Polish are listed. As I understand, 040 is a required field in Koha. However, 040$f is optional. Even though Library of Congress allows for the omission of 040$f, I can't see why it should be omitted, except for convenience. So I question about allowing to omit 040$f when 008/11=z. What causes the most trouble for me is, as mentioned earlier, that non-6XX fields are hardcoded to LCSH. This creates problems for libraries outside the US, as we typically do not use LCSH names. I wish we could skip thesaurus search for those fields. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #21 from Nick Clemens --- Created attachment 148531 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148531=edit Bug 33277: (follow-up) Fix notdefined search for Zebra and ES -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #20 from Nick Clemens --- Created attachment 148530 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148530=edit Bug 33277: Search thesaurus for 'notspecified' -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #19 from Nick Clemens --- Created attachment 148529 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148529=edit Bug 33277: Add comments and missing thesauri -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #18 from Nick Clemens --- Created attachment 148528 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148528=edit Bug 33277: Support custom thesaurus authority with no 040 specified -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Nick Clemens changed: What|Removed |Added Attachment #148492|0 |1 is obsolete|| Attachment #148493|0 |1 is obsolete|| Attachment #148494|0 |1 is obsolete|| Attachment #148495|0 |1 is obsolete|| Attachment #148500|0 |1 is obsolete|| --- Comment #17 from Nick Clemens --- Created attachment 148527 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148527=edit Bug 33277: Do not search using thesaurus when linking non 6XX fields This patch sets thesaurus as undefined when linking any field except 6XX This fixes the case where authrotiy records don't have the thesaurus defined Consequently - this means that Koha does not support multiple thesaurus records for authorities outside of subjects i.e. Using the default linker, and having both an LCSH and Sears record for 'Shakespeare,William' A 100 entry will find two results and the heading won't be linked. Previously we always linked to the LCSH To test: 1 - Import the attached auths and biblio (from bug 33159 comment 24) 2 - Set system preferences: RequireChoosingExistingAuthority - don't require AutoCreateAuthorities - don't generate CatalogModuleRelink - Do LinkerKeepStale - Don't LinkerModule - default LinkerRelink - do 3 - Edit the imported bib 4 - Save it 5 - Headings are not linked except 600 6 - Apply patch 7 - Restart all 8 - Edit and save record 9 - Headings are successfully linked -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #16 from Phil Ringnalda --- I've never seen an authority record for a heading intended to be coded with ind2=4, so I don't have a practical opinion. If someone ordered me to create one, I would have used 008/11=z and an empty 040$f to say "this is an Other thesaurus, but I cannot speak its name." But if Janusz has experience with ones using 008/11=| I'm quite happy to defer to experience, and wait to see if someone using z objects later. (Because I think horrible thoughts, a WhatSubjectThesaurusCodeIsUnknown preference came to mind to solve disagreements, but anyone who feels we need that really ought to either be forced to write it, or be forced to pay someone handsomely for writing it.) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #15 from Nick Clemens --- (In reply to Janusz Kaczmarek from comment #12) > I can confirm -- it does not work with Zebra for the case of 008/11 = 'z' > and no 040 $f. The second attempt is made, but without modification of > zebra auth index definition there will be 'notdefined' in the index instead > of 'z' (authority-zebra-indexdefs.xsl): I believe this is fixed, adjusted the mapping in ES to translate the value so that Zebra works directly - since we already translated most values in ES QueryBuilder and pass the word rather than code in other instances it seems to be more consistent > > With ES it seems OK. > > The only doubt that I still have is putting nothing in the query for 6XX 2nd > ind == '4'. As the consequence, if we had more than one auth records with > the same 1XX, we would get more than one from the query, which is bad. This was true before Koha knew about thesauri at all too - and we still have to deal with the separate bug 31925. I will defer to you and Phil on whether | or nothing is more appropriate, but will attach a patch using pipe so we can decide -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #14 from Nick Clemens --- Created attachment 148500 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148500=edit Bug 33277: (follow-up) Fix notdefined search for Zebra and ES -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #13 from Nick Clemens --- (In reply to Janusz Kaczmarek from comment #10) > (In reply to Nick Clemens from comment #8) > > Created attachment 148495 [details] [review] [review] > > Bug 33277: Don't search thesaurus when heading 'notspecified' > > Nick, in this case (i.e. 2nd ind == 4), according to my knowledge and > experience, if we want to be strict (and we want, I suppose) we should look > for an authority record with 008/11 = '|', and non simply 'not n' I misunderstood what you were saying on bug 33159 comment 33 - I thought you meant either pipe or non n The description here: https://www.loc.gov/marc/bibliographic/bd600.html Seems like it could be either z or | when looking here: https://www.loc.gov/marc/authority/ad008.html So I am happy to go either way - I will submit a patch to use | tomorrow -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #12 from Janusz Kaczmarek --- I can confirm -- it does not work with Zebra for the case of 008/11 = 'z' and no 040 $f. The second attempt is made, but without modification of zebra auth index definition there will be 'notdefined' in the index instead of 'z' (authority-zebra-indexdefs.xsl): notdefined With ES it seems OK. The only doubt that I still have is putting nothing in the query for 6XX 2nd ind == '4'. As the consequence, if we had more than one auth records with the same 1XX, we would get more than one from the query, which is bad. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #11 from Phil Ringnalda --- With the patches applied, the 655 _ 7 ‡aLiteratura angielska‡2DBN heading doesn't get linked. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #10 from Janusz Kaczmarek --- (In reply to Nick Clemens from comment #8) > Created attachment 148495 [details] [review] > Bug 33277: Don't search thesaurus when heading 'notspecified' Nick, in this case (i.e. 2nd ind == 4), according to my knowledge and experience, if we want to be strict (and we want, I suppose) we should look for an authority record with 008/11 = '|', and non simply 'not n' For 6XX we read (e.g. https://www.loc.gov/marc/bibliographic/bd650.html): 2nd ind == "4 - Source not specified Subject added entry conforms to a controlled list that cannot be identified by second indicator values 0-3, 5-6 or by a code in subfield $2." So it does not mean "I, the librarian, do not care, take anything you can find", It means rather "there is no appropriate thesaurus identification", so, I would interpret it as "not coded" == '|'. It is not straightforward to do a search for '|', but it is possible, both in Zebra and ES. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Phil Ringnalda changed: What|Removed |Added Blocks||33296 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33296 [Bug 33296] Linker should search for authority records with an appropriate 008/14,15,16 value -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #9 from Nick Clemens --- (In reply to Phil Ringnalda from comment #4) > Not sure how many other cases it would enable, but the bug 33159 comment 28 > thing of searching for Heading-use-main-or-added-entry = 'a' while linking > 1xx/7xx would work for LCSH plus FAST, since the FAST name authority records > correctly have 008/14 set to b. Please file as a new bug :-) dependent on this one -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Nick Clemens changed: What|Removed |Added Blocks||33295 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33295 [Bug 33295] We need a way to exclude 'inappropriate' heading from linking searches -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #8 from Nick Clemens --- Created attachment 148495 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148495=edit Bug 33277: Don't search thesaurus when heading 'notspecified' -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #7 from Nick Clemens --- Created attachment 148494 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148494=edit Bug 33277: Add comments and missing thesauri -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #6 from Nick Clemens --- Created attachment 148493 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148493=edit Bug 33277: Support custom thesaurus authority with no 040 specified -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Nick Clemens changed: What|Removed |Added Attachment #148417|0 |1 is obsolete|| --- Comment #5 from Nick Clemens --- Created attachment 148492 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148492=edit Bug 33277: Do not search using thesaurus when linking non 6XX fields This patch sets thesaurus as undefined when linking any field except 6XX This fixes the case where authrotiy records don't have the thesaurus defined Consequently - this means that Koha does not support multiple thesaurus records for authorities outside of subjects i.e. Using the default linker, and having both an LCSH and Sears record for 'Shakespeare,William' A 100 entry will find two results and the heading won't be linked. Previously we always linked to the LCSH To test: 1 - Import the attached auths and biblio (from bug 33159 comment 24) 2 - Set system preferences: RequireChoosingExistingAuthority - don't require AutoCreateAuthorities - don't generate CatalogModuleRelink - Do LinkerKeepStale - Don't LinkerModule - default LinkerRelink - do 3 - Edit the imported bib 4 - Save it 5 - Headings are not linked except 600 6 - Apply patch 7 - Restart all 8 - Edit and save record 9 - Headings are successfully linked -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Tomás Cohen Arazi changed: What|Removed |Added CC||tomasco...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Frank Hansen changed: What|Removed |Added CC||frank.han...@ub.lu.se -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Phil Ringnalda changed: What|Removed |Added CC||p...@chetcolibrary.org --- Comment #4 from Phil Ringnalda --- Not sure how many other cases it would enable, but the bug 33159 comment 28 thing of searching for Heading-use-main-or-added-entry = 'a' while linking 1xx/7xx would work for LCSH plus FAST, since the FAST name authority records correctly have 008/14 set to b. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Nick Clemens changed: What|Removed |Added CC||janus...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #3 from Nick Clemens --- Created attachment 148419 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148419=edit Bib record for testing -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #2 from Nick Clemens --- Created attachment 148418 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148418=edit Authority records for testing -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Nick Clemens changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |n...@bywatersolutions.com |ity.org | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 --- Comment #1 from Nick Clemens --- Created attachment 148417 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148417=edit Bug 33277: Do not search using thesaurus when linking non 6XX fields This patch sets thesaurus as undefined when linking any field except 6XX This fixes the case where authrotiy records don't have the thesaurus defined Consequently - this means that Koha does not support multiple thesaurus records for authorities outside of subjects i.e. Using the default linker, and having both an LCSH and Sears record for 'Shakespeare,William' A 100 entry will find two results and the heading won't be linked. Previously we always linked to the LCSH To test: 1 - Import the attached auths and biblio (from bug 33159 comment 24) 2 - Set system preferences: RequireChoosingExistingAuthority - don't require AutoCreateAuthorities - don't generate CatalogModuleRelink - Do LinkerKeepStale - Don't LinkerModule - default LinkerRelink - do 3 - Edit the imported bib 4 - Save it 5 - Headings are not linked except 600 6 - Apply patch 7 - Restart all 8 - Edit and save record 9 - Headings are successfully linked -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Nick Clemens changed: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 33277] Correctly handle linking subfields with no defined thesaurus
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33277 Nick Clemens changed: What|Removed |Added Depends on||33159 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33159 [Bug 33159] Thesaurus is not defined by second indicator for controlled fields outside of 6XX -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
