put original message into else

qgis · Jan 8, 2019 · a16e5f27051f438b57873798f9c16bb6a35e4eae · a16e5f2
1 parent dba3b28
commit a16e5f27051f438b57873798f9c16bb6a35e4eae
Unified Split

Showing with 5 additions and 2 deletions.

+5 −2 src/providers/ogr/qgsgeopackagedataitems.cpp
diff --git a/src/providers/ogr/qgsgeopackagedataitems.cpp b/src/providers/ogr/qgsgeopackagedataitems.cpp
@@ -525,11 +525,10 @@ bool QgsGeoPackageAbstractLayerItem::deleteLayer()
   }
   else
   {
-    QMessageBox::information( nullptr, tr( "Delete Layer" ), tr( "Layer <b>%1</b> deleted successfully." ).arg( mName ) );
     if ( mParent )
     {
       QgsGeoPackageConnectionItem *connectionParentItem = qobject_cast<QgsGeoPackageConnectionItem *>( mParent );
-      if( connectionParentItem )
+      if ( connectionParentItem )
       {
         if ( QMessageBox::question( nullptr, QObject::tr( "Delete Layer" ), QObject::tr( "The layer <b>%1</b> deleted successfully."
                                     " Compact database (VACUUM) <b>%2</b> now?" ).arg( mName, connectionParentItem->name() ), QMessageBox::Yes | QMessageBox::No, QMessageBox::No ) == QMessageBox::Yes )
@@ -539,6 +538,10 @@ bool QgsGeoPackageAbstractLayerItem::deleteLayer()
       }
       mParent->refreshConnections();
     }
+    else
+    {
+      QMessageBox::information( nullptr, tr( "Delete Layer" ), tr( "Layer <b>%1</b> deleted successfully." ).arg( mName ) );
+    }
   }
   return true;
 }