refactor: make transactionNestingLevel for easier overriding

[Crovitche-1623]

Q	A
Type	improvement
Fixed issues	Close #6885

Summary

See #6885

[Crovitche-1623]

May I also remove the @internal annotation from the __construct() method as the class can be used using wrapper_class?

[morozov]

Where else is this method expected to be invoked from?

[Crovitche-1623]

@morozov I used the convertException method from the connect and commit methods that I overrode.

[Crovitche-1623]

@morozov Another solution would be to make the handleDriverException method protected instead of private.

[morozov]

May I also remove the @internal annotation from the __construct() method as the class can be used using wrapper_class?

I don't think this is a good idea. It would make the shape of $params part of the public API which we discussed in #6885 (comment).

[Crovitche-1623]

May I also remove the @internal annotation from the __construct() method as the class can be used using wrapper_class?

I don't think this is a good idea. It would make the shape of $params part of the public API which we discussed in #6885 (comment).

@morozov Okay, in that case, do you have any idea how I can silence the deprecation message that indicates the class is internal?

[morozov]

Okay, in that case, do you have any idea how I can silence the deprecation message that indicates the class is internal?

What does this message look like? I don't think the DBAL itself emits any messages when an internal method is invoked from outside. It must be some 3rd-party tooling.

[Crovitche-1623]

Okay, in that case, do you have any idea how I can silence the deprecation message that indicates the class is internal?

What does this message look like? I don't think the DBAL itself emits any messages when an internal method is invoked from outside. It must be some 3rd-party tooling.

Yep it's the Symfony\Component\Config\Resource\ClassExistenceResource on line 76.

I have no clue what this class does though.