Fix calculation of certain aggregates from expressions when no

matching features exist Eg count should return 0 in this case rather than null (cherry-picked from 9ba41e9)
qgis · Aug 29, 2016 · 638de65 · 638de65
1 parent 2588679
commit 638de65
Show file tree

Hide file tree

Showing 3 changed files with 68 additions and 1 deletion.
diff --git a/src/core/qgsaggregatecalculator.cpp b/src/core/qgsaggregatecalculator.cpp
@@ -103,7 +103,7 @@ QVariant QgsAggregateCalculator::calculate( QgsAggregateCalculator::Aggregate ag
       //no matching features
       if ( ok )
         *ok = true;
-      return QVariant();
+      return defaultValue( aggregate );
     }
 
     if ( context )
@@ -458,6 +458,41 @@ QVariant QgsAggregateCalculator::concatenateStrings( QgsFeatureIterator& fit, in
   return result;
 }
 
+QVariant QgsAggregateCalculator::defaultValue( QgsAggregateCalculator::Aggregate aggregate ) const
+{
+  // value to return when NO features are aggregated:
+  switch ( aggregate )
+  {
+      // sensible values:
+    case Count:
+    case CountDistinct:
+    case CountMissing:
+      return 0;
+
+    case StringConcatenate:
+      return ""; // zero length string - not null!
+
+      // undefined - nothing makes sense here
+    case Sum:
+    case Min:
+    case Max:
+    case Mean:
+    case Median:
+    case StDev:
+    case StDevSample:
+    case Range:
+    case Minority:
+    case Majority:
+    case FirstQuartile:
+    case ThirdQuartile:
+    case InterQuartileRange:
+    case StringMinimumLength:
+    case StringMaximumLength:
+      return QVariant();
+  }
+  return QVariant();
+}
+
 QVariant QgsAggregateCalculator::calculateDateTimeAggregate( QgsFeatureIterator& fit, int attr, QgsExpression* expression,
     QgsExpressionContext* context, QgsDateTimeStatisticalSummary::Statistic stat )
 {

diff --git a/src/core/qgsaggregatecalculator.h b/src/core/qgsaggregatecalculator.h
@@ -169,6 +169,8 @@ class CORE_EXPORT QgsAggregateCalculator
                                QgsExpressionContext* context, bool* ok = nullptr );
     static QVariant concatenateStrings( QgsFeatureIterator& fit, int attr, QgsExpression* expression,
                                         QgsExpressionContext* context, const QString& delimiter );
+
+    QVariant defaultValue( Aggregate aggregate ) const;
 };
 
 #endif //QGSAGGREGATECALCULATOR_H

diff --git a/tests/src/python/test_qgsaggregatecalculator.py b/tests/src/python/test_qgsaggregatecalculator.py
@@ -351,6 +351,36 @@ def testExpression(self):
         self.assertTrue(ok)
         self.assertEqual(val, 5)
 
+    def testExpressionNoMatch(self):
+        """ test aggregate calculation using an expression with no features """
+
+        # no features
+        layer = QgsVectorLayer("Point?field=fldint:integer", "layer", "memory")
+
+        # sum
+        agg = QgsAggregateCalculator(layer)
+        val, ok = agg.calculate(QgsAggregateCalculator.Sum, 'fldint * 2')
+        self.assertTrue(ok)
+        self.assertEqual(val, None)
+
+        # count
+        agg = QgsAggregateCalculator(layer)
+        val, ok = agg.calculate(QgsAggregateCalculator.Count, 'fldint * 2')
+        self.assertTrue(ok)
+        self.assertEqual(val, 0)
+
+        # count distinct
+        agg = QgsAggregateCalculator(layer)
+        val, ok = agg.calculate(QgsAggregateCalculator.CountDistinct, 'fldint * 2')
+        self.assertTrue(ok)
+        self.assertEqual(val, 0)
+
+        # count missing
+        agg = QgsAggregateCalculator(layer)
+        val, ok = agg.calculate(QgsAggregateCalculator.CountMissing, 'fldint * 2')
+        self.assertTrue(ok)
+        self.assertEqual(val, 0)
+
     def testStringToAggregate(self):
         """ test converting strings to aggregate types """