From 0f0f96a06500daff952fe07e1ccac8f77d56ed3e Mon Sep 17 00:00:00 2001
From: Johan Malm <jgm323@gmail.com>
Date: Sun, 2 Jul 2023 21:20:04 +0100
Subject: [PATCH] action.c: remove duplicate code

---
 src/action.c | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/src/action.c b/src/action.c
index 9aef122b..eef9d3d9 100644
--- a/src/action.c
+++ b/src/action.c
@@ -164,15 +164,12 @@ action_arg_from_xml_node(struct action *action, char *nodename, char *content)
 
 	switch (action->type) {
 	case ACTION_TYPE_EXECUTE:
-		if (!strcmp(argument, "command")) {
-			action_arg_add_str(action, argument, content);
-			goto cleanup;
-		} else if (!strcmp(argument, "execute")) {
-			/*
-			 * <action name="Execute"><execute>foo</execute></action>
-			 * is deprecated, but we support it anyway for backward
-			 * compatibility with old openbox-menu generators
-			 */
+		/*
+		 * <action name="Execute"> with an <execute> child is
+		 * deprecated, but we support it anyway for backward
+		 * compatibility with old openbox-menu generators
+		 */
+		if (!strcmp(argument, "command") || !strcmp(argument, "execute")) {
 			action_arg_add_str(action, "command", content);
 			goto cleanup;
 		}