Merge pull request #253 from peppy/verify-ranked-state

Add verify step to confirm correct `ranked` flag
ppy · Apr 15, 2024 · bcb42e0 · bcb42e0
2 parents 5303c55 + 0ce882a
commit bcb42e0
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/...erver.Queues.ScoreStatisticsProcessor/Commands/Maintenance/VerifyImportedScoresCommand.cs b/...erver.Queues.ScoreStatisticsProcessor/Commands/Maintenance/VerifyImportedScoresCommand.cs
@@ -95,6 +95,7 @@ public async Task<int> OnExecuteAsync(CancellationToken cancellationToken)
                     + "`legacy_total_score`, "
                     + "`total_score`, "
                     + "`has_replay`, "
+                    + "s.ranked,"
                     + "s.`rank`, "
                     + "s.`pp`, "
                     + "h.* "
@@ -188,6 +189,13 @@ public async Task<int> OnExecuteAsync(CancellationToken cancellationToken)
                             sqlBuffer.Append($"UPDATE scores SET pp = {importedScore.HighScore.pp.ToString() ?? "NULL"} WHERE id = {importedScore.id};");
                         }
 
+                        if (!check(importedScore.id, "ranked", importedScore.ranked, true))
+                        {
+                            Interlocked.Increment(ref fail);
+                            requiresIndexing = true;
+                            sqlBuffer.Append($"UPDATE scores SET ranked = 1 WHERE id = {importedScore.id};");
+                        }
+
                         if (!check(importedScore.id, "replay", importedScore.has_replay, importedScore.HighScore.replay))
                         {
                             Interlocked.Increment(ref fail);
@@ -300,6 +308,7 @@ public class ComparableScore
             public long? total_score;
             public bool has_replay;
             public ScoreRank rank;
+            public bool ranked;
             public float? pp;
 
             public HighScore? HighScore { get; set; }