Closed
Bug 217422
Opened 21 years ago
Closed 21 years ago
"0" is missing in "votes: 0" (MySQL 4)
Categories
(Bugzilla :: Creating/Changing Bugs, defect)
Bugzilla
Creating/Changing Bugs
Tracking
()
RESOLVED
FIXED
Bugzilla 2.16
People
(Reporter: jruderman, Assigned: justdave)
References
()
Details
(Whiteboard: [fixed in 2.16.4] [fixed in 2.17.5])
Attachments
(2 files)
654 bytes,
patch
|
bbaetz
:
review+
|
Details | Diff | Splinter Review |
703 bytes,
patch
|
bbaetz
:
review+
|
Details | Diff | Splinter Review |
In bugs with 0 votes, the "0" doesn't appear. In a bug with more than 0 votes,
the number does appear.
Dave Miller says:
mysql> select sum(votes.count) from votes where bug_id=216434;
+------------------+
| sum(votes.count) |
+------------------+
| NULL |
+------------------+
1 row in set (0.00 sec)
MySQL 4 returns NULL if there were no matching values to count. MySQL 3.23
returned 0.
Assignee | ||
Comment 1•21 years ago
|
||
This is in Bug.pm. fixing component
Component: Bugzilla-General → Creating/Changing Bugs
Assignee | ||
Comment 2•21 years ago
|
||
Assignee | ||
Updated•21 years ago
|
Attachment #130454 -
Flags: review?(myk)
Assignee | ||
Updated•21 years ago
|
Target Milestone: --- → Bugzilla 2.18
Comment 3•21 years ago
|
||
Comment on attachment 130454 [details] [diff] [review]
Patch
Won't work in pg, but r=bbaetz since we don't have an abstraction layer for
hose sort of non-standard mysql-isms yet.
Attachment #130454 -
Flags: review?(myk) → review+
Assignee | ||
Comment 4•21 years ago
|
||
Checking in Bug.pm;
/cvsroot/mozilla/webtools/bugzilla/Bug.pm,v <-- Bug.pm
new revision: 1.33; previous revision: 1.32
done
Status: NEW → RESOLVED
Closed: 21 years ago
Flags: approval+
Resolution: --- → FIXED
Assignee | ||
Comment 5•21 years ago
|
||
Low risk, needed for MySQL 4 compatibility. Nominating for 2.16 branch.
Whiteboard: [wanted for 2.16.4] [fixed in 2.17.5]
Target Milestone: Bugzilla 2.18 → Bugzilla 2.16
Assignee | ||
Comment 6•21 years ago
|
||
Assignee | ||
Updated•21 years ago
|
Attachment #132158 -
Flags: review?(bbaetz)
Updated•21 years ago
|
Attachment #132158 -
Flags: review?(bbaetz) → review+
Assignee | ||
Comment 7•21 years ago
|
||
checked in on 2.16 branch
Checking in Bug.pm;
/cvsroot/mozilla/webtools/bugzilla/Bug.pm,v <-- Bug.pm
new revision: 1.12.2.1; previous revision: 1.12
done
OS: Windows XP → All
Hardware: PC → All
Whiteboard: [wanted for 2.16.4] [fixed in 2.17.5] → [fixed in 2.16.4] [fixed in 2.17.5]
Assignee | ||
Comment 8•21 years ago
|
||
Comment on attachment 132158 [details] [diff] [review]
Patch for 2.16 branch (incomplete - see comment 8)
FYI to anyone watching this bug, the patch for the 2.16 branch on this bug is
broken/incomplete. There is a corrective patch on bug 225474.
Attachment #132158 -
Attachment description: Patch for 2.16 branch → Patch for 2.16 branch (incomplete - see comment 8)
Updated•12 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•