From 8101f35f22732718cf2d405983e879c10177d8b3 Mon Sep 17 00:00:00 2001
From: wmayer <wmayer@users.sourceforge.net>
Date: Fri, 9 Feb 2018 23:10:40 +0100
Subject: [PATCH] avoid superfluous transaction on PD primitive creation

---
 src/Mod/PartDesign/Gui/TaskPrimitiveParameters.cpp | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/Mod/PartDesign/Gui/TaskPrimitiveParameters.cpp b/src/Mod/PartDesign/Gui/TaskPrimitiveParameters.cpp
index 4c5a6a541c..1017c48ac6 100644
--- a/src/Mod/PartDesign/Gui/TaskPrimitiveParameters.cpp
+++ b/src/Mod/PartDesign/Gui/TaskPrimitiveParameters.cpp
@@ -638,10 +638,9 @@ void  TaskBoxPrimitives::setPrimitive(QString name)
         }
 
         // Execute the Python block
-        QString prim = tr("Create primitive");
-        Gui::Application::Instance->activeDocument()->openCommand(prim.toUtf8());
+        // No need to open a transaction because this is already done in the command
+        // class or when starting to edit a primitve.
         Gui::Command::runCommand(Gui::Command::Doc, cmd.toUtf8());
-        Gui::Application::Instance->activeDocument()->commitCommand();
         Gui::Command::runCommand(Gui::Command::Doc, "App.ActiveDocument.recompute()");
     }
     catch (const Base::PyException& e) {