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

RESOLVED FIXED in Bugzilla 2.18

Status

()

Bugzilla
Query/Bug List
P2
major
RESOLVED FIXED
15 years ago
5 years ago

People

(Reporter: John Levon, Assigned: Joel Peshkin)

Tracking

2.17.1
Bugzilla 2.18

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

15 years ago
e.g. set "comment contains string" to "history",
email commenter is "moz@compsoc.man.ac.uk" :

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.
(Assignee)

Comment 1

15 years ago
Created attachment 105698 [details] [diff] [review]
Patch


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

The table aliases now differentiate more specifically.

Comment 2

15 years ago
Created attachment 105699 [details] [diff] [review]
Make the alias in the email section unique
(Assignee)

Comment 3

15 years ago
This should go into 2.17.1
Assignee: endico → bugreport
Severity: normal → major
Priority: -- → P2
Target Milestone: --- → Bugzilla 2.18

Comment 4

15 years ago
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= :)
Attachment #105698 - Flags: review+

Updated

15 years ago
Attachment #105699 - Attachment is obsolete: true
(Assignee)

Updated

15 years ago
Blocks: 179196
Status: NEW → ASSIGNED
(Assignee)

Updated

15 years ago
No longer blocks: 179196
(Assignee)

Updated

15 years ago
Blocks: 179176
a=myk
(Assignee)

Comment 6

15 years ago

/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Search.pm,v  <--  Search.pm
new revision: 1.29; previous revision: 1.28
done                                                         
Status: ASSIGNED → RESOLVED
Last Resolved: 15 years ago
OS: Linux → All
Hardware: PC → All
Resolution: --- → FIXED
REOPEN - there are two branches to that if statement; you need to fix both
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

Comment 8

15 years ago
Created attachment 105726 [details] [diff] [review]
Handle the else block

Updated

15 years ago
Attachment #105726 - Flags: review?
Comment on attachment 105726 [details] [diff] [review]
Handle the else block

yeah, thats what I had too
Attachment #105726 - Flags: review?
Attachment #105726 - Flags: review+

Comment 10

15 years ago
Checking in Search.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Search.pm,v  <--  Search.pm
new revision: 1.32; previous revision: 1.31
done
Status: REOPENED → RESOLVED
Last Resolved: 15 years ago15 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.