[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 Mason Jameschanged: What|Removed |Added CC||m...@kohaaloha.com --- Comment #11 from Mason James --- Enhancement, skipping for 16.05.x -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 Kyle M Hallchanged: What|Removed |Added CC||k...@bywatersolutions.com Status|Passed QA |Pushed to Master --- Comment #10 from Kyle M Hall --- Pushed to master for 16.11, thanks Jonathan! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 Nick Clemenschanged: What|Removed |Added CC||n...@bywatersolutions.com Attachment #56418|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 --- Comment #9 from Nick Clemens--- Created attachment 56418 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56418=edit 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 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: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 Martin Renvoizechanged: What|Removed |Added Attachment #56339|0 |1 is obsolete|| --- Comment #8 from Martin Renvoize --- Created attachment 56415 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56415=edit [PASSED QA] 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 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 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 Martin Renvoizechanged: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 Jonathan Druartchanged: What|Removed |Added Attachment #54178|0 |1 is obsolete|| --- Comment #7 from Jonathan Druart --- Created attachment 56339 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=56339=edit 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 Tested the delete_patrons.pl script and cleanborrowers.pl too. Tests (are relevant and) pass and the qa scripts are happy too :-D -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 --- Comment #6 from Owen Leonard--- (In reply to Jonathan Druart from comment #5) > Could you confirm that it correctly works please? > I have just retested and the cronjob script seems to work as expected. I defer to Tomás on the signoff, since I'm never confident that errors like that aren't a configuration problem with my gitified package installation. However, if there is more I can do to test I would be happy to do so. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 --- Comment #5 from Jonathan Druart--- (In reply to Owen Leonard from comment #3) > (In reply to Jonathan Druart from comment #2) > > 3/ Same as previously but using the cronjob > > misc/cronjobs/delete_patrons.pl. > > At this step I'm getting "No method move_to_deleted! at > misc/cronjobs/delete_patrons.pl line 77." The patch has been signed off in the meanwhile. Could you confirm that it correctly works please? I have just retested and the cronjob script seems to work as expected. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 Tomás Cohen Arazichanged: What|Removed |Added Attachment #53522|0 |1 is obsolete|| --- Comment #4 from Tomás Cohen Arazi --- Created attachment 54178 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=54178=edit 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 Tested the delete_patrons.pl script and cleanborrowers.pl too. Tests (are relevant and) pass and the qa scripts are happy too :-D -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 Tomás Cohen Arazichanged: What|Removed |Added Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 Owen Leonardchanged: What|Removed |Added Status|Needs Signoff |Failed QA --- Comment #3 from Owen Leonard --- (In reply to Jonathan Druart from comment #2) > 3/ Same as previously but using the cronjob > misc/cronjobs/delete_patrons.pl. At this step I'm getting "No method move_to_deleted! at misc/cronjobs/delete_patrons.pl line 77." -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 Jonathan Druartchanged: What|Removed |Added Attachment #53249|0 |1 is obsolete|| --- Comment #2 from Jonathan Druart --- Created attachment 53522 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53522=edit 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. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 Jonathan Druartchanged: What|Removed |Added Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 Owen Leonardchanged: What|Removed |Added Status|Needs Signoff |Patch doesn't apply -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 Jonathan Druartchanged: What|Removed |Added Blocks||16907 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16907 [Bug 16907] Move DelMember and HandleDelBorrower to Koha::Patron -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 --- Comment #1 from Jonathan Druart--- Created attachment 53249 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53249=edit 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. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 Jonathan Druartchanged: What|Removed |Added Status|ASSIGNED|Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 16891] Move MoveMemberToDeleted to Koha::Patron->move_to_deleted
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16891 Jonathan Druartchanged: What|Removed |Added Blocks||16846 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16846 [Bug 16846] Move patron related code to Koha::Patron -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/