From 4eac253d812b875422e29d6cecdbf57acecf3c81 Mon Sep 17 00:00:00 2001 From: Julian Maurice Date: Wed, 16 May 2018 09:30:42 +0200 Subject: [PATCH] Bug 15774: Show additional field values in edit form MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Séverine QUEUNE --- acqui/basketheader.pl | 2 +- .../prog/en/modules/serials/subscription-add.tt | 4 ++-- serials/subscription-add.pl | 17 +++++++++++++++-- 3 files changed, 18 insertions(+), 5 deletions(-) diff --git a/acqui/basketheader.pl b/acqui/basketheader.pl index da2129f..7b5b160 100755 --- a/acqui/basketheader.pl +++ b/acqui/basketheader.pl @@ -100,7 +100,7 @@ if ( $op eq 'add_form' ) { $template->param( is_an_edit => 1); $template->param( additional_field_values => { map { - $_->field->name => $_->value + $_->field->id => $_->value } Koha::Acquisition::Baskets->find($basketno)->additional_field_values }, ); } else { 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 5248f72..32d8252 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 @@ -444,9 +444,9 @@ fieldset.rows li.radio { width: 100%; } /* override staff-global.css */ - [% IF additional_fields_for_subscription %] + [% IF additional_fields %]
- [% INCLUDE 'additional-fields-entry.inc' available=additional_fields_for_subscription values=additional_fields %] + [% INCLUDE 'additional-fields-entry.inc' available=additional_fields values=additional_field_values %]
[% END %] diff --git a/serials/subscription-add.pl b/serials/subscription-add.pl index d9e9327..3a8534b 100755 --- a/serials/subscription-add.pl +++ b/serials/subscription-add.pl @@ -64,9 +64,10 @@ my $sub_on; my $subs; our $firstissuedate; +my $subscriptionid = $query->param('subscriptionid'); + if ($op eq 'modify' || $op eq 'dup' || $op eq 'modsubscription') { - my $subscriptionid = $query->param('subscriptionid'); $subs = GetSubscription($subscriptionid); output_and_exit( $query, $cookie, $template, 'unknown_subscription') @@ -137,7 +138,19 @@ $template->param( locations_loop=>$locations_loop, ); -$template->param( additional_fields_for_subscription => [ Koha::AdditionalFields->search( { tablename => 'subscription' } ) ] ); +my @additional_fields = Koha::AdditionalFields->search({ tablename => 'subscription' }); +my %additional_field_values; +if ($subscriptionid) { + my $subscription = Koha::Subscriptions->find($subscriptionid); + foreach my $value ($subscription->additional_field_values) { + $additional_field_values{$value->field_id} = $value->value; + } +} + +$template->param( + additional_fields => \@additional_fields, + additional_field_values => \%additional_field_values, +); my $typeloop = { map { $_->{itemtype} => $_ } @{ Koha::ItemTypes->search_with_localization->unblessed } }; -- 2.1.4