Skip to content

Commit fcbfbb1

Browse files
ker0xnicolas-grekas
authored andcommitted
[FrameworkBundle] Rename service notifier.logger_notification_listener to notifier.notification_logger_listener
1 parent 071bddb commit fcbfbb1

File tree

5 files changed

+14
-4
lines changed

5 files changed

+14
-4
lines changed

UPGRADE-6.3.md

+5
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,11 @@ FrameworkBundle
3838
</framework:config>
3939
```
4040

41+
FrameworkBundle
42+
---------------
43+
44+
* Deprecate the `notifier.logger_notification_listener` service, use the `notifier.notification_logger_listener` service instead
45+
4146
HttpKernel
4247
----------
4348

src/Symfony/Bundle/FrameworkBundle/CHANGELOG.md

+1
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ CHANGELOG
99
* Add `--format` option to the `debug:config` command
1010
* Add support to pass namespace wildcard in `framework.messenger.routing`
1111
* Deprecate `framework:exceptions` tag, unwrap it and replace `framework:exception` tags' `name` attribute by `class`
12+
* Deprecate the `notifier.logger_notification_listener` service, use the `notifier.notification_logger_listener` service instead
1213

1314
6.2
1415
---

src/Symfony/Bundle/FrameworkBundle/Resources/config/notifier.php

+5-1
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,11 @@
126126
->args([service('texter.transports')])
127127
->tag('messenger.message_handler', ['handles' => PushMessage::class])
128128

129-
->set('notifier.logger_notification_listener', NotificationLoggerListener::class)
129+
->set('notifier.notification_logger_listener', NotificationLoggerListener::class)
130130
->tag('kernel.event_subscriber')
131+
132+
->alias('notifier.logger_notification_listener', 'notifier.notification_logger_listener')
133+
->deprecate('symfony/framework-bundle', '6.3', 'The "%alias_id%" service is deprecated, use "notifier.notification_logger_listener" instead.')
134+
131135
;
132136
};

src/Symfony/Bundle/FrameworkBundle/Resources/config/notifier_debug.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
return static function (ContainerConfigurator $container) {
1717
$container->services()
1818
->set('notifier.data_collector', NotificationDataCollector::class)
19-
->args([service('notifier.logger_notification_listener')])
19+
->args([service('notifier.notification_logger_listener')])
2020
->tag('data_collector', ['template' => '@WebProfiler/Collector/notifier.html.twig', 'id' => 'notifier'])
2121
;
2222
};

src/Symfony/Bundle/FrameworkBundle/Test/NotificationAssertionsTrait.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -91,8 +91,8 @@ public static function getNotifierMessage(int $index = 0, string $transportName
9191
public static function getNotificationEvents(): NotificationEvents
9292
{
9393
$container = static::getContainer();
94-
if ($container->has('notifier.logger_notification_listener')) {
95-
return $container->get('notifier.logger_notification_listener')->getEvents();
94+
if ($container->has('notifier.notification_logger_listener')) {
95+
return $container->get('notifier.notification_logger_listener')->getEvents();
9696
}
9797

9898
static::fail('A client must have Notifier enabled to make notifications assertions. Did you forget to require symfony/notifier?');

0 commit comments

Comments
 (0)