Hello,

here is the workaround I implemented :

Index: roundcubemail/program/steps/mail/compose.inc

--- roundcubemail/program/steps/mail/compose.inc (revision 1193) +++ roundcubemail/program/steps/mail/compose.inc (working copy) @@ -547,8 +547,8 @@ "</tbody></table><br>", Q($MESSAGE['subject']), Q($MESSAGE['headers']->date),

•                 Q($IMAP->decode_header($MESSAGE['headers']->from)),
  

•                 Q($IMAP->decode_header($MESSAGE['headers']->to)));
  

str_replace('&','&amp;',Q($IMAP->decode_header($MESSAGE['headers']->from))),

str_replace('&','&amp;',Q($IMAP->decode_header($MESSAGE['headers']->to)))); }

// add attachments @@ -925,4 +925,4 @@ $OUTPUT->set_env('contacts', $a_contacts); } parse_template('compose');

Trouble is, as soon as you switch to text mode the email address is lost. But at least this way we get it in HTML mode. I don't think this is a good way to fix this, perhaps have another function that can be called that will double up on the entity conversions?

David

On Mon, 17 Mar 2008 17:47:40 -0400, dev-bounces+roundcube.net=spam.bounceme.net@lists.roundcube.net (nobody) wrote:

Hello,

In the HTML editor, the original email address is lost when forwarding in HTML mode.

Trouble is this :

Example email : david roundcube.net@spam.bounceme.net

Email from IMAP : david roundcube.net@spam.bounceme.ne Email After it comes from Q() function : david roundcube.net@spam.bounceme.net Email as it appears in the textarea : david roundcube.net@spam.bounceme.net

When the wysiwig editor comes on, the <round.... > is thought invalid, so the editor drops it.

We would need to double htmlentity the email address. Once to get it to display in the textarea and once for the WYSIWIG parser.

The email address in the plain text mode should be passed by htmlentity once instead of never.

David

---

List info: http://lists.roundcube.net/dev/

List info: http://lists.roundcube.net/dev/