From dddf50a274e1fe21d3626b9286053f552c1f1df1 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 --- .../prog/en/modules/serials/subscription-add.tt | 4 ++-- serials/subscription-add.pl | 18 +++++++++++++++--- 2 files changed, 17 insertions(+), 5 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 39c1a0b05f..960b258884 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 @@ -436,9 +436,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 1c10528777..196c789cc0 100755 --- a/serials/subscription-add.pl +++ b/serials/subscription-add.pl @@ -64,9 +64,9 @@ my $sub_on; my $subs; our $firstissuedate; -if ($op eq 'modify' || $op eq 'dup' || $op eq 'modsubscription') { +my $subscriptionid = $query->param('subscriptionid'); - my $subscriptionid = $query->param('subscriptionid'); +if ($op eq 'modify' || $op eq 'dup' || $op eq 'modsubscription') { $subs = GetSubscription($subscriptionid); ## FIXME : Check rights to edit if mod. Could/Should display an error message. @@ -134,7 +134,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.14.2