[koha-commits] main Koha release repository branch 3.16.x updated. v3.16.06-116-g62a253f

Fri Jan 23 08:02:00 CET 2015

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  62a253fc44583125cff03cbbe53fe6525ff23683 (commit)
       via  e7eacdfaf5e5cab1a40841e2f5bdc91cca3e53c6 (commit)
       via  558e7aaff2ea60ae54ee94c2760826d2b1001c76 (commit)
       via  82c6fd5642aee1fc2d3fae1d9835d680b4b79c2a (commit)
       via  d561213146334dfc3d4013a5fc86dee19ac7552c (commit)
       via  72d0eb1f869bc5e2607ab1d2c900a0f18fc08cce (commit)
      from  a00af8d2861958cb036c2fdf0a7d4ffee647a2bd (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 62a253fc44583125cff03cbbe53fe6525ff23683
Author: Mark Tompsett <mtompset at hotmail.com>
Date:   Wed Sep 3 19:09:46 2014 -0400

    Bug 12868: Improving t/db_dependent/Member.t

    The mock function of GetMemberAccountRecord did not properly
    account for the undef case. This was corrected.

    Then all 4 combinations of borrower number and card number being
    defined or not were called to GetMemberDetail.

    The problematic test case is where the borrower number is
    undefined and the cardnumber is defined.

    TEST PLAN
    ---------
    1) Apply just this first patch.
    2) prove -v t/db_dependent/Member.t
       -- This should fail!
    3) Run koha QA test tools.

    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Signed-off-by: Mason James <mtj at kohaaloha.com>

commit e7eacdfaf5e5cab1a40841e2f5bdc91cca3e53c6
Author: Olli-Antti Kivilahti <olli-antti.kivilahti at jns.fi>
Date:   Mon Sep 15 11:38:32 2014 +0300

    Bug 12922 - Do not DIE the advance_notices.pl -cronjob if no letter of type is found!!

    We failed to deliver advance_notices because a template for sms's is undefined, because we don't support
    sending sms' as advance_notice.
    This crashed the cronjob because digests are set to die instead of the warn used in non-digest.
    And we get angry customers asking for compensation!

    This patch replaces the die with warn.

    TEST PREPARATION:

    0. Edit the ODUEDGST letter, find an undefined letter for any trasport type.

    TEST PLAN:

    1. Find a borrower and from the messaging preferences set the "Advance notice" transport type to
       the undefined digest. Set the "Days in Advance" to 1.
    2. Check-out something for that borrower and set the due date for tomorrow.
    3. Run "misc/cronjobs/advance_notices.pl -c -n -v" from the terminal.
    4. BEFORE THIS PATCH: You get an error
       "No circulation PREDUEDGST letter transported by sms at /home/koha/kohaclone/C4/Letters.pm line 609."
       and the script dies.
    4. AFTER THIS PATCH: You get an error
       "No circulation PREDUEDGST letter transported by sms at /home/koha/kohaclone/C4/Letters.pm line 609."
       but the script keep on going!

    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Mason James <mtj at kohaaloha.com>

commit 558e7aaff2ea60ae54ee94c2760826d2b1001c76
Author: Colin Campbell <colin.campbell at ptfs-europe.com>
Date:   Tue Jan 6 11:40:46 2015 +0000

    Bug 13522: Make it explicit that scalar containd a hash ref

    Prior to perl 5.12 keys can only operate on a hash. So although
    $data[0] ( thats an abysmal variable name! ) will contain a hash ref
    the perl compiler cannot deduce that from the context and gives
    a syntax error. Add the hash sigil to make the context explicit and
    the compiler can generate the correct code.

    Signed-off-by: Jonathan Druart <jonathan.druart at biblibre.com>

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Mason James <mtj at kohaaloha.com>

commit 82c6fd5642aee1fc2d3fae1d9835d680b4b79c2a
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Wed Dec 3 11:35:36 2014 +0100

    Bug 13378: Add a filter to search suggestions not linked to a fund

    This patch adds a "None" option for the fund filter.

    Test plan:
    1/ Go on the suggestion search page
    2/ Search suggestions not linked to a fund using the "None" option.
    3/ Search all suggestions (linked or not to a fund) using the "Any" option.

    Works as expected.
    Signed-off-by: Marc Veron <veron at veron.ch>

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Mason James <mtj at kohaaloha.com>

commit d561213146334dfc3d4013a5fc86dee19ac7552c
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Tue Dec 2 10:51:08 2014 +0100

    Bug 13369: table should been highlighted correctly when row are grouped

    The css used to highlight the rows comes from staff-global.css
    We need a more specific rule to be used.

    Test plan:
    Go on the fund list view and confirm that the rows are correctly
    highlighted.

    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Mason James <mtj at kohaaloha.com>

commit 72d0eb1f869bc5e2607ab1d2c900a0f18fc08cce
Author: Chris Nighswonger <chris.nighswonger at gmail.com>
Date:   Fri Dec 19 07:47:40 2014 -0500

    Bug 13407 Bumping required version of PDF::Reuse

    Signed-off-by: Chris <chris at bigballofwax.co.nz>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
    Signed-off-by: Mason James <mtj at kohaaloha.com>

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

Summary of changes:
 C4/Installer/PerlDependencies.pm                   |    2 +-
 C4/Suggestions.pm                                  |   17 +-
 koha-tmpl/intranet-tmpl/prog/en/css/datatables.css |   14 +-
 .../prog/en/modules/admin/aqbudgets.tt             |    2 +-
 .../prog/en/modules/suggestion/suggestion.tt       |    9 +-
 misc/cronjobs/advance_notices.pl                   |   12 +-
 t/db_dependent/Members.t                           |   54 +++-
 t/db_dependent/Suggestions.t                       |  286 +++++++++++++++++++-
 tools/viewlog.pl                                   |   23 ++
 9 files changed, 386 insertions(+), 33 deletions(-)

hooks/post-receive
-- 
main Koha release repository