Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
logger.channel.default:
class: Drupal\Core\Logger\LoggerChannel
factory_method: get
factory_service: logger.factory
arguments: ['system']
logger.channel.php:
class: Drupal\Core\Logger\LoggerChannel
factory_method: get
factory_service: logger.factory
arguments: ['php']
logger.channel.image:
class: Drupal\Core\Logger\LoggerChannel
factory_method: get
factory_service: logger.factory
arguments: ['image']
logger.channel.cron:
class: Drupal\Core\Logger\LoggerChannel
factory_method: get
factory_service: logger.factory
arguments: ['cron']
logger.channel.form:
class: Drupal\Core\Logger\LoggerChannel
factory_method: get
factory_service: logger.factory
arguments: ['form']
Proposed resolution
logger.channel.default:
parent: logger.channel_base
arguments: ['system']
logger.channel.php:
parent: logger.channel_base
arguments: ['php']
logger.channel.image:
parent: logger.channel_base
arguments: ['image']
logger.channel.cron:
parent: logger.channel_base
arguments: ['cron']
logger.channel.form:
parent: logger.channel_base
arguments: ['form']
Remaining tasks
review/commit
User interface changes
None
API changes
None
Comment | File | Size | Author |
---|---|---|---|
#8 | drupal-2341701-5.patch | 2.5 KB | ParisLiakos |
#8 | interdiff-2341701.txt | 385 bytes | ParisLiakos |
#4 | drupal-2341701-3.patch | 2.49 KB | ParisLiakos |
#3 | drupal-2341701.patch | 1.35 KB | ParisLiakos |
#3 | interdiff-2341701.patch | 1.13 KB | ParisLiakos |
Comments
Comment #1
ParisLiakos CreditAttribution: ParisLiakos commentedComment #2
dawehnerAwesome idea!
Note: there are also logger defined in aggregator, dblog, rest and syslog you might could update as well.
Comment #3
ParisLiakos CreditAttribution: ParisLiakos commentedoooh indeed, totally forgot those, thanks!! not syslog though ;)
Comment #4
ParisLiakos CreditAttribution: ParisLiakos commentedmeh, wrong patch:) interdiff is correct
Comment #5
dawehnerCool, thank you!
Comment #8
ParisLiakos CreditAttribution: ParisLiakos commentedthats embarassing :/
Comment #9
tim.plunkettThis is great, thanks!
Comment #10
catchCommitted/pushed to 8.0.x, thanks!