Super trivial fix

Comments surrounding RouteProcessorCsrf talk of an inbound path processor while the class implements the OutBoundRouteProcessorInterface.

CommentFileSizeAuthor
#3 inOut-2380349-3.patch536 bytesmartin107
inOut-0.patch553 bytesmartin107
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Needs review » Needs work

Actually I do not think that this one-line documentation is accurate at all. Looking at the code, I don't think that is what it does. Can we possibly fix it to be accurate?

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Title: 'in' ... 'out'...err I mean shake it all about! » Fix one-line documentation for RouteProcessorCsrf
martin107’s picture

martin107 CreditAttribution: martin107 commented
Status: Needs work » Needs review
FileSize
inOut-2380349-3.patch536 bytes

Sorry was busy until now.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

Seems valid

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

This issue is a minor bug fix, and doesn't include any disruptive changes, so it is allowed per https://www.drupal.org/core/beta-changes. Committed 3162cf9 and pushed to 8.0.x. Thanks!

  • alexpott committed 3162cf9 on 8.0.x 
    Issue #2380349 by martin107: Fix one-line documentation for...

Status: Fixed » Closed (fixed)

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