Task #48880
Story #48874: As a developer I want a PSR-3 compatible Logging API
Implement LoggerInterface from PSR-3
100%
Description
PSR-3 expects the \TYPO3\CMS\Core\Log\Logger to extend AbstractLogger which implements LoggerInterface. AbstractLogger contains the shorthand log methods like debug(), warning() or error(). These shorthands call the log() mehthod which is implemented in Logger.
Updated by Ernesto Baschny about 9 years ago
It's not a requirement to extend AbstractLogger, we need simply to implement LoggerInterface one way of the other. But since the AbstractLogger is rather simple, it probably don't hurt to do it this way.
Updated by Helmut Hummel about 9 years ago
Ernesto Baschny wrote:
It's not a requirement to extend AbstractLogger,
I agree.
we need simply to implement LoggerInterface one way of the other. But since the AbstractLogger is rather simple, it probably don't hurt to do it this way.
We should only do it, if we think it makes sense for us. I don't see a real benefit in doing so.
Updated by Ernesto Baschny about 9 years ago
Helmut Hummel wrote:
We should only do it, if we think it makes sense for us. I don't see a real benefit in doing so.
Agreed.
Updated by Gerrit Code Review about 9 years ago
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21827
Updated by Gerrit Code Review about 9 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21827
Updated by Steffen Müller about 9 years ago
- Status changed from Under Review to Resolved
Applied in changeset 24a458dc
Updated by Steffen Müller about 9 years ago
- Subject changed from PSR-3: Introduce AbstractLogger and LoggerInterface to Implement LoggerInterface from PSR-3