From d38b29ed4fd2dab544369b90fc5b502264896221 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 20 Apr 2015 16:59:05 +0200 Subject: [PATCH] Bug 14022: Does not reset the subscription length on editing a subscription The subscription length is always set to the first/default value ('issues') when editing a subscription. Even if you selected another value on creating/editing it, the form will show you the first option. Test plan: 1/ Add/Edit a subscription, select a subscription length different of the first value (issues) 2/ Edit the subscription and confirm the value is kept Bonus: issues, week and months become translatable. --- .../prog/en/modules/serials/subscription-add.tt | 30 +++++++++++++++++----- serials/subscription-add.pl | 18 ++++--------- 2 files changed, 29 insertions(+), 19 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt index 8401968..6bfe8d4 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt @@ -652,13 +652,31 @@ $(document).ready(function() {
  • diff --git a/serials/subscription-add.pl b/serials/subscription-add.pl index fa75f06..893a045 100755 --- a/serials/subscription-add.pl +++ b/serials/subscription-add.pl @@ -59,8 +59,6 @@ my ($template, $loggedinuser, $cookie) my $sub_on; -my @subscription_types = (qw(issues weeks months)); -my @sub_type_data; my $subs; our $firstissuedate; @@ -164,17 +162,11 @@ if ($op eq 'addsubscription') { } elsif ($op eq 'modsubscription') { redirect_mod_subscription(); } else { - while (@subscription_types) { - my $sub_type = shift @subscription_types; - my %row = ( 'name' => $sub_type ); - if ( defined $sub_on and $sub_on eq $sub_type ) { - $row{'selected'} = ' selected'; - } else { - $row{'selected'} = ''; - } - push( @sub_type_data, \%row ); - } - $template->param(subtype => \@sub_type_data); + + $template->param( + subtypes => [ qw( numberlength weeklength monthlength ) ], + subtype => $sub_on, + ); if ( $op ne 'modsubscription' && $op ne 'dup' && $op ne 'modify' ) { my $letters = get_letter_loop(); -- 2.1.0