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

Price slider displaying incorrect amount on products. #539

Closed
vortexunsworth opened this issue Sep 21, 2017 · 2 comments
Closed

Price slider displaying incorrect amount on products. #539

vortexunsworth opened this issue Sep 21, 2017 · 2 comments
Assignees
Labels

Comments

@vortexunsworth
Copy link

The price slider is displaying incorrect product result count compared to actual results and result count when listing products by price excluding tax.

Preconditions

Magento Version : CE 2.1.7

ElasticSuite Version : 2.3

Environment : Developer and Production

Third party modules : N/A

Steps to reproduce

  1. Set Sales > Tax > Calculation Settings > Catalog Prices = Excluding Tax
  2. Go to product list page.
  3. Filter products by price slider.

Expected result

  1. Price filter displayed expected results to be actual result count.

Actual result

  1. Price slider expected result count is incorrect to actual results.
    image

The problem "seems" to be that the slider carrier on filtering with VAT applied to products. Tried various solutions but none that worked including PR #381 .

@romainruaud
Copy link
Collaborator

Hello @vortexunsworth and thank you for submitting this issue.

I am able to reproduce it on my local environment.

This is not especially due to the "Excluding tax" display configuration, but rather a duplicate of #376 and incorrect slider up and down limit.

The PR #381 is incomplete for now, as it does not resolve completely the bug (that's why the PR has not been merged for now).

We'll continue to work on this PR.

Best regards

@romainruaud romainruaud self-assigned this Sep 25, 2017
@afoucret
Copy link
Contributor

Merged in branch 2.5.x and master.
Will be part of release 2.5.3.

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

No branches or pull requests

3 participants