delimited text provider: only add finite points to spatial index (fixes #11708)

jef-n · jef-n · commit 98ca2dcc8656 · 2015-01-10T18:02:13.000+01:00
diff --git a/src/providers/delimitedtext/qgsdelimitedtextprovider.cpp b/src/providers/delimitedtext/qgsdelimitedtextprovider.cpp
@@ -520,7 +520,7 @@ void QgsDelimitedTextProvider::scanFile( bool buildIndexes )
             mGeometryType = QGis::Point;
           }
           mNumberFeatures++;
-          if ( buildSpatialIndex )
+          if ( buildSpatialIndex && qIsFinite( pt.x() ) && qIsFinite( pt.y() ) )
           {
             QgsFeature f;
             f.setFeatureId( mFile->recordId() );
@@ -856,6 +856,7 @@ bool QgsDelimitedTextProvider::pointFromXY( QString &sX, QString &sY, QgsPoint &
     pt.setY( y );
     return true;
   }
+
   return false;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -520,7 +520,7 @@ void QgsDelimitedTextProvider::scanFile( bool buildIndexes )`
`520`	`520`	`mGeometryType = QGis::Point;`
`521`	`521`	`}`
`522`	`522`	`mNumberFeatures++;`
`523`		`- if ( buildSpatialIndex )`
	`523`	`+ if ( buildSpatialIndex && qIsFinite( pt.x() ) && qIsFinite( pt.y() ) )`
`524`	`524`	`{`
`525`	`525`	`QgsFeature f;`
`526`	`526`	`f.setFeatureId( mFile->recordId() );`
`@@ -856,6 +856,7 @@ bool QgsDelimitedTextProvider::pointFromXY( QString &sX, QString &sY, QgsPoint &`
`856`	`856`	`pt.setY( y );`
`857`	`857`	`return true;`
`858`	`858`	`}`
	`859`	`+`
`859`	`860`	`return false;`
`860`	`861`	`}`
`861`	`862`