Link to sibling metarecord bibs in record detail page

Bug #1573734 reported by Jeff Godin
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned

Bug Description

Given a record detail page for a bib:

If that bib is a member of a metarecord with other bibs:

And those bibs "should be visible" at the given scope ([potentially some hand waving here]):

It could be useful to list those bibs (potentially grouped by format) and link to the individual bibs or to a results page displaying multiple bibs (potentially with "show more details" enabled).

Creating a wishlist bug from the back of a talk on search due to at least three people thinking this is a good idea and at least two people in the room thinking that this is achievable. :-)

Revision history for this message
Kathy Lussier (klussier) wrote :

Adding a screenshot with a starting mock-up on how this feature may look.

https://s3.amazonaws.com/media-p.slid.es/uploads/319986/images/2509753/alternate-formats.png

Kathy Lussier (klussier)
Changed in evergreen:
status: New → Confirmed
Revision history for this message
Terran McCanna (tmccanna) wrote :

Yes please!!! :)

Revision history for this message
Kathy Lussier (klussier) wrote :

The problem with my earlier mock-up is that it assumes there are only a few bib records in the metarecord group. However, when you're working with a classic, like Great Expectations, you are likely to way too many titles in the group to display them well on a bib record. In fact, our largest metarecord group has 3,000+ records attached to it (a result of not including the parts subfields in the fingerprint).

Here is an alternate mock-up for how we would like to implement this feature.

http://masslnc.org/system/files/u71/repositioned-alternate-formats-editions.png

Rather than listing individual records, provide links to their format groups and language groups.

Revision history for this message
Terran McCanna (tmccanna) wrote :

I like it - very clear and easy to understand.

Blake GH (bmagic)
Changed in evergreen:
assignee: nobody → Blake GH (blake-j)
Revision history for this message
Blake GH (bmagic) wrote :

I have some work on this:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/blake/LP1573734_Link_to_sibling_metarecord_bibs_in_record_detail_page

I would like to get some feedback from the developers on this one. Use this for example:

http://sb1.missourievergreen.org/eg/opac/record/1647802

The hyperlink for each of the formats is working just fine with blank search criteria but when you want to see all of the bibs in the metarecord without searching for anything, the server search code rejects the search:

View all (12) formats and editions:
http://sb1.missourievergreen.org/eg/opac/results?locg=1;detail_record_view=0;modifier=metabib;metarecord=261488
(no results)

If I change the link slightly:
http://sb1.missourievergreen.org/eg/opac/results?locg=1;detail_record_view=0;modifier=metabib;metarecord=261488;query=*

It works but it's not ideal. I don't like "hacking" it by searching for asterisk. Any suggestions on this? Should I edit the search code to allow for a blank query as long as there is a metarecord specified? Perhaps search for * and not display it in the TPAC? What is the best approach?

Revision history for this message
Mike Rylander (mrylander) wrote :

This is looking really good, Blake!

For your question, if you add "fi:from_metarecord=261488" as a CGI parameter to all , you get results and it leaves the query box empty. It's redundant in the URL, but that's not a big deal IMO. Thoughts?

Revision history for this message
Blake GH (bmagic) wrote :

Mike

That sounds fine to me. It's "better" than changing the search core to allow a blank query under "special" circumstances.

Kathy Lussier (klussier)
Changed in evergreen:
milestone: 2.next → 2.12-beta
Revision history for this message
Blake GH (bmagic) wrote :

This code is firmed up. One outstanding question is:
search_format vs mr_hold_format vs icon_format

There is a line in misc_util.tt2:
formats_xpath = '//*[local-name()="attributes"]/*[local-name()="field"][@name="' _ icon_style _ '"]';

The variable icon_style is set by a global_flag "opac.icon_attr" which is seeded with "icon_format"

It was natural to use the loop structure that was already in place in misc_util.tt2 for the icon format.

This means that the formats that are displayed are defined by icon_format. When you click on the hyperlink to display the result page of sibling bibs that match that format, we use search_format. There is a potential disconnect in definitions between icon_format and search_format.

Would it be OK if the search results were not created by search_format but instead icon_format?

AKA:

eg/opac/results?locg=1;modifier=metabib;metarecord=261488;fi%3Afrom_metarecord=261488;fi%3Aicon_format=cdaudiobook

Or should it be mr_hold_format? If we use search_format then the siblings should be displayed using search_format as well.

Perhaps a new global flag and make it configurable?

Revision history for this message
Kathy Lussier (klussier) wrote :

Adding a link to an IRC discussion on this question:

http://irc.evergreen-ils.org/evergreen/2017-02-09#i_288173

Revision history for this message
Blake GH (bmagic) wrote :

All,

A potential pitfall for using "opac.metarecord.holds.format_attr" - the default seeded database does not define any electronic formats in the "mr_hold_format". This was brought up on IRC and was decided to use icon_format instead.

I pushed my branch to the working repo. The link is the same as above. This branch includes transcendent bibs, suppress the summary when the bib is the only member of the metarecord, and uses icon_format for the grouping for both the summary and the grouped results.

Blake GH (bmagic)
Changed in evergreen:
assignee: Blake GH (blake-j) → nobody
tags: added: pullrequest
Revision history for this message
Kathy Lussier (klussier) wrote :

With the merge of bug 1629108, we no longer need to run this code on top of the other two branches. Pushed a new branch that only contains the sibling link work.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/kmlussier/LP1573734_Link_to_sibling_metarecord_bibs_in_record_detail_page

Revision history for this message
Kathy Lussier (klussier) wrote :

I tested the above branch and it work as expected. I made one minor tweak to change the heading in summary.tt2 to sentence case instead of title cast, matching the style of other similar headings on that page.

Merged to master for inclusion in 2.12.

Changed in evergreen:
status: Confirmed → Fix Committed
Revision history for this message
Kathy Lussier (klussier) wrote :

Just realized I forgot to check for / create release notes. Adding the needsreleasenotes tag as a reminder to myself to include them.

tags: added: needsreleasenote
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.