Skip to content

[9.x] UUID and ULID support for Eloquent #44074

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.

Sign up for GitHub

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

Merged
merged 10 commits into from
Sep 13, 2022
Merged

[9.x] UUID and ULID support for Eloquent #44074

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
8f15950
Eloquen UUID & ULID
driesvints Sep 9, 2022
78fb93b
wip
driesvints Sep 9, 2022
c22597a
wip
driesvints Sep 9, 2022
4e37b71
Update tests/Integration/Database/EloquentPrimaryUlidTest.php
driesvints Sep 12, 2022
08d0c03
wip
driesvints Sep 12, 2022
0c4bcbb
wip
driesvints Sep 12, 2022
1866016
wip
driesvints Sep 12, 2022
be6df86
wip
driesvints Sep 12, 2022
1cf1148
tweaking
taylorotwell Sep 13, 2022
003b1a3
update composer file
taylorotwell Sep 13, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
update composer file
  • Loading branch information
@taylorotwell
taylorotwell committed Sep 13, 2022
commit 003b1a37149934414148a7e83bed0bd84241955e
2 changes: 1 addition & 1 deletion src/Illuminate/Support/composer.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@
"league/commonmark": "Required to use Str::markdown() and Stringable::markdown() (^2.0.2).",
"ramsey/uuid": "Required to use Str::uuid() (^4.2.2).",
"symfony/process": "Required to use the composer class (^6.0).",
"symfony/uid": "Required to use an ULID as a primary key for Eloquent (^6.0).",
"symfony/uid": "Required to generate ULIDs for Eloquent (^6.0).",
Copy link
Contributor

@tanthammar tanthammar Sep 16, 2022 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there something wrong with the requirement?
I had to manually composer require symfony/uid to get Str::ulid() to work.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is in the suggests session. So to use its functionality you should manually install this dependency.

"symfony/var-dumper": "Required to use the dd function (^6.0).",
"vlucas/phpdotenv": "Required to use the Env class and env helper (^5.4.1)."
},
Expand Down
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy