Ensure creator username is updated on set updates

Important in cases such as name changes.

This existed in old BSS
(https://github.com/peppy/osu-web-10/blob/2723bd9f63e243c506336494509d1b72fb92872d/www/web/osu-osz2-bmsubmit-upload.php#L222-L224),
not sure why I didn't port that across.

Author: bdach

osu.Server.BeatmapSubmission/BeatmapSubmissionController.cs

@@ -404,7 +404,8 @@ private static async Task reviveBeatmapSet(MySqlConnection connection, uint user
         private async Task<bool> updateBeatmapSetFromArchiveAsync(osu_beatmapset beatmapSet, Stream beatmapStream, MySqlConnection db)
         {
             using var archiveReader = new ZipArchiveReader(beatmapStream);
-            var parser = new BeatmapPackageParser(await db.GetUsernameAsync(beatmapSet.user_id));
+            string username = await db.GetUsernameAsync(beatmapSet.user_id);
+            var parser = new BeatmapPackageParser(username);
             var parseResult = parser.Parse(beatmapSet.beatmapset_id, archiveReader);
 
             checkPackageSize(beatmapStream.Length, parseResult);
@@ -421,6 +422,7 @@ private async Task<bool> updateBeatmapSetFromArchiveAsync(osu_beatmapset beatmap
 
                 if (fileSet.SetEquals(parseResult.Files))
                 {
+                    beatmapSet.creator = username;
                     await db.MarkBeatmapSetUpdatedAsync(beatmapSet, transaction);
                     await transaction.CommitAsync();
                     return false;
@@ -454,6 +456,7 @@ private async Task<bool> updateBeatmapSetFromArchiveAsync(osu_beatmapset beatmap
             if (beatmapIds.Count > 0)
                 throw new InvariantException($"Beatmap package is missing .osu files for beatmaps with IDs: {string.Join(", ", beatmapIds)}", LogLevel.Warning);
 
+            parseResult.BeatmapSet.creator = username;
             await db.UpdateBeatmapSetAsync(parseResult.BeatmapSet, transaction);
 
             await transaction.CommitAsync();

osu.Server.BeatmapSubmission/DatabaseOperationExtensions.cs

@@ -286,7 +286,7 @@ public static Task UpdateBeatmapSetAsync(this MySqlConnection db, osu_beatmapset
                 UPDATE `osu_beatmapsets`
                 SET
                     `artist` = @artist, `artist_unicode` = @artist_unicode, `title` = @title, `title_unicode` = @title_unicode,
-                    `source` = @source, `tags` = @tags, `video` = @video, `storyboard` = @storyboard,
+                    `creator` = @creator, `source` = @source, `tags` = @tags, `video` = @video, `storyboard` = @storyboard,
                     `storyboard_hash` = @storyboard_hash, `bpm` = @bpm, `filename` = @filename, `displaytitle` = @displaytitle,
                     `body_hash` = NULL, `header_hash` = NULL, `osz2_hash` = NULL, `active` = 1, `last_update` = CURRENT_TIMESTAMP
                 WHERE `beatmapset_id` = @beatmapset_id
@@ -296,14 +296,15 @@ public static Task UpdateBeatmapSetAsync(this MySqlConnection db, osu_beatmapset
         }
 
         /// <summary>
-        /// Only bumps the <paramref name="beatmapset"/>'s <c>last_update</c> date without performing any other changes.
+        /// Only updates the <paramref name="beatmapset"/>'s <c>last_update</c> date and <c>creator</c> name without performing any other changes.
         /// Used in scenarios where a submission occurred for a beatmap with no changes.
         /// Bumping the update date is important in such cases, because if it is e.g. an update after reviving the beatmap,
         /// not bumping the update date will cause the beatmap to move to the graveyard again on the next day after the update.
+        /// Updating the creator name is important in cases of name changes, especially ones with username history scrubbed.
         /// </summary>
         public static Task MarkBeatmapSetUpdatedAsync(this MySqlConnection db, osu_beatmapset beatmapset, MySqlTransaction? transaction = null)
         {
-            return db.ExecuteAsync("UPDATE `osu_beatmapsets` SET `last_update` = CURRENT_TIMESTAMP WHERE `beatmapset_id` = @beatmapset_id", beatmapset, transaction);
+            return db.ExecuteAsync("UPDATE `osu_beatmapsets` SET `last_update` = CURRENT_TIMESTAMP, `creator` = @creator WHERE `beatmapset_id` = @beatmapset_id", beatmapset, transaction);
         }
 
         public static async Task<ulong> InsertBeatmapsetFileAsync(this MySqlConnection db, beatmapset_file file, MySqlTransaction? transaction = null)