summaryrefslogtreecommitdiffstats
path: root/extensions/BMO
diff options
context:
space:
mode:
authorByron Jones <bjones@mozilla.com>2013-03-06 09:40:29 +0100
committerByron Jones <bjones@mozilla.com>2013-03-06 09:40:29 +0100
commit4ac63bb2181e9e07851bf73520cd068818947b6a (patch)
tree8d6098dbfec3248686519c7f5577eb8c47115e3c /extensions/BMO
parent6b5c91b2f020e9a1ab0e9efd5d4d339fa8f7b22d (diff)
downloadbugzilla-4ac63bb2181e9e07851bf73520cd068818947b6a.tar.gz
bugzilla-4ac63bb2181e9e07851bf73520cd068818947b6a.tar.xz
Bug 848217: The product component search field is not putting bugzilla.mozilla.org components before upstream Bugzilla components
Diffstat (limited to 'extensions/BMO')
-rw-r--r--extensions/BMO/lib/WebService.pm81
1 files changed, 0 insertions, 81 deletions
diff --git a/extensions/BMO/lib/WebService.pm b/extensions/BMO/lib/WebService.pm
index a9387b330..cefcde2f6 100644
--- a/extensions/BMO/lib/WebService.pm
+++ b/extensions/BMO/lib/WebService.pm
@@ -100,87 +100,6 @@ sub getBugsVerifier {
return \%users;
}
-sub prod_comp_search {
- my ($self, $params) = @_;
- my $user = Bugzilla->user;
- my $dbh = Bugzilla->switch_to_shadow_db();
-
- my $search = $params->{'search'};
- $search || ThrowCodeError('param_required',
- { function => 'Bug.prod_comp_search', param => 'search' });
-
- my $limit = detaint_natural($params->{'limit'})
- ? $dbh->sql_limit($params->{'limit'})
- : '';
-
- # We do this in the DB directly as we want it to be fast and
- # not have the overhead of loading full product objects
-
- # All products which the user has "Entry" access to.
- my $enterable_ids = $dbh->selectcol_arrayref(
- 'SELECT products.id FROM products
- LEFT JOIN group_control_map
- ON group_control_map.product_id = products.id
- AND group_control_map.entry != 0
- AND group_id NOT IN (' . $user->groups_as_string . ')
- WHERE group_id IS NULL
- AND products.isactive = 1');
-
- if (scalar @$enterable_ids) {
- # And all of these products must have at least one component
- # and one version.
- $enterable_ids = $dbh->selectcol_arrayref(
- 'SELECT DISTINCT products.id FROM products
- WHERE ' . $dbh->sql_in('products.id', $enterable_ids) .
- ' AND products.id IN (SELECT DISTINCT components.product_id
- FROM components
- WHERE components.isactive = 1)
- AND products.id IN (SELECT DISTINCT versions.product_id
- FROM versions
- WHERE versions.isactive = 1)');
- }
-
- return { products => [] } if !scalar @$enterable_ids;
-
- my @list;
- foreach my $word (split(/[\s,]+/, $search)) {
- if ($word ne "") {
- my $sql_word = $dbh->quote($word);
- trick_taint($sql_word);
- # XXX CONCAT_WS is MySQL specific
- my $field = "CONCAT_WS(' ', products.name, components.name, components.description)";
- push(@list, $dbh->sql_iposition($sql_word, $field) . " > 0");
- }
- }
-
- my $products = $dbh->selectall_arrayref("
- SELECT products.name AS product,
- components.name AS component
- FROM products
- INNER JOIN components ON products.id = components.product_id
- WHERE (" . join(" AND ", @list) . ")
- AND products.id IN (" . join(",", @$enterable_ids) . ")
- ORDER BY products.name $limit",
- { Slice => {} });
-
- # To help mozilla staff file bmo administration bugs into the right
- # component, sort bmo in front of bugzilla.
- if ($user->in_group('mozilla-corporation') || $user->in_group('mozilla-foundation')) {
- $products = [
- sort {
- return 1 if $a->{product} eq 'Bugzilla'
- && $b->{product} eq 'bugzilla.mozilla.org';
- return -1 if $b->{product} eq 'Bugzilla'
- && $a->{product} eq 'bugzilla.mozilla.org';
- return lc($a->{product}) cmp lc($b->{product})
- || lc($a->{component}) cmp lc($b->{component});
- } @$products
- ];
- }
-
- return { products => $products };
-}
-
1;
__END__