If graduated ranges are inconsistent or overlapping then don't allow

changes via the histogram. Prevents ranges from getting even more mucked up.
qgis · Jun 12, 2015 · 787e925 · 787e925
1 parent 604f7e5
commit 787e925
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/src/gui/symbology-ng/qgsgraduatedhistogramwidget.cpp b/src/gui/symbology-ng/qgsgraduatedhistogramwidget.cpp
@@ -82,11 +82,21 @@ void QgsGraduatedHistogramWidget::drawHistogram()
   if ( !mRenderer )
     return;
 
-  setGraduatedRanges( mRenderer->ranges() );
+  bool pickerEnabled = false;
+  if ( !mRenderer->rangesOverlap() && !mRenderer->rangesHaveGaps() && !mRenderer->ranges().isEmpty() )
+  {
+    setGraduatedRanges( mRenderer->ranges() );
+    pickerEnabled = true;
+  }
+  else
+  {
+    setGraduatedRanges( QgsRangeList() );
+  }
   QgsHistogramWidget::drawHistogram();
 
   // histo picker
-  mHistoPicker->setEnabled( true );
+  mHistoPicker->setEnabled( pickerEnabled );
+  mFilter->blockSignals( !pickerEnabled );
 }
 
 void QgsGraduatedHistogramWidget::mousePress( double value )