Search highlighting affects OPAC title display

[Dan Wells]

With the new search infrastructure, the way highlighting is turned on and off via different data paths leads to some display oddities. For example, in the Concerto set, do a search for 'gaiman', go in a record, then turn highlighting on/off. When highlighting is 'on', the page title is just the title, but when it is off, it reverts back to the complete 245 field. (Obviously this isn't particular to these records, but is actually likely to happen more often than not for most records.) I, for one, would be happy to finally jettison the pre-formatted silliness that is MARC field 245 (well, most of MARC, unfortunately), but to have it tied to the highlight state is a bit odd.

It seems like the impetus here is to make the 'title' be consistent from display down to index, which makes a ton of sense, but the good old 245 has a lot of history, so maybe we continue to treat "page title" (i.e. 245) as a special case, and as a slightly different thing from the "real title"? The other obvious option is to make the non-highlight display a closer match for the highlighted version so that the switch isn't as jarring.

[Mike Rylander (mrylander)]

I was confused for a while ... the difference is that we're depending on the titleNonfiling XSLT template in the mods32 stylesheet (in stock) to pull out the title string when using the Display Field highlighting, which restricts the subfields to a, b, f, g, and k, while the previous mechanism used a rather big hammer of get_graphic_880s() defined in misc_util.tt2. There are other titles extracted elsewhere, but that one in particular is, indeed, "all of 245, minus e, w, 0, 4, 5, 6, 7, 8, and 9". While I could certainly see adding n and p to the titleNonfiling XSLT definition, we don't want to go back to showing subfield "h", for instance.

I mentioned this in IRC, but IMO, the way forward is to always use the Display Field data when we have it, but only use the highlighted version when highlighting is turned on. The logic to do that would actually be a simplification of what's in there now, with just an adjustment to the DF mapping code in misc_util.js to choose between the appropriate versions, and removal of any other "are we highlighting" tests in the rest of the templates.

I will work on that soon, so I'll just assign myself now.

[Mike Rylander (mrylander)]

Here's what I'm thinking:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/miker/lp-1752434-use_more_display_fields

[Remington Steed (rjs7)]

I gave Mike's branch a trial run, and it seems to fix the issue. Here's my sign-off:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/rsteed/lp-1752434-use_more_display_fields

[Galen Charlton (gmc)]

Pushed to master and rel_3_1, along with a patch that fixes a regression introduced in the control of highlighting in the author field. Thanks, Mike and Remington!