rixty/bygfoot
1
1
Fork 0
mirror of https://github.com/tstellar/bygfoot.git synced 2025-01-09 13:42:38 +01:00
Code Issues Projects Releases Wiki Activity

Fix bug in query_cup_begins()

Browse Source 
The function incorrectly assumed that cups where all the scheduled games
had been played wre complete, and did not take into account the fact
that games for later rounds may not have been scheduled yet.
This commit is contained in:
Tom Stellard 2021-02-20 16:34:21 -08:00 committed by Tom Stellard
parent 4ad92b9884
commit 0f25c31ee5
1 changed files with 22 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
src/cup.c
View File

@ -1235,19 +1235,34 @@ query_cup_begins(const Cup *cup)
for(i=0;i<cup_round->choose_teams->len;i++)
if(!cup_choose_team_should_generate(&g_array_index(cup_round->choose_teams,CupChooseTeam, i)))
{
const Fixture *last_fixture;
cup_get_choose_team_league_cup(
&g_array_index(cup_round->choose_teams,
CupChooseTeam, i), &league, &cup_temp);
if((cup_temp == NULL &&
if(cup_temp == NULL &&
query_league_active(league) &&
g_array_index(league->fixtures, Fixture,
league->fixtures->len - 1).attendance == -1) ||
(league == NULL &&
((cup_temp->fixtures->len > 0 &&
g_array_index(cup_temp->fixtures, Fixture,
cup_temp->fixtures->len - 1).attendance == -1) ||
cup_temp->fixtures->len == 0)))
league->fixtures->len - 1).attendance == -1)
return FALSE;
/* Handle cups */
if (!cup_temp)
continue;
if (!cup_temp->fixtures->len)
return FALSE;
last_fixture = &g_array_index(cup_temp->fixtures, Fixture,
cup_temp->fixtures->len - 1);
/* attendance == 1 means there are scheduled games that have
* not been played yet. If the last fixture is not for the
* last round of the cup that means there are still games
* that have not been scheduled yet.
*/
if (last_fixture->attendance == -1 ||
last_fixture->round != cup_temp->rounds->len - 1)
return FALSE;
}
}