Merge pull request #1667 from elpaso/bugfix11510

NathanW2 · NathanW2 · commit 34af9788a4ac · 2014-10-28T21:24:43.000+10:00
Fix #11510 graduated renderer causes crash: added check for geometry
diff --git a/src/core/symbology-ng/qgsgraduatedsymbolrendererv2.cpp b/src/core/symbology-ng/qgsgraduatedsymbolrendererv2.cpp
@@ -965,7 +965,11 @@ QList<double> QgsGraduatedSymbolRendererV2::getDataValues( QgsVectorLayer *vlaye
   else
     lst = expression->referencedColumns();
 
-  QgsFeatureIterator fit = vlayer->getFeatures( QgsFeatureRequest().setSubsetOfAttributes( lst, vlayer->pendingFields() ) );
+  QgsFeatureIterator fit = vlayer->getFeatures( QgsFeatureRequest()
+                                                .setFlags( expression->needsGeometry() ?
+                                                             QgsFeatureRequest::NoFlags :
+                                                             QgsFeatureRequest::NoGeometry  )
+                                                .setSubsetOfAttributes( lst, vlayer->pendingFields() ) );
 
   // create list of non-null attribute values
   while ( fit.nextFeature( f ) )
