Posted by our1944 on October 30, 2009 at 7:06am
Jump to:
| Project: | Mailhandler |
| Version: | 6.x-1.8 |
| Component: | Mailhandler |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs work |
Issue Summary
I made a patch against mailhandler 6.x-1.8 to enable the commands in html email. I hope it is helpful to someone.
Thanks,
Sebastian
| Attachment | Size |
|---|---|
| mailhandler.retrieve.inc_.html_command.patch | 1.63 KB |
Comments
#1
The main problem I see with this is that it no longer assumes commands will be on the first line of the body. This makes the distinction between an email without commands and an email with commands less clear, since commands could now be anywhere in the body (by allowing an arbitrary commands start line, and not assuming it's the first line). That said, there's already the assumption that the first line cannot start with a word followed by a colon unless it's supposed to be a command. The problem w/ stripping all tags from whole body before iterating is that it makes stripping the commands and signature from the body more difficult (line numbers are arbitrary). Maybe the only way around this is to allow specification of an optional commands delimiter.
#2
Marked http://drupal.org/node/567790 as duplicate.
#3
Ramblings:
This is basically more an implementation issue. Using the body of the email for commands is a nice original design however brings along these indicated disadvantages. The email header is actually the place where such things belong. Here the difficulty is that most people have a hard time adjusting their email headers either manually or via a small script.
A HTML mail should actually also contain the plain mail and that is where the command should reside and picked up not from the HTML part, however also not easy for others to craft such an email
#4
Marked #976084: Node body HTML getting cut off on import as dupe.