From ff6fc8b9e579b7196fc3a3a79d2f3f85732d0ddf Mon Sep 17 00:00:00 2001 From: Martin Stenberg Date: Tue, 18 Aug 2015 16:41:42 +0200 Subject: [PATCH] Bug 14675: Don't update details if no changes made if no changes has been made for personal details, bring user back to details page and inform them that no changes has been made. Signed-off-by: Aleisha --- .../bootstrap/en/modules/opac-memberentry.tt | 3 ++ opac/opac-memberentry.pl | 42 ++++++++++++-------- 2 files changed, 28 insertions(+), 17 deletions(-) diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt index 4eed652..d5408d5 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt @@ -44,6 +44,9 @@ [% UNLESS OPACPatronDetails %]
To make changes to your record please contact the library.
[% END %] + [% IF nochanges %] +
No changes were made.
+ [% END %] [% END %] [% IF empty_mandatory_fields %] diff --git a/opac/opac-memberentry.pl b/opac/opac-memberentry.pl index 116ba7f..f696208 100755 --- a/opac/opac-memberentry.pl +++ b/opac/opac-memberentry.pl @@ -182,26 +182,34 @@ elsif ( $action eq 'update' ) { $template->param( action => 'edit' ); } else { - ( $template, $borrowernumber, $cookie ) = get_template_and_user( - { - template_name => "opac-memberentry-update-submitted.tt", - type => "opac", - query => $cgi, - authnotrequired => 1, - } - ); - my %borrower_changes = DelUnchangedFields( $borrowernumber, %borrower ); + if (%borrower_changes) { + ( $template, $borrowernumber, $cookie ) = get_template_and_user( + { + template_name => "opac-memberentry-update-submitted.tt", + type => "opac", + query => $cgi, + authnotrequired => 1, + } + ); - my $m = - Koha::Borrower::Modifications->new( - borrowernumber => $borrowernumber ); + my $m = + Koha::Borrower::Modifications->new( + borrowernumber => $borrowernumber ); - $m->DelModifications; - $m->AddModifications(\%borrower_changes); - $template->param( - borrower => GetMember( borrowernumber => $borrowernumber ), - ); + $m->DelModifications; + $m->AddModifications(\%borrower_changes); + $template->param( + borrower => GetMember( borrowernumber => $borrowernumber ), + ); + } + else { + $template->param( + action => 'edit', + nochanges => 1, + borrower => GetMember( borrowernumber => $borrowernumber ), + ); + } } } elsif ( $action eq 'edit' ) { #Display logged in borrower's data -- 1.7.10.4