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

Product enabled status only checked for default storeview #71

Closed
npluimers opened this issue May 27, 2024 · 1 comment
Closed

Product enabled status only checked for default storeview #71

npluimers opened this issue May 27, 2024 · 1 comment

Comments

@npluimers
Copy link

Describe the bug
Products that are enabled on default, and disabled on a specific storeview are being included in the product feed, causing missing products.

Used versions

  • Magento Version number(eg 2.3.5): 2.4.5-p6
  • Open source/Enterprise/B2b: Open Source
  • Sooqr version number (Check configuration): 2.1.1

To Reproduce
Steps to reproduce the behavior:

  1. Add Product A, enabled on default, and disable on one storeview
  2. Generate feeds
  3. Check feed of storeview where product A is disabled

Expected behavior
Product A is not included in the feed since the product is not enabled on the specific storeview

Actual behavior
Product A is included in the feed

Additional context
We have created a patch: https://gist.github.com/npluimers/f8d6475369514d34629af8d620ce3353

@Marvin-Magmodules
Copy link
Member

Marvin-Magmodules commented May 29, 2024

Hi @npluimers, thank you for opening this issue and also supplying the patch! Just released v2.1.2 that includes this patch. We added some extra code to this patch to also check for visibility fallback on default store, as while testing we found out that part was still not working properly.

We are closing this issue now but please feel free to reopen if it doesn't fit your needs.

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

No branches or pull requests

2 participants