Closed Bug 225474 Opened 21 years ago Closed 21 years ago

xml.cgi script has syntax problems

Categories

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

2.16.4
defect

Tracking

()

RESOLVED FIXED
Bugzilla 2.16

People

(Reporter: jch_2000, Assigned: justdave)

References

Details

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

Attachments

(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: 1. 2. 3. The following line is the patch required to fix it in the file Bug.pm: 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
F#$@$!!!!! 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 Bug.pm 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
Status: UNCONFIRMED → ASSIGNED
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 Bug.pm) to make sure it still worked, and it did.
Attached patch Patch (deleted) — Splinter Review
Comment on attachment 135346 [details] [diff] [review] Patch 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 Bug.pm; /cvsroot/mozilla/webtools/bugzilla/Bug.pm,v <-- Bug.pm new revision: 1.12.2.2; previous revision: 1.12.2.1 done Checking in bug_form.pl; /cvsroot/mozilla/webtools/bugzilla/Attic/bug_form.pl,v <-- bug_form.pl new revision: 1.93.2.4; previous revision: 1.93.2.3 done
Status: ASSIGNED → RESOLVED
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 globals.pl 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 Bug.pm 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.

Attachment

General

Created:
Updated:
Size: