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

Remove status filter from Product Collection in CatalogSearch Layer #3060

Conversation

elidrissidev
Copy link
Member

@elidrissidev elidrissidev commented Mar 3, 2023

Description (*)

A followup to #2662 which removed the status filter from Product Collection in Catalog Layer to fix the issue discussed in #2603 (comment).

This issue could be fixed by #2660, but this change is just to keep the consistency between Catalog and CatalogSearch Layers.

Related Pull Requests

Contribution checklist (*)

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All automated tests passed successfully (all builds are green)
  • Add yourself to contributors list

@github-actions github-actions bot added the Component: CatalogSearch Relates to Mage_CatalogSearch label Mar 3, 2023
@elidrissidev elidrissidev requested a review from fballiano March 3, 2023 08:56
@fballiano
Copy link
Contributor

question, are we loosing the status filter completely or it was duplicate so this way we're avoiding the duplicate?

@elidrissidev
Copy link
Member Author

question, are we loosing the status filter completely or it was duplicate so this way we're avoiding the duplicate?

We're avoiding the duplicate only when flat tables are enabled, else the filter is never added to the collection.

@elidrissidev
Copy link
Member Author

An important step to reproduce this issue is to make sure price attribute has "Use In Search Results Layered Navigation" value set to "Yes", otherwise this issue will not occur.

@fballiano
Copy link
Contributor

but we should have the status filter if flat is not enabled no?

@elidrissidev
Copy link
Member Author

It wasn't there before, and it was also removed from the Catalog view, so Idk...

@fballiano
Copy link
Contributor

I thought I added the filter on status, cause disabled product should never be visible on frotend, and there was the method for that but the method was empty... I don't know...

@fballiano fballiano merged commit 84361cf into OpenMage:1.9.4.x Mar 9, 2023
@fballiano
Copy link
Contributor

merged and v20ed

@elidrissidev elidrissidev deleted the fix/catalogsearch-layer-status-filter branch March 9, 2023 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: CatalogSearch Relates to Mage_CatalogSearch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants