Attachment 135070 Details for Bug 28152 – Bug 28152: Log the "duplicate item barcode" error

[patch] Bug 28152: Log the "duplicate item barcode" error

Bug-28152-Log-the-duplicate-item-barcode-error.patch (text/plain), 3.21 KB, created by Jonathan Druart on 2022-05-17 12:37:46 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2022-05-17 12:37:46 UTC

Size: 3.21 KB

patch

obsolete

>From eb0044dbc18c3344a573365cc610534b6acaaec5 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Mon, 10 Jan 2022 11:22:54 +0100
>Subject: [PATCH] Bug 28152: Log the "duplicate item barcode" error
>
>Looking at the code this 'import_error' column is empty for the last 9 years. Not sure it makes much sense to have this single error now.
>
>commit 1dba9c6409d78cb1f90de6c1300cb5b63fb1b851
>Date:   Wed Oct 10 14:21:22 2012 -0500
>    Bug 7131: teach MARC import how to overlay items
>---
> C4/ImportBatch.pm | 14 +++++++++-----
> 1 file changed, 9 insertions(+), 5 deletions(-)
>
>diff --git a/C4/ImportBatch.pm b/C4/ImportBatch.pm
>index e17c1a07e29..6a8fc526f23 100644
>--- a/C4/ImportBatch.pm
>+++ b/C4/ImportBatch.pm
>@@ -770,13 +770,14 @@ sub BatchCommitItems {
>         my $duplicate_barcode = exists( $item->{'barcode'} ) && Koha::Items->find({ barcode => $item->{'barcode'} });
>         my $duplicate_itemnumber = exists( $item->{'itemnumber'} );
> 
>-        my $updsth = $dbh->prepare("UPDATE import_items SET status = ?, itemnumber = ? WHERE import_items_id = ?");
>+        my $updsth = $dbh->prepare("UPDATE import_items SET status = ?, itemnumber = ?, import_error = ? WHERE import_items_id = ?");
>         if ( $action eq "replace" && $duplicate_itemnumber ) {
>             # Duplicate itemnumbers have precedence, that way we can update barcodes by overlaying
>             ModItemFromMarc( $item_marc, $biblionumber, $item->{itemnumber} );
>             $updsth->bind_param( 1, 'imported' );
>             $updsth->bind_param( 2, $item->{itemnumber} );
>-            $updsth->bind_param( 3, $row->{'import_items_id'} );
>+            $updsth->bind_param( 3, undef );
>+            $updsth->bind_param( 4, $row->{'import_items_id'} );
>             $updsth->execute();
>             $updsth->finish();
>             $num_items_replaced++;
>@@ -785,14 +786,16 @@ sub BatchCommitItems {
>             ModItemFromMarc( $item_marc, $biblionumber, $itemnumber );
>             $updsth->bind_param( 1, 'imported' );
>             $updsth->bind_param( 2, $item->{itemnumber} );
>-            $updsth->bind_param( 3, $row->{'import_items_id'} );
>+            $updsth->bind_param( 3, undef );
>+            $updsth->bind_param( 4, $row->{'import_items_id'} );
>             $updsth->execute();
>             $updsth->finish();
>             $num_items_replaced++;
>         } elsif ($duplicate_barcode) {
>             $updsth->bind_param( 1, 'error' );
>             $updsth->bind_param( 2, undef );
>-            $updsth->bind_param( 3, $row->{'import_items_id'} );
>+            $updsth->bind_param( 3, 'duplicate item barcode' );
>+            $updsth->bind_param( 4, $row->{'import_items_id'} );
>             $updsth->execute();
>             $num_items_errored++;
>         } else {
>@@ -804,7 +807,8 @@ sub BatchCommitItems {
>             if( $itemnumber ) {
>                 $updsth->bind_param( 1, 'imported' );
>                 $updsth->bind_param( 2, $itemnumber );
>-                $updsth->bind_param( 3, $row->{'import_items_id'} );
>+                $updsth->bind_param( 3, undef );
>+                $updsth->bind_param( 4, $row->{'import_items_id'} );
>                 $updsth->execute();
>                 $updsth->finish();
>                 $num_items_added++;
>-- 
>2.25.1

Actions: View | Diff | Splinter Review

Attachments on bug 28152: 119609 | 129243 | 135070 | 135253 | 135254 | 135267 | 135268 | 135294 | 135295