Update Logger.php

[cobaltjeh]

The Symfony LoggerInterface is deprecated, that class hints that the PsrLogger should be used instead.

[stof]

@jmikola note that this change will bump the Symfony requirement to 2.3+ (Symfony was not relying on PSR-3 before 2.3) so the composer constraint should be updated when merging

[jmikola]

@stof: Thanks for the heads up.

[jmikola]

@stof: If I understand correctly, since our 2.0 release is only compatible with Symfony 2.0, and the 3.0 (currently master branch) is compatibly with ~2.1, merging this would mean we'd have no release with support for 2.1.x and 2.2.x. I think the correct thing to do would be slate this for a 4.0.0 milestone to be merged after we tag a 3.0.0 release (which is waiting on ODM 1.0.0). Does that make sense?

[stof]

@jmikola IMO, the best solution would be to copy this logger to the ODM itself with a PSR-3 typehint (it is not specific to Symfony2 anymore), and keep the existing class as is but marking it deprecated. This way, the bundle can choose the right class depending on the version, users of the ODM have a PSR-3 logger available by default, and you don't need a 4.0 milestone yet

[jmikola]

@stof: That sounds like a great idea. Thanks!

[alcaeus]

Closing this in favor of #323. The upcoming 3.1 version will bump Symfony requirement to 2.3+.