[PR #8898] Throw exception NotSupported Exception for UuidGenerator with doctr… #11237

New Issue

admin · 2026-01-22T16:10:07+01:00

admin commented

2026-01-22 16:10:07 +01:00

Original Pull Request: https://github.com/doctrine/orm/pull/8898

State: closed
Merged: Yes

Let's fix another issue:

[0;31mERROR[0m: UndefinedMethod - lib/Doctrine/ORM/Id/UuidGenerator.php:33:59 - Method Doctrine\DBAL\Platforms\AbstractPlatform::getGuidExpression does not exist (see https://psalm.dev/022)
        $sql  = 'SELECT ' . $conn->getDatabasePlatform()->[97;41mgetGuidExpression[0m();

Method has been removed with https://github.com/doctrine/dbal/pull/3211.
Lets throw exception if method do not exists. Generator itself is already deprecated.

Questions:

May it would be better to throw it in constructor?
What about unit tests? Should we cover also this case?

It looks like there will be few similar issues, so let's start from this one to create a template for the next PRs :)

**Original Pull Request:** https://github.com/doctrine/orm/pull/8898 **State:** closed **Merged:** Yes --- Let's fix another issue: ``` [0;31mERROR[0m: UndefinedMethod - lib/Doctrine/ORM/Id/UuidGenerator.php:33:59 - Method Doctrine\DBAL\Platforms\AbstractPlatform::getGuidExpression does not exist (see https://psalm.dev/022) $sql = 'SELECT ' . $conn->getDatabasePlatform()->[97;41mgetGuidExpression[0m(); ``` Method has been removed with https://github.com/doctrine/dbal/pull/3211. Lets throw exception if method do not exists. Generator itself is already deprecated. Questions: 1. May it would be better to throw it in constructor? 2. What about unit tests? Should we cover also this case? It looks like there will be few similar issues, so let's start from this one to create a template for the next PRs :)

admin added the pull-request label 2026-01-22 16:10:07 +01:00

admin referenced this issue

2026-01-22 16:15:29 +01:00

[PR #11237] Add error handling when misuse of find() with array values #12878

Sign in to join this conversation.