__toString() in UuidInterface

[kacper-wojtaszczyk]

The problem

I try to use explicit typecasting to basic types where possible because i find it clearer and more intuitive. With that in mind I always use (string) $uuid for casting the UUIDs to strings. The problem is that the magic method __toString() is only referenced in the actual implementations so when I try to typehint a variable with UuidInterface I get a warning about an unknown method on every cast I made.

The solution

Add public function __toString(): string; to the Interface (happy to make a pull request, just wanted to know if it doesn't break some adopted convention)

The mic drop

[ramsey]

@kacper-wojtaszczyk, thank you for your suggestion. This has been discussed a great deal already. The short answer is that __toString() has to wait until a 4.0.0 release, since it's a major BC break to add it to the interface.

Please take a look at the conversations on #272, #238, #111, the discussion at 4c467ce#commitcomment-31174263, and the commit feb3eb5.

The __toString() method is already on UuidInterface in the 4.x branch, but the comment here has me rethinking that, since the explicitness of ->toString() has typing and long-term stability benefits: ShittySoft/symfony-live-berlin-2018-doctrine-tutorial#3 (comment)

[kacper-wojtaszczyk]

Any ETA on 4.0?

[ramsey]

No ETA yet. Sorry.