Code cleanup
This commit is contained in:
@@ -76,20 +76,12 @@ internal sealed class StateWorkflowService(AppDbContext db)
|
||||
return EndpointHelpers.BadRequestError("Only admins can move backward.");
|
||||
|
||||
var appState = await db.AppState.FirstAsync();
|
||||
var shouldSave = EndpointHelpers.ReconcilePlayerPhase(player, appState.ResultsOpen);
|
||||
_ = EndpointHelpers.ReconcilePlayerPhase(player, appState.ResultsOpen);
|
||||
|
||||
try
|
||||
{
|
||||
player.CurrentPhase = PrevPhase(player.CurrentPhase);
|
||||
player.VotesFinal = false;
|
||||
shouldSave = true;
|
||||
return Results.Ok(new PhaseTransitionResponse(player.CurrentPhase, appState.ResultsOpen));
|
||||
}
|
||||
finally
|
||||
{
|
||||
if (shouldSave)
|
||||
await db.SaveChangesAsync();
|
||||
}
|
||||
return Results.Ok(new PhaseTransitionResponse(player.CurrentPhase, appState.ResultsOpen));
|
||||
}
|
||||
|
||||
private static Phase NextPhase(Phase current) => current switch
|
||||
|
||||
Reference in New Issue
Block a user