Bug 41763 - Use Koha::Biblioframework->fill_with_default_values instead of C4::Acquisition::FillWithDefaultValue
Summary: Use Koha::Biblioframework->fill_with_default_values instead of C4::Acquisitio...
Status: Needs Signoff
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Johanna Räisä
QA Contact: Testopia
URL:
Keywords:
Depends on: 41757
Blocks:
  Show dependency treegraph
 
Reported: 2026-02-04 05:49 UTC by Johanna Räisä
Modified: 2026-02-06 09:17 UTC (History)
0 users

See Also:
GIT URL:
Initiative type: ---
Sponsorship status: ---
Comma delimited list of Sponsors:
Crowdfunding goal: 0
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 41763: use fill_default_value instead of FillWithDefaultValues (10.05 KB, patch)
2026-02-06 09:17 UTC, Johanna Räisä
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Johanna Räisä 2026-02-04 05:49:12 UTC
There are no single method to fill default values from framework. Every module has there own way to do it. One method helps to keep code clean and robust.
Comment 1 Johanna Räisä 2026-02-06 09:17:45 UTC
Created attachment 192588 [details] [review]
Bug 41763: use fill_default_value instead of  FillWithDefaultValues

This patch adds unified version of the code to fill default values in acquisition order creation.
It uses the fill_default_value method from the Koha::Biblioframeworks instead of the C4::Acquisition::FillWithDefaultValues module.

Test plan:
1. Create or modify acquisition framework ACQ with a default value for a field (for instance 099$z=1).
2. Create a new acquisition order from a new record.
3. Verify the default value exist in the order detail page.

Sponsored-by: Koha-Suomi Oy