Commit 8bdb86fd authored by Max Kanat-Alexander's avatar Max Kanat-Alexander

Bug 577602: Don't put multi-select fields into the GROUP BY in Search.pm,

because they are created by an aggregate. (This fixes multi-select buglist columns on Pg.) r=mkanat, a=mkanat (module owner)
parent 68f48be8
......@@ -1119,8 +1119,12 @@ sub init {
foreach my $field (@fields) {
# These fields never go into the GROUP BY (bug_id goes in
# explicitly, below).
next if (grep($_ eq $field, EMPTY_COLUMN,
qw(bug_id actual_time percentage_complete flagtypes.name keywords)));
my @skip_group_by = (EMPTY_COLUMN,
qw(bug_id actual_time percentage_complete flagtypes.name
keywords));
push(@skip_group_by, map { $_->name } @$multi_select_fields);
next if grep { $_ eq $field } @skip_group_by;
my $col = COLUMNS->{$field}->{name};
push(@groupby, $col) if !grep($_ eq $col, @groupby);
}
......
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