FW: Long Lines on ticket display page

Sorry about that . It is 3.8.2 (mod_perl2) on a sun solaris machine with an Oracle Db.

Let me know if you need any other info.

Regards,
-AshishFrom: Jesse Vincent [jesse@bestpractical.com]
Sent: Wednesday, November 25, 2009 10:31 PM
To: Potla, Ashish Bassaliel
Cc: rt-users@bestpractical.com
Subject: Re: [rt-users] Long Lines on ticket display page

Hello,

Some bizarre things have been happening and would like to know if anyone has seen similar cases.

What version of RT are you folks running?

-Jesse

  1.   When a long line is in the message body, it runs off the screen.
    
  2.   Sometimes, long words, like path names are unnecessarily truncated and wrapped to the next line even though the end
    

is not reached.

  1.   Inconsistent new lines get added to the message body(especially to indented previous messages) and keeps making the
    

email longer and longer each time correspondence is added to it.

Please help me out!

Thank you,

-Ashish


http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-users

Community help: http://wiki.bestpractical.com
Commercial support: sales@bestpractical.com

Discover RT’s hidden secrets with RT Essentials from O’Reilly Media.
Buy a copy at http://rtbook.bestpractical.com

Sorry about that . It is 3.8.2 (mod_perl2) on a sun solaris machine with an Oracle Db.

We’ve definitely made wrapping improvements in the past couple point
releases. And one since that last release:

commit 4eb846f4b7da66940f6bc479a0367d226969e99d
Author: Jesse Vincent jesse@bestpractical.com

RT was accidentally injecting too many newlines when rendering plaintext messages without <pre>.

This commit fixes the regex.

diff --git a/share/html/Ticket/Elements/ShowMessageStanza b/share/html/Ticket/Elements/ShowMessageStanza
index e9b57bb…0d4fe61 100755
— a/share/html/Ticket/Elements/ShowMessageStanza
+++ b/share/html/Ticket/Elements/ShowMessageStanza
@@ -98,7 +98,7 @@ my $print_content = sub {
$m->callback( content => $ref, %ARGS );
$m->comp(’/Elements/MakeClicky’, content => $ref, ticket => $ticket, %ARGS);
unless ( $plain_text_pre || $plain_text_mono ) {

  •    $$ref =~ s{(?=\r*\n)}{<br />}g if defined $$ref;
    
  •    $$ref =~ s{(\r?\n)}{<br />}g if defined $$ref;
    
    }
    $m->out( $$ref );
    };