TPAC: Restore OpenSearch and RSS feeds for searches

Bug #1269070 reported by Dan Scott
20
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Undecided
Unassigned

Bug Description

* Evergreen 2.4, 2.5, master

JSPAC used to include OpenSearch support so that one could add your Evergreen instance as a search engine. It would be nice to have that back.

IIRC, JSPAC also used to offer up RSS feeds for search results. Not so sure whether we really need that, but thought it might be worth mentioning.

For OpenSearch support, Ben Shum pointed me back at a conversation we had a few months ago: http://irc.evergreen-ils.org/evergreen/2013-08-16#i_22600

Roughly, dropping "<link type="application/opensearchdescription+xml" rel='search' title="[% l('Evergreen OpenSearch') %]" href="/opac/extras/opensearch/1.1/-/osd.xml" />" into base.tt2 should do the trick... in theory.

Tags: pullrequest
Revision history for this message
Dan Scott (denials) wrote :

Pushed http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dbs/tpac_restore_opensearch with a fix for this problem.

I had to make a couple of small changes to Open-ILS/WWW/SuperCat.pm:
* We pass in the hostname now, so we can use an absolute path.
* I've opted to use HTTPS to protect searcher privacy.

Seems to work, but eyes / testing are greatly appreciated.

Changed in evergreen:
milestone: none → 2.6.0-alpha1
Revision history for this message
Dan Scott (denials) wrote :

Ben Shum caught an extraneous " that could be confusing to browsers; I've force-pushed the fix to the branch. Thanks Ben!

Revision history for this message
Dan Wells (dbw2) wrote :

Added pullrequest as this bug seemed ready for it.

tags: added: pullrequest
Changed in evergreen:
milestone: 2.6.0-alpha1 → 2.6.0-beta1
Ben Shum (bshum)
Changed in evergreen:
status: New → In Progress
assignee: nobody → Ben Shum (bshum)
Revision history for this message
Ben Shum (bshum) wrote :

Works for me, I'm picking it to master for upcoming 2.6 as new functionality. I also added a small commit with a tiny statement for the release notes.

Changed in evergreen:
status: In Progress → Fix Committed
assignee: Ben Shum (bshum) → nobody
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.