[koha-commits] main Koha release repository branch master updated. v3.14.00-824-g836f3ce

Sun Apr 20 17:00:08 CEST 2014

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, master has been updated
       via  836f3ce992ef2c3bffbcf80c73bcacbbe479ea85 (commit)
       via  734846e4cb371cf45c5025e9d61e66bd4144936d (commit)
       via  7ca91683854127be5f15d083d7c7e55a44325b74 (commit)
       via  b62a793580292455e78dc21ca27df17e2748e95e (commit)
      from  536345c3b84ac91540b9f20f6e090a7e91343ec6 (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 836f3ce992ef2c3bffbcf80c73bcacbbe479ea85
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Wed Mar 5 15:08:13 2014 +0100

    Bug 8258: Use patron's library's notice for DUEDGST and PREDUEDGST

    If a notice is defined for the library of the patron, it should be
    used.

    Without this patch, the notice used is the one defined for all
    libraries.

    Test plan:
    1/ Set the advanced notice for a patron using digest.
    2/ Check one item out to this patron (backdate the return date according
    the days in advance value).
    3/ launch advance_notices.pl -c
    4/ Verify the notice used is the default one.
    5/ Define a notice for the library of the patron for PREDUEDGST
    6/ launch advance_notices.pl -c
    7/ Verify the notice used is the one previously defined.
    8/ Check one item out to this patron (date due = today)
    9/ launch advance_notices.pl -c
    10/ Verify the notice used is the default one.
    11/ Define a notice for the library of the patron for DUEDGST
    12/ launch advance_notices.pl -c
    13/ Verify the notice used is the one previously defined.

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Passes all tests  and QA script.
    Test case: User from Library A, checked out books
    - in library A from A and B
    - in library B from B
    Verified, that the 'all libraries' notice is still used,
    when no specific notice is defined.
    Verified, that the patron's home library noticed is used,
    when defined.

    Note: Before and after the patch we print the branch information
    from the patron's home library, so also using the template from
    this branch, seems logical. All items over all branches are
    processed into one single reminder email, before and after the patch.

    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit 734846e4cb371cf45c5025e9d61e66bd4144936d
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Wed Nov 20 10:12:56 2013 +0100

    Bug 11243: UT: unit tests for GetBasketsInfosByBookseller

    Verify prove t/db_dependent/Acquisition/GetBasketsInfosByBookseller.t
    returns green.

    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit 7ca91683854127be5f15d083d7c7e55a44325b74
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Wed Nov 20 10:12:00 2013 +0100

    Bug 11243: update vendor list to also display canceled bib counts

    Same as previous patch but for the biblios count.

    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit b62a793580292455e78dc21ca27df17e2748e95e
Author: Jonathan Druart <jonathan.druart at biblibre.com>
Date:   Wed Nov 13 14:57:55 2013 +0100

    Bug 11243: make vendor list distinguish between active and canceled items

    On the vendor result list, the "Item count" columns contain the sum of
    all items ordered for a basket. But if an order is canceled, the item
    count is not really meaningful.

    This patch just adds, in parenthesis, the number of items canceled.

    Test plan:
    - create a basket and 3 orders with different number of items
    - cancel 1 order
    - verify on the supplier list that the number of items is correct and
      the number of canceled items is correct too.

    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    Signed-off-by: Chris Cormack <chris at bigballofwax.co.nz>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Passes all tests and QA script.

    Note: In case the biblio was deleted when the order was cancelled,
    the number of biblios will be off.

    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

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

Summary of changes:
 C4/Acquisition.pm                                  |   25 +++++-
 .../prog/en/modules/acqui/booksellers.tt           |   14 ++-
 misc/cronjobs/advance_notices.pl                   |   38 +++++---
 .../Acquisition/GetBasketsInfosByBookseller.t      |   93 ++++++++++++++++++++
 4 files changed, 151 insertions(+), 19 deletions(-)
 create mode 100644 t/db_dependent/Acquisition/GetBasketsInfosByBookseller.t

hooks/post-receive
-- 
main Koha release repository