[koha-commits] main Koha release repository branch master updated. v17.11.00-246-gc70270b

Tue Jan 2 16:18:13 CET 2018

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  c70270b3aa52f2956efbf62013905af4c632d732 (commit)
       via  5baf823935848a43bdcd7ed63449084075f54d47 (commit)
       via  78462ff16faef5d7f24181c79cc53d10db29d75e (commit)
       via  3a5534fcf5c945e744a86422eb03251b40ea5894 (commit)
       via  9782fa1846575bba8a53e934a16ca94a55bea8b1 (commit)
       via  19e5eeb8e5580dbcf7c344adf5798b63ea720a88 (commit)
       via  c78746d40d584be2a74dc666b39acfa451728803 (commit)
       via  44ba06bdb163ae70db53f7f7747735fb7c8f6ea9 (commit)
       via  91990b04b9f79b130996053c0f4f9a9479338e3c (commit)
      from  0853b50f016107d48b33a8e6a0fa5ff6b1006fa7 (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 c70270b3aa52f2956efbf62013905af4c632d732
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Wed Oct 4 20:22:13 2017 -0300

    Bug 15378: Allow removal of lost items from course reserves

    It seems that this restriction is not necessary, it should be possible
    to remove lost items from course reserves.

    Test plan:
    Create a course and add reserve with a lost item

    => Without this patch the "Remove" button is disabled and the item
    cannot be removed from the course
    => With this patch applied the button is enable and the item can be
    removed

    Signed-off-by: Séverine QUEUNE <severine.queune at bulac.fr>

    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 5baf823935848a43bdcd7ed63449084075f54d47
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Tue Feb 28 14:26:22 2017 +0100

    Bug 18183: jQuery append error related to script tags in cloneItemBlock

    See the detailed explanation of the error on Bugzilla.
    As a workaround, we move the append of clone to outeritemblock before the
    append of data (long string with item fields and script tags) to clone.

    This patch also moves two vars inside the success function, since they are
    not used outside of it.

    Test plan:
    [1] Set AcqCreateItem to receiving.
    [2] Receive an order. Verify that adding, updating or deleting an item block
        still works as expected.

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 78462ff16faef5d7f24181c79cc53d10db29d75e
Author: Josef Moravec <josef.moravec at gmail.com>
Date:   Tue Jan 2 08:44:39 2018 +0000

    Bug 19304: (QA follow-up) Fix number of tests in Members.t

    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 3a5534fcf5c945e744a86422eb03251b40ea5894
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Tue Sep 12 15:41:56 2017 -0300

    Bug 19304: Move C4::Members::GetNoticeEmailAddress to Koha::Patron->notice_email_address

    This subroutine is quite trivial and can be replaced easily with a new
    method of Koha::Patron

    Test plan:
    Overdue notices and shelf sharing must be send the to an email address,
    according to the value of the pref AutoEmailPrimaryAddress

    Signed-off-by: David Bourgault <david.bourgault at inlibro.com>

    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 9782fa1846575bba8a53e934a16ca94a55bea8b1
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Tue Sep 12 15:28:07 2017 -0300

    Bug 19303: Move C4::Members::GetFirstValidEmailAddress to Koha::Patron->first_valid_email_address

    This new Koha::Patron->first_valid_email_address already exists, it
    should be called directly

    Test plan:
    - With a logged in user who have an email address defined, send a basket.
    The email should be marked sent by this email address

    - On the "Holds awaiting pickup", the email address must be displayed as well

    Signed-off-by: David Bourgault <david.bourgault at inlibro.com>

    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 19e5eeb8e5580dbcf7c344adf5798b63ea720a88
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Mon Sep 11 14:16:07 2017 -0300

    Bug 19300: Replace C4::Reserves::OPACItemHoldsAllowed - Remove the subroutine

    At this point the subroutine is not longer in use, we can remove it

    Signed-off-by: Alex Buckley <alexbuckley at catalyst.net.nz>

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit c78746d40d584be2a74dc666b39acfa451728803
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Mon Sep 11 14:15:41 2017 -0300

    Bug 19300: Replace C4::Reserves::OPACItemHoldsAllowed

    This patchset move The OPACItemHoldsAllowed logic
    (issuingrules.opacitemholds) to a new class method of
    Koha::IssuingRules: get_opacitemholds_policy

    On the way, this patch will certainly fix the same problem as bug
    19298 with onshelfholds.

    Test plan:
    Make sure the opacitemholds policy is correct when placing a hold at the
    OPAC or the staff interface.

    Followed test plan which worked as described
    Signed-off-by: Alex Buckley <alexbuckley at catalyst.net.nz>

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 44ba06bdb163ae70db53f7f7747735fb7c8f6ea9
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Mon Sep 11 13:59:11 2017 -0300

    Bug 19300: Replace C4::Reserves::OPACItemHoldsAllowed - tests

    This patch proves that we will not introduce trivial regression.
    With the same tests, we will execute the existing code and the new code.

    Test plan:
    With only this patch applied, prove t/db_dependent/Koha/IssuingRules.t
    should return green

    Followed test plan, patches worked as described
    Signed-off-by: Alex Buckley <alexbuckley at catalyst.net.nz>

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 91990b04b9f79b130996053c0f4f9a9479338e3c
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Tue Dec 19 15:51:42 2017 -0300

    Bug 19798: Fix print transfer slip

    ... and maybe other bugs as well
    What happens here is that $itemnumber is no longer set when barcode is
    passed.

    Test plan:
    1. Check in available/not on hold item at a non owning library.
    2. Click print slip
    3. Item's info must be displayed on the slip

    Signed-off-by: Björn Nylén <bjorn.nylen at ub.lu.se>
    Signed-off-by: Björn Nylén <bjorn.nylen at ub.lu.se>
    Signed-off-by: Jon Knight <J.P.Knight at lboro.ac.uk>

    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

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

Summary of changes:
 C4/Letters.pm                                      |    2 +-
 C4/Members.pm                                      |   50 -------------------
 C4/Reserves.pm                                     |   52 +-------------------
 Koha/IssuingRules.pm                               |   29 +++++++++++
 Koha/Patron.pm                                     |   27 ++++++++++
 circ/returns.pl                                    |    1 +
 .../en/modules/course_reserves/course-details.tt   |    5 --
 .../prog/en/modules/reserve/request.tt             |    2 +-
 koha-tmpl/intranet-tmpl/prog/js/additem.js         |    6 +--
 misc/cronjobs/notice_unprocessed_suggestions.pl    |    3 +-
 misc/cronjobs/overdue_notices.pl                   |    5 +-
 opac/opac-reserve.pl                               |    8 +--
 opac/opac-sendbasket.pl                            |    2 +-
 opac/opac-shareshelf.pl                            |    7 +--
 reserve/request.pl                                 |    6 ++-
 t/db_dependent/Koha/IssuingRules.t                 |   51 ++++++++++++++++++-
 t/db_dependent/Koha/Patrons.t                      |   16 +++++-
 t/db_dependent/Members.t                           |   15 +-----
 18 files changed, 147 insertions(+), 140 deletions(-)

hooks/post-receive
-- 
main Koha release repository