[Koha-bugs] [Bug 32478] Remove Koha::Config::SysPref->find since bypasses cache

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32478

--- Comment #14 from Marcel de Rooy <m.de.rooy at rijksmuseum.nl> ---
(In reply to David Gustafsson from comment #13)
> Ok! Suppose due to forgot to fix the tests that still uses the 'NULL' value,
> will do so. Why is it odd? null is valid json, NULL is not.

Some things are hard to explain :)
The fact that you need to type null as a valid JSON expression in a textarea is
not very appealing as to user interface. (Or even stricter, valid YAML.) But I
could certainly live with it :) Just document it clearly.

And going bit out of scope, but why are we specifying a: [ b, c ] instead of
the pure YAML approach with something like:
a:
- b
- c
We probably need some discussion/consensus about handling all such preferences.

The fact that is all kind of YAML now, does however make a difference. See next
comment.

-- 
You are receiving this mail because:
You are watching all bug changes.