From 267faf14f19cd3765541a3e115f23f45f73ef8ad Mon Sep 17 00:00:00 2001 From: Byron Jones Date: Fri, 7 Jun 2013 13:28:24 +0800 Subject: [PATCH] Bug 880315: Fix malformed sql generated by the fix for bug 879055 r=LpSolit, a=LpSolit --- Bugzilla/Search.pm | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Bugzilla/Search.pm b/Bugzilla/Search.pm index 6c5e04088b..fba6ea07d0 100644 --- a/Bugzilla/Search.pm +++ b/Bugzilla/Search.pm @@ -2903,14 +2903,14 @@ sub _anywordsubstr { my ($self, $args) = @_; my @terms = $self->_substring_terms($args); - $args->{term} = '(' . join("\n\tOR ", @terms) . ')'; + $args->{term} = @terms ? '(' . join("\n\tOR ", @terms) . ')' : ''; } sub _allwordssubstr { my ($self, $args) = @_; my @terms = $self->_substring_terms($args); - $args->{term} = '(' . join("\n\tAND ", @terms) . ')'; + $args->{term} = @terms ? '(' . join("\n\tAND ", @terms) . ')' : ''; } sub _nowordssubstr { @@ -2927,14 +2927,14 @@ sub _anywords { # Because _word_terms uses AND, we need to parenthesize its terms # if there are more than one. @terms = map("($_)", @terms) if scalar(@terms) > 1; - $args->{term} = '(' . join("\n\tOR ", @terms) . ')'; + $args->{term} = @terms ? '(' . join("\n\tOR ", @terms) . ')' : ''; } sub _allwords { my ($self, $args) = @_; my @terms = $self->_word_terms($args); - $args->{term} = '(' . join("\n\tAND ", @terms) . ')'; + $args->{term} = @terms ? '(' . join("\n\tAND ", @terms) . ')' : ''; } sub _nowords { -- 2.47.2