From 993f0b3529d4c708fcbaa3fc8efcdeee9e1775d5 Mon Sep 17 00:00:00 2001 From: "mkanat%kerio.com" <> Date: Wed, 20 Apr 2005 06:44:03 +0000 Subject: Bug 284599: Use of REGEXP search is not consistent wrt case sensitivity Patch By Tomas Kopal r=mkanat, a=justdave --- Bugzilla/DB.pm | 6 ++++-- Bugzilla/DB/Pg.pm | 4 ++-- Bugzilla/Search.pm | 6 +++--- 3 files changed, 9 insertions(+), 7 deletions(-) (limited to 'Bugzilla') diff --git a/Bugzilla/DB.pm b/Bugzilla/DB.pm index 7e45e56e4..f1747b39d 100644 --- a/Bugzilla/DB.pm +++ b/Bugzilla/DB.pm @@ -1199,7 +1199,8 @@ formatted SQL command have prefix C. All other methods have prefix C. =item C Description: Outputs SQL regular expression operator for POSIX regex - searches in format suitable for a given database. + searches (case insensitive) in format suitable for a given + database. Abstract method, should be overriden by database specific code. Params: none Returns: formatted SQL for regular expression search (e.g. REGEXP) @@ -1208,7 +1209,8 @@ formatted SQL command have prefix C. All other methods have prefix C. =item C Description: Outputs SQL regular expression operator for negative POSIX - regex searches in format suitable for a given database. + regex searches (case insensitive) in format suitable for a given + database. Abstract method, should be overriden by database specific code. Params: none Returns: formatted SQL for negative regular expression search diff --git a/Bugzilla/DB/Pg.pm b/Bugzilla/DB/Pg.pm index e635096f2..4fe8d2244 100644 --- a/Bugzilla/DB/Pg.pm +++ b/Bugzilla/DB/Pg.pm @@ -87,11 +87,11 @@ sub bz_last_key { } sub sql_regexp { - return "~"; + return "~*"; } sub sql_not_regexp { - return "!~" + return "!~*" } sub sql_limit { diff --git a/Bugzilla/Search.pm b/Bugzilla/Search.pm index bebbfd94a..b12d714f9 100644 --- a/Bugzilla/Search.pm +++ b/Bugzilla/Search.pm @@ -1000,10 +1000,10 @@ sub init { $term = $dbh->sql_position(lc($q), "LOWER($ff)") . " = 0"; }, ",regexp" => sub { - $term = "LOWER($ff) " . $dbh->sql_regexp() . " $q"; + $term = "$ff " . $dbh->sql_regexp() . " $q"; }, ",notregexp" => sub { - $term = "LOWER($ff) " . $dbh->sql_not_regexp() . " $q"; + $term = "$ff " . $dbh->sql_not_regexp() . " $q"; }, ",lessthan" => sub { $term = "$ff < $q"; @@ -1506,7 +1506,7 @@ sub GetByWordList { $word =~ s/^'//; $word =~ s/'$//; $word = '(^|[^a-z0-9])' . $word . '($|[^a-z0-9])'; - push(@list, "lower($field) " . $dbh->sql_regexp() . " '$word'"); + push(@list, "$field " . $dbh->sql_regexp() . " '$word'"); } } -- cgit v1.2.3-24-g4f1b