Closed Bug 225474 Opened 21 years ago Closed 21 years ago

xml.cgi script has syntax problems


(Bugzilla :: Bug Import/Export & Moving, defect, P1)




Bugzilla 2.16


(Reporter: jch_2000, Assigned: justdave)



(Keywords: regression, Whiteboard: [fixed in 2.16.5] [does not affect trunk])


(1 file)

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.
Version: unspecified → 2.16.4

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.
Severity: normal → critical
Depends on: 217422
Ever confirmed: true
Keywords: regression
OS: Windows XP → All
Priority: -- → P1
Hardware: PC → All
Whiteboard: [wanted for 2.16.5] [does not affect trunk]
Target Milestone: --- → Bugzilla 2.16
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.
Attached patch PatchSplinter Review
Comment on attachment 135346 [details] [diff] [review]

Joy. r=bbaetz
Attachment #135346 - Flags: review+
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 :(
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:
Closed: 21 years ago
Flags: approval+
Resolution: --- → FIXED
Whiteboard: [wanted for 2.16.5] [does not affect trunk] → [fixed in 2.16.5] [does not affect trunk]
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.
*** Bug 226979 has been marked as a duplicate of this bug. ***
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....
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?
correct.  glad you got it working :)
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.