Developers shouldn't have to explicitly pass parameters to FacetapiUrlProcessor::normalizeParams() before resetting them. This method should be baked into the FacetapiAdapter::setParams() method so normalization happens automatically. Testing fails due to the current application flow.

CommentFileSizeAuthor
#1 facetapi-1344860-1.patch966 bytescpliakas

Comments

cpliakas’s picture

cpliakas commented
Status: Active » Needs review
StatusFileSize
new facetapi-1344860-1.patch966 bytes

The attached patch changes the application flow.

cpliakas’s picture

cpliakas commented
Status: Needs review » Fixed

Committed at 7080fd6

Status: Fixed » Closed (fixed)

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