diff options
author | lpsolit%gmail.com <> | 2005-04-13 05:21:44 +0200 |
---|---|---|
committer | lpsolit%gmail.com <> | 2005-04-13 05:21:44 +0200 |
commit | add70bda3bba58c8e39711215e9d6f31b42bf598 (patch) | |
tree | c89fa02b306f1b109cab3f7e5fb437bc220854c0 /Bugzilla | |
parent | 703a821f34f126ad281ef60e4ff3ec1b237a7798 (diff) | |
download | bugzilla-add70bda3bba58c8e39711215e9d6f31b42bf598.tar.gz bugzilla-add70bda3bba58c8e39711215e9d6f31b42bf598.tar.xz |
Bug 289729: fix several "uninitialized value" messages - Patch by Frédéric Buclin <LpSolit@gmail.com> r=wurblzap a=justdave
Diffstat (limited to 'Bugzilla')
-rw-r--r-- | Bugzilla/Flag.pm | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/Bugzilla/Flag.pm b/Bugzilla/Flag.pm index 0aca49c87..29215876a 100644 --- a/Bugzilla/Flag.pm +++ b/Bugzilla/Flag.pm @@ -280,7 +280,10 @@ sub validate { && trim($cgi->param("requestee-$id"))) { my $requestee_email = trim($cgi->param("requestee-$id")); - if ($requestee_email ne $flag->{'requestee'}->{'email'}) { + my $old_requestee = + $flag->{'requestee'} ? $flag->{'requestee'}->login : ''; + + if ($old_requestee ne $requestee_email) { # We know the requestee exists because we ran # Bugzilla::User::match_field before getting here. my $requestee = Bugzilla::User->new_from_login($requestee_email); @@ -551,7 +554,7 @@ sub modify { my $flag = get($id); my $status = $cgi->param("flag-$id"); - my $requestee_email = trim($cgi->param("requestee-$id")); + my $requestee_email = trim($cgi->param("requestee-$id") || ''); # Ignore flags the user didn't change. There are two components here: |