Broaden search result for other sorting than "relevance" (resolves #10)

[avstudnitz]

This is relevant for search queries containing two words or more where
the words are found in different product attributes. In this case, there
is a mechanism in the search result which searches for the single words
if no results are found otherwise. The mechanism was only used for search
by relevance which is extended to other sort orders with this commit.
Additionally, there was a bug in this mechanism - the parameter "mm=0%"
has to be set for these additional requests.

[avstudnitz]

Successfully tested in a client shop.

[schmengler]

Is there a conceptual difference between $broaden and $attributesToReset, that justifies passing one via setter and one via parameter to the build method? It looks confusing.

If no, I think, the setter is the better way, as it fits the builder pattern better:

1. configure the builder
2. build the object based on builder configuration

Then it could look like this:

            $this->getParamsBuilder()
                ->setBroaden($this->broaden)
-               ->buildAsArray($this->getAttributetoReset())
+               ->setAttributesToReset($this->getAttributetoReset())
+               ->buildAsArray()

[schmengler]

should be SearchParamsBuilder now, because methods not in the interface are used (which is okay in this case, as discussed)

[avstudnitz]

Okay, I changed this. See also integer-net/solr-pro#7 as I had to do changes to the pro library too due to changed interface.

[avstudnitz]

I can't change it to SearchParamsBuilder easily. It crashes with the Autosuggest.

[schmengler]

If I understood it correctly, setAttributesToReset is only needed in one implementation. I think it should not be implemented in the abstract params builder and probably not even used in the abstract query builder then.

Here we can easily remove the line and move them to setAttributeToReset() above:

     public function setAttributeToReset($attributeToReset)
     {
-        $this->attributetoReset = $attributeToReset;
+        $this->paramsBuilder->setAttributeToReset($attributeToReset);
         return $this;
     }

Then move that method down to the class(es) where it is actually needed and we know that the type of paramsBuilder supports it: SearchQueryBuilder and CategoryQueryBuilder(?)