TwigExtension::getUrl() should declare that it returns render array rather than string.
https://git.drupalcode.org/project/drupal/blob/8.7.6/core/lib/Drupal/Cor...

CommentFileSizeAuthor
#2 3073410-2.patch696 bytesshashikant_chauhan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Chi created an issue. See original summary.

shashikant_chauhan’s picture

shashikant_chauhan CreditAttribution: shashikant_chauhan as a volunteer commented
Status: Active » Needs review
FileSize
3073410-2.patch696 bytes

Adding patch.

Chi’s picture

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

Looks good. Thanks.

  • larowlan committed 08ee5a9 on 8.8.x 
    Issue #3073410 by shashikant_chauhan: TwigExtension::getUrl() declares...
larowlan’s picture

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

Committed 08ee5a9 and pushed to 8.8.x. Thanks!

c/p to 8.7.x

  • larowlan committed f926c1a on 8.7.x 
    Issue #3073410 by shashikant_chauhan: TwigExtension::getUrl() declares...

Status: Fixed » Closed (fixed)

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