Actually use the "isNewField" flag

qgis · Sep 11, 2020 · 33a6472 · 33a6472
1 parent d32369f
commit 33a6472
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/analysis/processing/qgsalgorithmfieldcalculator.cpp b/src/analysis/processing/qgsalgorithmfieldcalculator.cpp
@@ -124,7 +124,7 @@ bool QgsFieldCalculatorAlgorithm::prepareAlgorithm( const QVariantMap &parameter
 
   QVariant::Type fieldType = fieldTypes[fieldTypeIdx];
 
-  // this is to keep backwards compatibility.
+  // this is to keep backwards compatibility, "NEW_FIELD" flags what how "FIELD_NAME" should be treated
   // In a perfect universe there would be only "EXISTING_FIELD_NAME" and "NEW_FIELD_NAME"
   if ( !parameters.contains( QStringLiteral( "NEW_FIELD" ) ) )
   {
@@ -147,7 +147,7 @@ bool QgsFieldCalculatorAlgorithm::prepareAlgorithm( const QVariantMap &parameter
 
   int fieldIdx = mFields.lookupField( field.name() );
 
-  if ( fieldIdx < 0 )
+  if ( isNewField || fieldIdx < 0 )
     mFields.append( field );
 
   QString dest;