From fb5412c529a64ee8ccbe0acf14213ec07d14559f Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Wed, 25 Aug 2010 14:36:20 -0400 Subject: [PATCH] Fix for Bug 4849 - Cannot delete records from Labeled MARC view The template checks for a "count" variable (item count) to see whether it should allow deletion of the biblio record. For some reason labeledMARCdetail.pl hard-coded this as 1, which meant the template always thought was 1 item. This patch also adds an item count check on the display of "Delete all items," disabling it if there are no items. Signed-off-by: Galen Charlton --- catalogue/labeledMARCdetail.pl | 6 +++++- koha-tmpl/intranet-tmpl/prog/en/includes/cat-toolbar.inc | 2 +- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/catalogue/labeledMARCdetail.pl b/catalogue/labeledMARCdetail.pl index de67677fb2..e72a6d115d 100755 --- a/catalogue/labeledMARCdetail.pl +++ b/catalogue/labeledMARCdetail.pl @@ -25,6 +25,7 @@ use C4::Auth; use C4::Context; use C4::Output; use C4::Biblio; +use C4::Items; use C4::Search; # enabled_staff_search_views my $query = new CGI; @@ -51,7 +52,10 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user( } ); -$template->param( count => 1, bibliotitle => $biblio->{title} ); +#count of item linked +my $itemcount = GetItemsCount($biblionumber); +$template->param( count => $itemcount, + bibliotitle => $biblio->{title}, ); #Getting the list of all frameworks my $queryfwk = diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/cat-toolbar.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/cat-toolbar.inc index eda798548e..228ebf35eb 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/cat-toolbar.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/cat-toolbar.inc @@ -87,7 +87,7 @@ function confirm_items_deletion() { { text: _("Attach Item"), url: "/cgi-bin/koha/cataloguing/moveitem.pl?biblionumber=" }, { text: _("Edit as New (Duplicate)"), url: "/cgi-bin/koha/cataloguing/addbiblio.pl?biblionumber=&frameworkcode=&op=duplicate" }, { text: _("Delete Record"), onclick: {fn: confirm_deletion },id:'disabled' }, - { text: _("Delete all Items"), onclick: {fn: confirm_items_deletion } } + { text: _("Delete all Items"), onclick: {fn: confirm_items_deletion },id:'disabled' } ]; var savemenu = [ -- 2.39.5