148093 - Editmilestones shows 'xyzzy' as the product bug count

Status: RESOLVED FIXED

(Bugzilla :: Administration, task)

Description

[Jouni Heikniemi]

Go to editmilestones.cgi without a parameter, and you'll see a bug table with
bug counts written as 'xyzzy'.  

A ridiculously old minor bug, but I'll attach a patch anyway.

Comment 1

[Jouni Heikniemi]

Attachment: Patch

Takes a bug count instead :-)

Comment 2

[Bradley Baetz (:bbaetz)]

Comment on attachment 85573 [details] [diff] [review]
Patch

How does this affect bug 153445? Won't this cause the same problem for
editMilestones as we have for editproducts?

Comment 3

[Jouni Heikniemi]

Probably yes, but I don't think that's an issue. When 153445 gets fixed, it's
easy to fix this as well. Another approach is to totally remove bug counts (the
xyzzy column) from this page. I really don't care that much, but I think Asa's
perf problem in 153445 is practically bmo-specific since no other installation
has that many bugs. If a reasonable number of other installations can benefit
from querying the bug counts, we should do it.

What do you want to do with this? 

Comment 4

[Bradley Baetz (:bbaetz)]

I don't know - the fix for the other bug is likly to involve removing the
feature, ast least until we use a db with row level locking.

Comment 5

[Jouni Heikniemi]

Attachment: v2: kill

Ok, let's just remove the bug counts then.

By the way, MySQL sucks. :-)

Comment 6

[Jouni Heikniemi]

Attachment: v3: Removed bitrot

Here's a version without the bitrot.

Comment 7

[Jouni Heikniemi]

Comment on attachment 111076 [details] [diff] [review]
v3: Removed bitrot

You were the only one interested in this months ago when I made the last patch.
Now pay the price and r=. ;-)

Comment 8

[Bradley Baetz (:bbaetz)]

Comment on attachment 111076 [details] [diff] [review]
v3: Removed bitrot

r=bbaetz. Wehn we support non-sucky dbs we could turn this back on
conditionally

Comment 9

[Jouni Heikniemi]

Checking in editmilestones.cgi;
/cvsroot/mozilla/webtools/bugzilla/editmilestones.cgi,v  <--  editmilestones.cgi
new revision: 1.13; previous revision: 1.12
done