From 891508cca90fb2334625b9c3021b0a9a3ff9ef68 Mon Sep 17 00:00:00 2001
From: jvyden <jvyden@jvyden.xyz>
Date: Sun, 11 Aug 2024 16:04:47 -0400
Subject: [PATCH 1/2] Track level completions

Closes #616
---
 Refresh.GameServer/Database/GameDatabaseContext.Relations.cs   | 3 +++
 .../Endpoints/Game/DataTypes/Response/GameLevelResponse.cs     | 3 +++
 2 files changed, 6 insertions(+)

diff --git a/Refresh.GameServer/Database/GameDatabaseContext.Relations.cs b/Refresh.GameServer/Database/GameDatabaseContext.Relations.cs
index a29b4a82..3b176b8c 100644
--- a/Refresh.GameServer/Database/GameDatabaseContext.Relations.cs
+++ b/Refresh.GameServer/Database/GameDatabaseContext.Relations.cs
@@ -414,6 +414,9 @@ public int GetTotalPlaysForLevelByUser(GameLevel level, GameUser user) =>
     public int GetUniquePlaysForLevel(GameLevel level, bool includingAuthor = true) =>
         this.UniquePlayLevelRelations.Count(r => r.Level == level && (includingAuthor || r.User != level.Publisher));
 
+    public int GetTotalCompletionsForLevel(GameLevel level) =>
+        this.GameSubmittedScores.Count(s => s.Level == level);
+
     #endregion
 
     #region Comments
diff --git a/Refresh.GameServer/Endpoints/Game/DataTypes/Response/GameLevelResponse.cs b/Refresh.GameServer/Endpoints/Game/DataTypes/Response/GameLevelResponse.cs
index 0a207923..ae74c0dc 100644
--- a/Refresh.GameServer/Endpoints/Game/DataTypes/Response/GameLevelResponse.cs
+++ b/Refresh.GameServer/Endpoints/Game/DataTypes/Response/GameLevelResponse.cs
@@ -46,6 +46,7 @@ public class GameLevelResponse : IDataConvertableFrom<GameLevelResponse, GameLev
     [XmlElement("heartCount")] public required int HeartCount { get; set; }
     
     [XmlElement("playCount")] public required int TotalPlayCount { get; set; }
+    [XmlElement("completionCount")] public required int CompletionCount { get; set; }
     [XmlElement("uniquePlayCount")] public required int UniquePlayCount { get; set; }
 
     [XmlElement("yourDPadRating")] public int YourRating { get; set; }
@@ -116,6 +117,7 @@ public static GameLevelResponse FromHash(string hash, DataContext dataContext)
             MaxPlayers = 4,
             HeartCount = 0,
             TotalPlayCount = 0,
+            CompletionCount = 0,
             UniquePlayCount = 0,
             YayCount = 0,
             BooCount = 0,
@@ -161,6 +163,7 @@ public static GameLevelResponse FromHash(string hash, DataContext dataContext)
             SameScreenGame = old.SameScreenGame,
             HeartCount = dataContext.Database.GetFavouriteCountForLevel(old),
             TotalPlayCount = dataContext.Database.GetTotalPlaysForLevel(old),
+            CompletionCount = dataContext.Database.GetTotalCompletionsForLevel(old),
             UniquePlayCount = dataContext.Database.GetUniquePlaysForLevel(old),
             YayCount = dataContext.Database.GetTotalRatingsForLevel(old, RatingType.Yay),
             BooCount = dataContext.Database.GetTotalRatingsForLevel(old, RatingType.Boo),

From 1ef45cb5e47fb8d3807c4999382394071ae92f38 Mon Sep 17 00:00:00 2001
From: jvyden <jvyden@jvyden.xyz>
Date: Sun, 11 Aug 2024 16:07:56 -0400
Subject: [PATCH 2/2] Implement yourlbp1PlayCount and yourlbp3PlayCount

Closes #456. Hopefully.
---
 .../DataTypes/Response/GameLevelResponse.cs     | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/Refresh.GameServer/Endpoints/Game/DataTypes/Response/GameLevelResponse.cs b/Refresh.GameServer/Endpoints/Game/DataTypes/Response/GameLevelResponse.cs
index ae74c0dc..b0435302 100644
--- a/Refresh.GameServer/Endpoints/Game/DataTypes/Response/GameLevelResponse.cs
+++ b/Refresh.GameServer/Endpoints/Game/DataTypes/Response/GameLevelResponse.cs
@@ -53,9 +53,10 @@ public class GameLevelResponse : IDataConvertableFrom<GameLevelResponse, GameLev
     [XmlElement("thumbsup")] public required int YayCount { get; set; }
     [XmlElement("thumbsdown")] public required int BooCount { get; set; }
     [XmlElement("yourRating")] public int YourStarRating { get; set; }
-    
-    // 1 by default since this will break reviews if set to 0 for GameLevelResponses that do not have extra data being filled in
-    [XmlElement("yourlbp2PlayCount")] public int YourLbp2PlayCount { get; set; } = 1;
+
+    [XmlElement("yourlbp1PlayCount")] public int YourLbp1PlayCount { get; set; }
+    [XmlElement("yourlbp2PlayCount")] public int YourLbp2PlayCount { get; set; }
+    [XmlElement("yourlbp3PlayCount")] public int YourLbp3PlayCount { get; set; }
     
     [XmlArray("customRewards")]
     [XmlArrayItem("customReward")]
@@ -206,10 +207,16 @@ public static GameLevelResponse FromHash(string hash, DataContext dataContext)
         if (dataContext.User != null)
         {
             RatingType? rating = dataContext.Database.GetRatingByUser(old, dataContext.User);
-            
+
             response.YourRating = rating?.ToDPad() ?? (int)RatingType.Neutral;
             response.YourStarRating = rating?.ToLBP1() ?? 0;
-            response.YourLbp2PlayCount = dataContext.Database.GetTotalPlaysForLevelByUser(old, dataContext.User);
+
+            // this is technically invalid, but specifying this for all games ensures they all have the capacity to review if played.
+            // we don't store the game's version in play relations, so this is the best we can do
+            int plays = dataContext.Database.GetTotalPlaysForLevelByUser(old, dataContext.User);
+            response.YourLbp1PlayCount = plays;
+            response.YourLbp2PlayCount = plays;
+            response.YourLbp3PlayCount = plays;
         }
         
         response.PlayerCount = dataContext.Match.GetPlayerCountForLevel(RoomSlotType.Online, response.LevelId);