Attachment 68293 Details for Bug 19487 – Bug 19487: Do not return an item if not checked out

[patch] Bug 19487: Do not return an item if not checked out

Bug-19487-Do-not-return-an-item-if-not-checked-out.patch (text/plain), 1.90 KB, created by Nick Clemens (kidclamp) on 2017-10-19 10:58:23 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2017-10-19 10:58:23 UTC

Size: 1.90 KB

patch

obsolete

>From 85739cdab79c5ead60f47a162180ad63bd505dbb Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Wed, 18 Oct 2017 13:59:56 -0300
>Subject: [PATCH] Bug 19487: Do not return an item if not checked out
>
>To recreate:
>1 - Manually add a lost fine to a ptron and include a barcode
>2 - Attempt to write off the fine
>3 - Internal server error
>4 - Checkout an item and mark lost to checkin and fine
>5 - Attempt to write off line
>6 - Internal server error
>
>Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
>---
> C4/Circulation.pm | 13 +++++--------
> 1 file changed, 5 insertions(+), 8 deletions(-)
>
>diff --git a/C4/Circulation.pm b/C4/Circulation.pm
>index b915b34..18d9a44 100644
>--- a/C4/Circulation.pm
>+++ b/C4/Circulation.pm
>@@ -2129,6 +2129,11 @@ routine in C<C4::Accounts>.
> sub MarkIssueReturned {
>     my ( $borrowernumber, $itemnumber, $dropbox_branch, $returndate, $privacy ) = @_;
> 
>+
>+    # Retrieve the issue
>+    my $issue = Koha::Checkouts->find( { itemnumber => $itemnumber } ) or return;
>+    my $issue_id = $issue->issue_id;
>+
>     my $anonymouspatron;
>     if ( $privacy == 2 ) {
>         # The default of 0 will not work due to foreign key constraints
>@@ -2143,11 +2148,6 @@ sub MarkIssueReturned {
>     my $schema   = $database->schema;
>     my $dbh   = C4::Context->dbh;
> 
>-    my $issue_id = $dbh->selectrow_array(
>-        q|SELECT issue_id FROM issues WHERE itemnumber = ?|,
>-        undef, $itemnumber
>-    );
>-
>     my $query = 'UPDATE issues SET returndate=';
>     my @bind;
>     if ($dropbox_branch) {
>@@ -2170,9 +2170,6 @@ sub MarkIssueReturned {
>         # Update the returndate
>         $dbh->do( $query, undef, @bind );
> 
>-        # Retrieve the issue
>-        my $issue = Koha::Checkouts->find( $issue_id ); # FIXME should be fetched earlier
>-
>         # Create the old_issues entry
>         my $old_checkout = Koha::Old::Checkout->new($issue->unblessed)->store;
> 
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 19487: 68275 | 68293 | 68294 | 68312 | 68313 | 68314 | 68330