From 0a69dda648b921e787774623051c41ea0b0458de Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 23 Mar 2018 13:58:24 -0300 Subject: [PATCH] Bug 20363: (bug 16966 follow-up) better messages handling User views the "Your privacy" -page, it shows "No reading history to delete", even when the user has history. Test plan: - Turn OPACPrivacy on - Go to opac-privacy.pl - Click the "Immediate deletion button" to delete the reading history => If the patron has reading history you must get "Your reading history has been deleted." Otherwise "No reading history to delete" Note that this patch reintroduce the "something went wrong" message if the deletion failed for whatever reason. Signed-off-by: David Bourgault --- koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-privacy.tt | 4 +++- opac/opac-privacy.pl | 9 +++++++-- 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-privacy.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-privacy.tt index 98346f6..143f4e0 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-privacy.tt +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-privacy.tt @@ -27,7 +27,9 @@ [% IF deleted %]
Your reading history has been deleted.
- [% ELSE %] + [% ELSIF history_not_deleted %] +
The deletion of your reading history failed, because there is a problem with the configuration of this feature. Please help to fix the system by informing your library of this error
+ [% ELSIF nothing_to_delete %]
No reading history to delete
[% END %] diff --git a/opac/opac-privacy.pl b/opac/opac-privacy.pl index ff0d328..8796a7a 100755 --- a/opac/opac-privacy.pl +++ b/opac/opac-privacy.pl @@ -63,8 +63,13 @@ elsif ( $op eq "delete_record" ) { my $rows = eval { Koha::Patrons->search({ 'me.borrowernumber' => $borrowernumber })->anonymise_issue_history; }; - $rows = $@ ? 0 : int($rows); - $template->param( 'deleted' => $rows ); + $template->param( + ( + $@ ? ( history_not_deleted => 1 ) + : $rows ? ( deleted => int($rows) ) + : ( nothing_to_delete => 1 ) + ) + ); } # get borrower privacy .... -- 2.7.4