Attachment 77060 Details for Bug 16111 – Bug 16111: (follow-up) Fixes for simplicity and validation

[patch] Bug 16111: (follow-up) Fixes for simplicity and validation

Bug-16111-follow-up-Fixes-for-simplicity-and-valid.patch (text/plain), 19.36 KB, created by Owen Leonard on 2018-07-17 16:58:02 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2018-07-17 16:58:02 UTC

Size: 19.36 KB

patch

obsolete

>From 607d20bf77c09f91e2d116e21c23381fe6078d73 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Tue, 17 Jul 2018 15:54:06 +0000
>Subject: [PATCH] Bug 16111: (follow-up) Fixes for simplicity and validation
>
>This patch simplifies the OPAC search results feed template by adding
>BLOCKs for a couple of repeated sections. The same is done for the feed
>URL in the search results template.
>
>This patch also makes some changes to the markup of the rss and Atom
>feeds to improve their validity.
>
>To test, apply the patch and perform a search in the OPAC. Clicking the
>RSS icon at the top of the search results page should bring up the RSS
>feed, which should render correctly.
>
>If you validate the source it should return only a couple of
>"recommendations:"
> - "Self reference doesn't match document location"
> - "Unregistered link relationship: search"
>
>Perform the same test with the Atom feed by modifying the URL of the RSS
>feed in your browser so that it ends in "format=atom" instead of
>"format=rss"
>
>The Atom feed has some more serious validation issues but should be
>functional.
>---
> .../bootstrap/en/modules/opac-opensearch.tt | 129 ++++++++++++++-------
> .../opac-tmpl/bootstrap/en/modules/opac-results.tt | 10 +-
> 2 files changed, 91 insertions(+), 48 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-opensearch.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-opensearch.tt
>index 07d7cf9..1a4f5d2 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-opensearch.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-opensearch.tt
>@@ -1,5 +1,47 @@
> <?xml version='1.0' encoding='utf-8' ?>
> [% USE Koha %]
>+[% BLOCK item_description %]
>+ <![CDATA[
>+ [% IF ( OPACAmazonCoverImages ) %]
>+ [% IF ( SEARCH_RESULT.normalized_isbn ) %]
>+ <img src="https://images-na.ssl-images-amazon.com/images/P/[% SEARCH_RESULT.normalized_isbn %].01.TZZZZZZZ.jpg" alt="" />
>+ [% END %]
>+ [% END %]
>+ [% IF ( SyndeticsEnabled ) %]
>+ [% IF ( SyndeticsCoverImages ) %]
>+ [% IF ( SEARCH_RESULT.content_identifier_exists ) %]
>+ <img src="http://www.syndetics.com/index.aspx?isbn=[% SEARCH_RESULT.normalized_isbn %]/SC.GIF&amp;client=[% SEARCH_RESULT.SyndeticsClientCode %]&amp;type=xw10[% IF ( SEARCH_RESULT.normalized_upc ) %]&amp;upc=[% SEARCH_RESULT.normalized_upc %][% END %][% IF ( SEARCH_RESULT.normalized_oclc ) %]&amp;oclc=[% SEARCH_RESULT.normalized_oclc %][% END %]" alt="" />
>+ [% ELSE %]
>+ <img src="http://www.syndetics.com/index.aspx?isbn=[% SEARCH_RESULT.normalized_isbn %]/SC.GIF&amp;client=[% SyndeticsClientCode %]&amp;type=xw10&amp;upc=[% SEARCH_RESULT.normalized_upc %]&amp;oclc=[% SEARCH_RESULT.normalized_oclc %]" alt="" />
>+ [% END %]
>+ [% END %]
>+ [% END %]
>+ [% bt_id = ( SEARCH_RESULT.normalized_upc || SEARCH_RESULT.normalized_isbn ) %]
>+ [% IF ( BakerTaylorEnabled ) %]
>+ [% IF bt_id %]
>+ <a href="https://[% BakerTaylorBookstoreURL |url %][% bt_id %]"><img alt="See Baker &amp; Taylor" src="[% BakerTaylorImageURL |url %][% bt_id %]" /></a>
>+ [% END %]
>+ [% END %]
>+ 
>+ [% IF ( SEARCH_RESULT.author ) %]By [% SEARCH_RESULT.author |html %]. [% END %]
>+ [% IF ( SEARCH_RESULT.place ) %][% SEARCH_RESULT.place |html %] [% END %]
>+ [% IF ( SEARCH_RESULT.publishercode ) %][% SEARCH_RESULT.publishercode |html %][% END %]
>+ [% IF ( SEARCH_RESULT.publicationyear ) %]
>+ [% SEARCH_RESULT.publicationyear |html %]
>+ [% ELSE %]
>+ [% IF ( SEARCH_RESULT.copyrightdate ) %] [% SEARCH_RESULT.copyrightdate |html %][% END %]
>+ [% END %]
>+ [% IF ( SEARCH_RESULT.pages ) %]. [% SEARCH_RESULT.pages |html %][% END %]
>+ [% IF ( SEARCH_RESULT.notes ) %], [% SEARCH_RESULT.notes |html %][% END %]
>+ [% IF ( SEARCH_RESULT.size ) %] [% SEARCH_RESULT.size |html %]. [% END %]
>+ [% IF ( SEARCH_RESULT.isbn ) %] [% SEARCH_RESULT.isbn |html %][% END %]
>+ 
>+ [% IF Koha.Preference( 'RequestOnOpac' ) == 1 %]
>+ <a href="[% OPACBaseURL %]/cgi-bin/koha/opac-reserve.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]">Place Hold on [% SEARCH_RESULT.title %]</a>
>+ [% END %]
>+ ]]>
>+[% END %]
>+[% BLOCK feed_url %][% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% query_cgi | html %][% limit_cgi |uri %]&amp;count=[% countrss |uri %]&amp;offset=[% offset %]&amp;sort_by=acqdate_dsc&amp;format=[% END %]
> [% IF ( opensearchdescription ) %]
> <OpenSearchDescription xmlns="http://a9.com/-/spec/opensearch/1.1/">
> <ShortName>[% LibraryName |html %] Search</ShortName>
>@@ -18,10 +60,10 @@
> xmlns:dc="http://purl.org/dc/elements/1.1/"
> xmlns:atom="http://www.w3.org/2005/Atom">
> <channel>
>- <title><![CDATA[[% LibraryName |html %] Search [% IF ( query_desc ) %]for '[% query_desc |html %]'[% END %][% IF ( limit_desc ) %] with limit(s): '[% limit_desc |html %]'[% END %]]]></title>
>- <link>[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% query_cgi |html %][% limit_cgi |html %]&amp;format=rss</link>
>- <atom:link rel="self" type="application/rss+xml" href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% query_cgi |uri %][% limit_cgi |uri %]&amp;sort_by=[% sort_by | uri %]&amp;format=atom"/>
>- <description><![CDATA[ Search results [% IF ( query_desc ) %]for '[% query_desc |html %]'[% END %][% IF ( limit_desc ) %] with limit(s): '[% limit_desc |html %]'[% END %] at [% LibraryName |html %]]]></description>
>+ <title><![CDATA[[% LibraryName |html %] Search [% IF ( query_desc ) %]for '[% query_desc %]'[% END %][% IF ( limit_desc ) %] with limit(s): '[% limit_desc |html %]'[% END %]]]></title>
>+ <link>[% PROCESS feed_url %]rss</link>
>+ <atom:link rel="self" type="application/rss+xml" href="[% PROCESS feed_url %]rss"/>
>+ <description><![CDATA[ Search results [% IF ( query_desc ) %]for '[% query_desc %]'[% END %][% IF ( limit_desc ) %] with limit(s): '[% limit_desc |html %]'[% END %] at [% LibraryName |html %]]]></description>
> <opensearch:totalResults>[% total %]</opensearch:totalResults>
> <opensearch:startIndex>[% offset %]</opensearch:startIndex>
> [% IF ( results_per_page ) %]
>@@ -29,34 +71,26 @@
> [% ELSE %]
> <opensearch:itemsPerPage>20</opensearch:itemsPerPage>
> [% END %]
>-	 [% FOREACH SEARCH_RESULT IN SEARCH_RESULTS %]
> <atom:link rel="search" type="application/opensearchdescription+xml" href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% SEARCH_RESULT.query_cgi |uri %][% SEARCH_RESULT.limit_cgi |uri %]&amp;sort_by=[% SEARCH_RESULT.sort_by |uri %]&amp;format=opensearchdescription"/>
> <opensearch:Query role="request" searchTerms="[% SEARCH_RESULT.query_desc |uri %][% SEARCH_RESULT.limit_desc |uri %]" startPage="[% SEARCH_RESULT.page %]" />
>+ [% FOREACH SEARCH_RESULT IN SEARCH_RESULTS %]
> <item>
> <title>[% SEARCH_RESULT.title |html %] [% FOREACH subtitl IN SEARCH_RESULT.subtitle %], [% subtitl.subfield |html %][% END %]</title>
> <dc:identifier>ISBN [% SEARCH_RESULT.isbn |html %]</dc:identifier>
>- <link>[% IF ( SEARCH_RESULT.BiblioDefaultViewmarc ) %][% OPACBaseURL %]/cgi-bin/koha/opac-MARCdetail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %][% ELSE %][% IF ( SEARCH_RESULT.BiblioDefaultViewisbd ) %][% OPACBaseURL %]/cgi-bin/koha/opac-ISBDdetail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %][% ELSE %][% OPACBaseURL %]/cgi-bin/koha/opac-detail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %][% END %][% END %]</link>
>- <description><![CDATA[
>-[% IF ( OPACAmazonCoverImages ) %][% IF ( SEARCH_RESULT.normalized_isbn ) %]<img src="https://images-na.ssl-images-amazon.com/images/P/[% SEARCH_RESULT.normalized_isbn %].01.TZZZZZZZ.jpg" alt="" />[% END %][% END %]
>-[% IF ( SyndeticsEnabled ) %][% IF ( SyndeticsCoverImages ) %][% IF ( SEARCH_RESULT.content_identifier_exists ) %]<img src="http://www.syndetics.com/index.aspx?isbn=[% SEARCH_RESULT.normalized_isbn %]/SC.GIF&amp;client=[% SEARCH_RESULT.SyndeticsClientCode %]&amp;type=xw10[% IF ( SEARCH_RESULT.normalized_upc ) %]&amp;upc=[% SEARCH_RESULT.normalized_upc %][% END %][% IF ( SEARCH_RESULT.normalized_oclc ) %]&amp;oclc=[% SEARCH_RESULT.normalized_oclc %][% END %]" alt="" />
>-[% ELSE %]
>-<img src="http://www.syndetics.com/index.aspx?isbn=[% SEARCH_RESULT.normalized_isbn %]/SC.GIF&amp;client=[% SyndeticsClientCode %]&amp;type=xw10&amp;upc=[% SEARCH_RESULT.normalized_upc %]&amp;oclc=[% SEARCH_RESULT.normalized_oclc %]" alt="" />
>-[% END %][% END %][% END %]
>-
>-[% bt_id = ( SEARCH_RESULT.normalized_upc || SEARCH_RESULT.normalized_isbn ) %]
>-[% IF ( BakerTaylorEnabled ) %][% IF bt_id %]<a href="https://[% BakerTaylorBookstoreURL |url %][% bt_id %]"><img alt="See Baker &amp; Taylor" src="[% BakerTaylorImageURL |url %][% bt_id %]" /></a>[% END %][% END %]
>-	 [% IF ( SEARCH_RESULT.author ) %]By [% SEARCH_RESULT.author |html %]. [% END %]
>-	 [% IF ( SEARCH_RESULT.place ) %][% SEARCH_RESULT.place |html %] [% END %][% IF ( SEARCH_RESULT.publishercode ) %][% SEARCH_RESULT.publishercode |html %][% END %][% IF ( SEARCH_RESULT.publicationyear ) %] [% SEARCH_RESULT.publicationyear |html %]
>- [% ELSE %][% IF ( SEARCH_RESULT.copyrightdate ) %] [% SEARCH_RESULT.copyrightdate |html %][% END %][% END %]
>- [% IF ( SEARCH_RESULT.pages ) %]. [% SEARCH_RESULT.pages |html %][% END %]
>- [% IF ( SEARCH_RESULT.notes ) %], [% SEARCH_RESULT.notes |html %][% END %]
>- [% IF ( SEARCH_RESULT.size ) %] [% SEARCH_RESULT.size |html %]. [% END %]
>- [% IF ( SEARCH_RESULT.isbn ) %] [% SEARCH_RESULT.isbn |html %][% END %]
>- 
>-[% IF Koha.Preference( 'RequestOnOpac' ) == 1 %]
>-<a href="[% OPACBaseURL %]/cgi-bin/koha/opac-reserve.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]">Place Hold on [% SEARCH_RESULT.title %]</a>
>-[% END %]
>-						]]></description>
>+ <link>
>+ [% IF ( SEARCH_RESULT.BiblioDefaultViewmarc ) %]
>+ [% OPACBaseURL %]/cgi-bin/koha/opac-MARCdetail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]">
>+ [% ELSE %]
>+ [% IF ( SEARCH_RESULT.BiblioDefaultViewisbd ) %]
>+ [% OPACBaseURL %]/cgi-bin/koha/opac-ISBDdetail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]
>+ [% ELSE %]
>+ [% OPACBaseURL %]/cgi-bin/koha/opac-detail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]
>+ [% END %]
>+ [% END %]
>+ </link>
>+ <description>
>+ [% PROCESS item_description %]
>+ </description>
> <guid>[% OPACBaseURL %]/cgi-bin/koha/opac-detail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]</guid>
> </item>
> 	 [% END %]
>@@ -68,39 +102,46 @@
> <feed xmlns="http://www.w3.org/2005/Atom"
> xmlns:opensearch="http://a9.com/-/spec/opensearch/1.1/">
> <id>[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% query_cgi |html %][% limit_cgi |html %]&amp;format=atom</id>
>- <title>[% LibraryName |html %] Search [% IF ( query_desc ) %]for '[% query_desc |html %]'[% END %][% IF ( limit_desc ) %]&nbsp;with limit(s):&nbsp;'[% limit_desc |html %]'[% END %]</title>
>- <link href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% query_cgi |uri %][% limit_cgi |uri %]&amp;format=atom"/>
>- <updated>[% timestamp %]</updated>
>+ <title>[% LibraryName |html %] Search [% IF ( query_desc ) %]for '[% query_desc %]'[% END %][% IF ( limit_desc ) %]&nbsp;with limit(s):&nbsp;'[% limit_desc |html %]'[% END %]</title>
>+ <link href="[% PROCESS feed_url %]atom"/>
>+ [% IF ( timestamp ) %]<updated>[% timestamp %]</updated>[% END %]
> <author>
> <name>[% LibraryName |html %]</name>
> </author>
> 
> <opensearch:totalResults>[% total %]</opensearch:totalResults>
>- <opensearch:startIndex>[% page %]</opensearch:startIndex>
>+ <opensearch:startIndex>[% offset %]</opensearch:startIndex>
> [% IF ( results_per_page ) %]
> <opensearch:itemsPerPage>[% results_per_page %]</opensearch:itemsPerPage>
> [% ELSE %]
> <opensearch:itemsPerPage>20</opensearch:itemsPerPage>
> [% END %]
>- <opensearch:Query role="request" searchTerms="[% SEARCH_RESULT.query_desc |url %] [% SEARCH_RESULT.limit_desc |url%]" startPage="[% page |html %]" />
>- <link rel="alternate" href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% SEARCH_RESULT.query_cgi %][% SEARCH_RESULT.limit_cgi %]pw=[% page %]&amp;format=atom" type="application/atom+xml"/>
>- <link rel="self" href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% SEARCH_RESULT.query_cgi %][% SEARCH_RESULT.limit_cgi %]&amp;format=atom" type="application/atom+xml"/>
>- <link rel="first" href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% SEARCH_RESULT.query_cgi %][% SEARCH_RESULT.limit_cgi %]pw=1&amp;format=atom" type="application/atom+xml"/>
>- <link rel="previous" href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% SEARCH_RESULT.query_cgi %][% SEARCH_RESULT.limit_cgi %]pw=2&amp;format=atom" type="application/atom+xml"/>
>- <link rel="next" href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% SEARCH_RESULT.query_cgi %][% SEARCH_RESULT.limit_cgi %]pw=4&amp;format=atom" type="application/atom+xml"/>
>- <link rel="last" href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% SEARCH_RESULT.query_cgi %][% SEARCH_RESULT.limit_cgi %]pw=42299&amp;format=atom" type="application/atom+xml"/>
>+ <opensearch:Query role="request" searchTerms="[% query_desc |url %][% limit_desc |url%]" startPage="[% page |html %]" />
>+ <link rel="alternate" href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% query_cgi | html %][% limit_cgi | html %]pw=[% page %]&amp;format=atom" type="application/atom+xml"/>
>+ <link rel="self" href="[% PROCESS feed_url %]atom" type="application/atom+xml"/>
>+ <link rel="first" href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% query_cgi | html %][% limit_cgi | html %]pw=1&amp;format=atom" type="application/atom+xml"/>
>+ <link rel="previous" href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% query_cgi | html %][% limit_cgi | html %]pw=2&amp;format=atom" type="application/atom+xml"/>
>+ <link rel="next" href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% query_cgi | html %][% limit_cgi | html %]pw=4&amp;format=atom" type="application/atom+xml"/>
>+ <link rel="last" href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% query_cgi | html %][% limit_cgi | html %]pw=42299&amp;format=atom" type="application/atom+xml"/>
> <link rel="search" type="application/opensearchdescription+xml" href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?format=opensearchdescription"/>
> 
> [% FOREACH SEARCH_RESULT IN SEARCH_RESULTS %]
> <entry>
> <title>[% SEARCH_RESULT.title |html %] [% FOREACH subtitl IN SEARCH_RESULT.subtitle %], [% subtitl.subfield |html %][% END %]</title>
>- <link>[% IF ( SEARCH_RESULT.BiblioDefaultViewmarc ) %][% OPACBaseURL %]/cgi-bin/koha/opac-MARCdetail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %][% ELSE %][% IF ( SEARCH_RESULT.BiblioDefaultViewisbd ) %][% OPACBaseURL %]/cgi-bin/koha/opac-ISBDdetail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %][% ELSE %][% OPACBaseURL %]/cgi-bin/koha/opac-detail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %][% END %][% END %]</link>
>- 
>- <updated>[% SEARCH_RESULT.timestamp %]</updated>
>+ [% IF ( SEARCH_RESULT.BiblioDefaultViewmarc ) %]
>+ <link href="[% OPACBaseURL %]/cgi-bin/koha/opac-MARCdetail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]" />
>+ [% ELSE %]
>+ [% IF ( SEARCH_RESULT.BiblioDefaultViewisbd ) %]
>+ <link href="[% OPACBaseURL %]/cgi-bin/koha/opac-ISBDdetail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]" />
>+ [% ELSE %]
>+ <link href="[% OPACBaseURL %]/cgi-bin/koha/opac-detail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]" />
>+ [% END %]
>+ [% END %]
>+ [% IF ( SEARCH_RESULT.dateaccessioned ) %]<updated>[% SEARCH_RESULT.dateaccessioned %]</updated>[% END %]
> <id>[% OPACBaseURL %]/cgi-bin/koha/opac-detail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]</id>
>- <content type="text">
>-
>- </content>
>+ <summary type="html">
>+ [% PROCESS item_description %]
>+ </summary>
> </entry>
> [% END %]
> </feed>
>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 4ad361a..9a9db45 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt
>@@ -16,8 +16,10 @@
> [% Asset.css("css/jquery.rating.css") %]
> [% END %]
> [% END %]
>-
>-<link rel="alternate" type="application/rss+xml" title="[% LibraryName |html %] Search RSS feed" href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% query_cgi | url %][% limit_cgi |uri %]&amp;count=[% countrss |uri %]&amp;sort_by=acqdate_dsc&amp;format=rss" />
>+[% BLOCK rss_feed %]
>+ [% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% query_cgi | url %][% limit_cgi |uri %]&amp;count=[% countrss |uri %]&amp;offset=[% offset %]&amp;sort_by=acqdate_dsc&amp;format=rss
>+[% END %]
>+<link rel="alternate" type="application/rss+xml" title="[% LibraryName |html %] Search RSS feed" href="[% PROCESS rss_feed %]" />
> </head>
> 
> [% INCLUDE 'bodytag.inc' bodyid='results' bodyclass='scrollto' %]
>@@ -42,7 +44,7 @@
> No results found!
> 
> [% IF ( searchdesc ) %]
>- No results found for that in [% LibraryName %] catalog. <a href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% query_cgi | url %][% limit_cgi | html | url %]&amp;format=rss" class="rsssearchlink noprint"><img src="[% interface %]/[% theme %]/images/feed-icon-16x16.png" alt="Subscribe to this search" title="Subscribe to this search" border="0" class="rsssearchicon"/></a>
>+ No results found for that in [% LibraryName %] catalog. <a href="[% PROCESS rss_feed %]" class="rsssearchlink noprint"><img src="[% interface %]/[% theme %]/images/feed-icon-16x16.png" alt="Subscribe to this search" title="Subscribe to this search" border="0" class="rsssearchicon"/></a>
> [% ELSE %]
> You did not specify any search criteria.
> [% END %]
>@@ -86,7 +88,7 @@
> [% END %]
> ).
> [% END %]
>- <a href="[% OPACBaseURL %]/cgi-bin/koha/opac-search.pl?[% query_cgi | url %][% limit_cgi |html | url %]&amp;count=[% countrss |html %]&amp;sort_by=acqdate_dsc&amp;format=rss" class="rsssearchlink noprint"><img src="[% interface %]/[% theme %]/images/feed-icon-16x16.png" alt="Subscribe to this search" title="Subscribe to this search" class="rsssearchicon"/></a>
>+ <a href=[% PROCESS rss_feed %]" class="rsssearchlink noprint"><img src="[% interface %]/[% theme %]/images/feed-icon-16x16.png" alt="Subscribe to this search" title="Subscribe to this search" class="rsssearchicon"/></a>
> [% END # / IF total %]
> 
> [% END # / IF searchdesc %]
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 16111: 49400 | 49587 | 49629 | 49837 | 76148 | 76160 | 76161 | 76162 | 77057 | 77058 | 77059 | 77060 | 78578 | 78579 | 78580 | 78581 | 78689 | 81285 | 81286 | 81287 | 81288 | 92050 | 92362 | 92363 | 92366 | 92435 | 92436 | 92437 | 92438 | 92439 | 92618 | 92619 | 92620 | 92621 | 92622 | 92623 | 92624