Bug #44521
ObjectPathMappingRepository always checks case-sensitive
| Status: | Resolved | Start date: | 2013-01-14 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | Bastian Waidelich | % Done: | 100% |
|
| Category: | MVC - Routing | |||
| Target version: | TYPO3 Flow Base Distribution - 2.0.1 | |||
| PHP Version: | Complexity: | |||
| Has patch: | No | Affected Flow version: | (any) |
Description
The ObjectPathMappingRepository always checks for matches using a case-sensitive check. This worked fine for most people so far, because the most common collation on MySQL is probably utf8_unicode_ci, which is case-*in*sensitive anway.
Related issues
Associated revisions
[BUGFIX] Respect case-sensitivity in object routing
By default dynamic route parts are lower-cased unless the
"toLowerCase" flag is set to TRUE for the route or a single
route part.
This change makes sure that this flag is actually taken into
account for routes using the ObjectPathMapping.
Change-Id: Ia9a2a864881b2ee94f1335cc141fc5c0b5cf632e
Fixes: #44521
Release: master, 2.0
[BUGFIX] Respect case-sensitivity in object routing
By default dynamic route parts are lower-cased unless the
"toLowerCase" flag is set to TRUE for the route or a single
route part.
This change makes sure that this flag is actually taken into
account for routes using the ObjectPathMapping.
Change-Id: Ia9a2a864881b2ee94f1335cc141fc5c0b5cf632e
Fixes: #44521
Release: master, 2.1, 2.0
History
#1 Updated by Karsten Dambekalns over 2 years ago
See https://review.typo3.org/17526
Bastian, can you take this further? To make it work reliably, the used table should probably switch to lutf8_unicode_bin@…
#2 Updated by Bastian Waidelich over 2 years ago
- Status changed from Needs Feedback to Accepted
Yes, I'll take care
#3 Updated by Karsten Dambekalns almost 2 years ago
- Target version changed from 2.0 to 2.0.1
#4 Updated by Gerrit Code Review almost 2 years ago
- Status changed from Accepted to Under Review
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17526
#5 Updated by Gerrit Code Review almost 2 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17526
#6 Updated by Gerrit Code Review almost 2 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17526
#7 Updated by Karsten Dambekalns almost 2 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 9ec9717a5ba73e7dab90f74fc9e167333930ed87.
#8 Updated by Gerrit Code Review almost 2 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch 2.0 has been pushed to the review server.
It is available at https://review.typo3.org/24977
#9 Updated by Gerrit Code Review over 1 year ago
Patch set 2 for branch 2.0 of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at https://review.typo3.org/24977
#10 Updated by Gerrit Code Review over 1 year ago
Patch set 3 for branch 2.0 of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at https://review.typo3.org/24977
#11 Updated by Gerrit Code Review over 1 year ago
Patch set 4 for branch 2.0 of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at https://review.typo3.org/24977
#12 Updated by Karsten Dambekalns over 1 year ago
- Status changed from Under Review to Resolved
Applied in changeset 56c372045e97248a487d30867fe431d7a53dc6cc.