Merge pull request #319 from peppy/pp-back-to-model

Ensure pp is saved back to model when calculated
ppy · Jan 23, 2025 · b47e5eb · b47e5eb
2 parents ce1032b + 817cfb2
commit b47e5eb
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/osu.Server.Queues.ScoreStatisticsProcessor/Processors/ScorePerformanceProcessor.cs b/osu.Server.Queues.ScoreStatisticsProcessor/Processors/ScorePerformanceProcessor.cs
@@ -165,20 +165,24 @@ public async Task<bool> ProcessScoreAsync(SoloScore score, MySqlConnection conne
 
                 if (score.is_legacy_score && write_legacy_score_pp)
                 {
+                    score.pp = performanceAttributes.Total;
+
                     var helper = LegacyDatabaseHelper.GetRulesetSpecifics(score.ruleset_id);
                     await connection.ExecuteAsync($"UPDATE scores SET pp = @Pp WHERE id = @ScoreId; UPDATE {helper.HighScoreTable} SET pp = @Pp WHERE score_id = @LegacyScoreId", new
                     {
                         ScoreId = score.id,
                         LegacyScoreId = score.legacy_score_id,
-                        Pp = performanceAttributes.Total,
+                        Pp = score.pp
                     }, transaction: transaction);
                 }
                 else
                 {
+                    score.pp = performanceAttributes.Total;
+
                     await connection.ExecuteAsync("UPDATE scores SET pp = @Pp WHERE id = @ScoreId", new
                     {
                         ScoreId = score.id,
-                        Pp = performanceAttributes.Total
+                        Pp = score.pp,
                     }, transaction: transaction);
                 }