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

core_url_rewrite.request_path should be case-sensitive #252

Closed
colinmollenhour opened this issue Feb 28, 2013 · 3 comments
Closed

core_url_rewrite.request_path should be case-sensitive #252

colinmollenhour opened this issue Feb 28, 2013 · 3 comments
Assignees

Comments

@colinmollenhour
Copy link

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:

ALTER TABLE  `core_url_rewrite` CHANGE  `request_path`  `request_path` VARCHAR( 255 ) CHARACTER SET utf8 COLLATE utf8_bin NULL DEFAULT NULL COMMENT  'Request Path'

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.

@verklov verklov assigned verklov and piotrekkaminski and unassigned verklov Apr 18, 2014
@verklov
Copy link
Contributor

verklov commented Apr 18, 2014

@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.

@verklov
Copy link
Contributor

verklov commented Jan 21, 2015

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.

@verklov verklov closed this as completed Jan 21, 2015
magento-team pushed a commit that referenced this issue Apr 30, 2015
@wakhay
Copy link

wakhay commented Jun 16, 2015

Hi @verklov , any updates on this?
Also, how can I view the ticket MAGETWO-23727 ? (may be I'm searching at the wrong place for it)

Thanks!

magento-team pushed a commit that referenced this issue Dec 22, 2015
[NORD] Sample Data tests coverage. Bugfixes
slavvka pushed a commit that referenced this issue May 24, 2020
MQE-2105: Test steps' merging order is not consistent
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants