fu: Close-up of Fu, bringing a scoop of water to her mouth (Default)
fu ([personal profile] fu) wrote in [site community profile] changelog2010-11-26 04:24 pm

[dw-free] Wrap links in email notifications with spaces

[commit: http://hg.dwscoalition.org/dw-free/rev/2d7f013b42b0]

http://bugs.dwscoalition.org/show_bug.cgi?id=3253

Add spaces around the system-generated link to the entry, so that it doesn't
confuse email clients.

Patch by [personal profile] ninetydegrees.

Files modified:
  • cgi-bin/LJ/Comment.pm
--------------------------------------------------------------------------------
diff -r 2dd136674a6b -r 2d7f013b42b0 cgi-bin/LJ/Comment.pm
--- a/cgi-bin/LJ/Comment.pm	Sat Nov 27 00:15:11 2010 +0800
+++ b/cgi-bin/LJ/Comment.pm	Sat Nov 27 00:24:06 2010 +0800
@@ -1358,7 +1358,7 @@ sub _format_mail_both {
         $vars->{closelink} = "</a>";
     } else {
         $vars->{openlink}  = '';
-        $vars->{closelink} = " ($talkurl)";
+        $vars->{closelink} = " ( $talkurl )";
     }
 
     my $subject = $is_html ? $entry->subject_html : $entry->subject_text;
--------------------------------------------------------------------------------