Bug 35913 - Item order prices do not fall back to MarcFieldsToOrder if not set by MarcItemFieldsToOrder
Summary: Item order prices do not fall back to MarcFieldsToOrder if not set by MarcIte...
Status: Pushed to stable
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: Main
Hardware: All All
: P5 - low major (vote)
Assignee: Nick Clemens (kidclamp)
QA Contact: Victor Grousset/tuxayo
URL:
Keywords:
Depends on: 33170
Blocks:
  Show dependency treegraph
 
Reported: 2024-01-25 19:56 UTC by Nick Clemens (kidclamp)
Modified: 2024-04-12 11:35 UTC (History)
6 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
24.05.00,23.11.04


Attachments
Bug 35913 - Restore price fall back to MarcFieldsToOrder if not set by MarcItemFieldsToOrder (1.64 KB, patch)
2024-01-25 20:01 UTC, Nick Clemens (kidclamp)
Details | Diff | Splinter Review
Test order file (6.28 KB, application/xml)
2024-01-25 20:01 UTC, Nick Clemens (kidclamp)
Details
Bug 35913 - Restore price fall back to MarcFieldsToOrder if not set by MarcItemFieldsToOrder (1.70 KB, patch)
2024-02-01 17:50 UTC, ByWater Sandboxes
Details | Diff | Splinter Review
Bug 35913: Restore price fall back to MarcFieldsToOrder if not set by MarcItemFieldsToOrder (1.76 KB, patch)
2024-02-21 05:02 UTC, Victor Grousset/tuxayo
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Clemens (kidclamp) 2024-01-25 19:56:08 UTC
Bug 33170 removed a fallback to the order level price when an item level price was not found in MarcItemFields To order
Comment 1 Nick Clemens (kidclamp) 2024-01-25 20:01:03 UTC
Created attachment 161464 [details] [review]
Bug 35913 - Restore price fall back to MarcFieldsToOrder if not set by MarcItemFieldsToOrder

To test:
Setup -- Set systempreferences below

MarcFieldsToOrder:
price: 949$g
quantity: 949$k
budget_code: 949$l
discount: 949$m
sort1: 949$n
sort2: 949$q

MarcItemFieldsToOrder:
homebranch: 949$a
holdingbranch: 949$b
itype: 949$y
nonpublic_note: 949$x
public_note: 949$z
loc: 949$c
ccode: 949$8
notforloan: 949$7
uri: 949$u
copyno: 949$t
price: 949$g
replacementprice: 949$v
itemcallnumber: 949$o
quantity: 949$k
budget_code: 949$l

Stage the attached bib-303.marcxml file
Add to basket from the staged file
Select the biblio
Note items have prices
Cancel
Remove 'price' mapping from MarcItemFieldsToOrder
Apply patch
Add to basket from file again
Note item prices, and other fields, come through as expected
Comment 2 Nick Clemens (kidclamp) 2024-01-25 20:01:28 UTC
Created attachment 161465 [details]
Test order file
Comment 3 ByWater Sandboxes 2024-02-01 17:50:08 UTC
Created attachment 161717 [details] [review]
Bug 35913 - Restore price fall back to MarcFieldsToOrder if not set by MarcItemFieldsToOrder

To test:
Setup -- Set systempreferences below

MarcFieldsToOrder:
price: 949$g
quantity: 949$k
budget_code: 949$l
discount: 949$m
sort1: 949$n
sort2: 949$q

MarcItemFieldsToOrder:
homebranch: 949$a
holdingbranch: 949$b
itype: 949$y
nonpublic_note: 949$x
public_note: 949$z
loc: 949$c
ccode: 949$8
notforloan: 949$7
uri: 949$u
copyno: 949$t
price: 949$g
replacementprice: 949$v
itemcallnumber: 949$o
quantity: 949$k
budget_code: 949$l

Stage the attached bib-303.marcxml file
Add to basket from the staged file
Select the biblio
Note items have prices
Cancel
Remove 'price' mapping from MarcItemFieldsToOrder
Apply patch
Add to basket from file again
Note item prices, and other fields, come through as expected

Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org>
Comment 4 Victor Grousset/tuxayo 2024-02-21 05:02:48 UTC
Created attachment 162313 [details] [review]
Bug 35913: Restore price fall back to MarcFieldsToOrder if not set by MarcItemFieldsToOrder

To test:
Setup -- Set systempreferences below

MarcFieldsToOrder:
price: 949$g
quantity: 949$k
budget_code: 949$l
discount: 949$m
sort1: 949$n
sort2: 949$q

MarcItemFieldsToOrder:
homebranch: 949$a
holdingbranch: 949$b
itype: 949$y
nonpublic_note: 949$x
public_note: 949$z
loc: 949$c
ccode: 949$8
notforloan: 949$7
uri: 949$u
copyno: 949$t
price: 949$g
replacementprice: 949$v
itemcallnumber: 949$o
quantity: 949$k
budget_code: 949$l

Stage the attached bib-303.marcxml file
Add to basket from the staged file
Select the biblio
Note items have prices
Cancel
Remove 'price' mapping from MarcItemFieldsToOrder
Apply patch
Add to basket from file again
Note item prices, and other fields, come through as expected

Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org>
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Comment 5 Victor Grousset/tuxayo 2024-02-21 05:03:42 UTC
Works, makes sense, QA script happy, code looks good, passing QA :)
Comment 6 Katrin Fischer 2024-03-07 15:37:56 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 7 Fridolin Somers 2024-03-11 09:57:31 UTC
Pushed to 23.11.x for 23.11.04