[koha-commits] main Koha release repository branch master updated. v3.22.00-866-ge8036c9

Git repo owner gitmaster at git.koha-community.org
Sun Mar 13 00:33:45 CET 2016


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  e8036c97949692838f1e7452b83b2d950b32e5f4 (commit)
       via  aa73c96aed863637531355a548467fd67cd63b83 (commit)
       via  bff0ca0b0b91fd066ebc87a59d88f92f6143f7ee (commit)
      from  186f630407573fd5a48522457685cebd13abc6db (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 e8036c97949692838f1e7452b83b2d950b32e5f4
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Fri Mar 11 18:35:47 2016 +0000

    Bug 15656 [QA Followup] - Return without searching if patron has no guarantor
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    
    Signed-off-by: Brendan Gallagher brendan at bywatersolutions.com

commit aa73c96aed863637531355a548467fd67cd63b83
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Mon Jan 25 15:41:20 2016 +0000

    Bug 15656: Move guarantor/guarantees code - GetMemberRelatives
    
    Note:
    QA question: Does the Koha::Patron->siblings method should return undef
    if there is no guarantor?
    It would avoid the weird  != undef, = $borrowernumber conditions.
    
    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    
    Signed-off-by: Brendan Gallagher brendan at bywatersolutions.com

commit bff0ca0b0b91fd066ebc87a59d88f92f6143f7ee
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Mon Jan 25 13:25:42 2016 +0000

    Bug 15656: Move guarantor/guarantees code - GetGuarantees
    
    2 subroutines of C4::Members deal with guarantor/guarantees:
    GetGuarantees and GetMemberRelatives.
    Since we already have a Koha::Patron->guarantor method, it makes sense
    to move these 2 subroutines to this module.
    
    This first patch deals with GetGuarantees.
    
    Test plan for the entire patch set:
    1/ Create 5 patrons A (adult), B (child), C (child), D (child), E
    (child), F (adult)
    2/ Add relation between them: A is father of B, C and D.
    E does not have a guarantor
    F does not have guarantees
    3/ Check some items out for all of these patrons
    4/ On the "Check out" and "Details" tabs, you should not see any
    differences with these patch applied : The "Relatives' checkouts" tabs
    should list all of the guarantor/guarantee/siblings checkouts
    
    Note:
    $template->param('C' => 1);
    I have not found any reference of this 'C' in the template.
    It seems it's an old c/p from members/memberentrygen.tt
    
    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    
    Signed-off-by: Brendan Gallagher brendan at bywatersolutions.com

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

Summary of changes:
 C4/Members.pm                                      |   75 --------------
 Koha/Patron.pm                                     |   40 ++++++++
 circ/circulation.pl                                |   10 +-
 .../prog/en/modules/members/moremember.tt          |   22 ++--
 members/moremember.pl                              |   51 +++-------
 t/db_dependent/Koha/Patrons.t                      |  106 ++++++++++++++++++++
 6 files changed, 183 insertions(+), 121 deletions(-)
 create mode 100644 t/db_dependent/Koha/Patrons.t


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list