In function apachesolr_search_execute() we cache the current query after the prepare hook, but then we let it be further modified on apachesolr_modify_query() (e.g. adding nodeaccess and other filters) before executing the query.

This leads to a bug where those later filters are exposed in the breadcrumb since we use $query->get_breadcrumb() with the changes introduced by apachesolr_modify_query().

Comments

pwolanin’s picture

Status: Active » Needs review
StatusFileSize
new1.35 KB
pwolanin’s picture

Version: 6.x-1.x-dev » 6.x-2.x-dev
Status: Needs review » Patch (to be ported)

committed to 6.x-1.x

robertdouglass’s picture

Version: 6.x-2.x-dev » 5.x-2.x-dev

#667124 by pwolanin: Fixed Use current query not altered query for the breadcrumb.
Committed to 6.2.

claudiu.cristea’s picture

Status: Patch (to be ported) » Fixed
StatusFileSize
new2.6 KB

Committed to 5.2 in #318128.
Patch attached.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.