Bug #47534

Algorithms::generateUUID() creates uppercase characters in UUIDs

Added by Karsten Dambekalns over 2 years ago. Updated over 2 years ago.

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

related to TYPO3.Flow - Bug #46862: generateUUID can be broken by seeding mt_rand Resolved 2013-04-02

Associated revisions

Revision 654e0f4b
Added by Karsten Dambekalns over 2 years ago

[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

Revision 60ea1313
Added by Karsten Dambekalns over 2 years ago

[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

#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

Also available in: Atom PDF