[processing] correctly handle null gems in meancoords alg

Fixes #17026
qgis · Aug 22, 2017 · 667ba4255e87e8e05051e7fa8b4851633b3da4b7 · 667ba42
1 parent 7ca8de9
commit 667ba4255e87e8e05051e7fa8b4851633b3da4b7
Unified Split

Showing with 8 additions and 8 deletions.

+8 −8 python/plugins/processing/algs/qgis/MeanCoords.py
diff --git a/python/plugins/processing/algs/qgis/MeanCoords.py b/python/plugins/processing/algs/qgis/MeanCoords.py
@@ -124,14 +124,14 @@ def processAlgorithm(self, progress):
         current = 0
         total = 100.0 / len(means)
         for (clazz, values) in means.iteritems():
-            outFeat = QgsFeature()
-            cx = values[0] / values[2]
-            cy = values[1] / values[2]
-            meanPoint = QgsPoint(cx, cy)
-
-            outFeat.setGeometry(QgsGeometry.fromPoint(meanPoint))
-            outFeat.setAttributes([cx, cy, clazz])
-            writer.addFeature(outFeat)
+            if values[2]:
+                outFeat = QgsFeature()
+                cx = values[0] / values[2]
+                cy = values[1] / values[2]
+                meanPoint = QgsPoint(cx, cy)
+                outFeat.setGeometry(QgsGeometry.fromPoint(meanPoint))
+                outFeat.setAttributes([cx, cy, clazz])
+                writer.addFeature(outFeat)
             current += 1
             progress.setPercentage(int(current * total))