Instead of explicitly calling the l function and setting the #markup property we should be employing the link theme function. This allows developers to modify the link output, including changing the link text and adding additional attributes.

CommentFileSizeAuthor
#1 commerce_addressbook-theme_links-1774086-1.patch2.28 KBcmaddalozzo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cmaddalozzo’s picture

cmaddalozzo CreditAttribution: cmaddalozzo commented
FileSize
commerce_addressbook-theme_links-1774086-1.patch2.28 KB

I've attached a patch with the refactored code.

cmaddalozzo’s picture

cmaddalozzo CreditAttribution: cmaddalozzo commented
Status: Active » Needs review
jsacksick’s picture

jsacksick CreditAttribution: jsacksick commented
Status: Needs review » Fixed

Committed thanks !

Status: Fixed » Closed (fixed)

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