As reported by drunken monkey at #1216928: Address possible regressions, Facet API does not support minumum facet counts.

CommentFileSizeAuthor
#2 facetapi-1219150-2.patch4.1 KBdrunken monkey
#1 facetapi-1219150-1.patch4.05 KBcpliakas

Comments

cpliakas’s picture

cpliakas commented
StatusFileSize
new facetapi-1219150-1.patch4.05 KB

The attached patch adds support for minumum facet counts. Marking as "needs work" because validation needs to happen of the mincount value passed by the user.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Active » Needs review
StatusFileSize
new facetapi-1219150-2.patch4.1 KB

Ultra-lazy version.
If someone wants to set a negative min count, I guess it really doesn't do any harm. *shrug*

cpliakas’s picture

cpliakas commented
Status: Needs review » Fixed

Committed fix at 756973c. Need some tests, but not critical by any means.

Status: Fixed » Closed (fixed)

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