130151 - collectstats.pl header incorrect.

Status: RESOLVED DUPLICATE of bug 16009

(Bugzilla :: Reporting/Charting, defect)

Description

[Chema Celorio]

The header that collectstats is generating does not corresponds to the columns,
not only the header has one column less but the order of the fields do not match
because the order in which they are listed in collectstats is different than the
order in which they are listed in the default configuration.

Collect stats should not hardcode severity and resolutions. You'll find a patch
attached that solves this issue in a generic way.

regards,
Chema

Comment 1

[Chema Celorio]

Attachment: patch against collect stats

Comment 2

[Gervase Markham [:gerv]]

> The header that collectstats is generating does not corresponds to the 
> columns, not only the header has one column less but the order of the fields 
> do not match because the order in which they are listed in collectstats is 
> different than the order in which they are listed in the default 
> configuration.

I'm very sceptical that this is the case, because it would mean everyone's
charts would be wrong. To convince me, you will need to be more specific. :-)

What do you mean by "the order they are listed in the default configuration"?
Please give both orders, and point out how they are different, and what problems
it causes.

> Collect stats should not hardcode severity and resolutions. You'll find a patch
> attached that solves this issue in a generic way.

Even if there is a bug, this patch cannot be checked in, because there would be
serious migration issues. There is no guarantee that the lists you use would
produce the same fields in the same order, and so people's data fields may get
messed up.

Gerv

Comment 3

[Chema Celorio]

Hmm, yes.

I think you are right, i backported a patch to the Ximian's bugzilla
installation and i overlooked a detail. I'll provide a new patch that doesn't
break current installations.

my mistake.
Chema

Comment 4

[Gervase Markham [:gerv]]

Please don't provide another patch until you explain what the problem is more
clearly :-)

The fact that collectstats.pl is not generic is not, in itself, a problem - this
will go away when we get generic charting, and the data moves into the database.
This is bug 16009.

Gerv

Comment 5

[Chema Celorio]

I'm closing the bug as a dulpicate of 16009

*** This bug has been marked as a duplicate of 16009 ***