Skip to content
Snippets Groups Projects
Commit caad6202 authored by Demian Katz's avatar Demian Katz
Browse files

Fix bug: field-specific facet limit interfered with deep facet paging.

parent 06806dd2
No related merge requests found
...@@ -251,6 +251,18 @@ class Params extends \VuFind\Search\Base\Params ...@@ -251,6 +251,18 @@ class Params extends \VuFind\Search\Base\Params
$this->facetLimit = $l; $this->facetLimit = $l;
} }
/**
* Set Facet Limit by Field
*
* @param array $new Associative array of $field name => $limit
*
* @return void
*/
public function setFacetLimitByField(array $new)
{
$this->facetLimitByField = $new;
}
/** /**
* Set Facet Offset * Set Facet Offset
* *
......
...@@ -308,6 +308,8 @@ class Results extends \VuFind\Search\Base\Results ...@@ -308,6 +308,8 @@ class Results extends \VuFind\Search\Base\Results
// Manipulate facet settings temporarily: // Manipulate facet settings temporarily:
$params->resetFacetConfig(); $params->resetFacetConfig();
$params->setFacetLimit($limit); $params->setFacetLimit($limit);
// Clear field-specific limits, as they can interfere with retrieval:
$params->setFacetLimitByField([]);
if (null !== $page && $limit != -1) { if (null !== $page && $limit != -1) {
$offset = ($page - 1) * $limit; $offset = ($page - 1) * $limit;
$params->setFacetOffset($offset); $params->setFacetOffset($offset);
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment