expressions: keep brackets (fix #11475)

jef-n · jef-n · commit 2c7378bb16f7 · 2014-10-23T21:03:08.000+02:00
diff --git a/src/core/qgsexpression.cpp b/src/core/qgsexpression.cpp
@@ -2414,7 +2414,7 @@ QString QgsExpression::NodeBinaryOperator::dump() const
   QString fmt;
   fmt += lOp && lOp->precedence() < precedence() ? "(%1)" : "%1";
   fmt += " %2 ";
-  fmt += rOp && rOp->precedence() < precedence() ? "(%3)" : "%3";
+  fmt += rOp && rOp->precedence() <= precedence() ? "(%3)" : "%3";
 
   return fmt.arg( mOpLeft->dump() ).arg( BinaryOperatorText[mOp] ).arg( mOpRight->dump() );
 }

Original file line number	Diff line number	Diff line change
`@@ -2414,7 +2414,7 @@ QString QgsExpression::NodeBinaryOperator::dump() const`
`2414`	`2414`	`QString fmt;`
`2415`	`2415`	`fmt += lOp && lOp->precedence() < precedence() ? "(%1)" : "%1";`
`2416`	`2416`	`fmt += " %2 ";`
`2417`		`- fmt += rOp && rOp->precedence() < precedence() ? "(%3)" : "%3";`
	`2417`	`+ fmt += rOp && rOp->precedence() <= precedence() ? "(%3)" : "%3";`
`2418`	`2418`
`2419`	`2419`	`return fmt.arg( mOpLeft->dump() ).arg( BinaryOperatorText[mOp] ).arg( mOpRight->dump() );`
`2420`	`2420`	`}`