-
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
core_url_rewrite.request_path should be case-sensitive #252
Comments
@colinmollenhour, thank you for the idea! I assigned this issue to a PM responsible for the area and created a ticket in the backlog. I will let you know of the decision taken. |
There is a ticket MAGETWO-23727 entered in the product backlog. We are closing this request on the GitHub now. We will update you on any changes on this request after the PM responsible for the area reviews it and makes a decision according to the priorities we have. |
Hi @verklov , any updates on this? Thanks! |
[NORD] Sample Data tests coverage. Bugfixes
MQE-2105: Test steps' merging order is not consistent
I tried to add two rewrites for the same url with different case (customers kept entering it wrong) and I couldn't because the collation on the request_path is utf8_general_ci (ci for case-insensitive) so the unique index prevented it. I changed the collation like so:
And was then able to add the rewrite. There may be other unexpected side-effects of this column being ci, and I can't see any harm in making it case-sensitive.
The text was updated successfully, but these errors were encountered: