fix #2947

MerginMaps · Dec 7, 2023 · efdb598 · efdb598 · inputapp-bot · Dec 7, 2023
1 parent 2a37b1b
commit efdb598
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/app/attributes/attributecontroller.cpp b/app/attributes/attributecontroller.cpp
@@ -1056,15 +1056,13 @@ bool AttributeController::deleteFeature()
   if ( !mFeatureLayerPair.layer() )
     return false;
 
-  bool rv = true;
-
   if ( !startEditing() )
   {
-    rv = false;
+    return false;
   }
 
   bool isDeleted = mFeatureLayerPair.layer()->deleteFeature( mFeatureLayerPair.feature().id() );
-  rv = commit();
+  bool rv = commit();
 
   if ( !isDeleted || !rv )
   {