I needed to theme the HTML email to include the name of the person forwarding a page. Unfortunately, the yname field was not available in $vars so I created a one-line patch to add it in.

CommentFileSizeAuthor
forward.patch1010 bytesjumpfightgo@groups.drupal.org

Comments

jumpfightgo@groups.drupal.org’s picture

jumpfightgo@groups.drupal.org commented

I wonder if both yname and yemail should be filtered at this step, before being passed the the theme function, or should we let the theme function filter the content? Since yemail was being assed unfiltered already, I followed left yname unfiltered and handled it with my template's theme, but it's probably better practive to do it the other way around.

seanr’s picture

seanr
he/him
commented
Status: Needs review » Fixed

Thanks. This is committed and will be in the next release.

Status: Fixed » Closed (fixed)

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