Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
followup 5c267f2: apply @wonder-sk 's comments
  • Loading branch information
nirvn committed Jun 11, 2018
1 parent 03151f8 commit d1fa150
Show file tree
Hide file tree
Showing 7 changed files with 19 additions and 15 deletions.
6 changes: 3 additions & 3 deletions python/core/auto_generated/qgsmaplayerstylemanager.sip.in
Expand Up @@ -224,7 +224,7 @@ Emitted when the current style has been changed
};


class QgsLayerStyleOverride
class QgsMapLayerStyleOverride
{
%Docstring
Restore overridden layer style on destruction.
Expand All @@ -237,13 +237,13 @@ Restore overridden layer style on destruction.
%End
public:

QgsLayerStyleOverride( QgsMapLayer *layer );
QgsMapLayerStyleOverride( QgsMapLayer *layer );
%Docstring
Construct a style override object associated with a map layer.
The overridden style will be restored upon object destruction.
%End

~QgsLayerStyleOverride();
~QgsMapLayerStyleOverride();

void setOverrideStyle( const QString &style );
%Docstring
Expand Down
2 changes: 1 addition & 1 deletion src/core/dxf/qgsdxfexport.cpp
Expand Up @@ -979,7 +979,7 @@ void QgsDxfExport::writeEntities()
continue;
}

QgsLayerStyleOverride styleOverride( vl );
QgsMapLayerStyleOverride styleOverride( vl );
if ( mMapSettings.layerStyleOverrides().contains( vl->id() ) )
{
QgsDebugMsg( QString( "%1: apply override style" ).arg( vl->id() ) );
Expand Down
2 changes: 1 addition & 1 deletion src/core/layertree/qgslayertreemodel.cpp
Expand Up @@ -1186,7 +1186,7 @@ void QgsLayerTreeModel::addLegendToLayer( QgsLayerTreeLayer *nodeL )
if ( !layerLegend )
return;

QgsLayerStyleOverride styleOverride( ml );
QgsMapLayerStyleOverride styleOverride( ml );
if ( mLayerStyleOverrides.contains( ml->id() ) )
styleOverride.setOverrideStyle( mLayerStyleOverrides.value( ml->id() ) );

Expand Down
2 changes: 1 addition & 1 deletion src/core/qgsmaphittest.cpp
Expand Up @@ -101,7 +101,7 @@ bool QgsMapHitTest::legendKeyVisible( const QString &ruleKey, QgsVectorLayer *la

void QgsMapHitTest::runHitTestLayer( QgsVectorLayer *vl, SymbolSet &usedSymbols, SymbolSet &usedSymbolsRuleKey, QgsRenderContext &context )
{
QgsLayerStyleOverride styleOverride( vl );
QgsMapLayerStyleOverride styleOverride( vl );
if ( mSettings.layerStyleOverrides().contains( vl->id() ) )
styleOverride.setOverrideStyle( mSettings.layerStyleOverrides().value( vl->id() ) );

Expand Down
18 changes: 11 additions & 7 deletions src/core/qgsmaplayerstylemanager.h
Expand Up @@ -201,20 +201,20 @@ class CORE_EXPORT QgsMapLayerStyleManager : public QObject
*
* \since QGIS 3.2
*/
class CORE_EXPORT QgsLayerStyleOverride
class CORE_EXPORT QgsMapLayerStyleOverride
{
public:

/**
* Construct a style override object associated with a map layer.
* The overridden style will be restored upon object destruction.
*/
QgsLayerStyleOverride( QgsMapLayer *layer )
QgsMapLayerStyleOverride( QgsMapLayer *layer )
: mLayer( layer )
{
}

~QgsLayerStyleOverride()
~QgsMapLayerStyleOverride()
{
if ( mLayer && mStyleOverridden )
mLayer->styleManager()->restoreOverrideStyle();
Expand All @@ -226,10 +226,14 @@ class CORE_EXPORT QgsLayerStyleOverride
*/
void setOverrideStyle( const QString &style )
{
if ( mLayer && mStyleOverridden )
mLayer->styleManager()->restoreOverrideStyle();
mLayer->styleManager()->setOverrideStyle( style );
mStyleOverridden = true;
if ( mLayer )
{
if ( mStyleOverridden )
mLayer->styleManager()->restoreOverrideStyle();

mLayer->styleManager()->setOverrideStyle( style );
mStyleOverridden = true;
}
}

private:
Expand Down
2 changes: 1 addition & 1 deletion src/core/qgsmaprendererjob.cpp
Expand Up @@ -296,7 +296,7 @@ LayerRenderJobs QgsMapRendererJob::prepareJobs( QPainter *painter, QgsLabelingEn
if ( mFeatureFilterProvider )
job.context.setFeatureFilterProvider( mFeatureFilterProvider );

QgsLayerStyleOverride styleOverride( ml );
QgsMapLayerStyleOverride styleOverride( ml );
if ( mSettings.layerStyleOverrides().contains( ml->id() ) )
styleOverride.setOverrideStyle( mSettings.layerStyleOverrides().value( ml->id() ) );

Expand Down
2 changes: 1 addition & 1 deletion src/core/qgsmapthemecollection.cpp
Expand Up @@ -365,7 +365,7 @@ QMap<QString, QString> QgsMapThemeCollection::mapThemeStyleOverrides( const QStr
if ( layerRec.usingCurrentStyle )
{
QgsMapLayer *layer = layerRec.layer();
QgsLayerStyleOverride styleOverride( layer );
QgsMapLayerStyleOverride styleOverride( layer );
styleOverride.setOverrideStyle( layerRec.currentStyle );

// set the checked legend nodes
Expand Down

0 comments on commit d1fa150

Please sign in to comment.