Closed issue2550805 (Postgresql should search title case insensitive), by Tom Ekberg.

Author: bernhardreiter

CHANGES.txt

@@ -51,6 +51,8 @@ Features:
 
 Fixed:
 
+- issue2550805: Postgres should search title attribute case insensitive 
+  like sqlite. Reported and fixed by Tom Ekberg. (Bernhard Reiter)
 - Removed some old left over "rlog" references in documentation and code.
   Makes the debugging.txt advise for the database unit tests work again.
 - Fixed OpenPGP support for modern versions of libgpgme. (Bernhard Reiter)

roundup/backends/back_postgresql.py

@@ -285,6 +285,7 @@ def clear(self):
 
 class PostgresqlClass:
     order_by_null_values = '(%s is not NULL)'
+    case_insensitive_like = 'ILIKE'
 
 class Class(PostgresqlClass, rdbms_common.Class):
     pass

roundup/backends/rdbms_common.py

@@ -2440,7 +2440,14 @@ def _filter_sql (self, search_matches, filterspec, srt=[], grp=[], retr=0):
 
                     # now add to the where clause
                     where.append('('
-                        +' and '.join(["_%s._%s LIKE '%s'"%(pln, k, s) for s in v])
+                        +' and '.join(["_%s._%s %s '%s'"%(
+                                    pln,
+                                    k,
+                                    # For many databases the LIKE operator
+                                    # ignores case.  Postgres and Oracle have
+                                    # an ILIKE operator to support this.
+                                    getattr(self,'case_insensitive_like','LIKE'),
+                                    s) for s in v])
                         +')')
                     # note: args are embedded in the query string now
                 if 'sort' in p.need_for:

test/db_test_base.py

@@ -1235,6 +1235,23 @@ def testFilteringString(self):
             ae(filt(None, {'title': ['one', 'two']}, ('+','id'), (None,None)),
                 [])
 
+    def testFilteringStringCase(self):
+        """
+        Similar to testFilteringString except the search parameters
+        have different capitalization.
+        """
+        ae, filter, filter_iter = self.filteringSetup()
+        for filt in filter, filter_iter:
+            ae(filt(None, {'title': ['One']}, ('+','id'), (None,None)), ['1'])
+            ae(filt(None, {'title': ['Issue One']}, ('+','id'), (None,None)),
+                ['1'])
+            ae(filt(None, {'title': ['ISSUE', 'ONE']}, ('+','id'), (None,None)),
+                ['1'])
+            ae(filt(None, {'title': ['iSSUE']}, ('+','id'), (None,None)),
+                ['1','2','3'])
+            ae(filt(None, {'title': ['One', 'Two']}, ('+','id'), (None,None)),
+                [])
+
     def testFilteringLink(self):
         ae, filter, filter_iter = self.filteringSetup()
         a = 'assignedto'