Commit ee88638a authored by preed%sigkill.com's avatar preed%sigkill.com

Bug 192877 - State changes on bugs w/ dependancies cause "Use of uninitialized…

Bug 192877 - State changes on bugs w/ dependancies cause "Use of uninitialized values" in BugMail.pm; r=justdave, bbaetz, a=justdave
parent bdb93447
...@@ -773,28 +773,31 @@ sub NewProcessOnePerson ($$$$$$$$$$$$$) { ...@@ -773,28 +773,31 @@ sub NewProcessOnePerson ($$$$$$$$$$$$$) {
my $difftext = ""; my $difftext = "";
my $diffheader = ""; my $diffheader = "";
my $add_diff; my $add_diff;
foreach my $diff (@diffparts) { foreach my $diff (@diffparts) {
$add_diff = 0; $add_diff = 0;
if ($diff->{'fieldname'} eq 'estimated_time' || if (exists($diff->{'fieldname'}) &&
$diff->{'fieldname'} eq 'remaining_time' || ($diff->{'fieldname'} eq 'estimated_time' ||
$diff->{'fieldname'} eq 'work_time') { $diff->{'fieldname'} eq 'remaining_time' ||
$diff->{'fieldname'} eq 'work_time')) {
if (UserInGroup(Param("timetrackinggroup"), $userid)) { if (UserInGroup(Param("timetrackinggroup"), $userid)) {
$add_diff = 1; $add_diff = 1;
} }
} else { } else {
$add_diff = 1; $add_diff = 1;
} }
if ($add_diff) { if ($add_diff) {
if ($diffheader ne $diff->{'header'}) { if (exists($diff->{'header'}) &&
($diffheader ne $diff->{'header'})) {
$diffheader = $diff->{'header'}; $diffheader = $diff->{'header'};
$difftext .= $diffheader; $difftext .= $diffheader;
} }
$difftext .= $diff->{'text'}; $difftext .= $diff->{'text'};
} }
} }
if ($difftext eq "" && $newcomments eq "") { if ($difftext eq "" && $newcomments eq "") {
# Whoops, no differences! # Whoops, no differences!
return; return;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment