Re: [RCD] Problem with html entities

[nobody]

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,
[EMAIL PROTECTED] (nobody)
wrote:
> Hello,
> 
> In the HTML editor, the original email address is lost when forwarding in
> HTML mode.
> 
> Trouble is this :
> 
> Example email : david <[EMAIL PROTECTED]>
> 
> Email from IMAP : david <[EMAIL PROTECTED]>
> Email After it comes from Q() function : david
> <[EMAIL PROTECTED]>
> Email as it appears in the textarea    : david
> <[EMAIL PROTECTED]>
> 
> 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/