From 6600a7250278f62daf2bb2896da7705a15637adf Mon Sep 17 00:00:00 2001 From: Lari Taskula Date: Mon, 2 Oct 2017 19:06:23 +0300 Subject: [PATCH] Bug 19400: Reminder to unset gone no address flag after patron makes a modification request Our librarians requested a reminder to unset "gone no address" flag from patron's record once the patron has made a modification request to update their address. I propose adding a message box under patron modification request to notify librarians about patrons that have gone no address flag on, and an option to unset the flag without the need of having to navigate into patron's details. To test: 1. Apply patch 2. Set "Gone no address" flag for your test patron. You can do this by going to patron modification screen in staff client. 3. Go to OPAC with your test patron 4. Make a modification request for your personal details 5. Go to staff client and see pending modification requests 6. Open the request you just created 7. Observe a message dialog that says this patron has gone no address flag set 8. Check the checkbox to unset the flag and approve the modification request 9. Click Submit 10. Observe your test patron no longer has gone no address flag set 11. Repeat steps 2-7 12. Do not check the checkbox, but approve the modification request 13. Observe your test patron still has gone no address flag set 14. Remove the gone no address flag from your test patron 15. Repeat steps 3-6 16. Observe there is no message dialog for gone no address Followed test plan, patch worked as described. Also ran QA test tools and all modified files passed Signed-off-by: Alex Buckley --- .../intranet-tmpl/prog/en/modules/members/members-update.tt | 8 ++++++++ members/members-update-do.pl | 9 +++++++++ 2 files changed, 17 insertions(+) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/members-update.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/members-update.tt index 40fc42d..39a5982 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/members-update.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/members-update.tt @@ -166,6 +166,14 @@ [% END %] + [% IF borrowers.$borrowernumber.gonenoaddress > 0 %] +
+

Patron's address in doubt

+

The Gone no address flag is set for this patron. If this modification request resolves the issue, please unset the flag.

+ + +
+ [% END %] [% END %] diff --git a/members/members-update-do.pl b/members/members-update-do.pl index 6a6e826..bd1757f 100755 --- a/members/members-update-do.pl +++ b/members/members-update-do.pl @@ -49,6 +49,15 @@ foreach my $param (@params) { if ( $action eq 'approve' ) { my $m = Koha::Patron::Modifications->find( { borrowernumber => $borrowernumber } ); + + if ($query->param("unset_gna_$borrowernumber")) { + # Unset gone no address + ModMember( + borrowernumber => $borrowernumber, + gonenoaddress => undef + ); + } + $m->approve() if $m; } elsif ( $action eq 'deny' ) { -- 2.1.4