Revert "Show the turn dialog before updating the GUI for the incoming…

… player."

This reverts commit aa34fcc.

Fixes #4238

src/play_controller.cpp

@@ -1192,10 +1192,6 @@ void play_controller::play_turn()
 
 		// If a side is empty skip over it.
 		if (!current_team().is_empty()) {
-			// Show the turn dialog now, before the minimap is redrawn and before healing is animated.
-			if(current_team().is_local_human() && current_team().is_proxy_human()) {
-				show_turn_dialog();
-			}
 			init_side_begin();
 			if(gamestate_->init_side_done()) {
 				// This is the case in a reloaded game where the side was initialized before saving the game.

src/play_controller.hpp

@@ -293,7 +293,6 @@ class play_controller : public controller_base, public events::observer, public
 	void fire_start();
 	void start_game();
 	virtual void init_gui();
-	virtual void show_turn_dialog() {}
 	void finish_side_turn();
 	void finish_turn(); //this should not throw an end turn or end level exception
 	bool enemies_visible() const;

src/playmp_controller.cpp

@@ -122,6 +122,7 @@ void playmp_controller::play_human_turn()
 	const std::unique_ptr<countdown_clock> timer(saved_game_.mp_settings().mp_countdown
         ? new countdown_clock(current_team())
         : nullptr);
+	show_turn_dialog();
 	if(undo_stack().can_undo()) {
 		// If we reload a networked mp game we cannot undo moves made before the save
 		// because other players already received them

src/playsingle_controller.cpp

@@ -458,6 +458,8 @@ void playsingle_controller::execute_gotos()
 }
 
 void playsingle_controller::play_human_turn() {
+	show_turn_dialog();
+
 	if (!preferences::disable_auto_moves()) {
 		execute_gotos();
 	}

src/playsingle_controller.hpp

@@ -65,7 +65,7 @@ class playsingle_controller : public play_controller
 protected:
 	virtual void play_side_impl() override;
 	void before_human_turn();
-	virtual void show_turn_dialog() override;
+	void show_turn_dialog();
 	void execute_gotos();
 	virtual void play_human_turn();
 	virtual void after_human_turn();