Skip to content

bugzilla
bugzilla
Commit c000c0a4 authored by bbaetz%acm.org's avatar bbaetz%acm.org
Browse files
Options

Bug 204104 - internal error in Search.pm when searching for invalid

keywords/email patch by mike.morgan@oregonstate.edu (michael morgan) r=bbaetz a=justdave
parent 2f45b9d0
Hide whitespace changes
Inline Side-by-side
Showing with 6 additions and 5 deletions
Bugzilla/Search.pm
View file @ c000c0a4
...@@ -34,6 +34,7 @@ use vars qw($userid); ...@@ -34,6 +34,7 @@ use vars qw($userid);
package Bugzilla::Search; package Bugzilla::Search;
use Bugzilla::Config; use Bugzilla::Config;
use Bugzilla::Error;
use Bugzilla::Util; use Bugzilla::Util;
use Date::Format; use Date::Format;
...@@ -103,7 +104,7 @@ sub init { ...@@ -103,7 +104,7 @@ sub init {
my $c = trim($params->param('votes')); my $c = trim($params->param('votes'));
if ($c ne "") { if ($c ne "") {
if ($c !~ /^[0-9]*$/) { if ($c !~ /^[0-9]*$/) {
&::ThrowUserError("illegal_at_least_x_votes", ThrowUserError("illegal_at_least_x_votes",
{ value => $c }); { value => $c });
} }
push(@specialchart, ["votes", "greaterthan", $c - 1]); push(@specialchart, ["votes", "greaterthan", $c - 1]);
...@@ -217,7 +218,7 @@ sub init { ...@@ -217,7 +218,7 @@ sub init {
my $c = trim($params->param('changedin')); my $c = trim($params->param('changedin'));
if ($c ne "") { if ($c ne "") {
if ($c !~ /^[0-9]*$/) { if ($c !~ /^[0-9]*$/) {
&::ThrowUserError("illegal_changed_in_last_x_days", ThrowUserError("illegal_changed_in_last_x_days",
{ value => $c }); { value => $c });
} }
push(@specialchart, ["changedin", push(@specialchart, ["changedin",
...@@ -460,10 +461,10 @@ sub init { ...@@ -460,10 +461,10 @@ sub init {
$t = "greaterthan"; $t = "greaterthan";
} }
if ($field eq "ispatch" && $v ne "0" && $v ne "1") { if ($field eq "ispatch" && $v ne "0" && $v ne "1") {
&::ThrowUserError("illegal_attachment_is_patch"); ThrowUserError("illegal_attachment_is_patch");
} }
if ($field eq "isobsolete" && $v ne "0" && $v ne "1") { if ($field eq "isobsolete" && $v ne "0" && $v ne "1") {
&::ThrowUserError("illegal_is_obsolete"); ThrowUserError("illegal_is_obsolete");
} }
$f = "$table.$field"; $f = "$table.$field";
}, },
...@@ -992,7 +993,7 @@ sub SqlifyDate { ...@@ -992,7 +993,7 @@ sub SqlifyDate {
} }
my $date = str2time($str); my $date = str2time($str);
if (!defined($date)) { if (!defined($date)) {
&::ThrowUserError("illegal_date", { date => $str }); ThrowUserError("illegal_date", { date => $str });
} }
return time2str("%Y-%m-%d %H:%M:%S", $date); return time2str("%Y-%m-%d %H:%M:%S", $date);
} }
......
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