Commit dcca89e5 authored by mkanat%bugzilla.org's avatar mkanat%bugzilla.org

Bug 518459: Remove certain QuickSearch features that are no longer needed (after…

Bug 518459: Remove certain QuickSearch features that are no longer needed (after the re-work) and may be confusing Patch by Max Kanat-Alexander <mkanat@bugzilla.org> r=LpSolit, a=LpSolit
parent 9d6f961b
...@@ -110,8 +110,6 @@ use constant FIELD_OPERATOR => { ...@@ -110,8 +110,6 @@ use constant FIELD_OPERATOR => {
}; };
# We might want to put this into localconfig or somewhere # We might want to put this into localconfig or somewhere
use constant PLATFORMS => ('pc', 'sun', 'macintosh', 'mac');
use constant OPSYSTEMS => ('windows', 'win', 'linux');
use constant PRODUCT_EXCEPTIONS => ( use constant PRODUCT_EXCEPTIONS => (
'row', # [Browser] 'row', # [Browser]
# ^^^ # ^^^
...@@ -273,22 +271,6 @@ sub _handle_status_and_resolution { ...@@ -273,22 +271,6 @@ sub _handle_status_and_resolution {
elsif ($words->[0] eq 'OPEN') { elsif ($words->[0] eq 'OPEN') {
shift @$words; shift @$words;
} }
elsif ($words->[0] =~ /^\+[A-Z]+(,[A-Z]+)*$/) {
# e.g. +DUP,FIX
if (matchPrefixes(\%states,
\%resolutions,
[split(/,/, substr($words->[0], 1))],
\@closedStates,
$legal_resolutions)) {
shift @$words;
# Allowing additional resolutions means we need to keep
# the "no resolution" resolution.
$resolutions{'---'} = 1;
}
else {
# Carry on if no match found.
}
}
elsif ($words->[0] =~ /^[A-Z]+(,[A-Z]+)*$/) { elsif ($words->[0] =~ /^[A-Z]+(,[A-Z]+)*$/) {
# e.g. NEW,ASSI,REOP,FIX # e.g. NEW,ASSI,REOP,FIX
undef %states; undef %states;
...@@ -326,10 +308,6 @@ sub _handle_special_first_chars { ...@@ -326,10 +308,6 @@ sub _handle_special_first_chars {
my $baseWord = substr($qsword, 1); my $baseWord = substr($qsword, 1);
my @subWords = split(/[\|,]/, $baseWord); my @subWords = split(/[\|,]/, $baseWord);
if ($firstChar eq '+') {
addChart('short_desc', 'substring', $_, $negate) foreach (@subWords);
return 1;
}
if ($firstChar eq '#') { if ($firstChar eq '#') {
addChart('short_desc', 'substring', $baseWord, $negate); addChart('short_desc', 'substring', $baseWord, $negate);
addChart('content', 'matches', $baseWord, $negate); addChart('content', 'matches', $baseWord, $negate);
...@@ -458,27 +436,14 @@ sub _translate_field_name { ...@@ -458,27 +436,14 @@ sub _translate_field_name {
sub _special_field_syntax { sub _special_field_syntax {
my ($word, $negate) = @_; my ($word, $negate) = @_;
# Platform and operating system
if (grep { lc($word) eq $_ } PLATFORMS
or grep { lc($word) eq $_ } OPSYSTEMS)
{
addChart('rep_platform', 'substring', $word, $negate);
addChart('op_sys', 'substring', $word, $negate);
return 1;
}
# Priority
my $legal_priorities = get_legal_field_values('priority');
if (grep { lc($_) eq lc($word) } @$legal_priorities) {
addChart('priority', 'equals', $word, $negate);
return 1;
}
# P1-5 Syntax # P1-5 Syntax
if ($word =~ m/^P(\d+)(?:-(\d+))?$/i) { if ($word =~ m/^P(\d+)(?:-(\d+))?$/i) {
my $start = $1 - 1; my $start = $1 - 1;
$start = 0 if $start < 0; $start = 0 if $start < 0;
my $end = $2 - 1; my $end = $2 - 1;
my $legal_priorities = get_legal_field_values('priority');
$end = scalar(@$legal_priorities) - 1 $end = scalar(@$legal_priorities) - 1
if $end > (scalar @$legal_priorities - 1); if $end > (scalar @$legal_priorities - 1);
my $prios = $legal_priorities->[$start]; my $prios = $legal_priorities->[$start];
...@@ -489,13 +454,6 @@ sub _special_field_syntax { ...@@ -489,13 +454,6 @@ sub _special_field_syntax {
return 1; return 1;
} }
# Severity
my $legal_severities = get_legal_field_values('bug_severity');
if (grep { lc($word) eq substr($_, 0, 3)} @$legal_severities) {
addChart('bug_severity', 'substring', $word, $negate);
return 1;
}
# Votes (votes>xx) # Votes (votes>xx)
if ($word =~ m/^votes>([0-9]+)$/) { if ($word =~ m/^votes>([0-9]+)$/) {
addChart('votes', 'greaterthan', $1, $negate); addChart('votes', 'greaterthan', $1, $negate);
......
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