Bug 3606: fixes 'last 50 items only' link in members/readingrec.tmpl.
If a staff member clicked on 'Show all items' in a patron's reading record, and then clicked on 'show last 50 items only,' all of the records would still be shown. The link for the 'last 50' items contains a switch for 'limit=full' instead of 'limit=50'. Also fixes some XHTML errors. Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
This commit is contained in:
parent
5de6ec3c15
commit
5056cd1f53
1 changed files with 3 additions and 3 deletions
|
@ -17,12 +17,12 @@
|
||||||
<h1>Reading Record</h1>
|
<h1>Reading Record</h1>
|
||||||
<form action="/cgi-bin/koha/members/readingrec.pl" method="get"><input type="hidden" name="borrowernumber" id="borrowernumber" value="<!-- TMPL_VAR NAME="borrowernumber" -->" /></form>
|
<form action="/cgi-bin/koha/members/readingrec.pl" method="get"><input type="hidden" name="borrowernumber" id="borrowernumber" value="<!-- TMPL_VAR NAME="borrowernumber" -->" /></form>
|
||||||
<p><!-- TMPL_IF NAME="showfulllink" -->Showing All Items |
|
<p><!-- TMPL_IF NAME="showfulllink" -->Showing All Items |
|
||||||
<a href="readingrec.pl?borrowernumber=<!-- TMPL_VAR name="borrowernumber" -->&limit=full">Show Last 50 Items Only</a> <!-- TMPL_ELSE --> Showing Last 50 Items |
|
<a href="readingrec.pl?borrowernumber=<!-- TMPL_VAR name="borrowernumber" -->&limit=50">Show Last 50 Items Only</a> <!-- TMPL_ELSE --> Showing Last 50 Items |
|
||||||
<a href="readingrec.pl?borrowernumber=<!-- TMPL_VAR name="borrowernumber" -->&limit=full">Show All Items</a><!-- /TMPL_IF --></p>
|
<a href="readingrec.pl?borrowernumber=<!-- TMPL_VAR name="borrowernumber" -->&limit=full">Show All Items</a><!-- /TMPL_IF --></p>
|
||||||
<table>
|
<table>
|
||||||
<tr>
|
<tr>
|
||||||
<th>
|
<th>
|
||||||
<a href="readingrec.pl?borrowernumber=<!-- TMPL_VAR name="borrowernumber" -->&order=title&amp;limit=<!-- TMPL_VAR name="limit" -->">Title</a>
|
<a href="readingrec.pl?borrowernumber=<!-- TMPL_VAR name="borrowernumber" -->&order=title&limit=<!-- TMPL_VAR name="limit" -->">Title</a>
|
||||||
</th>
|
</th>
|
||||||
<th>
|
<th>
|
||||||
<a href="readingrec.pl?borrowernumber=<!-- TMPL_VAR name="borrowernumber" -->&order=author&limit=<!-- TMPL_VAR name="limit" -->">Author</a>
|
<a href="readingrec.pl?borrowernumber=<!-- TMPL_VAR name="borrowernumber" -->&order=author&limit=<!-- TMPL_VAR name="limit" -->">Author</a>
|
||||||
|
@ -32,7 +32,7 @@
|
||||||
<th>Number of Renewals</th>
|
<th>Number of Renewals</th>
|
||||||
<th>Date Due</th>
|
<th>Date Due</th>
|
||||||
<th>
|
<th>
|
||||||
<a href="readingrec.pl?borrowernumber=<!-- TMPL_VAR name="borrowernumber" -->&order=returndate DESC&limit=<!-- TMPL_VAR name="limit" -->">Return Date</a>
|
<a href="readingrec.pl?borrowernumber=<!-- TMPL_VAR name="borrowernumber" -->&order=returndate%20DESC&limit=<!-- TMPL_VAR name="limit" -->">Return Date</a>
|
||||||
</th>
|
</th>
|
||||||
</tr>
|
</tr>
|
||||||
<!-- TMPL_LOOP name="loop_reading" -->
|
<!-- TMPL_LOOP name="loop_reading" -->
|
||||||
|
|
Loading…
Reference in a new issue