Skip to content

Commit

Permalink
Adds missing styleChanged connection
Browse files Browse the repository at this point in the history
Remove useless if ( mRenderContext ) in onAttributeValueChanged and
onStyleChanged
  • Loading branch information
lbartoletti committed Oct 25, 2018
1 parent d33052a commit a091e64
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions src/core/qgstracer.cpp
Expand Up @@ -617,6 +617,7 @@ void QgsTracer::setLayers( const QList<QgsVectorLayer *> &layers )
connect( layer, &QgsVectorLayer::geometryChanged, this, &QgsTracer::onGeometryChanged );
connect( layer, &QgsVectorLayer::attributeValueChanged, this, &QgsTracer::onAttributeValueChanged );
connect( layer, &QgsVectorLayer::dataChanged, this, &QgsTracer::onDataChanged );
connect( layer, &QgsVectorLayer::styleChanged, this, &QgsTracer::onStyleChanged );
connect( layer, &QObject::destroyed, this, &QgsTracer::onLayerDestroyed );
}

Expand Down Expand Up @@ -705,8 +706,7 @@ void QgsTracer::onAttributeValueChanged( QgsFeatureId fid, int idx, const QVaria
Q_UNUSED( fid );
Q_UNUSED( idx );
Q_UNUSED( value );
if ( mRenderContext )
invalidateGraph();
invalidateGraph();
}

void QgsTracer::onDataChanged( )
Expand All @@ -716,8 +716,7 @@ void QgsTracer::onDataChanged( )

void QgsTracer::onStyleChanged( )
{
if ( mRenderContext )
invalidateGraph();
invalidateGraph();
}

void QgsTracer::onLayerDestroyed( QObject *obj )
Expand Down

0 comments on commit a091e64

Please sign in to comment.