Used named parameters for with_variable function

qgis · Aug 23, 2018 · 89cd785 · 89cd785
1 parent 34bb08c
commit 89cd785
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/resources/function_help/json/with_variable b/resources/function_help/json/with_variable
@@ -5,7 +5,7 @@
   "arguments": [
                  {"arg":"name","description":"the name of the variable to set"},
                  {"arg":"value","description":"the value to set"},
-                 {"arg":"node","description":"the expression for which the variable will be available"}
+                 {"arg":"expression","description":"the expression for which the variable will be available"}
                ],
   "examples": [ { "expression":"with_variable('my_sum', 1 + 2 + 3, @my_sum * 2 + @my_sum * 5)", "returns":"42"}]
 }
diff --git a/src/core/expression/qgsexpressionfunction.cpp b/src/core/expression/qgsexpressionfunction.cpp
@@ -4863,7 +4863,11 @@ bool QgsArrayForeachExpressionFunction::prepare( const QgsExpressionNodeFunction
 }
 
 QgsWithVariableExpressionFunction::QgsWithVariableExpressionFunction()
-  : QgsExpressionFunction( QStringLiteral( "with_variable" ), 3, QCoreApplication::tr( "General" ) )
+  : QgsExpressionFunction( QStringLiteral( "with_variable" ), QgsExpressionFunction::ParameterList() <<
+                           QgsExpressionFunction::Parameter( QStringLiteral( "name" ) )
+                           << QgsExpressionFunction::Parameter( QStringLiteral( "value" ) )
+                           << QgsExpressionFunction::Parameter( QStringLiteral( "expression" ) ),
+                           QCoreApplication::tr( "General" ) )
 {
 
 }

diff --git a/tests/src/core/testqgsexpression.cpp b/tests/src/core/testqgsexpression.cpp
@@ -1287,7 +1287,7 @@ class TestQgsExpression: public QObject
       QTest::newRow( "'a' not  like 'a%'" ) << QStringLiteral( "'a' not  like 'a%'" ) << false << QVariant( 0 );
 
       // with_variable
-      QTest::newRow( "with_variable('five', 5, @five * 2)" ) << QStringLiteral( "with_variable('five', 5, @five * 2)" ) << false << QVariant( 10 );
+      QTest::newRow( "with_variable(name:='five', value:=5, expression:=@five * 2)" ) << QStringLiteral( "with_variable(name:='five', value:=5, expression:=@five * 2)" ) << false << QVariant( 10 );
       QTest::newRow( "with_variable('nothing', NULL, COALESCE(@nothing, 'something'))" ) << QStringLiteral( "with_variable('nothing', NULL, COALESCE(@nothing, 'something'))" ) << false << QVariant( "something" );
 
       // array_first, array_last