But the node was being looked for in the wrong place
Function system_mail
line 1727 & 1728 currently read
if (isset($params['node'])) {
$node = $params['node'];

should read
if (isset($context['node'])) {
$node = $context['node'];

This means that when the workflow module sends a mail, the node variables such as %node_url are not replaced

Comments

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Title: // Node-based variable translation is only available if we have a node. » system_mail() should get the node form $context not $params
Version: 6.11 » 7.x-dev
Priority: Critical » Normal

I am not sure your suggestion is right (did not check in detail), but retitling and moving to Drupal 7, so it can be looked at there and backported.

marcingy’s picture

marcingy commented
Version: 7.x-dev » 6.x-dev

The implementation has change entirely in D7 and no processing of node is done there. Moving back to D6 as the issue seems specific to that version.

Status: Active » Closed (outdated)

Automatically closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.