Update QgsMapToolShowHideLabels to work with new vector api

- Works now, but should have worked before this fix
qgis · Feb 8, 2013 · 3676a38 · 3676a38
1 parent c90870d
commit 3676a38
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/src/app/qgsmaptoolshowhidelabels.cpp b/src/app/qgsmaptoolshowhidelabels.cpp
@@ -210,16 +210,14 @@ bool QgsMapToolShowHideLabels::selectedFeatures( QgsVectorLayer* vlayer,
   QgsDebugMsg( "Selection layer: " + vlayer->name() );
   QgsDebugMsg( "Selection polygon: " + selectGeomTrans.exportToWkt() );
 
-  QgsFeatureIterator fit = vlayer->getFeatures( QgsFeatureRequest().setFilterRect( selectGeomTrans.boundingBox() ).setFlags( QgsFeatureRequest::NoGeometry | QgsFeatureRequest::ExactIntersect ).setSubsetOfAttributes( QgsAttributeList() ) );
+  QgsFeatureIterator fit = vlayer->getFeatures( QgsFeatureRequest()
+                           .setFilterRect( selectGeomTrans.boundingBox() )
+                           .setFlags( QgsFeatureRequest::ExactIntersect )
+                           .setSubsetOfAttributes( QgsAttributeList() ) );
 
   QgsFeature f;
   while ( fit.nextFeature( f ) )
   {
-    QgsGeometry* g = f.geometry();
-
-    if ( !selectGeomTrans.intersects( g ) )
-      continue;
-
     selectedFeatIds.insert( f.id() );
   }