Bugzilla – Attachment 17209 Details for
Bug 7131
way to overlay items in in marc import
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 7131 - Followup 2 - Allow overlaying by barcode
Bug-7131---Followup-2---Allow-overlaying-by-barcod.patch (text/plain), 2.96 KB, created by
Kyle M Hall
on 2013-04-05 11:55:58 UTC
(
hide
)
Description:
Bug 7131 - Followup 2 - Allow overlaying by barcode
Filename:
MIME Type:
Creator:
Kyle M Hall
Created:
2013-04-05 11:55:58 UTC
Size:
2.96 KB
patch
obsolete
>From 7d4b35bb4b5b03c273779c18153b411fb84d627b Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Fri, 5 Apr 2013 07:53:47 -0400 >Subject: [PATCH] Bug 7131 - Followup 2 - Allow overlaying by barcode > >This patch adds the ability to overlay by either itemnumber, >or barcode. Itemnumbers take precendence over barcodes, which >allows us to batch update item barcodes with an overlay. >--- > C4/ImportBatch.pm | 17 ++++++++++++++--- > 1 files changed, 14 insertions(+), 3 deletions(-) > >diff --git a/C4/ImportBatch.pm b/C4/ImportBatch.pm >index 3f1ceab..f127d7a 100644 >--- a/C4/ImportBatch.pm >+++ b/C4/ImportBatch.pm >@@ -687,19 +687,20 @@ sub BatchCommitItems { > my $dbh = C4::Context->dbh; > > my ( $num_items_added, $num_items_errored, $num_items_replaced ) = 0; >- my $sth = $dbh->prepare(" >+ my $sth = $dbh->prepare( " > SELECT import_items_id, import_items.marcxml, encoding > FROM import_items > JOIN import_records USING (import_record_id) > WHERE import_record_id = ? > ORDER BY import_items_id >- "); >+ " ); > $sth->bind_param( 1, $import_record_id ); > $sth->execute(); >+ > while ( my $row = $sth->fetchrow_hashref() ) { > my $item_marc = MARC::Record->new_from_xml( StripNonXmlChars( $row->{'marcxml'} ), 'UTF-8', $row->{'encoding'} ); > >- #delete date_due subfield as to not accidentally delete item checkout due dates >+ # Delete date_due subfield as to not accidentally delete item checkout due dates > my ( $MARCfield, $MARCsubfield ) = GetMarcFromKohaField( 'items.onloan', GetFrameworkCode($biblionumber) ); > $item_marc->field($MARCfield)->delete_subfield( code => $MARCsubfield ); > >@@ -710,6 +711,7 @@ sub BatchCommitItems { > > my $updsth = $dbh->prepare("UPDATE import_items SET status = ?, itemnumber = ? 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} ); >@@ -717,6 +719,15 @@ sub BatchCommitItems { > $updsth->execute(); > $updsth->finish(); > $num_items_replaced++; >+ } elsif ( $action eq "replace" && $duplicate_barcode ) { >+ my $itemnumber = GetItemnumberFromBarcode( $item->{'barcode'} ); >+ 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->execute(); >+ $updsth->finish(); >+ $num_items_replaced++; > } elsif ($duplicate_barcode) { > $updsth->bind_param( 1, 'error' ); > $updsth->bind_param( 2, 'duplicate item barcode' ); >-- >1.7.2.5
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 7131
:
14344
|
14475
|
15954
|
15957
|
16887
|
16889
|
17208
|
17209
|
17210
|
17211
|
17212
|
18708
|
18709
|
18710
|
18711
|
18967
|
18968
|
18969
|
20455
|
21797