From c11d443d04a2b89c4336a84740e4e9e727bda866 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Thu, 9 May 2013 15:08:41 -0400 Subject: [PATCH] [SIGNED-OFF] Bug 10225 - If an item isn't editable, don't show the onclick Edit/Delete menu The user should not be shown edit/delete links when clicking on a row in additems which they cannot edit because of the IndependantBranches setting. This patch adds a class to the row to exclude such rows in the JavaScript onclick handler. To test: - Turn IndependantBranches on and edit items for a title which has holdings from various branches including your own. - Click a row for an item which is held by your library. You should see the "Edit/Delete" links appear. - Click a row for an item which is held by another library. No links should appear. Signed-off-by: Nicole C. Engard All test pass --- .../prog/en/modules/cataloguing/additem.tt | 20 ++++++++++++++------ 1 files changed, 14 insertions(+), 6 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt index 97d06a5..1043965 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt @@ -13,7 +13,7 @@ $(document).ready(function(){ $("fieldset.rows input").keydown(function(e){ return checkEnter(e); }); /* Inline edit/delete links */ var biblionumber = $("input[name='biblionumber']").attr("value"); - $("td").click(function(event){ + $("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"); @@ -134,11 +134,19 @@ $(document).ready(function() { [% END %] [% FOREACH item_loo IN item_loop %] - [% IF ( item_loo.itemnumber == itemnumber) %] - - [% ELSE %] - - [% END %] + [% IF ( item_loo.itemnumber == itemnumber) %] + [% IF item_loo.nomod %] + + [% ELSE %] + + [% END %] + [% ELSE %] + [% IF item_loo.nomod %] + + [% ELSE %] + + [% END %] + [% END %] [% IF ( item_loo.nomod ) %]  [% ELSE %][% IF ( item_loo.hostitemflag ) %]Edit in host Delink [% ELSE %]Edit -- 1.7.2.3