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;
--------------------------------------------------------------------------------

Post a comment in response:

This account has disabled anonymous posting.
If you don't have an account you can create one now.
HTML doesn't work in the subject.
More info about formatting

If you are unable to use this captcha for any reason, please contact us by email at support@dreamwidth.org