Bug 15868: Ask for confirmation when deleting a MMT action

Test plan:
Create marc modification template
Add an action
Delete it
With this patch you must get a confirmation mesg

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Brendan Gallagher brendan@bywatersolutions.com
This commit is contained in:
Jonathan Druart 2016-03-08 11:07:43 +00:00 committed by Brendan Gallagher
parent fdac3eecf8
commit 6ca12c8a72
2 changed files with 9 additions and 5 deletions

View File

@ -185,9 +185,12 @@ function clearFormElements(divId) {
}
function confirmDeleteAction() {
return confirm( MSG_MMT_CONFIRM_DEL_TEMPLATE_ACTION );
}
function confirmDelete() {
var agree = confirm( MSG_MMT_CONFIRM_DEL_TEMPLATE );
return agree;
return confirm( MSG_MMT_CONFIRM_DEL_TEMPLATE );
}
var modaction_legend_innerhtml;
@ -285,4 +288,4 @@ function setSelectByValue( selectId, value ) {
s.selectedIndex = i;
}
}
}
}

View File

@ -11,7 +11,8 @@
var MSG_MMT_SOURCE_FIELD = _("The source field should be filled.");
var MSG_MMT_EVERY = _("Every");
var MSG_MMT_ALL = _("All");
var MSG_MMT_CONFIRM_DEL_TEMPLATE= _("Are you sure you wish to delete this template?");
var MSG_MMT_CONFIRM_DEL_TEMPLATE = _("Are you sure you wish to delete this template?");
var MSG_MMT_CONFIRM_DEL_TEMPLATE_ACTION = _("Are you sure you wish to delete this template action?");
var MSG_MMT_EDIT_ACTION = _("Edit action %s");
var MSG_MMT_UPDATE_ACTION = _("Update action");
//]]>
@ -162,7 +163,7 @@
"[% ActionsLoo.conditional_regex |replace('\\\\', '\\\\') |replace("'", "\'") |replace('"', '\"') |replace('\n', '\\n') |replace('\r', '\\r') %]",
"[% ActionsLoo.description |replace('\\\\', '\\\\') |replace("'", "\'") |replace('"', '\"') |replace('\n', '\\n') |replace('\r', '\\r') %]"
);updateAllEvery();'>Edit</a></td>
<td><a href="marc_modification_templates.pl?template_id=[% ActionsLoo.template_id %]&op=delete_action&mmta_id=[% ActionsLoo.mmta_id %]">Delete</a></td>
<td><a href="marc_modification_templates.pl?template_id=[% ActionsLoo.template_id %]&op=delete_action&mmta_id=[% ActionsLoo.mmta_id %]" onclick="return confirmDeleteAction();">Delete</a></td>
</tr>
[% END %]
</table>