'current_player' shouldn't default to 'save_id'

this doesn't make sense, specially because 'current_player' is a string
used to display in dialogs and 'save_id' is an internal id string

https://gna.org/bugs/?22324,

src/team.cpp

@@ -184,9 +184,6 @@ void team::team_info::read(const config &cfg)
 	if(save_id.empty()) {
 		save_id = id;
 	}
-	if (current_player.empty()) {
-		current_player = save_id;
-	}
 
 	income_per_village = cfg["village_gold"].to_int(game_config::village_income);
 	recall_cost = cfg["recall_cost"].to_int(game_config::recall_cost);