Koha/admin/edi_ean_accounts.pl
Owen Leonard 900a1aecbb
Bug 36995: Correct parameter name for library EAN deletion
This patches removes the "cud-" prefix from the "delete_confirm"
parameter check in Library EAN management. The confirm step is a GET
operation.

To test, apply the patch and restart services;

- Go to Administration -> Library EANs.
- Add an entry if necessary, then click "Delete."
- You should be taken to a confirmation page: "Confirm deletion of
  EAN..."
- Confirm deletion and verify that the EAN was deleted.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
2024-06-06 15:18:44 +01:00

124 lines
3.7 KiB
Perl
Executable file

#!/usr/bin/perl
# Copyright 2012, 2014 Mark Gavillet & PTFS Europe Ltd
#
# 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;
use C4::Auth qw( get_template_and_user );
use C4::Output qw( output_html_with_http_headers );
use Koha::Database;
my $input = CGI->new();
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
{
template_name => 'admin/edi_ean_accounts.tt',
query => $input,
type => 'intranet',
flagsrequired => { acquisition => 'edi_manage' },
}
);
my $schema = Koha::Database->new()->schema();
my $id = scalar $input->param('id');
my $op = scalar $input->param('op') || 'display';
if ( $op eq 'ean_form' ) {
my $e = $schema->resultset('EdifactEan')->find($id);
my @branches = $schema->resultset('Branch')->search(
undef,
{
columns => [ 'branchcode', 'branchname' ],
order_by => 'branchname',
}
);
$template->param(
ean_form => 1,
branches => \@branches,
ean => $e,
);
}
elsif ( $op eq 'delete_confirm' ) {
my $e = $schema->resultset('EdifactEan')->find($id);
$template->param(
delete_confirm => 1,
ean => $e,
);
}
else {
if ( $op eq 'cud-save' ) {
my $change = $id;
if ($change) {
$schema->resultset('EdifactEan')->find($id)->update(
{
branchcode => scalar $input->param('branchcode') || undef,
description => scalar $input->param('description'),
ean => scalar $input->param('ean'),
id_code_qualifier => scalar $input->param('id_code_qualifier'),
}
);
}
else {
my $new_ean = $schema->resultset('EdifactEan')->new(
{
branchcode => scalar $input->param('branchcode') || undef,
description => scalar $input->param('description'),
ean => scalar $input->param('ean'),
id_code_qualifier => scalar $input->param('id_code_qualifier'),
}
);
$new_ean->insert();
}
}
elsif ( $op eq 'cud-delete_confirmed' ) {
my $e = $schema->resultset('EdifactEan')->find($id);
$e->delete if $e;
}
my @eans = $schema->resultset('EdifactEan')->search(
{},
{
join => 'branch',
}
);
$template->param( display => 1 );
$template->param( eans => \@eans );
}
$template->param(
code_qualifiers => [
{
code => '14',
description => 'EAN International',
},
{
code => '31B',
description => 'US SAN Agency',
},
{
code => '91',
description => 'Assigned by supplier',
},
{
code => '92',
description => 'Assigned by buyer',
},
]
);
output_html_with_http_headers( $input, $cookie, $template->output );