Bugzilla – Attachment 41435 Details for
Bug 10005
inventory checks items in without warning
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[SIGNED OFF] Bug 10005: Inventory - Add an option to not check in items
SIGNED-OFF-Bug-10005-Inventory---Add-an-option-to-.patch (text/plain), 3.51 KB, created by
Jason Robb
on 2015-08-08 14:34:54 UTC
(
hide
)
Description:
[SIGNED OFF] Bug 10005: Inventory - Add an option to not check in items
Filename:
MIME Type:
Creator:
Jason Robb
Created:
2015-08-08 14:34:54 UTC
Size:
3.51 KB
patch
obsolete
>From e36a753ad859bd9b70d85f13875414bbbf3c6c17 Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Tue, 14 Jul 2015 15:02:42 +0100 >Subject: [PATCH] [SIGNED OFF] Bug 10005: Inventory - Add an option to not > check in items > >The inventory tools automatically check in items, this patch adds it as >an option. > >Test plan: >1/ Check an item out, fill a file with its barcode, and use this file in >the inventory tools. >2/ Check the new checkbox and confirm that the item is not checked in >3/ Repeat again and don't check it, the behavior should be the same as before >this patch. > >Signed-off-by: Jason Robb <jrobb@sekls.org> >--- > .../prog/en/modules/tools/inventory.tt | 3 +++ > tools/inventory.pl | 19 +++++++++++-------- > 2 files changed, 14 insertions(+), 8 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt >index fdac5fa..fb0fc2b 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt >@@ -214,6 +214,9 @@ $(document).ready(function(){ > <label for="compareinv2barcd">Compare barcodes list to results: </label> > <input type="checkbox" name="compareinv2barcd" id="compareinv2barcd" /> > </li> >+ <li> >+ <label for="dont_checkin">Do not check in items scanned during inventory: </label> >+ <input type="checkbox" name="dont_checkin" id="dont_checkin" /> > </ol> > </fieldset> > <input type="hidden" name="op" value="do_it" /> >diff --git a/tools/inventory.pl b/tools/inventory.pl >index a4bf639..2192dbd 100755 >--- a/tools/inventory.pl >+++ b/tools/inventory.pl >@@ -52,6 +52,7 @@ my $branchcode = $input->param('branchcode') || ''; > my $branch = $input->param('branch'); > my $op = $input->param('op'); > my $compareinv2barcd = $input->param('compareinv2barcd'); >+my $dont_checkin = $input->param('dont_checkin'); > > my ( $template, $borrowernumber, $cookie ) = get_template_and_user( > { template_name => "tools/inventory.tt", >@@ -209,14 +210,16 @@ if ( $uploadbarcodes && length($uploadbarcodes) > 0 ) { > ModItem( { datelastseen => $date }, undef, $item->{'itemnumber'} ); > push @scanned_items, $item; > $count++; >- $qonloan->execute($barcode); >- if ($qonloan->rows){ >- my $data = $qonloan->fetchrow_hashref; >- my ($doreturn, $messages, $iteminformation, $borrower) =AddReturn($barcode, $data->{homebranch}); >- if ($doreturn){ >- push @errorloop, {'barcode'=>$barcode,'ERR_ONLOAN_RET'=>1} >- } else { >- push @errorloop, {'barcode'=>$barcode,'ERR_ONLOAN_NOT_RET'=>1} >+ unless ( $dont_checkin ) { >+ $qonloan->execute($barcode); >+ if ($qonloan->rows){ >+ my $data = $qonloan->fetchrow_hashref; >+ my ($doreturn, $messages, $iteminformation, $borrower) =AddReturn($barcode, $data->{homebranch}); >+ if ($doreturn){ >+ push @errorloop, {'barcode'=>$barcode,'ERR_ONLOAN_RET'=>1} >+ } else { >+ push @errorloop, {'barcode'=>$barcode,'ERR_ONLOAN_NOT_RET'=>1} >+ } > } > } > } else { >-- >1.7.10.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 10005
:
40990
|
41435
|
42443