Tickets :: [#9928] Unable to resume some html drafts correcly

6.0.0-beta1

7/4/25

Summary	Unable to resume some html drafts correcly
Queue	IMP
Queue Version	Git master
Type	Bug
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	rui.carneiro (at) portugalmail (dot) net
Created	04/18/2011 (5191 days ago)
Due
Updated	04/20/2011 (5189 days ago)
Assigned
Resolved	04/19/2011 (5190 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

04/20/2011 11:45:06 AM	rui (dot) carneiro (at) portugalmail (dot) net	Comment #5	Reply to this comment
Changes have been made in Git for this ticket: Fix resuming reply drafts in HTML mode (~~Bug #9928~~). 3 files changed, 33 insertions(+), 12 deletions(-) http://git.horde.org/horde-git/-/commit/12ed6821d898b4a6979ff012465a10472f34f918 Fixed. Thanks!

04/19/2011 07:22:21 PM	Michael Slusarz	Assigned to Michael Slusarz State ⇒ Resolved

04/19/2011 07:22:07 PM	Git Commit	Comment #4	Reply to this comment
Changes have been made in Git for this ticket: Fix resuming reply drafts in HTML mode (~~Bug #9928~~). 3 files changed, 33 insertions(+), 12 deletions(-) http://git.horde.org/horde-git/-/commit/12ed6821d898b4a6979ff012465a10472f34f918

04/19/2011 08:53:04 AM	rui (dot) carneiro (at) portugalmail (dot) net	Comment #3	Reply to this comment
Works for me. If your preferences are set to compose in text/plain by default (vs. HTML), the message will be resumed in text/plain, not text/html. I could be convinced otherwise (e.g. resuming in the richest format possible), but this is not a bug; this is expected behavior. My compose_html value is 1 and is locked. I already debugged this code. My problem is that $contents->findBody('html') is returning 2 for me. With this value my mode will never be 'html'.

04/19/2011 12:51:14 AM	Michael Slusarz	Comment #2	Reply to this comment
Works for me. If your preferences are set to compose in text/plain by default (vs. HTML), the message will be resumed in text/plain, not text/html. I could be convinced otherwise (e.g. resuming in the richest format possible), but this is not a bug; this is expected behavior.

04/18/2011 11:50:43 AM	rui (dot) carneiro (at) portugalmail (dot) net	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ Unable to resume some html drafts correcly Queue ⇒ IMP Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
After this commit (http://git.horde.org/horde-git/-/commit/1240d1c1618061647027c0940f9b5df484b46935) I am unable to resume some html messages. Test case: 1- Reply to one message from Inbox 2- Save that message as draft 3- Resume it The problem is that _getMessageText is not detecting html correctly.