Fix for Bug 4138, Adding or deleting suggestions in the OPAC does not properly refresh the page
- Using redirect() and adding URL parameter to enable the display of messages on the screen confirming the action. - Also correcting for JavaScript error in form validation routine. Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
This commit is contained in:
parent
d91ebf445f
commit
65540255d8
2 changed files with 12 additions and 3 deletions
|
@ -52,7 +52,7 @@ $.tablesorter.addParser({
|
|||
}
|
||||
|
||||
if (_alertString.length==0) {
|
||||
document.Aform.submit();
|
||||
f.submit();
|
||||
} else {
|
||||
alertString2 = _("Form not submitted because of the following problem(s)\n");
|
||||
alertString2 += "------------------------------------------------------------------------------------\n\n";
|
||||
|
@ -100,6 +100,10 @@ $.tablesorter.addParser({
|
|||
|
||||
<!-- TMPL_IF name="op_else" -->
|
||||
<h1><!-- TMPL_UNLESS NAME="OPACViewOthersSuggestions" --><!-- TMPL_IF NAME="loggedinusername" -->My <!-- /TMPL_IF --><!-- /TMPL_UNLESS -->Purchase Suggestions</h1>
|
||||
|
||||
<!-- TMPL_IF NAME="deleted" --><div class="dialog message">The selected suggestions have been deleted.</div><!-- /TMPL_IF -->
|
||||
<!-- TMPL_IF NAME="submitted" --><div class="dialog message">Your suggestion has been submitted.</div><!-- /TMPL_IF -->
|
||||
|
||||
<!-- TMPL_IF NAME="suggestions_loop" -->
|
||||
<!-- TMPL_IF NAME="OPACViewOthersSuggestions" -->
|
||||
<form action="/cgi-bin/koha/opac-suggestions.pl" method="get">
|
||||
|
|
|
@ -35,7 +35,8 @@ delete $$suggestion{$_} foreach qw<op suggestedbyme>;
|
|||
$op = 'else' unless $op;
|
||||
|
||||
my ( $template, $borrowernumber, $cookie );
|
||||
|
||||
my $deleted = $input->param('deleted');
|
||||
my $submitted = $input->param('submitted');
|
||||
|
||||
if ( C4::Context->preference("AnonSuggestions") ) {
|
||||
( $template, $borrowernumber, $cookie ) = get_template_and_user(
|
||||
|
@ -84,6 +85,8 @@ if ( $op eq "add_confirm" ) {
|
|||
&SearchSuggestion( $suggestion );
|
||||
}
|
||||
$op = 'else';
|
||||
print $input->redirect("/cgi-bin/koha/opac-suggestions.pl?op=else&submitted=1");
|
||||
exit;
|
||||
}
|
||||
|
||||
if ( $op eq "delete_confirm" ) {
|
||||
|
@ -92,6 +95,8 @@ if ( $op eq "delete_confirm" ) {
|
|||
&DelSuggestion( $borrowernumber, $delete_field );
|
||||
}
|
||||
$op = 'else';
|
||||
print $input->redirect("/cgi-bin/koha/opac-suggestions.pl?op=else&deleted=1");
|
||||
exit;
|
||||
}
|
||||
map{ $_->{'branchcodesuggestedby'}=GetBranchInfo($_->{'branchcodesuggestedby'})->[0]->{'branchname'}} @$suggestions_loop;
|
||||
my $supportlist=GetSupportList();
|
||||
|
@ -118,7 +123,7 @@ $template->param(
|
|||
suggestions_loop => $suggestions_loop,
|
||||
showall => $allsuggestions,
|
||||
"op_$op" => 1,
|
||||
suggestionsview => 1
|
||||
suggestionsview => 1,
|
||||
);
|
||||
|
||||
|
||||
|
|
Loading…
Reference in a new issue