[koha-commits] main Koha release repository branch master updated. v16.05.00-1101-gb9af2e9

[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, master has been updated
       via  b9af2e9731ef17af3d35090e5bae3af65a8d08c4 (commit)
       via  c4a923bdf7eacce4de1efd4c5bdd2fe4010ad4b1 (commit)
       via  26c034d1f50762f0548ce3ee61767d64139cd311 (commit)
       via  20c44f005138d2f585eeb1336a93920629b52d3c (commit)
      from  6fb994257ecf74f65b0951564341561dd40df3a5 (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 b9af2e9731ef17af3d35090e5bae3af65a8d08c4
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Tue Aug 9 10:17:18 2016 +0100

    Bug 16907: Make sure the log will be written only if the patron has been deleted
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit c4a923bdf7eacce4de1efd4c5bdd2fe4010ad4b1
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Mon Jul 11 21:35:15 2016 +0100

    Bug 16907: Koha::Patrons - Move HandleDelBorrower to ->delete
    
    This job should be done each time patron data are deleted. It's better
    to do it just before deleting the patron than assuming the caller did
    the job by itself.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit 26c034d1f50762f0548ce3ee61767d64139cd311
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Mon Jul 11 21:18:56 2016 +0100

    Bug 16907: Koha::Patrons - Move DelMember to ->delete
    
    This patch moves the C4::Members::DelMember subroutine to the
    Koha::Patron module.
    The delete method must be overwritten to permit handling of patron's
    holds.
    
    Test plan:
    (With the 2 patches applied)
    1/ Create a patron with holds and owner of lists
    2/ Delete patrons using the web interface:
     - More > Delete on a patron page
     - Batch patron deletion tools
    3/ and the cronjob script
     - perl misc/cronjobs/delete_patrons.pl -c [more options]
    
    The patron should have been moved to the deletedborrowers table, his/her
    holds and lists should have been deleted.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit 20c44f005138d2f585eeb1336a93920629b52d3c
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Sat Jul 9 16:03:11 2016 +0100

    Bug 16891: Move C4::Members::MoveMemberToDeleted to Koha::Patron->move_to_deleted
    
    This patch removes the C4::Members::MoveMemberToDeleted subroutine in
    order to replace it with the Koha::Patron->move_to_deleted method.
    Next after this change, we will move C4::Members::HandleDelBorrower and
    C4::Members::DelMember to the same module to simplify the code in
    members/deletemem.pl and misc/cronjobs/delete_patrons.pl
    
    Test plan:
    1/ Delete a patron from the staff interface and make sure (s)he has been moved to
    the deletedborrowers table.
    2/ Use the "Batch patron deletion" tool (tools/cleanborrowers.pl) to
    remove patron. Make sure the "Permanently delete these patrons" and "Move
    these patrons to the trash" options work as before
    3/ Same as previously but using the cronjob
    misc/cronjobs/delete_patrons.pl.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    Tested the delete_patrons.pl script and cleanborrowers.pl too.
    Tests (are relevant and) pass and the qa scripts are happy too :-D
    
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

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

Summary of changes:
 C4/Members.pm                                      |   94 +-------------------
 Koha/Patron.pm                                     |   62 +++++++++++++
 members/deletemem.pl                               |    7 +-
 misc/cronjobs/delete_expired_opac_registrations.pl |    1 -
 misc/cronjobs/delete_patrons.pl                    |   43 +++------
 .../delete_unverified_opac_registrations.pl        |    1 -
 t/db_dependent/Auth_with_ldap.t                    |    4 +-
 t/db_dependent/Koha/Patrons.t                      |   48 +++++++++-
 t/db_dependent/Members.t                           |   33 +------
 tools/cleanborrowers.pl                            |    7 +-
 10 files changed, 136 insertions(+), 164 deletions(-)


hooks/post-receive
-- 
main Koha release repository