Bug 15784: Library deletion warning is incorrectly styled

When deleting a library in Administration ->  Libraries and groups, the
deletion confirmation message is not styled with the standard "dialog
alert" <div>. This patch modifies the confirmation markup to match the
standard.

To test, apply the patch and go to Administration -> Libraries and
groups.

- Try deleting a library. The confirmation box should have the standard
  "dialog alert" class.
- Test the 'confirm' and 'cancel' actions and make sure both complete
  correctly.

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
This commit is contained in:
Owen Leonard 2016-02-03 08:40:20 -05:00 committed by Julian Maurice
parent 5edc50888d
commit cfb46e45cb
1 changed files with 11 additions and 4 deletions

View File

@ -164,12 +164,19 @@ tinyMCE.init({
[% END %]
[% IF ( delete_confirm ) %]
<form action="[% action %]" method="post">
<fieldset><legend>Confirm deletion of [% branchname %] ([% branchcode %])?</legend><input type="hidden" name="op" value="delete_confirmed" />
<div class="dialog alert">
<form action="[% action %]" method="post">
<h3>Are you sure you want to delete [% branchname %] ([% branchcode %])?</h3>
<input type="hidden" name="op" value="delete_confirmed" />
<input type="hidden" name="branchcode" value="[% branchcode |html %]" />
<input type="hidden" name="branchname" value="[% branchname |html %]">
<fieldset class="action"><input type="submit" value="Delete Library" /> <a class="cancel" href="/cgi-bin/koha/admin/branches.pl">Cancel</a></fieldset>
</fieldset></form>
<input type="submit" class="approve" value="Yes, delete" />
</form>
<form action="/cgi-bin/koha/admin/branches.pl" method="get">
<input type="submit" class="deny" value="No, do not delete"/>
</form>
</div>
[% END %]
[% IF ( else ) %]