[Koha-bugs] [Bug 22824] Replace YAML::Syck with YAML::XS

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

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

--- Comment #40 from Julian Maurice <julian.maurice at biblibre.com> ---
(In reply to Jonathan Druart from comment #39)
> (In reply to Julian Maurice from comment #36)
> > There are some issues in misc/admin/koha-preferences:
> > - It now allows to set any value for boolean (YesNo) preferences. In master
> > there is a warning message and value is set to 0 when an invalid value is
> > given
> 
> I think we are good with the warning I added on the about page, don't you
> think?
> I don't really want to reintroduce the C4::Boolean module (or equivalent
> code) only for this script.

I think the script should prevent to set wrong value for those sysprefs. It
could at least check that the value given is either "0" or "1".

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