https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41834
--- Comment #3 from Jonathan Druart <[email protected]> --- Created attachment 194393 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=194393&action=edit Bug 41834: Set systempreferences's options, explanation and type to NULL 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). Instead of fixing the discrepancies (again, see bug 41800) we should NULL the 3 columns in sysprefs.sql and so at the DB level to rely only on the yaml files. This patch does several things: 1. Remove the 3 columns from sysprefs.sql to use the default NULL 2. Atomic update: * Set to NULL for existing installations as well * Rename UseICUStyleQUotes => UseICUStyleQuotes * Delete OpacMoreSearches and OPACMySummaryNote, old prefs that have been moved to additional contents 3. Fix some errors in yaml files (mostly empty entries) 4. Move parsing of yaml files to a dedicated Koha::Devel::Syspref module 5. Retrieve all yaml content using new methods added to Koha::Config::SysPrefs (that will be reused in follow-up bugs) Test plan: 1. Run updatedatabase and confirm that now all systempreferences rows in DB have options, explanation and type set to NULL (except local-use prefs) 2. `prove t/db_dependent/check_sysprefs.t` should return green 3. Confirm that the UI is still working correctly -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
