
jesse at bestpractical
Nov 25, 2009, 9:17 AM
Post #2 of 2
(336 views)
Permalink
|
|
Re: FW: Long Lines on ticket display page
[In reply to]
|
|
On Wed, Nov 25, 2009 at 09:10:30AM -0800, Potla, Ashish Bassaliel wrote: > 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 [at] bestpractical> Date: Fri Oct 30 14:54:21 2009 -0400 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 ); }; _______________________________________________ http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-users Community help: http://wiki.bestpractical.com Commercial support: sales [at] bestpractical Discover RT's hidden secrets with RT Essentials from O'Reilly Media. Buy a copy at http://rtbook.bestpractical.com
|