[Bug 29539] New: UNIMARCslimUtils.xsl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Bug ID: 29539 Summary: UNIMARCslimUtils.xsl Change sponsored?: --- Product: Koha Version: 21.05 Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: OPAC Assignee: oleonard@myacpl.org Reporter: rafael.antonio@sapo.pt QA Contact: testopia@bugs.koha-community.org When it hapens UNIMARC record to have $9 after $a on biblio_metadata record like <datafield tag="606" ind1=" " ind2=" "> <subfield code="a">Portugal</subfield> <subfield code="9">12005</subfield> </datafield> OPAC details display the authority number and not the term. If we change record for <datafield tag="606" ind1=" " ind2=" "> <subfield code="9">12005</subfield> <subfield code="a">Portugal</subfield> </datafield> it display the term. Is it a bug? The only piece of XSLT code I see for such reason is at UNIMARCslimUtils.xsl file <xsl:choose> <xsl:when test="marc:subfield[@code=9]"> Any comment? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|UNIMARCslimUtils.xsl |UNIMARC: authority number | |in $9 displays for 606 | |instead of content of $a -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #1 from Rafael Antonio <rafael.antonio@sapo.pt> --- This problem hapens with all fields that are validate through thesaurus and use $9 to link to authority, not only 606 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|UNIMARC: authority number |UNIMARC: authority number |in $9 displays for 606 |in $9 displays for |instead of content of $a |thesaurus controlled fields | |instead of content of $a -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 George Veranis <gveranis@dataly.gr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |gveranis@dataly.gr --- Comment #2 from George Veranis <gveranis@dataly.gr> --- I confirm that problem exist on version 20.05 and after for all unimarc installations. And the problem should be on template onesubject at UNIMARCslimUtils.xsl on opac , at intranet is working as expected. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #3 from Rafael Antonio <rafael.antonio@sapo.pt> --- I have tried to compare UNIMARCslimUtils release 20 and release 18 but it seems there is no difference on template onesubject. Testing intranet UNIMARCslimUtils on OPAC it does not work. So the only way is to try to find the bug or reson for such. As you can see this is the result on OPAC Country: Portugal. Publication: Lisboa : Gabinete da Inspecção-Geral do Trabalho, 1992 Subject - Topical Name: 9504 | 11070 | 9501 | 9200 | 11252 | 11305 | 8439 | 6383 | 8128 And similar display for Intranet Country: Portugal. Publication: Lisboa : Gabinete da Inspecção-Geral do Trabalho, 1992 Subject - Topical Name: Inspecção-Geral do Trabalho | Relatório | Inspecção do trabalho | Higiene do trabalho | Saúde no trabalho | Segurança no trabalho | Doença profissional | Acidente de trabalho | Dados estatisticos Who can help? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fridolin.somers@biblibre.co | |m --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- i had a look at the XSLT code here, but it is a bit complicated. Adding Frido as he might have more UNIMARC knowledge than I. If you compare files, please note that there are 2 versions of UNIMARCslimUtils.xsl - one on staff and one on OPAC side. So maybe comparing those 2 would give some more hints. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #5 from Rafael Antonio <rafael.antonio@sapo.pt> --- I am not so sure if the problem is on XSLT file or any perl function that was changed and this can be a side efect -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #6 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- The display is rendered from the MARCXML record in the database using XSLT. There are some replacements where authorised values are linked to fields, but they should not be to blame here I think. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #7 from George Veranis <gveranis@dataly.gr> --- For the moment I solve that problem by using the template "tag_subject" from the MARC21slimUtils.xsl and it works fine. and adding also the same codes and subdivCodes from template tag_onesubject that seems to have the problem, but my xslt knowledge is not the best at the moment to solve it or to understand what is wrong, so I decided to use the same template as MARC21. <xsl:template name="tag_subject"> <xsl:param name="tag" /> <xsl:param name="label" /> <xsl:param name="spanclass" /> <xsl:if test="marc:datafield[@tag=$tag]"> <span class="results_summary subjects {$spanclass}"> <span class="label"><xsl:value-of select="$label"/>: </span> <xsl:for-each select="marc:datafield[@tag=$tag]"> <a> <xsl:choose> <xsl:when test="marc:subfield[@code=9]"> <xsl:attribute name="href">/cgi-bin/koha/opac-search.pl?q=an:<xsl:value-of select="marc:subfield[@code=9]"/></xsl:attribute> </xsl:when> <xsl:otherwise> <xsl:attribute name="href">/cgi-bin/koha/opac-search.pl?q=su:<xsl:value-of select="marc:subfield[@code='a']"/></xsl:attribute> </xsl:otherwise> </xsl:choose> <xsl:call-template name="chopPunctuation"> <xsl:with-param name="chopString"> <xsl:call-template name="subfieldSelect"> <xsl:with-param name="codes">abcdfijkmpnptvxyz</xsl:with-param> <xsl:with-param name="subdivCodes">ijknpxyz</xsl:with-param> <xsl:with-param name="subdivDelimiter">-- </xsl:with-param> </xsl:call-template> </xsl:with-param> </xsl:call-template> </a> <xsl:if test="not (position()=last())"> <xsl:text> | </xsl:text> </xsl:if> </xsl:for-each> </span> </xsl:if> </xsl:template> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #8 from Fridolin Somers <fridolin.somers@biblibre.com> --- Hi, This is a very strange feature in UNIMARC XSLT that aim to look for several $9 in authority fields. We should remove it. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #9 from Rafael Antonio <rafael.antonio@sapo.pt> --- Use of $9 is very important for searching using Thesaurus (authority records). May be we need to review XSLT but if Vearnis fix works we can wait for next release we open to be solved -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 George Veranis <gveranis@dataly.gr> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #10 from George Veranis <gveranis@dataly.gr> --- Created attachment 155604 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=155604&action=edit Bug 29539: UNIMARC: authority number in $9 displays for thesaurus controlled fields instead of content of $a When you try to dislpay a bibliographic record on unimarc that has subjects linked with authorities then only the $9 is displayed as link instead of the content of $a and it's subdivisions, if any. To test: 1) You will need to have a bibliographic record with at least one subject autority connected in unimarc framework. 2) View that record on OPAC on detail display. The subject will display as a number ( $9 ) and you cannot see the text/term of the subject ( $a ) 3) Apply patch 4) Repeat step 2 5) The subject display in a normal way based on content $a - or more subfields Sponsored-by: National Library of Greece -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|oleonard@myacpl.org |gveranis@dataly.gr CC| |david@davidnind.com --- Comment #11 from David Nind <david@davidnind.com> --- I've changed the assignee. I couldn't replicate the issue on the current master (using the sample data in koha-testing-docker) - the topical terms for a record displayed correctly in the OPAC. Is this for 21.05 only? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Andreas Roussos <a.roussos@dataly.gr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |a.roussos@dataly.gr -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Thibault Keromnès <thibault.keromnes@univ-paris8.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA CC| |thibault.keromnes@univ-pari | |s8.fr --- Comment #12 from Thibault Keromnès <thibault.keromnes@univ-paris8.fr> --- Couldn't replicate either on a sandbox (display is ok whether it's 606$a and then 606$9 or 606$9 and then 606$a) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Andreas Roussos <a.roussos@dataly.gr> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|21.05 |master --- Comment #13 from Andreas Roussos <a.roussos@dataly.gr> --- (In reply to David Nind from comment #11)
I've changed the assignee. Thanks, David.
I couldn't replicate the issue on the current master (using the sample data in koha-testing-docker) - the topical terms for a record displayed correctly in the OPAC. I just managed to replicate the issue using KTD. Since I already had a MARC21 KTD container up and running on the current master b54da05c73 and didn't want to lose my bash history, I ran these commands to set it up with the UNIMARC sample data and fix the Zebra index configuration:
reset_all unimarc sed -i.safe 's/marc21/unimarc/g' \ /etc/koha/sites/kohadev/zebra-authorities-dom.cfg \ /etc/koha/sites/kohadev/zebra-biblios-dom.cfg koha-rebuild-zebra -a -b -f -v kohadev I then linked the same Topical Name subject with field 606 of a test biblio twice, and changed the order of the subfields so that in the first 606 the $9 would appear after the $a and in the second 606 the $a would appear after the $9. In the OPAC details view for that biblio I got exactly what Rafael describes in this Bug's Description.
Is this for 21.05 only? No, it manifests on the current master, too. I've changed the Version.
-- You are receiving this mail because: You are watching all bug changes.
Couldn't replicate either on a sandbox (display is ok whether it's 606$a and then 606$9 or 606$9 and then 606$a) That's interesting, I actually replicated the problem today by setting up a UNIMARC flavour instance on ByWater Solutions' sandbox server (on
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #14 from Andreas Roussos <a.roussos@dataly.gr> --- (In reply to Thibault Keromnès from comment #12) the current master b54da05c73 and without applying George's patch). As you can see here the first Topical subject does not display properly: https://opac-abefore9.sandbox.bywatersolutions.com/cgi-bin/koha/opac-detail.... The order of the relevant subfields in that biblio is as follows: 606 _a écosystèmes _9 6410 606 _9 6410 _a écosystèmes -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #155604|0 |1 is obsolete| | --- Comment #15 from David Nind <david@davidnind.com> --- Created attachment 161262 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=161262&action=edit Bug 29539: UNIMARC: authority number in $9 displays for thesaurus controlled fields instead of content of $a When you try to dislpay a bibliographic record on unimarc that has subjects linked with authorities then only the $9 is displayed as link instead of the content of $a and it's subdivisions, if any. To test: 1) You will need to have a bibliographic record with at least one subject autority connected in unimarc framework. 2) View that record on OPAC on detail display. The subject will display as a number ( $9 ) and you cannot see the text/term of the subject ( $a ) 3) Apply patch 4) Repeat step 2 5) The subject display in a normal way based on content $a - or more subfields Sponsored-by: National Library of Greece Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This fixes the display of release notes| |authority terms in the OPAC | |for UNIMARC systems. The | |authority record number was | |displaying instead of the | |term, depending on the | |order of the $9 and $a | |subfields (example for a | |606 entry: if $a then $9, | |the authority number was | |displayed; if $9 then $a, | |the authority term was | |displayed). --- Comment #16 from David Nind <david@davidnind.com> --- Thanks Andreas for the steps on how to reproduce the issue! I have now tested and signed off using KTD (koha-testing-docker). Updated test plan: 1. KTD startup: edit your .env file and change KOHA_MARC_FLAVOUR=unimarc 2. Start up KTD as normal: ktd pull and ktd up 3. Edit any record so that it has two 606 entries - NOTE: The order displayed in the editor is important: - For record 560 ([Le ]Jardin des Henderson : roman [LIVR] / Hermary-Vieille Catherine), remove the existing values in the 606 field - Add these values: 606 _a écosystèmes _9 6410 606 _9 6410 _a écosystèmes NOTE: The sample data for UNIMARC records is very incomplete - you will need to: 1. Add data for the mandatory fields before you can save the record. 2. Update the item(s) for the record so that any values with dropdown lists have valid values (otherwise you will have errors in the OPAC when you try to view the record). 4. View the record in the OPAC and staff interface: - OPAC: Subject - Topical Name: 6410 | écosystèmes - Staff interface: Subject - Topical Name: écosystèmes | écosystèmes 5. Apply the patch. 6. Restart everything. 7. View the record again in the OPAC and staff interface - OPAC: Subject - Topical Name: écosystèmes | écosystèmes - Staff interface: Subject - Topical Name: écosystèmes | écosystèmes 8. Experiment with changing the order and adding new terms - these should now display correctly, irrespective of the $a and $9 order in the record. 9. Sign off. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |m.de.rooy@rijksmuseum.nl |y.org | CC| |m.de.rooy@rijksmuseum.nl -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #17 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- - <xsl:call-template name="tag_onesubject"> This is the last reference, right? So we are using it no longer. Please remove dead code too. This template contains a lot of additional logic. Could you or Fridolin explain what its purpose should have been? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #18 from Fridolin Somers <fridolin.somers@biblibre.com> --- (In reply to Marcel de Rooy from comment #17)
This template contains a lot of additional logic. Could you or Fridolin explain what its purpose should have been?
This was a try to show composed authority fields, with method called Rameau. An field like 600 may contain an authority link for $a and another authority link for $x. In this case the $9 must be before the linked field : $9 888 $a History $9 999 $x Antiquity But this is not implemented in Koha record edition so it is logical to remove this behavior. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #19 from Fridolin Somers <fridolin.somers@biblibre.com> --- Created attachment 161622 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=161622&action=edit Bug 29539: (follow-up) remove tag_onesubject template Also: Restores the <span class="value"> Replaces "not(position()=last())" with the more used "position() != last()" Removes unecessary change in <xsl:param name="spanclass" /> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #20 from Fridolin Somers <fridolin.somers@biblibre.com> --- Thanks everyone for moving this dirty bug forward ;) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #161262|0 |1 is obsolete| | --- Comment #21 from David Nind <david@davidnind.com> --- Created attachment 161768 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=161768&action=edit Bug 29539: UNIMARC: authority number in $9 displays for thesaurus controlled fields instead of content of $a When you try to dislpay a bibliographic record on unimarc that has subjects linked with authorities then only the $9 is displayed as link instead of the content of $a and it's subdivisions, if any. To test: 1) You will need to have a bibliographic record with at least one subject autority connected in unimarc framework. 2) View that record on OPAC on detail display. The subject will display as a number ( $9 ) and you cannot see the text/term of the subject ( $a ) 3) Apply patch 4) Repeat step 2 5) The subject display in a normal way based on content $a - or more subfields Sponsored-by: National Library of Greece Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #161622|0 |1 is obsolete| | --- Comment #22 from David Nind <david@davidnind.com> --- Created attachment 161769 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=161769&action=edit Bug 29539: (follow-up) remove tag_onesubject template Also: Restores the <span class="value"> Replaces "not(position()=last())" with the more used "position() != last()" Removes unecessary change in <xsl:param name="spanclass" /> Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #161768|0 |1 is obsolete| | --- Comment #23 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 163191 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163191&action=edit Bug 29539: UNIMARC: authority number in $9 displays for thesaurus controlled fields instead of content of $a When you try to dislpay a bibliographic record on unimarc that has subjects linked with authorities then only the $9 is displayed as link instead of the content of $a and it's subdivisions, if any. To test: 1) You will need to have a bibliographic record with at least one subject autority connected in unimarc framework. 2) View that record on OPAC on detail display. The subject will display as a number ( $9 ) and you cannot see the text/term of the subject ( $a ) 3) Apply patch 4) Repeat step 2 5) The subject display in a normal way based on content $a - or more subfields Sponsored-by: National Library of Greece Signed-off-by: David Nind <david@davidnind.com> Bug 29539: (follow-up) remove tag_onesubject template Also: Restores the <span class="value"> Replaces "not(position()=last())" with the more used "position() != last()" Removes unecessary change in <xsl:param name="spanclass" /> Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Patch doesn't apply --- Comment #24 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Marcel, I tried to apply the patches in the attached sequence and the other way around, I always end up with: Apply? [(y)es, (n)o, (i)nteractive] i Applying: Bug 29539: (follow-up) remove tag_onesubject template error: sha1 information is lacking or useless (koha-tmpl/opac-tmpl/bootstrap/en/xslt/UNIMARCslimUtils.xsl). error: could not build fake ancestor Patch failed at 0001 Bug 29539: (follow-up) remove tag_onesubject template 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-29539-follow-up-remove-tagonesubject-template-daluwkeh.patch kohadev-koha@kohadevbox:koha(master|AM 1/1)$ git log Can you please check/rebase on latest master? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #25 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Katrin Fischer from comment #24)
Hi Marcel, I tried to apply the patches in the attached sequence and the other way around, I always end up with:
Apply? [(y)es, (n)o, (i)nteractive] i Applying: Bug 29539: (follow-up) remove tag_onesubject template error: sha1 information is lacking or useless (koha-tmpl/opac-tmpl/bootstrap/en/xslt/UNIMARCslimUtils.xsl). error: could not build fake ancestor Patch failed at 0001 Bug 29539: (follow-up) remove tag_onesubject template 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-29539-follow-up-remove-tagonesubject-template-daluwkeh.patch kohadev-koha@kohadevbox:koha(master|AM 1/1)$ git log
Can you please check/rebase on latest master?
I would be happy to leave that to the author. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #26 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- That's alright, just easier for someone with the branch still around. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #27 from Andreas Roussos <a.roussos@dataly.gr> --- There's a single sign off in Comment 23 which appears to involve two patches (instead of two separate sign offs, one per patch). I think that's the reason the current set of patches (attachment ids 161768 and 163191) does not apply cleanly. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #161769|0 |1 is obsolete| | --- Comment #28 from David Nind <david@davidnind.com> --- Comment on attachment 161769 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=161769 Bug 29539: (follow-up) remove tag_onesubject template Can confirm that the combined patch still applies. I have obsoleted the separate follow-up patch and will change the status back to Passed QA. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |24.11.00 released in| | Status|Passed QA |Pushed to main -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #29 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Thanks for all the hard work! Pushed to main for the next 24.11.00 release as RM Assistant -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lucas@bywatersolutions.com Status|Pushed to main |Pushed to stable Version(s)|24.11.00 |24.11.00,24.05.02 released in| | --- Comment #30 from Lucas Gass <lucas@bywatersolutions.com> --- Backported to 24.05.x for upcoming 24.05.02 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)|24.11.00,24.05.02 |24.11.00,24.05.02,23.11.07 released in| | Status|Pushed to stable |Pushed to oldstable --- Comment #31 from Fridolin Somers <fridolin.somers@biblibre.com> --- Pushed to 23.11.x for 23.11.07 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 wainuiwitikapark@catalyst.net.nz changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |wainuiwitikapark@catalyst.n | |et.nz --- Comment #32 from wainuiwitikapark@catalyst.net.nz --- not backporting to 23.05.x unless requested -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Tiago Simões <tmb.simoes@fct.unl.pt> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tmb.simoes@fct.unl.pt --- Comment #33 from Tiago Simões <tmb.simoes@fct.unl.pt> --- Hi! Would it be possible to backport it to 22.11? At our library, we've been struggling with this problem for a long time and I'm glad a solution was finally found, as we have a lot of records in the situation described by Rafael António. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #34 from Lucas Gass <lucas@bywatersolutions.com> --- (In reply to Tiago Simões from comment #33)
Hi! Would it be possible to backport it to 22.11? At our library, we've been struggling with this problem for a long time and I'm glad a solution was finally found, as we have a lot of records in the situation described by Rafael António.
Tiago, Wainu declined to backported this to 23.05. Although it sounds like she could be persuaded to do so? If it can be backported to 23.05 there is a chance it will also backported to 22.11. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #35 from Tiago Simões <tmb.simoes@fct.unl.pt> --- (In reply to Lucas Gass from comment #34)
(In reply to Tiago Simões from comment #33)
Hi! Would it be possible to backport it to 22.11? At our library, we've been struggling with this problem for a long time and I'm glad a solution was finally found, as we have a lot of records in the situation described by Rafael António.
Tiago, Wainu declined to backported this to 23.05. Although it sounds like she could be persuaded to do so?
If it can be backported to 23.05 there is a chance it will also backported to 22.11.
I understand. If I'm the only one wanting this, it's understandable that she doesn't backport it. I'll try and make the case to our IT department to upgrade to a more recent version sooner than planned, then. Thanks anyway! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #36 from wainuiwitikapark@catalyst.net.nz --- I can have a look at backporting this to 23.11 for the next release -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #37 from wainuiwitikapark@catalyst.net.nz --- Could I please have a more detailed test plan for 23.05? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #38 from wainuiwitikapark@catalyst.net.nz --- (In reply to wainuiwitikapark from comment #37)
Could I please have a more detailed test plan for 23.05?
For KTD -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #39 from Tiago Simões <tmb.simoes@fct.unl.pt> --- Not sure if the comments above were for me since I expressed interest in backporting it to previous versions than 23.11, but I can try and test in KTD. The problem is that I don't have any experience with docker so it might take a while for me to come up with a test plan as I need to familiarize myself with it. Any kind of help in doing so is welcome, if only to speed things up. If not, as far as I'm concerned, I can wait for an upgrade to a supported version. If the comments weren't directed at me, then please ignore. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #40 from wainuiwitikapark@catalyst.net.nz --- (In reply to Tiago Simões from comment #39)
Not sure if the comments above were for me since I expressed interest in backporting it to previous versions than 23.11, but I can try and test in KTD. The problem is that I don't have any experience with docker so it might take a while for me to come up with a test plan as I need to familiarize myself with it.
Any kind of help in doing so is welcome, if only to speed things up. If not, as far as I'm concerned, I can wait for an upgrade to a supported version.
If the comments weren't directed at me, then please ignore.
Hi Tiago, Thanks for your response! My comments were just directed at anyone really :) I am having trouble setting up the following: You will need to have a bibliographic record with at least one subject autority connected in unimarc framework. I'm not very familiar with unimarc and unsure how to change the marcflavour on my ktd and having it persist through a reset_all. Any help from anyone would be greatly appreciated :) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #41 from Tiago Simões <tmb.simoes@fct.unl.pt> --- (In reply to Wainui Witika-Park from comment #40)
(In reply to Tiago Simões from comment #39)
Not sure if the comments above were for me since I expressed interest in backporting it to previous versions than 23.11, but I can try and test in KTD. The problem is that I don't have any experience with docker so it might take a while for me to come up with a test plan as I need to familiarize myself with it.
Any kind of help in doing so is welcome, if only to speed things up. If not, as far as I'm concerned, I can wait for an upgrade to a supported version.
If the comments weren't directed at me, then please ignore.
Hi Tiago, Thanks for your response!
My comments were just directed at anyone really :)
I am having trouble setting up the following: You will need to have a bibliographic record with at least one subject autority connected in unimarc framework.
I'm not very familiar with unimarc and unsure how to change the marcflavour on my ktd and having it persist through a reset_all.
Any help from anyone would be greatly appreciated :)
Hi Wainui, I've finally set up ktd with the 23.05 version and with the UNIMARC flavour. I've initially started it with marc21 and then switched to unimarc by accessing the .env file in the koha-testing-docker directory and changing the KOHA_MARC_FLAVOUR parameter from marc21 to unimarc. Everything seemed to work fine. I noticed the sample bibliographic records were different (all in french as far as I could tell) and the records' information was filled in the right unimarc fields (again, as far as I could tell). I even tried reset_all and reset_all_unimarc and, again, no issues. But I did notice that there were so little MARC bibliographic frameworks created (Default, Acquisitions and Fast Cataloguing), so the fields were kind of all over the place. Lots of mandatory unimarc fields that should not be mandatory, which makes the process of reproducing the bug difficult, since you first need to edit a record and switch the order in which the 606$9 and 606$a appear. If you still want to go through with backporting to older versions, I can create a bibliographic framework on my ktd environment, at least for books, just to make the process less of a hassle. If not, I completely understand. Either way, would it be worth it to ask for these frameworks to be added to ktd? I'd also add another step between 2) and 3), which is: Edit one or more records and switch the order of $9 and $a so the latter comes before the former. This can be tested with just the 606 field, as it is the most commonly used one in the 6 class. I hope my reply was helpful and not too confusing :). -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #42 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Tiago, there are some other bugs about improving the default UNIMARC frameworks and their behavior filed and in progress already. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #43 from Tiago Simões <tmb.simoes@fct.unl.pt> --- (In reply to Katrin Fischer from comment #42)
Hi Tiago, there are some other bugs about improving the default UNIMARC frameworks and their behavior filed and in progress already.
Hi Katrin, you're right, sorry, I forgot to do a search on bugzilla for them. Thanks for the heads-up. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #44 from Wainui Witika-Park <wainuiwitikapark@catalyst.net.nz> --- I might leave this out of 23.05 if that's okay? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 --- Comment #45 from Tiago Simões <tmb.simoes@fct.unl.pt> --- (In reply to Wainui Witika-Park from comment #44)
I might leave this out of 23.05 if that's okay?
Sure, no problem, I understand. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29539 Wainui Witika-Park <wainuiwitikapark@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to oldstable |Needs documenting --- Comment #46 from Wainui Witika-Park <wainuiwitikapark@catalyst.net.nz> --- Not backporting to 22.11 unless requested -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org