Mark project as dirty when changing layer subset string

qgis · May 28, 2018 · a525b2d · a525b2d
1 parent e25c813
commit a525b2d
Showing 1 changed file with 19 additions and 14 deletions.
diff --git a/src/gui/qgsquerybuilder.cpp b/src/gui/qgsquerybuilder.cpp
@@ -14,6 +14,7 @@
  ***************************************************************************/
 #include "qgsquerybuilder.h"
 #include "qgslogger.h"
+#include "qgsproject.h"
 #include "qgssettings.h"
 #include "qgsvectorlayer.h"
 #include "qgsvectordataprovider.h"
@@ -243,23 +244,27 @@ void QgsQueryBuilder::test()
 
 void QgsQueryBuilder::accept()
 {
-  if ( !mLayer->setSubsetString( txtSQL->text() ) )
+  if ( txtSQL->text() != mOrigSubsetString )
   {
-    //error in query - show the problem
-    if ( mLayer->dataProvider()->hasErrors() )
+    if ( !mLayer->setSubsetString( txtSQL->text() ) )
     {
-      QMessageBox::warning( this,
-                            tr( "Query Result" ),
-                            tr( "An error occurred when executing the query." )
-                            + tr( "\nThe data provider said:\n%1" ).arg( mLayer->dataProvider()->errors().join( QStringLiteral( "\n" ) ) ) );
-      mLayer->dataProvider()->clearErrors();
-    }
-    else
-    {
-      QMessageBox::warning( this, tr( "Query Result" ), tr( "Error in query. The subset string could not be set." ) );
+      //error in query - show the problem
+      if ( mLayer->dataProvider()->hasErrors() )
+      {
+        QMessageBox::warning( this,
+                              tr( "Query Result" ),
+                              tr( "An error occurred when executing the query." )
+                              + tr( "\nThe data provider said:\n%1" ).arg( mLayer->dataProvider()->errors().join( QStringLiteral( "\n" ) ) ) );
+        mLayer->dataProvider()->clearErrors();
+      }
+      else
+      {
+        QMessageBox::warning( this, tr( "Query Result" ), tr( "Error in query. The subset string could not be set." ) );
+      }
+
+      return;
     }
-
-    return;
+    QgsProject::instance()->setDirty( true );
   }
 
   QDialog::accept();