From 9aa674c14a1f4cdaa4f9c9caf83b474f7247ac70 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Fr=C3=A9d=C3=A9ric=20Demians?= Date: Tue, 26 Nov 2013 11:26:54 +0100 Subject: [PATCH] Bug 11304: fix display of detail page holdings tab in translated Bootstrap theme Detail page tabs are not displayed properly because JavaScript is broken in French. This is due to _(' JS string construction used in place of _(". This patch fixes: - opac-detail.tt page - xt/single_quotes to test bootstrap theme Signed-off-by: Owen Leonard Tested by applying the patch, running translate update, then translate install. Saw no JavaScript error on the French detail page. Signed-off-by: Katrin Fischer Switches from single to double quotes for translatatable strings. Unit tests for xt/single_quotes passes now. Signed-off-by: Galen Charlton (cherry picked from commit 1b80e77f84dd864342220b89acd1210ac53a3f70) Signed-off-by: Fridolin SOMERS --- koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt | 6 +++--- xt/single_quotes.t | 7 +++++-- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt index 8b7cfddf0e..867cb741ea 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt @@ -1467,17 +1467,17 @@ }, function (data) { if (data.auth_status != 'ok') { - window.alert(_('Your CGI session cookie is not current. Please refresh the page and try again.')); + window.alert(_("Your CGI session cookie is not current. Please refresh the page and try again.")); } else { $("#rating_value").val(data.rating_value); if (data.rating_value) { - $("#rating_value_text").text(_('your rating: ') + data.rating_value + ', '); + $("#rating_value_text").text(_("your rating: ") + data.rating_value + ", "); } else { $("#rating_value_text").text(''); } - $("#rating_text").text(_('average rating: ') + data.rating_avg_int + ' (' + data.rating_total + ' ' + _("votes") + ')'); + $("#rating_text").text(_("average rating: ") + data.rating_avg_int + " (" + data.rating_total + ' ' + _("votes") + ")"); } }, "json"); diff --git a/xt/single_quotes.t b/xt/single_quotes.t index b123a6d28c..89b539c426 100755 --- a/xt/single_quotes.t +++ b/xt/single_quotes.t @@ -28,8 +28,11 @@ find( open my $fh, $_ or die "Could not open $_: $!"; my @lines = sort grep /\_\(\'/, <$fh>; push @files, { name => "$_", lines => \@lines } if @lines; - }, - ( "./koha-tmpl/opac-tmpl/prog/en", "./koha-tmpl/intranet-tmpl/prog/en" ) + },qw# + ./koha-tmpl/opac-tmpl/prog/en + ./koha-tmpl/opac-tmpl/bootstrap/en + ./koha-tmpl/intranet-tmpl/prog/en + # ); ok( !@files, "Files do not contain single quotes _(' " ) -- 2.39.5