Re: [VuFind-Tech] Combined search confuses breadcrumbs

[Demian K. <dem...@vi...>]

Good catch - I've implemented a fix here:

https://github.com/vufind-org/vufind/commit/95eef448216f1f1b49cb102e2b752bacd4c01b5b

It would probably also make sense to refactor VuFind\Search\Memory as a ZF service rather than the current static-driven approach... but for now I wanted to implement the least disruptive fix possible.  I'll open a separate ticket for the refactoring, to be addressed at a future date.

- Demian

From: Joe Atzberger [mailto:jo...@bo...]
Sent: Wednesday, September 04, 2013 4:39 PM
To: vufind-tech Tech
Subject: [VuFind-Tech] Combined search confuses breadcrumbs

I've been running a Combined search via AJAX and it appears to be disturbing the breadcrumb links.

The code in module/VuFind/src/VuFind/View/Helper/Root/GetLastSearchLink.php pulls a cached value based on the last search performed, but each time I last performed two kinds of searches (or in practice, performed the AJAX one after the regular one and have presented links to hits for both).  What's the fix?  Are these features incompatible?

--Joe