Commit 54641cb
committed
bug symfony#57887 [Uid] Ensure UuidV1 is created in lowercase (smnandre)
This PR was submitted for the 7.2 branch but it was merged into the 5.4 branch instead.
Discussion
----------
[Uid] Ensure UuidV1 is created in lowercase
| Q | A
| ------------- | ---
| Branch? | 7.2
| Bug fix? | no
| New feature? | no
| Deprecations? | no
| Issues | Fix symfony#57878
| License | MIT
Ensure `$this->uid` is in lowercase in UuidV1 (see symfony#57878 for context)
(strtolower is already called in the parent::__construct... but not when $args is null. And the uuid extension generates uppercase values --at least sometimes--).
Not sure if should be considered a "bug" / which version to flag ?
Commits
-------
9abfd25 [Uid] Ensure UuidV1 is created in lowercase2 files changed
+10
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
90 | 90 | | |
91 | 91 | | |
92 | 92 | | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
93 | 102 | | |
94 | 103 | | |
95 | 104 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
28 | | - | |
| 28 | + | |
29 | 29 | | |
30 | 30 | | |
31 | 31 | | |
| |||
0 commit comments