If there are over 50 items in the holdings or Other holdings tab, the
warning and link 'This record has many physical items. Click here to
view them all.' is only shown for the first tab.
Test plan:
- Switch on the OpacSeparateHoldings pref.
- Go on a biblio detail page at the OPAC with more than 50 items
- Check that the 'view all' link appears on the second tab.
Signed-off-by: Srdjan <srdjan@catalyst.net.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, passes all tests and QA script.
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
<div id="holdings">
[% IF ( itemloop.size ) %]
<div id="holdings">
[% IF ( itemloop.size ) %]
- [% IF ( lotsofitems ) %]
- <p>This record has many physical items. <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% biblionumber %]&viewallitems=1#holdings">Click here to view them all.</a></p>
+ [% IF ( lotsofholdingsitems ) %]
+ <p>This record has many physical items. <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% biblionumber %]&viewallitems=1#holdings">Click here to view them all.</a></p>
[% ELSE %]
[% INCLUDE items_table items=itemloop tab="holdings" %]
[% END %]
[% ELSE %]
[% INCLUDE items_table items=itemloop tab="holdings" %]
[% END %]
[% IF (SeparateHoldings) %]
<div id="otherholdings">
[% IF (otheritemloop.size) %]
[% IF (SeparateHoldings) %]
<div id="otherholdings">
[% IF (otheritemloop.size) %]
- [% INCLUDE items_table items=otheritemloop tab="otherholdings" %]
+ [% IF lotsofothersholdingsitems %]
+ <p>This record has many physical items. <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% biblionumber %]&viewallitems=1#otherholdings">Click here to view them all.</a></p>
+ [% ELSE %]
+ [% INCLUDE items_table items=otheritemloop tab="otherholdings" %]
+ [% END %]
[% ELSE %]
No other items.
[% END %]
[% ELSE %]
No other items.
[% END %]
# TODO: The limit of 50 could be a syspref
my $viewallitems = $query->param('viewallitems');
if (scalar(@itemloop) >= 50 && !$viewallitems) {
# TODO: The limit of 50 could be a syspref
my $viewallitems = $query->param('viewallitems');
if (scalar(@itemloop) >= 50 && !$viewallitems) {
- $template->param('lotsofitems' => 1);
+ $template->param('lotsofholdingsitems' => 1);
+}
+if (scalar(@otheritemloop) >= 50 && !$viewallitems) {
+ $template->param('lotsofothersholdingsitems' => 1);
}
## get notes and subjects from MARC record
}
## get notes and subjects from MARC record