On 2/27/2014 12:09 PM, John Hardin wrote:
At best, I think this is added in Message.pm with my $text =
$self->get_header ('subject') || "\n";
That's been in SA for a *long* time.
Yeah, I had a feeling. I even seem to remember figuring this out eons
ago ;-)
Perhaps we need to add a tflag to not add subject for body rules to
ignore this step.
That would pretty much require two $texts, wouldn't it? body rules
without that tflag would expect to run against the header as well...
To "properly" implement that tflag, in addition to not doing the
concatenation wouldn't all body rules not having the tflag also have
to be explicitly run against the Subject header so they maintain the
current semantics?
The current implementation would remain as-is. the tflag would ne a
negation such as tflag <rulename> do_not_add_subject which would only
affect body type rules. No need for add_subject as that's existing
behavior.
My thought process at least.
