From 044d90b4d058f98564ba59db0c3f767a5c4d88e7 Mon Sep 17 00:00:00 2001
From: LemonBoy <thatlemon@gmail.com>
Date: Mon, 29 Jan 2024 18:32:36 +0100
Subject: [PATCH] Make the random color assignment undo-able

Create a transaction in the currently selected document to allow the
user to roll-back the color assignment.

Closes #11689
---
 src/Gui/CommandFeat.cpp | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/src/Gui/CommandFeat.cpp b/src/Gui/CommandFeat.cpp
index c197fc5f36..e80dae50f7 100644
--- a/src/Gui/CommandFeat.cpp
+++ b/src/Gui/CommandFeat.cpp
@@ -88,26 +88,31 @@ void StdCmdRandomColor::activated(int iMsg)
 
     // get the complete selection
     std::vector<SelectionSingleton::SelObj> sel = Selection().getCompleteSelection();
+
+    Command::openCommand(QT_TRANSLATE_NOOP("Command", "Set Random Color"));
     for (const auto & it : sel) {
         auto fMax = (float)RAND_MAX;
         auto fRed = (float)rand()/fMax;
         auto fGrn = (float)rand()/fMax;
         auto fBlu = (float)rand()/fMax;
+        auto objColor = App::Color(fRed, fGrn, fBlu);
 
         ViewProvider* view = Application::Instance->getDocument(it.pDoc)->getViewProvider(it.pObject);
         auto vpLink = dynamic_cast<ViewProviderLink*>(view);
         if(vpLink) {
             if(!vpLink->OverrideMaterial.getValue())
-                cmdGuiObjectArgs(it.pObject, "OverrideMaterial = True");
-            cmdGuiObjectArgs(it.pObject, "ShapeMaterial.DiffuseColor=(%.2f,%.2f,%.2f)", fRed, fGrn, fBlu);
+                vpLink->OverrideMaterial.setValue(true);
+            vpLink->ShapeMaterial.setDiffuseColor(objColor);
             continue;
         }
         auto color = dynamic_cast<App::PropertyColor*>(view->getPropertyByName("ShapeColor"));
         if (color) {
             // get the view provider of the selected object and set the shape color
-            cmdGuiObjectArgs(it.pObject, "ShapeColor=(%.2f,%.2f,%.2f)", fRed, fGrn, fBlu);
+            color->setValue(objColor);
         }
     }
+
+    Command::commitCommand();
 }
 
 bool StdCmdRandomColor::isActive()