63/63 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

------ ----------------------------------------------------------------------------------------------------------------------------------------------
Line modules/redirect_404/src/EventSubscriber/Redirect404Subscriber.php
------ ----------------------------------------------------------------------------------------------------------------------------------------------
102 Parameter $event of method Drupal\redirect_404\EventSubscriber\Redirect404Subscriber::onKernelException() has typehint with deprecated class
Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent:
since Symfony 4.3, use ExceptionEvent instead
104 Call to deprecated method getException() of class Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent:
since Symfony 4.4, use getThrowable instead
------ ----------------------------------------------------------------------------------------------------------------------------------------------

Line modules/redirect_domain/src/EventSubscriber/DomainRedirectRequestSubscriber.php
------ ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------
68 Parameter $event of method Drupal\redirect_domain\EventSubscriber\DomainRedirectRequestSubscriber::onKernelRequestCheckDomainRedirect() has typehint with deprecated class
Symfony\Component\HttpKernel\Event\GetResponseEvent:
since Symfony 4.3, use RequestEvent instead
112 Parameter $event of method Drupal\redirect_domain\EventSubscriber\DomainRedirectRequestSubscriber::setResponse() has typehint with deprecated class
Symfony\Component\HttpKernel\Event\GetResponseEvent:
since Symfony 4.3, use RequestEvent instead
------ ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------

------ ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Line src/EventSubscriber/RedirectRequestSubscriber.php
------ ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
111 Parameter $event of method Drupal\redirect\EventSubscriber\RedirectRequestSubscriber::onKernelRequestCheckRedirect() has typehint with deprecated class
Symfony\Component\HttpKernel\Event\GetResponseEvent:
since Symfony 4.3, use RequestEvent instead
180 Parameter $event of method Drupal\redirect\EventSubscriber\RedirectRequestSubscriber::setResponse() has typehint with deprecated class Symfony\Component\HttpKernel\Event\GetResponseEvent:
since Symfony 4.3, use RequestEvent instead
------ ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

------ -----------------------------------------------------------------------------------------------------------------------------------------------------------
Line src/EventSubscriber/RouteNormalizerRequestSubscriber.php
------ -----------------------------------------------------------------------------------------------------------------------------------------------------------
86 Parameter $event of method Drupal\redirect\EventSubscriber\RouteNormalizerRequestSubscriber::onKernelRequestRedirect() has typehint with deprecated class
Symfony\Component\HttpKernel\Event\GetResponseEvent:
since Symfony 4.3, use RequestEvent instead
------ -----------------------------------------------------------------------------------------------------------------------------------------------------------

Comments

kapilkumar0324 created an issue. See original summary.

kapilv’s picture

StatusFileSize
new6.63 KB

Status: Needs review » Needs work

The last submitted patch, 2: 3130096-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

saphemmy’s picture

Status: Needs work » Needs review
StatusFileSize
new6.85 KB

The issues have been fixed as stated above

Status: Needs review » Needs work

The last submitted patch, 4: 3130096-4.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

saphemmy’s picture

Most of these tests were already failing before my patch.

askibinski’s picture

Title: Class deprecated » Deprecated Symfony Event classes
Status: Needs work » Needs review
StatusFileSize
new12.53 KB
new5.27 KB

@saphemmy The tests still used the deprecated classes as return types which conflicts.

holist’s picture

Status: Needs review » Reviewed & tested by the community

Fixes deprecation notices. Works best when combined with https://www.drupal.org/project/redirect/issues/3173618, with both changes I got no more deprecation notices for the module.

berdir’s picture

Status: Reviewed & tested by the community » Needs work

This means the module then requires Drupal 9, which is fine, but all info.yml files and composer.json need to be updated to reflect that.

  • Berdir committed 11613ac on 8.x-1.x authored by askibinski
    Issue #3130096 by askibinski, KapilV, saphemmy: Deprecated Symfony Event...
berdir’s picture

Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.