oops. Patch #2078285: Add short-cut methods to the \Drupal class for generating URLs and links from routes didn't declare the l() method static.

CommentFileSizeAuthor
#4 2080841-4.patch1.34 KBpwolanin
#1 2080841-1.patch665 bytespwolanin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Status: Active » Needs review
FileSize
2080841-1.patch665 bytes
neclimdul’s picture

neclimdul
He/Him
CreditAttribution: neclimdul commented
Title: l() method on \Druapl needs to be static » l() method on \Drupal needs to be static

had to fix it.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented

Maybe we should convert an existing call in order to be sure.

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
FileSize
2080841-4.patch1.34 KB

Here's an example conversion too.

dawehner’s picture

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

Perfect.

alexpott’s picture

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

Committed 07525d7 and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

Updated issue summary.