[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

Jonathan Druart <jonathan.druart at bugs.koha-community.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |In Discussion
                 CC|                            |julian.maurice at biblibre.com
                   |                            |,
                   |                            |kyle at bywatersolutions.com,
                   |                            |m.de.rooy at rijksmuseum.nl,
                   |                            |nick at bywatersolutions.com,
                   |                            |tomascohen at gmail.com

--- Comment #16 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
I am sorry, I lost myself!

This was supposed to be a quite trivial patch and it ended up huge!

I think the move is interesting and going into the right direction.
But I have the feeling that it's not finished and some tricky checks need to be
done on the syspref table/.pref.

This patch set is dealing with the yes/no, on/off weird options we have,
historically I bet, in our table and .pref files. In this patch we are removing
all of that and expect a boolean to be 1 or 0 (wow!), and it removes the
C4::Boolean weirdness.

So far the prefs, installer and translate scripts seem to be working correctly.
The edge cases may be hard to catch!

Don't be afraid by the number of patches, they are all very trivial!

What do you think?

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