Permalink
Browse files

Bug 6234: Fixing missing biblionumber for links using biblio-default-…

…view.inc

Fixes various pages where the biblionumber was missing in links to the detail page of a record.
Should also fix 6241.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
I made the changes Owen suggested
- added the class="ratiolimit" back
- fixed additional link in waitingreserve.tt
Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
  • Loading branch information...
1 parent 0eaa391 commit 93a18939e8753b578b97cc122010e57a995ae590 Katrin Fischer committed with ranginui Apr 21, 2011
@@ -43,7 +43,7 @@
[% overduesloo.date_due %]
</td>
<td>
- [% INCLUDE 'biblio-default-view.inc' %][% overduesloo.title |html %] [% IF ( overduesloo.subtitle ) %][% overduesloo.subtitle %][% END %]</a> [% IF ( overduesloo.author ) %] by [% overduesloo.author %][% END %]
+ [% INCLUDE 'biblio-default-view.inc' biblionumber = overduesloo.biblionumber %][% overduesloo.title |html %] [% IF ( overduesloo.subtitle ) %][% overduesloo.subtitle %][% END %]</a> [% IF ( overduesloo.author ) %] by [% overduesloo.author %][% END %]
<br />Barcode : [% overduesloo.barcode %]
</td>
@@ -120,7 +120,7 @@
[% IF ( overdueloo.email ) %][<a href="mailto:[% overdueloo.email %]?subject=Overdue: [% overdueloo.title |html %]">email</a>][% END %]
([% IF ( overdueloo.phone ) %][% overdueloo.phone %][% ELSIF ( overdueloo.mobile ) %][% overdueloo.mobile %][% ELSIF ( overdueloo.phonepro ) %][% overdueloo.phonepro %][% END %])</td>
<td>[% overdueloo.branchcode %]</td>
- <td>[% INCLUDE 'biblio-default-view.inc' %][% overdueloo.title |html %] [% overdueloo.subtitle %]</a> [% IF ( overdueloo.author ) %], by [% overdueloo.author %][% END %]
+ <td>[% INCLUDE 'biblio-default-view.inc' biblionumber = overdueloo.biblionumber %][% overdueloo.title |html %] [% overdueloo.subtitle %]</a> [% IF ( overdueloo.author ) %], by [% overdueloo.author %][% END %]
</td>
<td><a href="/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=[% overdueloo.biblionumber %]&amp;itemnumber=[% overdueloo.itemnum %]#item[% overdueloo.itemnum %]">[% overdueloo.barcode %]</a></td>
<td>[% overdueloo.itemcallnumber %]</td>
@@ -118,7 +118,7 @@ $.tablesorter.addParser({
<td>[% reserveloo.rcount %]</td>
<td>
<p>
- [% INCLUDE 'biblio-default-view.inc' %]
+ [% INCLUDE 'biblio-default-view.inc' biblionumber = reserveloo.biblionumber %]
[% reserveloo.title |html %] [% reserveloo.subtitle %]
</a>
</p>
@@ -91,7 +91,7 @@ $.tablesorter.addParser({
<td><p>[% reserveloo.reservecount %]</p></td>
<td><p>[% reserveloo.itemcount %]</p></td>
<td><p class="ratiolimit">[% reserveloo.thisratio %]</p></td>
- <td>[% INCLUDE 'biblio-default-view.inc' %][% reserveloo.title |html %] [% IF ( reserveloo.subtitle ) %][% FOREACH subtitl IN reserveloo.subtitle %][% subtitl.subfield %][% END %][% END %]</a>[% IF ( reserveloo.author ) %] by [% reserveloo.author %][% END %]
+ <td> [% INCLUDE 'biblio-default-view.inc' biblionumber = reserveloo.biblionumber %][% reserveloo.title |html %] [% IF ( reserveloo.subtitle ) %][% FOREACH subtitl IN reserveloo.subtitle %][% subtitl.subfield %][% END %][% END %]</a>[% IF ( reserveloo.author ) %] by [% reserveloo.author %][% END %]
</td>
<td><p>[% reserveloo.listbranch %]</p></td>
<td><p>[% reserveloo.location %]</p></td>
@@ -59,7 +59,7 @@ $(document).ready(function() {
[% END %]
<td><p>[% reser.datetransfer %]</p> [% IF ( reser.messcompa ) %]<span class="error">Transfer is [% reser.diff %] days late</span>[% END %]</td>
<td><span style="display:none;">[% reser.title |html %]</span><!-- invisible title for tablesorter -->
- [% INCLUDE 'biblio-default-view.inc' %][% reser.title |html %] [% IF ( reser.subtitle ) %] [% FOREACH subtitl IN reser.subtitle %][% subtitl.subfield %][% END %][% END %]</a> [% IF ( reser.author ) %]by [% reser.author %][% END %]
+ [% INCLUDE 'biblio-default-view.inc' biblionumber = reser.biblionumber %][% reser.title |html %] [% IF ( reser.subtitle ) %] [% FOREACH subtitl IN reser.subtitle %][% subtitl.subfield %][% END %][% END %]</a> [% IF ( reser.author ) %]by [% reser.author %][% END %]
[% IF ( reser.itemtype ) %]&nbsp; (<b>[% reser.itemtype %]</b>)[% END %]
<br />Barcode: [% reser.barcode %]
</td>
@@ -84,7 +84,7 @@ $.tablesorter.addParser({
<tbody>[% FOREACH reserveloo IN reserveloop %]
<tr>
<td><p>[% reserveloo.waitingdate %]</p></td>
- <td>[% INCLUDE 'biblio-default-view.inc' %]
+ <td>[% INCLUDE 'biblio-default-view.inc' biblionumber = reserveloo.biblionumber %]
[% reserveloo.title |html %] [% reserveloo.subtitle |html %]
</a>
&nbsp; (<b>[% reserveloo.itemtype %]</b>)
@@ -129,7 +129,7 @@ $.tablesorter.addParser({
<tbody>[% FOREACH overloo IN overloop %]
<tr>
<td><p>[% overloo.waitingdate %]</p></td>
- <td>[% INCLUDE 'biblio-default-view.inc' %][% overloo.title |html %] [% overloo.subtitle |html %]
+ <td>[% INCLUDE 'biblio-default-view.inc' biblionumber = overloo.biblionumber %][% overloo.title |html %] [% overloo.subtitle |html %]
</a>
[% UNLESS ( item_level_itypes ) %][% IF ( overloo.itemtype ) %]&nbsp; (<b>[% overloo.itemtype %]</b>)[% END %][% END %]
<br />Barcode: [% overloo.barcode %]
@@ -196,7 +196,7 @@ function placeHold () {
[% UNLESS ( noItemTypeImages ) %]<img src="[% itemsloo.imageurl %]" alt="[% itemsloo.description %]" title="[% itemsloo.description %]" />[% END %][% itemsloo.description %]
</td>[% END %]
<td>
- [% INCLUDE 'biblio-default-view.inc' %]
+ [% INCLUDE 'biblio-default-view.inc' biblionumber = itemsloo.biblionumber %]
[% itemsloo.title |html %][% FOREACH subtitl IN itemsloo.subtitle %] [% subtitl.subfield %][% END %]</a>
</td>
<td>[% itemsloo.author %]</td>

0 comments on commit 93a1893

Please sign in to comment.