Bug 41834 - Remove systempreferences's options, explanation and type DB values
Summary: Remove systempreferences's options, explanation and type DB values
Status: ASSIGNED
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement
Assignee: Jonathan Druart
QA Contact: Testopia
URL:
Keywords:
Depends on: 41837 41833
Blocks:
  Show dependency treegraph
 
Reported: 2026-02-12 09:56 UTC by Jonathan Druart
Modified: 2026-03-02 22:53 UTC (History)
3 users (show)

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 41834: Set systempreferences's options, explanation and type to NULL (209.93 KB, patch)
2026-03-02 15:59 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2026-02-12 09:56:19 UTC
We currently have to set options, explanation and type in sysprefs.sql and the atomic/dbrev files.
However we (almost) never use them, we rely on the values in the yml files.

We could remove them and don't require them when when add new sysprefs.

It will avoid the mess we currently have (mismatch between new and updated installations, see bug 41682 and bug 41800).
Comment 1 Jonathan Druart 2026-03-02 15:59:47 UTC
Created attachment 194343 [details] [review]
Bug 41834: Set systempreferences's options, explanation and type to NULL

Patch from commit 637b6a5
Comment 2 David Cook 2026-03-02 22:53:30 UTC
This sounds really good. It drives me a bit crazy that we duplicate things between the DB and the YAML files.