@@ -, +, @@ - Do a search with multiple results - Go to the detail page (make sure results browser shows up!) - Use the "Edit items" link from the toolbar - Delete item (try both way) - Choose "cancel" --- .../prog/en/modules/cataloguing/additem.tt | 43 +++++++++++----------- 1 file changed, 22 insertions(+), 21 deletions(-) --- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt @@ -21,17 +21,24 @@ $(document).ready(function(){ [% END %] [% END %] $("fieldset.rows input, fieldset.rows select").addClass("noEnterSubmit"); - /* Inline edit/delete links */ - var biblionumber = $("input[name='biblionumber']").attr("value"); - $("tr.editable td").click(function(event){ - var $tgt = $(event.target); - if($tgt.is("a")||$tgt.is(":first-child")||$tgt.is(":nth-child(2)")){ return true; } else { - var rowid = $(this).parent().attr("id"); - num_rowid = rowid.replace("row",""); - $(".linktools").remove(); - $(this).append("" + _("Edit item") + " " + _("Delete item") + ""); - } - }); + /* Inline edit/delete links */ + var biblionumber = $("input[name='biblionumber']").attr("value"); + $("tr.editable").each(function(){ + $(this).find("td:not(:first)").on('click', function(){ + var rowid = $(this).parent().attr("id"); + num_rowid = rowid.replace("row",""); + $(".linktools").remove(); + var edit_link = $('' + _("Edit item") + ''); + var delete_link = $('' + _("Delete item") + ''); + $(delete_link).on('click', function(e) { + return confirm_deletion(); + }); + var tools_node = $(''); + $(tools_node).append(edit_link); + $(tools_node).append(delete_link); + $(this).append(tools_node); + }); + }); var itemst = $("#itemst").dataTable($.extend(true, {}, dataTablesDefaults, { "aoColumnDefs": [ @@ -79,16 +86,10 @@ function Dopop(link,i) { newin=window.open(link+"&result="+defaultvalue,"valuebuilder",'width=500,height=400,toolbar=false,scrollbars=yes'); } -function confirm_deletion(biblionumber,itemnumber) { - var original = $("#row"+itemnumber).attr("class"); - $("#row"+itemnumber).attr("class","confirm"); - var is_confirmed = confirm(_("Are you sure you want to delete this item?")); - if (is_confirmed) { - window.location = "additem.pl?op=delitem&biblionumber="+biblionumber+"&itemnumber="+itemnumber; - } else { - $("#row"+itemnumber).attr("class",""); - } +function confirm_deletion() { + return confirm(_("Are you sure you want to delete this item?")); } + //]]> @@ -161,7 +162,7 @@ function confirm_deletion(biblionumber,itemnumber) { [% IF ( item_loo.countanalytics ) %] View analytics [% ELSE %] - Delete + Delete [% END %] [% END %] --