-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Cms block cache key does not contain the store id #22299
Comments
Hi @tzyganu. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
For more details, please, review the Magento Contributor Assistant documentation. @tzyganu do you confirm that you was able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
Hi @engcom-backlog-nazar. Thank you for working on this issue.
|
HI @tzyganu thank you for you report, Would you like to create Pull Request with this ? |
@engcom-backlog-nazar If you really want me to do it, sure. Unless you want to take over. if you do, I won't oppose. |
✅ Confirmed by @engcom-backlog-nazar Issue Available: @engcom-backlog-nazar, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself. |
Hi @engcom-backlog-nazar. Thank you for working on this issue.
|
Hi @tzyganu. Thank you for working on this issue.
|
@magento-engcom-team give me 2.3-develop instance - upcoming 2.3.x release |
Hi @dipti2jcommerce. Thank you for your request. I'm working on Magento 2.3-develop instance for you |
Hi @tzyganu. Thank you for your report. The fix will be available with the upcoming 2.2.9 release. |
Preconditions (*)
{{trans}}
directive. Or 2 blocks with the same identifier but each one available on a different store viewSteps to reproduce (*)
Expected result (*)
Actual result (*)
Additional info:
This happens because the cache key of the cms block is determined by
Magento\Framework\View\Element\AbstractBlock::getCacheKey
which callsgetCacheKeyInfo
from the same class which returns an array with the name of the block in the layout.Possible solution:
Adding the store id to the cache key or to the array returned by
getCacheKeyInfo
.The text was updated successfully, but these errors were encountered: