Add support for modulo operator (%) in rule based rendering. See ticket #3845.

Sandro Santilli · jef-n · commit c2757d31bc3a · 2011-05-26T21:21:02.000+02:00
diff --git a/src/core/qgssearchstringlexer.ll b/src/core/qgssearchstringlexer.ll
@@ -109,7 +109,7 @@ string      "'"{str_char}*"'"
 
 "||"   { return CONCAT; }
 
-[+-/*^]    { return yytext[0]; }
+[+-/*^%]    { return yytext[0]; }
 
 [()]      { return yytext[0]; }
 
diff --git a/src/core/qgssearchstringparser.yy b/src/core/qgssearchstringparser.yy
@@ -171,6 +171,7 @@ scalar_exp:
       }
     | scalar_exp '^' scalar_exp   { $$ = new QgsSearchTreeNode(QgsSearchTreeNode::opPOW,  $1, $3); joinTmpNodes($$,$1,$3); }
     | scalar_exp '*' scalar_exp   { $$ = new QgsSearchTreeNode(QgsSearchTreeNode::opMUL,  $1, $3); joinTmpNodes($$,$1,$3); }
+    | scalar_exp '%' scalar_exp   { $$ = new QgsSearchTreeNode(QgsSearchTreeNode::opMOD,  $1, $3); joinTmpNodes($$,$1,$3); }
     | scalar_exp '/' scalar_exp   { $$ = new QgsSearchTreeNode(QgsSearchTreeNode::opDIV,  $1, $3); joinTmpNodes($$,$1,$3); }
     | scalar_exp '+' scalar_exp   { $$ = new QgsSearchTreeNode(QgsSearchTreeNode::opPLUS, $1, $3); joinTmpNodes($$,$1,$3); }
     | scalar_exp '-' scalar_exp   { $$ = new QgsSearchTreeNode(QgsSearchTreeNode::opMINUS,$1, $3); joinTmpNodes($$,$1,$3); }
diff --git a/src/core/qgssearchtreenode.cpp b/src/core/qgssearchtreenode.cpp
@@ -268,6 +268,7 @@ QString QgsSearchTreeNode::makeSearchString()
         case opPLUS:  str += "+"; break;
         case opMINUS: str += "-"; break;
         case opMUL:   str += "*"; break;
+        case opMOD:   str += "%"; break;
         case opDIV:   str += "/"; break;
         case opPOW:   str += "^"; break;
 
@@ -781,6 +782,10 @@ QgsSearchTreeValue QgsSearchTreeNode::valueAgainst( const QgsFieldMap& fields, Q
           return QgsSearchTreeValue( val1 - val2 );
         case opMUL:
           return QgsSearchTreeValue( val1 * val2 );
+        case opMOD:
+          // NOTE: we _might_ support float operators, like postgresql does
+          // see 83c94a886c059 commit in postgresql git repo for more info
+          return QgsSearchTreeValue( int(val1) % int(val2) );
         case opDIV:
           if ( val2 == 0 )
             return QgsSearchTreeValue( 2, "" ); // division by zero
diff --git a/src/core/qgssearchtreenode.h b/src/core/qgssearchtreenode.h
@@ -66,6 +66,7 @@ class CORE_EXPORT QgsSearchTreeNode
       opPLUS,
       opMINUS,
       opMUL,
+      opMOD,
       opDIV,
       opPOW,
       opSQRT,

Original file line number	Diff line number	Diff line change
`@@ -171,6 +171,7 @@ scalar_exp:`
`171`	`171`	`}`
`172`	`172`	`\| scalar_exp '^' scalar_exp { $$ = new QgsSearchTreeNode(QgsSearchTreeNode::opPOW, $1, $3); joinTmpNodes($$,$1,$3); }`
`173`	`173`	`\| scalar_exp '*' scalar_exp { $$ = new QgsSearchTreeNode(QgsSearchTreeNode::opMUL, $1, $3); joinTmpNodes($$,$1,$3); }`
	`174`	`+ \| scalar_exp '%' scalar_exp { $$ = new QgsSearchTreeNode(QgsSearchTreeNode::opMOD, $1, $3); joinTmpNodes($$,$1,$3); }`
`174`	`175`	`\| scalar_exp '/' scalar_exp { $$ = new QgsSearchTreeNode(QgsSearchTreeNode::opDIV, $1, $3); joinTmpNodes($$,$1,$3); }`
`175`	`176`	`\| scalar_exp '+' scalar_exp { $$ = new QgsSearchTreeNode(QgsSearchTreeNode::opPLUS, $1, $3); joinTmpNodes($$,$1,$3); }`
`176`	`177`	`\| scalar_exp '-' scalar_exp { $$ = new QgsSearchTreeNode(QgsSearchTreeNode::opMINUS,$1, $3); joinTmpNodes($$,$1,$3); }`
-Original file line number
+Diff line change
       opPLUS,
       opMINUS,
       opMUL,
 +      opMOD,
       opDIV,
       opPOW,
       opSQRT,