'query' => 'path=' . $vars['path']

should be:

'query' => array('path' => $vars['path'])

There are other cases as well in the function theme_forward_email($variables)

Comments

SolomonGifford’s picture

SolomonGifford commented 
  $vars = $variables['vars'];

should also be:

  $vars = $variables;
john.oltman’s picture

john.oltman commented
Status: Active » Postponed (maintainer needs more info)

Thomas, I assume you are using the old D5-based theme functions? Because in the D7 module I'm not seeing this. The code in #1 in particular is actually correct in D7.

john.oltman’s picture

john.oltman commented
Assigned: Unassigned » john.oltman
Status: Postponed (maintainer needs more info) » Active

I was looking in the forward.module file - I see the issue now in the forward.theme file. I think #1 is OK but the original issue remains. Changing back to active.

SolomonGifford’s picture

SolomonGifford commented

Yeah, its in the forward.theme file - it seems it may have some legacy naming in there.

john.oltman’s picture

john.oltman commented
Status: Active » Fixed

Committed fix to 7.x branch and will be in next nightly DEV build

Status: Fixed » Closed (fixed)

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