Martin Renvoize <> changed:

           What    |Removed                     |Added
  Attachment #56339|0                           |1
        is obsolete|                            |

--- Comment #8 from Martin Renvoize <> ---
Created attachment 56415
[PASSED QA] Bug 16891: Move C4::Members::MoveMemberToDeleted to

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/ and misc/cronjobs/

Test plan:
1/ Delete a patron from the staff interface and make sure (s)he has been moved
the deletedborrowers table.
2/ Use the "Batch patron deletion" tool (tools/ 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

Signed-off-by: Tomas Cohen Arazi <>
Tested the script and too.
Tests (are relevant and) pass and the qa scripts are happy too :-D

Signed-off-by: Martin Renvoize <>

You are receiving this mail because:
You are watching all bug changes.
Koha-bugs mailing list
website :
git :
bugs :

Reply via email to