More reliable trigger of refresh when changing styles

Previously when changing the presets map canvas would not be redrawn

Author: wonder-sk

src/app/qgsmaplayerstyleguiutils.cpp

@@ -103,8 +103,6 @@ void QgsMapLayerStyleGuiUtils::useStyle()
   bool res = layer->styleManager()->setCurrentStyle( name );
   if ( !res )
     QgsDebugMsg( "Failed to set current style: " + name );
-
-  layer->triggerRepaint();
 }
 
 
@@ -128,12 +126,7 @@ void QgsMapLayerStyleGuiUtils::removeStyle()
   if ( name == defaultStyleName() )
     name.clear();
 
-  bool needsRefresh = ( layer->styleManager()->currentStyle() == name );
-
   bool res = layer->styleManager()->removeStyle( name );
   if ( !res )
     QgsDebugMsg( "Failed to remove style: " + name );
-
-  if ( needsRefresh )
-    layer->triggerRepaint();
 }

src/core/qgsmaplayerstylemanager.cpp

@@ -131,6 +131,7 @@ bool QgsMapLayerStyleManager::setCurrentStyle( const QString& name )
   syncCurrentStyle(); // sync before unloading it
   mCurrentStyle = name;
   mStyles[mCurrentStyle].writeToLayer( mLayer );
+  mLayer->triggerRepaint();
   return true;
 }