Comment 7 for bug 1454884

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

Thank you Terran! It works for me. I had a couple of thoughts when testing it.

1. In some cases, I almost wanted to get rid of the subfield b in the display as well as it could sometimes be misleading. However, I think if we were to do so, we should see the same display in the initial metarecord search results display so that the two are consistent. Also, in many cases, it was misleading because the metarecord grouping is a bit imperfect, which should be the topic of another bug.

2. I haven't seen this happen in the wild yet, but I imagine there may be times where the foreign language version of a title is the master record and ultimately displays here. I think that's a more difficult issue to address.

Overall, though, your patch addresses the majority of use cases and provides more accuracy on the screen. Merged to master and backported to 2.10, 2.9 and 2.8.