-
Notifications
You must be signed in to change notification settings - Fork 330
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade Symfony calls to MAIN_REQUEST, LogoutEvent, TestUserProvider, Kernel RoutingConfigurator #6561
Upgrade Symfony calls to MAIN_REQUEST, LogoutEvent, TestUserProvider, Kernel RoutingConfigurator #6561
Conversation
46cb722
to
2543b7d
Compare
2543b7d
to
e24e8a9
Compare
8254d8b
to
9f407b1
Compare
@@ -41,6 +41,6 @@ public function dispatchDomainEventWithSpecificEventName(DomainEvent $event): vo | |||
// allows to register listeners that listen to all domain events. | |||
// this subscriber additionally dispatches the event with a specific event-name such as TagRemovedEvent::class | |||
// to allow for registering listeners for a specific type of event. | |||
$this->eventDispatcher->dispatch($event, \get_class($event)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
get_class is the default behaviour of the event dispatcher that is why rector is replacing it.
@@ -35,12 +35,12 @@ public function __construct(TranslatorInterface $translator) | |||
/** | |||
* @param mixed[] $parameters | |||
*/ | |||
public function trans($id, array $parameters = [], $domain = null, $locale = null): string | |||
public function trans(string $id, array $parameters = [], string $domain = null, string $locale = null): string |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this class is @internal
and should so make no difference here.
9f407b1
to
257b9fd
Compare
257b9fd
to
ecb3e1d
Compare
What's in this PR?
Upgrade deprecated symfony method calls.
Why?
To make sulu compatible with symfony 6 in future.