Skip to content

Commit d5ef91f

Browse files
committedOct 6, 2016
Fix rule based renderer shows 0 count on first click of "show counts"
(cherry-picked from 291f7bd)
1 parent b7e24e3 commit d5ef91f

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed
 

‎src/gui/symbology-ng/qgsrulebasedrendererv2widget.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -504,8 +504,6 @@ void QgsRuleBasedRendererV2Widget::countFeatures()
504504
countMap[rule].duplicateCount = 0;
505505
}
506506

507-
QgsFeatureRequest req = QgsFeatureRequest().setFilterExpression( mRenderer->filter( mLayer->fields() ) );
508-
509507
QgsRenderContext renderContext;
510508
renderContext.setRendererScale( 0 ); // ignore scale
511509

@@ -525,10 +523,11 @@ void QgsRuleBasedRendererV2Widget::countFeatures()
525523
context << QgsExpressionContextUtils::layerScope( mLayer );
526524

527525
renderContext.setExpressionContext( context );
528-
req.setExpressionContext( context );
529526

530527
mRenderer->startRender( renderContext, mLayer->fields() );
531-
528+
// QgsRuleBasedRenderer::filter must be called after startRender
529+
QgsFeatureRequest req = QgsFeatureRequest().setFilterExpression( mRenderer->filter( mLayer->fields() ) );
530+
req.setExpressionContext( context );
532531
req.setSubsetOfAttributes( mRenderer->usedAttributes(), mLayer->fields() );
533532
QgsFeatureIterator fit = mLayer->getFeatures( req );
534533

0 commit comments

Comments
 (0)
Please sign in to comment.