For an issue, it was the biblionumber which was saved and not the itemnumber
This is quite not what we want to store
This patch fixes that.
---
 C4/Circulation.pm |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/C4/Circulation.pm b/C4/Circulation.pm
index d6dbbb3..3bb8522 100644
--- a/C4/Circulation.pm
+++ b/C4/Circulation.pm
@@ -1060,7 +1060,7 @@ sub AddIssue {
         }
     }
 
-    logaction("CIRCULATION", "ISSUE", $borrower->{'borrowernumber'}, 
$biblio->{'biblionumber'})
+    logaction( "CIRCULATION", "ISSUE", $borrower->{'borrowernumber'}, 
$item->{'itemnumber'}  )
         if C4::Context->preference("IssueLog");
   }
   return ($datedue);   # not necessarily the same as when it came in!
@@ -1565,10 +1565,10 @@ sub AddReturn {
             branch   => $branch,
         });
     }
-    
-    logaction("CIRCULATION", "RETURN", $borrowernumber, 
$item->{'biblionumber'})
-        if C4::Context->preference("ReturnLog");
-    
+
+    logaction( "CIRCULATION", "RETURN", $borrowernumber, $item->{'itemnumber'} 
)
+      if C4::Context->preference("ReturnLog");
+
     # FIXME: make this comment intelligible.
     #adding message if holdingbranch is non equal a userenv branch to return 
the document to homebranch
     #we check, if we don't have reserv or transfert for this document, if not, 
return it to homebranch .
-- 
1.6.3.3

_______________________________________________
Koha-patches mailing list
Koha-patches@lists.koha.org
http://lists.koha.org/mailman/listinfo/koha-patches

Reply via email to