Removed unnecessary unnecessary if in Client::OnInputCommand.

Change name from Server::parseEvent to Server::parseOnInputCommand

include/Engine/Network/Client.h

@@ -81,8 +81,9 @@ private:
     bool hasMappedEntity(EntityID entityID);
     // Events
     EventBroker* m_EventBroker;
+   
     EventRelay<Client, Events::InputCommand> m_EInputCommand;
-    bool OnInputCommand(const Events::InputCommand &e);
+    bool OnInputCommand(const Events::InputCommand& e);
 };
 
 #endif

include/Engine/Network/MessageType.h

@@ -12,6 +12,7 @@ enum class MessageType
 	Message,
 	Snapshot,
 	Event,
+    OnInputCommand
 };
 
 #endif

include/Engine/Network/Server.h

@@ -68,7 +68,7 @@ private:
     void checkForTimeOuts();
     void disconnect(int i);
     void parseMessageType(Packet& packet);
-    void parseEvent(Packet& packet);
+    void parseOnInputCommand(Packet& packet);
     void parseConnect(Packet& packet);
     void parseDisconnect();
     void parseClientPing();

src/Engine/Network/Client.cpp

@@ -254,7 +254,6 @@ int Client::receive(char* data, size_t length)
     if (error) {
         LOG_ERROR("receive: %s", error.message().c_str());
     }
-
     return bytesReceived;
 }
 
@@ -297,18 +296,18 @@ void Client::moveMessageHead(char*& data, size_t& length, size_t stepSize)
 
 bool Client::OnInputCommand(const Events::InputCommand & e)
 {
-    if (e.Command == "Forward" || e.Command == "Right") {
-        Packet packet(MessageType::Event, m_SendPacketID);
+    if (e.Command == "ConnectToServer") { // Connect for now
+        connect();
+        LOG_INFO("Client::OnInputCommand: Command is %s. Value is %f. PlayerID is %i.", e.Command.c_str(), e.Value, e.PlayerID);
+        return true;
+    } else {
+        Packet packet(MessageType::OnInputCommand, m_SendPacketID);
         packet.WriteString(e.Command);
         packet.WritePrimitive(e.PlayerID);
         packet.WritePrimitive(e.Value);
         send(packet);
         LOG_INFO("Client::OnInputCommand: Command is %s. Value is %f. PlayerID is %i.", e.Command.c_str(), e.Value, e.PlayerID);
         return true;
-    } else if (e.Command == "ConnectToServer") { // Connect for now
-        connect();
-        LOG_INFO("Client::OnInputCommand: Command is %s. Value is %f. PlayerID is %i.", e.Command.c_str(), e.Value, e.PlayerID);
-        return true;
     }
     return false;
 }

src/Engine/Network/Server.cpp

@@ -81,8 +81,10 @@ void Server::parseMessageType(Packet& packet)
         parseDisconnect();
         break;
     case MessageType::Event:
-        parseEvent(packet);
         break;
+    case MessageType::OnInputCommand:
+        parseOnInputCommand(packet);
+        break;;
     default:
         break;
     }
@@ -218,7 +220,7 @@ void Server::disconnect(int i)
     m_PlayerDefinitions[i].Name = "";
 }
 
-void Server::parseEvent(Packet& packet)
+void Server::parseOnInputCommand(Packet& packet)
 {
     size_t i;
     for (i = 0; i < MAXCONNECTIONS; i++) {