Bug 903682

Summary:	Double-quoting message with HTML and text
Product:	[Fedora] Fedora	Reporter:	Matthew Saltzman <mjs>
Component:	evolution	Assignee:	Matthew Barnes <mbarnes>
Status:	CLOSED UPSTREAM	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	18	CC:	fedora, kalevlember, lucilanga, mbarnes, mclasen, mcrha, phatina, tpopela
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-01-29 10:07:17 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Matthew Saltzman 2013-01-24 15:01:31 UTC

Description of problem:
When replying to a message with both text and html bodies, the reply quotes the body twice.

Version-Release number of selected component (if applicable):
evolution-3.6.3-1.fc18.x86_64

How reproducible:
Always

Steps to Reproduce:
1. Reply to a message that has text and HTML copies.
2.
3.
  
Actual results:
Body of message is quoted twice

Expected results:
Body of message is quoted once.

Additional info:
This is a regression from F17.

Comment 1 Milan Crha 2013-01-24 17:12:55 UTC

Thanks for a bug report. After a quick testing it works fine for me. I think this is a webkit issue, because once evolution generates the page it doesn't touch it, definitely not during scrolling - as far as I know. I use webkitgtk3-1.10.1-1.fc18.x86_64.

Comment 2 Milan Crha 2013-01-24 17:15:04 UTC

(In reply to comment #1)
> Thanks for a bug report. After a quick testing it works fine for me. I think
> this is a webkit issue, because once evolution generates the page it doesn't
> touch it, definitely not during scrolling - as far as I know. I use
> webkitgtk3-1.10.1-1.fc18.x86_64.

Err, I'm sorry, thus was supposed to come to bug #903698

Comment 3 Milan Crha 2013-01-25 17:22:33 UTC

I suppose this is fixed with evolution 3.6.3-2, which has a fix for bug #903469, which I believe is related. Could you retest, please? (Works for me on git master, which has the fix included too.)

Comment 4 Matthew Saltzman 2013-01-26 14:23:47 UTC

No, this version still exhibits the problem.  Maybe some other patch still not released?

Comment 5 Milan Crha 2013-01-28 08:07:18 UTC

Hmm, either that, or a your message is a "corner case". Could you check settings in Edit->Preferences->Mail Preferences->HTML Messages tab, Plain Text Mode section, what you've set there, please? I have there
  [x] Show suppressed HTML parts as attachments
  Html Mode [ Show HTML if present     | v ]

And in Composer Settings, in section Replies and Forwards I've:
  Reply style: [ Quoted   | v ]

Comment 6 Matthew Saltzman 2013-01-28 14:11:55 UTC

I have "show suppressed..." checked and HTML mode set to "show plain text if present."  Reply style is "quoted". Nothing else is checked in Composer Preferences except "group replies go to mailing list". (No character encoding is selected.)

I tried changing HTML mode to "show HTML if present" and that does change the behavior.  So it is the "show plain text" setting that exhibits the problem.

Is that really considered a "corner case"?

Comment 7 Milan Crha 2013-01-29 10:07:17 UTC

Hmm, I thought about other issue there, but as it does this with show plain text only, then it's a bug. I'm moving this upstream, as [1].

[1] https://bugzilla.gnome.org/show_bug.cgi?id=692775