fixed for OGo bug #1906

[sope] / sope-gdl1 / GDLContentStore / EOQualifier+GCS.m

diff --git a/sope-gdl1/GDLContentStore/EOQualifier+GCS.m b/sope-gdl1/GDLContentStore/EOQualifier+GCS.m
index ec0103cc4a84f8c9d6f5cff601896aa15064c252..88343350e50536baafc9bd578b1fdaf9eb0c34be 100644 (file)
--- a/sope-gdl1/GDLContentStore/EOQualifier+GCS.m
+++ b/sope-gdl1/GDLContentStore/EOQualifier+GCS.m
@@ -1,5 +1,5 @@
 /*
-  Copyright (C) 2004-2005 SKYRIX Software AG
+  Copyright (C) 2004-2007 SKYRIX Software AG
 
   This file is part of OpenGroupware.org.
 
@@ -64,30 +64,34 @@
   toString:(NSMutableString *)_ms
 {
   id val;
-  
-  [_ms appendString:[_q key]];
-  
+  NSString *qKey, *qOperator, *qValue, *qFormat;
+  BOOL isCI;
+
+  qKey = [_q key];
+
   if ((val = [_q value])) {
     SEL op = [_q selector];
-    
+
     if ([val isNotNull]) {
+      isCI = NO;
+
       if (sel_eq(op, EOQualifierOperatorEqual))
-       [_ms appendString:@" = "];
+       qOperator = @"=";
       else if (sel_eq(op, EOQualifierOperatorNotEqual))
-       [_ms appendString:@" != "];
+       qOperator = @"!=";
       else if (sel_eq(op, EOQualifierOperatorLessThan))
-       [_ms appendString:@" < "];
+       qOperator = @"<";
       else if (sel_eq(op, EOQualifierOperatorGreaterThan))
-       [_ms appendString:@" > "];
+       qOperator = @">";
       else if (sel_eq(op, EOQualifierOperatorLessThanOrEqualTo))
-       [_ms appendString:@" <= "];
+       qOperator = @"<=";
       else if (sel_eq(op, EOQualifierOperatorGreaterThanOrEqualTo))
-       [_ms appendString:@" >= "];
+       qOperator = @">=";
       else if (sel_eq(op, EOQualifierOperatorLike))
-       [_ms appendString:@" LIKE "];
+       qOperator = @"LIKE";
       else if (sel_eq(op, EOQualifierOperatorCaseInsensitiveLike)) {
-       // TODO: this is PostgreSQL specific, use UPPER?
-       [_ms appendString:@" ILIKE "];
+       isCI = YES;
+       qOperator = @"LIKE";
       }
       else {
        [self errorWithFormat:@"%s: unsupported operation for null: %@",
@@ -95,11 +99,9 @@
       }
 
       if ([val isKindOfClass:[NSNumber class]])
-       [_ms appendString:[val stringValue]];
+       qValue = [val stringValue];
       else if ([val isKindOfClass:[NSString class]]) {
-       [_ms appendString:@"'"];
-       [_ms appendString:val];
-       [_ms appendString:@"'"];
+       qValue = [NSString stringWithFormat: @"'%@'", val];
       }
       else {
        [self errorWithFormat:@"%s: unsupported value class: %@",
@@ -107,18 +109,31 @@
       }
     }
     else {
-      if (sel_eq(op, EOQualifierOperatorEqual))
-       [_ms appendString:@" IS NULL"];
-      else if (sel_eq(op, EOQualifierOperatorEqual))
-       [_ms appendString:@" IS NOT NULL"];
+      if (sel_eq(op, EOQualifierOperatorEqual)) {
+       qOperator = @"IS";
+       qValue = @"NULL";
+      }
+      else if (sel_eq(op, EOQualifierOperatorEqual)) {
+       qOperator = @"IS NOT";
+       qValue = @"NULL";
+      }
       else {
        [self errorWithFormat:@"%s: invalid operation for null: %@",
              __PRETTY_FUNCTION__, NSStringFromSelector(op)];
       }
     }
   }
+  else {
+    qOperator = @"IS";
+    qValue = @"NULL";
+  }
+
+  if (isCI)
+    qFormat = @"UPPER(%@) %@ UPPER(%@)";
   else
-    [_ms appendString:@" IS NULL"];
+    qFormat = @"%@ %@ %@";
+
+  [_ms appendFormat: qFormat, qKey, qOperator, qValue];
 }
 
 - (void)_appendQualifier:(EOQualifier *)_q toString:(NSMutableString *)_ms {