Merge lp:~widelands-dev/widelands/bug-1821506-new-scenario-crash into lp:widelands

[GunChleoc]

Allow random player tribes in scenarios.

This is for Build 20, because Widelands will crash if somebody creates a scenario and forgets to assign a tribename to a player.

[bunnybot (widelandsofficial)]

Continuous integration builds have changed state:

Travis build 4636. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/510574309.
Appveyor build 4423. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1821506_new_scenario_crash-4423.

[hessenfarmer (stephan-lutz)]

Tested this. Works well. Code LGTM.

@bunnybot merge

review: Approve (test review)

=== modified file 'src/logic/game.cc'
--- src/logic/game.cc	2019-03-09 10:55:24 +0000
+++ src/logic/game.cc	2019-03-24 12:06:57 +0000
@@ -232,7 +232,14 @@
 	loader_ui.step(_("Creating players"));
 	PlayerNumber const nr_players = map().get_nrplayers();
 	iterate_player_numbers(p, nr_players) {
-		add_player(p, 0, map().get_scenario_player_tribe(p), map().get_scenario_player_name(p));
+		// If tribe name is empty, pick a random tribe
+		std::string tribe = map().get_scenario_player_tribe(p);
+		if (tribe.empty()) {
+			log("Setting random tribe for Player %d\n", static_cast<unsigned int>(p));
+			const DescriptionIndex random = std::rand() % tribes().nrtribes();
+			tribe = tribes().get_tribe_descr(random)->name();
+		}
+		add_player(p, 0, tribe, map().get_scenario_player_name(p));
 		get_player(p)->set_ai(map().get_scenario_player_ai(p));
 	}
 	win_condition_displayname_ = "Scenario";

=== modified file 'src/logic/playersmanager.cc'
--- src/logic/playersmanager.cc	2019-02-23 11:00:49 +0000
+++ src/logic/playersmanager.cc	2019-03-24 12:06:57 +0000
@@ -75,8 +75,12 @@
 			number_of_players_--;
 		}
 	}
+	const TribeDescr* player_tribe = egbase_.tribes().get_tribe_descr(egbase_.tribes().tribe_index(tribe));
+	if (player_tribe == nullptr) {
+		throw wexception("Tribe '%s' for player %d '%s' does not exist!", tribe.c_str(), static_cast<unsigned int>(player_number), name.c_str());
+	}
 	p = new Player(egbase_, player_number, initialization_index,
-	               *egbase_.tribes().get_tribe_descr(egbase_.tribes().tribe_index(tribe)), name);
+	               *player_tribe, name);
 	p->set_team_number(team);
 	if (player_number <= UserSettings::highest_playernum()) {
 		number_of_players_++;