in \Drupal\Core\Routing\UrlGenerator There is missing documentation for one of the constructor params and it's named using camel case instead of snake case.

CommentFileSizeAuthor
#1 2304801-1.patch1.65 KBpwolanin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Issue summary: View changes
Status: Active » Needs review
FileSize
2304801-1.patch1.65 KB
jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Needs review » Reviewed & tested by the community

Looks fine, thanks!

alexpott’s picture

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

Committed 4ac137d and pushed to 8.x. Thanks!

  • alexpott committed 4ac137d on 8.x 
    Issue #2304801 by pwolanin: Fixed Missing @param in UrlGenerator for...

Status: Fixed » Closed (fixed)

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