RSS links should point to opac record format by default instead of htmholdings-full

Bug #1183972 reported by Kathy Lussier
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned
2.4
Fix Released
Undecided
Unassigned

Bug Description

By default, links in Evergreen RSS feeds point to records in the htmlholdings-full format. An example record in this format can be seen at:

https://jasondev.mvlcstaff.org/opac/extras/unapi?id=tag:open-ils.org,2013-05-24:biblio-record_entry/490977/MVLC&format=htmlholdings-full

We would like these links to use the opac format by default, which, after https://bugs.launchpad.net/evergreen/+bug/1089479 is resolved, would bring the user to the bib record in the catalog.

There was some discussion in IRC this morning regarding the use of YAOUS to identify which format should be used. However, it sounded like the use of a setting might be complicated to implement. It might be first useful to have a discussion about whether a setting is even needed to control this behavior.

 I personally can't think of any users who would want to view the htmlholdings-full format of a record. I think most users would expect to arrive at the record in the catalog.

Ben Shum (bshum)
Changed in evergreen:
importance: Undecided → Wishlist
status: New → Triaged
Revision history for this message
Mike Rylander (mrylander) wrote :

Below you'll find a branch that does what Ben describes, as well as adjusting the <title> element of bookbag feeds to contain just the bookbag name. The description is still available and wrapped in context-providing text.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/miker/supercat-tpac-links

This should close the above-referenced bug 1089479 as well.

tags: added: opac pullrequest
Changed in evergreen:
milestone: none → 2.5.0-alpha1
Revision history for this message
Mike Rylander (mrylander) wrote :

I might also argue that this is a bug fix, at least for 2.4 ... but I'll leave that alone others agree. If you do, feel free to target this bug at 2.4 as well.

Revision history for this message
Lebbeous Fogle-Weekley (lebbeous) wrote :

This works well in my tests, so I have signed-off and pushed to master.

Since it's not targeted at 2.4 yet, I didn't merge it there, although if somebody comes along before bit rot sets in, I don't see why it would be unreasonable to merge into 2.4

Changed in evergreen:
status: Triaged → Fix Committed
Revision history for this message
Lebbeous Fogle-Weekley (lebbeous) wrote :

On second thought, as this fixes bug 1089479 at the same time, and nobody wants links to JSPAC links if they're running 2.4+, I think this really ought to be backported. So doing that and marking the other bug closed.

Ben Shum (bshum)
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.