[koha-commits] main Koha release repository branch 3.16.x updated. v3.16.02-8-g24be2de

[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.16.x has been updated
       via  24be2de308729a79f4738d8a0ac7ecfd0ec54a68 (commit)
       via  c2ddcf83b64c1d23d3d9f83b40f578ca243e75bf (commit)
       via  462c33aeb6a71eb8c5d9afa98d05b7f48b7d742f (commit)
       via  90328d0811c8c253188a4cb6192d9a6915fcd68a (commit)
      from  c00d74306f69ac99cf45feee52d00d16f6f26de0 (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 24be2de308729a79f4738d8a0ac7ecfd0ec54a68
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Tue Jul 22 12:15:35 2014 +0200

    Bug 12622: Fix export report as ods
    
    Bug 11679 introduces an ods export for reports.
    
    It looks quite buggy:
    1/ You get a Perl error if the report has no results
    2/ The ods file contains 1 lines less than the total results.
    
    Test plan:
    After applying this patch, try to export a report which has no results
    and verify you get an ods file with headers only.
    Try to export a report which has a least 1 result and verify the
    generated ods has the right number of lines.
    
    Signed-off-by: Robert Higgins <robert.higgins at catalyst.net.nz>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Fixes the reported problem, passes all tests and QA script.
    Added the sign-off line for Robert.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit c2ddcf83b64c1d23d3d9f83b40f578ca243e75bf
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date:   Fri Jul 25 09:55:14 2014 -0300

    Bug 12596: (followup) add reference to bug # and license note
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit 462c33aeb6a71eb8c5d9afa98d05b7f48b7d742f
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Thu Jul 17 10:57:06 2014 -0400

    Bug 12596 - Backdating returns with SpecifiyReturnDate causes fines for items not overdue!
    
    When using the backdating of returns feature, an item that is not
    overdue is treated as being as many days overdue as it is *not* overdue.
    This is due to the fact that _get_chargeable_units appears to return the
    difference between the return date and the due date without
    consideration the return date being earlier than the due date.
    
    Test Plan:
    1) Apply the unit test patch
    2) prove t/db_dependent/Circulation.t
    3) Note the failure
    4) Apply the second patch
    5) prove t/db_dependent/Circulation.t
    6) Note there are no failures
    
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>
    Fixes some badly named variables also
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    In order to test this, you need to activate SpecifyReturnDate.
    I confirmed the problem and verified that the bug fixes it
    by running the tests, but also by testing in staff.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit 90328d0811c8c253188a4cb6192d9a6915fcd68a
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Thu Jul 17 12:17:48 2014 -0400

    Bug 12596 - Unit Test
    
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

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

Summary of changes:
 C4/Overdues.pm               |   14 +++++++++-----
 reports/guided_reports.pl    |   11 +++++++----
 t/db_dependent/Circulation.t |   40 +++++++++++++++++++++++++++++++---------
 3 files changed, 47 insertions(+), 18 deletions(-)


hooks/post-receive
-- 
main Koha release repository