Koha/serials/subscription-batchedit.pl
Pedro Amorim afe1d0a4c6
Bug 35044: (QA follow-up): Allow for '0' in serials batch edit
This commit also ensures any existing '0' value additional fields are
kept if batch editing and submitting the input empty

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
2024-08-23 18:21:27 +02:00

107 lines
3.8 KiB
Perl
Executable file

#!/usr/bin/perl
# Copyright 2017 BibLibre
#
# This file is part of Koha.
#
# Koha is free software; you can redistribute it and/or modify it
# under the terms of the GNU General Public License as published by
# the Free Software Foundation; either version 3 of the License, or
# (at your option) any later version.
#
# Koha is distributed in the hope that it will be useful, but
# WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with Koha; if not, see <http://www.gnu.org/licenses>.
use Modern::Perl;
use CGI qw( -utf8 );
use C4::Auth qw( get_template_and_user );
use C4::Output qw( output_html_with_http_headers );
use C4::Serials qw( can_edit_subscription );
use Koha::Subscriptions;
use Koha::Acquisition::Booksellers;
use Koha::AdditionalFields;
use Koha::DateUtils qw( dt_from_string );
my $cgi = CGI->new;
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
{
template_name => 'serials/subscription-batchedit.tt',
query => $cgi,
type => 'intranet',
flagsrequired => { serials => 'edit_subscription' },
}
);
my @subscriptionids = $cgi->multi_param('subscriptionid');
my @subscriptions;
foreach my $subscriptionid (@subscriptionids) {
my $subscription = Koha::Subscriptions->find($subscriptionid);
push @subscriptions, $subscription if $subscription;
}
my $available_additional_fields = Koha::AdditionalFields->search( { tablename => 'subscription' } );
my $op = $cgi->param('op') || q{};
if ( $op eq 'cud-batchedit' ) {
my %params = (
aqbooksellerid => scalar $cgi->param('booksellerid'),
location => scalar $cgi->param('location'),
branchcode => scalar $cgi->param('branchcode'),
itemtype => scalar $cgi->param('itemtype'),
notes => scalar $cgi->param('notes'),
internalnotes => scalar $cgi->param('internalnotes'),
serialsadditems => scalar $cgi->param('serialsadditems'),
);
# If we convert a blank string we get todays date, we should only convert enddate if it is not blank
$params{'enddate'} = dt_from_string( scalar $cgi->param('enddate') ) if $cgi->param('enddate');
my $field_values = {};
while ( my $available_field = $available_additional_fields->next ) {
my @submitted_fields = $cgi->param( 'additional_field_' . $available_field->id );
my @submitted_fields_array;
foreach my $submitted_field (@submitted_fields) {
push @submitted_fields_array, $submitted_field if defined $submitted_field && $submitted_field ne '';
}
$field_values->{ $available_field->id } = \@submitted_fields_array;
}
foreach my $subscription (@subscriptions) {
next
unless C4::Serials::can_edit_subscription( $subscription->unblessed )
; # This should be moved to Koha::Subscription->can_edit
while ( my ( $key, $value ) = each %params ) {
if ( defined $value and $value ne '' ) {
$subscription->$key($value);
}
}
$subscription->add_additional_fields($field_values, 'subscription');
$subscription->store;
}
my $redirect_url = $cgi->param('referrer') // '/cgi-bin/koha/serials/serials-home.pl';
print $cgi->redirect($redirect_url);
exit;
}
$template->param(
subscriptions => \@subscriptions,
booksellers => [ Koha::Acquisition::Booksellers->search->as_list ],
available_additional_fields => Koha::AdditionalFields->search( { tablename => 'subscription' } ),
referrer => scalar $cgi->param('referrer'),
);
output_html_with_http_headers $cgi, $cookie, $template->output;