Closed
Bug 191051
Opened 22 years ago
Closed 22 years ago
emailtype2 substring search returns no hits
Categories
(Bugzilla :: Query/Bug List, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.18
People
(Reporter: danm.moz, Assigned: myk)
Details
Attachments
(1 file)
675 bytes,
patch
|
justdave
:
review+
|
Details | Diff | Splinter Review |
A search for bugs with bug owner containing danm and cc: list member containing
jst finds no bugs. So
http://bugzilla.mozilla.org/buglist.cgi?bug_status=NEW&
emailassigned_to1=1&emailtype1=substring&email1=danm&
emailcc2=1&emailtype2=substring&email2=jst
gets nothing while
http://bugzilla.mozilla.org/buglist.cgi?bug_status=NEW&
emailassigned_to1=1&emailtype1=substring&email1=danm&
emailcc2=1&emailtype2=substring&email2=jst@netscape.com
does find several bugs. (Without the embedded linefeeds, of course.) The only
difference between the two search strings is the last field, where the second
email field substring value is specied. So a substring of "jst" finds nothing,
while a substring of "jst@netscape.com" does.
Assignee | ||
Comment 3•22 years ago
|
||
The function that handles "anyexact" and "substring" "cc" field searches is
doing "anyexact" in both cases. This fix uses replaces the hard-coded
"anyexact" with the term variable ($t).
Assignee | ||
Updated•22 years ago
|
Attachment #113140 -
Flags: review?(justdave)
Comment 4•22 years ago
|
||
but since Myk actually fixed it, we'll give it to him anyway :-)
Assignee: endico → myk
Updated•22 years ago
|
Attachment #113140 -
Flags: review?(justdave) → review+
Updated•22 years ago
|
Flags: approval+
Assignee | ||
Comment 6•22 years ago
|
||
Yes, thanks for pinging me about it.
Checking in Bugzilla/Search.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Search.pm,v <-- Search.pm
new revision: 1.38; previous revision: 1.37
done
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•