617133 - Reporting for components causes errors

Status: RESOLVED DUPLICATE of bug 567953

(Bugzilla :: Reporting/Charting, defect)

Description

[miketosh]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.12) Gecko/20101026 (CK-xxxxxxxxxx) Firefox/3.6.12 (.NET CLR 3.5.30729)
Build Identifier: 

If you have components in products that have the same name, the reporting tool will show multiple rows for that component, as many as exist, whether the numbers belong to that component or not.

Reproducible: Always

Steps to Reproduce:
1. Create 3 products all with the same component
2. Create bugs against one of them
3. go to the report page and use components as your Y axis.

report.cgi?y_axis_field=component&cumulate=0&z_axis_field=&format=bar&x_axis_field=&query_format=report-graph&resolution=---&action=wrap
Actual Results:  
You get multiple rows/columns for each of the unique components with the same number in them.  If you have 5 bugs against product1, component 'ALL', and 6 products with a component 'ALL', the report will show you 6 rows, each with the same 5 bugs listed.

Expected Results:  
only one row.

I've fixed this with the following change to report.cgi:

At the top, call 
  use List::MoreUtils qw(uniq);

At the bottom, change old line:
     $fields{$field->name} = \@names ;
to be:
     $fields{$field->name} = [ uniq @names ];

Comment 1

[miketosh]

I can provide the two line patch, but I can't check it in.

Comment 2

[Frédéric Buclin]

Which version is that?

Comment 3

[miketosh]

Bugzilla 4.0rc1, and trunk.  I think a change to Bugzilla::Field->legal_values may have caused this.

Comment 4

[Frédéric Buclin]

Is Bugzilla 3.6.3 also affected?

Comment 5

[Frédéric Buclin]

This is actually a dupe.

Comment 6

[miketosh]

(In reply to comment #4)
> Is Bugzilla 3.6.3 also affected?

No.  3.6* isn't affected, just 4.0rc1+.  If you'd like me to add a patch to 567953 for approval, just let me know.