To be sure we work correctly with AND/OR facets and move complex facets, each fq should be wrapped in parentheses if moved to q.alt.

CommentFileSizeAuthor
#1 864146-parens-1.patch520 bytespwolanin

Comments

pwolanin’s picture

pwolanin commented
Status: Active » Needs review
StatusFileSize
new 864146-parens-1.patch520 bytes
pwolanin’s picture

pwolanin commented
Title: When moving fq to q.alt, we shoudl parenthisize each fq » When moving fq to q.alt, we should parenthesize each fq

fix title

jpmckinney’s picture

jpmckinney commented

Looks like a good fix to http://drupal.org/cvs?commit=360436 #552152-12: OR facet counts are confusing

jpmckinney’s picture

jpmckinney commented
Status: Needs review » Reviewed & tested by the community
robertdouglass’s picture

robertdouglass commented
Version: 6.x-2.x-dev » 6.x-1.1
Status: Reviewed & tested by the community » Patch (to be ported)

#864146 by pwolanin | jpmckinney: Fixed When moving fq to q.alt, we should parenthesize each fq.

Committed to 6.2. Was this intended for 6.1 as well?

pwolanin’s picture

pwolanin commented

Can't hurt.

jpmckinney’s picture

jpmckinney commented
Version: 6.x-1.1 » 6.x-1.x-dev

Committed to 7.x

jpmckinney’s picture

jpmckinney commented
Status: Patch (to be ported) » Fixed

Status: Fixed » Closed (fixed)

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