Open
Description
Preconditions and environment
- Magento version 2.4.7-p3
- Do not use the $oldStore to fetch existing urlwrites
Steps to reproduce
- Have multiple Websites and multiples stores
- Create a custom redirection for a website A on a URL common to different websites (blog to blogs for example)
- Go to Website B, and on the blog page, make a store switch
Expected result
The Rewrite url created for another website A must not have any link with Website B
Actual result
For code in file : vendor/magento/module-url-rewrite/Model/StoreSwitcher/RewriteUrl.php ligne 80
Currently my store IDs on my website B are 7 and 25.
The code returns me a redirect to website A
So we have $existingRewrite is yes and $currentRewrite is false . So we are sent back to the BaseUrl
The problem in the $existing write variable, we will not filter with the $oldStoreId
To correct this I added the $oldStoreId in the filter
Additional information
This my fix
switcher.zip
Release note
No response
Triage and priority
- Severity: S0 - Affects critical data or functionality and leaves users without workaround.Severity: S1 - Affects critical data or functionality and forces users to employ a workaround.Severity: S2 - Affects non-critical data or functionality and forces users to employ a workaround.Severity: S3 - Affects non-critical data or functionality and does not force users to employ a workaround.Severity: S4 - Affects aesthetics, professional look and feel, “quality” or “usability”.
Metadata
Metadata
Assignees
Labels
Gate 3 Passed. Manual verification of the issue completed. Issue is confirmedMay be fixed according to the position in the backlog.Indicates original Magento version for the Issue report.The issue has been reproduced on latest 2.4-develop branchIssue related to Developer Experience and needs help with Triage to Confirm or Reject it
Activity
m2-assistant commentedon Dec 12, 2024
Hi @Yoann-D. Thank you for your report.
To speed up processing of this issue, make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce.
@magento I am working on this
Join Magento Community Engineering Slack and ask your questions in #github channel.
⚠️ According to the Magento Contribution requirements, all issues must go through the Community Contributions Triage process. Community Contributions Triage is a public meeting.
🕙 You can find the schedule on the Magento Community Calendar page.
📞 The triage of issues happens in the queue order. If you want to speed up the delivery of your contribution, join the Community Contributions Triage session to discuss the appropriate ticket.
m2-assistant commentedon Dec 12, 2024
Hi @engcom-Hotel. Thank you for working on this issue.
In order to make sure that issue has enough information and ready for development, please read and check the following instruction: 👇
Area: XXXXX
label to the ticket, indicating the functional areas it may be related to.2.4-develop
branchDetails
- If the issue is reproducible on2.4-develop
branch, please, add the labelReproduced on 2.4.x
.- If the issue is not reproducible, add your comment that issue is not reproducible and close the issue and stop verification process here!
Issue: Confirmed
once verification is complete.engcom-Hotel commentedon Dec 12, 2024
Hello @Yoann-D,
Thanks for the report and collaboration!
We request you to please elaborate more on the below steps (If possible share screencast to demonstrate):
Thanks
24 remaining items