Correctly set winning team

Gets rid of simple score check which could be equal. Checks game match winner instead (fixes #14)

Author: Bakkes

AutoReplayUploader/AutoReplayUploaderPlugin.cpp

@@ -420,6 +420,20 @@ string AutoReplayUploaderPlugin::SetReplayName(ServerWrapper& server, ReplaySocc
 	// Get Team scores
 	match.Team0Score = soccarReplay.GetTeam0Score();
 	match.Team1Score = soccarReplay.GetTeam1Score();
+	
+	if (auto matchWinner = server.GetMatchWinner(); !matchWinner.IsNull())
+	{
+		match.WinningTeam = matchWinner.GetTeamIndex();
+	}
+	else if (auto gameWinner = server.GetGameWinner(); !gameWinner.IsNull())
+	{
+		match.WinningTeam = gameWinner.GetTeamIndex();
+	}
+	else
+	{
+		match.WinningTeam = match.Team0Score > match.Team1Score ? 0 : 1;
+	}
+
 
 	// Get current Sequence number
 	auto seq = *templateSequence;

Uploader/Match.h

@@ -17,7 +17,7 @@ class Match
 
 	int Team0Score;
 	int Team1Score;
-
+	int WinningTeam;
 	Match();
 	~Match();
 };

Uploader/Player.cpp

@@ -8,7 +8,7 @@ Player::~Player()
 {
 }
 
-bool Player::WonMatch(int team0Score, int team1Score)
+bool Player::WonMatch(int winningTeam)
 {
-	return Team == 0 ? team0Score > team1Score : team1Score > team0Score;
+	return Team == winningTeam;
 }

Uploader/Player.h

@@ -21,5 +21,5 @@ class Player
 	Player();
 	~Player();
 
-	bool WonMatch(int team0Score, int team1Score);
+	bool WonMatch(int winningTeam);
 };

Uploader/Replay.cpp

@@ -63,7 +63,7 @@ string ApplyNameTemplate(string& nameTemplate, Match& match, int* matchIndex)
 	min.insert(min.begin(), 2 - min.length(), '0');
 
 	// Calculate Win/Loss string
-	auto won = match.PrimaryPlayer.WonMatch(match.Team0Score, match.Team1Score);
+	auto won = match.PrimaryPlayer.WonMatch(match.WinningTeam);
 	auto winloss = won ? string("Win") : string("Loss");
 	auto wl = won ? string("W") : string("L");