Jonathan Druart
759790c97a
Due to how moredetail.pl was written it was hard to know which action
was triggered.
Test plan:
- Set MarkLostItemsAsReturned to "from the items tab of the catalog
module."
- Check an item out to a patron
- Use additem.pl to set that item to a lost status without returning it
- go to moredetail.pl for that item
- mark item as either damaged or withdrawn without interacting with the
lost dropdown
- note that item has not been returned
Note for QA: Maybe we could have just tested if exists $item_changes->{'itemlost'}
to call LostItem
Signed-off-by: Claudie Trégouët <claudie.tregouet@biblibre.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
86 lines
3.1 KiB
Perl
Executable file
86 lines
3.1 KiB
Perl
Executable file
#!/usr/bin/perl
|
|
|
|
# $Id: updateitem.pl,v 1.9.2.1.2.4 2006/10/05 18:36:50 kados Exp $
|
|
# Copyright 2006 LibLime
|
|
#
|
|
# 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;
|
|
use C4::Context;
|
|
use C4::Biblio;
|
|
use C4::Items;
|
|
use C4::Output;
|
|
use C4::Circulation;
|
|
use C4::Reserves;
|
|
|
|
my $cgi= new CGI;
|
|
|
|
checkauth($cgi, 0, {circulate => 'circulate_remaining_permissions'}, 'intranet');
|
|
|
|
my $op = $cgi->param('op') || "";
|
|
my $biblionumber=$cgi->param('biblionumber');
|
|
my $itemnumber=$cgi->param('itemnumber');
|
|
my $biblioitemnumber=$cgi->param('biblioitemnumber');
|
|
my $itemlost=$cgi->param('itemlost');
|
|
my $itemnotes=$cgi->param('itemnotes');
|
|
my $itemnotes_nonpublic=$cgi->param('itemnotes_nonpublic');
|
|
my $withdrawn=$cgi->param('withdrawn');
|
|
my $damaged=$cgi->param('damaged');
|
|
|
|
my $confirm=$cgi->param('confirm');
|
|
my $dbh = C4::Context->dbh;
|
|
|
|
# get the rest of this item's information
|
|
my $item_data_hashref = Koha::Items->find($itemnumber)->unblessed;
|
|
|
|
# make sure item statuses are set to 0 if empty or NULL
|
|
for ($damaged,$itemlost,$withdrawn) {
|
|
if (!$_ or $_ eq "") {
|
|
$_ = 0;
|
|
}
|
|
}
|
|
|
|
# modify MARC item if input differs from items table.
|
|
my $item_changes = {};
|
|
if ( $op eq "set_non_public_note" ) {
|
|
checkauth($cgi, 0, {editcatalogue => 'edit_items'}, 'intranet');
|
|
if ((not defined $item_data_hashref->{'itemnotes_nonpublic'}) or $itemnotes_nonpublic ne $item_data_hashref->{'itemnotes_nonpublic'}) {
|
|
$item_changes->{'itemnotes_nonpublic'} = $itemnotes_nonpublic;
|
|
}
|
|
}
|
|
elsif ( $op eq "set_public_note" ) { # i.e., itemnotes parameter passed from form
|
|
checkauth($cgi, 0, {editcatalogue => 'edit_items'}, 'intranet');
|
|
if ((not defined $item_data_hashref->{'itemnotes'}) or $itemnotes ne $item_data_hashref->{'itemnotes'}) {
|
|
$item_changes->{'itemnotes'} = $itemnotes;
|
|
}
|
|
} elsif ( $op eq "set_lost" && $itemlost ne $item_data_hashref->{'itemlost'}) {
|
|
$item_changes->{'itemlost'} = $itemlost;
|
|
} elsif ( $op eq "set_withdrawn" && $withdrawn ne $item_data_hashref->{'withdrawn'}) {
|
|
$item_changes->{'withdrawn'} = $withdrawn;
|
|
} elsif ( $op eq "set_damaged" && $damaged ne $item_data_hashref->{'damaged'}) {
|
|
$item_changes->{'damaged'} = $damaged;
|
|
} else {
|
|
#nothings changed, so do nothing.
|
|
print $cgi->redirect("moredetail.pl?biblionumber=$biblionumber&itemnumber=$itemnumber#item$itemnumber");
|
|
exit;
|
|
}
|
|
|
|
ModItem($item_changes, $biblionumber, $itemnumber);
|
|
|
|
LostItem($itemnumber, 'moredetail') if $op eq "set_lost";
|
|
|
|
print $cgi->redirect("moredetail.pl?biblionumber=$biblionumber&itemnumber=$itemnumber#item$itemnumber");
|