a_guilhem

Results 18 comments of a_guilhem

Indeed, we could distinguish the type of exception thrown depending on whether there is a constructor or not. - if constructor invalid, the object being not possible to instantiate we...

@issei-m for sure. I will also take into account this PR https://github.com/symfony/symfony/pull/48821

@TerjeBr PR https://github.com/symfony/symfony/pull/48821 doesn't solve your issue ? I can have a look at it :) sorry for the late reply

Would the added test case correspond to your case in https://github.com/symfony/symfony/pull/50192 ?

@nicolas-grekas @dunglas ok I'll update it and try to decorate the Connection object

> I don't like the approach, because as @nicolas-grekas mentioned, this will keep pinging the DB every request, even if normally request wouldn't need connection. Can't we make this DBAL...

@ostrolucky Before refactoring (taking into account `DoctrinePingConnectionMiddleware`), I'm not entirely sure I grasp the specific middleware you're referring to. Are you referring to those related to `Doctrine\DBAL\Driver` `Doctrine\DBAL\Driver\Connection`

> Are you planning to add some tests? in https://github.com/doctrine/DoctrineBundle/pull/1739 PR

closing the PR : see https://github.com/doctrine/dbal/pull/6351 and https://github.com/doctrine/DoctrineBundle/pull/1739