Bug 15853 - Add author and link columns to opac course reserves table

To test:
1 - Add some course reserves
2 - View the table in staff and opac, there is no column for author or
link
3 - Apply patch
4 - You shoudl see these columns
5 - Link should contain item uri if present, fallback to title 856$u,
  then be blank if niether

Sponsored by:
    Southeastern Wisconsin Information Technology Exchange, Inc (SWITCH)
    (http://caspian.switchinc.org/)

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Might be good to add this table to the column visibility tool if you
plan to add more columns.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
This commit is contained in:
Nick Clemens 2016-06-03 14:49:25 -04:00 committed by Kyle M Hall
parent dba6cfe441
commit b89a601e57
2 changed files with 19 additions and 0 deletions

View file

@ -73,6 +73,7 @@
<thead> <thead>
<tr> <tr>
<th class="antithe">Title</th> <th class="antithe">Title</th>
<th>Author</th>
<th>Barcode</th> <th>Barcode</th>
<th>Call number</th> <th>Call number</th>
[% IF item_level_itypes %]<th>Item type</th>[% END %] [% IF item_level_itypes %]<th>Item type</th>[% END %]
@ -84,6 +85,7 @@
[% IF CAN_user_coursereserves_add_reserves %]<th class="NoSort">&nbsp;<!-- Edit --></th>[% END %] [% IF CAN_user_coursereserves_add_reserves %]<th class="NoSort">&nbsp;<!-- Edit --></th>[% END %]
[% IF CAN_user_coursereserves_delete_reserves %]<th class="NoSort">&nbsp;<!-- Remove --></th>[% END %] [% IF CAN_user_coursereserves_delete_reserves %]<th class="NoSort">&nbsp;<!-- Remove --></th>[% END %]
<th class="NoSort">Other course reserves</th> <th class="NoSort">Other course reserves</th>
<th>Link</th>
</tr> </tr>
</thead> </thead>
@ -91,6 +93,7 @@
[% FOREACH cr IN course_reserves %] [% FOREACH cr IN course_reserves %]
<tr> <tr>
<td><a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% cr.item.biblionumber %]">[% cr.item.title %]</a></td> <td><a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% cr.item.biblionumber %]">[% cr.item.title %]</a></td>
<td>[% cr.item.author %]</td>
<td><a href="/cgi-bin/koha/catalogue/moredetail.pl?itemnumber=[% cr.item.itemnumber %]&amp;biblionumber=[% cr.item.biblionumber %]&amp;bi=[% cr.item.biblioitemnumber %]">[% cr.item.barcode %]</a></td> <td><a href="/cgi-bin/koha/catalogue/moredetail.pl?itemnumber=[% cr.item.itemnumber %]&amp;biblionumber=[% cr.item.biblionumber %]&amp;bi=[% cr.item.biblioitemnumber %]">[% cr.item.barcode %]</a></td>
<td>[% cr.item.itemcallnumber %]</td> <td>[% cr.item.itemcallnumber %]</td>
[% IF item_level_itypes %] [% IF item_level_itypes %]
@ -194,6 +197,13 @@
[% END %] [% END %]
[% END %] [% END %]
</td> </td>
<td>
[% IF (cr.item.uri) %]
<a href="[% cr.item.uri %]">Item URI</a>
[% ELSIF (cr.item.url) %]
<a href="[% cr.item.url %]">Record URL</a>
[% END %]
</td>
</tr> </tr>
[% END %] [% END %]
</tbody> </tbody>

View file

@ -50,6 +50,7 @@
<thead> <thead>
<tr> <tr>
<th class="anti-the">Title</th> <th class="anti-the">Title</th>
<th>Author</th>
<th>Item type</th> <th>Item type</th>
<th>Location</th> <th>Location</th>
<th>Collection</th> <th>Collection</th>
@ -58,6 +59,7 @@
<th>Status</th> <th>Status</th>
<th class="title-string">Date due</th> <th class="title-string">Date due</th>
<th>Notes</th> <th>Notes</th>
<th>Link</th>
</tr> </tr>
</thead> </thead>
@ -65,6 +67,7 @@
[% FOREACH cr IN course_reserves %] [% FOREACH cr IN course_reserves %]
<tr> <tr>
<td><a href="opac-detail.pl?biblionumber=[% cr.item.biblionumber %]">[% cr.item.title %]</a></td> <td><a href="opac-detail.pl?biblionumber=[% cr.item.biblionumber %]">[% cr.item.title %]</a></td>
<td>[% cr.item.author %]</td>
<td>[% ItemTypes.GetDescription( cr.item.itype ) %]</td> <td>[% ItemTypes.GetDescription( cr.item.itype ) %]</td>
<td>[% Branches.GetName( cr.item.holdingbranch ) %] <br/> <i>[% AuthorisedValues.GetByCode( 'LOC', cr.item.location ) %]</i></td> <td>[% Branches.GetName( cr.item.holdingbranch ) %] <br/> <i>[% AuthorisedValues.GetByCode( 'LOC', cr.item.location ) %]</i></td>
<td>[% AuthorisedValues.GetByCode( 'CCODE', cr.item.ccode ) %]</td> <td>[% AuthorisedValues.GetByCode( 'CCODE', cr.item.ccode ) %]</td>
@ -78,6 +81,12 @@
[% cr.item.itemnotes %] [% cr.item.itemnotes %]
[% END %] [% END %]
</td> </td>
<td>[% IF (cr.item.uri) %]
<a href="[% cr.item.uri %]">Item URI</a>
[% ELSIF (cr.item.url) %]
<a href="[% cr.item.url %]">Record URL</a>
[% END %]
</td>
</tr> </tr>
[% END %] [% END %]
</tbody> </tbody>