Bug 35912 - Item prices not populating order form when adding to a basket from a staged file
Summary: Item prices not populating order form when adding to a basket from a staged file
Status: RESOLVED FIXED
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: Marcel de Rooy
URL:
Keywords:
Depends on: 33170
Blocks:
  Show dependency treegraph
 
Reported: 2024-01-25 19:36 UTC by Nick Clemens (kidclamp)
Modified: 2024-02-08 20:32 UTC (History)
6 users (show)

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


Attachments
Bug 35912: Corretc field name form itemprice to price in addorderiso2709 template (2.54 KB, patch)
2024-01-25 19:39 UTC, Nick Clemens (kidclamp)
Details | Diff | Splinter Review
Test Order file (6.28 KB, application/xml)
2024-01-25 19:40 UTC, Nick Clemens (kidclamp)
Details
Bug 35912: Corretc field name form itemprice to price in addorderiso2709 template (2.60 KB, patch)
2024-01-25 22:01 UTC, ByWater Sandboxes
Details | Diff | Splinter Review
Bug 35912: Corretc field name form itemprice to price in addorderiso2709 template (2.69 KB, patch)
2024-01-26 08:02 UTC, Marcel de Rooy
Details | Diff | Splinter Review
Bug 35912: Corretc field name form itemprice to price in addorderiso2709 template (2.71 KB, patch)
2024-02-01 17:44 UTC, ByWater Sandboxes
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:36:50 UTC
Bug 33170 refactored addorderiso2709 to make MarcItemFieldsToOrder easier to adjust, however, it missed that when listing we renamed 'price' as 'itemprice' so prices for items are not being populated
Comment 1 Nick Clemens (kidclamp) 2024-01-25 19:39:33 UTC
Created attachment 161462 [details] [review]
Bug 35912: Corretc field name form itemprice to price in addorderiso2709 template

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 do no have prices
Cancel
Apply patch
Add to basket from file again
Note prices, and other fields, come through as expected
Comment 2 Nick Clemens (kidclamp) 2024-01-25 19:40:18 UTC
Created attachment 161463 [details]
Test Order file
Comment 3 ByWater Sandboxes 2024-01-25 22:01:09 UTC
Created attachment 161470 [details] [review]
Bug 35912: Corretc field name form itemprice to price in addorderiso2709 template

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 do no have prices
Cancel
Apply patch
Add to basket from file again
Note prices, and other fields, come through as expected

Signed-off-by: Michelle Spinney <mspinney@clamsnet.org>
Comment 4 Marcel de Rooy 2024-01-26 08:00:15 UTC
(In reply to ByWater Sandboxes from comment #3)
> Bug 35912: Corretc field name form itemprice to price in addorderiso2709

Yeah, corretc :)
Comment 5 Marcel de Rooy 2024-01-26 08:02:42 UTC
Created attachment 161483 [details] [review]
Bug 35912: Corretc field name form itemprice to price in addorderiso2709 template

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 do no have prices
Cancel
Apply patch
Add to basket from file again
Note prices, and other fields, come through as expected

Signed-off-by: Michelle Spinney <mspinney@clamsnet.org>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 6 Katrin Fischer 2024-01-26 14:18:38 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 7 Fridolin Somers 2024-01-31 12:53:12 UTC
Pushed to 23.11.x for 23.11.03
Comment 8 ByWater Sandboxes 2024-02-01 17:44:27 UTC
Created attachment 161716 [details] [review]
Bug 35912: Corretc field name form itemprice to price in addorderiso2709 template

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 do no have prices
Cancel
Apply patch
Add to basket from file again
Note prices, and other fields, come through as expected

Signed-off-by: Michelle Spinney <mspinney@clamsnet.org>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org>
Comment 9 Andrew Fuerste-Henry 2024-02-01 17:53:09 UTC
Apologies for the stray sign-off, it was a sandbox error.
Comment 10 Liz Rea 2024-02-08 16:28:44 UTC
May we have a backport for this to 23.05?
Comment 11 Nick Clemens (kidclamp) 2024-02-08 18:37:06 UTC
This should not be needed for 23.05 - caused by 33170 - so only in 23.11 and after
Comment 12 Lucas Gass 2024-02-08 18:52:36 UTC
Missing dependency for 23.05.x, no backport.