Last Comment Bug 225474 - xml.cgi script has syntax problems
: xml.cgi script has syntax problems
[fixed in 2.16.5] [does not affect tr...
: regression
Product: Bugzilla
Classification: Server Software
Component: Bug Import/Export & Moving (show other bugs)
: 2.16.4
: All All
: P1 critical (vote)
: Bugzilla 2.16
Assigned To: Dave Miller [:justdave] (
: default-qa
: 226979 (view as bug list)
Depends on: 217422
  Show dependency treegraph
Reported: 2003-11-12 10:36 PST by Julio
Modified: 2012-12-18 20:46 PST (History)
4 users (show)
justdave: approval+
See Also:
QA Whiteboard:
Iteration: ---
Points: ---

Patch (1.37 KB, patch)
2003-11-12 14:04 PST, Dave Miller [:justdave] (
bbaetz: review+
Details | Diff | Splinter Review

Description Julio 2003-11-12 10:36:55 PST
User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; Q312461; .NET CLR 1.0.3705; .NET CLR 1.1.4322)
Build Identifier: 

when invoking the xml.cgi script in version 2.16.4 perl rejects it both due to 
a syntax problem in the query, and 2nd because of a misnamed column.

Reproducible: Always

Steps to Reproduce:

The following line is the patch required to fix it in the file

line 115 should be changed to:

groupset, delta_ts, ifnull(sum(votes.count),0)

No comma at the end, and the name of the count column is different.
Comment 1 Dave Miller [:justdave] ( 2003-11-12 13:44:41 PST

regression from bug 217422.

Looking at the patch that got checked in (which I wrote) that looks so damn
obvious, I could shoot myself for that.

My first thought was "why the hell is show_bug still working then?" because that
error should have killed show_bug as well.

Then it dawned on me that show_bug on the 2.16 branch wasn't using yet. 
Which means the original patch didn't fix what it was intended to fix anyway. (A
MySQL 4 compatibility issue on the show_bug screen).

New patch coming up.
Comment 2 Dave Miller [:justdave] ( 2003-11-12 13:50:27 PST
For the record, my test box for Bugzilla 2.16 only has MySQL 3.23.54 on it, so
the symptoms of the MySQL 4 problem (votes showing up blank) weren't there.  I
applied the patch, tried show_bug (completely forgetting that it didn't use to make sure it still worked, and it did.
Comment 3 Dave Miller [:justdave] ( 2003-11-12 14:04:49 PST
Created attachment 135346 [details] [diff] [review]
Comment 4 Bradley Baetz (:bbaetz) 2003-11-12 19:58:13 PST
Comment on attachment 135346 [details] [diff] [review]

Joy. r=bbaetz
Comment 5 Dave Miller [:justdave] ( 2003-11-12 20:10:08 PST
Julio: care to give this a shot and make sure it still works for you?  Since
most of us on the dev team aren't using 2.16.x on a regular basis these days
it's hard for us to test things thoroughly on it anymore :(
Comment 6 Dave Miller [:justdave] ( 2003-11-14 23:40:42 PST
Well, it works on my test install...

Checking in;
/cvsroot/mozilla/webtools/bugzilla/,v  <--
new revision:; previous revision:
Checking in;
/cvsroot/mozilla/webtools/bugzilla/Attic/,v  <--
new revision:; previous revision:
Comment 7 Ulrich Windl 2003-12-02 05:27:43 PST
Hi, I thought I had reported the same bug as well, but couldn't find it again 8-(
However I've just tested your patch for 2.16.4 and mySQL 4.0.15: I'm still getting
SQL errors:
LEFT JOIN cc select at line 271 (#1)
from bugs left join votes using(bug_id)
2.16.3 worked for the same version of mysql however. My OS is SuSE Linux 9.0, so
I guess some more people will have the same problems.
Comment 8 Dave Miller [:justdave] ( 2003-12-13 21:59:29 PST
*** Bug 226979 has been marked as a duplicate of this bug. ***
Comment 9 Dave Miller [:justdave] ( 2003-12-13 22:02:21 PST
Ulrich: what's the entire error message?  that looks abbreviated, and there's
not enough of it there for me to tell what you're hitting....
Comment 10 Ulrich Windl 2003-12-14 23:32:21 PST
I did not read the patch careful enough (I had applied it manually from one
screen to another): After removing the trailing comma in at line 115,
xml.cgi works again now. As you had already said, the bug was not obvious at
all. I guess now that the MySQL error is gone as well, you do no longer care to
see the full error messages from Apache's logs, do you?
Comment 11 Dave Miller [:justdave] ( 2003-12-14 23:35:04 PST
correct.  glad you got it working :)

Note You need to log in before you can comment on or make changes to this bug.