2010-10-08 Adam Barth <abarth@webkit.org>
authorabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 9 Oct 2010 00:18:37 +0000 (00:18 +0000)
committerabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 9 Oct 2010 00:18:37 +0000 (00:18 +0000)
        Reviewed by Oliver Hunt.

        Bugmail for new attachments should link to the reviewtool
        https://bugs.webkit.org/show_bug.cgi?id=47440

        As requested by olliej himself.  This patch was constructed by pure
        reason.  I have no way to actually run this code.

        * Bugzilla/BugMail.pm:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@69434 268f45cc-cd09-0410-ab3c-d52691b4dbfc

BugsSite/Bugzilla/BugMail.pm
BugsSite/ChangeLog

index d4c9a47..6f78bc2 100644 (file)
@@ -609,7 +609,7 @@ sub sendMail {
         my $showattachurlbase =
             Bugzilla->params->{'urlbase'} . "attachment.cgi?id=";
 
-        $newcomments =~ s/(Created an attachment \(id=([0-9]+)\))/$1\n --> \(${showattachurlbase}$2\)/g;
+        $newcomments =~ s/(Created an attachment \(id=([0-9]+)\))/$1\n --> \(${showattachurlbase}$2&action=review\)/g;
     }
 
     my $diffs = $difftext . "\n\n" . $newcomments;
index ac2c098..c30095e 100644 (file)
@@ -1,3 +1,15 @@
+2010-10-08  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Oliver Hunt.
+
+        Bugmail for new attachments should link to the reviewtool
+        https://bugs.webkit.org/show_bug.cgi?id=47440
+
+        As requested by olliej himself.  This patch was constructed by pure
+        reason.  I have no way to actually run this code.
+
+        * Bugzilla/BugMail.pm:
+
 2010-09-28  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
 
         Reviewed by Csaba Osztrogonác.