The substitution of the email variables doesn't work.
In abuse_admin_moderate_content(), in abuse.admin.inc at the t-function for the description of the body-fields, eg. line 592 it is mentioned to use the schema !variable
'#description' => t('available fields are !title, !url, !name, and !id. You can copy from the list below.'),
but in abuse_mail() where the substitution should happen via strtr(), the array keys were written with @variable
...
$vars = array(
'@title' => $object->title,
'@url' => $object->link,
'@name' => $account->name,
'@id' => $object->link,
);
...
$body = strtr($params['body'], $vars);
so no substitution takes place.
In case of a flagged comment i received in the abuse-mail a node/683#comment-21 when i used @url instead of !url, perhaps it should be absolute with a leading http://.
See attached patch which solve both cases.
| Comment | File | Size | Author |
|---|---|---|---|
| abuse.admin_.inc-abuse_mail.patch | 768 bytes | fl3a |
Comments
Comment #1
buzzman commentedanyone tested this patch yet ... ? thanks