[Bug 25082] New: Unknown language code if 041 $a is linked to an authorized value list
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Bug ID: 25082 Summary: Unknown language code if 041 $a is linked to an authorized value list Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: MARC Bibliographic data support Assignee: koha-bugs@lists.koha-community.org Reporter: pablo.bianchi@gmail.com QA Contact: testopia@bugs.koha-community.org
From Bug 20364, comment 13:
When 041 $a (Language code of text/sound track or separate title) is associated with the LANG authorized value list, then the detail view of the record (OPAC and Intranet) show an "Unknown language code", even if the value of the subfield is a valid language code. This text comes from MARC21Languages.xsl. This behavior was expected? Maybe MARC21Languages.xsl code → Language list should be just a fallback if the code is not present/translated on associated list (probably LANG). -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Pablo AB <pablo.bianchi@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |arees@stlukes.nsw.edu.au, | |fridolin.somers@biblibre.co | |m, | |katrin.fischer@bsz-bw.de, | |m.de.rooy@rijksmuseum.nl, | |marie-luce.laflamme@inlibro | |.com, | |martin.renvoize@ptfs-europe | |.com, | |pablo.bianchi@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Pablo AB <pablo.bianchi@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |20364 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20364 [Bug 20364] Show languages from MARC21 field 041 in intranet -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #1 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Pablo, I have no way to tell in the XSLT if yo uhave mapped the field to an authorised language list. The issue here is not that it's a valid language code - the issue is the XSLT contains some magic - so we end with the description in the XSLT that is processed (which will never be a valid language code...) What I can do is say: if you don't know the code, display 041$a as it was stored. Someone else might be able to provide a more elaborate solution. I'll have a go and you can see if it works better for you. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #2 from Katrin Fischer <katrin.fischer@bsz-bw.de> ---
display 041$a as it was stored.
Hm, stored is not quite right: as it was passed to the XSLT -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- What happens: - When LANG (or any authorised value) is used with a subfield, Koha will replace the codes in the MARCXML with the descriptions, before it is processed by the XSLT, so 041$ager Turns into: 041$aGerman The XSLT currently expects language codes, not descriptions. So when it gets handed a description, it will turn it into "Unknown language". -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 102959 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=102959&action=edit Bug 25082: Make 041 descriptions display, if authorised value list is used When 041$a or another subfield is linked to an authorised value list, Koha will replace the codes with the descriptions from the authorised value in the MARCXML before passing it to the XSLT. The XSLT was made to translate codes into descriptions - as there is no code now, this fails and 'unknown language' is shown. The patch changes the handling so that when the XSLT is handled something that doesn't match a known code, it displays the subfield without processing. To test: - Create some records with 041. Examples: http://www.loc.gov/marc/bibliographic/bd041.html - Verify this displays nicely in staff and OPAC - Apply patch - Verify display is unchanged - Change one of your MARC framworks (use another framework than for your first test records) and link the 041 subfields to the LANG authorised value list - Go to administration and create some entries for LANG with language codes and descriptions - Catalog a 041 using the normal editor (not advanced) - You can now use a pull down to select the values you want - Verify the display is now 'Unknown language' for this record - Apply patch - Verify all records display the language descriptions now, using an authorised value or not - Run misc/maintenance/generate_MARC21Languages.pl Verify the output at the end reads like: <xsl:otherwise> <!-- when 041\$a is mapped to an authorised value list, this will show the description --> <xsl:value-of select="\$code" /> </xsl:otherwise> -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |katrin.fischer@bsz-bw.de |ity.org | -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |rel_20_05_target -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |bgkriegel@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Bernardo Gonzalez Kriegel <bgkriegel@gmail.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=25082 Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #102959|0 |1 is obsolete| | --- Comment #5 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- Created attachment 102988 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=102988&action=edit Bug 25082: Make 041 descriptions display, if authorised value list is used When 041$a or another subfield is linked to an authorised value list, Koha will replace the codes with the descriptions from the authorised value in the MARCXML before passing it to the XSLT. The XSLT was made to translate codes into descriptions - as there is no code now, this fails and 'unknown language' is shown. The patch changes the handling so that when the XSLT is handled something that doesn't match a known code, it displays the subfield without processing. To test: - Create some records with 041. Examples: http://www.loc.gov/marc/bibliographic/bd041.html - Verify this displays nicely in staff and OPAC - Apply patch - Verify display is unchanged - Remove the patch - Change one of your MARC frameworks (use another framework than for your first test records) and link the 041 subfields to the LANG authorised value list - Go to administration and create some entries for LANG with language codes and descriptions - Catalog a 041 using the normal editor (not advanced) - You can now use a pull down to select the values you want - Verify the display is now 'Unknown language' for this record - Apply patch - Verify all records display the language descriptions now, using an authorised value or not - Run misc/maintenance/generate_MARC21Languages.pl Verify the output at the end reads like: <xsl:otherwise> <!-- when 041\$a is mapped to an authorised value list, this will show the description --> <xsl:value-of select="\$code" /> </xsl:otherwise> Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Amended test plan (add remove patch) to verify the error Work as described following test plan, no errors. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #6 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Thx Bernardo! - already spotting more typos in the commit message *looks away quickly* -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #7 from Pablo AB <pablo.bianchi@gmail.com> --- Thanks to both! Will be backported to stable releases? I don't know where to check this info. I'm not sure if this bug is related to Bug 20307. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #8 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I've marked it as a bug - if the Release maintainers agree it will be likely to be backported. But at the moment we still need it to go through QA first :) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org --- Comment #9 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Katrin, I see AV's code, should not it be displayed AV's description? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #10 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- This should work like this: When you don't use AVs, it will display the descriptions from the XSLT file. If you are using AVs, it will display the descriptions of the AVs. Does that make sense? You should only see codes if you had entered a code that is neither in the XSLT nor in the AV list. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #11 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- How did you test? Did you link your 041$a to the AV before testing this? Note: Using the AV is not standard and there is no complete LANG list we ship with Koha, so I'd say not using the AV is more standard. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #12 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- I created a new LANG code=xxx, description=description xxx I set a 041$a to xxx and "xxx" was displayed on the detail page. I had other 041 with "correct" value (translated into correct language string). IIRC I used 041##$adut$afre$ager$aita$aspa$beng and replaced fre with xxx -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #13 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- I may have missed something however, I can retest if you think I tested it wrong. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #14 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- The AV translation will only work if you change your frameworks: You need to linkt he LANG authorised value category to 041$a -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #15 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Katrin Fischer from comment #14)
The AV translation will only work if you change your frameworks: You need to linkt he LANG authorised value category to 041$a
Yes of course. I did that. Ok retrying now. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #16 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- - Apply the patch - Edit a record, add 041 _ _ ‡adut‡axxx‡ager‡aita‡aspa‡beng - Create a LANG AV xxx="description xxx" - link 041$a with LANG - view record: Language: , descriptionxxx, , , . Summary language: English. - remove the AV Language: , , , , . Summary language: English. - remove the AV from $041$a Language: Dutch, xxx, German, Italian, Spanish. Summary language: English. - Create a LANG AV xxx="description xxx" Language: Dutch, xxx, German, Italian, Spanish. Summary language: English. (Hum??) - restart_all! Language: Dutch, xxx, German, Italian, Spanish. Summary language: English. (Hum??) I have no idea what's going on, but I never get the expected display, should be Language: Dutch, description xxx, German, Italian, Spanish. Summary language: English. right? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #17 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hm, I will have a go later. - Apply the patch - Edit a record, add 041 _ _ ‡adut‡axxx‡ager‡aita‡aspa‡beng - Create a LANG AV xxx="description xxx" - link 041$a with LANG Correct. - view record: Language: , descriptionxxx, , , . Summary language: English. I'd have expected the codes to show here, but I wonder what is sent to the XSLT in these cases. If the code that switches the code for the description in the MARCXML record changes an unknown code to "", that would explain what you see there. - remove the AV Language: , , , , . Summary language: English. See above. - remove the AV from $041$a Language: Dutch, xxx, German, Italian, Spanish. Summary language: English. That's the expected display. xxx is not in the XSLT file (no valid language code in MARC, so we display the code. - Create a LANG AV xxx="description xxx" Language: Dutch, xxx, German, Italian, Spanish. Summary language: English. (Hum??) Now it's not linked to the framework, so it has no effect. Displays the same as without the AV, this is expected. - restart_all! Language: Dutch, xxx, German, Italian, Spanish. Summary language: English. (Hum??) Still expected. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #18 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- After this patch applied, LANG linked with 041$a, I think we are expecting Language: Dutch, description xxx, German, Italian, Spanish. Summary language: English. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #19 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Jonathan Druart from comment #18)
After this patch applied, LANG linked with 041$a, I think we are expecting
Language: Dutch, description xxx, German, Italian, Spanish. Summary language: English.
What Koha does if you use an authorised value is a replacement in the MARCXML that is processed by XSLT. The question here is: What happens if you haven't added a code to the AV? Does it leave the code or does it ""? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #20 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Or in other words: What does the MARCXML look like that hits the XSLT? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #102988|0 |1 is obsolete| | --- Comment #21 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 104355 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=104355&action=edit Bug 25082: Make 041 descriptions display, if authorised value list is used When 041$a or another subfield is linked to an authorised value list, Koha will replace the codes with the descriptions from the authorised value in the MARCXML before passing it to the XSLT. The XSLT was made to translate codes into descriptions - as there is no code now, this fails and 'unknown language' is shown. The patch changes the handling so that when the XSLT is handled something that doesn't match a known code, it displays the subfield without processing. To test: - Create some records with 041. Examples: http://www.loc.gov/marc/bibliographic/bd041.html - Verify this displays nicely in staff and OPAC - Apply patch - Verify display is unchanged - Remove the patch - Change one of your MARC frameworks (use another framework than for your first test records) and link the 041 subfields to the LANG authorised value list - Go to administration and create some entries for LANG with language codes and descriptions - Catalog a 041 using the normal editor (not advanced) - You can now use a pull down to select the values you want - Verify the display is now 'Unknown language' for this record - Apply patch - Verify all records display the language descriptions now, using an authorised value or not - Run misc/maintenance/generate_MARC21Languages.pl Verify the output at the end reads like: <xsl:otherwise> <!-- when 041\$a is mapped to an authorised value list, this will show the description --> <xsl:value-of select="\$code" /> </xsl:otherwise> Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Amended test plan (add remove patch) to verify the error Work as described following test plan, no errors. Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #22 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- I now see "Language: descriptionxyz. Summary language: English." Because $b is not linked. There is a missing space (should be "description xyz"), but it's something else, maybe bug 25373. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #23 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (I was editing the default framework, the biblio record was using BKS. The confusing was because the adv editor is displaying the dropdown list). -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Failed QA --- Comment #24 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- <cait1> I need to change this line here: <cait1> + <xsl:with-param name="code" select="translate(., ' .-;|#', '')"/> <cait1> please fail it - it doesn't work for language descriptions with mulitple words <cait1> we don't want to replace the _ <cait1> the space -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Katrin Fischer <katrin.fischer@bsz-bw.de> 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=25082 --- Comment #25 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 104379 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=104379&action=edit Bug 25082: Don't remove characters from string At the moment, show-lang-041 will either get a code or an AV description to deal with. If it's a description, we don't want to remove spaces and other chars from it. To test: - Repeat tests from first patch with an AV in LANG category that consists of multiple words, has a - and other possible punctuation. - The description should show correctly in OPAC and staff. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #26 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Sorry but still wrong. Summary, using 041 _ _ ‡adut‡axxx‡ager‡aita‡aspa‡beng * No mapping Before: Language: Dutch, Unknown language code, German, Italian, Spanish. Summary language: English. After: Language: Dutch, xxx, German, Italian, Spanish. Summary language: English. Mapped with LANG (no AV) * Mapped with LANG (no AVs yet) Before: Language: Unknown language code, Unknown language code, Unknown language code, Unknown language code, Unknown language code. Summary language: English. After: Language: , , , , . Summary language: English. * Create 2 LANG (xxx=description xyz, spa=Spanish) Before: Language: Unknown language code, Unknown language code, Unknown language code, Unknown language code, Unknown language code. Summary language: English. After: Language: , description xyz, , , Spanish. Summary language: English. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #27 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Jonathan, trying to explain. Please have a look at Please have a look at: http://git.koha-community.org/gitweb/?p=koha.git;a=blob;f=C4/XSLT.pm;h=1b394... to start with. It explains what happens, when you map an authorised value to a MARC field. (In reply to Jonathan Druart from comment #26)
Sorry but still wrong.
Summary, using 041 _ _ ‡adut‡axxx‡ager‡aita‡aspa‡beng
* No mapping Before: Language: Dutch, Unknown language code, German, Italian, Spanish. Summary language: English. After: Language: Dutch, xxx, German, Italian, Spanish. Summary language: English. Mapped with LANG (no AV)
We display the code instead of the "Unknown language", that was an intentional change. The code displaying here will not always be a code, if the AV is mapped, it's the description.
* Mapped with LANG (no AVs yet) Before: Language: Unknown language code, Unknown language code, Unknown language code, Unknown language code, Unknown language code. Summary language: English. After: Language: , , , , . Summary language: English.
I assume, and you might be able to verify, that transformMARCXML4XSLT is to blame for this, not the XSLT. I think if the language code doesn't exist in LANG, it gives us "". I cannot do anything about this on XSLT level.
* Create 2 LANG (xxx=description xyz, spa=Spanish) Before: Language: Unknown language code, Unknown language code, Unknown language code, Unknown language code, Unknown language code. Summary language: English. After: Language: , description xyz, , , Spanish. Summary language: English.
Ok, this is harder. What I could do, if we get "" from transformMARCXML4XSLT is a fallback if code = "" display "Unknown language". Would this be ok? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #104355|0 |1 is obsolete| | Attachment #104379|0 |1 is obsolete| | --- Comment #28 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 104497 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=104497&action=edit Bug 25082: Make 041 descriptions display, if authorised value list is used When 041$a or another subfield is linked to an authorised value list, Koha will replace the codes with the descriptions from the authorised value in the MARCXML before passing it to the XSLT. The XSLT was made to translate codes into descriptions - as there is no code now, this fails and 'unknown language' is shown. The patch changes the handling so that when the XSLT is handled something that doesn't match a known code, it displays the subfield without processing. To test: - Create some records with 041. Examples: http://www.loc.gov/marc/bibliographic/bd041.html - Verify this displays nicely in staff and OPAC - Apply patch - Verify display is unchanged - Remove the patch - Change one of your MARC frameworks (use another framework than for your first test records) and link the 041 subfields to the LANG authorised value list - Go to administration and create some entries for LANG with language codes and descriptions - Catalog a 041 using the normal editor (not advanced) - You can now use a pull down to select the values you want - Verify the display is now 'Unknown language' for this record - Apply patch - Verify all records display the language descriptions now, using an authorised value or not - Run misc/maintenance/generate_MARC21Languages.pl Verify the output at the end reads like: <xsl:otherwise> <!-- when 041\$a is mapped to an authorised value list, this will show the description --> <xsl:value-of select="\$code" /> </xsl:otherwise> Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Amended test plan (add remove patch) to verify the error Work as described following test plan, no errors. Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #29 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 104498 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=104498&action=edit Bug 25082: Don't remove characters from string At the moment, show-lang-041 will either get a code or an AV description to deal with. If it's a description, we don't want to remove spaces and other chars from it. To test: - Repeat tests from first patch with an AV in LANG category that consists of multiple words, has a - and other possible punctuation. - The description should show correctly in OPAC and staff. Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA --- Comment #30 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Let this one go as it and see later if we need to deal with edge cases. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |20.05.00 released in| | Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 --- Comment #31 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Nice work everyone! Pushed to master for 20.05 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Joy Nelson <joy@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |joy@bywatersolutions.com Version(s)|20.05.00 |20.05.00, 19.11.06 released in| | Status|Pushed to master |Pushed to stable --- Comment #32 from Joy Nelson <joy@bywatersolutions.com> --- Backported to 19.11.x for 19.11.06 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED CC| |lucas@bywatersolutions.com Status|Pushed to stable |RESOLVED --- Comment #33 from Lucas Gass <lucas@bywatersolutions.com> --- missing dependencies for 19.05.x, no backport -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25082 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|rel_20_05_candidate | -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org