Comment 0 for bug 1914116

Revision history for this message
Jason Boyer (jboyer) wrote : An audit of query parameters is needed

Evergreen 3.5, 3.6, and master are affected.

So, recently we fixed bug 1687545 and now can sure that all query parameters consistently use ampersands rather than semicolons. Hurray!

But it turns out all of these semicolons were hiding areas where we were not using the necessary filters in our .tt2 files, so now things like the "Show more copies" link in the opac has changed from

.../eg/opac/record/2?locg=1;detail_record_view=0;query=concerto;copy_limit=50;copy_offset=0

to the new and exciting

.../eg/opac/record/2?locg=1&detail_record_view=0&query=concerto©_limit=50©_offset=0

which works about as well as one might expect.

The minimum required fix to correct this is to run anything that will end up in an 'a' tag href attribute through the html filter, though there may be times where multiple filters may be needed.