The 2 files have diverged in logic: opac-tmpl/bootstrap/en/includes/i18n.inc koha-tmpl/bootstrap/en/includes/i18n.inc Adding bug 35716 as "See also" as this logic should ideally be DRY.
Created attachment 162955 [details] [review] Bug 36282: [DO NOT PUSH] For test plan only
Created attachment 162956 [details] [review] Bug 36282: Chomp whitespace around opac i18n strings Same logic as bug 26816 but to OPAC Test plan: 1) Apply test plan patch only 2) Visit opac home: <opac_url>/cgi-bin/koha/opac-main.pl 3) Notice there's a space between 'Translated string' and ':', even though the markup does not contain a whitespace 4) Apply 2nd patch 5) Notice the white space is gone
Created attachment 163010 [details] [review] Bug 36282: Chomp whitespace around opac i18n strings Same logic as bug 26816 but to OPAC Test plan: 1) Apply test plan patch only 2) Visit opac home: <opac_url>/cgi-bin/koha/opac-main.pl 3) Notice there's a space between 'Translated string' and ':', even though the markup does not contain a whitespace 4) Apply 2nd patch 5) Notice the white space is gone Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Created attachment 165321 [details] [review] Bug 36282: Chomp whitespace around opac i18n strings Same logic as bug 26816 but to OPAC Test plan: 1) Apply test plan patch only 2) Visit opac home: <opac_url>/cgi-bin/koha/opac-main.pl 3) Notice there's a space between 'Translated string' and ':', even though the markup does not contain a whitespace 4) Apply 2nd patch 5) Notice the white space is gone Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Obvious code correction, no regressions. Passing QA
Pushed for 24.05! Well done everyone, thank you!
Pushed to 23.11.x for 23.11.06
Backported to 23.05.x for upcoming 23.05.12
Not backporting to 22.11 unless requested