diff options
author | Byron Jones <bjones@mozilla.com> | 2013-10-16 16:26:31 +0200 |
---|---|---|
committer | Byron Jones <bjones@mozilla.com> | 2013-10-16 16:26:31 +0200 |
commit | 4b5b03e55429a6e1c531650aa5be00f624b2bfa2 (patch) | |
tree | 11fce978297727eb1f1054c812ec20f87de2f628 /buglist.cgi | |
parent | bebd1cb6430faa08b9937f1bf2c65af8c8f823ef (diff) | |
download | bugzilla-4b5b03e55429a6e1c531650aa5be00f624b2bfa2.tar.gz bugzilla-4b5b03e55429a6e1c531650aa5be00f624b2bfa2.tar.xz |
Bug 926109: Error when searching for many columns at once (MariaDB can only use 61 tables in a join)
Diffstat (limited to 'buglist.cgi')
-rwxr-xr-x | buglist.cgi | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/buglist.cgi b/buglist.cgi index 6ffbea94f..9c7281822 100755 --- a/buglist.cgi +++ b/buglist.cgi @@ -546,16 +546,7 @@ if (defined $params->param('columnlist')) { if ($params->param('columnlist') eq "all") { # If the value of the CGI parameter is "all", display all columns, # but remove the redundant "short_desc" column. - # BMO: Skip tracking flag columns when retrieving all columns - # MySQL bombs on greater than 61 joins. - my @non_tf_columns; - foreach my $column (keys %$columns) { - next if $column eq 'short_desc'; - next if ($column =~ /^cf_(blocking|tracking|status)/ - && $columns->{$column}->{name} =~ /^COALESCE/); - push(@non_tf_columns, $column); - } - @displaycolumns = @non_tf_columns; + @displaycolumns = grep($_ ne 'short_desc', keys(%$columns)); } else { @displaycolumns = split(/[ ,]+/, $params->param('columnlist')); |