Bug #47534
Algorithms::generateUUID() creates uppercase characters in UUIDs
Status: | Resolved | Start date: | 2013-04-24 | |
---|---|---|---|---|
Priority: | Must have | Due date: | ||
Assigned To: | Karsten Dambekalns | % Done: | 100% |
|
Category: | Utility | |||
Target version: | TYPO3 Flow Base Distribution - 2.0 | |||
PHP Version: | Complexity: | |||
Has patch: | No | Affected Flow version: | Flow 2.0.0 beta 2 |
Description
The UUIDs generated by that method were always lowercase, now they are uppercase. As soon as you do a case-sensitive comparison on those UUIDs as strings, this will now break, even if the UUIDs are the same semantically.
Related issues
Associated revisions
[BUGFIX] generateUUID() creates uppercase characters in UUIDs
The UUIDs generated by that method were always lowercase, now they are
uppercase (since change I1237fdfbee75cbfca2f2082cf30a69383dbc7fd5). As
soon as you do a case-sensitive comparison on those UUIDs as strings,
this will now break, even if the UUIDs are the same semantically.
This change makes them lowercase again.
Change-Id: I0ec62f8710cc3c84fb726e3b08e6e08fc1638e39
Fixes: #47534
Related: #46862
Releases: master, 2.0
[BUGFIX] generateUUID() creates uppercase characters in UUIDs
The UUIDs generated by that method were always lowercase, now they are
uppercase (since change I1237fdfbee75cbfca2f2082cf30a69383dbc7fd5). As
soon as you do a case-sensitive comparison on those UUIDs as strings,
this will now break, even if the UUIDs are the same semantically.
This change makes them lowercase again.
Change-Id: I0ec62f8710cc3c84fb726e3b08e6e08fc1638e39
Fixes: #47534
Related: #46862
Releases: master, 2.0
History
#1 Updated by Gerrit Code Review over 2 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20124
#2 Updated by Gerrit Code Review over 2 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20124
#3 Updated by Gerrit Code Review over 2 years ago
Patch set 1 for branch 2.0 has been pushed to the review server.
It is available at https://review.typo3.org/20127
#4 Updated by Karsten Dambekalns over 2 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 654e0f4bb2c557b066538bc53588461c1c3aad28.
#5 Updated by Gerrit Code Review over 2 years ago
- Status changed from Resolved to Under Review
Patch set 2 for branch 2.0 has been pushed to the review server.
It is available at https://review.typo3.org/20127
#6 Updated by Karsten Dambekalns over 2 years ago
- Status changed from Under Review to Resolved
Applied in changeset 60ea1313d0206c8528830872eec2c7a712424b6e.