From: lpsolit%gmail.com <>
Date: Mon, 2 Mar 2009 20:23:34 +0000 (+0000)
Subject: Bug 480999: Irrelevant username check in the 'Email Addresses, Bug Numbers, and Votes... 
X-Git-Tag: bugzilla-3.3.4~13
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=57886ccea2f55e517bd06e7a8f5b90bbf407ab97;p=thirdparty%2Fbugzilla.git

Bug 480999: Irrelevant username check in the 'Email Addresses, Bug Numbers, and Votes' section if no checkbox is checked - Patch by Frédéric Buclin <LpSolit@gmail.com> r/a=mkanat
---

diff --git a/Bugzilla/Search.pm b/Bugzilla/Search.pm
index 2f86d2502b..0c0a76562b 100644
--- a/Bugzilla/Search.pm
+++ b/Bugzilla/Search.pm
@@ -237,15 +237,7 @@ sub init {
             next;
         }
         my $type = $params->param("emailtype$id");
-        if ($type eq "exact") {
-            $type = "anyexact";
-            foreach my $name (split(',', $email)) {
-                $name = trim($name);
-                if ($name) {
-                    login_to_id($name, THROW_ERROR);
-                }
-            }
-        }
+        $type = "anyexact" if ($type eq "exact");
 
         my @clist;
         foreach my $field ("assigned_to", "reporter", "cc", "qa_contact") {
@@ -259,6 +251,17 @@ sub init {
         if (@clist) {
             push(@specialchart, \@clist);
         }
+        else {
+            # No field is selected. Nothing to see here.
+            next;
+        }
+
+        if ($type eq "anyexact") {
+            foreach my $name (split(',', $email)) {
+                $name = trim($name);
+                login_to_id($name, THROW_ERROR) if $name;
+            }
+        }
     }
 
     my $chfieldfrom = trim(lc($params->param('chfieldfrom'))) || '';