Bug 533018: "Confirm match" displays full email address to logged-out users in reques...

r/a=mkanat

diff --git a/Bugzilla/User.pm b/Bugzilla/User.pm
index 2ae12efcc36c3cba3af5ed79e7d854b6dd268b5e..b55313c6d0b58fafc6d18b55113c40d826d12ab2 100644 (file)
--- a/Bugzilla/User.pm
+++ b/Bugzilla/User.pm
@@ -1026,6 +1026,7 @@ sub match {
     my $wildstr = $str;
 
     if ($wildstr =~ s/\*/\%/g # don't do wildcards if no '*' in the string
+        && $user->id
         # or if we only want exact matches
         && Bugzilla->params->{'usermatchmode'} ne 'off') 
     {
@@ -1068,6 +1069,7 @@ sub match {
 
     # then try substring search
     if ((scalar(@users) == 0)
+        && $user->id
         && (Bugzilla->params->{'usermatchmode'} eq 'search')
         && (length($str) >= 3))
     {

diff --git a/template/en/default/global/confirm-user-match.html.tmpl b/template/en/default/global/confirm-user-match.html.tmpl
index 5b209dfce8f558c0b6184fd66ebfebe3bd84b185..52607614c589598e8fbf7e1ea608d92cede66deb 100644 (file)
--- a/template/en/default/global/confirm-user-match.html.tmpl
+++ b/template/en/default/global/confirm-user-match.html.tmpl
@@ -41,7 +41,7 @@
 [% IF matchsuccess == 1 %]
   [% PROCESS global/header.html.tmpl title="Confirm Match" %]
 
-[% USE Bugzilla %]
+  [% USE Bugzilla %]
 
   <form method="post" 
   [% IF script -%]
@@ -70,9 +70,13 @@
   [% PROCESS global/header.html.tmpl title="Match Failed" %]
   <p>
     [% terms.Bugzilla %] was unable to make any match at all for one or more of
-    the names and/or email addresses you entered on the previous page.<br>
-    Please go back and try other names or email addresses.
+    the names and/or email addresses you entered on the previous page.
+    [% IF !user.id %]
+      <b>Note: You are currently logged out. Only exact matches against e-mail
+      addresses will be performed.</b>
+    [% END %]
   </p>
+  <p>Please go back and try other names or email addresses.</p>
 [% END %]
 
   <table border="0">