Merge lp:~widelands-dev/widelands/bug-1714681-multiplayer-tribe-team into lp:widelands

[GunChleoc]

Always update MultiPlayerPlayerGroup when a NoteGameSettings::Action::kUser notification is received

This fixes a bug where players could still edit the tribe and team for a shared slot after moving away from it

[bunnybot (widelandsofficial)]

Continuous integration builds have changed state:

Travis build 3425. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/373389863.
Appveyor build 3230. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1714681_multiplayer_tribe_team-3230.

[Notabilis (notabilis27)]

Diff looks good and testing works as desired.

review: Approve

[GunChleoc (gunchleoc)]

Thanks!

@bunnybot merge

=== modified file 'src/wui/multiplayersetupgroup.cc'
--- src/wui/multiplayersetupgroup.cc	2018-04-07 16:59:00 +0000
+++ src/wui/multiplayersetupgroup.cc	2018-05-01 08:06:08 +0000
@@ -227,19 +227,23 @@
 
 		subscriber_ = Notifications::subscribe<NoteGameSettings>([this](
 		   const NoteGameSettings& note) {
-			const std::vector<PlayerSettings>& players = settings_->settings().players;
+			if (settings_->settings().players.empty()) {
+				// No map/savegame yet
+				return;
+			}
+
 			switch (note.action) {
 			case NoteGameSettings::Action::kMap:
 				// We don't care about map updates, since we receive enough notifications for the
 				// slots.
 				break;
+			case NoteGameSettings::Action::kUser:
+				// We might have moved away from a slot, so we need to update the previous slot too. Since we can't track the slots here, we just update everything.
+				update();
+				break;
 			default:
-				if (players.empty()) {
-					// No map/savegame yet
-					return;
-				}
 				if (id_ == note.position ||
-				    (id_ < players.size() && players.at(id_).state == PlayerSettings::State::kShared)) {
+					(id_ < settings_->settings().players.size() && settings_->settings().players.at(id_).state == PlayerSettings::State::kShared)) {
 					update();
 				}
 			}