[processing] skip invalid geometries when buffering (fix #13763)

qgis · Jan 15, 2016 · 3bc3f52 · 3bc3f52 · m-kuhn · Jan 15, 2016
1 parent 0c02f18
commit 3bc3f52
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/python/plugins/processing/algs/qgis/Buffer.py b/python/plugins/processing/algs/qgis/Buffer.py
@@ -26,6 +26,8 @@
 __revision__ = '$Format:%H$'
 
 from qgis.core import QgsFeature, QgsGeometry
+
+from processing.core.ProcessingLog import ProcessingLog
 from processing.tools import vector
 
 
@@ -55,6 +57,9 @@ def buffering(progress, writer, distance, field, useField, layer, dissolve,
                 value = distance
 
             inGeom = QgsGeometry(inFeat.geometry())
+            if inGeom.isGeosEmpty() or not inGeom.isGeosValid():
+                ProcessingLog.addToLog('Feature {} has empty or invalid geometry. Skipping...'.format(inFeat.id()), ProcessingLog.LOG_WARNING)
+                continue
             outGeom = inGeom.buffer(float(value), segments)
             if first:
                 tempGeom = QgsGeometry(outGeom)
@@ -77,6 +82,10 @@ def buffering(progress, writer, distance, field, useField, layer, dissolve,
             else:
                 value = distance
             inGeom = QgsGeometry(inFeat.geometry())
+            if inGeom.isGeosEmpty() or not inGeom.isGeosValid():
+                ProcessingLog.addToLog('Feature {} has empty or invalid geometry. Skipping...'.format(inFeat.id()), ProcessingLog.LOG_WARNING)
+                continue
+
             outGeom = inGeom.buffer(float(value), segments)
             outFeat.setGeometry(outGeom)
             outFeat.setAttributes(attrs)