Problem happens in Koha 3.8 but maybe it happens also in 3.10. In UNIMARC installations the authors have a double comma separating the subfield a and the subfield b. The problem is the following: - UNIMARC authors are 7XX fields - In UNIMARC 7XXa is always filled with the surname and a comma, like "SURNAME," - The facets are build with 700ab, 701ab and 702ab - The facets use a comma to separate the subfield a and subfield b, so there a double comma: the separator and the comma from subfield a Solution: Change the method getFacets of Koha.pm - separator for UNIMARC authors will be a space instead of a comma.
Created attachment 14389 [details] [review] Bug 9341: Problem with UNIMARC authors facets In UNIMARC installations the authors have a double comma separating the subfield a and the subfield b. Test plan: - After applying the patch do a search and check the double comma in authors facets. - Apply the patch - Do a new search - Check if authors facets as a double comma Sponsored-by: KEEP SOLUTIONS
Hello In France we don't add any ponctuation in 7XX$a, so I was surprised. But, you are right, except it is only a recommandation, and not a part of the format itself. The UNIMARC manual (2008) says : "There are no standards for punctuation in this field. However, it is recommended that punctuation be retained where it is available in the source format. In source formats where punctuation is not available, but is printed out by an algorithm generated from the definition of the subfields, it is recommended that the punctuation which would be used in display be included in the UNIMARC record. Because there are no standards for punctuation in this field, recipients of records in the UNIMARC format will have to be aware of the practices adopted by the agency preparing the record; therefore agencies distributing records should attempt to be consistent in their own records." But in French version of the manual, http://www.bnf.fr/documents/B700-6-2010.pdf we read : "NdT : les exemples provenant de catalogues anglo-saxons contiennent la ponctuation prescrite, qui a été saisie et non pas générée par programme comme le veut l’usage français." (Translator note : examples from anglo-saxon catalogs contains punctuation, included in the record, and not generated by an algorithm as wants french use) So maybe it should be a syspref ? M. Saby
Hi Mathieu, You're right. I've checked out this with some librarians and there isn't a rule for the filling of UNIMARC authors. So this should be a preference. I will do the change today.
Created attachment 14413 [details] [review] Bug 9341: Problem with UNIMARC authors facets In UNIMARC installations the authors have a double comma separating the subfield a and the subfield b. Test plan: - After applying the patch do a search and check the double comma in authors facets. - Apply the patch - Do a new search - Check if authors facets as a double comma Sponsored-by: KEEP SOLUTIONS
Hi Vitor, your patch does not apply: Applying: Bug 9341: Problem with UNIMARC authors facets fatal: sha1 information is lacking or useless (C4/Koha.pm). Repository lacks necessary blobs to fall back on 3-way merge. Cannot fall back to three-way merge. Could your reformat and resubmit it please?
Hi Jonathan, I don't see any problem with the patch. What could be wrong? C4/Koha.pm only has one simple change that worked in my installation.
Jonathan, Can you try now? I was because I put the first patch obsolete.
QA Comment: I agree with the syspref but I think the default value should be the comma in order not to change the current behavior. Why do you create a specific syspref for UNIMARC? We could create a syspref named AuthorsFacetsSeparator and use it for other formats too. Marked as Failed QA.
Jonathan, The default value is ', ', like it was now. Check the MySQL insert in the updatedatabase.pl. The problem only happens in UNIMARC, and that's in MARC21 the authors are only listed in 100a, 110a and 700a. So the separator is never used.
(In reply to comment #9) > Jonathan, > > The default value is ', ', like it was now. Yes, you are right. > Check the MySQL insert in the updatedatabase.pl. > The problem only happens in UNIMARC, and that's in MARC21 the authors are > only listed in 100a, 110a and 700a. So the separator is never used. Yep, ok
Created attachment 14448 [details] [review] Bug 9341: Problem with UNIMARC authors facets In UNIMARC installations the authors have a double comma separating the subfield a and the subfield b. Test plan: - After applying the patch do a search and check the double comma in authors facets. - Apply the patch - Do a new search - Check if authors facets as a double comma Sponsored-by: KEEP SOLUTIONS Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Created attachment 14449 [details] [review] Bug 9341: Problem with UNIMARC authors facets In UNIMARC installations the authors have a double comma separating the subfield a and the subfield b. Test plan: - After applying the patch do a search and check the double comma in authors facets. - Apply the patch - Do a new search - Check if authors facets as a double comma Sponsored-by: KEEP SOLUTIONS Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Hi Vitor, I am sorry, but I can't get the second patch to apply, with or without the first patch: Apply? [y]es/[n]o/[e]dit/[v]iew patch/[a]ccept all y Applying: Bug 9341: Problem with UNIMARC authors facets fatal: sha1 information is lacking or useless (C4/Koha.pm). Repository lacks necessary blobs to fall back on 3-way merge. Cannot fall back to three-way merge. Patch failed at 0001 Bug 9341: Problem with UNIMARC authors facets When you have resolved this problem run "git am --resolved". If you would prefer to skip this patch, instead run "git am --skip". To restore the original branch and stop patching run "git am --abort". Can you please rebase and resubmit? If you can rebase without bigger problems (updatedatabase etc. is ok) switch back to Signed off.
Created attachment 15497 [details] [review] Bug 9341: Problem with UNIMARC authors facets In UNIMARC installations the authors have a double comma separating the subfield a and the subfield b. Test plan: - After applying the patch do a search and check the double comma in authors facets. - Apply the patch - Do a new search - Check if authors facets as a double comma Sponsored-by: KEEP SOLUTIONS
Created attachment 16279 [details] [review] Bug 9341: Problem with UNIMARC authors facets In UNIMARC installations the authors have a double comma separating the subfield a and the subfield b. Test plan: - After applying the patch do a search and check the double comma in authors facets. - Apply the patch - Do a new search - Check if authors facets as a double comma Sponsored-by: KEEP SOLUTIONS Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
QA comments: * passes koha-qa.pl * default behaviour unchanged, new behaviour driven by a (needed) syspref passed QA
This patch has been pushed to master.
Pushed to 3.10.x and 3.8.x will be in 3.10.5 and 3.8.12