Bugzilla – Attachment 159333 Details for
Bug 33171
Add coded_location_qualifier, barcode, and enumchron to MarcItemFieldsToOrder
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 33171: Fix missing price when mapped from MarcItemFieldsToOrder
Bug-33171-Fix-missing-price-when-mapped-from-MarcI.patch (text/plain), 2.16 KB, created by
Matt Blenkinsop
on 2023-11-29 11:34:17 UTC
(
hide
)
Description:
Bug 33171: Fix missing price when mapped from MarcItemFieldsToOrder
Filename:
MIME Type:
Creator:
Matt Blenkinsop
Created:
2023-11-29 11:34:17 UTC
Size:
2.16 KB
patch
obsolete
>From 80c6ebbeef14df12e5f809292783695bef51e560 Mon Sep 17 00:00:00 2001 >From: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> >Date: Wed, 29 Nov 2023 11:11:37 +0000 >Subject: [PATCH] Bug 33171: Fix missing price when mapped from > MarcItemFieldsToOrder > >This patch fixes an issue where the price is not passed to the UI when read from MarcItemFieldsToOrder. > >Test plan: >1) In Administration, copy and paste the following into MarcItemFieldsToOrder: >quantity: 975$q >budget_code: 975$h >price: 975$p >itype: 975$y >coded_location_qualifier: 975$a >barcode: 975$b >enumchron: 975$c >2) Save >3) Navigate to Acquisitions and find a basket >4) Click Add to basket and select From a new file >5) Choose the file attached to this bug >6) Upload file and click Stage for import >7) Click Add staged files to basket once the job is finished >8) Check the box next to the record >9) The price should show 10.00 under Item record 1 >10) coded location qualifier, barcode, enumchron should also have values as per the other commit on this bug >11) Save and import, the basket should not have a value of 0 as the price has been read correctly >12) Check the database, the newest item should have values for price, coded_location_qualifier, barcode and enumchron > >Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> >--- > acqui/addorderiso2709.pl | 2 ++ > 1 file changed, 2 insertions(+) > >diff --git a/acqui/addorderiso2709.pl b/acqui/addorderiso2709.pl >index 8810c7990e..518a924069 100755 >--- a/acqui/addorderiso2709.pl >+++ b/acqui/addorderiso2709.pl >@@ -535,6 +535,8 @@ sub import_biblios_list { > # Handle incorrectly named original parameters for MarcItemFieldsToOrder > $iteminfos->{location} = delete $iteminfos->{loc} if $iteminfos->{loc}; > $iteminfos->{copynumber} = delete $iteminfos->{copyno} if $iteminfos->{copyno}; >+ # Price is handled as "itemprice" in the UI form to distinguish from MarcFieldsToOrder >+ $iteminfos->{itemprice} = delete $iteminfos->{price} if $iteminfos->{price}; > > # Convert budge code to a budget id > my $item_budget_code = delete $iteminfos->{budget_code}; >-- >2.37.1 (Apple Git-137.1)
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 33171
:
147964
|
147966
|
153294
|
159331
|
159332
|
159333
|
159429
|
159430
|
160458
|
160459