Use isEmpty() to test for null values

qgis · Mar 4, 2013 · 029c403 · 029c403
1 parent f7bf3f2
commit 029c403
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/core/symbology-ng/qgsgraduatedsymbolrendererv2.cpp b/src/core/symbology-ng/qgsgraduatedsymbolrendererv2.cpp
@@ -193,7 +193,7 @@ QgsSymbolV2* QgsGraduatedSymbolRendererV2::symbolForFeature( QgsFeature& feature
   }
 
   // Null values should not be categorized
-  if ( attrs[mAttrNum] == "")
+  if ( attrs[mAttrNum].toString().isEmpty() )
     return NULL;
 
   // find the right category    
@@ -809,7 +809,7 @@ QgsGraduatedSymbolRendererV2* QgsGraduatedSymbolRendererV2::createRenderer(
 
     // create list of non-null attribute values
     while ( fit.nextFeature( f ) )
-      if ( f.attribute( attrNum ) != "" )
+      if ( !f.attribute( attrNum ).toString().isEmpty() )
         values.append( f.attribute( attrNum ).toDouble() );
 
     // calculate the breaks