From 7de5c17fb679d222c80451ec048904fc365bbfe7 Mon Sep 17 00:00:00 2001
From: Gene Walters <genewalt@amazon.com>
Date: Sun, 24 Oct 2021 23:07:58 -0700
Subject: [PATCH] The editor might not be the connector so make sure to connect
 to the actual MP simulation even if the editor isn't the editor-server
 connect (if editorsv_launch=true then the editor-server will connect to the
 editor)

Signed-off-by: Gene Walters <genewalt@amazon.com>
---
 .../Editor/MultiplayerEditorConnection.cpp    | 26 ++++++++-----------
 .../MultiplayerEditorSystemComponent.cpp      |  2 +-
 2 files changed, 12 insertions(+), 16 deletions(-)

diff --git a/Gems/Multiplayer/Code/Source/Editor/MultiplayerEditorConnection.cpp b/Gems/Multiplayer/Code/Source/Editor/MultiplayerEditorConnection.cpp
index fb0b86af6a..0c226445a4 100644
--- a/Gems/Multiplayer/Code/Source/Editor/MultiplayerEditorConnection.cpp
+++ b/Gems/Multiplayer/Code/Source/Editor/MultiplayerEditorConnection.cpp
@@ -18,7 +18,6 @@
 #include <AzCore/Utils/Utils.h>
 #include <AzCore/Serialization/SerializeContext.h>
 #include <AzCore/Serialization/Utils.h>
-#include <AzFramework/Spawnable/Spawnable.h>
 #include <AzNetworking/ConnectionLayer/IConnection.h>
 #include <AzNetworking/Framework/INetworking.h>
 
@@ -47,7 +46,7 @@ namespace Multiplayer
             ConnectionId editorServerToEditorConnectionId = m_networkEditorInterface->Connect(IpAddress(LocalHost.data(), editorsv_port, ProtocolType::Tcp));
             
             // If there wasn't an Editor waiting for this server to start, then assume this is an editor-server launched by hand... listen and wait for the editor to request a connection
-            if (editorServerToEditorConnectionId == AzNetworking::InvalidConnectionId)
+            if (editorServerToEditorConnectionId == InvalidConnectionId)
             {
                 m_networkEditorInterface->Listen(editorsv_port);
             }
@@ -154,21 +153,18 @@ namespace Multiplayer
         [[maybe_unused]] MultiplayerEditorPackets::EditorServerReady& packet
     )
     {
-        if (connection->GetConnectionRole() == ConnectionRole::Connector)
-        {
-            // Receiving this packet means Editor sync is done, disconnect
-            connection->Disconnect(AzNetworking::DisconnectReason::TerminatedByClient, AzNetworking::TerminationEndpoint::Local);
+        // Receiving this packet means Editor sync is done, disconnect
+        connection->Disconnect(AzNetworking::DisconnectReason::TerminatedByClient, AzNetworking::TerminationEndpoint::Local);
 
-            if (auto console = AZ::Interface<AZ::IConsole>::Get(); console)
+        if (const auto console = AZ::Interface<AZ::IConsole>::Get())
+        {
+            AZ::CVarFixedString remoteAddress;
+            uint16_t remotePort;
+            if (console->GetCvarValue("editorsv_serveraddr", remoteAddress) != AZ::GetValueResult::ConsoleVarNotFound &&
+                console->GetCvarValue("sv_port", remotePort) != AZ::GetValueResult::ConsoleVarNotFound)
             {
-                AZ::CVarFixedString remoteAddress;
-                uint16_t remotePort;
-                if (console->GetCvarValue("editorsv_serveraddr", remoteAddress) != AZ::GetValueResult::ConsoleVarNotFound &&
-                    console->GetCvarValue("sv_port", remotePort) != AZ::GetValueResult::ConsoleVarNotFound)
-                    {
-                        // Connect the Editor to the editor server for Multiplayer simulation
-                        AZ::Interface<IMultiplayer>::Get()->Connect(remoteAddress.c_str(), remotePort);
-                    }
+                // Connect the Editor to the editor server for Multiplayer simulation
+                AZ::Interface<IMultiplayer>::Get()->Connect(remoteAddress.c_str(), remotePort);
             }
         }
         return true;
diff --git a/Gems/Multiplayer/Code/Source/Editor/MultiplayerEditorSystemComponent.cpp b/Gems/Multiplayer/Code/Source/Editor/MultiplayerEditorSystemComponent.cpp
index 6213e8d725..469bfa21aa 100644
--- a/Gems/Multiplayer/Code/Source/Editor/MultiplayerEditorSystemComponent.cpp
+++ b/Gems/Multiplayer/Code/Source/Editor/MultiplayerEditorSystemComponent.cpp
@@ -199,7 +199,7 @@ namespace Multiplayer
                 {
                     AZ_Warning(
                         "MultiplayerEditor", false,
-                        "Launching EditorServer skipped because incompatible cvars. editorsv_launch=true, meaning you want to launch an editor-server on this machine, but the editorsv_serveraddr is %s instead of the local address (127.0.0.1)."
+                        "Launching EditorServer skipped because incompatible cvars. editorsv_launch=true, meaning you want to launch an editor-server on this machine, but the editorsv_serveraddr is %s instead of the local address (127.0.0.1). "
                         "Please either set editorsv_launch=false and keep the remote editor-server, or set editorsv_launch=true and editorsv_serveraddr=127.0.0.1.",
                         remoteAddress.c_str())
                     return;