Correctly emit presetsChanged() from QgsVisibilityPresetCollection

nyalldawson · nyalldawson · commit b51d70ab956e · 2016-05-13T10:05:05.000+10:00
diff --git a/src/core/qgsvisibilitypresetcollection.cpp b/src/core/qgsvisibilitypresetcollection.cpp
@@ -57,6 +57,7 @@ void QgsVisibilityPresetCollection::insert( const QString& name, const QgsVisibi
   mPresets.insert( name, state );
 
   reconnectToLayersStyleManager();
+  emit presetsChanged();
 }
 
 void QgsVisibilityPresetCollection::update( const QString& name, const PresetRecord& state )
@@ -67,6 +68,7 @@ void QgsVisibilityPresetCollection::update( const QString& name, const PresetRec
   mPresets[name] = state;
 
   reconnectToLayersStyleManager();
+  emit presetsChanged();
 }
 
 void QgsVisibilityPresetCollection::removePreset( const QString& name )
@@ -77,13 +79,15 @@ void QgsVisibilityPresetCollection::removePreset( const QString& name )
   mPresets.remove( name );
 
   reconnectToLayersStyleManager();
+  emit presetsChanged();
 }
 
 void QgsVisibilityPresetCollection::clear()
 {
   mPresets.clear();
 
   reconnectToLayersStyleManager();
+  emit presetsChanged();
 }
 
 QStringList QgsVisibilityPresetCollection::presets() const
@@ -231,6 +235,7 @@ void QgsVisibilityPresetCollection::readXML( const QDomDocument& doc )
   }
 
   reconnectToLayersStyleManager();
+  emit presetsChanged();
 }
 
 void QgsVisibilityPresetCollection::writeXML( QDomDocument& doc )
@@ -286,6 +291,7 @@ void QgsVisibilityPresetCollection::registryLayersRemoved( const QStringList& la
       rec.mPerLayerCurrentStyle.remove( layerID );
     }
   }
+  emit presetsChanged();
 }
 
 void QgsVisibilityPresetCollection::layerStyleRenamed( const QString& oldName, const QString& newName )
@@ -308,4 +314,5 @@ void QgsVisibilityPresetCollection::layerStyleRenamed( const QString& oldName, c
         rec.mPerLayerCurrentStyle[layerID] = newName;
     }
   }
+  emit presetsChanged();
 }

Original file line number	Diff line number	Diff line change
`@@ -57,6 +57,7 @@ void QgsVisibilityPresetCollection::insert( const QString& name, const QgsVisibi`
`57`	`57`	`mPresets.insert( name, state );`
`58`	`58`
`59`	`59`	`reconnectToLayersStyleManager();`
	`60`	`+ emit presetsChanged();`
`60`	`61`	`}`
`61`	`62`
`62`	`63`	`void QgsVisibilityPresetCollection::update( const QString& name, const PresetRecord& state )`
`@@ -67,6 +68,7 @@ void QgsVisibilityPresetCollection::update( const QString& name, const PresetRec`
`67`	`68`	`mPresets[name] = state;`
`68`	`69`
`69`	`70`	`reconnectToLayersStyleManager();`
	`71`	`+ emit presetsChanged();`
`70`	`72`	`}`
`71`	`73`
`72`	`74`	`void QgsVisibilityPresetCollection::removePreset( const QString& name )`
`@@ -77,13 +79,15 @@ void QgsVisibilityPresetCollection::removePreset( const QString& name )`
`77`	`79`	`mPresets.remove( name );`
`78`	`80`
`79`	`81`	`reconnectToLayersStyleManager();`
	`82`	`+ emit presetsChanged();`
`80`	`83`	`}`
`81`	`84`
`82`	`85`	`void QgsVisibilityPresetCollection::clear()`
`83`	`86`	`{`
`84`	`87`	`mPresets.clear();`
`85`	`88`
`86`	`89`	`reconnectToLayersStyleManager();`
	`90`	`+ emit presetsChanged();`
`87`	`91`	`}`
`88`	`92`
`89`	`93`	`QStringList QgsVisibilityPresetCollection::presets() const`
`@@ -231,6 +235,7 @@ void QgsVisibilityPresetCollection::readXML( const QDomDocument& doc )`
`231`	`235`	`}`
`232`	`236`
`233`	`237`	`reconnectToLayersStyleManager();`
	`238`	`+ emit presetsChanged();`
`234`	`239`	`}`
`235`	`240`
`236`	`241`	`void QgsVisibilityPresetCollection::writeXML( QDomDocument& doc )`
`@@ -286,6 +291,7 @@ void QgsVisibilityPresetCollection::registryLayersRemoved( const QStringList& la`
`286`	`291`	`rec.mPerLayerCurrentStyle.remove( layerID );`
`287`	`292`	`}`
`288`	`293`	`}`
	`294`	`+ emit presetsChanged();`
`289`	`295`	`}`
`290`	`296`
`291`	`297`	`void QgsVisibilityPresetCollection::layerStyleRenamed( const QString& oldName, const QString& newName )`
`@@ -308,4 +314,5 @@ void QgsVisibilityPresetCollection::layerStyleRenamed( const QString& oldName, c`
`308`	`314`	`rec.mPerLayerCurrentStyle[layerID] = newName;`
`309`	`315`	`}`
`310`	`316`	`}`
	`317`	`+ emit presetsChanged();`
`311`	`318`	`}`