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

Don't skip attribute options with a value of 0 from the layered navigation #7578

Conversation

dcabrejas
Copy link

@dcabrejas dcabrejas commented Nov 26, 2016

Bug Fix

Steps to replicate :

  1. Create an attribute to be used in the Layered Navigation using the source_model Magento\Eav\Model\Entity\Attribute\Source\Boolean.

  2. When you are on a category page, on the layered navigation you are not given the option to filter by 'No' even though there are products in the category which have this value, you can only filter by 'Yes' because all the attribute options with a falsy value won't be available. 'No' has a falsy value of 0;

screen shot 2016-11-26 at 15 36 37

## Code Explanation

Make sure that attribute options with a numeric value are not skipped. Empty strings will still be skipped.

@magento-cicd2
Copy link
Contributor

magento-cicd2 commented Nov 26, 2016

CLA assistant check
All committers have signed the CLA.

@okorshenko
Copy link
Contributor

@dcabrejasazagra thank you for your contribution

@dcabrejas dcabrejas deleted the bugfix/layered-nav-attribute-empty-value branch June 16, 2018 10:10
magento-devops-reposync-svc pushed a commit that referenced this pull request Apr 21, 2022
…Ql_13042022

Hammer quality backlog graph ql 13042022
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.

6 participants