@@ -, +, @@ --- acqui/basket.pl | 5 ++--- acqui/basketheader.pl | 19 +++++++++---------- .../prog/en/modules/acqui/basketheader.tt | 1 + 3 files changed, 12 insertions(+), 13 deletions(-) --- a/acqui/basket.pl +++ a/acqui/basket.pl @@ -426,9 +426,8 @@ if ( $op eq 'list' ) { duplinbatch => $duplinbatch, csv_profiles => Koha::CsvProfiles->search({ type => 'sql', used_for => 'export_basket' }), available_additional_fields => Koha::AdditionalFields->search( { tablename => 'aqbasket' } ), - additional_field_values => { map { - $_->field->name => $_->value - } Koha::Acquisition::Baskets->find($basketno)->additional_field_values->as_list }, + additional_field_values => + Koha::Acquisition::Baskets->find($basketno)->get_additional_field_values_for_template, ); } --- a/acqui/basketheader.pl +++ a/acqui/basketheader.pl @@ -96,11 +96,8 @@ if ( $op eq 'add_form' ) { } } $template->param( is_an_edit => 1); - $template->param( - additional_field_values => { map { - $_->field->id => $_->value - } Koha::Acquisition::Baskets->find($basketno)->additional_field_values->as_list }, - ); + $template->param( additional_field_values => + Koha::Acquisition::Baskets->find($basketno)->get_additional_field_values_for_template ); } else { #new basket my $basket; @@ -168,11 +165,13 @@ if ( $op eq 'add_form' ) { my @additional_fields; my $basket_fields = Koha::AdditionalFields->search({ tablename => 'aqbasket' }); while ( my $field = $basket_fields->next ) { - my $value = $input->param('additional_field_' . $field->id); - push @additional_fields, { - id => $field->id, - value => $value, - }; + my @field_values = $input->param( 'additional_field_' . $field->id ); + foreach my $value (@field_values) { + push @additional_fields, { + id => $field->id, + value => $value, + } if $value; + } } Koha::Acquisition::Baskets->find($basketno)->set_additional_fields(\@additional_fields); --- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketheader.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketheader.tt @@ -162,6 +162,7 @@ [% MACRO jsinclude BLOCK %] [% Asset.js("js/acquisitions-menu.js") | $raw %] [% Asset.js("js/acq.js") | $raw %] + [% Asset.js("js/additional-fields-entry.js") | $raw %] [% INCLUDE 'select2.inc' %]