Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding the ability to use the Manual mode for price navigation step, … #111

Merged
merged 1 commit into from
Sep 14, 2016

Conversation

romainruaud
Copy link
Collaborator

Add the ability to use the Magento configuration for the Price Navigation Step Calculation : the "Manual" mode is now supported and will use the "Default Price Navigation Step" field when rendering the price slider.

This can allow to have much lighter aggregations when having a wide range of prices displayed, like in #77

Could be merged only in 2.2.x since it's more a new feature than a bugfix.

…with a default price navigation step defined in configuration.
@romainruaud romainruaud force-pushed the feature_price-interval branch from b385c94 to 8074cb5 Compare September 14, 2016 08:30
@romainruaud romainruaud merged commit f78211e into Smile-SA:2.2.x Sep 14, 2016
@southerncomputer
Copy link
Contributor

southerncomputer commented Sep 27, 2016

how about adding min_doc_countclass Histogram
in core/../Histogram.php

{
/**
* Build the aggregation.
*
* @param BucketInterface $bucket Histogram bucket.
*
* @return array
*/
public function buildBucket(BucketInterface $bucket)
{
return ['histogram' => ['field' => $bucket->getField(), 'interval' => $bucket->getInterval(),'min_doc_count'=>1]];
}
}
`
this helped with massive amounts of price variance!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants