Attachment 156466 Details for Bug 34961 – Bug 34961: Add sort parameter to OPAC search results RSS feed link

[patch] Bug 34961: Add sort parameter to OPAC search results RSS feed link

Bug-34961-Add-sort-parameter-to-OPAC-search-result.patch (text/plain), 3.06 KB, created by David Nind on 2023-10-02 23:49:23 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: David Nind

Created: 2023-10-02 23:49:23 UTC

Size: 3.06 KB

patch

obsolete

>From 97fcf91b87ceb368ac972dcf33c0f624b06f4c7f Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Fri, 29 Sep 2023 18:33:59 +0000
>Subject: [PATCH] Bug 34961: Add sort parameter to OPAC search results RSS feed
> link
>
>This patch corrects two RSS links in the OPAC search results template so
>that they include the correct parameters, including the descending sort
>by acquisition date.
>
>To test, apply the patch and go to the OPAC.
>
>- Perform a catalog search which will return search results.
>- Next to the page heading "Your search returned X results" is an RSS
>  link. Check the link to confirm that it has all the expected
>  parameters:
>
>  opac-search.pl?idx=kw&q=nude&count=50&sort_by=acqdate_dsc&format=rss
>
>- Perform a search which will return no results and check the RSS link
>  on that page. It should be the same.
>
>Signed-off-by: David Nind <david@davidnind.com>
>---
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>index c58dee9e18..b6bf16c468 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>@@ -68,7 +68,7 @@
>                         <p>
>                             [% IF ( searchdesc ) %]
>                                 No results found for that in [% LibraryName | html %] catalog.
>-                                <a href="[% OPACBaseURL | url %]/cgi-bin/koha/opac-search.pl?[% query_cgi | $raw %][% limit_cgi | $raw %]&amp;format=rss" aria-label="Subscribe to this search" class="btn btn-link rsssearchlink noprint">
>+                                <a href="[% OPACBaseURL | url %]/cgi-bin/koha/opac-search.pl?[% query_cgi | $raw %][% limit_cgi | $raw %]&amp;count=[% countrss |uri %]&amp;sort_by=acqdate_dsc&amp;format=rss" aria-label="Subscribe to this search" class="btn btn-link rsssearchlink noprint">
>                                     <i class="fa fa-rss rsssearchicon" aria-hidden="true" title="Subscribe to this search"></i>
>                                 </a>
>                             [% ELSE %]
>@@ -112,7 +112,7 @@
>                     [% IF ( total ) %]
>                         <h1 id="numresults">
>                             Your search returned [% total | html %] results.
>-                            <a href="[% OPACBaseURL | url %]/cgi-bin/koha/opac-search.pl?[% query_cgi | $raw %][% limit_cgi | $raw %]&amp;format=rss" class="btn btn-link rsssearchlink noprint" aria-label="Subscribe to this search">
>+                            <a href="[% OPACBaseURL | url %]/cgi-bin/koha/opac-search.pl?[% query_cgi | $raw %][% limit_cgi | $raw %]&amp;count=[% countrss |uri %]&amp;sort_by=acqdate_dsc&amp;format=rss" class="btn btn-link rsssearchlink noprint" aria-label="Subscribe to this search">
>                                 <i class="fa fa-rss rsssearchicon" aria-hidden="true" title="Subscribe to this search"></i>
>                             </a>
>                         </h1>
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 34961: 156455 | 156466 | 156493