179242 - Searching for comment contains string + email gives "Not unique table/alias: 'longdescs_' at globals.pl line 242"

Status: RESOLVED FIXED

(Bugzilla :: Query/Bug List, defect, P2)

Description

[John Levon]

e.g. set "comment contains string" to "history",
email commenter is "[e-mail]" :

http://bugzilla.mozilla.org/buglist.cgi?short_desc_type=allwordssubstr&short_desc=&long_desc_type=substring&long_desc=history&bug_file_loc_type=allwordssubstr&bug_file_loc=&status_whiteboard_type=allwordssubstr&status_whiteboard=&keywords_type=allwords&keywords=&bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&bug_status=RESOLVED&emaillongdesc1=1&emailtype1=exact&email1=moz%40compsoc.man.ac.uk&emailtype2=exact&email2=&bugidtype=include&bug_id=&votes=&changedin=&chfieldfrom=&chfieldto=Now&chfieldvalue=&cmdtype=doit&remaction=run&namedcmd=cc%27d&order=Reuse+same+sort+as+last+time&field0-0-0=noop&type0-0-0=noop&value0-0-0=

and you get the error given in the summary.

Comment 1

[Joel Peshkin]

Attachment: Patch

OK, this results from the join of longdescs from the chart-based email matches.

The table aliases now differentiate more specifically.

Comment 2

[Jacob Steenhagen]

Attachment: Make the alias in the email section unique

Comment 3

[Joel Peshkin]

This should go into 2.17.1

Comment 4

[Jacob Steenhagen]

Comment on attachment 105698 [details] [diff] [review]
Patch

My patch was identenital (minus the fact I forgot the -u in my diff command) so
I think we can call that an r= :)

Comment 5

[Myk Melez [:myk] [@mykmelez]]

a=myk

Comment 6

[Joel Peshkin]

/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Search.pm,v  <--  Search.pm
new revision: 1.29; previous revision: 1.28
done                                                         

Comment 7

[Bradley Baetz (:bbaetz)]

REOPEN - there are two branches to that if statement; you need to fix both

Comment 8

[Jacob Steenhagen]

Attachment: Handle the else block

Comment 9

[Bradley Baetz (:bbaetz)]

Comment on attachment 105726 [details] [diff] [review]
Handle the else block

yeah, thats what I had too

Comment 10

[Jacob Steenhagen]

Checking in Search.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Search.pm,v  <--  Search.pm
new revision: 1.32; previous revision: 1.31
done