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 Pages used in settings are not excluded from sitemap #38666
Comments
Hi @thomas-kl1. Thank you for your report.
Join Magento Community Engineering Slack and ask your questions in #github channel. |
Hi @engcom-Dash. Thank you for working on this issue.
|
Hi @engcom-Delta. Thank you for working on this issue.
|
Hi @thomas-kl1 , Verified the issue on 2.4-develop and it is not reproducible. Steps to reproduce:- 1.Create 2 websites (a & b) with a specific home page configured for each (e.g: home-a for global, and home-b for website Please refer the attached screenshot and let us if we are missing something. |
@magento give me 2.4-develop instance |
Hi @thomas-kl1. Thank you for your request. I'm working on Magento instance for you. |
Hi @thomas-kl1, here is your Magento Instance: https://40fa56ac7e83b8154495257198e8cf7d.instances-prod.magento-community.engineering |
Hi @thomas-kl1 , Based on your comment #38666 (comment) , we are closing this issue.Feel free to raise a fresh ticket or reopen this ticket if you face the same issue again. Thanks, |
Preconditions and environment
The error is because the settings use the following source:
Magento\Cms\Model\Config\Source\Page
which pipes the page id to the identifier. E.g:So when
\Magento\Cms\Model\GetUtilityPageIdentifiers::execute
is called:the scoped values (if not the default one provided in
vendor/magento/module-cms/etc/config.xml
is going to have this format : |<page_id>Which is incompatible with the query format here (used to generate the cms page sitemap):
\Magento\Sitemap\Model\ResourceModel\Cms\Page::getCollection
More specifically:
It won't prevent to return the given pages because the format does not match!
Steps to reproduce
Expected result
The sitemap A does not have the home-a url
The sitemap B does not have the home-b url
Actual result
The sitemap A does not have the home-a url (OK)
The sitemap B does have the home-b url (NOK)
Additional information
Also true for:
Release note
No response
Triage and priority
The text was updated successfully, but these errors were encountered: