Active
Project:
Variable Email
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
21 Jan 2013 at 04:45 UTC
Updated:
3 Jun 2013 at 20:36 UTC
It looks like this for me, out of the box: http://share.aidanlister.com/MJHC
I've hunted around, and I think I see the problem. I'm not sure what "text_format" is, I cannot find it defined anywhere in the variable module. I see that the mail_text type uses "text" for the field, which looks correct if you want the previews displayed in HTML. If you want the previews displayed escaped, then use "string".
index 9a9dd2a..3eeac05 100644
--- a/sites/all/modules/contrib/variable_email/variable_email.variable.inc
+++ b/sites/all/modules/contrib/variable_email/variable_email.variable.inc
@@ -25,7 +25,7 @@ function variable_email_build_mail_html($variable, $options = array()) {
$name = str_replace('[mail_part]', '', $variable['name']);
// For mail text, children have different types
$variable['children'][$name . 'subject']['type'] = 'string';
- $variable['children'][$name . 'body']['type'] = 'text_format';
+ $variable['children'][$name . 'body']['type'] = 'text';
$variable = variable_build_multiple($variable, $options);
return $variable;
}
Comments
Comment #1
imiksuWould it be possible that someone has edited the mail body with text format which you don't have access into. In that case it works as designed.
However, other email templates are tending to use just text format as you mentioned, so it wouldn't be bad idea to stay consistent.
Comment #2
roam2345 commentedthis issue is also effecting commerce here #1451680: Notice: Undefined index: module en variable_type_include().