[Bug 31718] New: Change the IF ELSE values in marc_subfields_structure breadcrumbs to facilitate translation
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 Bug ID: 31718 Summary: Change the IF ELSE values in marc_subfields_structure breadcrumbs to facilitate translation Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Templates Assignee: oleonard@myacpl.org Reporter: caroline.cyr-la-rose@inlibro.com QA Contact: testopia@bugs.koha-community.org This section of the breadcrumb in marc_subfields_structure.tt controls if it says we are in the default framework structure or the XYZ framework structure. [% IF ( frameworkcode ) %] [% frameworkcode | html %] [% ELSE %] <span>Default</span> [% END %] <span>framework structure</span> Before bug 29602, the string to translate looked like this %s %s %s Default %s framework structure Now, the 'Default' and the 'framework structure' are two different strings. However, in French, the correct syntax is 'framework structure default'. Before bug 29602, we could play around with the %s to accomodate the different syntax structure. Since the option to move the %s was removed, I propose we write the whole phrase 'Default framework structure' or 'XYZ framework structure' in the IF ELSE. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |Academy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 --- Comment #1 from Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> --- Something like [% IF ( frameworkcode ) %] [% frameworkcode | html %] <span>framework structure</span> [% ELSE %] <span>Default framework structure</span> [% END %] -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 --- Comment #2 from Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> --- (In reply to Caroline Cyr La Rose from comment #1)
Something like
[% IF ( frameworkcode ) %] [% frameworkcode | html %] <span>framework structure</span> [% ELSE %] <span>Default framework structure</span> [% END %]
The span needs to be around the [% frameworkcode | html %] part so that we can move it around in the string. <span>[% frameworkcode | html %] framework structure</span> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 --- Comment #3 from Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> --- Created attachment 141526 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=141526&action=edit Bug 31718: Change the IF ELSE values in marc_subfields_structure breadcrumbs to facilitate translation This patch changes the strings in the IF ELSE for the framework name in the breadcrumbs on the marc_subfields_structure.pl page. To test: 1) Apply patch 2) In the intranet, go to Administration > MARC bibliographic framework 3) Click 'Actions' next to the default framework and choose 'MARC structure' 4) Click 'Actions' next to a MARC tag and choose 'View subfields' 5) Check the breadcrumbs, they should show Home > Administration > MARC frameworks > Default framework structure > ... 6) Redo steps 2-5, using a different framework, e.g. ACQ The breadcrumbs for ACQ should look like Home > Administration > MARC frameworks > ACQ framework structure > ... -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|oleonard@myacpl.org |caroline.cyr-la-rose@inlibr | |o.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This enhancement changes release notes| |the strings in the IF ELSE | |for the framework name in | |the breadcrumbs on the | |marc_subfields_structure.pl | |page. There is no change to | |the page in English. | |However, it will facilitate | |the translation by having | |complete strings. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 --- Comment #4 from Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> --- For Release Maintainers, this is needed as far back as 22.05. Before that it's not needed. For translators, the two strings will now be Default framework structure and %s framework structure In French, the translations will be Structure de la grille par défaut and Structure de la grille %s -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |30733 CC| |fridolin.somers@biblibre.co | |m Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30733 [Bug 30733] Simplify translatable strings -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 --- Comment #5 from Fridolin Somers <fridolin.somers@biblibre.com> --- Created attachment 141536 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=141536&action=edit Bug 31718: Others facilitate translation in frameworks administration Some other IF/ELSE/END can be changed to facilitate translation Test plan : Always test with default framework and with another framework and compare with and without patch 1) View subfields of field => Check the h1 in page 2) Perform actions on a field (create, modify, delete) => Check breadcrumbs and h1 in page 3) View the framework table => Check columns Repeatable, Mandatory, Important -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 --- Comment #6 from Fridolin Somers <fridolin.somers@biblibre.com> --- I signed first patch and added a new one for some other IF/ELSE/END that can facilitate translation. Like did Bug 30733 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 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=31718 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #141526|0 |1 is obsolete| | --- Comment #7 from David Nind <david@davidnind.com> --- Created attachment 141556 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=141556&action=edit Bug 31718: Change the IF ELSE values in marc_subfields_structure breadcrumbs to facilitate translation This patch changes the strings in the IF ELSE for the framework name in the breadcrumbs on the marc_subfields_structure.pl page. To test: 1) Apply patch 2) In the intranet, go to Administration > MARC bibliographic framework 3) Click 'Actions' next to the default framework and choose 'MARC structure' 4) Click 'Actions' next to a MARC tag and choose 'View subfields' 5) Check the breadcrumbs, they should show Home > Administration > MARC frameworks > Default framework structure > ... 6) Redo steps 2-5, using a different framework, e.g. ACQ The breadcrumbs for ACQ should look like Home > Administration > MARC frameworks > ACQ framework structure > ... 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=31718 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #141536|0 |1 is obsolete| | --- Comment #8 from David Nind <david@davidnind.com> --- Created attachment 141557 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=141557&action=edit Bug 31718: Others facilitate translation in frameworks administration Some other IF/ELSE/END can be changed to facilitate translation Test plan : Always test with default framework and with another framework and compare with and without patch 1) View subfields of field => Check the h1 in page 2) Perform actions on a field (create, modify, delete) => Check breadcrumbs and h1 in page 3) View the framework table => Check columns Repeatable, Mandatory, Important 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=31718 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com --- Comment #9 from David Nind <david@davidnind.com> --- Testing notes (using KTD) ------------------------- I checked the default framework and the BKS framework. First patch - breadcrumbs when viewing and editing subfields (changes to a single span tag for the framework name): - View the page source - Search for breadcrumbs - Before the patch is applied - default framework: <span>Default</span> <span>framework structure</span> - After the patch is applied - default framework: <span>Default framework structure</span> - Before the patch is applied - BKS framework: BKS <span>framework structure</span> - After the patch is applied - BKS framework: <span>BKS framework structure</span> Second patch - step 1 - view subfields: - View the page source - Search for <h1> - No change to output for h1 before and after the patch is applied: . Default framework: <h1>MARC subfield structure admin for 245 (default framework)</h1> . BKS framework: <h1>MARC subfield structure admin for 245 (framework BKS)</h1> . Updates code for template so that it is consistent with other changes (I think) - Breadcrumbs: changes as per the first patch Second patch - step 2 - add, edit and delete a MARC tag: - View the page source - Search for breadcrumbs - After patch is applied: <span> tags added to breadcrumbs where required Second patch - step 3: Framework table: - View the page source - Search for Repeatable - Check the first row of results: ==> should now have <span> tags around all the Nos and Yeses I also noticed that when deleting a tag from the view subfields page that there is an 'i' added to the breadcrumb, for example: - Home > Administration > MARC frameworks > BKS framework structure > Tag 245 Subfield structure > Confirm deletion of subfield ci - Should be ... Confirm deletion of subfield c I've created bug 31724 for this. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |katrin.fischer@bsz-bw.de |y.org | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #141556|0 |1 is obsolete| | --- Comment #10 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 141924 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=141924&action=edit Bug 31718: Change the IF ELSE values in marc_subfields_structure breadcrumbs to facilitate translation This patch changes the strings in the IF ELSE for the framework name in the breadcrumbs on the marc_subfields_structure.pl page. To test: 1) Apply patch 2) In the intranet, go to Administration > MARC bibliographic framework 3) Click 'Actions' next to the default framework and choose 'MARC structure' 4) Click 'Actions' next to a MARC tag and choose 'View subfields' 5) Check the breadcrumbs, they should show Home > Administration > MARC frameworks > Default framework structure > ... 6) Redo steps 2-5, using a different framework, e.g. ACQ The breadcrumbs for ACQ should look like Home > Administration > MARC frameworks > ACQ framework structure > ... Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #141557|0 |1 is obsolete| | --- Comment #11 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 141925 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=141925&action=edit Bug 31718: Others facilitate translation in frameworks administration Some other IF/ELSE/END can be changed to facilitate translation Test plan : Always test with default framework and with another framework and compare with and without patch 1) View subfields of field => Check the h1 in page 2) Perform actions on a field (create, modify, delete) => Check breadcrumbs and h1 in page 3) View the framework table => Check columns Repeatable, Mandatory, Important Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Change the IF ELSE values |Change the IF ELSE values |in marc_subfields_structure |in MARC subfields structure |breadcrumbs to facilitate |breadcrumbs to facilitate |translation |translation -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 --- Comment #12 from Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> --- Release maintainers, this is needed as far back as 22.05, but not before that. Thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |victor@tuxayo.net --- Comment #13 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Thanks for the fix Caroline! In 30733 I used the same trick in a few places where I noticed it would be an issue. I hope librarians aren't too unhappy with this missed case ^^
Release maintainers, this is needed as far back as 22.05, but not before that.
Good that you repeat, because RMaints are unlikely to read all the comment history. But don't worry, having bug 30733 in the "Depends on" field here takes care of ensuring that it's not backported too far. Also the way it will conflict when trying to backport would give a hint that it's missing a dependency. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |22.11.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=31718 --- Comment #14 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Pushed to master for 22.11. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 --- Comment #15 from Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> --- (In reply to Victor Grousset/tuxayo from comment #13)
Release maintainers, this is needed as far back as 22.05, but not before that.
Good that you repeat, because RMaints are unlikely to read all the comment history. But don't worry, having bug 30733 in the "Depends on" field here takes care of ensuring that it's not backported too far. Also the way it will conflict when trying to backport would give a hint that it's missing a dependency.
I mainly wanted to let Lucas know that it was important to add it in 22.05 and not skip it :) After that, I figured it wouldn't apply... -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 --- Comment #16 from Victor Grousset/tuxayo <victor@tuxayo.net> ---
After that, I figured it wouldn't apply...
It depends on the conflicts, if they are simple, Lucas will be able to solve them. Otherwise they will call for a backported patch here if someone needs it. You might then need to ask your dev colleagues to try to backport it and solve the conflict if you really need it. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31718 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED CC| |lucas@bywatersolutions.com Status|Pushed to master |RESOLVED -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org