Alex Buckley
db3ebe47e8
Removed the sql code from Itemtypes.pm and replaced it with DBIx database query in the itemtypes.pl administrative script Test plan: 1. In the staff interface, stage and manage MARC records for import 2. Try to delete an itemtype. If there are items of that itemtype in the database then a message telling you the number of items of that itemtype there are will be displayed. 3. Record that number 4. View the admin/itemtpes.pl script and confirm that there is sql code written in this file. 5. Apply this patch 6. View the admin/itemtypes.pl script and observe that there is no sql in this file. There is however DBIx code, for example $schema->resultset('Item')->search({ 'itype' => $itemtype_code} ); which is searching for items with the itype value matching $itemtype_code value. 7. In the staff interface try to delete the same itemtype 8. Record the number of items there are with that itemtype in the resulting message 9. The numbers recorded in steps 3 and 8 should match showing that the DBIx code is working as intended Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
174 lines
5.7 KiB
Perl
Executable file
174 lines
5.7 KiB
Perl
Executable file
#!/usr/bin/perl
|
|
|
|
# Copyright 2000-2002 Katipo Communications
|
|
# Copyright 2002 Paul Poulain
|
|
#
|
|
# 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>.
|
|
|
|
=head1 admin/itemtypes.pl
|
|
|
|
=cut
|
|
|
|
use Modern::Perl;
|
|
use CGI qw ( -utf8 );
|
|
|
|
use File::Spec;
|
|
|
|
use C4::Koha;
|
|
use C4::Context;
|
|
use C4::Auth;
|
|
use C4::Output;
|
|
use Koha::ItemTypes;
|
|
use Koha::ItemType;
|
|
use Koha::Localizations;
|
|
|
|
my $input = new CGI;
|
|
my $searchfield = $input->param('description');
|
|
my $itemtype_code = $input->param('itemtype');
|
|
my $op = $input->param('op') // 'list';
|
|
my @messages;
|
|
$searchfield =~ s/\,//g if $searchfield;
|
|
my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
|
|
{ template_name => "admin/itemtypes.tt",
|
|
query => $input,
|
|
type => "intranet",
|
|
authnotrequired => 0,
|
|
flagsrequired => { parameters => 'parameters_remaining_permissions' },
|
|
debug => 1,
|
|
}
|
|
);
|
|
|
|
my $dbh = C4::Context->dbh;
|
|
|
|
my $sip_media_type = $input->param('sip_media_type');
|
|
undef($sip_media_type) if defined($sip_media_type) and $sip_media_type =~ /^\s*$/;
|
|
|
|
if ( $op eq 'add_form' ) {
|
|
my $itemtype = Koha::ItemTypes->find($itemtype_code);
|
|
my $imagesets = C4::Koha::getImageSets( checked => ( $itemtype ? $itemtype->imageurl : undef ) );
|
|
my $searchcategory = GetAuthorisedValues("ITEMTYPECAT");
|
|
my $translated_languages = C4::Languages::getTranslatedLanguages( undef , C4::Context->preference('template') );
|
|
$template->param(
|
|
itemtype => $itemtype,
|
|
imagesets => $imagesets,
|
|
searchcategory => $searchcategory,
|
|
can_be_translated => ( scalar(@$translated_languages) > 1 ? 1 : 0 ),
|
|
);
|
|
} elsif ( $op eq 'add_validate' ) {
|
|
my $is_a_modif = $input->param('is_a_modif');
|
|
my $itemtype = Koha::ItemTypes->find($itemtype_code);
|
|
my $description = $input->param('description');
|
|
my $rentalcharge = $input->param('rentalcharge');
|
|
my $image = $input->param('image') || q||;
|
|
|
|
my $notforloan = $input->param('notforloan') ? 1 : 0;
|
|
my $imageurl =
|
|
$image eq 'removeImage' ? ''
|
|
: (
|
|
$image eq 'remoteImage' ? $input->param('remoteImage')
|
|
: $image
|
|
);
|
|
my $summary = $input->param('summary');
|
|
my $checkinmsg = $input->param('checkinmsg');
|
|
my $checkinmsgtype = $input->param('checkinmsgtype');
|
|
my $hideinopac = $input->param('hideinopac') // 0;
|
|
my $searchcategory = $input->param('searchcategory');
|
|
|
|
if ( $itemtype and $is_a_modif ) { # it's a modification
|
|
$itemtype->description($description);
|
|
$itemtype->rentalcharge($rentalcharge);
|
|
$itemtype->notforloan($notforloan);
|
|
$itemtype->imageurl($imageurl);
|
|
$itemtype->summary($summary);
|
|
$itemtype->checkinmsg($checkinmsg);
|
|
$itemtype->checkinmsgtype($checkinmsgtype);
|
|
$itemtype->sip_media_type($sip_media_type);
|
|
$itemtype->hideinopac($hideinopac);
|
|
$itemtype->searchcategory($searchcategory);
|
|
|
|
eval { $itemtype->store; };
|
|
|
|
if ($@) {
|
|
push @messages, { type => 'error', code => 'error_on_update' };
|
|
} else {
|
|
push @messages, { type => 'message', code => 'success_on_update' };
|
|
}
|
|
} elsif ( not $itemtype and not $is_a_modif ) {
|
|
my $itemtype = Koha::ItemType->new(
|
|
{ itemtype => $itemtype_code,
|
|
description => $description,
|
|
rentalcharge => $rentalcharge,
|
|
notforloan => $notforloan,
|
|
imageurl => $imageurl,
|
|
summary => $summary,
|
|
checkinmsg => $checkinmsg,
|
|
checkinmsgtype => $checkinmsgtype,
|
|
sip_media_type => $sip_media_type,
|
|
hideinopac => $hideinopac,
|
|
searchcategory => $searchcategory,
|
|
}
|
|
);
|
|
eval { $itemtype->store; };
|
|
|
|
if ($@) {
|
|
push @messages, { type => 'error', code => 'error_on_insert' };
|
|
} else {
|
|
push @messages, { type => 'message', code => 'success_on_insert' };
|
|
}
|
|
} else {
|
|
push @messages,
|
|
{ type => 'error',
|
|
code => 'already_exists',
|
|
};
|
|
}
|
|
|
|
$searchfield = '';
|
|
$op = 'list';
|
|
|
|
} elsif ( $op eq 'delete_confirm' ) {
|
|
my $ItemType = Koha::ItemTypes->find($itemtype_code);
|
|
my $overalltotal = $ItemType->can_be_deleted();
|
|
if ($overalltotal == 0) {
|
|
push @messages, { type => 'error', code => 'cannot_be_deleted'};
|
|
$op = 'list';
|
|
} else {
|
|
$template->param( itemtype => $ItemType, );
|
|
}
|
|
|
|
} elsif ( $op eq 'delete_confirmed' ) {
|
|
|
|
my $itemtype = Koha::ItemTypes->find($itemtype_code);
|
|
my $deleted = eval { $itemtype->delete };
|
|
if ( $@ or not $deleted ) {
|
|
push @messages, { type => 'error', code => 'error_on_delete' };
|
|
} else {
|
|
push @messages, { type => 'message', code => 'success_on_delete' };
|
|
}
|
|
|
|
$op = 'list';
|
|
}
|
|
|
|
if ( $op eq 'list' ) {
|
|
my $itemtypes = Koha::ItemTypes->search;
|
|
$template->param(
|
|
itemtypes => $itemtypes,
|
|
messages => \@messages,
|
|
);
|
|
}
|
|
|
|
$template->param( op => $op );
|
|
|
|
output_html_with_http_headers $input, $cookie, $template->output;
|