search

cesargb / laravel-magiclink

Create link for authenticate in Laravel without password or get private content
MIT License
385 stars 44 forks source link

Cast MagicLink's UUID to a string when initially set #113

Closed rossbearman closed 7 months ago

rossbearman commented 7 months ago

Resolves #112.

rossbearman commented 7 months ago

The failing PHPStan test is unrelated to this change.

cesargb commented 7 months ago

@rossbearman Its true, the fail with PhpStan is not related with this PR.

cesargb commented 7 months ago

@rossbearman Can your reopen this change?

rossbearman commented 7 months ago

Hi @cesargb, I realised the issue wasn't directly related to laravel-magiclink. The failure was only happening at my end because declare(strict_types=1); was being injected everywhere by a rogue Pint configuration.

The issue isn't present after pulling in a clean version of the package, as LazyUuidFromString gets cast automatically when declare(strict_types=1); isn't present.