Skip to content

Commit 0a3583d

Browse files
committed
It does not make sense to classify into zero classes, does it?
1 parent 8d28b6e commit 0a3583d

File tree

3 files changed

+17
-2
lines changed

3 files changed

+17
-2
lines changed

src/core/symbology-ng/qgsgraduatedsymbolrendererv2.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -690,6 +690,8 @@ QgsGraduatedSymbolRendererV2* QgsGraduatedSymbolRendererV2::createRenderer(
690690
QgsSymbolV2* symbol,
691691
QgsVectorColorRampV2* ramp )
692692
{
693+
if ( classes < 1 )
694+
return NULL;
693695

694696
int attrNum = vlayer->fieldNameIndex( attrName );
695697

src/gui/symbology-ng/qgsgraduatedsymbolrendererv2widget.cpp

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -176,9 +176,16 @@ void QgsGraduatedSymbolRendererV2Widget::classifyGraduated()
176176
mode = QgsGraduatedSymbolRendererV2::Quantile;
177177

178178
// create and set new renderer
179+
QgsGraduatedSymbolRendererV2* r = QgsGraduatedSymbolRendererV2::createRenderer(
180+
mLayer, attrName, classes, mode, mGraduatedSymbol, ramp );
181+
if ( !r )
182+
{
183+
QMessageBox::critical( this, tr( "Error" ), tr( "Renderer creation has failed." ) );
184+
return;
185+
}
186+
179187
delete mRenderer;
180-
mRenderer = QgsGraduatedSymbolRendererV2::createRenderer(
181-
mLayer, attrName, classes, mode, mGraduatedSymbol, ramp );
188+
mRenderer = r;
182189

183190
populateRanges();
184191
}

src/ui/qgsgraduatedsymbolrendererv2widget.ui

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,12 @@
7070
<verstretch>0</verstretch>
7171
</sizepolicy>
7272
</property>
73+
<property name="minimum">
74+
<number>1</number>
75+
</property>
76+
<property name="maximum">
77+
<number>999</number>
78+
</property>
7379
<property name="value">
7480
<number>5</number>
7581
</property>

0 commit comments

Comments
 (0)