Bugzilla – Attachment 87237 Details for
Bug 10345
Copy number should be incremented when adding multiple items at once
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 10345: Increment copy number when adding multiple copies.
Bug-10345-Increment-copy-number-when-adding-multip.patch (text/plain), 2.38 KB, created by
Pierre-Marc Thibault
on 2019-03-29 20:55:07 UTC
(
hide
)
Description:
Bug 10345: Increment copy number when adding multiple copies.
Filename:
MIME Type:
Creator:
Pierre-Marc Thibault
Created:
2019-03-29 20:55:07 UTC
Size:
2.38 KB
patch
obsolete
>From 2cf9ab51fae74fc930ecf004b82ef5c39c93d77a Mon Sep 17 00:00:00 2001 >From: =?UTF-8?q?Bj=C3=B6rn=20Nyl=C3=A9n?= <bjorn.nylen@ub.lu.se> >Date: Thu, 28 Mar 2019 10:28:29 +0100 >Subject: [PATCH] Bug 10345: Increment copy number when adding multiple copies. > >To test: >1. Add multiple copies of an item with data in the 'Copy number' field. Note that tha data will be identical for all items. >2. Apply patch. >3. Add multiple copies of an item with a positive integer (ie. only digits) in the 'Copy number' field. Note that the copy number is incremented for each item. >4. Add multiple copies of an item with some other type of data in the 'Copy number' field. Note that the copy number field remains unchanged for the added items. > >Signed-off-by: Pierre-Marc Thibault <pierre-marc.thibault@inLibro.com> >--- > cataloguing/additem.pl | 8 ++++++++ > 1 file changed, 8 insertions(+) > >diff --git a/cataloguing/additem.pl b/cataloguing/additem.pl >index 7b6d5e4..48417aa 100755 >--- a/cataloguing/additem.pl >+++ b/cataloguing/additem.pl >@@ -536,8 +536,10 @@ if ($op eq "additem") { > > use C4::Barcodes; > my $barcodeobj = C4::Barcodes->new; >+ my $copynumber = $addedolditem->{'copynumber'}; > my $oldbarcode = $addedolditem->{'barcode'}; > my ($tagfield,$tagsubfield) = &GetMarcFromKohaField("items.barcode",$frameworkcode); >+ my ($copytagfield,$copytagsubfield) = &GetMarcFromKohaField("items.copynumber",$frameworkcode); > > # If there is a barcode and we can't find their new values, we can't add multiple copies > my $testbarcode; >@@ -571,6 +573,10 @@ if ($op eq "additem") { > # Checking if the barcode already exists > $exist_itemnumber = get_item_from_barcode($barcodevalue); > } >+ # Updating record with the new copynumber >+ if ( $copynumber ){ >+ $record->field($copytagfield)->update($copytagsubfield => $copynumber); >+ } > > # Adding the item > if (!$exist_itemnumber) { >@@ -581,6 +587,8 @@ if ($op eq "additem") { > # That way, all items are added, even if there was some already existing barcodes > # FIXME : Please note that there is a risk of infinite loop here if we never find a suitable barcode > $i++; >+ # Only increment copynumber if item was really added >+ $copynumber++ if ( $copynumber && $copynumber =~ m/^\d+$/ ); > } > > # Preparing the next iteration >-- >2.7.4
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 10345
:
87153
|
87237
|
87353