493049 - Remove original quoted email text from incoming email

Status: NEW

(Bugzilla :: Incoming Email, enhancement, P3)

Description

[Tom Ryan]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US) AppleWebKit/525.19 (KHTML, like Gecko) Chrome/1.0.154.65 Safari/525.19
Build Identifier: 

When people reply to a Bugzilla email, often they forget to delete the "original message" at the bottom of their response. This makes a complete mess of the Bugzilla comments database.

This problem is especially common among new, unsophisticated or casual users of Bugzilla. It is those same users that often benefit the most from the incoming email feature.

The proposal is to automatically truncate the end of an email message that, in the case of Microsoft Outlook, usually begins with the string "Original Message" . If this code is invoked, an email should be sent back to the sender to warn them that the truncation occurred.

For extra credit, I would like to see a system preference for this feature so that it could get rolled into the main source tree. The preference would have the following options:

Enable/Disable
Truncation Strings
Truncation Warning email text

The incoming email feature is extremely useful, it just needs a little fine tuning.

Reproducible: Always

Comment 1

[Max Kanat-Alexander]

Note that we should only truncate if it's quoted text all the way down after the matching line.

The matches would just be a series of regexes that match the common text that starts at the top of replies.

Comment 2

[Michael Ching]

Attachment: Remove quoted content at end of a reply in email_in.pl

This patch should contain a regex which:
1.  Detects a few common "start of quote" markers.
2.  Checks that lines after that start with a '>' or are all whitespace.
3.  Disregards previous rule past a '--' signature marker.

Comment 3

[Tom Ryan]

Michael, can you post the latest version?

Comment 4

[Michael Ching]

Attachment: Remove quoted content at end of a reply in email_in.pl

Adds run-time flags to control behavior, cleans up various notifications.

Comment 5

[Max Kanat-Alexander]

Hey Michael. If you're interested in this patch becoming a part of Bugzilla, you'll want to read our development process:

  http://wiki.mozilla.org/Bugzilla:Developers

Comment 6

[Marc Schumann [:Wurblzap]]

Comment on attachment 387890 [details] [diff] [review]
Remove quoted content at end of a reply in email_in.pl

In no way do I want to keep this from going in, but I'd really love to see this being localizable.

(In reply to comment #5)

Ok, Michael, I hope you don't mind I'm asking for review for you.

Comment 7

[Max Kanat-Alexander]

Marc: I agree with you about localizability; we should be able to put regexes in messages.html.tmpl or strings.txt.tmpl.

Comment 8

[Frédéric Buclin]

Comment on attachment 387890 [details] [diff] [review]
Remove quoted content at end of a reply in email_in.pl

I don't think we need to reply to the sender that the quoted text has been removed. This should be done silently, which would fix the localization problem.