Fix for Bug 4218 - Staff client detail page does not show item hold status

Showing "on hold" in the holdings list (as we used to do) is misleading
if it is a bibilo-level hold. However seeing that a title has holds
is important information for this view. This patch adds a line to the
biblio-level information display with a count of the title's outstanding
holds.

Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
This commit is contained in:
Owen Leonard 2010-09-21 10:45:03 -04:00 committed by Chris Cormack
parent eaddd603e4
commit 6466ac847f
3 changed files with 8 additions and 2 deletions

View file

@ -102,6 +102,9 @@ my $dbh = C4::Context->dbh;
my @items = &GetItemsInfo( $biblionumber, 'intra' );
my $dat = &GetBiblioData($biblionumber);
# get count of holds
my ( $holdcount, $holds ) = GetReservesFromBiblionumber($biblionumber,1);
#coping with subscriptions
my $subscriptionsnumber = CountSubscriptionFromBiblionumber($biblionumber);
my @subscriptions = GetSubscriptions( $dat->{title}, $dat->{issn}, $biblionumber );
@ -215,6 +218,7 @@ $template->param(
volinfo => $itemfields{enumchron},
itemdata_itemnotes => $itemfields{itemnotes},
z3950_search_params => C4::Search::z3950_search_args($dat),
holdcount => $holdcount,
C4::Search::enabled_staff_search_views,
);

View file

@ -1901,3 +1901,6 @@ ul.budget_hierarchy li:last-child:after {
ul.budget_hierarchy li:first-child:after {
content: "";
}
.holdcount { font-size : 105%; line-height : 200%; }
.holdcount a { border : 1px solid #a4bedd; background-color : #e4ecf5; font-weight : bold; -moz-border-radius: 4px; padding : .1em .4em; text-decoration : none; }
.holdcount a:hover { background-color : #ebeff7; }

View file

@ -195,11 +195,10 @@ function verify_images() {
</ul>
</li>
<!-- /TMPL_IF -->
<!-- TMPL_IF NAME="holdcount" --><li><strong>Holds:</strong> <span class="holdcount"><a href="/cgi-bin/koha/reserve/request.pl?biblionumber=<!-- TMPL_VAR NAME="biblionumber" -->"><!-- TMPL_VAR NAME="holdcount" --></a></span></li><!-- TMPL_ELSE --><!-- /TMPL_IF -->
</ul>
<!-- /TMPL_IF -->
</div>
</div>
<div id="bibliodetails" class="toptabs">
<ul>