Fix inconsistent-missing-override warning

jgrocha · jgrocha · commit bbf6ec05b9a4 · 2017-01-29T23:28:21.000Z
diff --git a/src/core/qgsvectordataprovider.cpp b/src/core/qgsvectordataprovider.cpp
@@ -700,7 +700,7 @@ bool QgsVectorDataProvider::isDeleteStyleFromDBSupported() const
   return false;
 }
 
-void QgsVectorDataProvider::deleteStyleById( const QString& uri, QString styleId, QString& errCause )
+void QgsVectorDataProvider::deleteStyleById( const QString& uri, QString styleId, QString& errCause ) const
 {
   Q_UNUSED( uri );
   Q_UNUSED( styleId );
diff --git a/src/core/qgsvectordataprovider.h b/src/core/qgsvectordataprovider.h
@@ -479,7 +479,7 @@ class CORE_EXPORT QgsVectorDataProvider : public QgsDataProvider
     /**
      * Deletes an existing style from the provider
      */
-    virtual void deleteStyleById( const QString& uri, QString styleId, QString& errCause );
+    virtual void deleteStyleById( const QString& uri, QString styleId, QString& errCause ) const;
 
     static QVariant convertValue( QVariant::Type type, const QString& value );
 
diff --git a/src/providers/postgres/qgspostgresprovider.cpp b/src/providers/postgres/qgspostgresprovider.cpp
@@ -4632,12 +4632,8 @@ QGISEXTERN int listStyles( const QString &uri, QStringList &ids, QStringList &na
   return numberOfRelatedStyles;
 }
 
-void QgsPostgresProvider::deleteStyleById( const QString &uri, QString styleId, QString &errCause )
+void QgsPostgresProvider::deleteStyleById( const QString &uri, QString styleId, QString &errCause ) const
 {
-  QgsDebugMsg( "deleteStyleById" );
-  QgsDebugMsg( styleId );
-  QgsDebugMsg( uri );
-
   QgsDataSourceUri dsUri( uri );
 
   QgsPostgresConn *conn = QgsPostgresConn::connectDb( dsUri.connectionInfo( false ), false );
diff --git a/src/providers/postgres/qgspostgresprovider.h b/src/providers/postgres/qgspostgresprovider.h
@@ -133,7 +133,7 @@ class QgsPostgresProvider : public QgsVectorDataProvider
     bool isValid() const override;
     virtual bool isSaveAndLoadStyleToDBSupported() const override { return true; }
     virtual bool isDeleteStyleFromDBSupported() const override { return true; }
-    void deleteStyleById( const QString& uri, QString styleId, QString& errCause );
+    void deleteStyleById( const QString& uri, QString styleId, QString& errCause ) const override;
     QgsAttributeList attributeIndexes() const override;
     QgsAttributeList pkAttributeIndexes() const override { return mPrimaryKeyAttrs; }
     QString defaultValueClause( int fieldId ) const override;

Original file line number	Diff line number	Diff line change
`@@ -700,7 +700,7 @@ bool QgsVectorDataProvider::isDeleteStyleFromDBSupported() const`
`700`	`700`	`return false;`
`701`	`701`	`}`
`702`	`702`
`703`		`-void QgsVectorDataProvider::deleteStyleById( const QString& uri, QString styleId, QString& errCause )`
	`703`	`+void QgsVectorDataProvider::deleteStyleById( const QString& uri, QString styleId, QString& errCause ) const`
`704`	`704`	`{`
`705`	`705`	`Q_UNUSED( uri );`
`706`	`706`	`Q_UNUSED( styleId );`
Original file line number	Diff line number	Diff line change
`@@ -4632,12 +4632,8 @@ QGISEXTERN int listStyles( const QString &uri, QStringList &ids, QStringList &na`
`4632`	`4632`	`return numberOfRelatedStyles;`
`4633`	`4633`	`}`
`4634`	`4634`
`4635`		`-void QgsPostgresProvider::deleteStyleById( const QString &uri, QString styleId, QString &errCause )`
	`4635`	`+void QgsPostgresProvider::deleteStyleById( const QString &uri, QString styleId, QString &errCause ) const`
`4636`	`4636`	`{`
`4637`		`- QgsDebugMsg( "deleteStyleById" );`
`4638`		`- QgsDebugMsg( styleId );`
`4639`		`- QgsDebugMsg( uri );`
`4640`		`-`
`4641`	`4637`	`QgsDataSourceUri dsUri( uri );`
`4642`	`4638`
`4643`	`4639`	`QgsPostgresConn *conn = QgsPostgresConn::connectDb( dsUri.connectionInfo( false ), false );`