|
Text to go in the release notes:
|
This fixes several system preferences discrepancies and adds tests, including:
* Setting options to NULL when options=""
* Fixing the explanation when different
* Fixing the wrong order (some rows had options=explanation)
* Fixing the wrong type "Yes/No" => "YesNo"
* Removed StaffLoginBranchBasedOnIP: both StaffLoginLibraryBasedOnIP and StaffLoginBranchBasedOnIP are in the database for upgraded installs
* Adding a description for ApiKeyLog
* Fixing 'integer' vs. 'Integer' inconsistency
* Fixing 'cancelation' typo
* Improving the tests:
* Compare sysprefs.sql and the database content for options, explanation and type
* Catch type not defined
* Catch incorrect YesNo values (must be 0 or 1)
An example where discrepancies have crept in during upgrades includes warnings in the About Koha > System information - the system preferences had no value (either 1 or 0) in the 'value' field:
* Warning System preference 'ILLHistoryCheck' must be '0' or '1', but is ''.
* Warning System preference 'ILLOpacUnauthenticatedRequest' must be '0' or '1', but is ''.
* Warning System preference 'SeparateHoldingsByGroup' must be '0' or '1', but is ''.
|