[bugfix] Categorized style issue using numeric fields with NULL values

Fixes #17442
qgis · Nov 24, 2017 · 29abd9d · 29abd9d
1 parent 8b15c73
commit 29abd9d
Showing 1 changed file with 7 additions and 13 deletions.
diff --git a/src/gui/symbology/qgscategorizedsymbolrendererwidget.cpp b/src/gui/symbology/qgscategorizedsymbolrendererwidget.cpp
@@ -604,28 +604,22 @@ static void _createCategories( QgsCategoryList &cats, QList<QVariant> &values, Q
 
   int num = values.count();
 
-  bool hasNull = false;
-
   for ( int i = 0; i < num; i++ )
   {
     QVariant value = values[i];
-    if ( value.toString().isNull() )
+    QgsSymbol *newSymbol = symbol->clone();
+    if ( ! value.isNull() )
     {
-      hasNull = true;
+      cats.append( QgsRendererCategory( value, newSymbol, value.toString(), true ) );
     }
-    QgsSymbol *newSymbol = symbol->clone();
-
-    cats.append( QgsRendererCategory( value, newSymbol, value.toString(), true ) );
   }
 
-  // add null (default) value if not exists
-  if ( !hasNull )
-  {
-    QgsSymbol *newSymbol = symbol->clone();
-    cats.append( QgsRendererCategory( QVariant( "" ), newSymbol, QString(), true ) );
-  }
+  // add null (default) value
+  QgsSymbol *newSymbol = symbol->clone();
+  cats.append( QgsRendererCategory( QVariant( "" ), newSymbol, QString(), true ) );
 }
 
+
 void QgsCategorizedSymbolRendererWidget::addCategories()
 {
   QString attrName = mExpressionWidget->currentField();