[Commands] Cleanup #emote Command. (#2535)

* [Commands] Cleanup #emote Command.

- Cleanup messages and logic.
- Allow `^` separator to send multiple messages by name, world, or zone.

* Update emote.cpp

zone/command.cpp

@@ -132,7 +132,7 @@ int command_init(void)
 		command_add("dz", "Manage expeditions and dynamic zone instances", AccountStatus::QuestTroupe, command_dz) ||
 		command_add("dzkickplayers", "Removes all players from current expedition. (/kickplayers alternative for pre-RoF clients)", AccountStatus::Player, command_dzkickplayers) ||
 		command_add("editmassrespawn", "[name-search] [second-value] - Mass (Zone wide) NPC respawn timer editing command", AccountStatus::GMAdmin, command_editmassrespawn) ||
-		command_add("emote", "['name'/'world'/'zone'] [type] [message] - Send an emote message", AccountStatus::QuestTroupe, command_emote) ||
+		command_add("emote", "[Name|World|Zone] [type] [message] - Send an emote message by name, to the world, or to your zone (^ separator allows multiple messages to be sent at once)", AccountStatus::QuestTroupe, command_emote) ||
 		command_add("emotesearch", "[Search Criteria] - Search for NPC Emotes", AccountStatus::QuestTroupe, command_emotesearch) ||
 		command_add("emoteview", "Lists all NPC Emotes", AccountStatus::QuestTroupe, command_emoteview) ||
 		command_add("emptyinventory", "Clears your or your target's entire inventory (Equipment, General, Bank, and Shared Bank)", AccountStatus::GMImpossible, command_emptyinventory) ||

zone/gm_commands/emote.cpp

@@ -5,41 +5,53 @@ extern WorldServer worldserver;
 
 void command_emote(Client *c, const Seperator *sep)
 {
-	if (sep->arg[3][0] == 0) {
+	auto arguments = sep->argnum;
-		c->Message(Chat::White, "Usage: #emote [name | world | zone] type# message");
+	if (arguments < 3 || !sep->IsNumber(2)) {
+		c->Message(Chat::White, "Usage: #emote [Name] [Type] [Message]");
+		c->Message(Chat::White, "Usage: #emote world [Type] [Message]");
+		c->Message(Chat::White, "Usage: #emote zone [Type] [Message]");
+		return;
 	}
-	else {
+
-		if (strcasecmp(sep->arg[1], "zone") == 0) {
+	if (!worldserver.Connected()) {
-			char *newmessage = 0;
+		c->Message(Chat::White, "Error: World server disconnected!");
-			if (strstr(sep->arg[3], "^") == 0) {
+		return;
-				entity_list.Message(0, atoi(sep->arg[2]), sep->argplus[3]);
+	}
-			}
+
-			else {
+	bool is_world = !strcasecmp(sep->arg[1], "world");
-				for (newmessage = strtok((char *) sep->arg[3], "^");
+	bool is_zone = !strcasecmp(sep->arg[1], "zone");
-					newmessage != nullptr;
+
-					newmessage = strtok(nullptr, "^"))
+	const std::string emote_message = sep->argplus[3];
-					entity_list.Message(0, atoi(sep->arg[2]), newmessage);
+	const auto emote_type = std::stoul(sep->arg[2]);
-			}
+
+	if (is_zone) {
+		if (!Strings::Contains(emote_message, "^")) {
+			entity_list.Message(0, emote_type, emote_message.c_str());
+			return;
 		}
-		else if (!worldserver.Connected()) {
+
-			c->Message(Chat::White, "Error: World server disconnected");
+		for (const auto& m : Strings::Split(emote_message, "^")) {
+			entity_list.Message(0, emote_type, m.c_str());
 		}
-		else if (!strcasecmp(sep->arg[1], "world")) {
+	} else {
+		if (!Strings::Contains(emote_message, "^")) {
 			worldserver.SendEmoteMessage(
+				is_world ? 0 : sep->arg[1],
 				0,
-				0,
+				emote_type,
-				atoi(sep->arg[2]),
+				emote_message.c_str()
-				sep->argplus[3]
 			);
+			return;
 		}
-		else {
+
+		for (const auto& m : Strings::Split(emote_message, "^")) {
 			worldserver.SendEmoteMessage(
-				sep->arg[1],
+				is_world ? 0 : sep->arg[1],
 				0,
-				atoi(sep->arg[2]),
+				emote_type,
-				sep->argplus[3]
+				m.c_str()
 			);
-		}
+ 		}
 	}
 }