context.handler:
  class: Drupal\Core\Plugin\Context\ContextHandler
  arguments: ['@typed_data_manager']

ContextHandler class does not even have a constructor.

CommentFileSizeAuthor
#3 3002470.patch492 byteslongwave
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Chi created an issue. See original summary.

Chi’s picture

Chi CreditAttribution: Chi commented
Issue summary: View changes
longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave as a volunteer commented
Status: Active » Needs review
FileSize
3002470.patch492 bytes

I checked git history and the constructor was removed several years ago in #2293205: Consider removing usage of DataDefinition in ContextHandler so this is just a forgotten relic.

Chi’s picture

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

Thanks.

  • alexpott committed 1b317fd on 8.7.x 
    Issue #3002470 by longwave, Chi: Remove unused argument from context....

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 3: 3002470.patch, failed testing. View results

Chi’s picture

Chi CreditAttribution: Chi commented
Status: Needs work » Fixed

It seems already fixed.

Status: Fixed » Closed (fixed)

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