Problem/Motivation

AjaxRenderer declares a $controllerResolver member variable but never sets or accesses it.

Proposed resolution

Remove the declaration.

CommentFileSizeAuthor
#3 remove-unused-controllerResolver-variable-from-AjaxRenderer-2992542-3.patch574 bytesfoxtrotcharlie
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tstoeckler created an issue. See original summary.

foxtrotcharlie’s picture

foxtrotcharlie CreditAttribution: foxtrotcharlie as a volunteer commented
Assigned: Unassigned » foxtrotcharlie
foxtrotcharlie’s picture

foxtrotcharlie CreditAttribution: foxtrotcharlie as a volunteer commented
Status: Active » Needs review
FileSize
remove-unused-controllerResolver-variable-from-AjaxRenderer-2992542-3.patch574 bytes

$controllerResolver variable declaration removed from AjaxRenderer.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler at bio.logis Genetic Information Management GmbH commented
Status: Needs review » Reviewed & tested by the community

Perfect, thank you!

  • larowlan committed da66fcb on 8.7.x 
    Issue #2992542 by foxtrotcharlie: AjaxRenderer declares a $...
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan at PreviousNext commented
Version: 8.6.x-dev » 8.7.x-dev
Status: Reviewed & tested by the community » Fixed

Committed da66fcb and pushed to 8.7.x. Thanks!

Status: Fixed » Closed (fixed)

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