Attachment 129243 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), 2.74 KB, created by Jonathan Druart on 2022-01-10 10:58:32 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2022-01-10 10:58:32 UTC

Size: 2.74 KB

patch

obsolete

>From 49e047da5c625b0ca28b94af50a56b245ea085f2 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 | 11 +++++++----
> 1 file changed, 7 insertions(+), 4 deletions(-)
>
>diff --git a/C4/ImportBatch.pm b/C4/ImportBatch.pm
>index 99fadf875ce..39fbb3a656c 100644
>--- a/C4/ImportBatch.pm
>+++ b/C4/ImportBatch.pm
>@@ -769,13 +769,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++;
>@@ -784,14 +785,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 {
>-- 
>2.25.1

Actions: View | Diff | Splinter Review

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