[Koha-bugs] [Bug 35812] Should specify canonical URLs to help search indexers

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35812

--- Comment #9 from M <schodkowy.omegi-0r at icloud.com> ---
Testing notes:

1. Test whether affected pages now have the new <link> tags in HTML source
2. Check whether the canonical URL for bib links is correct and that the URL it
points to works (http://LIBRARY.TLD/bib/12345)
3. For patch 4, if you open up searching on http://LIBRARY.TLD/search page, you
should see nothing in `/var/log/koha/INSTANCE/plack.log`. After applying the
patch, you should see `/opac/opac-search.pl` entries. This confirms the request
now goes through Plack, similar to how `/bib/*` links should do already.

As for backports, patch 4 could be backported as technically it fixes a bug
related to canonical links. The rest of the changes might be deemed not
applicable for backport though.

I originally planned to also have canonical URL specified for /search itself
too similar to /bib/, but I backed down from it because:
- its usefulness would be limited if search results have noindex attribute now
- it'd be actually somewhat messy to specify proper URL query params correctly,
the current template parameters used for RSS link are not ideal, this would be
lots of rework and rethink for no clear benefit, so...

-- 
You are receiving this mail because:
You are watching all bug changes.