Bug 28453: Adjust OPAC pagination

Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
This commit is contained in:
Jonathan Druart 2025-03-10 09:51:28 +01:00 committed by Katrin Fischer
parent 9c44fe1257
commit c73e0b30bd
Signed by: kfischer
GPG key ID: 0EF6E2C03357A834
3 changed files with 5 additions and 4 deletions

View file

@ -46,7 +46,7 @@
<h1>Authority search results</h1>
[% IF total %]
<div class="pages">[% pagination_bar | $raw %]</div>
<div class="pages" id="pagination_top"> [% pagination_bar | $raw %] </div>
[% END %]
<div id="results">
[% IF ( total ) %]
@ -103,7 +103,7 @@
</div>
<!-- / .searchresults -->
<div class="pages">[% pagination_bar | $raw %]</div>
<div class="pages" id="pagination_bottom"> [% pagination_bar | $raw %] </div>
[% END # / IF total %]
</div>
<!-- / #userauthsearchresults -->

View file

@ -830,7 +830,7 @@
</tbody>
</table>
<!-- /.table -->
<div class="pages">[% pagination_bar | $raw %]</div>
<div class="pages" id="pagination_bottom"> [% pagination_bar | $raw %] </div>
[% ELSE %]
[% IF public %]
<p>No public lists.</p>

View file

@ -293,7 +293,8 @@ subtest 'XSS vulnerabilities in pagination' => sub {
$s->add_error_handler;
is( $alert_text, undef, 'No alert box displayed, even if evil intent' );
my $second_page = $driver->find_element('//div[@class="pages"]/span[@class="currentPage"]/following-sibling::a');
my $second_page =
$driver->find_element('//div[@class="pages"]/ul/li[@class="page-item active"]/following-sibling::li/a');
like(
$second_page->get_attribute('href'), qr{(?|&)public=1(&|$)},
'The second page should display category without the invalid value'