Remove duplicate QgsDataProvider::dataChanged( int ) signal

nyalldawson · nyalldawson · commit 5d78d60bf3fa · 2016-11-15T07:00:49.000+10:00
Was not needed and not emitted anywhere
diff --git a/doc/api_break.dox b/doc/api_break.dox
@@ -668,6 +668,12 @@ QgsDataItem        {#qgis_api_break_3_0_QgsDataItem}
 - emitBeginInsertItems(), emitEndInsertItems(), emitBeginRemoveItems(), emitEndRemoveItems(), emitDataChanged(), emitStateChanged() have been removed.
 - Favourites was renamed to Favorites
 
+QgsDataProvider        {#qgis_api_break_3_0_QgsDataProvider}
+---------------
+
+- The duplicate (and unused) dataChanged( int ) signal was removed. Use dataChanged() instead.
+
+
 QgsDataSourceURI        {#qgis_api_break_3_0_QgsDatasourceUri}
 ----------------
 
diff --git a/python/core/qgsdataprovider.sip b/python/core/qgsdataprovider.sip
@@ -326,12 +326,6 @@ class QgsDataProvider : QObject
      */
     void dataChanged();
 
-    /**
-     *   This is emitted whenever data or metadata (e.g. color table, extent) has changed
-     *   @param changed binary combination of changes
-     */
-    void dataChanged( int changed );
-
   protected:
 
     /** Add error message */
diff --git a/src/app/qgisapp.cpp b/src/app/qgisapp.cpp
@@ -10258,8 +10258,8 @@ void QgisApp::layersWereAdded( const QList<QgsMapLayer *>& theLayers )
 
     if ( provider )
     {
-      connect( provider, SIGNAL( dataChanged() ), layer, SLOT( triggerRepaint() ) );
-      connect( provider, SIGNAL( dataChanged() ), mMapCanvas, SLOT( refresh() ) );
+      connect( provider, &QgsDataProvider::dataChanged, layer, &QgsMapLayer::triggerRepaint );
+      connect( provider, &QgsDataProvider::dataChanged, mMapCanvas, &QgsMapCanvas::refresh );
     }
   }
 }
diff --git a/src/browser/qgsbrowser.cpp b/src/browser/qgsbrowser.cpp
@@ -442,8 +442,8 @@ void QgsBrowser::updateCurrentTab()
       QgsRasterLayer *rlayer = qobject_cast< QgsRasterLayer * >( mLayer );
       if ( rlayer )
       {
-        connect( rlayer->dataProvider(), SIGNAL( dataChanged() ), rlayer, SLOT( triggerRepaint() ) );
-        connect( rlayer->dataProvider(), SIGNAL( dataChanged() ), mapCanvas, SLOT( refresh() ) );
+        connect( rlayer->dataProvider(), &QgsRasterDataProvider::dataChanged, rlayer, &QgsRasterLayer::triggerRepaint );
+        connect( rlayer->dataProvider(), &QgsRasterDataProvider::dataChanged, mapCanvas, &QgsMapCanvas::refresh );
       }
     }
     mDirtyPreview = false;
diff --git a/src/core/qgsdataprovider.h b/src/core/qgsdataprovider.h
@@ -415,12 +415,6 @@ class CORE_EXPORT QgsDataProvider : public QObject
      */
     void dataChanged();
 
-    /**
-     *   This is emitted whenever data or metadata (e.g. color table, extent) has changed
-     *   @param changed binary combination of changes
-     */
-    void dataChanged( int changed );
-
   protected:
 
     /**
diff --git a/src/core/qgspointlocator.cpp b/src/core/qgspointlocator.cpp
@@ -631,7 +631,7 @@ QgsPointLocator::QgsPointLocator( QgsVectorLayer* layer, const QgsCoordinateRefe
   connect( mLayer, SIGNAL( featureAdded( QgsFeatureId ) ), this, SLOT( onFeatureAdded( QgsFeatureId ) ) );
   connect( mLayer, SIGNAL( featureDeleted( QgsFeatureId ) ), this, SLOT( onFeatureDeleted( QgsFeatureId ) ) );
   connect( mLayer, SIGNAL( geometryChanged( QgsFeatureId, const QgsGeometry& ) ), this, SLOT( onGeometryChanged( QgsFeatureId, const QgsGeometry& ) ) );
-  connect( mLayer, SIGNAL( dataChanged() ), this, SLOT( destroyIndex() ) );
+  connect( mLayer, &QgsVectorLayer::dataChanged, this, &QgsPointLocator::destroyIndex );
 }
 
 
diff --git a/src/core/qgstransaction.cpp b/src/core/qgstransaction.cpp
@@ -112,7 +112,7 @@ bool QgsTransaction::addLayer( QgsVectorLayer* layer )
     return false;
   }
 
-  connect( this, SIGNAL( afterRollback() ), layer->dataProvider(), SIGNAL( dataChanged() ) );
+  connect( this, &QgsTransaction::afterRollback, layer->dataProvider(), &QgsVectorDataProvider::dataChanged );
   connect( QgsMapLayerRegistry::instance(), SIGNAL( layersWillBeRemoved( QStringList ) ), this, SLOT( onLayersDeleted( QStringList ) ) );
   mLayers.insert( layer );
 
diff --git a/src/core/qgsvectorlayer.cpp b/src/core/qgsvectorlayer.cpp
@@ -1628,8 +1628,8 @@ bool QgsVectorLayer::setDataProvider( QString const & provider )
       mDataSource.chop( 10 );
   }
 
-  connect( mDataProvider, SIGNAL( dataChanged() ), this, SIGNAL( dataChanged() ) );
-  connect( mDataProvider, SIGNAL( dataChanged() ), this, SLOT( removeSelection() ) );
+  connect( mDataProvider, &QgsVectorDataProvider::dataChanged, this, &QgsVectorLayer::dataChanged );
+  connect( mDataProvider, &QgsVectorDataProvider::dataChanged, this, &QgsVectorLayer::removeSelection );
 
   return true;
 } // QgsVectorLayer:: setDataProvider

Original file line number	Diff line number	Diff line change
`@@ -10258,8 +10258,8 @@ void QgisApp::layersWereAdded( const QList<QgsMapLayer *>& theLayers )`
`10258`	`10258`
`10259`	`10259`	`if ( provider )`
`10260`	`10260`	`{`
`10261`		`- connect( provider, SIGNAL( dataChanged() ), layer, SLOT( triggerRepaint() ) );`
`10262`		`- connect( provider, SIGNAL( dataChanged() ), mMapCanvas, SLOT( refresh() ) );`
	`10261`	`+ connect( provider, &QgsDataProvider::dataChanged, layer, &QgsMapLayer::triggerRepaint );`
	`10262`	`+ connect( provider, &QgsDataProvider::dataChanged, mMapCanvas, &QgsMapCanvas::refresh );`
`10263`	`10263`	`}`
`10264`	`10264`	`}`
`10265`	`10265`	`}`
Original file line number	Diff line number	Diff line change
`@@ -442,8 +442,8 @@ void QgsBrowser::updateCurrentTab()`
`442`	`442`	`QgsRasterLayer rlayer = qobject_cast< QgsRasterLayer >( mLayer );`
`443`	`443`	`if ( rlayer )`
`444`	`444`	`{`
`445`		`- connect( rlayer->dataProvider(), SIGNAL( dataChanged() ), rlayer, SLOT( triggerRepaint() ) );`
`446`		`- connect( rlayer->dataProvider(), SIGNAL( dataChanged() ), mapCanvas, SLOT( refresh() ) );`
	`445`	`+ connect( rlayer->dataProvider(), &QgsRasterDataProvider::dataChanged, rlayer, &QgsRasterLayer::triggerRepaint );`
	`446`	`+ connect( rlayer->dataProvider(), &QgsRasterDataProvider::dataChanged, mapCanvas, &QgsMapCanvas::refresh );`
`447`	`447`	`}`
`448`	`448`	`}`
`449`	`449`	`mDirtyPreview = false;`
Original file line number	Diff line number	Diff line change
`@@ -631,7 +631,7 @@ QgsPointLocator::QgsPointLocator( QgsVectorLayer* layer, const QgsCoordinateRefe`
`631`	`631`	`connect( mLayer, SIGNAL( featureAdded( QgsFeatureId ) ), this, SLOT( onFeatureAdded( QgsFeatureId ) ) );`
`632`	`632`	`connect( mLayer, SIGNAL( featureDeleted( QgsFeatureId ) ), this, SLOT( onFeatureDeleted( QgsFeatureId ) ) );`
`633`	`633`	`connect( mLayer, SIGNAL( geometryChanged( QgsFeatureId, const QgsGeometry& ) ), this, SLOT( onGeometryChanged( QgsFeatureId, const QgsGeometry& ) ) );`
`634`		`- connect( mLayer, SIGNAL( dataChanged() ), this, SLOT( destroyIndex() ) );`
	`634`	`+ connect( mLayer, &QgsVectorLayer::dataChanged, this, &QgsPointLocator::destroyIndex );`
`635`	`635`	`}`
`636`	`636`
`637`	`637`
Original file line number	Diff line number	Diff line change
`@@ -112,7 +112,7 @@ bool QgsTransaction::addLayer( QgsVectorLayer* layer )`
`112`	`112`	`return false;`
`113`	`113`	`}`
`114`	`114`
`115`		`- connect( this, SIGNAL( afterRollback() ), layer->dataProvider(), SIGNAL( dataChanged() ) );`
	`115`	`+ connect( this, &QgsTransaction::afterRollback, layer->dataProvider(), &QgsVectorDataProvider::dataChanged );`
`116`	`116`	`connect( QgsMapLayerRegistry::instance(), SIGNAL( layersWillBeRemoved( QStringList ) ), this, SLOT( onLayersDeleted( QStringList ) ) );`
`117`	`117`	`mLayers.insert( layer );`
`118`	`118`
Original file line number	Diff line number	Diff line change
`@@ -1628,8 +1628,8 @@ bool QgsVectorLayer::setDataProvider( QString const & provider )`
`1628`	`1628`	`mDataSource.chop( 10 );`
`1629`	`1629`	`}`
`1630`	`1630`
`1631`		`- connect( mDataProvider, SIGNAL( dataChanged() ), this, SIGNAL( dataChanged() ) );`
`1632`		`- connect( mDataProvider, SIGNAL( dataChanged() ), this, SLOT( removeSelection() ) );`
	`1631`	`+ connect( mDataProvider, &QgsVectorDataProvider::dataChanged, this, &QgsVectorLayer::dataChanged );`
	`1632`	`+ connect( mDataProvider, &QgsVectorDataProvider::dataChanged, this, &QgsVectorLayer::removeSelection );`
`1633`	`1633`
`1634`	`1634`	`return true;`
`1635`	`1635`	`} // QgsVectorLayer:: setDataProvider`