Commit 294cff65 authored by Pami Ketolainen's avatar Pami Ketolainen Committed by Frédéric Buclin

Bug 860723: Custom fields are shown twice in report axis selectors

r/a=LpSolit
parent 28452a3f
...@@ -261,13 +261,6 @@ if (($cgi->param('query_format') || $cgi->param('format') || "") ...@@ -261,13 +261,6 @@ if (($cgi->param('query_format') || $cgi->param('format') || "")
$vars->{'category'} = Bugzilla::Chart::getVisibleSeries(); $vars->{'category'} = Bugzilla::Chart::getVisibleSeries();
} }
if ($cgi->param('format') && $cgi->param('format') =~ /^report-(table|graph)$/) {
# Get legal custom fields for tabular and graphical reports.
my @custom_fields_for_reports =
grep { $_->type == FIELD_TYPE_SINGLE_SELECT } Bugzilla->active_custom_fields;
$vars->{'custom_fields'} = \@custom_fields_for_reports;
}
$vars->{'known_name'} = $cgi->param('known_name'); $vars->{'known_name'} = $cgi->param('known_name');
$vars->{'columnlist'} = $cgi->param('columnlist'); $vars->{'columnlist'} = $cgi->param('columnlist');
......
...@@ -25,12 +25,5 @@ ...@@ -25,12 +25,5 @@
[% " selected" IF default.$name.0 == field %]> [% " selected" IF default.$name.0 == field %]>
[% field_descs.$field || field FILTER html %]</option> [% field_descs.$field || field FILTER html %]</option>
[% END %] [% END %]
[%# Single-select fields are also valid column names. %]
[% FOREACH field = custom_fields %]
<option value="[% field.name FILTER html %]"
[% " selected" IF default.$name.0 == field.name %]>
[% field.description FILTER html %]</option>
[% END %]
</select> </select>
[% END %] [% END %]
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment