[Koha-bugs] [Bug 27048] Add timestamps to verbose output of rebuild_zebra.pl
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Tue Feb 23 13:09:12 CET 2021
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27048
Martin Renvoize <martin.renvoize at ptfs-europe.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|Failed QA |Signed Off
--- Comment #8 from Martin Renvoize <martin.renvoize at ptfs-europe.com> ---
I have a better answer now..
1. Because there is already president for using POSIX in the script
2. Because if we were switching to DateTime we should use dt_from_string to
localise and be consistent with the rest of Koha.
3. Because DateTime::Durations don't print nicely without importing
DateTime::Format::Duration which would add another dependency for a reasonably
trivial functionality.
4. Because I hoped to 'keep it simple' by just working in seconds.. which it
did.
I've tried to write a patch to switch to using DateTime and it ends up
requiring extra dependencies, looking worse in the output and more complicated
to work with :(
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list