Re-fix for bug 29820: remove "Changed" from email subject line. Based on patch…

Re-fix for bug 29820: remove "Changed" from email subject line. Based on patch submitted by Stephan Niemz [faniz] <st.n@gmx.net>, r= jake, cyeh UPGRADE NOTES: when you install this update, you will need to change the subject line in your 'changedmail' and 'newchangedmail' params from the web by running editparams.cgi. The subject line needs to be changed from: Subject: [Bug %bugid%] %neworchanged% - %summary% to Subject: [Bug %bugid%] %neworchanged%%summary% Or whatever is appropriate for the subject you are using on your system. Note the removal of the " - " in the middle.

Re-fix for bug 29820: remove "Changed" from email subject line. Based on patch…
9818692e · justdave%syndicomm.com · 15bbeca6 · 9818692e · 9818692e
Commit 9818692e authored Apr 07, 2001 by justdave%syndicomm.com
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 6 deletions

defparams.pl defparams.pl +4 -4

processmail processmail +2 -2

No files found.
--- a/defparams.pl
+++ b/defparams.pl
@@ -381,8 +381,8 @@ separated by a comma (with duplication removed, if they're the same
 person).  %cc% gets replaced by the list of people on the CC list,
 separated by commas.  %bugid% gets replaced by the bug number.
 %diffs% gets replaced by the diff text from the old version to the new
-version of this bug.  %neworchanged% is either "New" or "Changed",
+version of this bug.  %neworchanged% is "New: " if this mail is
-depending on whether this mail is reporting a new bug or changes made
+reporting a new bug or empty if changes were made
 to an existing one.  %summary% gets replaced by the summary of this
 bug.  %<i>anythingelse</i>% gets replaced by the definition of that
 parameter (as defined on this page).},
@@ -390,7 +390,7 @@ parameter (as defined on this page).},
 "From: bugzilla-daemon
 To: %to%
 Cc: %cc%
-Subject: [Bug %bugid%] %neworchanged% - %summary%
+Subject: [Bug %bugid%] %neworchanged%%summary%
 %urlbase%show_bug.cgi?id=%bugid%
@@ -418,7 +418,7 @@ q{The same as 'changedmail', but used for the newemailtech stuff.},
 "From: bugzilla-daemon
 To: %to%
 Cc: %cc%
-Subject: [Bug %bugid%] %neworchanged% - %summary%
+Subject: [Bug %bugid%] %neworchanged%%summary%
 %urlbase%show_bug.cgi?id=%bugid%

--- a/processmail
+++ b/processmail
@@ -949,7 +949,7 @@ sub NewProcessOnePerson ($$$$$$$$$$) {
 # since if the bug didn't change, you wouldn't be getting mail
 # in the first place! see http://bugzilla.mozilla.org/show_bug.cgi?id=29820 
 # for details.
-    $substs{"neworchanged"} = $isnew ? "New" : "";
+    $substs{"neworchanged"} = $isnew ? 'New: ' : '';
    $substs{"to"} = $person;
    $substs{"cc"} = '';
    $substs{"bugid"} = $id;
@@ -1042,7 +1042,7 @@ sub ProcessOneBug {
                $substs{"diffs"} .= $_;
            }
            close DIFFS;
-            $substs{"neworchanged"} = $verb;
+            $substs{"neworchanged"} = ($verb eq "New") ? "New: " : "";
            $substs{"summary"} = $::bug{'short_desc'};
            my $msg = PerformSubsts(Param("changedmail"), \%substs);