From af05c00501d2cc245a3f3af929fe30c7dca4019a Mon Sep 17 00:00:00 2001 From: Fridolin Somers Date: Wed, 5 Mar 2014 14:26:14 +0100 Subject: [PATCH] Bug 11366: make "no group" option in acq basket group drop-down translatable in Acq baskets, there's a pull-down for basket groups. One of the entries in that pull-down is "No group", which is untranslatable. This string is hard-coded in Perl. This patch removes the string from Perl to set it has first option in select. To allow it to be default value, the option "Add new group" is moved to last position. Test plan : - Go to a closed aquisition basket in no basket group : cgi-bin/koha/acqui/basket.pl?basketno=x => You see "No group" selected in combobox "Basket group" - Cick on this combobox => You see "No group", then existing basket groups and then "Add new group" - Select a basket group and click on "change basket group" => You see the basket group name in combobox Use translation, for example fr-FR - go to src/misc/translator - run : perl translate update fr-FR => You find in PO file : #: intranet-tmpl/prog/en/modules/acqui/basket.tt:365 #, fuzzy, c-format msgid "No group" msgstr "Nom de groupe" - remove ", fuzzy" and correct translation : "Pas de groupe" - run : perl translate install fr-FR - Go to translated aquisition basket page => You see translated option in combobox Signed-off-by: Kyle M Hall Signed-off-by: Marcel de Rooy Signed-off-by: Galen Charlton (cherry picked from commit 21e6518d32477fa1fe59da64d735f3f07ae49bcd) Signed-off-by: Fridolin Somers Signed-off-by: Kyle M Hall (cherry picked from commit 1b760f8e4df32f73861b96ffc5ea4e2d0bd196a5) --- acqui/basket.pl | 7 ------- koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt | 3 ++- 2 files changed, 2 insertions(+), 8 deletions(-) diff --git a/acqui/basket.pl b/acqui/basket.pl index 9dad355301..c916138933 100755 --- a/acqui/basket.pl +++ b/acqui/basket.pl @@ -216,13 +216,6 @@ if ( $op eq 'delete_confirm' ) { $basketgroup = $bg; } } - my %emptygroup = ( id => undef, - name => "No group"); - if ( ! $basket->{basketgroupid} ) { - $emptygroup{default} = 1; - $emptygroup{nogroup} = 1; - } - unshift( @$basketgroups, \%emptygroup ); } # if the basket is closed, calculate estimated delivery date diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt index 6bfce77ded..52000805ac 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt @@ -225,7 +225,7 @@ [%- END -%] [% ELSE %] -- 2.39.5