Bug 4912: Redirect to the appropriate view after editing/deleting a list
There are 2 places where a list can be edited/deleted: on the list view and the list content view. After the edition, the user expect to be redirect to the previous page. This patch implements that. Test plan: At the OPAC, delete and edit a list from the 2 differents places. Confirm that you are redirect to the page you come from. With patch, redirects work as expected. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Brendan A Gallagher <brendan@bywatersolutions.com>
This commit is contained in:
parent
2923cfdff4
commit
6a2bf8d31b
2 changed files with 6 additions and 3 deletions
|
@ -177,6 +177,7 @@
|
|||
<span class="sep">|</span>
|
||||
<form method="get" action="/cgi-bin/koha/opac-shelves.pl" class="form-inline">
|
||||
<input type="hidden" name="op" value="edit_form" />
|
||||
<input type="hidden" name="referer" value="view" />
|
||||
<input type='hidden' name='category' value='[% category %]' />
|
||||
<input type="hidden" name="shelfnumber" value="[% shelf.shelfnumber | html %]" />
|
||||
<input type="submit" class="editshelf" value="Edit list" />
|
||||
|
@ -184,6 +185,7 @@
|
|||
|
||||
<form method="post" action="/cgi-bin/koha/opac-shelves.pl" class="form-inline">
|
||||
<input type="hidden" name="op" value="delete" />
|
||||
<input type="hidden" name="referer" value="list" />
|
||||
<input type='hidden' name='category' value='[% category %]' />
|
||||
<input type="hidden" name="shelfnumber" value="[% shelf.shelfnumber | html %]" />
|
||||
<input type="submit" class="deleteshelf" value="Delete list" onclick="return confirmDelete(MSG_CONFIRM_DELETE_LIST);"/>
|
||||
|
@ -194,6 +196,7 @@
|
|||
[% ELSIF category == PRIVATE # not manageshelf and private means shared %]
|
||||
<form action="/cgi-bin/koha/opac-shelves.pl" method="post" class="form-inline">
|
||||
<input type="hidden" name="op" value="remove_share" />
|
||||
<input type="hidden" name="referer" value="list" />
|
||||
<input type='hidden' name='category' value='[% category %]' />
|
||||
<input type="hidden" name="shelfnumber" value="[% shelf.shelfnumber | html %]" />
|
||||
<input type="submit" class="deleteshelf" onclick="return confirmDelete(MSG_CONFIRM_REMOVE_SHARE);" value="Remove share" />
|
||||
|
@ -545,7 +548,7 @@
|
|||
[% ELSE %]
|
||||
<legend>Editing <em>[% shelf.shelfname |html %]</em></legend>
|
||||
<input type="hidden" name="op" value="edit" />
|
||||
<input type="hidden" name="referer" value="view" />
|
||||
<input type="hidden" name="referer" value="[% referer %]" />
|
||||
<input type="hidden" name="shelfnumber" value="[% shelf.shelfnumber | html %]" />
|
||||
[% END %]
|
||||
<input type="hidden" name="owner" id="owner" value="[% loggedinusernumber %]" />
|
||||
|
|
|
@ -90,11 +90,11 @@ if ( $op eq 'add_form' ) {
|
|||
push @messages, { type => 'error', code => 'error_on_insert' };
|
||||
} else {
|
||||
push @messages, { type => 'message', code => 'success_on_insert' };
|
||||
$op = 'view';
|
||||
$op = $referer;
|
||||
}
|
||||
} else {
|
||||
push @messages, { type => 'error', code => 'unauthorized_on_insert' };
|
||||
$op = 'list';
|
||||
$op = $referer;
|
||||
}
|
||||
} elsif ( $op eq 'edit' ) {
|
||||
$shelfnumber = $query->param('shelfnumber');
|
||||
|
|
Loading…
Reference in a new issue