Symfony has deprecated \Symfony\Component\HttpKernel\Controller\ControllerResolverInterface::getArguments().

Since Drupal 8.6 The controller_resolver service should be replaced by http_kernel.controller.argument_resolver.

Patch attached fixes this.

Change record: https://www.drupal.org/node/2959408

CommentFileSizeAuthor
replace-controller_resolver_with_argument_resolver.patch4.59 KBjeroent

Comments

JeroenT created an issue. See original summary.

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Issue summary: View changes
jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Issue summary: View changes
rosinegrean’s picture

rosinegrean commented
Issue tags: -DevDaysCluj +DevDaysTransylvania
idebr’s picture

idebr commented
Status: Needs review » Reviewed & tested by the community

Patch applies cleanly and resolves all occurrences of the deprecation notice.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented

Since this change happened in 8.6 I'll wait till 8.8 to commit this change.

kristen pol’s picture

kristen pol
she/her
Primary language English
Location Santa Cruz, CA, USA
commented
Issue tags: +Drupal 9 compatibility

Per a Slack discussion with Gábor Hojtsy regarding usage of D9 tags (Drupal 9, Drupal 9 compatibility, Drupal 9 readiness, etc.), "Drupal 9 compatibility" should be used for contributed projects that need updating and "Drupal 9" was the old tag for D8 issues before the D9 branch was ready. Doing tag cleanup here based on that discussion.

  • joelpittet committed 8f93161 on 8.x-3.x authored by JeroenT 
    Issue #3060837 by JeroenT: Replace controller_resolver service with...
joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Reviewed & tested by the community » Fixed

Committed to dev for the next release, thanks for doing this!

Status: Fixed » Closed (fixed)

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