From 7fc2b3490cda32e13c407ad59e829d9813c7da95 Mon Sep 17 00:00:00 2001
From: timokoesters <timo@koesters.xyz>
Date: Tue, 23 Apr 2019 11:11:32 +0200
Subject: [PATCH] Answer with `WrongMessage` when trying to RequestState
 `Connected`

Former-commit-id: 8384caae9f19d05f174457ab8efc6046ce30bd8c
---
 server/src/lib.rs | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/server/src/lib.rs b/server/src/lib.rs
index f6b06cb61d..2a4fa2472a 100644
--- a/server/src/lib.rs
+++ b/server/src/lib.rs
@@ -265,7 +265,7 @@ impl Server {
                             ClientState::Connected => disconnect = true, // Default state
                             ClientState::Registered => match client.client_state {
                                 // Use ClientMsg::Register instead
-                                ClientState::Connected => client.error_state(RequestStateError::Impossible),
+                                ClientState::Connected => client.error_state(RequestStateError::WrongMessage),
                                 ClientState::Registered => client.error_state(RequestStateError::Already),
                                 ClientState::Spectator | ClientState::Character
                                     => client.allow_state(ClientState::Registered),
@@ -282,6 +282,7 @@ impl Server {
                         },
                         ClientMsg::Register { player } => match client.client_state {
                             ClientState::Connected => Self::initialize_player(state, entity, client, player),
+                            // Use RequestState instead (No need to send `player` again)
                             _ => client.error_state(RequestStateError::Impossible),
                         },
                         ClientMsg::Character(character) => match client.client_state {