Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
In mimemail.rules.inc in mimemail_rules_action_info() please set "sanitize" to FALSE for body param
'body' => array(
'type' => 'text',
'label' => t('Body'),
'description' => t('The mail\'s HTML body. Will be formatted using the text format selected on the <a href="@url">settings</a> page.', array('@url' => url('admin/config/system/mimemail'))),
'sanitize' => FALSE, //<--- HERE SHOULD BE FALSE
'optional' => TRUE,
'translatable' => TRUE,
),
It gives much problems for custom tokens which contain HTML markup.
Comments
Comment #2
TR CreditAttribution: TR commentedThat would break #1814922: Rule sanitizes the $body if populated by a parameter, which requires sanitize to be TRUE.
Can you give specific details about how to reproduce your problem?
Comment #3
TR CreditAttribution: TR commentedSet wrong status ...
Comment #4
Mykola DolynskyiI don`t understand what it would break.
All works fine if body sanitize = false.
Just now after update I killed 3 days again debugging this problem, because I forgot about this 'sanitize'=>true set in mimemail code. Hundreds of emails with broken markup with workflow notifications were sent because of this to people including bosses :(
Part of my comment from topic you showed
Custom module solution:
Comment #5
TR CreditAttribution: TR commentedAs I said, this reverses the changes made in #1814922: Rule sanitizes the $body if populated by a parameter so if you expect this to be fixed you really need to make an argument that the previous change was wrong, or you need to present a solution that works for that previous issue as well as this current issue.
Without steps to demonstrate this problem, we cannot do anything.