Comment 5 for bug 1623134

Revision history for this message
Jim Keenan (jkeenan) wrote :

I pushed a branch with a change so that location data shows for the item.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=commit;h=a7a76853b70e343346ae42714607dddf877d2e36

Here's what it looks like in a KPAC search for "peter pan" on our training server.

https://training.cwmars.org/eg/kpac/getit/1897644?query=peter%20pan;qtype=keyword

You can do multiple searches in the KPAC on our training server, if you like, to see how it's working: https://training.cwmars.org/eg/kpac/home (I have, just for testing.)

I did not change the "Get it!" wording. I thought it might be okay to leave it (it seemed to make sense in the context now in a way it didn't before) but changing it to something else is not a big problem.

There is a provision in the code for showing added content under the location data. I see in addedcontent.tt2, though, there's a caveat about enabling this and its potential unhappy effect on the display. (How is it that in fixing one issue we always uncover others?)

If interested people would like to test out this code change and see how it goes in your environment.