Closed (works as designed)
Project:
Rector
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
12 May 2022 at 18:38 UTC
Updated:
15 May 2022 at 09:03 UTC
Jump to comment: Most recent
Drupal 9
0.12.2
The @param is correctly replaced but the param strict tiping is full namespaced. It should add a use statement.
* Until the News page is ready we temporarily redirect to the search page.
* This way our users can already find the news they are looking for.
*
- * @param \Symfony\Component\HttpKernel\Event\GetResponseEvent $event
+ * @param \Symfony\Component\HttpKernel\Event\RequestEvent $event
* Response event.
*/
- public function onKernelRequest(GetResponseEvent $event) {
+ public function onKernelRequest(\Symfony\Component\HttpKernel\Event\RequestEvent $event) {
// Retrieve the request path, and do inbound processing so that language
// prefixes are removed.
$request = $event->getRequest();
Comments
Comment #2
claudiu.cristeaIt seems to be an upstream Rector issue. If I run PHPCBF after, this is fixed.
Comment #3
claudiu.cristeaClosed as is easily fixable by running PHPCBF after rector