WFS: case insesitive filters do not work

[assefa]

This is what is currently generated for case inseitive searches:

FILTER ("[NAME]" =~ /^syd.*/i)

But this seems to be invalid and gives errors such as:

msEvalExpression: Expression parser error. Failed to parse expression: "Tignish" =~ /^syd.*/i

Is there a way to use this operand (~=) in a case sensitive way?

Note that using filteritem and then just a regex /^{syd.*/i works.}

[sdlime]

What about trying:

  FILTER ("[NAME]" =~ "^syd.*"i)

Steve

[assefa]

Thanks Steve, I tried it but in both cases, I get a parsing error.

Failed to parse expression "Yarmouth" =~ "^{Syd.*" Failed to parse expression: "Yarmouth" =~ "}Syd.*"i

I am disabling (r10460) the matchCase for now until we figure out the correct syntax (if possible).

[sdlime]

Well, in 6.0 I'm aiming to add case insensitive operators for string and regex equality (=* and =~*) and do away with the friggin' trailing i's. Let me do some more testing and report back.

Steve

[bartvde]

Steve, Assefa,

a colleague of mine had a look and came up with the following patch to mapparser.y:

--- mapparser.y.orig    2010-08-23 13:25:22.000000000 +0200
+++ mapparser.y 2010-08-23 13:21:07.000000000 +0200
@@ -48,6 +48,7 @@
 %type <dblval> math_exp
 %type <strval> string_exp
 %type <strval> regular_exp
+%type <strval> iregular_exp
 %type <tmval> time_exp

 /* Bison/Yacc grammar */
@@ -67,6 +68,7 @@
 ;

 regular_exp: REGEX ;
+iregular_exp: IREGEX ;

 logical_exp:
        logical_exp OR logical_exp      {
@@ -139,6 +141,19 @@
                                       }
        | NOT logical_exp              { $$ = !$2; }
        | NOT math_exp                 { $$ = !$2; }
+       | string_exp RE iregular_exp     {
+                                         ms_regex_t re;
+
+                                         if(ms_regcomp(&re, $3, MS_REG_EXTENDED|MS_REG_NOSUB|MS_REG_ICASE) != 0)
+                                           $$ = MS_FALSE;
+
+                                         if(ms_regexec(&re, $1, 0, NULL, 0) == 0)
+                                          $$ = MS_TRUE;
+                                        else
+                                          $$ = MS_FALSE;
+
+                                         ms_regfree(&re);
+                                       }
        | string_exp RE regular_exp     {
                                          ms_regex_t re;

Are you able to look into this again? We really need to have this working. TIA.

[sdlime]

Bart: Which version are you wanting to work against?

Steve

[bartvde]

Hi Steve, trunk would be fine for us.

[sdlime]

Then it would make more sense for me to implement the case insensitive operators I mentioned earlier (=* and =~*) then... I can do that.

Steve

[sdlime]

Ok, I added a case insensitive regex operator to the parser/lexer. The syntax is:

('[county]' ~* /^A/)

This follows, fairly closely, the PostgreSQL syntax. This will likely change further with RFC 59 (no /'s necessary) if adopted but should good enough for testing now.

Assefa, can you try modifying the filter code?

Steve

[assefa]

out of the office last week ...

reactivate the islike operator r10488. Please give it a try.

[assefa]

This is fixed. closing.