Correct way to handle post match flow?

[GrimLothar]

Hey! I figure I would stop creating issues and actually post here, since these are questions and not bugs...

I'm trying to automate the progression of the tournament and was wondering if I could get some pointers. I'm starting with a simple 'single_elimination' bracket with no consolation final.

Whenever a match is finished, and assuming they won, I want to let the user know if they either have to go to the next match, or if they won the tournament

Would it be enough to just call manager.find.nextMatches(matchID) and if the response's length is 0 it means they've won the tournament?

Otherwise, send them to the matchID of the first result in the nextMatches array??

Thanks!

[Drarig29]

Hey! For this matter, both issues and discussions are OK 😉

And no, manager.find.nextMatches() doesn't take the result of the match into account. It only gives you, based on the structure of the tournament, what is the next match to be played.

But I'm working on an update now, and I think I'll add an optional participantId?: number parameter to manager.find.nextMatches(). If a participantId is given, it will give you the next matches from the point of view of the given participant.

Would it work for you?

[GrimLothar]

I'm guessing that would work, but it feels like what I have now is working? Let me share some code:

    static UpdateMatchWinner = async (tournamentID:number, matchID:number, winnerID:number):Promise<string> => {
        //First set the winner of the match
        await DB.TournamentManager.update.match({
            id:matchID,
            opponent1:{
                id:winnerID,
                result: 'win'
            }
        })

        //Then check if there are any follow-up matches
        const nextMatches = await DB.TournamentManager.find.nextMatches(matchID)

        if(nextMatches.length == 0){
            //No more matches after this one. Change tournaments status to finished and be done with it
            const success = await TournamentsController.SetState(tournamentID, TournamentState.Completed)
            return 'final'
        } else {
            //Check if next match is ready to start
            const nextMatch = nextMatches[0]
            if(nextMatch.opponent1 && nextMatch.opponent1.id && nextMatch.opponent2 && nextMatch.opponent2.id){
                //Both opponents available. Notify players they are ready to start
                //code to notify players
            } else {
                //both opponents not yet ready. Current player will have to wait.
                return 'wait'
            }
        }
    }

[Drarig29]

It would be almost correct. If you have a consolation final, then you would also have a nextMatches[1] that you currently don't take into account.

And you don't need the nextMatch.opponent1 && nextMatch.opponent1.id && nextMatch.opponent2 && nextMatch.opponent2.id condition, since it exactly equals to nextMatch.status === Status.Ready (cf. Status).

[GrimLothar]

oh nice! I'll use that status.