Bug 36053 - Replacement prices not populating when supplied from MarcItemFieldsToOrder
Summary: Replacement prices not populating when supplied from MarcItemFieldsToOrder
Status: Pushed to stable
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: Main
Hardware: All All
: P5 - low major with 15 votes (vote)
Assignee: Nick Clemens
QA Contact: Marcel de Rooy
URL:
Keywords: rel_23_05_candidate, rel_23_11_candidate
Depends on: 32166
Blocks:
  Show dependency treegraph
 
Reported: 2024-02-08 20:27 UTC by Nick Clemens
Modified: 2024-04-23 15:21 UTC (History)
14 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.05


Attachments
Bug 36053: Correct input name for item replacement fields (3.12 KB, patch)
2024-02-08 20:31 UTC, Nick Clemens
Details | Diff | Splinter Review
Bug 36053: Correct input name for item replacement fields (3.17 KB, patch)
2024-02-12 16:40 UTC, Brendan Lawlor
Details | Diff | Splinter Review
Bug 36053: Correct input name for item replacement fields (3.39 KB, patch)
2024-04-05 06:52 UTC, Marcel de Rooy
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Clemens 2024-02-08 20:27:07 UTC
Bug 32166 added the import_record_id to the name for the replacementprice field, however, this made it clash with the record level replacement price - that field doesn't populate when we are using MarcItemFieldsToOrder and so is $0

When fetching the item prices, we get an array which inlcudes that $0 and so the order replacement prices appear correctly in the form, but are not saved to the database

We need to rename the input field
Comment 1 Nick Clemens 2024-02-08 20:31:40 UTC
Created attachment 161955 [details] [review]
Bug 36053: Correct input name for item replacement fields

To test:
1 - Follow test plan and use sample record from bug 35912 ( or similar)
2 - Confirm that when the order is saved the replacement price is $0.00
3 - Apply patch, restart_all
4 - Delete previous orders, and stage and add to basket again
5 - Confirm that upon adding orders to basket the replacement price is saved correctly
Comment 2 Brendan Lawlor 2024-02-12 16:40:57 UTC
Created attachment 162063 [details] [review]
Bug 36053: Correct input name for item replacement fields

To test:
1 - Follow test plan and use sample record from bug 35912 ( or similar)
2 - Confirm that when the order is saved the replacement price is $0.00
3 - Apply patch, restart_all
4 - Delete previous orders, and stage and add to basket again
5 - Confirm that upon adding orders to basket the replacement price is saved correctly

Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org>
Comment 3 Erin A. 2024-03-19 16:17:49 UTC
We noticed this bug and are adding a comment to let you know we are interested in a fix! Thanks.
Comment 4 Katrin Fischer 2024-03-19 16:43:45 UTC
(In reply to Erin A. from comment #3)
> We noticed this bug and are adding a comment to let you know we are
> interested in a fix! Thanks.

Hi Erin :) This bug already has a patch and is in "Signed off", which means it will be reviewed by the QA team next and if it passes (everything works etc.) it will be included in Koha. As a bug it's likely to be backported into stable versions as well. Please keep commenting!
Comment 5 Nick Clemens 2024-04-04 17:03:24 UTC
This causes loss of data during order/record import - raising to major
Comment 6 Barbara Johnson 2024-04-04 17:12:58 UTC
+1 I very much agree that this is a major problem for us.
Comment 7 Marcel de Rooy 2024-04-05 06:52:23 UTC
Created attachment 164448 [details] [review]
Bug 36053: Correct input name for item replacement fields

To test:
1 - Follow test plan and use sample record from bug 35912 ( or similar)
2 - Confirm that when the order is saved the replacement price is $0.00
3 - Apply patch, restart_all
4 - Delete previous orders, and stage and add to basket again
5 - Confirm that upon adding orders to basket the replacement price is saved correctly

Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 8 Katrin Fischer 2024-04-05 09:52:40 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 9 Fridolin Somers 2024-04-18 07:23:57 UTC
Pushed to 23.11.x for 23.11.05
Comment 10 Barbara Johnson 2024-04-18 21:33:31 UTC
Just curious if this bug could be backported to 23.05?