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
Fixed anchor categories showing products of disabled subcategories #9002 #14320
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please, cover provided changes at least by unit test
* @return string | ||
* @since 101.0.0 | ||
*/ | ||
protected function makeTempCategoryTreeIndex() | ||
protected function makeTempCategoryTreeIndex(\Magento\Store\Model\Store $store) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can't add/remove any parameters to public and protected method, please, use guide for backward compatible developpmenet
* @param string $temporaryName | ||
* @since 101.0.0 | ||
*/ | ||
protected function fillTempCategoryTreeIndex($temporaryName) | ||
protected function fillTempCategoryTreeIndex($temporaryName, \Magento\Store\Model\Store $store) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See previous comment
@drashmk , I am closing this PR now due to inactivity. |
Hi @drashmk. Thank you for your contribution. |
Description
Incomplete query for the temporary table
temp_catalog_category_tree_index_xxxx
was causing products of disabled subcategories to be visible on anchor categories. It was missing the check foris_active
category attribute on the child category when creating the temporary category.Fixed Issues (if relevant)
#9002: Anchor categories are showing products of disabled subcategories
Manual testing scenarios
Contribution checklist