Implement level labels (#942)

This implements:
- An enum containing hopefully all labels, aswell as some methods to
(de)serialize them for the game
- The ability for publishers to set up to 5 labels for their levels
- The ability to filter levels by publisher labels in-game
- Migrates review labels to be stored using `List<Label>` instead of
`string`
- Adds some label sanitization
- When level statistics get recalculated, the 5 most recurring labels
among a level's reviews get saved in its statistics to have them show
under levels in-game aswell (the game differenciates between publisher
and non-publisher labels there)
- Deduplicates code used by both tags and labels
- Fixes tags to show up on levels in-game again

Author: jvyden

Refresh.Common/Constants/UgcLimits.cs

@@ -4,6 +4,7 @@ public static class UgcLimits
 {
     // Object limits
     public const int MaximumLevels = 9_999;
+    public const int MaximumLabels = 5;
 
     // String limits
     public const int TitleLimit = 64;

Refresh.Database/Extensions/LevelEnumerableExtensions.cs

@@ -88,6 +88,12 @@ public static IQueryable<GameLevel> FilterByLevelFilterSettings(this IQueryable<
         // Filter out sub levels that weren't published by self
         levels = levels.Where(l => !l.IsSubLevel || l.Publisher == user);
 
+        // Filter by labels
+        if (levelFilterSettings.Labels.Any())
+        {
+           levels = levels.Where(lvl => lvl.Labels.Any(lab => levelFilterSettings.Labels.Contains(lab))); 
+        }
+
         return levels;
     }
 
@@ -146,6 +152,12 @@ public static IEnumerable<GameLevel> FilterByLevelFilterSettings(this IEnumerabl
         // Filter out sub levels that weren't published by self
         levels = levels.Where(l => !l.IsSubLevel || l.Publisher == user);
 
+        // Filter by labels
+        if (levelFilterSettings.Labels.Any())
+        {
+           levels = levels.Where(lvl => lvl.Labels.Any(lab => levelFilterSettings.Labels.Contains(lab))); 
+        }
+
         return levels;
     }
 }

Refresh.Database/GameDatabaseContext.Levels.cs

@@ -38,6 +38,7 @@ public GameLevel AddLevel(ISerializedPublishLevel createInfo, TokenGame game, Ga
             IconHash = createInfo.IconHash,
             LocationX = createInfo.Location.X,
             LocationY = createInfo.Location.Y,
+            Labels = createInfo.FinalPublisherLabels.ToList() ?? [],
             RootResource = createInfo.RootResource,
             IsLocked = createInfo.IsLocked,
             IsCopyable = createInfo.IsCopyable == 1,
@@ -191,6 +192,12 @@ public GameLevel UpdateLevel(ISerializedPublishLevel updateInfo, GameLevel level
         level.SameScreenGame = updateInfo.SameScreenGame;
         level.BackgroundGuid = updateInfo.BackgroundGuid;
 
+        // Only update labels if this level is updated in a game which supports them, to not lose the labels
+        if (game is not TokenGame.LittleBigPlanet1 or TokenGame.LittleBigPlanetPSP)
+        {
+            level.Labels = updateInfo.FinalPublisherLabels.ToList();
+        }
+
         // If we're changing the actual contents of the level, update the game version and update date aswell
         if (updateInfo.RootResource != level.RootResource)
         {

Refresh.Database/GameDatabaseContext.Relations.cs

@@ -460,6 +460,20 @@ public void AddReviewToLevel(GameReview review, GameLevel level)
             review.Publisher.Statistics!.ReviewCount++;
         });
     }
+
+    public void MigrateReviewLabels(IEnumerable<GameReview> reviews)
+    {
+        foreach (GameReview review in reviews)
+        {
+#pragma warning disable CS0618 // LabelsString is obsolete          
+            if (string.IsNullOrWhiteSpace(review.LabelsString)) continue;
+
+            review.Labels = LabelExtensions.FromLbpCommaList(review.LabelsString).ToList();
+#pragma warning restore CS0618
+        }
+
+        this.SaveChanges();
+    }
 
     public void DeleteReviewsPostedByUser(GameUser user)
     {

Refresh.Database/GameDatabaseContext.Statistics.cs

@@ -1,5 +1,6 @@
 using System.Diagnostics;
 using System.Diagnostics.CodeAnalysis;
+using Refresh.Common.Constants;
 using Refresh.Database.Models.Comments;
 using Refresh.Database.Models.Levels;
 using Refresh.Database.Models.Playlists;
@@ -85,7 +86,7 @@ private void WriteEnsuringStatistics(GamePlaylist parent, GamePlaylist child, Ac
     }
 
     #region Levels
-    internal const int LevelStatisticsVersion = 3;
+    internal const int LevelStatisticsVersion = 4;
 
     public IEnumerable<GameLevel> GetLevelsWithStatisticsNeedingUpdates()
     {
@@ -168,6 +169,7 @@ private void RecalculateLevelStatisticsInternal(GameLevel level)
         level.Statistics.PhotoByPublisherCount = this.GetTotalPhotosInLevelByUser(level, level.Publisher);
         level.Statistics.ParentPlaylistCount = this.GetTotalPlaylistsContainingLevel(level);
         this.RecalculateLevelRatingStatisticsInternal(level);
+        this.RecalculateLevelRecurringLabels(level);
 
         level.Statistics.RecalculateAt = null;
         level.Statistics.Version = LevelStatisticsVersion;
@@ -186,6 +188,22 @@ private void RecalculateLevelRatingStatisticsInternal(GameLevel level)
         level.Statistics.Karma = level.Statistics.YayCount - level.Statistics.BooCount;
     }
 
+    private void RecalculateLevelRecurringLabels(GameLevel level)
+    {
+        Debug.Assert(level.Statistics != null);
+
+        // Take the most recurring labels among the reviews for the level
+        level.Statistics.RecurringLabels = this.GameReviews
+            .Where(r => r.LevelId == level.LevelId)
+            .SelectMany(r => r.Labels)
+            .GroupBy(r => r)
+            .Select(g => new { Label = g.Key, Count = g.Count() })
+            .OrderByDescending(g => g.Count)
+            .Take(UgcLimits.MaximumLabels)
+            .Select(g => g.Label)
+            .ToList();
+    }
+
     private void MarkLevelStatisticsDirty(GameLevel level)
     {
         Debug.Assert(this.ChangeTracker.HasChanges(), "should be called in write (no changes detected)");

Refresh.Database/Migrations/20250831164448_AddLevelLabels.cs

@@ -0,0 +1,43 @@
+using Microsoft.EntityFrameworkCore.Infrastructure;
+using Microsoft.EntityFrameworkCore.Migrations;
+
+#nullable disable
+
+namespace Refresh.Database.Migrations
+{
+    /// <inheritdoc />
+    [DbContext(typeof(GameDatabaseContext))]
+    [Migration("20250831164448_AddLevelLabels")]
+    public partial class AddLevelLabels : Migration
+    {
+        /// <inheritdoc />
+        protected override void Up(MigrationBuilder migrationBuilder)
+        {
+            migrationBuilder.AddColumn<byte[]>(
+                name: "RecurringLabels",
+                table: "GameLevelStatistics",
+                type: "smallint[]",
+                nullable: false,
+                defaultValue: new byte[0]);
+
+            migrationBuilder.AddColumn<byte[]>(
+                name: "Labels",
+                table: "GameLevels",
+                type: "smallint[]",
+                nullable: false,
+                defaultValue: new byte[0]);
+        }
+
+        /// <inheritdoc />
+        protected override void Down(MigrationBuilder migrationBuilder)
+        {
+            migrationBuilder.DropColumn(
+                name: "RecurringLabels",
+                table: "GameLevelStatistics");
+
+            migrationBuilder.DropColumn(
+                name: "Labels",
+                table: "GameLevels");
+        }
+    }
+}

Refresh.Database/Migrations/20250902151307_AddReviewLabelList.cs

@@ -0,0 +1,42 @@
+using Microsoft.EntityFrameworkCore.Infrastructure;
+using Microsoft.EntityFrameworkCore.Migrations;
+
+#nullable disable
+
+namespace Refresh.Database.Migrations
+{
+    /// <inheritdoc />
+    [DbContext(typeof(GameDatabaseContext))]
+    [Migration("20250902151307_AddReviewLabelList")]
+    public partial class AddReviewLabelList : Migration
+    {
+        /// <inheritdoc />
+        protected override void Up(MigrationBuilder migrationBuilder)
+        {
+            migrationBuilder.RenameColumn(
+                name: "Labels",
+                table: "GameReviews",
+                newName: "LabelsString");
+
+            migrationBuilder.AddColumn<byte[]>(
+                name: "Labels",
+                table: "GameReviews",
+                type: "smallint[]",
+                nullable: false,
+                defaultValue: new byte[0]);
+        }
+
+        /// <inheritdoc />
+        protected override void Down(MigrationBuilder migrationBuilder)
+        {
+            migrationBuilder.DropColumn(
+                name: "Labels",
+                table: "GameReviews");
+            
+            migrationBuilder.RenameColumn(
+                name: "LabelsString",
+                table: "GameReviews",
+                newName: "Labels");
+        }
+    }
+}

Refresh.Database/Migrations/GameDatabaseContextModelSnapshot.cs

@@ -18,7 +18,7 @@ protected override void BuildModel(ModelBuilder modelBuilder)
         {
 #pragma warning disable 612, 618
             modelBuilder
-                .HasAnnotation("ProductVersion", "9.0.7")
+                .HasAnnotation("ProductVersion", "9.0.8")
                 .HasAnnotation("Relational:MaxIdentifierLength", 63);
 
             NpgsqlModelBuilderExtensions.UseIdentityByDefaultColumns(modelBuilder);
@@ -212,9 +212,14 @@ protected override void BuildModel(ModelBuilder modelBuilder)
                     NpgsqlPropertyBuilderExtensions.UseIdentityByDefaultColumn(b.Property<int>("ReviewId"));
 
                     b.Property<string>("Content")
+                        .IsRequired()
                         .HasColumnType("text");
 
-                    b.Property<string>("Labels")
+                    b.PrimitiveCollection<byte[]>("Labels")
+                        .IsRequired()
+                        .HasColumnType("smallint[]");
+
+                    b.Property<string>("LabelsString")
                         .HasColumnType("text");
 
                     b.Property<int>("LevelId")
@@ -414,6 +419,10 @@ protected override void BuildModel(ModelBuilder modelBuilder)
                     b.Property<bool>("IsSubLevel")
                         .HasColumnType("boolean");
 
+                    b.PrimitiveCollection<byte[]>("Labels")
+                        .IsRequired()
+                        .HasColumnType("smallint[]");
+
                     b.Property<byte>("LevelType")
                         .HasColumnType("smallint");
 
@@ -1294,6 +1303,10 @@ protected override void BuildModel(ModelBuilder modelBuilder)
                     b.Property<DateTimeOffset?>("RecalculateAt")
                         .HasColumnType("timestamp with time zone");
 
+                    b.PrimitiveCollection<byte[]>("RecurringLabels")
+                        .IsRequired()
+                        .HasColumnType("smallint[]");
+
                     b.Property<int>("ReviewCount")
                         .HasColumnType("integer");
 

Refresh.Database/Models/Comments/GameReview.cs

@@ -9,18 +9,22 @@ public partial class GameReview : ISequentialId
 {
     [Key] public int ReviewId { get; set; }
 
-    [Required]
-    public GameLevel Level { get; set;  }
+    [Required, ForeignKey(nameof(LevelId))] public GameLevel Level { get; set; }
+    [Required] public int LevelId { get; set; }
 
     [Required]
     public GameUser Publisher { get; set; }
+
+    #nullable enable
 
     public DateTimeOffset PostedAt { get; set; }
 
-    public string Labels { get; set; }
-    
-    public string Content { get; set; }
-    
+    [Obsolete("Deprecated. This attribute only exists so BackfillReviewLabelsMigration could properly migrate labels at runtime.")]
+    public string? LabelsString { get; set; }
+    [Required] public List<Label> Labels { get; set; } = [];
+
+    public string Content { get; set; } = "";
+
     [NotMapped] public int SequentialId
     {
         get => this.ReviewId;

Refresh.Database/Models/Levels/GameLevel.cs

@@ -1,7 +1,5 @@
 using System.Diagnostics;
-using System.Xml.Serialization;
 using Refresh.Database.Models.Authentication;
-using Refresh.Database.Models.Comments;
 using Refresh.Database.Models.Statistics;
 using Refresh.Database.Models.Users;
 
@@ -22,6 +20,8 @@ public partial class GameLevel : ISequentialId
     public int LocationX { get; set; }
     public int LocationY { get; set; }
 
+    public List<Label> Labels { get; set; } = [];
+
     public string RootResource { get; set; } = string.Empty;
 
     /// <summary>