turniere
/
turniere-backend
mirror of https://github.com/turniere/turniere-backend.git
1
0
Fork 0
Code Issues Releases Wiki Activity

Fix second match winner advancing, even if match is not finished

This commit is contained in:
Daniel Schädler 2019-05-11 17:16:19 +02:00
parent 0573651919
commit 5d5a58c2a3
1 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
app/services/playoff_stage_service.rb
View File

@ -74,7 +74,13 @@ class PlayoffStageService
def self.populate_match_with_winners(match, first_match, second_match) def self.populate_match_with_winners(match, first_match, second_match)
match_scores = match.match_scores.sort_by(&:id) match_scores = match.match_scores.sort_by(&:id)
matches = [first_match, second_match].sort_by(&:position) matches = [first_match, second_match].sort_by(&:position)
winners = matches.map(&:winner) winners = []
if second_match.finished?
winners = matches.map(&:winner)
else
winners = matches.map{ |m|
m == first_match ? m.winner : nil }
end
# depending on the amount of match_scores already present we need to do different things # depending on the amount of match_scores already present we need to do different things
case match_scores.size case match_scores.size
@ -101,8 +107,8 @@ class PlayoffStageService
end end
# If a match is not decided yet, it will return nil as winner. # If a match is not decided yet, it will return nil as winner.
# This is not allowed in Database. The following code replaces MatchScores that contain nil as team with nil. # This is not allowed in Database. The following code filters out MatchScores that contain nil as team.
match_scores.map{ |ms| ms.team.nil? ? nil : ms} match_scores = match_scores.select { |ms| ms.team.present? }
match.match_scores = match_scores match.match_scores = match_scores
end end