Fix limited random color ramp always returns 1 less color than set

qgis · Aug 24, 2016 · 1d98b10 · 1d98b10
1 parent b7716aa
commit 1d98b10
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/core/qgscolorramp.cpp b/src/core/qgscolorramp.cpp
@@ -313,8 +313,11 @@ double QgsLimitedRandomColorRamp::value( int index ) const
 
 QColor QgsLimitedRandomColorRamp::color( double value ) const
 {
+  if ( value < 0 || value > 1 )
+    return QColor();
+
   int colorCnt = mColors.count();
-  int colorIdx = static_cast< int >( value * ( colorCnt - 1 ) );
+  int colorIdx = qMin( static_cast< int >( value * colorCnt ), colorCnt - 1 );
 
   if ( colorIdx >= 0 && colorIdx < colorCnt )
     return mColors.at( colorIdx );