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

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

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.