[koha-commits] main Koha release repository branch 3.22.x updated. v3.22.13-12-g3afe244

[Git repo owner]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".

The branch, 3.22.x has been updated
       via  3afe2446e1169be8539b484a8467c76b60c114a3 (commit)
       via  92ec0a73d96b34134e91d3b4fb80a15926b7952d (commit)
      from  a85fa89541f06a5bc2cc9562d711dc95457d446e (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 3afe2446e1169be8539b484a8467c76b60c114a3
Author: David Cook <dcook at prosentient.com.au>
Date:   Mon May 23 11:57:04 2016 +1000

    Bug 16376 - Koha::Calendar->is_holiday date truncation creates fatal errors for TZ America/Santiago
    
    Using a DateTime object with a timezone of America/Santiago
    was causing fatal errors for Koha::Calendar->is_holiday
    and Koha::Calendar->exception_holidays, when the objects
    were truncated to an invalid local time.
    
    Using a floating zone allows us to use the same day, month, year
    for comparison purposes without running into the possibility of
    creating an invalid local time and thus a fatal software error.
    
    Edit:
    
    While the changes to is_holiday and single_holiday make sense (Jonathan agrees too)
    I didn't manage to have them fail, because truncate is not failing in my trials, but
    days_between. So to me, it narrows down to have exception_holiday return floating tz
    datetime objects so it doesn't break days_between.
    
    Anyway, it is ok to push this patch, and the regression test I provide covers this scenario
    I'm describing.
    
    To test:
    - Apply the regression tests patch
    - Run:
      $ prove t/db_dependent/Holidays.t
    => FAIL: Unexpected error due to bad timezone/date combination
    - Apply this patch
    - Run:
      $ prove t/db_dependent/Holidays.t
    => SUCCESS: Tests pass
    
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    (cherry picked from commit 32354d8322afd05704d87cb38d9bc032ce8c4ec1)
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    (cherry picked from commit 5e9056a99cf37acad6c2d4fe5f3ebd1f61171327)
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>

commit 92ec0a73d96b34134e91d3b4fb80a15926b7952d
Author: Tomas Cohen Arazi <tomascohen at theke.io>
Date:   Tue Dec 6 11:29:08 2016 -0300

    Bug 16376: (regression tests)
    
    This patch introduces a regression test for exception_holidays. This routine
    returns a list of datetimes to be used in date comparison and some datetimes don't exist
    in some timezones, so floating timezones should be used instead.
    
    To test:
    - Apply the patch on master
    - Run:
      $ prove t/db_dependent/Holidays.t
    => FAIL: The new test fails
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    (cherry picked from commit c6a0848ff0b7acbec4d7bf817447d7cc84c0615e)
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    (cherry picked from commit aa5c67486f297dbdaa1ba7b6665916a3d2cc05f5)
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>

-----------------------------------------------------------------------

Summary of changes:
 Koha/Calendar.pm          |    6 ++++--
 t/db_dependent/Holidays.t |   48 +++++++++++++++++++++++++++++++++++++++++----
 2 files changed, 48 insertions(+), 6 deletions(-)


hooks/post-receive
-- 
main Koha release repository