lok: Simplify the check for command in sendDialogEvent.

Change-Id: I1d2c967b68113d2528b80e91c32170f749ed9335
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104434
Tested-by: Andras Timar <andras.timar@collabora.com>
Reviewed-by: Andras Timar <andras.timar@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/105738
Tested-by: Jenkins
Reviewed-by: Jan Holesovsky <kendy@collabora.com>

1 files changed, 25 insertions, 32 deletions

diff --git a/desktop/source/lib/init.cxx b/desktop/source/lib/init.cxx
index 9aec16e027cb..b31d68dc5391 100644
--- a/desktop/source/lib/init.cxx
+++ b/desktop/source/lib/init.cxx
@@ -3740,43 +3740,36 @@ static void doc_sendDialogEvent(LibreOfficeKitDocument* /*pThis*/, unsigned long
             WindowUIObject aUIObject(pWindow);
             std::unique_ptr<UIObject> pUIWindow(aUIObject.get_visible_child(aMap["id"]));
             if (pUIWindow) {
-                bool bIsClickAction = false;
+                OUString sAction((aMap.find("cmd") != aMap.end())? aMap["cmd"]: "");
 
-                if (aMap.find("cmd") != aMap.end()) {
-                    if (aMap["cmd"] == "selected")
-                    {
-                        aMap["POS"] = aMap["data"];
-                        aMap["TEXT"] = aMap["data"];
+                if (sAction == "selected")
+                {
+                    aMap["POS"] = aMap["data"];
+                    aMap["TEXT"] = aMap["data"];
 
-                        pUIWindow->execute(sSelectAction, aMap);
-                    }
-                    else if (aMap["cmd"] == "plus")
-                    {
-                        pUIWindow->execute(sUpAction, aMap);
-                    }
-                    else if (aMap["cmd"] == "minus")
-                    {
-                        pUIWindow->execute(sDownAction, aMap);
-                    }
-                    else if (aMap["cmd"] == "set")
-                    {
-                        aMap["TEXT"] = aMap["data"];
+                    pUIWindow->execute(sSelectAction, aMap);
+                }
+                else if (sAction == "plus")
+                {
+                    pUIWindow->execute(sUpAction, aMap);
+                }
+                else if (sAction == "minus")
+                {
+                    pUIWindow->execute(sDownAction, aMap);
+                }
+                else if (sAction == "set")
+                {
+                    aMap["TEXT"] = aMap["data"];
 
-                        pUIWindow->execute(sClearAction, aMap);
-                        pUIWindow->execute(sTypeAction, aMap);
-                    }
-                    else if (aMap["cmd"] == "value")
-                    {
-                        aMap["VALUE"] = aMap["data"];
-                        pUIWindow->execute(sValue, aMap);
-                    }
-                    else
-                        bIsClickAction = true;
+                    pUIWindow->execute(sClearAction, aMap);
+                    pUIWindow->execute(sTypeAction, aMap);
+                }
+                else if (sAction == "value")
+                {
+                    aMap["VALUE"] = aMap["data"];
+                    pUIWindow->execute(sValue, aMap);
                 }
                 else
-                    bIsClickAction = true;
-
-                if (bIsClickAction)
                     pUIWindow->execute(sClickAction, aMap);
             }
         }