Jérôme Tamarelle

Hello @yahyaizhar. Thanks for contributing this bugfix. Would you be able to add a test showing the bug, to avoid any regression?

Tracked in Jira to get time to review this PR and add tests: PHPORM-175

I've found some other by searching for `$callback(`: - The `Symfony\Component\Cache\LockRegistry::compute` have an immediately invoked calback, and the `return null` is unreachable. - In `Symfony\Component\Cache\Messenger\EarlyExpirationDispatcher::__invoke` - `Symfony\Component\Config\ConfigCacheFactoryInterface::cache` - `Symfony\Component\Config\Definition\Loader\DefinitionFileLoader::executeCallback`

I like the grey too. We want this options to be less visible in the list, not highlight them. And I don't see any reason for customizing the description prefix...

The function `Illuminate\Database\Query\Builder::update()` returns an `int` in Laravel (see [the code](https://github.com/laravel/framework/blob/869c1d9bfdeda586a62658f19b35a88e543c23d1/src/Illuminate/Database/Query/Builder.php#L3700-L3706)). The method `getChanges()` exists and is correctly updated when called on a `Model` instance. ```php $user = User::create(['name' =>...

That's a matter of adoption. Once major sdks will use this interface instead of requiring guzzle or symfony http-client directly (see https://github.com/FriendsOfPHP/well-known-implementations/issues/2), it will be possible to use them asynchronously...

What would be the syntax for using a constant in XML for a key?

Hello, thanks for the feedback. I'm tracking it in Jira PHPORM-180 You are right that this events are not triggered because `newModelInstance` is used. It might be possible to explicitly...

I rebased this PR on 6.4. Adding a new exception is a feature. The detection of circular dependency on even subscriber have been removed because this feature is deprecated.

I missed last comment from @stof. I can update the PR if we think this feature is useful. In fact, I would prefer getting rid of the doctrine event listeners...