https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21130 --- Comment #1 from David Bourgault <david.bourgault@inlibro.com> --- Created attachment 77321 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=77321&action=edit Bug 21130: Prevent HTML class from being translated Adds a xsl:value-of tag to prevent the value from being translated. TEST PLAN 0) Apply patch 1) Translate 2) In the opac, div.record has not changed NOTE: I don't know if this bug affects all translations, I noticed it when translating to fr-CA. The fix will work for all languages. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21130 --- Comment #2 from Blou <philippe.blouin@inlibro.com> --- Created attachment 78566 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=78566&action=edit Bug 21130: (Follow up OPAC Results) Prevent HTML class from being translated This fixes the same issue where "title" is being translated (in fr-CA at least) and thus break the javascript impacting the result display in the OPAC -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21130 Blou <philippe.blouin@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #79058|0 |1 is obsolete| | --- Comment #4 from Blou <philippe.blouin@inlibro.com> --- Created attachment 79110 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79110&action=edit Bug 21130: (Follow up OPAC Results) Prevent HTML class from being translated This fixes the same issue where "title" is being translated (in fr-CA at least) and thus break the javascript impacting the result display in the OPAC -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21130 --- Comment #6 from Blou <philippe.blouin@inlibro.com> --- Examine opac-tmrpl/bootstrap/en/xslt/MARC21slim2OPACDetail.xsl around line 110 (just below the "Wrapper div" comment. You'll find <xsl:attribute name="class">record</xsl:attribute> Translate, then look at the (fr-CA) version, you'll find <xsl:attribute name="class">notice</xsl:attribute> This will in turn create in the webpage some <div class="notice" instead of <div class="record" Bad. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21130 --- Comment #8 from M. Tompsett <mtompset@hotmail.com> --- (In reply to Blou from comment #6)

Examine opac-tmrpl/bootstrap/en/xslt/MARC21slim2OPACDetail.xsl around line 110 (just below the "Wrapper div" comment. You'll find

<xsl:attribute name="class">record</xsl:attribute>

Translate, then look at the (fr-CA) version, you'll find

Translate how? You did not give the steps.

<xsl:attribute name="class">notice</xsl:attribute>

This will in turn create in the webpage some <div class="notice" instead of <div class="record"

Bad.

Imagine I only how how to click and type. Tell me more detailed instructions starting from: 1) vagrant up jessie 2) cd kohaclone 3) git checkout -b bug_21130 origin/master 3a) Oops... forgot to do the opac steps first to find the page that is broken. 3b) something about looking for something on a page where it is going to be wrong. 4) git bz apply 21130 5) restart_all 6) sudo service apache2 restart 7) sudo systemctl daemon-reload 8) kshell 9) cd misc/translator 10) ./translate --help -- I'm stuck. 11) open the opac 12) change language -- oops... forgot to set some system preferences, but I'm playing ignorant, so I'm stuck, because there is no way to change anything. 13) inspect the page source -- looking for something gives me the line that should be corrected now. I need more details. I can fill in some of them, but this doesn't give me enough information to test still. Imagine the person is trying to set up Koha for the first time. Write instructions. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21130 Hayley Mapley <hayleymapley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #79110|0 |1 is obsolete| | --- Comment #9 from Hayley Mapley <hayleymapley@catalyst.net.nz> --- Created attachment 86421 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=86421&action=edit Bug 21130: (Follow up OPAC Results) Prevent HTML class from being translated This fixes the same issue where "title" is being translated (in fr-CA at least) and thus break the javascript impacting the result display in the OPAC Signed-off-by: Hayley Mapley <hayleymapley@catalyst.net.nz> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21130 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #77321|0 |1 is obsolete| | --- Comment #10 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 86886 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=86886&action=edit Bug 21130: Prevent HTML class from being translated Adds a xsl:value-of tag to prevent the value from being translated. TEST PLAN 0) Apply patch 1) Translate 2) In the opac, div.record has not changed NOTE: I don't know if this bug affects all translations, I noticed it when translating to fr-CA. The fix will work for all languages. 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=21130 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |m.de.rooy@rijksmuseum.nl QA Contact|testopia@bugs.koha-communit |m.de.rooy@rijksmuseum.nl |y.org | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21130 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable CC| |martin.renvoize@ptfs-europe | |.com --- Comment #13 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Pushed to 18.11.x for 18.11.04 -- You are receiving this mail because: You are watching all bug changes.