From 8918816c5be0d178d04e181e58a6a1f5f12996b5 Mon Sep 17 00:00:00 2001 From: Dave Lawrence Date: Fri, 18 Oct 2013 11:56:02 -0400 Subject: Bug 922684 - backport upstream bug 919852 to bmo/4.2 to allow quicksearch style bug searching in the webservice API --- Bugzilla/Search/Quicksearch.pm | 10 +++-- Bugzilla/WebService/Bug.pm | 83 +++++++++++++++++++++++++----------------- 2 files changed, 55 insertions(+), 38 deletions(-) diff --git a/Bugzilla/Search/Quicksearch.pm b/Bugzilla/Search/Quicksearch.pm index bc25bb4c0..9c252d04f 100644 --- a/Bugzilla/Search/Quicksearch.pm +++ b/Bugzilla/Search/Quicksearch.pm @@ -308,7 +308,7 @@ sub _bug_numbers_only { # Allow separation by comma or whitespace. $searchstring =~ s/[,\s]+/,/g; - if ($searchstring !~ /,/) { + if ($searchstring !~ /,/ && !i_am_webservice()) { # Single bug number; shortcut to show_bug.cgi. print $cgi->redirect( -uri => correct_urlbase() . "show_bug.cgi?id=$searchstring"); @@ -327,9 +327,11 @@ sub _handle_alias { if ($searchstring =~ /^([^,\s]+)$/) { my $alias = $1; # We use this direct SQL because we want quicksearch to be VERY fast. - my $is_alias = Bugzilla->dbh->selectrow_array( - q{SELECT 1 FROM bugs WHERE alias = ?}, undef, $alias); - if ($is_alias) { + my $bug_id = Bugzilla->dbh->selectrow_array( + q{SELECT bug_id FROM bugs WHERE alias = ?}, undef, $alias); + # If the user cannot see the bug or if we are using a webservice, + # do not resolve its alias. + if ($bug_id && Bugzilla->user->can_see_bug($bug_id) && !i_am_webservice()) { $alias = url_quote($alias); print Bugzilla->cgi->redirect( -uri => correct_urlbase() . "show_bug.cgi?id=$alias"); diff --git a/Bugzilla/WebService/Bug.pm b/Bugzilla/WebService/Bug.pm index b952f346b..8dfbe131e 100644 --- a/Bugzilla/WebService/Bug.pm +++ b/Bugzilla/WebService/Bug.pm @@ -39,9 +39,11 @@ use Bugzilla::Milestone; use Bugzilla::Status; use Bugzilla::Token qw(issue_hash_token); use Bugzilla::Search; +use Bugzilla::Search::Quicksearch; use List::Util qw(max); use List::MoreUtils qw(uniq); +use Storable qw(dclone); ############# # Constants # @@ -461,79 +463,86 @@ sub search { Bugzilla->switch_to_shadow_db(); - if ( defined($params->{offset}) and !defined($params->{limit}) ) { + my $match_params = dclone($params); + delete $match_params->{include_fields}; + delete $match_params->{exclude_fields}; + + # Determine whether this is a quicksearch query + if (exists $match_params->{quicksearch}) { + my $quicksearch = quicksearch($match_params->{'quicksearch'}); + my $cgi = Bugzilla::CGI->new($quicksearch); + $match_params = $cgi->Vars; + } + + if ( defined($match_params->{offset}) and !defined($match_params->{limit}) ) { ThrowCodeError('param_required', { param => 'limit', function => 'Bug.search()' }); } my $max_results = Bugzilla->params->{max_search_results}; - unless (defined $params->{limit} && $params->{limit} == 0) { - if (!defined $params->{limit} || $params->{limit} > $max_results) { - $params->{limit} = $max_results; + unless (defined $match_params->{limit} && $match_params->{limit} == 0) { + if (!defined $match_params->{limit} || $match_params->{limit} > $max_results) { + $match_params->{limit} = $max_results; } } else { - delete $params->{limit}; - delete $params->{offset}; + delete $match_params->{limit}; + delete $match_params->{offset}; } - $params = Bugzilla::Bug::map_fields($params); + $match_params = Bugzilla::Bug::map_fields($match_params); my %options = ( fields => ['bug_id'] ); # Find the highest custom field id - my @field_ids = grep(/^f(\d+)$/, keys %$params); + my @field_ids = grep(/^f(\d+)$/, keys %$match_params); my $last_field_id = @field_ids ? max @field_ids + 1 : 1; # Do special search types for certain fields. - if (my $change_when = delete $params->{'delta_ts'}) { - $params->{"f${last_field_id}"} = 'delta_ts'; - $params->{"o${last_field_id}"} = 'greaterthaneq'; - $params->{"v${last_field_id}"} = $change_when; + if (my $change_when = delete $match_params->{'delta_ts'}) { + $match_params->{"f${last_field_id}"} = 'delta_ts'; + $match_params->{"o${last_field_id}"} = 'greaterthaneq'; + $match_params->{"v${last_field_id}"} = $change_when; $last_field_id++; } - if (my $creation_when = delete $params->{'creation_ts'}) { - $params->{"f${last_field_id}"} = 'creation_ts'; - $params->{"o${last_field_id}"} = 'greaterthaneq'; - $params->{"v${last_field_id}"} = $creation_when; + if (my $creation_when = delete $match_params->{'creation_ts'}) { + $match_params->{"f${last_field_id}"} = 'creation_ts'; + $match_params->{"o${last_field_id}"} = 'greaterthaneq'; + $match_params->{"v${last_field_id}"} = $creation_when; $last_field_id++; } # Some fields require a search type such as short desc, keywords, etc. foreach my $param (qw(short_desc longdesc status_whiteboard bug_file_loc)) { - if (defined $params->{$param} && !defined $params->{$param . '_type'}) { - $params->{$param . '_type'} = 'allwordssubstr'; + if (defined $match_params->{$param} && !defined $match_params->{$param . '_type'}) { + $match_params->{$param . '_type'} = 'allwordssubstr'; } } - if (defined $params->{'keywords'} && !defined $params->{'keywords_type'}) { - $params->{'keywords_type'} = 'allwords'; + if (defined $match_params->{'keywords'} && !defined $match_params->{'keywords_type'}) { + $match_params->{'keywords_type'} = 'allwords'; } # Backwards compatibility with old method regarding role search - $params->{'reporter'} = delete $params->{'creator'} if $params->{'creator'}; + $match_params->{'reporter'} = delete $match_params->{'creator'} if $match_params->{'creator'}; foreach my $role (qw(assigned_to reporter qa_contact longdesc cc)) { - next if !exists $params->{$role}; - my $value = delete $params->{$role}; - $params->{"f${last_field_id}"} = $role; - $params->{"o${last_field_id}"} = "anywordssubstr"; - $params->{"v${last_field_id}"} = ref $value ? join(" ", @{$value}) : $value; + next if !exists $match_params->{$role}; + my $value = delete $match_params->{$role}; + $match_params->{"f${last_field_id}"} = $role; + $match_params->{"o${last_field_id}"} = "anywordssubstr"; + $match_params->{"v${last_field_id}"} = ref $value ? join(" ", @{$value}) : $value; $last_field_id++; } - my %match_params = %{ $params }; - delete $match_params{include_fields}; - delete $match_params{exclude_fields}; - # If no other parameters have been passed other than limit and offset # then we throw error if system is configured to do so. - if (!grep(!/^(limit|offset)$/, keys %match_params) + if (!grep(!/^(limit|offset)$/, keys %$match_params) && !Bugzilla->params->{search_allow_no_criteria}) { ThrowUserError('buglist_parameters_required'); } - $options{order} = [ split(/\s*,\s*/, delete $match_params{order}) ] if $match_params{order}; - $options{params} = \%match_params; + $options{order} = [ split(/\s*,\s*/, delete $match_params->{order}) ] if $match_params->{order}; + $options{params} = $match_params; my $search = new Bugzilla::Search(%options); my ($data) = $search->data; @@ -2753,6 +2762,10 @@ Status Whiteboard field. C If count_only set to true, only a single hash key called C will be returned which is the number of bugs that matched the search. +=item C + +C Search for bugs using quicksearch syntax. + =back =item B @@ -2794,9 +2807,11 @@ by system configuration. =item REST API call added in Bugzilla B<5.0>. -=item Updated to allow for full search capability similar to the Bugzilla UI +=item Updated to allow for full search capability similar to the Bugzilla UI in Bugzilla B<5.0>. +=item Updated to allow quicksearch capability in Bugzilla B<5.0>. + =back =back -- cgit v1.2.3-24-g4f1b