The documentation for ContextDefinition mentions the following:

Remove spaces after @ in your actual plugin - these are put into this sample code so that it is not recognized as an annotation.

However there are no spaces after the @ symbol. I tracked this down and this remark was introduced in the original issue #2281635: \Drupal\Core\Plugin\Context\Context should use a ContextDefinition class instead of arrays in comment #31. The spaces after the @ symbol were removed again in comment #38 but this was not spotted in subsequent reviews.

CommentFileSizeAuthor
#2 2877259-2.patch584 bytespfrenssen
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pfrenssen created an issue. See original summary.

pfrenssen’s picture

Status: Active » Needs review
FileSize
584 bytes
bander2’s picture

Status: Needs review » Reviewed & tested by the community

Patch applied cleanly.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed 3f01062 to 8.4.x and 27a9589 to 8.3.x. Thanks!

  • alexpott committed 3f01062 on 8.4.x
    Issue #2877259 by pfrenssen: Remove outdated remark in documentation of...

  • alexpott committed 27a9589 on 8.3.x
    Issue #2877259 by pfrenssen: Remove outdated remark in documentation of...

Status: Fixed » Closed (fixed)

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