From 7691ecb281f74ff984af34ba7f40738336bc44d1 Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Wed, 5 Jun 2024 11:10:35 +0200
Subject: [PATCH 01/18] begin local records

---
 EvoSC.sln                                     |  10 +-
 .../Database/Models/DbLocalRecord.cs          |  34 +++++
 .../Repository/LocalRecordRepository.cs       | 117 ++++++++++++++++++
 .../Interfaces/ILocalRecord.cs                |  12 ++
 .../Interfaces/ILocalRecordRepository.cs      |  14 +++
 .../LocalRecordsModule/LocalRecordsModule.cs  |   8 ++
 .../LocalRecordsModule.csproj                 |  26 ++++
 .../LocalRecordsModule/Localization.resx      |  19 +++
 .../202406050955_AddLocalRecordsTable.cs      |  16 +++
 .../Templates/LocalRecordsWidget.mt           |   5 +
 src/Modules/LocalRecordsModule/info.toml      |  11 ++
 .../Database/Models/DbPlayerRecord.cs         |  58 +++++++++
 .../Interfaces/Models/IPlayerRecord.cs        |   5 +-
 13 files changed, 333 insertions(+), 2 deletions(-)
 create mode 100644 src/Modules/LocalRecordsModule/Database/Models/DbLocalRecord.cs
 create mode 100644 src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
 create mode 100644 src/Modules/LocalRecordsModule/Interfaces/ILocalRecord.cs
 create mode 100644 src/Modules/LocalRecordsModule/Interfaces/ILocalRecordRepository.cs
 create mode 100644 src/Modules/LocalRecordsModule/LocalRecordsModule.cs
 create mode 100644 src/Modules/LocalRecordsModule/LocalRecordsModule.csproj
 create mode 100644 src/Modules/LocalRecordsModule/Localization.resx
 create mode 100644 src/Modules/LocalRecordsModule/Migrations/202406050955_AddLocalRecordsTable.cs
 create mode 100644 src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
 create mode 100644 src/Modules/LocalRecordsModule/info.toml

diff --git a/EvoSC.sln b/EvoSC.sln
index 6d42c6c99..c64ed88ef 100644
--- a/EvoSC.sln
+++ b/EvoSC.sln
@@ -112,6 +112,9 @@ Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "MapListModule", "src\Module
 EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "MapListModule.Tests", "tests\Modules\MapListModule.Tests\MapListModule.Tests.csproj", "{098D1F9B-054D-4158-BB6C-AC908C4595F6}"
 EndProject
+Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "LocalRecordsModule", "src/Modules/LocalRecordsModule/LocalRecordsModule.csproj", "{1D8DBFC8-EC21-4DDA-9D88-DE7CA04C8449}"
+EndProject
+
 
 
 
@@ -330,6 +333,10 @@ Global
 		{098D1F9B-054D-4158-BB6C-AC908C4595F6}.Debug|Any CPU.Build.0 = Debug|Any CPU
 		{098D1F9B-054D-4158-BB6C-AC908C4595F6}.Release|Any CPU.ActiveCfg = Release|Any CPU
 		{098D1F9B-054D-4158-BB6C-AC908C4595F6}.Release|Any CPU.Build.0 = Release|Any CPU
+		{1D8DBFC8-EC21-4DDA-9D88-DE7CA04C8449}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{1D8DBFC8-EC21-4DDA-9D88-DE7CA04C8449}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{1D8DBFC8-EC21-4DDA-9D88-DE7CA04C8449}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{1D8DBFC8-EC21-4DDA-9D88-DE7CA04C8449}.Release|Any CPU.Build.0 = Release|Any CPU
 	EndGlobalSection
 	GlobalSection(SolutionProperties) = preSolution
 		HideSolutionNode = FALSE
@@ -382,5 +389,6 @@ Global
 		{2D2D3AC6-C8BD-4615-BCC7-9A64007DB762} = {6D75D6A2-6ECD-4DE4-96C5-CAD7D134407A}
 		{AB316816-F301-4693-86E8-C31E78F53A59} = {DC47658A-F421-4BA4-B617-090A7DFB3900}
 		{098D1F9B-054D-4158-BB6C-AC908C4595F6} = {6D75D6A2-6ECD-4DE4-96C5-CAD7D134407A}
-	EndGlobalSection
+		{1D8DBFC8-EC21-4DDA-9D88-DE7CA04C8449} = {DC47658A-F421-4BA4-B617-090A7DFB3900}	
+EndGlobalSection
 EndGlobal
diff --git a/src/Modules/LocalRecordsModule/Database/Models/DbLocalRecord.cs b/src/Modules/LocalRecordsModule/Database/Models/DbLocalRecord.cs
new file mode 100644
index 000000000..b5946b412
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Database/Models/DbLocalRecord.cs
@@ -0,0 +1,34 @@
+using EvoSC.Common.Database.Models.Maps;
+using EvoSC.Common.Database.Models.Player;
+using EvoSC.Common.Interfaces.Models;
+using EvoSC.Modules.Official.LocalRecordsModule.Interfaces;
+using EvoSC.Modules.Official.PlayerRecords.Database.Models;
+using EvoSC.Modules.Official.PlayerRecords.Interfaces.Models;
+using LinqToDB.Mapping;
+
+namespace EvoSC.Modules.Official.LocalRecordsModule.Database.Models;
+
+public class DbLocalRecord : ILocalRecord
+{
+    [PrimaryKey, Identity]
+    public long Id { get; set; }
+    
+    [Column]
+    public long MapId { get; set; }
+    
+    [Column]
+    public long RecordId { get; set; }
+    
+    [Column]
+    public int Position { get; set; }
+
+    public IMap Map => DbMap;
+    
+    public IPlayerRecord Record => DbRecord;
+
+    [Association(ThisKey = nameof(MapId), OtherKey = nameof(DbMap.Id))]
+    public DbMap DbMap { get; set; }
+    
+    [Association(ThisKey = nameof(RecordId), OtherKey = nameof(DbPlayerRecord.Id))]
+    public DbPlayerRecord DbRecord { get; set; }
+}
diff --git a/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs b/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
new file mode 100644
index 000000000..154bb54d3
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
@@ -0,0 +1,117 @@
+using EvoSC.Common.Database.Models.Maps;
+using EvoSC.Common.Database.Models.Player;
+using EvoSC.Common.Database.Repository;
+using EvoSC.Common.Interfaces.Database;
+using EvoSC.Common.Interfaces.Models;
+using EvoSC.Modules.Official.LocalRecordsModule.Database.Models;
+using EvoSC.Modules.Official.LocalRecordsModule.Interfaces;
+using EvoSC.Modules.Official.PlayerRecords.Database.Models;
+using EvoSC.Modules.Official.PlayerRecords.Interfaces.Models;
+using LinqToDB;
+
+namespace EvoSC.Modules.Official.LocalRecordsModule.Database.Repository;
+
+public class LocalRecordRepository : DbRepository, ILocalRecordRepository
+{
+    protected LocalRecordRepository(IDbConnectionFactory dbConnFactory) : base(dbConnFactory)
+    {
+    }
+
+    public async Task<IEnumerable<DbLocalRecord>> GetLocalRecordsOfMapByIdAsync(long mapId) =>
+        await NewLoadAll()
+            .Where(r => r.Map.Id == mapId)
+            .ToArrayAsync();
+
+    public async Task<DbLocalRecord> AddOrUpdateRecordAsync(IMap map, IPlayerRecord record)
+    {
+        // check if old record is better or equal, and dont bother updating if so
+        var oldRecord = await GetRecordOfPlayerInMapAsync(record.Player, map);
+
+        if (oldRecord != null && oldRecord.Record.CompareTo(record) <= 0)
+        {
+            return oldRecord;
+        }
+
+        var localRecord = new DbLocalRecord
+        {
+            MapId = map.Id,
+            RecordId = record.Id,
+            Position = 0,
+            DbMap = new DbMap(map),
+            DbRecord = new DbPlayerRecord(record)
+        };
+        
+        var transaction = await Database.BeginTransactionAsync();
+
+        try
+        {
+            // remove old record and add the new one
+            if (oldRecord != null)
+            {
+                await Database.DeleteAsync(oldRecord);
+            }
+            
+            var id = await Database.InsertWithIdentityAsync(localRecord);
+            localRecord.Id = Convert.ToInt64(id);
+        }
+        catch (Exception ex)
+        {
+            await transaction.RollbackTransactionAsync();
+            throw;
+        }
+        
+        await RecalculatePositionsOfMapAsync(map);
+        return localRecord;
+    }
+
+    public async Task RecalculatePositionsOfMapAsync(IMap map)
+    {
+        var locals = await GetLocalRecordsOfMapByIdAsync(map.Id);
+        var sorted = locals.OrderBy(r => r.Record.Score).ToArray();
+        var updated = new List<DbLocalRecord>();
+        
+        for (var i = 0; i < sorted.Length; i++)
+        {
+            var newPos = i + 1;
+            
+            // don't update records that dont need it
+            if (sorted[i].Position == newPos)
+            {
+                continue;
+            }
+            
+            sorted[i].Position = newPos;
+            updated.Add(sorted[i]);
+        }
+
+        var transaction = await Database.BeginTransactionAsync();
+
+        try
+        {
+            foreach (var record in updated)
+            {
+                await Database.UpdateAsync(record);
+            }
+            
+            await transaction.CommitTransactionAsync();
+        }
+        catch (Exception ex)
+        {
+            await transaction.RollbackTransactionAsync();
+        }
+    }
+
+    public async Task<IEnumerable<DbLocalRecord>> GetRecordsByPlayerAsync(IPlayer player) =>
+        await NewLoadAll()
+            .Where(r => r.Record.Player.Id == player.Id)
+            .ToArrayAsync();
+
+    public async Task<DbLocalRecord?> GetRecordOfPlayerInMapAsync(IPlayer player, IMap map) =>
+        await NewLoadAll()
+            .FirstOrDefaultAsync(r => r.Record.Player.Id == player.Id);
+
+    private ILoadWithQueryable<DbLocalRecord, IPlayer> NewLoadAll() => Table<DbLocalRecord>()
+        .LoadWith(r => r.Map)
+        .LoadWith(r => r.Record)
+        .ThenLoad(r => r.Player);
+}
diff --git a/src/Modules/LocalRecordsModule/Interfaces/ILocalRecord.cs b/src/Modules/LocalRecordsModule/Interfaces/ILocalRecord.cs
new file mode 100644
index 000000000..1742a30af
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Interfaces/ILocalRecord.cs
@@ -0,0 +1,12 @@
+using EvoSC.Common.Interfaces.Models;
+using EvoSC.Modules.Official.PlayerRecords.Interfaces.Models;
+
+namespace EvoSC.Modules.Official.LocalRecordsModule.Interfaces;
+
+public interface ILocalRecord
+{
+    public long Id { get; }
+    public int Position { get; }
+    public IMap Map { get; }
+    public IPlayerRecord Record { get; }
+}
diff --git a/src/Modules/LocalRecordsModule/Interfaces/ILocalRecordRepository.cs b/src/Modules/LocalRecordsModule/Interfaces/ILocalRecordRepository.cs
new file mode 100644
index 000000000..74d32dbe2
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Interfaces/ILocalRecordRepository.cs
@@ -0,0 +1,14 @@
+using EvoSC.Common.Interfaces.Models;
+using EvoSC.Modules.Official.LocalRecordsModule.Database.Models;
+using EvoSC.Modules.Official.PlayerRecords.Interfaces.Models;
+
+namespace EvoSC.Modules.Official.LocalRecordsModule.Interfaces;
+
+public interface ILocalRecordRepository
+{
+    public Task<IEnumerable<DbLocalRecord>> GetLocalRecordsOfMapByIdAsync(long mapId);
+    public Task<DbLocalRecord> AddOrUpdateRecordAsync(IMap map, IPlayerRecord record);
+    public Task RecalculatePositionsOfMapAsync(IMap map);
+    public Task<IEnumerable<DbLocalRecord>> GetRecordsByPlayerAsync(IPlayer player);
+    public Task<DbLocalRecord?> GetRecordOfPlayerInMapAsync(IPlayer player, IMap map);
+}
diff --git a/src/Modules/LocalRecordsModule/LocalRecordsModule.cs b/src/Modules/LocalRecordsModule/LocalRecordsModule.cs
new file mode 100644
index 000000000..af409a695
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/LocalRecordsModule.cs
@@ -0,0 +1,8 @@
+using EvoSC.Modules.Attributes;
+
+namespace EvoSC.Modules.Official.LocalRecordsModule;
+
+[Module(IsInternal = true)]
+public class LocalRecordsModule : EvoScModule
+{
+}
diff --git a/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj b/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj
new file mode 100644
index 000000000..d9b3074db
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj
@@ -0,0 +1,26 @@
+<Project Sdk="Microsoft.NET.Sdk">
+  <PropertyGroup>
+    <TargetFramework>net8.0</TargetFramework>
+    <ImplicitUsings>enable</ImplicitUsings>
+    <Nullable>enable</Nullable>
+    <RootNamespace>EvoSC.Modules.Official.LocalRecordsModule</RootNamespace>
+    <GenerateAssemblyInfo>false</GenerateAssemblyInfo>
+    <AssemblyName>LocalRecordsModule</AssemblyName>
+    <Title>Local Records</Title>
+    <Description>Determine and display top records of a map from locally saved records.</Description>
+    <Version>1.0.0</Version>
+    <Authors>Evo</Authors>
+  </PropertyGroup>
+  <ItemGroup>
+    <EmbeddedResource Include="Templates\**\*" />
+  </ItemGroup>
+  <ItemGroup>
+    <EmbeddedResource Update="Localization.resx" />
+  </ItemGroup>
+  <ItemGroup>
+        <ProjectReference Include="../../EvoSC.Modules.SourceGeneration/EvoSC.Modules.SourceGeneration.csproj" OutputItemType="Analyzer" ReferenceOutputAssembly="false" />
+        <ProjectReference Include="../../EvoSC.Modules/EvoSC.Modules.csproj" Private="true" />
+        <ProjectReference Include="..\PlayerRecords\PlayerRecords.csproj" />
+  </ItemGroup>
+  
+  </Project>
diff --git a/src/Modules/LocalRecordsModule/Localization.resx b/src/Modules/LocalRecordsModule/Localization.resx
new file mode 100644
index 000000000..10e3157c7
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Localization.resx
@@ -0,0 +1,19 @@
+<?xml version="1.0" encoding="utf-8"?>
+<root>
+  <xsd:schema xmlns="" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:msdata="urn:schemas-microsoft-com:xml-msdata" id="root">
+    <xsd:element name="root" msdata:IsDataSet="true">
+    </xsd:element>
+  </xsd:schema>
+  <resheader name="resmimetype">
+    <value>text/microsoft-resx</value>
+  </resheader>
+  <resheader name="version">
+    <value>1.3</value>
+  </resheader>
+  <resheader name="reader">
+    <value>System.Resources.ResXResourceReader, System.Windows.Forms, Version=2.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089</value>
+  </resheader>
+  <resheader name="writer">
+    <value>System.Resources.ResXResourceWriter, System.Windows.Forms, Version=2.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089</value>
+  </resheader>
+</root>
diff --git a/src/Modules/LocalRecordsModule/Migrations/202406050955_AddLocalRecordsTable.cs b/src/Modules/LocalRecordsModule/Migrations/202406050955_AddLocalRecordsTable.cs
new file mode 100644
index 000000000..f22f5e511
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Migrations/202406050955_AddLocalRecordsTable.cs
@@ -0,0 +1,16 @@
+using FluentMigrator;
+
+namespace EvoSC.Modules.Official.LocalRecordsModule;
+
+[Tags("Production")]
+[Migration(1717574116)]
+public class AddLocalRecordsTable : Migration
+{
+    public override void Up()
+    {
+    }
+
+    public override void Down()
+    {
+    }
+}
diff --git a/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt b/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
new file mode 100644
index 000000000..2ba8d97ee
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
@@ -0,0 +1,5 @@
+<component>
+  <template>
+    
+  </template>
+</component>
diff --git a/src/Modules/LocalRecordsModule/info.toml b/src/Modules/LocalRecordsModule/info.toml
new file mode 100644
index 000000000..8c4d95a58
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/info.toml
@@ -0,0 +1,11 @@
+[info]
+# A unique name for this module, this is used as a identifier
+name = "LocalRecordsModule"
+# The title of the module
+title = "Local Records"
+# A short description of what the module is and does
+summary = "Determine and display top records of a map from locally saved records."
+# The current version of this module, using SEMVER
+version = "1.0.0"
+# The name of the author that created this module
+author = "Evo"
diff --git a/src/Modules/PlayerRecords/Database/Models/DbPlayerRecord.cs b/src/Modules/PlayerRecords/Database/Models/DbPlayerRecord.cs
index 7751b9051..c895bf8de 100644
--- a/src/Modules/PlayerRecords/Database/Models/DbPlayerRecord.cs
+++ b/src/Modules/PlayerRecords/Database/Models/DbPlayerRecord.cs
@@ -42,4 +42,62 @@ public class DbPlayerRecord : IPlayerRecord
     public DbMap DbMap;
 
     public IMap? Map => DbMap;
+
+    public DbPlayerRecord(){}
+
+    public DbPlayerRecord(IPlayerRecord? record)
+    {
+        if (record == null)
+        {
+            return;
+        }
+
+        Id = record.Id;
+        PlayerId = record.Player.Id;
+        MapId = record.Map.Id;
+        Score = record.Score;
+        RecordType = record.RecordType;
+        Checkpoints = record.Checkpoints;
+        CreatedAt = record.CreatedAt;
+        UpdatedAt = record.UpdatedAt;
+        DbPlayer = new DbPlayer(record.Player);
+        DbMap = new DbMap(record.Map);
+    }
+
+    public int CompareTo(IPlayerRecord? other)
+    {
+        if (other == null)
+        {
+            // better than "nothing"
+            return 1;
+        }
+        
+        if (RecordType != other.RecordType)
+        {
+            throw new InvalidOperationException("Cannot compare records of different types");
+        }
+
+        return RecordType switch
+        {
+            PlayerRecordType.Points => ComparePoints(),
+            PlayerRecordType.Time => CompareTime(),
+            _ => CompareTime()
+        };
+
+        // Compare time record type, lower is better
+        int CompareTime() => Score switch
+        {
+            _ when Score < other.Score => 1,
+            _ when Score > other.Score => 1,
+            _ => 0
+        };
+
+        // compare points record type, higher is better
+        int ComparePoints() => Score switch
+        {
+            _ when Score > other.Score => 1,
+            _ when Score < other.Score => 1,
+            _ => 0
+        };
+    }
 }
diff --git a/src/Modules/PlayerRecords/Interfaces/Models/IPlayerRecord.cs b/src/Modules/PlayerRecords/Interfaces/Models/IPlayerRecord.cs
index 90654397d..d1da8180f 100644
--- a/src/Modules/PlayerRecords/Interfaces/Models/IPlayerRecord.cs
+++ b/src/Modules/PlayerRecords/Interfaces/Models/IPlayerRecord.cs
@@ -2,11 +2,14 @@
 
 namespace EvoSC.Modules.Official.PlayerRecords.Interfaces.Models;
 
-public interface IPlayerRecord
+public interface IPlayerRecord : IComparable<IPlayerRecord>
 {
+    public long Id { get; }
     public IPlayer Player { get; }
     public IMap Map { get; }
     public int Score { get; }
     public PlayerRecordType RecordType { get; }
     public string Checkpoints { get; }
+    public DateTime CreatedAt { get; }
+    public DateTime UpdatedAt { get; }
 }

From a02f4869daa7052128c8a84584256e6273cd89f0 Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Wed, 5 Jun 2024 12:56:59 +0200
Subject: [PATCH 02/18] Add dynamic data changes to persistent manialinks.

---
 .../Interfaces/IManialinkManager.cs           | 22 ++++++
 .../Interfaces/Models/IPersistentManialink.cs |  9 +++
 .../Models/PersistentManialinkType.cs         |  7 ++
 src/EvoSC.Manialinks/ManialinkManager.cs      | 73 +++++++++++++++++--
 .../Models/PersistentManialink.cs             | 11 +++
 src/EvoSC/EvoSC.csproj                        |  1 +
 src/EvoSC/InternalModules.cs                  |  4 +-
 .../Controllers/ShowWidgetController.cs       | 18 +++++
 .../Database/Models/DbLocalRecord.cs          |  3 +
 .../Repository/LocalRecordRepository.cs       | 45 +++++++++++-
 .../{ => Database}/ILocalRecordRepository.cs  |  4 +-
 .../Services/ILocalRecordsService.cs          |  7 ++
 .../LocalRecordsModule/LocalRecordsModule.cs  | 14 +++-
 .../LocalRecordsModule.csproj                 |  3 +
 .../202406050955_AddLocalRecordsTable.cs      |  7 ++
 src/Modules/LocalRecordsModule/info.toml      |  3 +
 .../Repository/PlayerRecordsRepository.cs     |  5 ++
 .../Interfaces/IPlayerRecordsRepository.cs    |  4 +
 18 files changed, 228 insertions(+), 12 deletions(-)
 create mode 100644 src/EvoSC.Manialinks/Interfaces/Models/IPersistentManialink.cs
 create mode 100644 src/EvoSC.Manialinks/Interfaces/Models/PersistentManialinkType.cs
 create mode 100644 src/EvoSC.Manialinks/Models/PersistentManialink.cs
 create mode 100644 src/Modules/LocalRecordsModule/Controllers/ShowWidgetController.cs
 rename src/Modules/LocalRecordsModule/Interfaces/{ => Database}/ILocalRecordRepository.cs (75%)
 create mode 100644 src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs

diff --git a/src/EvoSC.Manialinks/Interfaces/IManialinkManager.cs b/src/EvoSC.Manialinks/Interfaces/IManialinkManager.cs
index fc6a33586..39b91fb77 100644
--- a/src/EvoSC.Manialinks/Interfaces/IManialinkManager.cs
+++ b/src/EvoSC.Manialinks/Interfaces/IManialinkManager.cs
@@ -102,6 +102,28 @@ public interface IManialinkManager
     /// <returns></returns>
     public Task SendPersistentManialinkAsync(string name);
 
+    /// <summary>
+    /// Send a manialink to all players which persists even if a player re-connects.
+    /// It will also automatically show for new players.
+    ///
+    /// This method allows for dynamic data updates by a callback method.
+    /// </summary>
+    /// <param name="name">Name of the template to show.</param>
+    /// <param name="setupData">Method that returns data to be sent.</param>
+    /// <returns></returns>
+    public Task SendPersistentManialinkAsync(string name, Func<Task<dynamic>> setupData);
+    
+    /// <summary>
+    /// Send a manialink to all players which persists even if a player re-connects.
+    /// It will also automatically show for new players.
+    ///
+    /// This method allows for dynamic data updates by a callback method.
+    /// </summary>
+    /// <param name="name"></param>
+    /// <param name="setupData">Method that returns data to be sent.</param>
+    /// <returns></returns>
+    public Task SendPersistentManialinkAsync(string name, Func<Task<IDictionary<string, object?>>> setupData);
+    
     /// <summary>
     /// Render a template and send it to a specific player.
     /// </summary>
diff --git a/src/EvoSC.Manialinks/Interfaces/Models/IPersistentManialink.cs b/src/EvoSC.Manialinks/Interfaces/Models/IPersistentManialink.cs
new file mode 100644
index 000000000..41291e4d8
--- /dev/null
+++ b/src/EvoSC.Manialinks/Interfaces/Models/IPersistentManialink.cs
@@ -0,0 +1,9 @@
+namespace EvoSC.Manialinks.Interfaces.Models;
+
+public interface IPersistentManialink
+{
+    public string Name { get; }
+    public PersistentManialinkType Type { get; }
+    public string CompiledOutput { get; }
+    public Func<Task<IDictionary<string, object?>>>? DynamicDataCallbackAsync { get; }
+}
diff --git a/src/EvoSC.Manialinks/Interfaces/Models/PersistentManialinkType.cs b/src/EvoSC.Manialinks/Interfaces/Models/PersistentManialinkType.cs
new file mode 100644
index 000000000..fa6e3deb6
--- /dev/null
+++ b/src/EvoSC.Manialinks/Interfaces/Models/PersistentManialinkType.cs
@@ -0,0 +1,7 @@
+namespace EvoSC.Manialinks.Interfaces.Models;
+
+public enum PersistentManialinkType
+{
+    Static,
+    Dynamic
+}
diff --git a/src/EvoSC.Manialinks/ManialinkManager.cs b/src/EvoSC.Manialinks/ManialinkManager.cs
index ccb6c6d97..e84da3e90 100644
--- a/src/EvoSC.Manialinks/ManialinkManager.cs
+++ b/src/EvoSC.Manialinks/ManialinkManager.cs
@@ -1,4 +1,6 @@
 using System.Collections.Concurrent;
+using System.Diagnostics;
+using System.Dynamic;
 using System.Reflection;
 using System.Text;
 using EvoSC.Common.Events;
@@ -33,7 +35,7 @@ public class ManialinkManager : IManialinkManager
     private readonly ManiaTemplateEngine _engine = new();
     private readonly Dictionary<string, IManialinkTemplateInfo> _templates = new();
     private readonly Dictionary<string, IManiaScriptInfo> _scripts = new();
-    private readonly ConcurrentDictionary<string, string> _persistentManialinks = new();
+    private readonly ConcurrentDictionary<string, IPersistentManialink> _persistentManialinks = new();
 
     private static IEnumerable<Assembly> s_defaultAssemblies = new[]
     {
@@ -207,7 +209,12 @@ public async Task SendPersistentManialinkAsync(string name, IDictionary<string,
         name = GetEffectiveName(name);
         var manialinkOutput = await PrepareAndRenderAsync(name, data);
         await _server.Remote.SendDisplayManialinkPageAsync(manialinkOutput, 0, false);
-        _persistentManialinks[name] = manialinkOutput;
+        _persistentManialinks[name] = new PersistentManialink
+        {
+            Name = name,
+            Type = PersistentManialinkType.Static,
+            CompiledOutput = manialinkOutput 
+        };
     }
 
     public async Task SendPersistentManialinkAsync(string name, dynamic data)
@@ -215,11 +222,39 @@ public async Task SendPersistentManialinkAsync(string name, dynamic data)
         name = GetEffectiveName(name);
         var manialinkOutput = await PrepareAndRenderAsync(name, data);
         await _server.Remote.SendDisplayManialinkPageAsync(manialinkOutput, 0, false);
-        _persistentManialinks[name] = manialinkOutput;
+        _persistentManialinks[name] = new PersistentManialink
+        {
+            Name = name,
+            Type = PersistentManialinkType.Static,
+            CompiledOutput = manialinkOutput 
+        };
     }
 
     public Task SendPersistentManialinkAsync(string name) => SendPersistentManialinkAsync(name, new { });
 
+    public Task SendPersistentManialinkAsync(string name, Func<Task<dynamic>> setupData) =>
+        SendPersistentManialinkAsync(name, async () =>
+        {
+            var data = await setupData();
+            return (IDictionary<string, object?>)data;
+        });
+
+    public async Task SendPersistentManialinkAsync(string name, Func<Task<IDictionary<string, object?>>> setupData)
+    {
+        name = GetEffectiveName(name);
+        
+        _persistentManialinks[name] = new PersistentManialink
+        {
+            Name = name,
+            Type = PersistentManialinkType.Static,
+            DynamicDataCallbackAsync = setupData
+        };
+
+        var data = await setupData();
+        var manialinkOutput = await PrepareAndRenderAsync(name, data);
+        await _server.Remote.SendDisplayManialinkPageAsync(manialinkOutput, 0, false);
+    }
+
     public async Task SendManialinkAsync(IPlayer player, string name, IDictionary<string, object?> data)
     {
         name = GetEffectiveName(name);
@@ -330,9 +365,37 @@ private async Task HandlePlayerConnectAsync(object sender, PlayerConnectGbxEvent
     {
         try
         {
-            foreach (var (_, output) in _persistentManialinks)
+            foreach (var (_, manialink) in _persistentManialinks)
             {
-                await _server.Remote.SendDisplayManialinkPageToLoginAsync(e.Login, output, 0, false);
+                string? output = null;
+                
+                switch (manialink.Type)
+                {
+                    case PersistentManialinkType.Static:
+                        output = manialink.CompiledOutput;
+                        break;
+                    case PersistentManialinkType.Dynamic:
+                        {
+                            var data = await manialink.DynamicDataCallbackAsync?.Invoke();
+                            output = await PrepareAndRenderAsync(manialink.Name, data);
+                        }
+                        break;
+                }
+
+                if (output == null)
+                {
+                    _logger.LogWarning("Failed to get output of persistent ({Type}) manialink: {Name}",
+                        manialink.Type switch
+                        {
+                            PersistentManialinkType.Dynamic => "Dynamic",
+                            PersistentManialinkType.Static => "Static",
+                            _ => "Unknown",
+                        },
+                        manialink.Name);
+                    continue;
+                }
+                
+                await _server.Remote.SendDisplayManialinkPageToLoginAsync(e.Login, manialink.CompiledOutput, 0, false);
             }
         }
         catch (Exception ex)
diff --git a/src/EvoSC.Manialinks/Models/PersistentManialink.cs b/src/EvoSC.Manialinks/Models/PersistentManialink.cs
new file mode 100644
index 000000000..8f11b2b91
--- /dev/null
+++ b/src/EvoSC.Manialinks/Models/PersistentManialink.cs
@@ -0,0 +1,11 @@
+using EvoSC.Manialinks.Interfaces.Models;
+
+namespace EvoSC.Manialinks.Models;
+
+public class PersistentManialink : IPersistentManialink
+{
+    public required string Name { get; init; }
+    public required PersistentManialinkType Type { get; init; }
+    public string CompiledOutput { get; init; }
+    public Func<Task<IDictionary<string, object?>>>? DynamicDataCallbackAsync { get; init; }
+}
diff --git a/src/EvoSC/EvoSC.csproj b/src/EvoSC/EvoSC.csproj
index 08d603228..25c2b72fc 100644
--- a/src/EvoSC/EvoSC.csproj
+++ b/src/EvoSC/EvoSC.csproj
@@ -26,6 +26,7 @@
         <ProjectReference Include="..\EvoSC.Modules\EvoSC.Modules.csproj" />
         <ProjectReference Include="..\Modules\ExampleModule\ExampleModule.csproj" />
         <ProjectReference Include="..\Modules\LiveRankingModule\LiveRankingModule.csproj" />
+        <ProjectReference Include="..\Modules\LocalRecordsModule\LocalRecordsModule.csproj" />
         <ProjectReference Include="..\Modules\MapListModule\MapListModule.csproj" />
         <ProjectReference Include="..\Modules\MapQueueModule\MapQueueModule.csproj" />
         <ProjectReference Include="..\Modules\MatchRankingModule\MatchRankingModule.csproj" />
diff --git a/src/EvoSC/InternalModules.cs b/src/EvoSC/InternalModules.cs
index abf3e7627..500e002c8 100644
--- a/src/EvoSC/InternalModules.cs
+++ b/src/EvoSC/InternalModules.cs
@@ -5,6 +5,7 @@
 using EvoSC.Modules.Official.ExampleModule;
 using EvoSC.Modules.Official.FastestCpModule;
 using EvoSC.Modules.Official.LiveRankingModule;
+using EvoSC.Modules.Official.LocalRecordsModule;
 using EvoSC.Modules.Official.MapListModule;
 using EvoSC.Modules.Official.MapQueueModule;
 using EvoSC.Modules.Official.MapsModule;
@@ -49,7 +50,8 @@ public static class InternalModules
         typeof(ASayModule),
         typeof(SpectatorTargetInfoModule),
         typeof(MapQueueModule),
-        typeof(MapListModule)
+        typeof(MapListModule),
+        typeof(LocalRecordsModule)
     };
 
     /// <summary>
diff --git a/src/Modules/LocalRecordsModule/Controllers/ShowWidgetController.cs b/src/Modules/LocalRecordsModule/Controllers/ShowWidgetController.cs
new file mode 100644
index 000000000..d57ace0d5
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Controllers/ShowWidgetController.cs
@@ -0,0 +1,18 @@
+using EvoSC.Common.Controllers;
+using EvoSC.Common.Controllers.Attributes;
+using EvoSC.Common.Events.Attributes;
+using EvoSC.Common.Interfaces.Controllers;
+using EvoSC.Common.Remote;
+using GbxRemoteNet.Events;
+
+namespace EvoSC.Modules.Official.LocalRecordsModule.Controllers;
+
+[Controller]
+public class ShowWidgetController : EvoScController<IEventControllerContext>
+{
+    [Subscribe(GbxRemoteEvent.PlayerConnect)]
+    public async Task OnPlayerConnectAsync(object sender, PlayerConnectGbxEventArgs e)
+    {
+        
+    }
+}
diff --git a/src/Modules/LocalRecordsModule/Database/Models/DbLocalRecord.cs b/src/Modules/LocalRecordsModule/Database/Models/DbLocalRecord.cs
index b5946b412..d61761537 100644
--- a/src/Modules/LocalRecordsModule/Database/Models/DbLocalRecord.cs
+++ b/src/Modules/LocalRecordsModule/Database/Models/DbLocalRecord.cs
@@ -8,8 +8,11 @@
 
 namespace EvoSC.Modules.Official.LocalRecordsModule.Database.Models;
 
+[Table(TableName)]
 public class DbLocalRecord : ILocalRecord
 {
+    public const string TableName = "LocalRecords";
+    
     [PrimaryKey, Identity]
     public long Id { get; set; }
     
diff --git a/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs b/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
index 154bb54d3..97df17304 100644
--- a/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
+++ b/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
@@ -3,19 +3,22 @@
 using EvoSC.Common.Database.Repository;
 using EvoSC.Common.Interfaces.Database;
 using EvoSC.Common.Interfaces.Models;
+using EvoSC.Common.Services.Attributes;
 using EvoSC.Modules.Official.LocalRecordsModule.Database.Models;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces;
+using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Database;
 using EvoSC.Modules.Official.PlayerRecords.Database.Models;
+using EvoSC.Modules.Official.PlayerRecords.Interfaces;
 using EvoSC.Modules.Official.PlayerRecords.Interfaces.Models;
 using LinqToDB;
 
 namespace EvoSC.Modules.Official.LocalRecordsModule.Database.Repository;
 
-public class LocalRecordRepository : DbRepository, ILocalRecordRepository
+[Service]
+public class LocalRecordRepository(IDbConnectionFactory dbConnFactory, IPlayerRecordsRepository recordsRepository)
+    : DbRepository(dbConnFactory), ILocalRecordRepository
 {
-    protected LocalRecordRepository(IDbConnectionFactory dbConnFactory) : base(dbConnFactory)
-    {
-    }
+    private readonly IPlayerRecordsRepository _recordsRepository = recordsRepository;
 
     public async Task<IEnumerable<DbLocalRecord>> GetLocalRecordsOfMapByIdAsync(long mapId) =>
         await NewLoadAll()
@@ -53,6 +56,8 @@ public async Task<DbLocalRecord> AddOrUpdateRecordAsync(IMap map, IPlayerRecord
             
             var id = await Database.InsertWithIdentityAsync(localRecord);
             localRecord.Id = Convert.ToInt64(id);
+
+            await transaction.CommitTransactionAsync();
         }
         catch (Exception ex)
         {
@@ -110,6 +115,38 @@ await NewLoadAll()
         await NewLoadAll()
             .FirstOrDefaultAsync(r => r.Record.Player.Id == player.Id);
 
+    public async Task DeleteRecordAsync(IPlayer player, IMap map)
+    {
+        var transaction = await Database.BeginTransactionAsync();
+
+        try
+        {
+            await Table<DbLocalRecord>().DeleteAsync(r => r.MapId == map.Id && r.Record.Player.Id == player.Id);
+            await recordsRepository.DeleteRecordAsync(player, map);
+        }
+        catch (Exception ex)
+        {
+            await transaction.RollbackTransactionAsync();
+            throw;
+        }
+    }
+
+    public async Task DeleteRecordAsync(ILocalRecord localRecord)
+    {
+        var transaction = await Database.BeginTransactionAsync();
+
+        try
+        {
+            await Database.DeleteAsync(localRecord);
+            await recordsRepository.DeleteRecordAsync(localRecord.Record);
+        }
+        catch (Exception ex)
+        {
+            await transaction.RollbackTransactionAsync();
+            throw;
+        }
+    }
+
     private ILoadWithQueryable<DbLocalRecord, IPlayer> NewLoadAll() => Table<DbLocalRecord>()
         .LoadWith(r => r.Map)
         .LoadWith(r => r.Record)
diff --git a/src/Modules/LocalRecordsModule/Interfaces/ILocalRecordRepository.cs b/src/Modules/LocalRecordsModule/Interfaces/Database/ILocalRecordRepository.cs
similarity index 75%
rename from src/Modules/LocalRecordsModule/Interfaces/ILocalRecordRepository.cs
rename to src/Modules/LocalRecordsModule/Interfaces/Database/ILocalRecordRepository.cs
index 74d32dbe2..0227882f4 100644
--- a/src/Modules/LocalRecordsModule/Interfaces/ILocalRecordRepository.cs
+++ b/src/Modules/LocalRecordsModule/Interfaces/Database/ILocalRecordRepository.cs
@@ -2,7 +2,7 @@
 using EvoSC.Modules.Official.LocalRecordsModule.Database.Models;
 using EvoSC.Modules.Official.PlayerRecords.Interfaces.Models;
 
-namespace EvoSC.Modules.Official.LocalRecordsModule.Interfaces;
+namespace EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Database;
 
 public interface ILocalRecordRepository
 {
@@ -11,4 +11,6 @@ public interface ILocalRecordRepository
     public Task RecalculatePositionsOfMapAsync(IMap map);
     public Task<IEnumerable<DbLocalRecord>> GetRecordsByPlayerAsync(IPlayer player);
     public Task<DbLocalRecord?> GetRecordOfPlayerInMapAsync(IPlayer player, IMap map);
+    public Task DeleteRecordAsync(IPlayer player, IMap map);
+    public Task DeleteRecordAsync(ILocalRecord localRecord);
 }
diff --git a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
new file mode 100644
index 000000000..02183fb76
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
@@ -0,0 +1,7 @@
+namespace EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
+
+public interface ILocalRecordsService
+{
+    public Task ShowWidgetAsync();
+    public Task<IEnumerable<ILocalRecord>> GetLocalsOfCurrentMapAsync();
+}
diff --git a/src/Modules/LocalRecordsModule/LocalRecordsModule.cs b/src/Modules/LocalRecordsModule/LocalRecordsModule.cs
index af409a695..2377145c1 100644
--- a/src/Modules/LocalRecordsModule/LocalRecordsModule.cs
+++ b/src/Modules/LocalRecordsModule/LocalRecordsModule.cs
@@ -1,8 +1,20 @@
+using EvoSC.Manialinks.Interfaces;
 using EvoSC.Modules.Attributes;
+using EvoSC.Modules.Interfaces;
+using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
 
 namespace EvoSC.Modules.Official.LocalRecordsModule;
 
 [Module(IsInternal = true)]
-public class LocalRecordsModule : EvoScModule
+public class LocalRecordsModule(IManialinkManager manialinkManager, ILocalRecordsService localRecordsService) : EvoScModule, IToggleable
 {
+    public Task EnableAsync() =>
+        manialinkManager.SendPersistentManialinkAsync("LocalRecordsModule.LocalRecordsWidget",
+            async () =>
+            {
+                var records = localRecordsService.GetLocalsOfCurrentMapAsync();
+                return new { records };
+            });
+
+    public Task DisableAsync() => manialinkManager.HideManialinkAsync("LocalRecordsModule.LocalRecordsWidget");
 }
diff --git a/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj b/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj
index d9b3074db..8d279fb72 100644
--- a/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj
+++ b/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj
@@ -22,5 +22,8 @@
         <ProjectReference Include="../../EvoSC.Modules/EvoSC.Modules.csproj" Private="true" />
         <ProjectReference Include="..\PlayerRecords\PlayerRecords.csproj" />
   </ItemGroup>
+  <ItemGroup>
+    <Folder Include="Services\" />
+  </ItemGroup>
   
   </Project>
diff --git a/src/Modules/LocalRecordsModule/Migrations/202406050955_AddLocalRecordsTable.cs b/src/Modules/LocalRecordsModule/Migrations/202406050955_AddLocalRecordsTable.cs
index f22f5e511..b40bb8a9d 100644
--- a/src/Modules/LocalRecordsModule/Migrations/202406050955_AddLocalRecordsTable.cs
+++ b/src/Modules/LocalRecordsModule/Migrations/202406050955_AddLocalRecordsTable.cs
@@ -1,3 +1,4 @@
+using EvoSC.Modules.Official.LocalRecordsModule.Database.Models;
 using FluentMigrator;
 
 namespace EvoSC.Modules.Official.LocalRecordsModule;
@@ -8,9 +9,15 @@ public class AddLocalRecordsTable : Migration
 {
     public override void Up()
     {
+        Create.Table(DbLocalRecord.TableName)
+            .WithColumn(nameof(DbLocalRecord.Id)).AsInt64().PrimaryKey().Identity()
+            .WithColumn(nameof(DbLocalRecord.MapId)).AsInt64().Indexed()
+            .WithColumn(nameof(DbLocalRecord.RecordId)).AsInt64().Indexed()
+            .WithColumn(nameof(DbLocalRecord.Position)).AsInt32();
     }
 
     public override void Down()
     {
+        Delete.Table(DbLocalRecord.TableName);
     }
 }
diff --git a/src/Modules/LocalRecordsModule/info.toml b/src/Modules/LocalRecordsModule/info.toml
index 8c4d95a58..fc5c8d2f7 100644
--- a/src/Modules/LocalRecordsModule/info.toml
+++ b/src/Modules/LocalRecordsModule/info.toml
@@ -9,3 +9,6 @@ summary = "Determine and display top records of a map from locally saved records
 version = "1.0.0"
 # The name of the author that created this module
 author = "Evo"
+
+[dependencies]
+RecordsModule = "1.0.0"
diff --git a/src/Modules/PlayerRecords/Database/Repository/PlayerRecordsRepository.cs b/src/Modules/PlayerRecords/Database/Repository/PlayerRecordsRepository.cs
index e97979ab5..192487279 100644
--- a/src/Modules/PlayerRecords/Database/Repository/PlayerRecordsRepository.cs
+++ b/src/Modules/PlayerRecords/Database/Repository/PlayerRecordsRepository.cs
@@ -52,4 +52,9 @@ public async Task<DbPlayerRecord> InsertRecordAsync(IPlayer player, IMap map, in
 
         return record;
     }
+
+    public Task DeleteRecordAsync(IPlayer player, IMap map) => Table<DbPlayerRecord>()
+        .DeleteAsync(r => r.MapId == map.Id && r.PlayerId == player.Id);
+
+    public Task DeleteRecordAsync(IPlayerRecord record) => Database.DeleteAsync(record);
 }
diff --git a/src/Modules/PlayerRecords/Interfaces/IPlayerRecordsRepository.cs b/src/Modules/PlayerRecords/Interfaces/IPlayerRecordsRepository.cs
index e5a2e68a4..2c4051512 100644
--- a/src/Modules/PlayerRecords/Interfaces/IPlayerRecordsRepository.cs
+++ b/src/Modules/PlayerRecords/Interfaces/IPlayerRecordsRepository.cs
@@ -1,5 +1,6 @@
 using EvoSC.Common.Interfaces.Models;
 using EvoSC.Modules.Official.PlayerRecords.Database.Models;
+using EvoSC.Modules.Official.PlayerRecords.Interfaces.Models;
 
 namespace EvoSC.Modules.Official.PlayerRecords.Interfaces;
 
@@ -26,4 +27,7 @@ public interface IPlayerRecordsRepository
     /// <param name="record">The record to add.</param>
     /// <returns></returns>
     public Task<DbPlayerRecord> InsertRecordAsync(IPlayer player, IMap map, int score, IEnumerable<int> checkpoints);
+
+    public Task DeleteRecordAsync(IPlayer player, IMap map);
+    public Task DeleteRecordAsync(IPlayerRecord record);
 }

From f8b48e3ed861498ff7a94b2b88a13ae9fdd97f7e Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Wed, 5 Jun 2024 14:08:40 +0200
Subject: [PATCH 03/18] fix persistent manialinks

---
 src/EvoSC.Manialinks/ManialinkManager.cs      | 17 +++++++++++---
 .../Templates/Containers/Widget.mt            |  6 ++---
 .../Controllers/ShowWidgetController.cs       |  5 ----
 .../Services/ILocalRecordsService.cs          |  1 -
 .../LocalRecordsModule.csproj                 |  3 ---
 .../Services/LocalRecordsService.cs           | 23 +++++++++++++++++++
 .../Templates/LocalRecordsWidget.mt           |  7 +++++-
 7 files changed, 46 insertions(+), 16 deletions(-)
 create mode 100644 src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs

diff --git a/src/EvoSC.Manialinks/ManialinkManager.cs b/src/EvoSC.Manialinks/ManialinkManager.cs
index e84da3e90..a2d5e7f50 100644
--- a/src/EvoSC.Manialinks/ManialinkManager.cs
+++ b/src/EvoSC.Manialinks/ManialinkManager.cs
@@ -1,8 +1,10 @@
 using System.Collections.Concurrent;
+using System.ComponentModel;
 using System.Diagnostics;
 using System.Dynamic;
 using System.Reflection;
 using System.Text;
+using System.Text.Json.Serialization;
 using EvoSC.Common.Events;
 using EvoSC.Common.Interfaces;
 using EvoSC.Common.Interfaces.Models;
@@ -23,6 +25,8 @@
 using ManiaTemplates;
 using ManiaTemplates.Lib;
 using Microsoft.Extensions.Logging;
+using Newtonsoft.Json;
+using JsonSerializer = System.Text.Json.JsonSerializer;
 
 namespace EvoSC.Manialinks;
 
@@ -233,10 +237,17 @@ public async Task SendPersistentManialinkAsync(string name, dynamic data)
     public Task SendPersistentManialinkAsync(string name) => SendPersistentManialinkAsync(name, new { });
 
     public Task SendPersistentManialinkAsync(string name, Func<Task<dynamic>> setupData) =>
-        SendPersistentManialinkAsync(name, async () =>
+        SendPersistentManialinkAsync(name, async Task<IDictionary<string, object?>> () =>
         {
-            var data = await setupData();
-            return (IDictionary<string, object?>)data;
+            var rawData = await setupData();
+            IDictionary<string, object?> data = new ExpandoObject();
+
+            foreach (PropertyDescriptor prop in TypeDescriptor.GetProperties(rawData.GetType()))
+            {
+                data.Add(prop.Name, prop.GetValue(rawData));
+            }
+            
+            return data;
         });
 
     public async Task SendPersistentManialinkAsync(string name, Func<Task<IDictionary<string, object?>>> setupData)
diff --git a/src/EvoSC.Manialinks/Templates/Containers/Widget.mt b/src/EvoSC.Manialinks/Templates/Containers/Widget.mt
index 0eb24922a..34a257334 100644
--- a/src/EvoSC.Manialinks/Templates/Containers/Widget.mt
+++ b/src/EvoSC.Manialinks/Templates/Containers/Widget.mt
@@ -2,15 +2,15 @@
   <property type="string" name="header" default="" />
   <property type="string" name="position" default="left" /> <!-- Values: left, right, top, bottom -->
   <property type="double" name="height" />
-  <property type="double" name="x" />
-  <property type="double" name="y" />
+  <property type="double" name="x" default="0" />
+  <property type="double" name="y" default="0" />
   <property type="string" name="style" default="normal" /> <!-- Values: normal, unstyled -->
   <property type="bool" name="ingrid" default="true" /> <!-- If true, position and size acts on the grid system -->
   <property type="double" name="gridsize" default="12" /> <!-- If true, position and size acts on the grid system -->
   <property type="double" name="cellsize" default="10" /> <!-- If true, position and size acts on the grid system -->
   
   <template>
-    <frame pos="{{ x*(cellsize + (160 - (160/gridsize - cellsize))/gridsize - cellsize) }} {{ y*(cellsize + (160 - (160/gridsize - cellsize))/gridsize - cellsize) }}">
+    <frame pos="">
       <frame if='header != ""'>
         <quad
                 bgcolor="28212F"
diff --git a/src/Modules/LocalRecordsModule/Controllers/ShowWidgetController.cs b/src/Modules/LocalRecordsModule/Controllers/ShowWidgetController.cs
index d57ace0d5..cc16853d8 100644
--- a/src/Modules/LocalRecordsModule/Controllers/ShowWidgetController.cs
+++ b/src/Modules/LocalRecordsModule/Controllers/ShowWidgetController.cs
@@ -10,9 +10,4 @@ namespace EvoSC.Modules.Official.LocalRecordsModule.Controllers;
 [Controller]
 public class ShowWidgetController : EvoScController<IEventControllerContext>
 {
-    [Subscribe(GbxRemoteEvent.PlayerConnect)]
-    public async Task OnPlayerConnectAsync(object sender, PlayerConnectGbxEventArgs e)
-    {
-        
-    }
 }
diff --git a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
index 02183fb76..05d3646a4 100644
--- a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
@@ -2,6 +2,5 @@
 
 public interface ILocalRecordsService
 {
-    public Task ShowWidgetAsync();
     public Task<IEnumerable<ILocalRecord>> GetLocalsOfCurrentMapAsync();
 }
diff --git a/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj b/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj
index 8d279fb72..d9b3074db 100644
--- a/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj
+++ b/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj
@@ -22,8 +22,5 @@
         <ProjectReference Include="../../EvoSC.Modules/EvoSC.Modules.csproj" Private="true" />
         <ProjectReference Include="..\PlayerRecords\PlayerRecords.csproj" />
   </ItemGroup>
-  <ItemGroup>
-    <Folder Include="Services\" />
-  </ItemGroup>
   
   </Project>
diff --git a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
new file mode 100644
index 000000000..ab269a42c
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
@@ -0,0 +1,23 @@
+using EvoSC.Common.Interfaces.Services;
+using EvoSC.Common.Services.Attributes;
+using EvoSC.Modules.Official.LocalRecordsModule.Interfaces;
+using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Database;
+using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
+
+namespace EvoSC.Modules.Official.LocalRecordsModule.Services;
+
+[Service]
+public class LocalRecordsService(IMapService mapService, ILocalRecordRepository localRecordRepository) : ILocalRecordsService
+{
+    public async Task<IEnumerable<ILocalRecord>> GetLocalsOfCurrentMapAsync()
+    {
+        var currentMap = await mapService.GetCurrentMapAsync();
+
+        if (currentMap == null)
+        {
+            throw new InvalidOperationException("Failed to get current map");
+        }
+        
+        return await localRecordRepository.GetLocalRecordsOfMapByIdAsync(currentMap.Id);
+    }
+}
diff --git a/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt b/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
index 2ba8d97ee..9bf845ab6 100644
--- a/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
+++ b/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
@@ -1,5 +1,10 @@
 <component>
+  <import component="EvoSC.Containers.Widget" as="Widget" />
+  <import component="EvoSC.Style.UIStyle" as="UIStyle" />
+  
   <template>
-    
+    <Widget header="local records" height="20">
+      
+    </Widget>
   </template>
 </component>

From 88d39b7dd427dc13f003d1bbf45ffb9cfadf22d5 Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Wed, 19 Jun 2024 15:10:21 +0200
Subject: [PATCH 04/18] local records widget template

---
 .../Util/TextFormatting/TextColor.cs          | 23 +++++++-
 .../Util/TextFormatting/TextStyling.cs        | 11 ++++
 .../Templates/Containers/Widget.mt            | 41 +++++++------
 .../Util/GlobalManialinkUtils.cs              | 29 ++++++++++
 .../ASayModule/Themes/DefaultASayTheme.cs     |  2 +-
 .../Templates/LocalRecordsWidget.mt           | 15 ++++-
 .../Templates/Style/LocalRecordsStyle.mt      | 11 ++++
 .../Templates/Widget/RecordRow.mt             | 58 +++++++++++++++++++
 .../Themes/DefaultLocalRecordsTheme.cs        | 25 ++++++++
 9 files changed, 190 insertions(+), 25 deletions(-)
 create mode 100644 src/Modules/LocalRecordsModule/Templates/Style/LocalRecordsStyle.mt
 create mode 100644 src/Modules/LocalRecordsModule/Templates/Widget/RecordRow.mt
 create mode 100644 src/Modules/LocalRecordsModule/Themes/DefaultLocalRecordsTheme.cs

diff --git a/src/EvoSC.Common/Util/TextFormatting/TextColor.cs b/src/EvoSC.Common/Util/TextFormatting/TextColor.cs
index a5c5973d0..97e69a5f7 100644
--- a/src/EvoSC.Common/Util/TextFormatting/TextColor.cs
+++ b/src/EvoSC.Common/Util/TextFormatting/TextColor.cs
@@ -29,9 +29,26 @@ public TextColor(Color color)
 
     public TextColor(string hex)
     {
-        this._r = byte.Parse(hex[0].ToString(), System.Globalization.NumberStyles.HexNumber);
-        this._g = byte.Parse(hex[1].ToString(), System.Globalization.NumberStyles.HexNumber);
-        this._b = byte.Parse(hex[2].ToString(), System.Globalization.NumberStyles.HexNumber);
+        if (hex.Length == 3)
+        {
+            this._r = byte.Parse(hex[0].ToString(), System.Globalization.NumberStyles.HexNumber);
+            this._g = byte.Parse(hex[1].ToString(), System.Globalization.NumberStyles.HexNumber);
+            this._b = byte.Parse(hex[2].ToString(), System.Globalization.NumberStyles.HexNumber);
+        } 
+        else if (hex.Length >= 6)
+        {
+            var r = Math.Floor(byte.Parse(hex[0..2], NumberStyles.HexNumber) / 255.0 * 0xF);
+            var g = Math.Floor(byte.Parse(hex[2..4], NumberStyles.HexNumber) / 255.0 * 0xF);
+            var b = Math.Floor(byte.Parse(hex[4..6], NumberStyles.HexNumber) / 255.0 * 0xF);
+
+            this._r = (byte)r;
+            this._g = (byte)g;
+            this._b = (byte)b;
+        }
+        else
+        {
+            throw new FormatException("Invalid color code");
+        }
     }
 
     /// <summary>
diff --git a/src/EvoSC.Common/Util/TextFormatting/TextStyling.cs b/src/EvoSC.Common/Util/TextFormatting/TextStyling.cs
index 76330bd10..14f135415 100644
--- a/src/EvoSC.Common/Util/TextFormatting/TextStyling.cs
+++ b/src/EvoSC.Common/Util/TextFormatting/TextStyling.cs
@@ -31,6 +31,17 @@ public TextStyling WithColor(TextColor color)
         return this;
     }
     
+    /// <summary>
+    /// Set the color of this style.
+    /// </summary>
+    /// <param name="color">The color to use.</param>
+    /// <returns></returns>
+    public TextStyling WithColor(string color)
+    {
+        _color = new TextColor(color);
+        return this;
+    }
+    
     /// <summary>
     /// Set the color of this style.
     /// </summary>
diff --git a/src/EvoSC.Manialinks/Templates/Containers/Widget.mt b/src/EvoSC.Manialinks/Templates/Containers/Widget.mt
index 34a257334..d6401cc93 100644
--- a/src/EvoSC.Manialinks/Templates/Containers/Widget.mt
+++ b/src/EvoSC.Manialinks/Templates/Containers/Widget.mt
@@ -2,53 +2,56 @@
   <property type="string" name="header" default="" />
   <property type="string" name="position" default="left" /> <!-- Values: left, right, top, bottom -->
   <property type="double" name="height" />
+  <property type="double" name="width" default="36" />
   <property type="double" name="x" default="0" />
   <property type="double" name="y" default="0" />
-  <property type="string" name="style" default="normal" /> <!-- Values: normal, unstyled -->
+  <property type="double" name="padding" default="1.5" />
+  <property type="string" name="headerStyle" default="normal" /> <!-- Values: normal, unstyled -->
+  <property type="string" name="bodyStyle" default="normal" /> <!-- Values: normal, unstyled -->
   <property type="bool" name="ingrid" default="true" /> <!-- If true, position and size acts on the grid system -->
   <property type="double" name="gridsize" default="12" /> <!-- If true, position and size acts on the grid system -->
   <property type="double" name="cellsize" default="10" /> <!-- If true, position and size acts on the grid system -->
   
   <template>
-    <frame pos="">
+    <frame pos='{{ (position == "left" ? -160 + padding : 160-width - padding ) }} {{ y }}'>
       <frame if='header != ""'>
         <quad
-                bgcolor="28212F"
-                size="37 5"
+                bgcolor="{{ Theme.UI_HeaderBg }}"
+                size="{{ width }} 4.5"
                 opacity="0.95"
-                pos="0.8 0"
-                if='style.ToLower() == "normal"'
+                pos="0.7 0"
+                if='headerStyle.ToLower() == "normal"'
         />
         <quad
-                bgcolor="ff0058"
-                size="0.8 5"
+                bgcolor="{{ Theme.UI_AccentPrimary }}"
+                size="0.7 4.5"
                 pos="0 0"
-                if='style.ToLower() == "normal"'
+                if='headerStyle.ToLower() == "normal"'
         />
         <label 
                 text="{{ header.ToUpper() }}" 
-                textfont="GameFontExtraBold"
-                textsize="0.8"
-                pos="18.5 -2.4"
+                textfont="{{ Font.Bold }}"
+                textsize="0.5"
+                pos="{{ width/2.0 + 1 }} -2.1"
                 halign="center"
                 valign="center"
-                if='style.ToLower() == "normal"'
+                if='headerStyle.ToLower() == "normal"'
         />
         <slot name="header" />
       </frame>
-      <frame pos='0 {{ -5 - (header == "" ? 0 : 0.3) }}'>
+      <frame pos='0 {{ -4.5 - (header == "" ? 0 : 0.3) }}'>
         <quad
-                size="37 {{ height }}"
+                size="{{ width }} {{ height }}"
                 bgcolor="2C2D34"
                 opacity="0.9"
-                pos="0.8 0"
-                if='style.ToLower() == "normal"'
+                pos="1 0"
+                if='bodyStyle.ToLower() == "normal"'
         />
         <quad
-                size="0.8 {{ height }}"
+                size="1 {{ height }}"
                 pos="0 0"
                 bgcolor="ff0058"
-                if='style.ToLower() == "normal"'
+                if='bodyStyle.ToLower() == "normal"'
         />
         <slot name="body" />
       </frame>
diff --git a/src/EvoSC.Manialinks/Util/GlobalManialinkUtils.cs b/src/EvoSC.Manialinks/Util/GlobalManialinkUtils.cs
index 0b5f9f2cd..f9fc15212 100644
--- a/src/EvoSC.Manialinks/Util/GlobalManialinkUtils.cs
+++ b/src/EvoSC.Manialinks/Util/GlobalManialinkUtils.cs
@@ -1,6 +1,8 @@
 using System.Globalization;
 using EvoSC.Common.Interfaces.Themes;
+using EvoSC.Common.Interfaces.Util;
 using EvoSC.Common.Util;
+using EvoSC.Common.Util.TextFormatting;
 
 namespace EvoSC.Manialinks.Util;
 
@@ -82,4 +84,31 @@ public bool HasItem(string items, string item, string splitter)
     }
 
     public bool HasItem(string items, string item) => HasItem(items, item, ",");
+
+    public string StyledTime(IRaceTime time)
+    {
+        var styledTime = new TextFormatter();
+
+        if (time.Hours > 0)
+        {
+            styledTime.AddText(time.Hours + ":", s => s.WithColor(_theme.UI_TextPrimary));
+        }
+
+        styledTime.AddText(time.Minutes.ToString().PadLeft(2, '0'), s => s
+            .WithColor(time.Minutes + time.Hours > 0 ? _theme.UI_TextPrimary : _theme.UI_TextMuted)
+        );
+
+        styledTime.AddText(":", s => s.WithColor(_theme.UI_TextPrimary));
+        styledTime.AddText(time.Seconds.ToString().PadLeft(2, '0'), s => s
+            .WithColor(time.Seconds + time.Minutes > 0 ? _theme.UI_TextPrimary : _theme.UI_TextMuted)
+        );
+        
+        styledTime.AddText(".", s => s.WithColor(_theme.UI_TextPrimary));
+        styledTime.AddText(time.Milliseconds.ToString().PadLeft(3, '0'), s => s
+            .WithColor(time.Milliseconds + time.Seconds > 0 ? _theme.UI_TextPrimary : _theme.UI_TextMuted)
+        );
+
+        Console.WriteLine(styledTime.ToString());
+        return styledTime.ToString();
+    }
 }
diff --git a/src/Modules/ASayModule/Themes/DefaultASayTheme.cs b/src/Modules/ASayModule/Themes/DefaultASayTheme.cs
index 7d771d8ec..008bb1a2d 100644
--- a/src/Modules/ASayModule/Themes/DefaultASayTheme.cs
+++ b/src/Modules/ASayModule/Themes/DefaultASayTheme.cs
@@ -14,7 +14,7 @@ public override Task ConfigureAsync()
         Set("ASayModule.Announcement.Default.TextSize").To(3);
         Set("ASayModule.Announcement.Default.IconSize").To(4);
         Set("ASayModule.Announcement.Default.Bg").To(_theme.UI_BgPrimary);
-        Set("ASayModule.Announcement.Default.BgSecondary").To(_theme.White);
+        Set("ASayModule.Announcement.Default.BgSecondary").To(_theme.UI_AccentSecondary);
         Set("ASayModule.Announcement.Default.Text").To(_theme.UI_TextPrimary);
         
         return Task.CompletedTask;
diff --git a/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt b/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
index 9bf845ab6..ccaa762e8 100644
--- a/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
+++ b/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
@@ -1,10 +1,21 @@
 <component>
+  <using namespace="EvoSC.Common.Interfaces.Util" />
+  <using namespace="EvoSC.Common.Util" />
+  
   <import component="EvoSC.Containers.Widget" as="Widget" />
   <import component="EvoSC.Style.UIStyle" as="UIStyle" />
   
+  <import component="LocalRecordsModule.Widget.RecordRow" as="RecordRow" />
+  <import component="LocalRecordsModule.Style.LocalRecordsStyle" as="LocalRecordsStyle" />
+  
   <template>
-    <Widget header="local records" height="20">
-      
+    <UIStyle />
+    <LocalRecordsStyle />
+    
+    <Widget header="local records" height="20" bodyStyle="unstyled" y="80">
+      <template slot="body">
+        <RecordRow pos="{{ __index + 140 }}" name="snixtho" y="{{ -__index*(4+0.3) }}" foreach="int i in Util.Range(10)" time="{{ RaceTime.FromMilliseconds(45236) }}" />
+      </template>
     </Widget>
   </template>
 </component>
diff --git a/src/Modules/LocalRecordsModule/Templates/Style/LocalRecordsStyle.mt b/src/Modules/LocalRecordsModule/Templates/Style/LocalRecordsStyle.mt
new file mode 100644
index 000000000..771809e4e
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Templates/Style/LocalRecordsStyle.mt
@@ -0,0 +1,11 @@
+<component>
+  <template>
+    <stylesheet>
+      <style
+              class="lr-body-primary"
+              bgcolor="{{ Theme.UI_BgPrimary }}"
+              opacity="0.9"
+      />
+    </stylesheet>
+  </template>
+</component>
\ No newline at end of file
diff --git a/src/Modules/LocalRecordsModule/Templates/Widget/RecordRow.mt b/src/Modules/LocalRecordsModule/Templates/Widget/RecordRow.mt
new file mode 100644
index 000000000..da565b74e
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Templates/Widget/RecordRow.mt
@@ -0,0 +1,58 @@
+<component>
+  <using namespace="EvoSC.Common.Interfaces.Util" />
+  
+  <import component="EvoSC.Controls.Panel" as="Panel" />
+  
+  <property type="double" name="width" default="36" />
+  <property type="double" name="height" default="4" />
+  <property type="double" name="y" default="0" />
+  
+  <property type="int" name="pos" />
+  <property type="string" name="name" />
+  <property type="IRaceTime" name="time" />
+  
+  <template>
+    <frame pos="0 {{ y }}">
+      <quad
+              bgcolor="{{ Theme.UI_LocalRecordsModule_Widget_AccentColor(pos) }}"
+              size="0.7 {{ height }}"
+              pos="0 0"
+      />
+      <quad
+              bgcolor="{{ Theme.UI_AccentSecondary }}"
+              size="{{ height }} {{ height }}"
+              opacity="0.9"
+              pos="0.7 0"
+      />
+      <label 
+              text="{{ pos }}" 
+              class="text-primary"
+              pos="{{ 0.6 + height/2 }} {{ -height/2.0 + 0.2 }}"
+              size="{{ height }} {{ height }}"
+              valign="center"
+              halign="center"
+              textcolor="{{ Theme.Black }}"
+      />
+      
+      <Panel width="{{ width-4 }}" height="4" className="lr-body-primary" x="4.7" bgColor="">
+        <label 
+                text="{{ name }}"
+                class="text-primary" 
+                textsize="0.5"
+                pos="1 {{ -height/2.0 + 0.2 }}"
+                valign="center"
+                size="{{ width-18 }} {{ height }}"
+        />
+
+        <label
+                text="{{ Util.StyledTime(time) }}"
+                class="text-primary"
+                textsize="0.5"
+                pos="{{ width-16.5 }} {{ -height/2.0 + 0.2 }}"
+                valign="center"
+                size="16 {{ height }}"
+        />
+      </Panel>
+    </frame>
+  </template>
+</component>
diff --git a/src/Modules/LocalRecordsModule/Themes/DefaultLocalRecordsTheme.cs b/src/Modules/LocalRecordsModule/Themes/DefaultLocalRecordsTheme.cs
new file mode 100644
index 000000000..86e36a317
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Themes/DefaultLocalRecordsTheme.cs
@@ -0,0 +1,25 @@
+using EvoSC.Common.Interfaces.Themes;
+using EvoSC.Common.Themes;
+using EvoSC.Common.Themes.Attributes;
+
+namespace EvoSC.Modules.Official.LocalRecordsModule.Themes;
+
+[Theme(Name = "Local Records", Description = "Default theme for the local records widget.")]
+public class DefaultLocalRecordsTheme(IThemeManager theme) : Theme<DefaultLocalRecordsTheme>
+{
+    private readonly dynamic _theme = theme.Theme;
+
+    public override Task ConfigureAsync()
+    {
+        Set("UI.LocalRecordsModule.Widget.AccentColor").To((int pos) => pos switch
+        {
+            1 => _theme.Gold,
+            2 => _theme.Silver,
+            3 => _theme.Bronze,
+            _ => _theme.UI_AccentPrimary
+        });
+
+        return Task.CompletedTask;
+    }
+}
+

From 88f6b062ad102e7941a9bbd5af13a45626117619 Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Wed, 19 Jun 2024 15:29:52 +0200
Subject: [PATCH 05/18] highlight self

---
 .../LocalRecordsModule/Templates/LocalRecordsWidget.mt   | 9 ++++++++-
 .../Templates/Style/LocalRecordsStyle.mt                 | 8 +++++++-
 .../LocalRecordsModule/Templates/Widget/RecordRow.mt     | 9 ++++++++-
 .../Themes/DefaultLocalRecordsTheme.cs                   | 4 ++++
 4 files changed, 27 insertions(+), 3 deletions(-)

diff --git a/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt b/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
index ccaa762e8..54933264c 100644
--- a/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
+++ b/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
@@ -14,7 +14,14 @@
     
     <Widget header="local records" height="20" bodyStyle="unstyled" y="80">
       <template slot="body">
-        <RecordRow pos="{{ __index + 140 }}" name="snixtho" y="{{ -__index*(4+0.3) }}" foreach="int i in Util.Range(10)" time="{{ RaceTime.FromMilliseconds(45236) }}" />
+        <RecordRow 
+                pos="{{ __index + 140 }}"
+                name="snixtho" 
+                y="{{ -__index*(4+0.3) }}" 
+                foreach="int i in Util.Range(10)"
+                time="{{ RaceTime.FromMilliseconds(45236) }}" 
+                self="{{ __index == 6 }}"
+        />
       </template>
     </Widget>
   </template>
diff --git a/src/Modules/LocalRecordsModule/Templates/Style/LocalRecordsStyle.mt b/src/Modules/LocalRecordsModule/Templates/Style/LocalRecordsStyle.mt
index 771809e4e..9ccbbfc34 100644
--- a/src/Modules/LocalRecordsModule/Templates/Style/LocalRecordsStyle.mt
+++ b/src/Modules/LocalRecordsModule/Templates/Style/LocalRecordsStyle.mt
@@ -3,7 +3,13 @@
     <stylesheet>
       <style
               class="lr-body-primary"
-              bgcolor="{{ Theme.UI_BgPrimary }}"
+              bgcolor="{{ Theme.UI_LocalRecordsModule_Widget_RowBg }}"
+              opacity="0.9"
+      />
+
+      <style
+              class="lr-body-highlight"
+              bgcolor="{{ Theme.UI_LocalRecordsModule_Widget_RowBgHighlight }}"
               opacity="0.9"
       />
     </stylesheet>
diff --git a/src/Modules/LocalRecordsModule/Templates/Widget/RecordRow.mt b/src/Modules/LocalRecordsModule/Templates/Widget/RecordRow.mt
index da565b74e..42cd6e9ed 100644
--- a/src/Modules/LocalRecordsModule/Templates/Widget/RecordRow.mt
+++ b/src/Modules/LocalRecordsModule/Templates/Widget/RecordRow.mt
@@ -10,6 +10,7 @@
   <property type="int" name="pos" />
   <property type="string" name="name" />
   <property type="IRaceTime" name="time" />
+  <property type="bool" name="self" />
   
   <template>
     <frame pos="0 {{ y }}">
@@ -34,7 +35,13 @@
               textcolor="{{ Theme.Black }}"
       />
       
-      <Panel width="{{ width-4 }}" height="4" className="lr-body-primary" x="4.7" bgColor="">
+      <Panel 
+              width="{{ width-4 }}" 
+              height="4" 
+              className='{{ self ? "lr-body-highlight" : "lr-body-primary" }}'
+              x="4.7"
+              bgColor=""
+      >
         <label 
                 text="{{ name }}"
                 class="text-primary" 
diff --git a/src/Modules/LocalRecordsModule/Themes/DefaultLocalRecordsTheme.cs b/src/Modules/LocalRecordsModule/Themes/DefaultLocalRecordsTheme.cs
index 86e36a317..5bacb1612 100644
--- a/src/Modules/LocalRecordsModule/Themes/DefaultLocalRecordsTheme.cs
+++ b/src/Modules/LocalRecordsModule/Themes/DefaultLocalRecordsTheme.cs
@@ -1,6 +1,7 @@
 using EvoSC.Common.Interfaces.Themes;
 using EvoSC.Common.Themes;
 using EvoSC.Common.Themes.Attributes;
+using EvoSC.Common.Util;
 
 namespace EvoSC.Modules.Official.LocalRecordsModule.Themes;
 
@@ -19,6 +20,9 @@ public override Task ConfigureAsync()
             _ => _theme.UI_AccentPrimary
         });
 
+        Set("UI.LocalRecordsModule.Widget.RowBg").To(_theme.UI_BgPrimary);
+        Set("UI.LocalRecordsModule.Widget.RowBgHighlight").To(ColorUtils.Lighten(_theme.UI_BgPrimary));
+
         return Task.CompletedTask;
     }
 }

From 277ffb4bcb9a3512a4790575f4de59e86d7fc281 Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Sat, 22 Jun 2024 11:07:18 +0200
Subject: [PATCH 06/18] fix maplist throwing maniascript error due to ID
 mismatch

---
 .../Controllers/ShowWidgetController.cs       | 13 --------
 .../Repository/LocalRecordRepository.cs       | 18 +++++-----
 .../LocalRecordsModule/LocalRecordsModule.cs  |  2 +-
 .../LocalRecordsModule.csproj                 |  3 ++
 .../Localization.nb-no.resx                   | 20 +++++++++++
 .../LocalRecordsModule/Localization.resx      |  6 ++++
 .../Templates/LocalRecordsWidget.mt           | 33 +++++++++++++++++--
 .../Templates/Style/LocalRecordsStyle.mt      |  2 +-
 .../MapListModule/Templates/MapList.mt        |  1 +
 .../Repository/PlayerRecordsRepository.cs     |  2 +-
 10 files changed, 71 insertions(+), 29 deletions(-)
 delete mode 100644 src/Modules/LocalRecordsModule/Controllers/ShowWidgetController.cs
 create mode 100644 src/Modules/LocalRecordsModule/Localization.nb-no.resx

diff --git a/src/Modules/LocalRecordsModule/Controllers/ShowWidgetController.cs b/src/Modules/LocalRecordsModule/Controllers/ShowWidgetController.cs
deleted file mode 100644
index cc16853d8..000000000
--- a/src/Modules/LocalRecordsModule/Controllers/ShowWidgetController.cs
+++ /dev/null
@@ -1,13 +0,0 @@
-using EvoSC.Common.Controllers;
-using EvoSC.Common.Controllers.Attributes;
-using EvoSC.Common.Events.Attributes;
-using EvoSC.Common.Interfaces.Controllers;
-using EvoSC.Common.Remote;
-using GbxRemoteNet.Events;
-
-namespace EvoSC.Modules.Official.LocalRecordsModule.Controllers;
-
-[Controller]
-public class ShowWidgetController : EvoScController<IEventControllerContext>
-{
-}
diff --git a/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs b/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
index 97df17304..3f46b4062 100644
--- a/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
+++ b/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
@@ -18,11 +18,9 @@ namespace EvoSC.Modules.Official.LocalRecordsModule.Database.Repository;
 public class LocalRecordRepository(IDbConnectionFactory dbConnFactory, IPlayerRecordsRepository recordsRepository)
     : DbRepository(dbConnFactory), ILocalRecordRepository
 {
-    private readonly IPlayerRecordsRepository _recordsRepository = recordsRepository;
-
     public async Task<IEnumerable<DbLocalRecord>> GetLocalRecordsOfMapByIdAsync(long mapId) =>
         await NewLoadAll()
-            .Where(r => r.Map.Id == mapId)
+            .Where(r => r.DbMap.Id == mapId)
             .ToArrayAsync();
 
     public async Task<DbLocalRecord> AddOrUpdateRecordAsync(IMap map, IPlayerRecord record)
@@ -72,7 +70,7 @@ public async Task<DbLocalRecord> AddOrUpdateRecordAsync(IMap map, IPlayerRecord
     public async Task RecalculatePositionsOfMapAsync(IMap map)
     {
         var locals = await GetLocalRecordsOfMapByIdAsync(map.Id);
-        var sorted = locals.OrderBy(r => r.Record.Score).ToArray();
+        var sorted = locals.OrderBy(r => r.DbRecord.Score).ToArray();
         var updated = new List<DbLocalRecord>();
         
         for (var i = 0; i < sorted.Length; i++)
@@ -108,12 +106,12 @@ public async Task RecalculatePositionsOfMapAsync(IMap map)
 
     public async Task<IEnumerable<DbLocalRecord>> GetRecordsByPlayerAsync(IPlayer player) =>
         await NewLoadAll()
-            .Where(r => r.Record.Player.Id == player.Id)
+            .Where(r => r.DbRecord.DbPlayer.Id == player.Id)
             .ToArrayAsync();
 
     public async Task<DbLocalRecord?> GetRecordOfPlayerInMapAsync(IPlayer player, IMap map) =>
         await NewLoadAll()
-            .FirstOrDefaultAsync(r => r.Record.Player.Id == player.Id);
+            .FirstOrDefaultAsync(r => r.DbRecord.DbPlayer.Id == player.Id);
 
     public async Task DeleteRecordAsync(IPlayer player, IMap map)
     {
@@ -121,7 +119,7 @@ public async Task DeleteRecordAsync(IPlayer player, IMap map)
 
         try
         {
-            await Table<DbLocalRecord>().DeleteAsync(r => r.MapId == map.Id && r.Record.Player.Id == player.Id);
+            await Table<DbLocalRecord>().DeleteAsync(r => r.MapId == map.Id && r.DbRecord.Player.Id == player.Id);
             await recordsRepository.DeleteRecordAsync(player, map);
         }
         catch (Exception ex)
@@ -148,7 +146,7 @@ public async Task DeleteRecordAsync(ILocalRecord localRecord)
     }
 
     private ILoadWithQueryable<DbLocalRecord, IPlayer> NewLoadAll() => Table<DbLocalRecord>()
-        .LoadWith(r => r.Map)
-        .LoadWith(r => r.Record)
-        .ThenLoad(r => r.Player);
+        .LoadWith(r => r.DbMap)
+        .LoadWith(r => r.DbRecord)
+        .ThenLoad(r => r.DbPlayer);
 }
diff --git a/src/Modules/LocalRecordsModule/LocalRecordsModule.cs b/src/Modules/LocalRecordsModule/LocalRecordsModule.cs
index 2377145c1..25c9b25c4 100644
--- a/src/Modules/LocalRecordsModule/LocalRecordsModule.cs
+++ b/src/Modules/LocalRecordsModule/LocalRecordsModule.cs
@@ -12,7 +12,7 @@ public Task EnableAsync() =>
         manialinkManager.SendPersistentManialinkAsync("LocalRecordsModule.LocalRecordsWidget",
             async () =>
             {
-                var records = localRecordsService.GetLocalsOfCurrentMapAsync();
+                var records = await localRecordsService.GetLocalsOfCurrentMapAsync();
                 return new { records };
             });
 
diff --git a/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj b/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj
index d9b3074db..09e8e8dc9 100644
--- a/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj
+++ b/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj
@@ -22,5 +22,8 @@
         <ProjectReference Include="../../EvoSC.Modules/EvoSC.Modules.csproj" Private="true" />
         <ProjectReference Include="..\PlayerRecords\PlayerRecords.csproj" />
   </ItemGroup>
+  <ItemGroup>
+    <Folder Include="Controllers\" />
+  </ItemGroup>
   
   </Project>
diff --git a/src/Modules/LocalRecordsModule/Localization.nb-no.resx b/src/Modules/LocalRecordsModule/Localization.nb-no.resx
new file mode 100644
index 000000000..6ac3a281d
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Localization.nb-no.resx
@@ -0,0 +1,20 @@
+<root>
+  <resheader name="resmimetype">
+    <value>text/microsoft-resx</value>
+  </resheader>
+  <resheader name="version">
+    <value>1.3</value>
+  </resheader>
+  <resheader name="reader">
+    <value>System.Resources.ResXResourceReader, System.Windows.Forms, Version=2.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089</value>
+  </resheader>
+  <resheader name="writer">
+    <value>System.Resources.ResXResourceWriter, System.Windows.Forms, Version=2.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089</value>
+  </resheader>
+  <data name="Widget.NoRecords" xml:space="preserve">
+    <value>Ingen rekorder for denne banen</value>
+  </data>
+  <data name="Widget.Title" xml:space="preserve">
+    <value>LOKALE REKORDER</value>
+  </data>
+</root>
\ No newline at end of file
diff --git a/src/Modules/LocalRecordsModule/Localization.resx b/src/Modules/LocalRecordsModule/Localization.resx
index 10e3157c7..f6b16b454 100644
--- a/src/Modules/LocalRecordsModule/Localization.resx
+++ b/src/Modules/LocalRecordsModule/Localization.resx
@@ -16,4 +16,10 @@
   <resheader name="writer">
     <value>System.Resources.ResXResourceWriter, System.Windows.Forms, Version=2.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089</value>
   </resheader>
+  <data name="Widget.NoRecords" xml:space="preserve">
+    <value>No records for this map</value>
+  </data>
+  <data name="Widget.Title" xml:space="preserve">
+    <value>LOCAL RECORDS</value>
+  </data>
 </root>
diff --git a/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt b/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
index 54933264c..1f41f0ab1 100644
--- a/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
+++ b/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
@@ -1,27 +1,54 @@
 <component>
   <using namespace="EvoSC.Common.Interfaces.Util" />
   <using namespace="EvoSC.Common.Util" />
+  <using namespace="EvoSC.Modules.Official.LocalRecordsModule.Interfaces" />
+  <using namespace="System.Linq" />
   
   <import component="EvoSC.Containers.Widget" as="Widget" />
   <import component="EvoSC.Style.UIStyle" as="UIStyle" />
+  <import component="EvoSC.Controls.Panel" as="Panel" />
   
   <import component="LocalRecordsModule.Widget.RecordRow" as="RecordRow" />
   <import component="LocalRecordsModule.Style.LocalRecordsStyle" as="LocalRecordsStyle" />
   
+  <property type="IEnumerable<ILocalRecord>" name="records" />
+  
   <template>
     <UIStyle />
     <LocalRecordsStyle />
-    
-    <Widget header="local records" height="20" bodyStyle="unstyled" y="80">
+      <Widget header="local records" height="20" bodyStyle="unstyled" y="80">
       <template slot="body">
         <RecordRow 
                 pos="{{ __index + 140 }}"
                 name="snixtho" 
                 y="{{ -__index*(4+0.3) }}" 
-                foreach="int i in Util.Range(10)"
+                foreach="ILocalRecord record in records"
                 time="{{ RaceTime.FromMilliseconds(45236) }}" 
                 self="{{ __index == 6 }}"
         />
+        
+        <frame if="records.Count() == 0">
+          <quad
+                  bgcolor="{{ Theme.UI_AccentPrimary }}"
+                  size="0.7 9"
+          />
+          <Panel
+                  width="36"
+                  height="9"
+                  x="0.7"
+                  className="lr-body-primary"
+                  bgColor=""
+          >
+            <label
+                    class="text-primary"
+                    text="No records for this map"
+                    valign="center"
+                    halign="center"
+                    pos="18 -4.2"
+                    size="36 9"
+            />
+          </Panel>
+        </frame>
       </template>
     </Widget>
   </template>
diff --git a/src/Modules/LocalRecordsModule/Templates/Style/LocalRecordsStyle.mt b/src/Modules/LocalRecordsModule/Templates/Style/LocalRecordsStyle.mt
index 9ccbbfc34..2015ce0dc 100644
--- a/src/Modules/LocalRecordsModule/Templates/Style/LocalRecordsStyle.mt
+++ b/src/Modules/LocalRecordsModule/Templates/Style/LocalRecordsStyle.mt
@@ -14,4 +14,4 @@
       />
     </stylesheet>
   </template>
-</component>
\ No newline at end of file
+</component>
diff --git a/src/Modules/MapListModule/Templates/MapList.mt b/src/Modules/MapListModule/Templates/MapList.mt
index 458aad365..1cb97e7b2 100644
--- a/src/Modules/MapListModule/Templates/MapList.mt
+++ b/src/Modules/MapListModule/Templates/MapList.mt
@@ -25,6 +25,7 @@ Shows a window with a list of maps available in the current match settings.
                  scrollHeight="{{ maps.Count()*8 }}"
                  width="96"
                  height="60"
+                 id="maplist-container"
       >
         <MapRow 
                 foreach="IMapListMap map in maps" 
diff --git a/src/Modules/PlayerRecords/Database/Repository/PlayerRecordsRepository.cs b/src/Modules/PlayerRecords/Database/Repository/PlayerRecordsRepository.cs
index 192487279..fafa5bb9f 100644
--- a/src/Modules/PlayerRecords/Database/Repository/PlayerRecordsRepository.cs
+++ b/src/Modules/PlayerRecords/Database/Repository/PlayerRecordsRepository.cs
@@ -21,7 +21,7 @@ public class PlayerRecordsRepository(DbConnectionFactory dbConnFactory, ILogger<
         Table<DbPlayerRecord>()
             .LoadWith(r => r.DbPlayer)
             .LoadWith(r => r.DbMap)
-            .SingleOrDefaultAsync(r => r.PlayerId == player.Id && r.MapId == map.Id);
+            .FirstOrDefaultAsync(r => r.PlayerId == player.Id && r.MapId == map.Id);
 
     public Task UpdateRecordAsync(DbPlayerRecord record) => Database.UpdateAsync(record);
 

From ec887ebaf2adbf6bc27859eb62892687a93d2b6a Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Sat, 22 Jun 2024 11:12:34 +0200
Subject: [PATCH 07/18] fix current map module throwing exceptions

---
 src/Modules/CurrentMapModule/Services/CurrentMapService.cs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/Modules/CurrentMapModule/Services/CurrentMapService.cs b/src/Modules/CurrentMapModule/Services/CurrentMapService.cs
index b8472aaf8..e203b9e57 100644
--- a/src/Modules/CurrentMapModule/Services/CurrentMapService.cs
+++ b/src/Modules/CurrentMapModule/Services/CurrentMapService.cs
@@ -37,9 +37,9 @@ public async Task HideWidgetAsync()
     private async Task ShowManialinkAsync(string mapUId)
     {
         var dbMap = await mapRepository.GetMapByUidAsync(mapUId);
-        var author = "";
+        string author;
         var worldRecord = await worldRecordService.GetRecordAsync();
-        if (dbMap.Author.NickName == dbMap.Author.AccountId)
+        if (dbMap?.Author?.NickName == dbMap?.Author?.AccountId)
         {
             var serverMap = await client.Remote.GetCurrentMapInfoAsync();
             author = serverMap.AuthorNickname.Length > 0 ? serverMap.AuthorNickname : serverMap.Author;

From e1571558c603008ee5f7c1a66c2ec1f14e00a097 Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Sat, 22 Jun 2024 12:23:41 +0200
Subject: [PATCH 08/18] add manialink transactions to optimize sending of bulk
 manialinks with different data

---
 .../Interfaces/IManialinkManager.cs           | 128 ++----------------
 .../Interfaces/IManialinkOperations.cs        | 122 +++++++++++++++++
 .../Interfaces/IManialinkTransaction.cs       |   6 +
 src/EvoSC.Manialinks/ManialinkManager.cs      |  41 +++---
 src/EvoSC.Manialinks/ManialinkTransaction.cs  | 122 +++++++++++++++++
 src/EvoSC.Manialinks/Util/ManialinkUtils.cs   |  19 +++
 .../Controllers/WidgetUpdateController.cs     |  19 +++
 .../Services/ILocalRecordsService.cs          |   6 +-
 .../LocalRecordsModule/LocalRecordsModule.cs  |   9 +-
 .../LocalRecordsModule.csproj                 |   3 -
 .../Services/LocalRecordsService.cs           |  45 +++++-
 11 files changed, 363 insertions(+), 157 deletions(-)
 create mode 100644 src/EvoSC.Manialinks/Interfaces/IManialinkOperations.cs
 create mode 100644 src/EvoSC.Manialinks/Interfaces/IManialinkTransaction.cs
 create mode 100644 src/EvoSC.Manialinks/ManialinkTransaction.cs
 create mode 100644 src/EvoSC.Manialinks/Util/ManialinkUtils.cs
 create mode 100644 src/Modules/LocalRecordsModule/Controllers/WidgetUpdateController.cs

diff --git a/src/EvoSC.Manialinks/Interfaces/IManialinkManager.cs b/src/EvoSC.Manialinks/Interfaces/IManialinkManager.cs
index 39b91fb77..e657668fc 100644
--- a/src/EvoSC.Manialinks/Interfaces/IManialinkManager.cs
+++ b/src/EvoSC.Manialinks/Interfaces/IManialinkManager.cs
@@ -6,7 +6,7 @@ namespace EvoSC.Manialinks.Interfaces;
 /// <summary>
 /// Manialink template & ManiaScript registry, and can render, display and hide Manialink to players.
 /// </summary>
-public interface IManialinkManager
+public interface IManialinkManager : IManialinkOperations
 {
     /// <summary>
     /// Add all the default templates from EvoSC.
@@ -54,27 +54,10 @@ public interface IManialinkManager
     public void RemoveManiaScript(string name);
     
     /// <summary>
-    /// Render a template and send it to all players.
-    /// </summary>
-    /// <param name="name">The name of the template.</param>
-    /// <param name="data">Any kind of data the template uses.</param>
-    /// <returns></returns>
-    public Task SendManialinkAsync(string name, IDictionary<string, object?> data);
-    
-    /// <summary>
-    /// Render a template and send it to all players.
-    /// </summary>
-    /// <param name="name">The name of the template.</param>
-    /// <param name="data">Any kind of data the template uses.</param>
-    /// <returns></returns>
-    public Task SendManialinkAsync(string name, dynamic data);
-    
-    /// <summary>
-    /// Render a template and send it to all players without data.
+    /// Pre-process all current templates registered.
     /// </summary>
-    /// <param name="name">The name of the template.</param>
     /// <returns></returns>
-    public Task SendManialinkAsync(string name);
+    public Task PreprocessAllAsync();
 
     /// <summary>
     /// Send a manialink to all players which persists even if a player re-connects.
@@ -123,107 +106,16 @@ public interface IManialinkManager
     /// <param name="setupData">Method that returns data to be sent.</param>
     /// <returns></returns>
     public Task SendPersistentManialinkAsync(string name, Func<Task<IDictionary<string, object?>>> setupData);
-    
-    /// <summary>
-    /// Render a template and send it to a specific player.
-    /// </summary>
-    /// <param name="player">The player to send to.</param>
-    /// <param name="name">The name of the template.</param>
-    /// <param name="data">Data which the template uses.</param>
-    /// <returns></returns>
-    public Task SendManialinkAsync(IPlayer player, string name, IDictionary<string, object?> data);
 
-    /// <summary>
-    /// Render a template and send it to a specific player.
-    /// </summary>
-    /// <param name="player">The player to send to.</param>
-    /// <param name="name">The name of the template.</param>
-    /// <param name="data">Data which the template uses</param>
-    /// <returns></returns>
-    public Task SendManialinkAsync(IPlayer player, string name, dynamic data);
-
-    /// <summary>
-    /// Render a template and send it to a specific player.
-    /// </summary>
-    /// <param name="playerLogin">The login to send to.</param>
-    /// <param name="name">The name of the template.</param>
-    /// <param name="data">Data which the template uses</param>
-    /// <returns></returns>
-    public Task SendManialinkAsync(string playerLogin, string name, dynamic data);
-
-    /// <summary>
-    /// Render a template and send it to a specific player without data.
-    /// </summary>
-    /// <param name="player">The player to send to.</param>
-    /// <param name="name">The name of the template.</param>
-    /// <returns></returns>
-    public Task SendManialinkAsync(IPlayer player, string name) => SendManialinkAsync(player, name, new { });
-    
-    /// <summary>
-    /// Render a template and send it to a set of players.
-    /// </summary>
-    /// <param name="players">The players to send to</param>
-    /// <param name="name">The name of the template.</param>
-    /// <param name="data">Data which the template uses</param>
-    /// <returns></returns>
-    public Task SendManialinkAsync(IEnumerable<IPlayer> players, string name, IDictionary<string, object?> data);
+    public Task RemovePersistentManialinkAsync(string name);
     
-    /// <summary>
-    /// Render a template and send it to a set of players.
-    /// </summary>
-    /// <param name="players">The players to send to</param>
-    /// <param name="name">The name of the template.</param>
-    /// <param name="data">Data which the template uses</param>
-    /// <returns></returns>
-    public Task SendManialinkAsync(IEnumerable<IPlayer> players, string name, dynamic data);
-    
-    /// <summary>
-    /// Render a template and send it to a set of players without data.
-    /// </summary>
-    /// <param name="players">The players to send to</param>
-    /// <param name="name">The name of the template.</param>
-    /// <returns></returns>
-    public Task SendManialinkAsync(IEnumerable<IPlayer> players, string name) =>
-        SendManialinkAsync(players, name, new { });
-
-    /// <summary>
-    /// Hide a manialink from all players.
-    /// </summary>
-    /// <param name="name">Name of the manialink to hide.</param>
-    /// <returns></returns>
-    public Task HideManialinkAsync(string name);
-    
-    /// <summary>
-    /// Hide a manialink from a player.
-    /// </summary>
-    /// <param name="player">The player to hide the manialink from.</param>
-    /// <param name="name">Name of the manialink to hide.</param>
-    /// <returns></returns>
-    public Task HideManialinkAsync(IPlayer player, string name);
-    
-    /// <summary>
-    /// Hide a manialink from a player.
-    /// </summary>
-    /// <param name="playerLogin">The player to hide the manialink from.</param>
-    /// <param name="name">Name of the manialink to hide.</param>
-    /// <returns></returns>
-    public Task HideManialinkAsync(string playerLogin, string name);
-    
-    /// <summary>
-    /// Hide a manialink from a set of players.
-    /// </summary>
-    /// <param name="players">The players to hide the manialink from.</param>
-    /// <param name="name">Name of the manialink to hide.</param>
-    /// <returns></returns>
-    public Task HideManialinkAsync(IEnumerable<IPlayer> players, string name);
-    
-    /// <summary>
-    /// Pre-process all current templates registered.
-    /// </summary>
-    /// <returns></returns>
-    public Task PreprocessAllAsync();
-
     public void AddGlobalVariable<T>(string name, T value);
     public void RemoveGlobalVariable(string name);
     public void ClearGlobalVariables();
+
+    public Task<string> PrepareAndRenderAsync(string name, IDictionary<string, object?> data);
+    public Task<string> PrepareAndRenderAsync(string name, dynamic data);
+    public string GetEffectiveName(string name);
+
+    public ManialinkTransaction CreateTransaction();
 }
diff --git a/src/EvoSC.Manialinks/Interfaces/IManialinkOperations.cs b/src/EvoSC.Manialinks/Interfaces/IManialinkOperations.cs
new file mode 100644
index 000000000..b27be90db
--- /dev/null
+++ b/src/EvoSC.Manialinks/Interfaces/IManialinkOperations.cs
@@ -0,0 +1,122 @@
+using EvoSC.Common.Interfaces.Models;
+
+namespace EvoSC.Manialinks.Interfaces;
+
+public interface IManialinkOperations
+{
+    /// <summary>
+    /// Render a template and send it to all players.
+    /// </summary>
+    /// <param name="name">The name of the template.</param>
+    /// <param name="data">Any kind of data the template uses.</param>
+    /// <returns></returns>
+    public Task SendManialinkAsync(string name, IDictionary<string, object?> data);
+    
+    /// <summary>
+    /// Render a template and send it to all players.
+    /// </summary>
+    /// <param name="name">The name of the template.</param>
+    /// <param name="data">Any kind of data the template uses.</param>
+    /// <returns></returns>
+    public Task SendManialinkAsync(string name, dynamic data);
+    
+    /// <summary>
+    /// Render a template and send it to all players without data.
+    /// </summary>
+    /// <param name="name">The name of the template.</param>
+    /// <returns></returns>
+    
+    public Task SendManialinkAsync(string name);
+    /// <summary>
+    /// Render a template and send it to a specific player.
+    /// </summary>
+    /// <param name="player">The player to send to.</param>
+    /// <param name="name">The name of the template.</param>
+    /// <param name="data">Data which the template uses.</param>
+    /// <returns></returns>
+    public Task SendManialinkAsync(IPlayer player, string name, IDictionary<string, object?> data);
+
+    /// <summary>
+    /// Render a template and send it to a specific player.
+    /// </summary>
+    /// <param name="player">The player to send to.</param>
+    /// <param name="name">The name of the template.</param>
+    /// <param name="data">Data which the template uses</param>
+    /// <returns></returns>
+    public Task SendManialinkAsync(IPlayer player, string name, dynamic data);
+
+    /// <summary>
+    /// Render a template and send it to a specific player.
+    /// </summary>
+    /// <param name="playerLogin">The login to send to.</param>
+    /// <param name="name">The name of the template.</param>
+    /// <param name="data">Data which the template uses</param>
+    /// <returns></returns>
+    public Task SendManialinkAsync(string playerLogin, string name, dynamic data);
+
+    /// <summary>
+    /// Render a template and send it to a specific player without data.
+    /// </summary>
+    /// <param name="player">The player to send to.</param>
+    /// <param name="name">The name of the template.</param>
+    /// <returns></returns>
+    public Task SendManialinkAsync(IPlayer player, string name) => SendManialinkAsync(player, name, new { });
+    
+    /// <summary>
+    /// Render a template and send it to a set of players.
+    /// </summary>
+    /// <param name="players">The players to send to</param>
+    /// <param name="name">The name of the template.</param>
+    /// <param name="data">Data which the template uses</param>
+    /// <returns></returns>
+    public Task SendManialinkAsync(IEnumerable<IPlayer> players, string name, IDictionary<string, object?> data);
+    
+    /// <summary>
+    /// Render a template and send it to a set of players.
+    /// </summary>
+    /// <param name="players">The players to send to</param>
+    /// <param name="name">The name of the template.</param>
+    /// <param name="data">Data which the template uses</param>
+    /// <returns></returns>
+    public Task SendManialinkAsync(IEnumerable<IPlayer> players, string name, dynamic data);
+    
+    /// <summary>
+    /// Render a template and send it to a set of players without data.
+    /// </summary>
+    /// <param name="players">The players to send to</param>
+    /// <param name="name">The name of the template.</param>
+    /// <returns></returns>
+    public Task SendManialinkAsync(IEnumerable<IPlayer> players, string name) =>
+        SendManialinkAsync(players, name, new { });
+
+    /// <summary>
+    /// Hide a manialink from all players.
+    /// </summary>
+    /// <param name="name">Name of the manialink to hide.</param>
+    /// <returns></returns>
+    public Task HideManialinkAsync(string name);
+    
+    /// <summary>
+    /// Hide a manialink from a player.
+    /// </summary>
+    /// <param name="player">The player to hide the manialink from.</param>
+    /// <param name="name">Name of the manialink to hide.</param>
+    /// <returns></returns>
+    public Task HideManialinkAsync(IPlayer player, string name);
+    
+    /// <summary>
+    /// Hide a manialink from a player.
+    /// </summary>
+    /// <param name="playerLogin">The player to hide the manialink from.</param>
+    /// <param name="name">Name of the manialink to hide.</param>
+    /// <returns></returns>
+    public Task HideManialinkAsync(string playerLogin, string name);
+    
+    /// <summary>
+    /// Hide a manialink from a set of players.
+    /// </summary>
+    /// <param name="players">The players to hide the manialink from.</param>
+    /// <param name="name">Name of the manialink to hide.</param>
+    /// <returns></returns>
+    public Task HideManialinkAsync(IEnumerable<IPlayer> players, string name);
+}
diff --git a/src/EvoSC.Manialinks/Interfaces/IManialinkTransaction.cs b/src/EvoSC.Manialinks/Interfaces/IManialinkTransaction.cs
new file mode 100644
index 000000000..164cf1155
--- /dev/null
+++ b/src/EvoSC.Manialinks/Interfaces/IManialinkTransaction.cs
@@ -0,0 +1,6 @@
+namespace EvoSC.Manialinks.Interfaces;
+
+public interface IManialinkTransaction
+{
+    public Task CommitAsync();
+}
diff --git a/src/EvoSC.Manialinks/ManialinkManager.cs b/src/EvoSC.Manialinks/ManialinkManager.cs
index a2d5e7f50..c15d3a542 100644
--- a/src/EvoSC.Manialinks/ManialinkManager.cs
+++ b/src/EvoSC.Manialinks/ManialinkManager.cs
@@ -1,10 +1,7 @@
 using System.Collections.Concurrent;
 using System.ComponentModel;
-using System.Diagnostics;
 using System.Dynamic;
 using System.Reflection;
-using System.Text;
-using System.Text.Json.Serialization;
 using EvoSC.Common.Events;
 using EvoSC.Common.Interfaces;
 using EvoSC.Common.Interfaces.Models;
@@ -23,10 +20,7 @@
 using GbxRemoteNet;
 using GbxRemoteNet.Events;
 using ManiaTemplates;
-using ManiaTemplates.Lib;
 using Microsoft.Extensions.Logging;
-using Newtonsoft.Json;
-using JsonSerializer = System.Text.Json.JsonSerializer;
 
 namespace EvoSC.Manialinks;
 
@@ -266,6 +260,13 @@ public async Task SendPersistentManialinkAsync(string name, Func<Task<IDictionar
         await _server.Remote.SendDisplayManialinkPageAsync(manialinkOutput, 0, false);
     }
 
+    public Task RemovePersistentManialinkAsync(string name)
+    {
+        name = GetEffectiveName(name);
+        _persistentManialinks.TryRemove(name, out _);
+        return Task.CompletedTask;
+    }
+
     public async Task SendManialinkAsync(IPlayer player, string name, IDictionary<string, object?> data)
     {
         name = GetEffectiveName(name);
@@ -307,7 +308,7 @@ public Task HideManialinkAsync(string name)
     {
         name = GetEffectiveName(name);
         _persistentManialinks.TryRemove(name, out _);
-        var manialinkOutput = CreateHideManialink(name);
+        var manialinkOutput = ManialinkUtils.CreateHideManialink(name);
         return _server.Remote.SendDisplayManialinkPageAsync(manialinkOutput, 3, true);
     }
 
@@ -315,21 +316,21 @@ public Task HideManialinkAsync(IPlayer player, string name)
     {
         name = GetEffectiveName(name);
         _persistentManialinks.TryRemove(name, out _);
-        var manialinkOutput = CreateHideManialink(name);
+        var manialinkOutput = ManialinkUtils.CreateHideManialink(name);
         return _server.Remote.SendDisplayManialinkPageToLoginAsync(player.GetLogin(), manialinkOutput, 3, true);
     }
 
     public Task HideManialinkAsync(string playerLogin, string name)
     {
         name = GetEffectiveName(name);
-        var manialinkOutput = CreateHideManialink(name);
+        var manialinkOutput = ManialinkUtils.CreateHideManialink(name);
         return _server.Remote.SendDisplayManialinkPageToLoginAsync(playerLogin, manialinkOutput, 3, true);
     }
 
     public Task HideManialinkAsync(IEnumerable<IPlayer> players, string name)
     {
         name = GetEffectiveName(name);
-        var manialinkOutput = CreateHideManialink(name);
+        var manialinkOutput = ManialinkUtils.CreateHideManialink(name);
         var multiCall = new MultiCall();
 
         foreach (var player in players)
@@ -467,32 +468,22 @@ private IEnumerable<Assembly> PrepareRender(string name)
         return assemblies;
     }
 
-    private async Task<string> PrepareAndRenderAsync(string name, IDictionary<string, object?> data)
+    public async Task<string> PrepareAndRenderAsync(string name, IDictionary<string, object?> data)
     {
         var assemblies = PrepareRender(name);
         return await _engine.RenderAsync(name, data, assemblies);
     }
     
-    private async Task<string> PrepareAndRenderAsync(string name, dynamic data)
+    public async Task<string> PrepareAndRenderAsync(string name, dynamic data)
     {
         var assemblies = PrepareRender(name);
         return await _engine.RenderAsync(name, data, assemblies);
     }
-    
-    private string CreateHideManialink(string name)
-    {
-        var sb = new StringBuilder()
-            .Append("<?xml version=\"1.0\" encoding=\"utf-8\" standalone=\"yes\" ?>\n")
-            .Append("<manialink version=\"3\" id=\"")
-            .Append(ManialinkNameUtils.KeyToId(name))
-            .Append("\">\n")
-            .Append("</manialink>\n");
 
-        return sb.ToString();
-    }
-
-    private string GetEffectiveName(string name) =>
+    public string GetEffectiveName(string name) =>
         _themeManager.ComponentReplacements.TryGetValue(name, out var effectiveName)
             ? effectiveName
             : name;
+
+    public ManialinkTransaction CreateTransaction() => new(this, _server);
 }
diff --git a/src/EvoSC.Manialinks/ManialinkTransaction.cs b/src/EvoSC.Manialinks/ManialinkTransaction.cs
new file mode 100644
index 000000000..1ffb0c65e
--- /dev/null
+++ b/src/EvoSC.Manialinks/ManialinkTransaction.cs
@@ -0,0 +1,122 @@
+using EvoSC.Common.Interfaces;
+using EvoSC.Common.Interfaces.Models;
+using EvoSC.Common.Util;
+using EvoSC.Manialinks.Interfaces;
+using EvoSC.Manialinks.Util;
+using GbxRemoteNet;
+using GbxRemoteNet.Interfaces;
+
+namespace EvoSC.Manialinks;
+
+public class ManialinkTransaction(IManialinkManager manialinkManager, IServerClient server) : IManialinkOperations, IManialinkTransaction
+{
+    private readonly MultiCall _serverCalls = new MultiCall();
+    private readonly HashSet<string> _persistentRemovals = [];
+
+    public Task SendManialinkAsync(string name, IDictionary<string, object?> data) => AddSendCallAsync(name, data);
+
+    public Task SendManialinkAsync(string name, dynamic data) => AddSendCallAsync(name, data);
+
+    public Task SendManialinkAsync(string name) => AddSendCallAsync(name, new { });
+
+    public Task SendManialinkAsync(IPlayer player, string name, IDictionary<string, object?> data) => AddSendCallAsync(player, name, data);
+
+    public Task SendManialinkAsync(IPlayer player, string name, dynamic data) => AddSendCallAsync(player, name, data);
+
+    public Task SendManialinkAsync(string playerLogin, string name, dynamic data) => AddSendCallAsync(playerLogin, name, data);
+
+    public async Task SendManialinkAsync(IEnumerable<IPlayer> players, string name, IDictionary<string, object?> data)
+    {
+        foreach (var player in players)
+        {
+            await AddSendCallAsync(player, name, data);
+        }
+    }
+
+    public async Task SendManialinkAsync(IEnumerable<IPlayer> players, string name, dynamic data)
+    {
+        foreach (var player in players)
+        {
+            await AddSendCallAsync(player, name, data);
+        }
+    }
+
+    public Task HideManialinkAsync(string name) => AddHideCallAsync(name);
+
+    public Task HideManialinkAsync(IPlayer player, string name) => AddHideCallAsync(player, name);
+
+    public Task HideManialinkAsync(string playerLogin, string name) => AddHideCallAsync(playerLogin, name);
+
+    public async Task HideManialinkAsync(IEnumerable<IPlayer> players, string name)
+    {
+        foreach (var player in players)
+        {
+            await AddHideCallAsync(player, name);
+        }
+    }
+
+    private async Task AddSendCallAsync(string name, IDictionary<string, object?> data)
+    {
+        var output = await GetSendOutputAsync(name, data);
+        _serverCalls.Add(nameof(IGbxRemoteClient.SendDisplayManialinkPageAsync), output, 0, false);
+    }
+    
+    private async Task AddSendCallAsync(string name, dynamic data)
+    {
+        var output = await GetSendOutputAsync(name, data);
+        _serverCalls.Add(nameof(IGbxRemoteClient.SendDisplayManialinkPageAsync), output, 0, false);
+    }
+    
+    private async Task AddSendCallAsync(string playerLogin, string name, IDictionary<string, object?> data)
+    {
+        var output = await GetSendOutputAsync(name, data);
+        _serverCalls.Add(nameof(IGbxRemoteClient.SendDisplayManialinkPageToLoginAsync), playerLogin, output, 0, false);
+    }
+    
+    private async Task AddSendCallAsync(string playerLogin, string name, dynamic data)
+    {
+        var output = await GetSendOutputAsync(name, data);
+        _serverCalls.Add(nameof(IGbxRemoteClient.SendDisplayManialinkPageToLoginAsync), playerLogin, output, 0, false);
+    }
+
+    private Task AddSendCallAsync(IPlayer player, string name, IDictionary<string, object?> data) =>
+        AddSendCallAsync(player.GetLogin(), name, data);
+    
+    private Task AddSendCallAsync(IPlayer player, string name, dynamic data) =>
+        AddSendCallAsync(player.GetLogin(), name, data);
+
+    private Task AddHideCallAsync(string name)
+    {
+        var output = PrepareAndGetHideCall(name);
+        _serverCalls.Add(nameof(IGbxRemoteClient.SendDisplayManialinkPageAsync), output, 3, true);
+        return Task.CompletedTask;
+    }
+    
+    private Task AddHideCallAsync(string playerLogin, string name)
+    {
+        var output = PrepareAndGetHideCall(name);
+        _serverCalls.Add(nameof(IGbxRemoteClient.SendDisplayManialinkPageToLoginAsync), playerLogin, output, 3, true);
+        return Task.CompletedTask;
+    }
+
+    private Task AddHideCallAsync(IPlayer player, string name) => AddHideCallAsync(player.GetLogin(), name);
+
+    private async Task<string> GetSendOutputAsync(string name, IDictionary<string, object?> data)
+    {
+        return await manialinkManager.PrepareAndRenderAsync(manialinkManager.GetEffectiveName(name), data);
+    }
+    
+    private async Task<string> GetSendOutputAsync(string name, dynamic data)
+    {
+        return await manialinkManager.PrepareAndRenderAsync(manialinkManager.GetEffectiveName(name), data);
+    }
+
+    private string PrepareAndGetHideCall(string name)
+    {
+        name = manialinkManager.GetEffectiveName(name);
+        _persistentRemovals.Add(name);
+        return ManialinkUtils.CreateHideManialink(name);
+    }
+
+    public Task CommitAsync() => server.Remote.MultiCallAsync(_serverCalls);
+}
diff --git a/src/EvoSC.Manialinks/Util/ManialinkUtils.cs b/src/EvoSC.Manialinks/Util/ManialinkUtils.cs
new file mode 100644
index 000000000..26c024fb1
--- /dev/null
+++ b/src/EvoSC.Manialinks/Util/ManialinkUtils.cs
@@ -0,0 +1,19 @@
+using System.Text;
+using ManiaTemplates.Lib;
+
+namespace EvoSC.Manialinks.Util;
+
+public static class ManialinkUtils
+{
+    public static string CreateHideManialink(string name)
+    {
+        var sb = new StringBuilder()
+            .Append("<?xml version=\"1.0\" encoding=\"utf-8\" standalone=\"yes\" ?>\n")
+            .Append("<manialink version=\"3\" id=\"")
+            .Append(ManialinkNameUtils.KeyToId(name))
+            .Append("\">\n")
+            .Append("</manialink>\n");
+
+        return sb.ToString();
+    }
+}
diff --git a/src/Modules/LocalRecordsModule/Controllers/WidgetUpdateController.cs b/src/Modules/LocalRecordsModule/Controllers/WidgetUpdateController.cs
new file mode 100644
index 000000000..efd92e7dd
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Controllers/WidgetUpdateController.cs
@@ -0,0 +1,19 @@
+using EvoSC.Common.Controllers;
+using EvoSC.Common.Controllers.Attributes;
+using EvoSC.Common.Events.Attributes;
+using EvoSC.Common.Interfaces.Controllers;
+using EvoSC.Common.Remote;
+using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
+using GbxRemoteNet.Events;
+
+namespace EvoSC.Modules.Official.LocalRecordsModule.Controllers;
+
+[Controller]
+public class WidgetUpdateController(ILocalRecordsService localRecords) : EvoScController<IEventControllerContext>
+{
+    [Subscribe(GbxRemoteEvent.PlayerConnect)]
+    public Task OnPlayerConnectAsync(object sender, PlayerConnectGbxEventArgs args)
+    {
+        return Task.CompletedTask;
+    }
+}
diff --git a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
index 05d3646a4..aef4dbe88 100644
--- a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
@@ -1,6 +1,10 @@
-namespace EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
+using EvoSC.Common.Interfaces.Models;
+
+namespace EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
 
 public interface ILocalRecordsService
 {
     public Task<IEnumerable<ILocalRecord>> GetLocalsOfCurrentMapAsync();
+    public Task ShowWidgetAsync(IPlayer player);
+    public Task ShowWidgetToAllAsync();
 }
diff --git a/src/Modules/LocalRecordsModule/LocalRecordsModule.cs b/src/Modules/LocalRecordsModule/LocalRecordsModule.cs
index 25c9b25c4..e3e7222e9 100644
--- a/src/Modules/LocalRecordsModule/LocalRecordsModule.cs
+++ b/src/Modules/LocalRecordsModule/LocalRecordsModule.cs
@@ -1,3 +1,4 @@
+using EvoSC.Common.Interfaces.Services;
 using EvoSC.Manialinks.Interfaces;
 using EvoSC.Modules.Attributes;
 using EvoSC.Modules.Interfaces;
@@ -8,13 +9,7 @@ namespace EvoSC.Modules.Official.LocalRecordsModule;
 [Module(IsInternal = true)]
 public class LocalRecordsModule(IManialinkManager manialinkManager, ILocalRecordsService localRecordsService) : EvoScModule, IToggleable
 {
-    public Task EnableAsync() =>
-        manialinkManager.SendPersistentManialinkAsync("LocalRecordsModule.LocalRecordsWidget",
-            async () =>
-            {
-                var records = await localRecordsService.GetLocalsOfCurrentMapAsync();
-                return new { records };
-            });
+    public Task EnableAsync() => localRecordsService.ShowWidgetToAllAsync();
 
     public Task DisableAsync() => manialinkManager.HideManialinkAsync("LocalRecordsModule.LocalRecordsWidget");
 }
diff --git a/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj b/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj
index 09e8e8dc9..d9b3074db 100644
--- a/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj
+++ b/src/Modules/LocalRecordsModule/LocalRecordsModule.csproj
@@ -22,8 +22,5 @@
         <ProjectReference Include="../../EvoSC.Modules/EvoSC.Modules.csproj" Private="true" />
         <ProjectReference Include="..\PlayerRecords\PlayerRecords.csproj" />
   </ItemGroup>
-  <ItemGroup>
-    <Folder Include="Controllers\" />
-  </ItemGroup>
   
   </Project>
diff --git a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
index ab269a42c..468484501 100644
--- a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
@@ -1,13 +1,22 @@
-using EvoSC.Common.Interfaces.Services;
+using EvoSC.Common.Interfaces.Models;
+using EvoSC.Common.Interfaces.Services;
 using EvoSC.Common.Services.Attributes;
+using EvoSC.Manialinks.Interfaces;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Database;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
+using GbxRemoteNet;
+using Microsoft.Extensions.Logging;
 
 namespace EvoSC.Modules.Official.LocalRecordsModule.Services;
 
 [Service]
-public class LocalRecordsService(IMapService mapService, ILocalRecordRepository localRecordRepository) : ILocalRecordsService
+public class LocalRecordsService(
+    IMapService mapService,
+    ILocalRecordRepository localRecordRepository,
+    IPlayerManagerService playerManagerService,
+    IManialinkManager manialinkManager,
+    ILogger<LocalRecordsService> logger) : ILocalRecordsService
 {
     public async Task<IEnumerable<ILocalRecord>> GetLocalsOfCurrentMapAsync()
     {
@@ -17,7 +26,37 @@ public async Task<IEnumerable<ILocalRecord>> GetLocalsOfCurrentMapAsync()
         {
             throw new InvalidOperationException("Failed to get current map");
         }
-        
+
         return await localRecordRepository.GetLocalRecordsOfMapByIdAsync(currentMap.Id);
     }
+
+    public Task ShowWidgetAsync(IPlayer player)
+    {
+        throw new NotImplementedException();
+    }
+
+    public async Task ShowWidgetToAllAsync()
+    {
+        var records = await GetLocalsOfCurrentMapAsync();
+        var onlinePlayers = await playerManagerService.GetOnlinePlayersAsync();
+        var transaction = manialinkManager.CreateTransaction();
+
+        try
+        {
+            foreach (var player in onlinePlayers)
+            {
+                await transaction.SendManialinkAsync(player, "LocalRecordsModule.LocalRecordsWidget", new
+                {
+                    currentPlayer = player,
+                    records
+                });
+            }
+            
+            await transaction.CommitAsync();
+        }
+        catch (Exception ex)
+        {
+            logger.LogError(ex, "Failed to send local records widget");
+        }
+    }
 }

From b5d8971f4e0aa8979c07138e3d5318c0aef02af8 Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Sat, 22 Jun 2024 13:56:36 +0200
Subject: [PATCH 09/18] show local records of a map

---
 .../Controllers/WidgetUpdateController.cs           | 12 +++++++++---
 .../Database/Repository/LocalRecordRepository.cs    |  2 +-
 .../Services/LocalRecordsService.cs                 | 13 ++++++-------
 .../Templates/LocalRecordsWidget.mt                 | 10 ++++++----
 4 files changed, 22 insertions(+), 15 deletions(-)

diff --git a/src/Modules/LocalRecordsModule/Controllers/WidgetUpdateController.cs b/src/Modules/LocalRecordsModule/Controllers/WidgetUpdateController.cs
index efd92e7dd..5497fe09a 100644
--- a/src/Modules/LocalRecordsModule/Controllers/WidgetUpdateController.cs
+++ b/src/Modules/LocalRecordsModule/Controllers/WidgetUpdateController.cs
@@ -2,18 +2,24 @@
 using EvoSC.Common.Controllers.Attributes;
 using EvoSC.Common.Events.Attributes;
 using EvoSC.Common.Interfaces.Controllers;
+using EvoSC.Common.Interfaces.Services;
 using EvoSC.Common.Remote;
+using EvoSC.Common.Util;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
 using GbxRemoteNet.Events;
 
 namespace EvoSC.Modules.Official.LocalRecordsModule.Controllers;
 
 [Controller]
-public class WidgetUpdateController(ILocalRecordsService localRecords) : EvoScController<IEventControllerContext>
+public class WidgetUpdateController(ILocalRecordsService localRecords, IPlayerManagerService playerManagerService) : EvoScController<IEventControllerContext>
 {
     [Subscribe(GbxRemoteEvent.PlayerConnect)]
-    public Task OnPlayerConnectAsync(object sender, PlayerConnectGbxEventArgs args)
+    public async Task OnPlayerConnectAsync(object sender, PlayerConnectGbxEventArgs args)
     {
-        return Task.CompletedTask;
+        var player = await playerManagerService.GetOnlinePlayerAsync(PlayerUtils.ConvertLoginToAccountId(args.Login));
+        await localRecords.ShowWidgetAsync(player);
     }
+
+    [Subscribe(GbxRemoteEvent.BeginMap)]
+    public Task OnBeginMapAsync(object sender, MapGbxEventArgs args) => localRecords.ShowWidgetToAllAsync();
 }
diff --git a/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs b/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
index 3f46b4062..22208fabd 100644
--- a/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
+++ b/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
@@ -1,5 +1,4 @@
 using EvoSC.Common.Database.Models.Maps;
-using EvoSC.Common.Database.Models.Player;
 using EvoSC.Common.Database.Repository;
 using EvoSC.Common.Interfaces.Database;
 using EvoSC.Common.Interfaces.Models;
@@ -21,6 +20,7 @@ public class LocalRecordRepository(IDbConnectionFactory dbConnFactory, IPlayerRe
     public async Task<IEnumerable<DbLocalRecord>> GetLocalRecordsOfMapByIdAsync(long mapId) =>
         await NewLoadAll()
             .Where(r => r.DbMap.Id == mapId)
+            .OrderBy(r => r.Position)
             .ToArrayAsync();
 
     public async Task<DbLocalRecord> AddOrUpdateRecordAsync(IMap map, IPlayerRecord record)
diff --git a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
index 468484501..ed453f607 100644
--- a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
@@ -18,6 +18,8 @@ public class LocalRecordsService(
     IManialinkManager manialinkManager,
     ILogger<LocalRecordsService> logger) : ILocalRecordsService
 {
+    private const string WidgetName = "LocalRecordsModule.LocalRecordsWidget";
+    
     public async Task<IEnumerable<ILocalRecord>> GetLocalsOfCurrentMapAsync()
     {
         var currentMap = await mapService.GetCurrentMapAsync();
@@ -30,9 +32,10 @@ public async Task<IEnumerable<ILocalRecord>> GetLocalsOfCurrentMapAsync()
         return await localRecordRepository.GetLocalRecordsOfMapByIdAsync(currentMap.Id);
     }
 
-    public Task ShowWidgetAsync(IPlayer player)
+    public async Task ShowWidgetAsync(IPlayer player)
     {
-        throw new NotImplementedException();
+        var records = await GetLocalsOfCurrentMapAsync();
+        await manialinkManager.SendManialinkAsync(player, WidgetName, new { currentPlayer = player, records });
     }
 
     public async Task ShowWidgetToAllAsync()
@@ -45,11 +48,7 @@ public async Task ShowWidgetToAllAsync()
         {
             foreach (var player in onlinePlayers)
             {
-                await transaction.SendManialinkAsync(player, "LocalRecordsModule.LocalRecordsWidget", new
-                {
-                    currentPlayer = player,
-                    records
-                });
+                await transaction.SendManialinkAsync(player, WidgetName, new { currentPlayer = player, records });
             }
             
             await transaction.CommitAsync();
diff --git a/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt b/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
index 1f41f0ab1..110a3eb68 100644
--- a/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
+++ b/src/Modules/LocalRecordsModule/Templates/LocalRecordsWidget.mt
@@ -1,4 +1,5 @@
 <component>
+  <using namespace="EvoSC.Common.Interfaces.Models" />
   <using namespace="EvoSC.Common.Interfaces.Util" />
   <using namespace="EvoSC.Common.Util" />
   <using namespace="EvoSC.Modules.Official.LocalRecordsModule.Interfaces" />
@@ -12,6 +13,7 @@
   <import component="LocalRecordsModule.Style.LocalRecordsStyle" as="LocalRecordsStyle" />
   
   <property type="IEnumerable<ILocalRecord>" name="records" />
+  <property type="IPlayer" name="currentPlayer" />
   
   <template>
     <UIStyle />
@@ -19,12 +21,12 @@
       <Widget header="local records" height="20" bodyStyle="unstyled" y="80">
       <template slot="body">
         <RecordRow 
-                pos="{{ __index + 140 }}"
-                name="snixtho" 
+                pos="{{ record.Position }}"
+                name="{{ record.Record.Player.NickName }}" 
                 y="{{ -__index*(4+0.3) }}" 
                 foreach="ILocalRecord record in records"
-                time="{{ RaceTime.FromMilliseconds(45236) }}" 
-                self="{{ __index == 6 }}"
+                time="{{ RaceTime.FromMilliseconds(record.Record.Score) }}" 
+                self="{{ record.Record.Player.Id == currentPlayer.Id }}"
         />
         
         <frame if="records.Count() == 0">

From a5fd8b249745d14987db3504baf7d939e047626b Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Sat, 22 Jun 2024 17:36:13 +0200
Subject: [PATCH 10/18] show local records based on player's position

---
 src/EvoSC.Manialinks/ManialinkController.cs   |  2 +-
 .../Config/ILocalRecordsSettings.cs           | 17 +++++++
 .../Services/ILocalRecordsService.cs          |  2 +-
 .../Services/LocalRecordsService.cs           | 47 ++++++++++++++++---
 4 files changed, 59 insertions(+), 9 deletions(-)
 create mode 100644 src/Modules/LocalRecordsModule/Config/ILocalRecordsSettings.cs

diff --git a/src/EvoSC.Manialinks/ManialinkController.cs b/src/EvoSC.Manialinks/ManialinkController.cs
index ac67a0459..8e6e055bd 100644
--- a/src/EvoSC.Manialinks/ManialinkController.cs
+++ b/src/EvoSC.Manialinks/ManialinkController.cs
@@ -30,7 +30,7 @@ public Task ShowAsync(IEnumerable<IOnlinePlayer> players, string maniaLink) =>
 
     public Task ShowAsync(IEnumerable<IOnlinePlayer> players, string maniaLink, object data) =>
         Context.ManialinkManager.SendManialinkAsync(players, maniaLink, PrepareManialinkData(data));
-
+    
     public Task ShowPersistentAsync(string name) => Context.ManialinkManager.SendPersistentManialinkAsync(name, PrepareManialinkData(new object()));
     
     public Task ShowPersistentAsync(string name, object data) => Context.ManialinkManager.SendPersistentManialinkAsync(name, PrepareManialinkData(data));
diff --git a/src/Modules/LocalRecordsModule/Config/ILocalRecordsSettings.cs b/src/Modules/LocalRecordsModule/Config/ILocalRecordsSettings.cs
new file mode 100644
index 000000000..bec5ecd9f
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Config/ILocalRecordsSettings.cs
@@ -0,0 +1,17 @@
+using System.ComponentModel;
+using Config.Net;
+using EvoSC.Common.Config.Models;
+using EvoSC.Modules.Attributes;
+
+namespace EvoSC.Modules.Official.LocalRecordsModule.Config;
+
+[Settings]
+public interface ILocalRecordsSettings
+{
+    [Option(DefaultValue = 10), Description("Max of rows to show in the local records widget.")]
+    public int MaxWidgetRows { get; }
+    
+    [Option(DefaultValue = 3), Description("Always show top N players in the widget.")]
+    public int WidgetShowTop { get; }
+}
+
diff --git a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
index aef4dbe88..00f73e338 100644
--- a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
@@ -4,7 +4,7 @@ namespace EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
 
 public interface ILocalRecordsService
 {
-    public Task<IEnumerable<ILocalRecord>> GetLocalsOfCurrentMapAsync();
+    public Task<IEnumerable<ILocalRecord>> GetLocalsOfCurrentMapFromPosAsync();
     public Task ShowWidgetAsync(IPlayer player);
     public Task ShowWidgetToAllAsync();
 }
diff --git a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
index ed453f607..ef0cdf2f7 100644
--- a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
@@ -2,6 +2,7 @@
 using EvoSC.Common.Interfaces.Services;
 using EvoSC.Common.Services.Attributes;
 using EvoSC.Manialinks.Interfaces;
+using EvoSC.Modules.Official.LocalRecordsModule.Config;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Database;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
@@ -16,11 +17,12 @@ public class LocalRecordsService(
     ILocalRecordRepository localRecordRepository,
     IPlayerManagerService playerManagerService,
     IManialinkManager manialinkManager,
-    ILogger<LocalRecordsService> logger) : ILocalRecordsService
+    ILogger<LocalRecordsService> logger,
+    ILocalRecordsSettings settings) : ILocalRecordsService
 {
     private const string WidgetName = "LocalRecordsModule.LocalRecordsWidget";
     
-    public async Task<IEnumerable<ILocalRecord>> GetLocalsOfCurrentMapAsync()
+    public async Task<IEnumerable<ILocalRecord>> GetLocalsOfCurrentMapFromPosAsync()
     {
         var currentMap = await mapService.GetCurrentMapAsync();
 
@@ -28,19 +30,23 @@ public async Task<IEnumerable<ILocalRecord>> GetLocalsOfCurrentMapAsync()
         {
             throw new InvalidOperationException("Failed to get current map");
         }
-
+        
         return await localRecordRepository.GetLocalRecordsOfMapByIdAsync(currentMap.Id);
     }
 
     public async Task ShowWidgetAsync(IPlayer player)
     {
-        var records = await GetLocalsOfCurrentMapAsync();
-        await manialinkManager.SendManialinkAsync(player, WidgetName, new { currentPlayer = player, records });
+        var records = await GetLocalsOfCurrentMapFromPosAsync();
+        var topRecords = records.Take(Math.Min(settings.WidgetShowTop, settings.MaxWidgetRows));
+        var playerRecords = GetRecordsForPlayer(records, topRecords, player);
+        await manialinkManager.SendManialinkAsync(player, WidgetName,
+            new { currentPlayer = player, records = playerRecords });
     }
 
     public async Task ShowWidgetToAllAsync()
     {
-        var records = await GetLocalsOfCurrentMapAsync();
+        var records = await GetLocalsOfCurrentMapFromPosAsync();
+        var topRecords = records.Take(Math.Min(settings.WidgetShowTop, settings.MaxWidgetRows));
         var onlinePlayers = await playerManagerService.GetOnlinePlayersAsync();
         var transaction = manialinkManager.CreateTransaction();
 
@@ -48,7 +54,8 @@ public async Task ShowWidgetToAllAsync()
         {
             foreach (var player in onlinePlayers)
             {
-                await transaction.SendManialinkAsync(player, WidgetName, new { currentPlayer = player, records });
+                var playerRecords = GetRecordsForPlayer(records, topRecords, player);
+                await transaction.SendManialinkAsync(player, WidgetName, new { currentPlayer = player, records = playerRecords });
             }
             
             await transaction.CommitAsync();
@@ -58,4 +65,30 @@ public async Task ShowWidgetToAllAsync()
             logger.LogError(ex, "Failed to send local records widget");
         }
     }
+
+    private IEnumerable<ILocalRecord> GetRecordsForPlayer(IEnumerable<ILocalRecord> records, IEnumerable<ILocalRecord> topRecords, IPlayer player)
+    {
+        var recordsAroundPlayer = GetRecordsAroundPlayer(player, records);
+        return topRecords.UnionBy(recordsAroundPlayer, r => r.Position);
+    }
+    
+    private IEnumerable<ILocalRecord> GetRecordsAroundPlayer(IPlayer player, IEnumerable<ILocalRecord> records)
+    {
+        var maxAroundRange = Math.Max(settings.MaxWidgetRows - settings.WidgetShowTop, 0);
+        var nRecords = records.Count();
+        var playerRecord = records.FirstOrDefault(r => r.Record.Player.Id == player.Id);
+        var position = playerRecord?.Position ?? 0;
+
+        if (playerRecord?.Position > 0)
+        {
+            var lower = Math.Max(position - maxAroundRange / 2, 0);
+            var left = Math.Max(maxAroundRange - (nRecords - lower), 0);
+            var intersect = Math.Max(settings.WidgetShowTop - lower, 0);
+            return records
+                .Skip(Math.Min(Math.Max(lower - left, 0), nRecords - 1))
+                .Take(maxAroundRange + intersect);
+        }
+
+        return Array.Empty<ILocalRecord>();
+    }
 }

From 3e6565700eeec3f21481e3f6b5e7d1c82f888d9e Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Sun, 23 Jun 2024 12:59:34 +0200
Subject: [PATCH 11/18] fix local records showing wrong in certain cases

---
 .../Services/ILocalRecordsService.cs          |  2 +-
 .../Services/LocalRecordsService.cs           | 68 ++++++++++++-------
 2 files changed, 44 insertions(+), 26 deletions(-)

diff --git a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
index 00f73e338..f46a2aad7 100644
--- a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
@@ -4,7 +4,7 @@ namespace EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
 
 public interface ILocalRecordsService
 {
-    public Task<IEnumerable<ILocalRecord>> GetLocalsOfCurrentMapFromPosAsync();
+    public Task<ILocalRecord[]> GetLocalsOfCurrentMapFromPosAsync();
     public Task ShowWidgetAsync(IPlayer player);
     public Task ShowWidgetToAllAsync();
 }
diff --git a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
index ef0cdf2f7..ca8e5a2ae 100644
--- a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
@@ -22,7 +22,7 @@ public class LocalRecordsService(
 {
     private const string WidgetName = "LocalRecordsModule.LocalRecordsWidget";
     
-    public async Task<IEnumerable<ILocalRecord>> GetLocalsOfCurrentMapFromPosAsync()
+    public async Task<ILocalRecord[]> GetLocalsOfCurrentMapFromPosAsync()
     {
         var currentMap = await mapService.GetCurrentMapAsync();
 
@@ -30,15 +30,15 @@ public async Task<IEnumerable<ILocalRecord>> GetLocalsOfCurrentMapFromPosAsync()
         {
             throw new InvalidOperationException("Failed to get current map");
         }
-        
-        return await localRecordRepository.GetLocalRecordsOfMapByIdAsync(currentMap.Id);
+
+        var records = (IEnumerable<ILocalRecord>)await localRecordRepository.GetLocalRecordsOfMapByIdAsync(currentMap.Id);
+        return records.ToArray();
     }
 
     public async Task ShowWidgetAsync(IPlayer player)
     {
         var records = await GetLocalsOfCurrentMapFromPosAsync();
-        var topRecords = records.Take(Math.Min(settings.WidgetShowTop, settings.MaxWidgetRows));
-        var playerRecords = GetRecordsForPlayer(records, topRecords, player);
+        var playerRecords = GetRecordsWithPlayer(player, records);
         await manialinkManager.SendManialinkAsync(player, WidgetName,
             new { currentPlayer = player, records = playerRecords });
     }
@@ -46,7 +46,6 @@ await manialinkManager.SendManialinkAsync(player, WidgetName,
     public async Task ShowWidgetToAllAsync()
     {
         var records = await GetLocalsOfCurrentMapFromPosAsync();
-        var topRecords = records.Take(Math.Min(settings.WidgetShowTop, settings.MaxWidgetRows));
         var onlinePlayers = await playerManagerService.GetOnlinePlayersAsync();
         var transaction = manialinkManager.CreateTransaction();
 
@@ -54,7 +53,7 @@ public async Task ShowWidgetToAllAsync()
         {
             foreach (var player in onlinePlayers)
             {
-                var playerRecords = GetRecordsForPlayer(records, topRecords, player);
+                var playerRecords = GetRecordsWithPlayer(player, records);
                 await transaction.SendManialinkAsync(player, WidgetName, new { currentPlayer = player, records = playerRecords });
             }
             
@@ -66,29 +65,48 @@ public async Task ShowWidgetToAllAsync()
         }
     }
 
-    private IEnumerable<ILocalRecord> GetRecordsForPlayer(IEnumerable<ILocalRecord> records, IEnumerable<ILocalRecord> topRecords, IPlayer player)
-    {
-        var recordsAroundPlayer = GetRecordsAroundPlayer(player, records);
-        return topRecords.UnionBy(recordsAroundPlayer, r => r.Position);
-    }
-    
-    private IEnumerable<ILocalRecord> GetRecordsAroundPlayer(IPlayer player, IEnumerable<ILocalRecord> records)
+    private ILocalRecord[] GetRecordsWithPlayer(IPlayer player, ILocalRecord[] records)
     {
-        var maxAroundRange = Math.Max(settings.MaxWidgetRows - settings.WidgetShowTop, 0);
-        var nRecords = records.Count();
+        if (records.Length == 0)
+        {
+            return records;
+        }
+        
         var playerRecord = records.FirstOrDefault(r => r.Record.Player.Id == player.Id);
-        var position = playerRecord?.Position ?? 0;
+        var topMaxRows = Math.Min(settings.MaxWidgetRows, records.Length);
+
+        // if player doesn't exist, just return the top players
+        if (playerRecord == null)
+        {
+            return records[..topMaxRows];
+        }
 
-        if (playerRecord?.Position > 0)
+        // if records around player overlaps with top players, just return first MaxRows records
+        if (playerRecord.Position <= settings.MaxWidgetRows)
         {
-            var lower = Math.Max(position - maxAroundRange / 2, 0);
-            var left = Math.Max(maxAroundRange - (nRecords - lower), 0);
-            var intersect = Math.Max(settings.WidgetShowTop - lower, 0);
-            return records
-                .Skip(Math.Min(Math.Max(lower - left, 0), nRecords - 1))
-                .Take(maxAroundRange + intersect);
+            return records[..topMaxRows];
+        }
+        
+        // return top records + records around the player
+        var topRecords = records[..Math.Min(settings.WidgetShowTop, records.Length)];
+
+        var maxRange = settings.MaxWidgetRows - settings.WidgetShowTop;
+        var half = maxRange / 2;
+        var r = maxRange % 2;
+
+        var lower = playerRecord.Position - half;
+        var upper = lower + maxRange;
+
+        // if the range is outside the upper bound, let's start from the last record instead
+        if (upper > records.Length)
+        {
+            lower -= upper - records.Length;
+            upper = records.Length;
+
+            lower++;
+            upper++;
         }
 
-        return Array.Empty<ILocalRecord>();
+        return [..topRecords, ..records[(lower - 1)..(upper - 1)]];
     }
 }

From 770d6cd5d8cb74583a89b1b87afe921e97317614 Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Sun, 23 Jun 2024 14:17:50 +0200
Subject: [PATCH 12/18] update local records when player makes a pb

---
 src/EvoSC.Common/Util/FormattingUtils.cs      |  4 +-
 .../Util/TextFormatting/FormattedText.cs      |  6 +-
 .../Config/ILocalRecordsSettings.cs           |  3 +
 .../Controllers/WidgetUpdateController.cs     |  4 ++
 .../Repository/LocalRecordRepository.cs       | 55 ++++++++++++----
 .../Database/ILocalRecordRepository.cs        |  4 +-
 .../Services/ILocalRecordsService.cs          |  2 +
 .../Services/LocalRecordsService.cs           | 65 ++++++++++++++++++-
 .../Controllers/QueueCommandsController.cs    |  2 +-
 .../Repository/PlayerRecordsRepository.cs     |  6 +-
 .../Interfaces/IPlayerRecordsRepository.cs    |  7 --
 .../Services/PlayerRecordsService.cs          |  8 ++-
 12 files changed, 130 insertions(+), 36 deletions(-)

diff --git a/src/EvoSC.Common/Util/FormattingUtils.cs b/src/EvoSC.Common/Util/FormattingUtils.cs
index 095abd297..475c4113f 100644
--- a/src/EvoSC.Common/Util/FormattingUtils.cs
+++ b/src/EvoSC.Common/Util/FormattingUtils.cs
@@ -49,9 +49,9 @@ public static TextFormatter FormatPlayerChatMessage(string nickname, string mess
     {
         var formattedMessage = new TextFormatter()
             .AddText("[")
-            .AddText(text => text.AsIsolated().AddText(nickname))
+            .AddText(text => text.AddText(nickname))
             .AddText("] ")
-            .AddText(text => text.AsIsolated().AddText(message));
+            .AddText(text => text.AddText(message));
 
         return formattedMessage;
     }
diff --git a/src/EvoSC.Common/Util/TextFormatting/FormattedText.cs b/src/EvoSC.Common/Util/TextFormatting/FormattedText.cs
index cf1e16423..f9d1c3720 100644
--- a/src/EvoSC.Common/Util/TextFormatting/FormattedText.cs
+++ b/src/EvoSC.Common/Util/TextFormatting/FormattedText.cs
@@ -9,7 +9,7 @@ namespace EvoSC.Common.Util.TextFormatting;
 public class FormattedText
 {
     public TextStyling? Style { get; set; }
-    public bool IsIsolated { get; set; }
+    public bool IsIsolated { get; set; } = true;
     public StringBuilder Text { get; set; } = new StringBuilder();
 
     public FormattedText(){}
@@ -127,9 +127,9 @@ public FormattedText WithStyle(Action<TextStyling> styleAction)
     /// interfere with later text.
     /// </summary>
     /// <returns></returns>
-    public FormattedText AsIsolated()
+    public FormattedText AsNotIsolated()
     {
-        IsIsolated = true;
+        IsIsolated = false;
         return this;
     }
 
diff --git a/src/Modules/LocalRecordsModule/Config/ILocalRecordsSettings.cs b/src/Modules/LocalRecordsModule/Config/ILocalRecordsSettings.cs
index bec5ecd9f..e77fe84f5 100644
--- a/src/Modules/LocalRecordsModule/Config/ILocalRecordsSettings.cs
+++ b/src/Modules/LocalRecordsModule/Config/ILocalRecordsSettings.cs
@@ -13,5 +13,8 @@ public interface ILocalRecordsSettings
     
     [Option(DefaultValue = 3), Description("Always show top N players in the widget.")]
     public int WidgetShowTop { get; }
+    
+    [Option(DefaultValue = 100), Description("Maximum number of local records to keep track of per map.")]
+    public int MaxRecordsPerMap { get; }
 }
 
diff --git a/src/Modules/LocalRecordsModule/Controllers/WidgetUpdateController.cs b/src/Modules/LocalRecordsModule/Controllers/WidgetUpdateController.cs
index 5497fe09a..0eb819641 100644
--- a/src/Modules/LocalRecordsModule/Controllers/WidgetUpdateController.cs
+++ b/src/Modules/LocalRecordsModule/Controllers/WidgetUpdateController.cs
@@ -6,6 +6,7 @@
 using EvoSC.Common.Remote;
 using EvoSC.Common.Util;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
+using EvoSC.Modules.Official.PlayerRecords.Events;
 using GbxRemoteNet.Events;
 
 namespace EvoSC.Modules.Official.LocalRecordsModule.Controllers;
@@ -22,4 +23,7 @@ public async Task OnPlayerConnectAsync(object sender, PlayerConnectGbxEventArgs
 
     [Subscribe(GbxRemoteEvent.BeginMap)]
     public Task OnBeginMapAsync(object sender, MapGbxEventArgs args) => localRecords.ShowWidgetToAllAsync();
+
+    [Subscribe(PlayerRecordsEvent.PbRecord)]
+    public Task OnPbAsync(object sender, PbRecordUpdateEventArgs args) => localRecords.UpdatePbAsync(args.Record);
 }
diff --git a/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs b/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
index 22208fabd..4b7e0d9b9 100644
--- a/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
+++ b/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
@@ -3,6 +3,7 @@
 using EvoSC.Common.Interfaces.Database;
 using EvoSC.Common.Interfaces.Models;
 using EvoSC.Common.Services.Attributes;
+using EvoSC.Modules.Official.LocalRecordsModule.Config;
 using EvoSC.Modules.Official.LocalRecordsModule.Database.Models;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Database;
@@ -14,7 +15,10 @@
 namespace EvoSC.Modules.Official.LocalRecordsModule.Database.Repository;
 
 [Service]
-public class LocalRecordRepository(IDbConnectionFactory dbConnFactory, IPlayerRecordsRepository recordsRepository)
+public class LocalRecordRepository(
+    IDbConnectionFactory dbConnFactory,
+    IPlayerRecordsRepository recordsRepository,
+    ILocalRecordsSettings settings)
     : DbRepository(dbConnFactory), ILocalRecordRepository
 {
     public async Task<IEnumerable<DbLocalRecord>> GetLocalRecordsOfMapByIdAsync(long mapId) =>
@@ -23,7 +27,7 @@ await NewLoadAll()
             .OrderBy(r => r.Position)
             .ToArrayAsync();
 
-    public async Task<DbLocalRecord> AddOrUpdateRecordAsync(IMap map, IPlayerRecord record)
+    public async Task<DbLocalRecord?> AddOrUpdateRecordAsync(IMap map, IPlayerRecord record)
     {
         // check if old record is better or equal, and dont bother updating if so
         var oldRecord = await GetRecordOfPlayerInMapAsync(record.Player, map);
@@ -33,6 +37,16 @@ public async Task<DbLocalRecord> AddOrUpdateRecordAsync(IMap map, IPlayerRecord
             return oldRecord;
         }
 
+        var worstRecord = await NewLoadAll()
+            .Where(r => r.DbMap.Id == map.Id)
+            .OrderByDescending(r => r.Position)
+            .FirstOrDefaultAsync();
+
+        if (worstRecord != null && worstRecord.Record.CompareTo(record) < 0)
+        {
+            return null;
+        }
+
         var localRecord = new DbLocalRecord
         {
             MapId = map.Id,
@@ -41,7 +55,7 @@ public async Task<DbLocalRecord> AddOrUpdateRecordAsync(IMap map, IPlayerRecord
             DbMap = new DbMap(map),
             DbRecord = new DbPlayerRecord(record)
         };
-        
+
         var transaction = await Database.BeginTransactionAsync();
 
         try
@@ -51,7 +65,7 @@ public async Task<DbLocalRecord> AddOrUpdateRecordAsync(IMap map, IPlayerRecord
             {
                 await Database.DeleteAsync(oldRecord);
             }
-            
+
             var id = await Database.InsertWithIdentityAsync(localRecord);
             localRecord.Id = Convert.ToInt64(id);
 
@@ -62,27 +76,37 @@ public async Task<DbLocalRecord> AddOrUpdateRecordAsync(IMap map, IPlayerRecord
             await transaction.RollbackTransactionAsync();
             throw;
         }
-        
-        await RecalculatePositionsOfMapAsync(map);
-        return localRecord;
+
+        var updated = await RecalculatePositionsOfMapAsync(map);
+        var updatedRecord = updated.FirstOrDefault(r => r.Id == localRecord.Id);
+        return updatedRecord;
     }
 
-    public async Task RecalculatePositionsOfMapAsync(IMap map)
+    public async Task<DbLocalRecord[]> RecalculatePositionsOfMapAsync(IMap map)
     {
         var locals = await GetLocalRecordsOfMapByIdAsync(map.Id);
         var sorted = locals.OrderBy(r => r.DbRecord.Score).ToArray();
         var updated = new List<DbLocalRecord>();
-        
+        var remove = new List<DbLocalRecord>();
+
+        var maxRecords = settings.MaxRecordsPerMap;
+
         for (var i = 0; i < sorted.Length; i++)
         {
+            if (i > maxRecords)
+            {
+                remove.Add(sorted[i]);
+                continue;
+            }
+
             var newPos = i + 1;
-            
+
             // don't update records that dont need it
             if (sorted[i].Position == newPos)
             {
                 continue;
             }
-            
+
             sorted[i].Position = newPos;
             updated.Add(sorted[i]);
         }
@@ -95,13 +119,20 @@ public async Task RecalculatePositionsOfMapAsync(IMap map)
             {
                 await Database.UpdateAsync(record);
             }
-            
+
+            foreach (var toRemove in remove)
+            {
+                await Database.DeleteAsync(toRemove);
+            }
+
             await transaction.CommitTransactionAsync();
         }
         catch (Exception ex)
         {
             await transaction.RollbackTransactionAsync();
         }
+
+        return updated.ToArray();
     }
 
     public async Task<IEnumerable<DbLocalRecord>> GetRecordsByPlayerAsync(IPlayer player) =>
diff --git a/src/Modules/LocalRecordsModule/Interfaces/Database/ILocalRecordRepository.cs b/src/Modules/LocalRecordsModule/Interfaces/Database/ILocalRecordRepository.cs
index 0227882f4..b0c224fec 100644
--- a/src/Modules/LocalRecordsModule/Interfaces/Database/ILocalRecordRepository.cs
+++ b/src/Modules/LocalRecordsModule/Interfaces/Database/ILocalRecordRepository.cs
@@ -7,8 +7,8 @@ namespace EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Database;
 public interface ILocalRecordRepository
 {
     public Task<IEnumerable<DbLocalRecord>> GetLocalRecordsOfMapByIdAsync(long mapId);
-    public Task<DbLocalRecord> AddOrUpdateRecordAsync(IMap map, IPlayerRecord record);
-    public Task RecalculatePositionsOfMapAsync(IMap map);
+    public Task<DbLocalRecord?> AddOrUpdateRecordAsync(IMap map, IPlayerRecord record);
+    public Task<DbLocalRecord[]> RecalculatePositionsOfMapAsync(IMap map);
     public Task<IEnumerable<DbLocalRecord>> GetRecordsByPlayerAsync(IPlayer player);
     public Task<DbLocalRecord?> GetRecordOfPlayerInMapAsync(IPlayer player, IMap map);
     public Task DeleteRecordAsync(IPlayer player, IMap map);
diff --git a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
index f46a2aad7..965c0ae8d 100644
--- a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
@@ -1,4 +1,5 @@
 using EvoSC.Common.Interfaces.Models;
+using EvoSC.Modules.Official.PlayerRecords.Interfaces.Models;
 
 namespace EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
 
@@ -7,4 +8,5 @@ public interface ILocalRecordsService
     public Task<ILocalRecord[]> GetLocalsOfCurrentMapFromPosAsync();
     public Task ShowWidgetAsync(IPlayer player);
     public Task ShowWidgetToAllAsync();
+    public Task UpdatePbAsync(IPlayerRecord record);
 }
diff --git a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
index ca8e5a2ae..3658d3e2a 100644
--- a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
@@ -1,12 +1,16 @@
-using EvoSC.Common.Interfaces.Models;
+using EvoSC.Common.Interfaces;
+using EvoSC.Common.Interfaces.Models;
 using EvoSC.Common.Interfaces.Services;
+using EvoSC.Common.Interfaces.Themes;
 using EvoSC.Common.Services.Attributes;
+using EvoSC.Common.Util;
+using EvoSC.Common.Util.TextFormatting;
 using EvoSC.Manialinks.Interfaces;
 using EvoSC.Modules.Official.LocalRecordsModule.Config;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Database;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
-using GbxRemoteNet;
+using EvoSC.Modules.Official.PlayerRecords.Interfaces.Models;
 using Microsoft.Extensions.Logging;
 
 namespace EvoSC.Modules.Official.LocalRecordsModule.Services;
@@ -18,7 +22,9 @@ public class LocalRecordsService(
     IPlayerManagerService playerManagerService,
     IManialinkManager manialinkManager,
     ILogger<LocalRecordsService> logger,
-    ILocalRecordsSettings settings) : ILocalRecordsService
+    ILocalRecordsSettings settings,
+    IServerClient server,
+    IThemeManager themeManager) : ILocalRecordsService
 {
     private const string WidgetName = "LocalRecordsModule.LocalRecordsWidget";
     
@@ -65,6 +71,59 @@ public async Task ShowWidgetToAllAsync()
         }
     }
 
+    public async Task UpdatePbAsync(IPlayerRecord record)
+    {
+        var oldRecord = await localRecordRepository.GetRecordOfPlayerInMapAsync(record.Player, record.Map);
+        var localRecord = await localRecordRepository.AddOrUpdateRecordAsync(record.Map, record);
+
+        if (localRecord == null)
+        {
+            // player did not get a local record good enough to be registered
+            return;
+        }
+
+        if (oldRecord == null)
+        {
+            await server.InfoMessageAsync(new TextFormatter()
+                .AddText(record.Player.NickName)
+                .AddText(" gained their ")
+                .AddText($"{localRecord.Position}.", s => s.WithColor(themeManager.Theme.Info))
+                .AddText(" local record ")
+                .AddText(RaceTime.FromMilliseconds(localRecord.Record.Score).ToString(), s => s.WithColor(themeManager.Theme.Info))
+                .ToString());
+            await ShowWidgetToAllAsync();
+            return;
+        }
+
+        if (record.Score < oldRecord.Record.Score)
+        {
+            await server.InfoMessageAsync(new TextFormatter()
+                .AddText(record.Player.NickName)
+                .AddText(" secured their ")
+                .AddText($"{localRecord.Position}.", s => s.WithColor(themeManager.Theme.Info))
+                .AddText(" local record ")
+                .AddText(RaceTime.FromMilliseconds(localRecord.Record.Score).ToString(), s => s.WithColor(themeManager.Theme.Info))
+                .AddText(" (")
+                .AddText($"{oldRecord.Position}.", s => s.WithColor(themeManager.Theme.Info))
+                .AddText(" - ")
+                .AddText($"{localRecord.Position}.", s => s.WithColor(themeManager.Theme.Info))
+                .AddText(" )")
+                .ToString());
+            await ShowWidgetToAllAsync();
+        }
+        else if (record.Score == oldRecord.Record.Score)
+        {
+            await server.InfoMessageAsync(new TextFormatter()
+                .AddText(record.Player.NickName)
+                .AddText(" equaled their ")
+                .AddText($"{localRecord.Position}.", s => s.WithColor(themeManager.Theme.Info))
+                .AddText(" local record ")
+                .AddText(RaceTime.FromMilliseconds(localRecord.Record.Score).ToString(),
+                    s => s.WithColor(themeManager.Theme.Info))
+                .ToString());
+        }
+    }
+
     private ILocalRecord[] GetRecordsWithPlayer(IPlayer player, ILocalRecord[] records)
     {
         if (records.Length == 0)
diff --git a/src/Modules/MapQueueModule/Controllers/QueueCommandsController.cs b/src/Modules/MapQueueModule/Controllers/QueueCommandsController.cs
index 18e48acbf..2cfb297bb 100644
--- a/src/Modules/MapQueueModule/Controllers/QueueCommandsController.cs
+++ b/src/Modules/MapQueueModule/Controllers/QueueCommandsController.cs
@@ -28,7 +28,7 @@ public async Task QueueListAsync()
         var message = new TextFormatter();
         var mapList = mapQueue.QueuedMaps
             .Select(m => new FormattedText(m.Name)
-                .AsIsolated()
+                .AsNotIsolated()
                 .WithStyle(style => style.WithColor(Color.Gray))
             );
 
diff --git a/src/Modules/PlayerRecords/Database/Repository/PlayerRecordsRepository.cs b/src/Modules/PlayerRecords/Database/Repository/PlayerRecordsRepository.cs
index fafa5bb9f..607cd0a76 100644
--- a/src/Modules/PlayerRecords/Database/Repository/PlayerRecordsRepository.cs
+++ b/src/Modules/PlayerRecords/Database/Repository/PlayerRecordsRepository.cs
@@ -21,10 +21,9 @@ public class PlayerRecordsRepository(DbConnectionFactory dbConnFactory, ILogger<
         Table<DbPlayerRecord>()
             .LoadWith(r => r.DbPlayer)
             .LoadWith(r => r.DbMap)
+            .OrderBy(r => r.Score)
             .FirstOrDefaultAsync(r => r.PlayerId == player.Id && r.MapId == map.Id);
 
-    public Task UpdateRecordAsync(DbPlayerRecord record) => Database.UpdateAsync(record);
-
     public async Task<DbPlayerRecord> InsertRecordAsync(IPlayer player, IMap map, int score, IEnumerable<int> checkpoints)
     {
         var record = new DbPlayerRecord
@@ -42,7 +41,8 @@ public async Task<DbPlayerRecord> InsertRecordAsync(IPlayer player, IMap map, in
 
         try
         {
-            await Database.InsertAsync(record);
+            var id = await Database.InsertWithIdentityAsync(record);
+            record.Id = Convert.ToInt64(id);
         }
         catch (Exception ex)
         {
diff --git a/src/Modules/PlayerRecords/Interfaces/IPlayerRecordsRepository.cs b/src/Modules/PlayerRecords/Interfaces/IPlayerRecordsRepository.cs
index 2c4051512..b8257542b 100644
--- a/src/Modules/PlayerRecords/Interfaces/IPlayerRecordsRepository.cs
+++ b/src/Modules/PlayerRecords/Interfaces/IPlayerRecordsRepository.cs
@@ -14,13 +14,6 @@ public interface IPlayerRecordsRepository
     /// <returns></returns>
     public Task<DbPlayerRecord?> GetRecordAsync(IPlayer player, IMap map);
     
-    /// <summary>
-    /// Update a record in the database.
-    /// </summary>
-    /// <param name="record">The record to update with new information.</param>
-    /// <returns></returns>
-    public Task UpdateRecordAsync(DbPlayerRecord record);
-    
     /// <summary>
     /// Add a new record to the database.
     /// </summary>
diff --git a/src/Modules/PlayerRecords/Services/PlayerRecordsService.cs b/src/Modules/PlayerRecords/Services/PlayerRecordsService.cs
index 3c0c3f9c0..9a4ab067a 100644
--- a/src/Modules/PlayerRecords/Services/PlayerRecordsService.cs
+++ b/src/Modules/PlayerRecords/Services/PlayerRecordsService.cs
@@ -55,7 +55,7 @@ public async Task<IMap> GetOrAddCurrentMapAsync()
 
         if (record == null)
         {
-            await recordsRepo.InsertRecordAsync(player, map, score, checkpoints);
+            record = await recordsRepo.InsertRecordAsync(player, map, score, checkpoints);
             return (record, RecordUpdateStatus.New);
         }
 
@@ -64,10 +64,12 @@ public async Task<IMap> GetOrAddCurrentMapAsync()
             return (record, score > record.Score ? RecordUpdateStatus.NotUpdated : RecordUpdateStatus.Equal);
         }
 
-        record.Score = score;
+        /* record.Score = score;
         record.Checkpoints = string.Join(',', checkpoints);
         record.UpdatedAt = DateTime.UtcNow;
-        await recordsRepo.UpdateRecordAsync(record);
+        await recordsRepo.UpdateRecordAsync(record); */
+        
+        record = await recordsRepo.InsertRecordAsync(player, map, score, checkpoints);
 
         return (record, RecordUpdateStatus.Updated);
     }

From d98e4c5a54c6617265274f620b34008ad4e891bf Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Sat, 29 Jun 2024 14:40:17 +0200
Subject: [PATCH 13/18] fix local record detection

---
 .../Repository/LocalRecordRepository.cs        | 10 ++++++++--
 .../Services/LocalRecordsService.cs            | 18 ++++++++++++------
 .../Database/Models/DbPlayerRecord.cs          |  4 ++--
 3 files changed, 22 insertions(+), 10 deletions(-)

diff --git a/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs b/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
index 4b7e0d9b9..9706e6160 100644
--- a/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
+++ b/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
@@ -11,6 +11,7 @@
 using EvoSC.Modules.Official.PlayerRecords.Interfaces;
 using EvoSC.Modules.Official.PlayerRecords.Interfaces.Models;
 using LinqToDB;
+using Microsoft.Extensions.Logging;
 
 namespace EvoSC.Modules.Official.LocalRecordsModule.Database.Repository;
 
@@ -18,7 +19,8 @@ namespace EvoSC.Modules.Official.LocalRecordsModule.Database.Repository;
 public class LocalRecordRepository(
     IDbConnectionFactory dbConnFactory,
     IPlayerRecordsRepository recordsRepository,
-    ILocalRecordsSettings settings)
+    ILocalRecordsSettings settings,
+    ILogger<LocalRecordRepository> logger)
     : DbRepository(dbConnFactory), ILocalRecordRepository
 {
     public async Task<IEnumerable<DbLocalRecord>> GetLocalRecordsOfMapByIdAsync(long mapId) =>
@@ -34,6 +36,7 @@ await NewLoadAll()
 
         if (oldRecord != null && oldRecord.Record.CompareTo(record) <= 0)
         {
+            logger.LogDebug("Player has old record that is better or equal");
             return oldRecord;
         }
 
@@ -42,8 +45,9 @@ await NewLoadAll()
             .OrderByDescending(r => r.Position)
             .FirstOrDefaultAsync();
 
-        if (worstRecord != null && worstRecord.Record.CompareTo(record) < 0)
+        if (worstRecord != null && worstRecord.Position >= settings.MaxRecordsPerMap && worstRecord.Record.CompareTo(record) < 0)
         {
+            logger.LogDebug("player got a new record that is worse than the worst local record of the map");
             return null;
         }
 
@@ -79,6 +83,8 @@ await NewLoadAll()
 
         var updated = await RecalculatePositionsOfMapAsync(map);
         var updatedRecord = updated.FirstOrDefault(r => r.Id == localRecord.Id);
+        
+        logger.LogDebug("player got a new local record");
         return updatedRecord;
     }
 
diff --git a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
index 3658d3e2a..986b10b5c 100644
--- a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
@@ -81,15 +81,22 @@ public async Task UpdatePbAsync(IPlayerRecord record)
             // player did not get a local record good enough to be registered
             return;
         }
+        
+        var localRaceTime = RaceTime.FromMilliseconds(localRecord.Record.Score).ToString();
+        
+        if (localRaceTime == null)
+        {
+            throw new InvalidOperationException($"Failed to convert {localRecord.Record.Score} to race time");
+        }
 
         if (oldRecord == null)
         {
             await server.InfoMessageAsync(new TextFormatter()
                 .AddText(record.Player.NickName)
-                .AddText(" gained their ")
+                .AddText(" gained the ")
                 .AddText($"{localRecord.Position}.", s => s.WithColor(themeManager.Theme.Info))
                 .AddText(" local record ")
-                .AddText(RaceTime.FromMilliseconds(localRecord.Record.Score).ToString(), s => s.WithColor(themeManager.Theme.Info))
+                .AddText(localRaceTime, s => s.WithColor(themeManager.Theme.Info))
                 .ToString());
             await ShowWidgetToAllAsync();
             return;
@@ -99,10 +106,10 @@ await server.InfoMessageAsync(new TextFormatter()
         {
             await server.InfoMessageAsync(new TextFormatter()
                 .AddText(record.Player.NickName)
-                .AddText(" secured their ")
+                .AddText(" improved the ")
                 .AddText($"{localRecord.Position}.", s => s.WithColor(themeManager.Theme.Info))
                 .AddText(" local record ")
-                .AddText(RaceTime.FromMilliseconds(localRecord.Record.Score).ToString(), s => s.WithColor(themeManager.Theme.Info))
+                .AddText(localRaceTime, s => s.WithColor(themeManager.Theme.Info))
                 .AddText(" (")
                 .AddText($"{oldRecord.Position}.", s => s.WithColor(themeManager.Theme.Info))
                 .AddText(" - ")
@@ -118,8 +125,7 @@ await server.InfoMessageAsync(new TextFormatter()
                 .AddText(" equaled their ")
                 .AddText($"{localRecord.Position}.", s => s.WithColor(themeManager.Theme.Info))
                 .AddText(" local record ")
-                .AddText(RaceTime.FromMilliseconds(localRecord.Record.Score).ToString(),
-                    s => s.WithColor(themeManager.Theme.Info))
+                .AddText(localRaceTime, s => s.WithColor(themeManager.Theme.Info))
                 .ToString());
         }
     }
diff --git a/src/Modules/PlayerRecords/Database/Models/DbPlayerRecord.cs b/src/Modules/PlayerRecords/Database/Models/DbPlayerRecord.cs
index c895bf8de..2c5c33d5a 100644
--- a/src/Modules/PlayerRecords/Database/Models/DbPlayerRecord.cs
+++ b/src/Modules/PlayerRecords/Database/Models/DbPlayerRecord.cs
@@ -87,7 +87,7 @@ public int CompareTo(IPlayerRecord? other)
         // Compare time record type, lower is better
         int CompareTime() => Score switch
         {
-            _ when Score < other.Score => 1,
+            _ when Score < other.Score => -1,
             _ when Score > other.Score => 1,
             _ => 0
         };
@@ -95,7 +95,7 @@ public int CompareTo(IPlayerRecord? other)
         // compare points record type, higher is better
         int ComparePoints() => Score switch
         {
-            _ when Score > other.Score => 1,
+            _ when Score > other.Score => -1,
             _ when Score < other.Score => 1,
             _ => 0
         };

From f76cafb9352216275451312c79ac9fea43e55303 Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Sat, 29 Jun 2024 15:32:17 +0200
Subject: [PATCH 14/18] add local /resetlocals command to reset local records
 if they are out of sync with pbs

---
 .../Database/Repository/Maps/MapRepository.cs |  5 +++
 .../Database/Repository/IMapRepository.cs     |  6 +++
 .../Interfaces/Services/IMapService.cs        |  6 +++
 src/EvoSC.Common/Services/MapService.cs       |  2 +
 src/Modules/LocalRecordsModule/AuditEvents.cs |  9 ++++
 .../Controllers/CommandsController.cs         | 21 ++++++++++
 .../LocalRecordsManialinkController.cs        | 42 +++++++++++++++++++
 .../Repository/LocalRecordRepository.cs       | 32 ++++++++++++++
 .../Database/ILocalRecordRepository.cs        |  2 +
 .../Services/ILocalRecordsService.cs          |  2 +
 .../Permissions/LocalRecordsPermission.cs     | 11 +++++
 .../Services/LocalRecordsService.cs           | 17 +++++++-
 .../Templates/Dialogs/ConfirmResetDialog.mt   | 19 +++++++++
 .../Repository/PlayerRecordsRepository.cs     |  8 ++++
 .../Interfaces/IPlayerRecordsRepository.cs    |  2 +
 15 files changed, 183 insertions(+), 1 deletion(-)
 create mode 100644 src/Modules/LocalRecordsModule/AuditEvents.cs
 create mode 100644 src/Modules/LocalRecordsModule/Controllers/CommandsController.cs
 create mode 100644 src/Modules/LocalRecordsModule/Controllers/LocalRecordsManialinkController.cs
 create mode 100644 src/Modules/LocalRecordsModule/Permissions/LocalRecordsPermission.cs
 create mode 100644 src/Modules/LocalRecordsModule/Templates/Dialogs/ConfirmResetDialog.mt

diff --git a/src/EvoSC.Common/Database/Repository/Maps/MapRepository.cs b/src/EvoSC.Common/Database/Repository/Maps/MapRepository.cs
index 6fabd5801..a1eea86f5 100644
--- a/src/EvoSC.Common/Database/Repository/Maps/MapRepository.cs
+++ b/src/EvoSC.Common/Database/Repository/Maps/MapRepository.cs
@@ -19,6 +19,11 @@ public class MapRepository(IDbConnectionFactory dbConnFactory, ILogger<MapReposi
         .LoadWith(t => t.DbDetails)
         .SingleOrDefaultAsync(m => m.Id == id);
 
+    public async Task<IMap[]> GetMapsAsync() => await Table<DbMap>()
+        .LoadWith(m => m.DbAuthor)
+        .LoadWith(m => m.DbDetails)
+        .ToArrayAsync();
+
     public async Task<IMap?> GetMapByUidAsync(string uid) => await Table<DbMap>()
         .LoadWith(t => t.DbAuthor)
         .LoadWith(t => t.DbDetails)
diff --git a/src/EvoSC.Common/Interfaces/Database/Repository/IMapRepository.cs b/src/EvoSC.Common/Interfaces/Database/Repository/IMapRepository.cs
index baca8ad32..84769c3a2 100644
--- a/src/EvoSC.Common/Interfaces/Database/Repository/IMapRepository.cs
+++ b/src/EvoSC.Common/Interfaces/Database/Repository/IMapRepository.cs
@@ -14,6 +14,12 @@ public interface IMapRepository
     /// <param name="id">The maps database ID.</param>
     /// <returns>a map if it exists in the database.</returns>
     public Task<IMap?> GetMapByIdAsync(long id);
+
+    /// <summary>
+    /// Get all maps in the database.
+    /// </summary>
+    /// <returns></returns>
+    public Task<IMap[]> GetMapsAsync();
     
     /// <summary>
     /// Gets a map from the database based on the provided UID.
diff --git a/src/EvoSC.Common/Interfaces/Services/IMapService.cs b/src/EvoSC.Common/Interfaces/Services/IMapService.cs
index f058d16d3..72929f840 100644
--- a/src/EvoSC.Common/Interfaces/Services/IMapService.cs
+++ b/src/EvoSC.Common/Interfaces/Services/IMapService.cs
@@ -86,4 +86,10 @@ public interface IMapService
     /// <param name="map">Map to get details of.</param>
     /// <returns></returns>
     public Task<IMapDetails> FetchMapDetailsAsync(IMap map);
+
+    /// <summary>
+    /// Get all maps in the database.
+    /// </summary>
+    /// <returns></returns>
+    public Task<IMap[]> GetCurrentMapListAsync();
 }
diff --git a/src/EvoSC.Common/Services/MapService.cs b/src/EvoSC.Common/Services/MapService.cs
index 22165b840..05cfad607 100644
--- a/src/EvoSC.Common/Services/MapService.cs
+++ b/src/EvoSC.Common/Services/MapService.cs
@@ -198,6 +198,8 @@ public async Task<IMapDetails> FetchMapDetailsAsync(IMap map)
         }
     }
 
+    public Task<IMap[]> GetCurrentMapListAsync() => mapRepository.GetMapsAsync();
+
     private static bool MapVersionExistsInDb(IMap map, MapMetadata mapMetadata)
     {
         return map.ExternalVersion == mapMetadata.ExternalVersion;
diff --git a/src/Modules/LocalRecordsModule/AuditEvents.cs b/src/Modules/LocalRecordsModule/AuditEvents.cs
new file mode 100644
index 000000000..e4b51e8fe
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/AuditEvents.cs
@@ -0,0 +1,9 @@
+using EvoSC.Common.Util.EnumIdentifier;
+
+namespace EvoSC.Modules.Official.LocalRecordsModule;
+
+public enum AuditEvents
+{
+    [Identifier(Name = "LocalRecords:ResetAll")]
+    ResetAll,
+}
diff --git a/src/Modules/LocalRecordsModule/Controllers/CommandsController.cs b/src/Modules/LocalRecordsModule/Controllers/CommandsController.cs
new file mode 100644
index 000000000..1e8fc9714
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Controllers/CommandsController.cs
@@ -0,0 +1,21 @@
+using EvoSC.Commands.Attributes;
+using EvoSC.Commands.Interfaces;
+using EvoSC.Common.Controllers;
+using EvoSC.Common.Controllers.Attributes;
+using EvoSC.Manialinks.Interfaces;
+using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
+using EvoSC.Modules.Official.LocalRecordsModule.Permissions;
+
+namespace EvoSC.Modules.Official.LocalRecordsModule.Controllers;
+
+[Controller]
+public class CommandsController(IManialinkManager manialinks) : EvoScController<ICommandInteractionContext>
+{
+    [ChatCommand("resetlocals", "Recalculates local records table on all maps based on pbs.",
+        LocalRecordsPermission.ResetLocals)]
+    public async Task ResetLocalsAsync()
+    {
+        await manialinks.SendManialinkAsync(Context.Player, "LocalRecordsModule.Dialogs.ConfirmResetDialog");
+        Context.AuditEvent.Cancel();
+    }
+}
diff --git a/src/Modules/LocalRecordsModule/Controllers/LocalRecordsManialinkController.cs b/src/Modules/LocalRecordsModule/Controllers/LocalRecordsManialinkController.cs
new file mode 100644
index 000000000..c8119c62c
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Controllers/LocalRecordsManialinkController.cs
@@ -0,0 +1,42 @@
+using EvoSC.Common.Controllers.Attributes;
+using EvoSC.Common.Interfaces;
+using EvoSC.Manialinks;
+using EvoSC.Manialinks.Attributes;
+using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
+using EvoSC.Modules.Official.LocalRecordsModule.Permissions;
+
+namespace EvoSC.Modules.Official.LocalRecordsModule.Controllers;
+
+[Controller]
+public class LocalRecordsManialinkController(ILocalRecordsService localRecordsService, IServerClient server) : ManialinkController
+{
+    [ManialinkRoute(Permission = LocalRecordsPermission.ResetLocals)]
+    public async Task ConfirmResetAsync(bool confirmed)
+    {
+        await HideAsync("LocalRecordsModule.Dialogs.ConfirmResetDialog");
+        
+        if (!confirmed)
+        {
+            return;
+        }
+        
+        Context.AuditEvent.WithEventName(AuditEvents.ResetAll);
+        await server.InfoMessageAsync("Resetting local records, this may take a while ...");
+
+        try
+        {
+            await localRecordsService.ResetLocalRecordsAsync();
+        }
+        catch (Exception ex)
+        {
+            await server.ErrorMessageAsync($"Failed to reset local records: {ex.Message}");
+            Context.AuditEvent.Error();
+            throw;
+        }
+        
+        await server.SuccessMessageAsync("Local records successfully reset!");
+        Context.AuditEvent.Success();
+
+        await localRecordsService.ShowWidgetToAllAsync();
+    }
+}
diff --git a/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs b/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
index 9706e6160..f408a3c93 100644
--- a/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
+++ b/src/Modules/LocalRecordsModule/Database/Repository/LocalRecordRepository.cs
@@ -88,6 +88,34 @@ await NewLoadAll()
         return updatedRecord;
     }
 
+    public async Task AddRecordsAsync(IMap map, IEnumerable<IPlayerRecord> records)
+    {
+        var transaction = await Database.BeginTransactionAsync();
+
+        try
+        {
+            foreach (var record in records)
+            {
+                await Database.InsertWithIdentityAsync(new DbLocalRecord
+                {
+                    MapId = map.Id,
+                    RecordId = record.Id,
+                    Position = 0,
+                    DbMap = new DbMap(map),
+                    DbRecord = new DbPlayerRecord(record)
+                });
+            }
+        }
+        catch (Exception ex)
+        {
+            await transaction.RollbackTransactionAsync();
+            throw;
+        }
+
+        await transaction.CommitTransactionAsync();
+        await RecalculatePositionsOfMapAsync(map);
+    }
+
     public async Task<DbLocalRecord[]> RecalculatePositionsOfMapAsync(IMap map)
     {
         var locals = await GetLocalRecordsOfMapByIdAsync(map.Id);
@@ -182,6 +210,10 @@ public async Task DeleteRecordAsync(ILocalRecord localRecord)
         }
     }
 
+    public Task DeleteRecordsAsync(IMap map) => Table<DbLocalRecord>()
+        .Where(r => r.MapId == map.Id)
+        .DeleteAsync();
+
     private ILoadWithQueryable<DbLocalRecord, IPlayer> NewLoadAll() => Table<DbLocalRecord>()
         .LoadWith(r => r.DbMap)
         .LoadWith(r => r.DbRecord)
diff --git a/src/Modules/LocalRecordsModule/Interfaces/Database/ILocalRecordRepository.cs b/src/Modules/LocalRecordsModule/Interfaces/Database/ILocalRecordRepository.cs
index b0c224fec..3a3b46f1b 100644
--- a/src/Modules/LocalRecordsModule/Interfaces/Database/ILocalRecordRepository.cs
+++ b/src/Modules/LocalRecordsModule/Interfaces/Database/ILocalRecordRepository.cs
@@ -8,9 +8,11 @@ public interface ILocalRecordRepository
 {
     public Task<IEnumerable<DbLocalRecord>> GetLocalRecordsOfMapByIdAsync(long mapId);
     public Task<DbLocalRecord?> AddOrUpdateRecordAsync(IMap map, IPlayerRecord record);
+    public Task AddRecordsAsync(IMap map, IEnumerable<IPlayerRecord> records);
     public Task<DbLocalRecord[]> RecalculatePositionsOfMapAsync(IMap map);
     public Task<IEnumerable<DbLocalRecord>> GetRecordsByPlayerAsync(IPlayer player);
     public Task<DbLocalRecord?> GetRecordOfPlayerInMapAsync(IPlayer player, IMap map);
     public Task DeleteRecordAsync(IPlayer player, IMap map);
     public Task DeleteRecordAsync(ILocalRecord localRecord);
+    public Task DeleteRecordsAsync(IMap map);
 }
diff --git a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
index 965c0ae8d..5b3879ec1 100644
--- a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
@@ -9,4 +9,6 @@ public interface ILocalRecordsService
     public Task ShowWidgetAsync(IPlayer player);
     public Task ShowWidgetToAllAsync();
     public Task UpdatePbAsync(IPlayerRecord record);
+
+    public Task ResetLocalRecordsAsync();
 }
diff --git a/src/Modules/LocalRecordsModule/Permissions/LocalRecordsPermission.cs b/src/Modules/LocalRecordsModule/Permissions/LocalRecordsPermission.cs
new file mode 100644
index 000000000..6cb85ba4a
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Permissions/LocalRecordsPermission.cs
@@ -0,0 +1,11 @@
+using System.ComponentModel;
+using EvoSC.Common.Permissions.Attributes;
+
+namespace EvoSC.Modules.Official.LocalRecordsModule.Permissions;
+
+[PermissionGroup]
+public enum LocalRecordsPermission
+{
+    [Description("Can reset the local records table on all maps.")]
+    ResetLocals
+}
diff --git a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
index 986b10b5c..adc6d2972 100644
--- a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
@@ -10,6 +10,7 @@
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Database;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
+using EvoSC.Modules.Official.PlayerRecords.Interfaces;
 using EvoSC.Modules.Official.PlayerRecords.Interfaces.Models;
 using Microsoft.Extensions.Logging;
 
@@ -24,7 +25,8 @@ public class LocalRecordsService(
     ILogger<LocalRecordsService> logger,
     ILocalRecordsSettings settings,
     IServerClient server,
-    IThemeManager themeManager) : ILocalRecordsService
+    IThemeManager themeManager,
+    IPlayerRecordsRepository playerRecordsRepository) : ILocalRecordsService
 {
     private const string WidgetName = "LocalRecordsModule.LocalRecordsWidget";
     
@@ -130,6 +132,19 @@ await server.InfoMessageAsync(new TextFormatter()
         }
     }
 
+    public async Task ResetLocalRecordsAsync()
+    {
+        var maps = await mapService.GetCurrentMapListAsync();
+
+        foreach (var map in maps)
+        {
+            var mapPbs = await playerRecordsRepository.GetRecordsOfMapAsync(map.Id);
+
+            await localRecordRepository.DeleteRecordsAsync(map);
+            await localRecordRepository.AddRecordsAsync(map, mapPbs);
+        }
+    }
+
     private ILocalRecord[] GetRecordsWithPlayer(IPlayer player, ILocalRecord[] records)
     {
         if (records.Length == 0)
diff --git a/src/Modules/LocalRecordsModule/Templates/Dialogs/ConfirmResetDialog.mt b/src/Modules/LocalRecordsModule/Templates/Dialogs/ConfirmResetDialog.mt
new file mode 100644
index 000000000..7d981fc34
--- /dev/null
+++ b/src/Modules/LocalRecordsModule/Templates/Dialogs/ConfirmResetDialog.mt
@@ -0,0 +1,19 @@
+<!--
+Shows a confirmation dialog when deleting a map from the leaderboard.
+-->
+<component>
+  <import component="EvoSC.Controls.ConfirmDialog" as="ConfirmDialog" />
+  <import component="EvoSC.Style.UIStyle" as="UIStyle" />
+  
+  <template>
+    <UIStyle />
+    
+    <ConfirmDialog 
+            title="Reset local records?"
+            text="This will re-calculate local records on all maps based PBs."
+            action="LocalRecordsManialinkController/ConfirmReset"
+    />
+  </template>
+  
+  <script resource="EvoSC.Scripts.UIScripts" main="true" />
+</component>
\ No newline at end of file
diff --git a/src/Modules/PlayerRecords/Database/Repository/PlayerRecordsRepository.cs b/src/Modules/PlayerRecords/Database/Repository/PlayerRecordsRepository.cs
index 607cd0a76..69f93cef7 100644
--- a/src/Modules/PlayerRecords/Database/Repository/PlayerRecordsRepository.cs
+++ b/src/Modules/PlayerRecords/Database/Repository/PlayerRecordsRepository.cs
@@ -57,4 +57,12 @@ public Task DeleteRecordAsync(IPlayer player, IMap map) => Table<DbPlayerRecord>
         .DeleteAsync(r => r.MapId == map.Id && r.PlayerId == player.Id);
 
     public Task DeleteRecordAsync(IPlayerRecord record) => Database.DeleteAsync(record);
+
+    public Task<DbPlayerRecord[]> GetRecordsOfMapAsync(long mapId) =>
+        Table<DbPlayerRecord>()
+            .LoadWith(r => r.DbMap)
+            .LoadWith(r => r.DbPlayer)
+            .Where(r => r.MapId == mapId)
+            .OrderBy(r => r.Score)
+            .ToArrayAsync();
 }
diff --git a/src/Modules/PlayerRecords/Interfaces/IPlayerRecordsRepository.cs b/src/Modules/PlayerRecords/Interfaces/IPlayerRecordsRepository.cs
index b8257542b..0602a3f00 100644
--- a/src/Modules/PlayerRecords/Interfaces/IPlayerRecordsRepository.cs
+++ b/src/Modules/PlayerRecords/Interfaces/IPlayerRecordsRepository.cs
@@ -23,4 +23,6 @@ public interface IPlayerRecordsRepository
 
     public Task DeleteRecordAsync(IPlayer player, IMap map);
     public Task DeleteRecordAsync(IPlayerRecord record);
+
+    public Task<DbPlayerRecord[]> GetRecordsOfMapAsync(long mapId);
 }

From e22f0d063257284a7de4d0d958e9b206d047e25c Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Sun, 30 Jun 2024 15:23:10 +0200
Subject: [PATCH 15/18] add service and controller tests

---
 EvoSC.sln                                     | 11 ++-
 .../Interfaces/IManialinkManager.cs           |  2 +-
 .../Interfaces/IManialinkTransaction.cs       |  2 +-
 src/EvoSC.Manialinks/ManialinkManager.cs      |  2 +-
 src/EvoSC.Manialinks/ManialinkTransaction.cs  |  2 +-
 .../Services/ILocalRecordsService.cs          |  2 +-
 .../Services/LocalRecordsService.cs           |  6 +-
 tests/EvoSC.CLI.Tests/EvoSC.CLI.Tests.csproj  |  4 +-
 .../EvoSC.Commands.Tests.csproj               |  4 +-
 .../EvoSC.Common.Tests.csproj                 |  4 +-
 .../EvoSC.Manialinks.Tests.csproj             |  4 +-
 .../EvoSC.Testing.Tests.csproj                |  4 +-
 tests/EvoSC.Tests/EvoSC.Tests.csproj          |  4 +-
 .../ASayModule.Tests/ASayModule.Tests.csproj  |  4 +-
 .../CurrentMapModule.Tests.csproj             |  4 +-
 .../FastestCp.Tests/FastestCp.Tests.csproj    |  4 +-
 .../Controllers/CommandsControllerTests.cs    | 27 +++++++
 .../LocalRecordsManialinkControllerTests.cs   | 46 ++++++++++++
 .../WidgetUpdateControllerTests.cs            | 70 +++++++++++++++++++
 .../MapListModule.Tests.csproj                |  4 +-
 .../MapQueueModuleTests.csproj                |  2 +-
 .../MapsModule.Tests/MapsModule.Tests.csproj  |  4 +-
 .../MatchManagerModule.Tests.csproj           |  4 +-
 .../MatchReadyModule.Tests.csproj             |  4 +-
 .../MatchTrackerModule.Tests.csproj           |  4 +-
 .../MotdModule.Tests/MotdModule.Tests.csproj  |  4 +-
 .../NextMapModule.Tests.csproj                |  4 +-
 .../OpenPlanetModule.Tests.csproj             |  4 +-
 .../Modules/Player.Tests/Player.Tests.csproj  |  4 +-
 .../SetName.Tests/SetName.Tests.csproj        |  4 +-
 30 files changed, 199 insertions(+), 49 deletions(-)
 create mode 100644 tests/Modules/LocalRecordsModule.Tests/Controllers/CommandsControllerTests.cs
 create mode 100644 tests/Modules/LocalRecordsModule.Tests/Controllers/LocalRecordsManialinkControllerTests.cs
 create mode 100644 tests/Modules/LocalRecordsModule.Tests/Controllers/WidgetUpdateControllerTests.cs

diff --git a/EvoSC.sln b/EvoSC.sln
index c64ed88ef..e1ff0f73c 100644
--- a/EvoSC.sln
+++ b/EvoSC.sln
@@ -114,6 +114,8 @@ Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "MapListModule.Tests", "test
 EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "LocalRecordsModule", "src/Modules/LocalRecordsModule/LocalRecordsModule.csproj", "{1D8DBFC8-EC21-4DDA-9D88-DE7CA04C8449}"
 EndProject
+Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "LocalRecordsModule.Tests", "tests\Modules\LocalRecordsModule.Tests\LocalRecordsModule.Tests.csproj", "{7401429B-B842-4316-B7A2-B77E9AD966CB}"
+EndProject
 
 
 
@@ -337,6 +339,10 @@ Global
 		{1D8DBFC8-EC21-4DDA-9D88-DE7CA04C8449}.Debug|Any CPU.Build.0 = Debug|Any CPU
 		{1D8DBFC8-EC21-4DDA-9D88-DE7CA04C8449}.Release|Any CPU.ActiveCfg = Release|Any CPU
 		{1D8DBFC8-EC21-4DDA-9D88-DE7CA04C8449}.Release|Any CPU.Build.0 = Release|Any CPU
+		{7401429B-B842-4316-B7A2-B77E9AD966CB}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{7401429B-B842-4316-B7A2-B77E9AD966CB}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{7401429B-B842-4316-B7A2-B77E9AD966CB}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{7401429B-B842-4316-B7A2-B77E9AD966CB}.Release|Any CPU.Build.0 = Release|Any CPU
 	EndGlobalSection
 	GlobalSection(SolutionProperties) = preSolution
 		HideSolutionNode = FALSE
@@ -389,6 +395,7 @@ Global
 		{2D2D3AC6-C8BD-4615-BCC7-9A64007DB762} = {6D75D6A2-6ECD-4DE4-96C5-CAD7D134407A}
 		{AB316816-F301-4693-86E8-C31E78F53A59} = {DC47658A-F421-4BA4-B617-090A7DFB3900}
 		{098D1F9B-054D-4158-BB6C-AC908C4595F6} = {6D75D6A2-6ECD-4DE4-96C5-CAD7D134407A}
-		{1D8DBFC8-EC21-4DDA-9D88-DE7CA04C8449} = {DC47658A-F421-4BA4-B617-090A7DFB3900}	
-EndGlobalSection
+		{1D8DBFC8-EC21-4DDA-9D88-DE7CA04C8449} = {DC47658A-F421-4BA4-B617-090A7DFB3900}
+		{7401429B-B842-4316-B7A2-B77E9AD966CB} = {6D75D6A2-6ECD-4DE4-96C5-CAD7D134407A}
+	EndGlobalSection
 EndGlobal
diff --git a/src/EvoSC.Manialinks/Interfaces/IManialinkManager.cs b/src/EvoSC.Manialinks/Interfaces/IManialinkManager.cs
index e657668fc..16643879d 100644
--- a/src/EvoSC.Manialinks/Interfaces/IManialinkManager.cs
+++ b/src/EvoSC.Manialinks/Interfaces/IManialinkManager.cs
@@ -117,5 +117,5 @@ public interface IManialinkManager : IManialinkOperations
     public Task<string> PrepareAndRenderAsync(string name, dynamic data);
     public string GetEffectiveName(string name);
 
-    public ManialinkTransaction CreateTransaction();
+    public IManialinkTransaction CreateTransaction();
 }
diff --git a/src/EvoSC.Manialinks/Interfaces/IManialinkTransaction.cs b/src/EvoSC.Manialinks/Interfaces/IManialinkTransaction.cs
index 164cf1155..6a85012ec 100644
--- a/src/EvoSC.Manialinks/Interfaces/IManialinkTransaction.cs
+++ b/src/EvoSC.Manialinks/Interfaces/IManialinkTransaction.cs
@@ -1,6 +1,6 @@
 namespace EvoSC.Manialinks.Interfaces;
 
-public interface IManialinkTransaction
+public interface IManialinkTransaction : IManialinkOperations
 {
     public Task CommitAsync();
 }
diff --git a/src/EvoSC.Manialinks/ManialinkManager.cs b/src/EvoSC.Manialinks/ManialinkManager.cs
index c15d3a542..87928429b 100644
--- a/src/EvoSC.Manialinks/ManialinkManager.cs
+++ b/src/EvoSC.Manialinks/ManialinkManager.cs
@@ -485,5 +485,5 @@ public string GetEffectiveName(string name) =>
             ? effectiveName
             : name;
 
-    public ManialinkTransaction CreateTransaction() => new(this, _server);
+    public IManialinkTransaction CreateTransaction() => new ManialinkTransaction(this, _server);
 }
diff --git a/src/EvoSC.Manialinks/ManialinkTransaction.cs b/src/EvoSC.Manialinks/ManialinkTransaction.cs
index 1ffb0c65e..9970f5012 100644
--- a/src/EvoSC.Manialinks/ManialinkTransaction.cs
+++ b/src/EvoSC.Manialinks/ManialinkTransaction.cs
@@ -8,7 +8,7 @@
 
 namespace EvoSC.Manialinks;
 
-public class ManialinkTransaction(IManialinkManager manialinkManager, IServerClient server) : IManialinkOperations, IManialinkTransaction
+public class ManialinkTransaction(IManialinkManager manialinkManager, IServerClient server) : IManialinkTransaction
 {
     private readonly MultiCall _serverCalls = new MultiCall();
     private readonly HashSet<string> _persistentRemovals = [];
diff --git a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
index 5b3879ec1..347dc895f 100644
--- a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
@@ -5,7 +5,7 @@ namespace EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
 
 public interface ILocalRecordsService
 {
-    public Task<ILocalRecord[]> GetLocalsOfCurrentMapFromPosAsync();
+    public Task<ILocalRecord[]> GetLocalsOfCurrentMapAsync();
     public Task ShowWidgetAsync(IPlayer player);
     public Task ShowWidgetToAllAsync();
     public Task UpdatePbAsync(IPlayerRecord record);
diff --git a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
index adc6d2972..b3d46b4f8 100644
--- a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
@@ -30,7 +30,7 @@ public class LocalRecordsService(
 {
     private const string WidgetName = "LocalRecordsModule.LocalRecordsWidget";
     
-    public async Task<ILocalRecord[]> GetLocalsOfCurrentMapFromPosAsync()
+    public async Task<ILocalRecord[]> GetLocalsOfCurrentMapAsync()
     {
         var currentMap = await mapService.GetCurrentMapAsync();
 
@@ -45,7 +45,7 @@ public async Task<ILocalRecord[]> GetLocalsOfCurrentMapFromPosAsync()
 
     public async Task ShowWidgetAsync(IPlayer player)
     {
-        var records = await GetLocalsOfCurrentMapFromPosAsync();
+        var records = await GetLocalsOfCurrentMapAsync();
         var playerRecords = GetRecordsWithPlayer(player, records);
         await manialinkManager.SendManialinkAsync(player, WidgetName,
             new { currentPlayer = player, records = playerRecords });
@@ -53,7 +53,7 @@ await manialinkManager.SendManialinkAsync(player, WidgetName,
 
     public async Task ShowWidgetToAllAsync()
     {
-        var records = await GetLocalsOfCurrentMapFromPosAsync();
+        var records = await GetLocalsOfCurrentMapAsync();
         var onlinePlayers = await playerManagerService.GetOnlinePlayersAsync();
         var transaction = manialinkManager.CreateTransaction();
 
diff --git a/tests/EvoSC.CLI.Tests/EvoSC.CLI.Tests.csproj b/tests/EvoSC.CLI.Tests/EvoSC.CLI.Tests.csproj
index 0dee7c98f..d428c99d4 100644
--- a/tests/EvoSC.CLI.Tests/EvoSC.CLI.Tests.csproj
+++ b/tests/EvoSC.CLI.Tests/EvoSC.CLI.Tests.csproj
@@ -8,12 +8,12 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
         <PackageReference Include="SonarAnalyzer.CSharp" Version="9.15.0.81779">
           <PrivateAssets>all</PrivateAssets>
           <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/EvoSC.Commands.Tests/EvoSC.Commands.Tests.csproj b/tests/EvoSC.Commands.Tests/EvoSC.Commands.Tests.csproj
index 7bd3513e0..6b942a975 100644
--- a/tests/EvoSC.Commands.Tests/EvoSC.Commands.Tests.csproj
+++ b/tests/EvoSC.Commands.Tests/EvoSC.Commands.Tests.csproj
@@ -8,12 +8,12 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
         <PackageReference Include="SonarAnalyzer.CSharp" Version="9.15.0.81779">
           <PrivateAssets>all</PrivateAssets>
           <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/EvoSC.Common.Tests/EvoSC.Common.Tests.csproj b/tests/EvoSC.Common.Tests/EvoSC.Common.Tests.csproj
index 82269616d..a298b4ccc 100644
--- a/tests/EvoSC.Common.Tests/EvoSC.Common.Tests.csproj
+++ b/tests/EvoSC.Common.Tests/EvoSC.Common.Tests.csproj
@@ -13,13 +13,13 @@
         <PackageReference Include="linq2db" Version="5.3.2" />
         <PackageReference Include="Microsoft.Data.Sqlite" Version="8.0.0" />
         <PackageReference Include="Microsoft.Extensions.Logging" Version="8.0.0" />
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
         <PackageReference Include="Moq" Version="4.20.70" />
         <PackageReference Include="SonarAnalyzer.CSharp" Version="9.15.0.81779">
           <PrivateAssets>all</PrivateAssets>
           <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/EvoSC.Manialinks.Tests/EvoSC.Manialinks.Tests.csproj b/tests/EvoSC.Manialinks.Tests/EvoSC.Manialinks.Tests.csproj
index f0c7412ba..4a5f4eb6e 100644
--- a/tests/EvoSC.Manialinks.Tests/EvoSC.Manialinks.Tests.csproj
+++ b/tests/EvoSC.Manialinks.Tests/EvoSC.Manialinks.Tests.csproj
@@ -9,12 +9,12 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
         <PackageReference Include="SonarAnalyzer.CSharp" Version="9.15.0.81779">
           <PrivateAssets>all</PrivateAssets>
           <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/EvoSC.Testing.Tests/EvoSC.Testing.Tests.csproj b/tests/EvoSC.Testing.Tests/EvoSC.Testing.Tests.csproj
index 4b1caa41b..8ceacb16d 100644
--- a/tests/EvoSC.Testing.Tests/EvoSC.Testing.Tests.csproj
+++ b/tests/EvoSC.Testing.Tests/EvoSC.Testing.Tests.csproj
@@ -10,12 +10,12 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
         <PackageReference Include="SonarAnalyzer.CSharp" Version="9.15.0.81779">
           <PrivateAssets>all</PrivateAssets>
           <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/EvoSC.Tests/EvoSC.Tests.csproj b/tests/EvoSC.Tests/EvoSC.Tests.csproj
index b5f47e83f..907dd812e 100644
--- a/tests/EvoSC.Tests/EvoSC.Tests.csproj
+++ b/tests/EvoSC.Tests/EvoSC.Tests.csproj
@@ -10,7 +10,7 @@
     <ItemGroup>
         <PackageReference Include="Flurl" Version="3.0.6" />
         <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="8.0.0" />
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
         <PackageReference Include="SonarAnalyzer.CSharp" Version="9.15.0.81779">
           <PrivateAssets>all</PrivateAssets>
           <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
@@ -19,7 +19,7 @@
           <PrivateAssets>all</PrivateAssets>
           <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/Modules/ASayModule.Tests/ASayModule.Tests.csproj b/tests/Modules/ASayModule.Tests/ASayModule.Tests.csproj
index 22753f6f3..ed5b14ac3 100644
--- a/tests/Modules/ASayModule.Tests/ASayModule.Tests.csproj
+++ b/tests/Modules/ASayModule.Tests/ASayModule.Tests.csproj
@@ -11,9 +11,9 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
         <PackageReference Include="Moq" Version="4.20.70" />
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/Modules/CurrentMapModule.Tests/CurrentMapModule.Tests.csproj b/tests/Modules/CurrentMapModule.Tests/CurrentMapModule.Tests.csproj
index e0fe50c23..78e1dfc16 100644
--- a/tests/Modules/CurrentMapModule.Tests/CurrentMapModule.Tests.csproj
+++ b/tests/Modules/CurrentMapModule.Tests/CurrentMapModule.Tests.csproj
@@ -9,13 +9,13 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
         <PackageReference Include="Moq" Version="4.20.70" />
         <PackageReference Include="SonarAnalyzer.CSharp" Version="9.15.0.81779">
           <PrivateAssets>all</PrivateAssets>
           <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/Modules/FastestCp.Tests/FastestCp.Tests.csproj b/tests/Modules/FastestCp.Tests/FastestCp.Tests.csproj
index 0db56029d..78c648182 100644
--- a/tests/Modules/FastestCp.Tests/FastestCp.Tests.csproj
+++ b/tests/Modules/FastestCp.Tests/FastestCp.Tests.csproj
@@ -9,13 +9,13 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
         <PackageReference Include="Moq" Version="4.20.70" />
         <PackageReference Include="SonarAnalyzer.CSharp" Version="9.15.0.81779">
           <PrivateAssets>all</PrivateAssets>
           <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/Modules/LocalRecordsModule.Tests/Controllers/CommandsControllerTests.cs b/tests/Modules/LocalRecordsModule.Tests/Controllers/CommandsControllerTests.cs
new file mode 100644
index 000000000..35e3313f8
--- /dev/null
+++ b/tests/Modules/LocalRecordsModule.Tests/Controllers/CommandsControllerTests.cs
@@ -0,0 +1,27 @@
+using EvoSC.Common.Interfaces.Models;
+using EvoSC.Manialinks.Interfaces;
+using EvoSC.Modules.Official.LocalRecordsModule.Controllers;
+using EvoSC.Testing.Controllers;
+using Moq;
+
+namespace LocalRecordsModule.Tests.Controllers;
+
+public class CommandsControllerTests : CommandInteractionControllerTestBase<CommandsController>
+{
+    private Mock<IOnlinePlayer> _actor = new();
+    private Mock<IManialinkManager> _manialinkManager = new();
+    
+    public CommandsControllerTests()
+    {
+        InitMock(_actor.Object, _manialinkManager);
+    }
+
+    [Fact]
+    public async Task Reset_Locals_Is_Confirmed()
+    {
+        await Controller.ResetLocalsAsync();
+        
+        _manialinkManager.Verify(m => m.SendManialinkAsync(_actor.Object, "LocalRecordsModule.Dialogs.ConfirmResetDialog"));
+        AuditEventBuilder.Verify(m => m.Cancel());
+    }
+}
diff --git a/tests/Modules/LocalRecordsModule.Tests/Controllers/LocalRecordsManialinkControllerTests.cs b/tests/Modules/LocalRecordsModule.Tests/Controllers/LocalRecordsManialinkControllerTests.cs
new file mode 100644
index 000000000..74671d24b
--- /dev/null
+++ b/tests/Modules/LocalRecordsModule.Tests/Controllers/LocalRecordsManialinkControllerTests.cs
@@ -0,0 +1,46 @@
+using EvoSC.Common.Interfaces;
+using EvoSC.Common.Interfaces.Models;
+using EvoSC.Manialinks.Interfaces.Models;
+using EvoSC.Modules.Official.LocalRecordsModule.Controllers;
+using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
+using EvoSC.Testing;
+using EvoSC.Testing.Controllers;
+using GbxRemoteNet.Interfaces;
+using Moq;
+
+namespace LocalRecordsModule.Tests.Controllers;
+
+public class LocalRecordsManialinkControllerTests : ManialinkControllerTestBase<LocalRecordsManialinkController>
+{
+    private readonly Mock<IManialinkActionContext> _manialinkActionContext = new();
+    private readonly Mock<IOnlinePlayer> _actor = new();
+    private readonly Mock<ILocalRecordsService> _localRecordsService = new();
+    private readonly (Mock<IServerClient> Server, Mock<IGbxRemoteClient> Remote)
+        _server = Mocking.NewServerClientMock();
+
+    public LocalRecordsManialinkControllerTests()
+    {
+        InitMock(_actor.Object, _manialinkActionContext.Object, _localRecordsService, _server.Server);
+    }
+
+    [Fact]
+    public async Task Reset_Records_Confirmation_Resets_All_Records()
+    {
+        await Controller.ConfirmResetAsync(true);
+        
+        ManialinkManager.Verify(m => m.HideManialinkAsync("LocalRecordsModule.Dialogs.ConfirmResetDialog"));
+        _localRecordsService.Verify(m => m.ResetLocalRecordsAsync());
+        AuditEventBuilder.Verify(m => m.Success());
+    }
+    
+    [Fact]
+    public async Task Reset_Records_Cancel_Does_Not_Resets_All_Records()
+    {
+        await Controller.ConfirmResetAsync(false);
+        
+        ManialinkManager.Verify(m => m.HideManialinkAsync("LocalRecordsModule.Dialogs.ConfirmResetDialog"));
+        _localRecordsService.Verify(m => m.ResetLocalRecordsAsync(), Times.Never);
+        AuditEventBuilder.Verify(m => m.Success(), Times.Never);
+        AuditEventBuilder.Verify(m => m.Error(), Times.Never);
+    }
+}
diff --git a/tests/Modules/LocalRecordsModule.Tests/Controllers/WidgetUpdateControllerTests.cs b/tests/Modules/LocalRecordsModule.Tests/Controllers/WidgetUpdateControllerTests.cs
new file mode 100644
index 000000000..546fa2137
--- /dev/null
+++ b/tests/Modules/LocalRecordsModule.Tests/Controllers/WidgetUpdateControllerTests.cs
@@ -0,0 +1,70 @@
+using EvoSC.Common.Interfaces.Models;
+using EvoSC.Common.Interfaces.Models.Enums;
+using EvoSC.Common.Interfaces.Services;
+using EvoSC.Common.Models.Maps;
+using EvoSC.Common.Models.Players;
+using EvoSC.Modules.Official.LocalRecordsModule.Controllers;
+using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
+using EvoSC.Modules.Official.PlayerRecords.Database.Models;
+using EvoSC.Modules.Official.PlayerRecords.Events;
+using EvoSC.Testing.Controllers;
+using GbxRemoteNet.Events;
+using Moq;
+
+namespace LocalRecordsModule.Tests.Controllers;
+
+public class WidgetUpdateControllerTests : EventControllerTestBase<WidgetUpdateController>
+{
+    private readonly Mock<ILocalRecordsService> _localRecordsService = new();
+    private readonly Mock<IPlayerManagerService> _playerManagerService = new();
+    
+    public WidgetUpdateControllerTests()
+    {
+        InitMock(_localRecordsService, _playerManagerService);
+    }
+
+    [Fact]
+    public async Task Widget_Shown_For_Player_On_Join()
+    {
+        var player = new OnlinePlayer
+        {
+            Id = 1,
+            AccountId = "a467a996-eba5-44bf-9e2b-8543b50f39ae",
+            NickName = "snixtho",
+            UbisoftName = "snixtho",
+            State = PlayerState.Playing
+        };
+        
+        _playerManagerService.Setup(m => m.GetOnlinePlayerAsync("a467a996-eba5-44bf-9e2b-8543b50f39ae"))
+            .ReturnsAsync(player);
+        
+        await Controller.OnPlayerConnectAsync(null, new PlayerConnectGbxEventArgs { Login = "pGepluulRL-eK4VDtQ85rg" });
+        
+        _localRecordsService.Verify(m => m.ShowWidgetAsync(player));
+    }
+
+    [Fact]
+    public async Task Widget_Is_Updated_When_New_Map_Starts()
+    {
+        await Controller.OnBeginMapAsync(null, new MapGbxEventArgs());
+        
+        _localRecordsService.Verify(m => m.ShowWidgetToAllAsync());
+    }
+
+    [Fact]
+    public async Task Local_Records_Are_Updated_When_Player_Gets_Pb()
+    {
+        var record = new DbPlayerRecord();
+        var player = new Player();
+        var map = new Map();
+
+        var args = new PbRecordUpdateEventArgs
+        {
+            Player = player, Map = map, Record = record, Status = RecordUpdateStatus.New
+        };
+
+        await Controller.OnPbAsync(null, args);
+        
+        _localRecordsService.Verify(m => m.UpdatePbAsync(record));
+    }
+}
diff --git a/tests/Modules/MapListModule.Tests/MapListModule.Tests.csproj b/tests/Modules/MapListModule.Tests/MapListModule.Tests.csproj
index 05167034a..478b4ee96 100644
--- a/tests/Modules/MapListModule.Tests/MapListModule.Tests.csproj
+++ b/tests/Modules/MapListModule.Tests/MapListModule.Tests.csproj
@@ -11,8 +11,8 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0"/>
-    <PackageReference Include="xunit" Version="2.6.3"/>
+    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
+    <PackageReference Include="xunit" Version="2.8.1" />
     <PackageReference Include="xunit.runner.visualstudio" Version="2.6.2">
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
       <PrivateAssets>all</PrivateAssets>
diff --git a/tests/Modules/MapQueueModuleTests/MapQueueModuleTests.csproj b/tests/Modules/MapQueueModuleTests/MapQueueModuleTests.csproj
index 98c22487c..a1b6670c6 100644
--- a/tests/Modules/MapQueueModuleTests/MapQueueModuleTests.csproj
+++ b/tests/Modules/MapQueueModuleTests/MapQueueModuleTests.csproj
@@ -11,7 +11,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0"/>
+    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
     <PackageReference Include="xunit.runner.visualstudio" Version="2.6.5">
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
       <PrivateAssets>all</PrivateAssets>
diff --git a/tests/Modules/MapsModule.Tests/MapsModule.Tests.csproj b/tests/Modules/MapsModule.Tests/MapsModule.Tests.csproj
index 596cf2d39..a8d287d31 100644
--- a/tests/Modules/MapsModule.Tests/MapsModule.Tests.csproj
+++ b/tests/Modules/MapsModule.Tests/MapsModule.Tests.csproj
@@ -11,12 +11,12 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
         <PackageReference Include="SonarAnalyzer.CSharp" Version="9.15.0.81779">
           <PrivateAssets>all</PrivateAssets>
           <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/Modules/MatchManagerModule.Tests/MatchManagerModule.Tests.csproj b/tests/Modules/MatchManagerModule.Tests/MatchManagerModule.Tests.csproj
index 4bff1a19b..85a98614d 100644
--- a/tests/Modules/MatchManagerModule.Tests/MatchManagerModule.Tests.csproj
+++ b/tests/Modules/MatchManagerModule.Tests/MatchManagerModule.Tests.csproj
@@ -10,8 +10,8 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/Modules/MatchReadyModule.Tests/MatchReadyModule.Tests.csproj b/tests/Modules/MatchReadyModule.Tests/MatchReadyModule.Tests.csproj
index 99c1e0f16..4f029621e 100644
--- a/tests/Modules/MatchReadyModule.Tests/MatchReadyModule.Tests.csproj
+++ b/tests/Modules/MatchReadyModule.Tests/MatchReadyModule.Tests.csproj
@@ -9,9 +9,9 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
         <PackageReference Include="Moq" Version="4.20.70" />
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/Modules/MatchTrackerModule.Tests/MatchTrackerModule.Tests.csproj b/tests/Modules/MatchTrackerModule.Tests/MatchTrackerModule.Tests.csproj
index 4611d84f1..60a5aa63c 100644
--- a/tests/Modules/MatchTrackerModule.Tests/MatchTrackerModule.Tests.csproj
+++ b/tests/Modules/MatchTrackerModule.Tests/MatchTrackerModule.Tests.csproj
@@ -11,8 +11,8 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/Modules/MotdModule.Tests/MotdModule.Tests.csproj b/tests/Modules/MotdModule.Tests/MotdModule.Tests.csproj
index 8d97ad595..078a6fa64 100644
--- a/tests/Modules/MotdModule.Tests/MotdModule.Tests.csproj
+++ b/tests/Modules/MotdModule.Tests/MotdModule.Tests.csproj
@@ -10,9 +10,9 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
         <PackageReference Include="MockHttpClient" Version="1.11.0" />
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/Modules/NextMapModule.Tests/NextMapModule.Tests.csproj b/tests/Modules/NextMapModule.Tests/NextMapModule.Tests.csproj
index d555b0d9c..8a0cf66e8 100644
--- a/tests/Modules/NextMapModule.Tests/NextMapModule.Tests.csproj
+++ b/tests/Modules/NextMapModule.Tests/NextMapModule.Tests.csproj
@@ -11,12 +11,12 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
         <PackageReference Include="SonarAnalyzer.CSharp" Version="9.15.0.81779">
             <PrivateAssets>all</PrivateAssets>
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/Modules/OpenPlanetModule.Tests/OpenPlanetModule.Tests.csproj b/tests/Modules/OpenPlanetModule.Tests/OpenPlanetModule.Tests.csproj
index f8570d763..b72b7e922 100644
--- a/tests/Modules/OpenPlanetModule.Tests/OpenPlanetModule.Tests.csproj
+++ b/tests/Modules/OpenPlanetModule.Tests/OpenPlanetModule.Tests.csproj
@@ -11,8 +11,8 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/Modules/Player.Tests/Player.Tests.csproj b/tests/Modules/Player.Tests/Player.Tests.csproj
index 2a7816f89..bc28570ad 100644
--- a/tests/Modules/Player.Tests/Player.Tests.csproj
+++ b/tests/Modules/Player.Tests/Player.Tests.csproj
@@ -11,12 +11,12 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
         <PackageReference Include="SonarAnalyzer.CSharp" Version="9.15.0.81779">
           <PrivateAssets>all</PrivateAssets>
           <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
diff --git a/tests/Modules/SetName.Tests/SetName.Tests.csproj b/tests/Modules/SetName.Tests/SetName.Tests.csproj
index 3dc204b12..471e9dbe9 100644
--- a/tests/Modules/SetName.Tests/SetName.Tests.csproj
+++ b/tests/Modules/SetName.Tests/SetName.Tests.csproj
@@ -11,12 +11,12 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
         <PackageReference Include="SonarAnalyzer.CSharp" Version="9.15.0.81779">
           <PrivateAssets>all</PrivateAssets>
           <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
-        <PackageReference Include="xunit" Version="2.6.3" />
+        <PackageReference Include="xunit" Version="2.8.1" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.5.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>

From 1ceb147dc8a8708d18d623910affd60b0e1494e8 Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Sun, 30 Jun 2024 15:42:47 +0200
Subject: [PATCH 16/18] add comments

---
 .../Database/ILocalRecordRepository.cs        | 59 +++++++++++++++++++
 .../Interfaces/ILocalRecord.cs                | 15 +++++
 .../Services/ILocalRecordsService.cs          | 26 ++++++++
 .../LocalRecordsManialinkControllerTests.cs   | 10 ++++
 .../LocalRecordsModuleTests.cs                | 32 ++++++++++
 5 files changed, 142 insertions(+)
 create mode 100644 tests/Modules/LocalRecordsModule.Tests/LocalRecordsModuleTests.cs

diff --git a/src/Modules/LocalRecordsModule/Interfaces/Database/ILocalRecordRepository.cs b/src/Modules/LocalRecordsModule/Interfaces/Database/ILocalRecordRepository.cs
index 3a3b46f1b..cccc0a91b 100644
--- a/src/Modules/LocalRecordsModule/Interfaces/Database/ILocalRecordRepository.cs
+++ b/src/Modules/LocalRecordsModule/Interfaces/Database/ILocalRecordRepository.cs
@@ -6,13 +6,72 @@ namespace EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Database;
 
 public interface ILocalRecordRepository
 {
+    /// <summary>
+    /// Get all local records from a map by its ID.
+    /// </summary>
+    /// <param name="mapId">ID of the map to get local records from.</param>
+    /// <returns></returns>
     public Task<IEnumerable<DbLocalRecord>> GetLocalRecordsOfMapByIdAsync(long mapId);
+    
+    /// <summary>
+    /// Add a new local record, or update it if its better than existing one.
+    /// </summary>
+    /// <param name="map">Map to add local record to.</param>
+    /// <param name="record">The record to add.</param>
+    /// <returns></returns>
     public Task<DbLocalRecord?> AddOrUpdateRecordAsync(IMap map, IPlayerRecord record);
+    
+    /// <summary>
+    /// Add multiple local records to a map.
+    /// </summary>
+    /// <param name="map">Map to add local records to.</param>
+    /// <param name="records">Records to add.</param>
+    /// <returns></returns>
     public Task AddRecordsAsync(IMap map, IEnumerable<IPlayerRecord> records);
+    
+    /// <summary>
+    /// Recalculate positions of all local records in a map. It will
+    /// also remove local records that does not fit within the configured
+    /// top N local records.
+    /// </summary>
+    /// <param name="map">Map to re-calculate local records for.</param>
+    /// <returns></returns>
     public Task<DbLocalRecord[]> RecalculatePositionsOfMapAsync(IMap map);
+    
+    /// <summary>
+    /// Get local records from a player.
+    /// </summary>
+    /// <param name="player">Player to get local records from.</param>
+    /// <returns></returns>
     public Task<IEnumerable<DbLocalRecord>> GetRecordsByPlayerAsync(IPlayer player);
+    
+    /// <summary>
+    /// Get the local record of a player for a map.
+    /// </summary>
+    /// <param name="player">The player to get the local record of.</param>
+    /// <param name="map">The map to get the local record for.</param>
+    /// <returns></returns>
     public Task<DbLocalRecord?> GetRecordOfPlayerInMapAsync(IPlayer player, IMap map);
+    
+    /// <summary>
+    /// Delete a local record for a player in a map.
+    /// </summary>
+    /// <param name="player">The player that has the local record.</param>
+    /// <param name="map">The map that contains the local record.</param>
+    /// <returns></returns>
     public Task DeleteRecordAsync(IPlayer player, IMap map);
+    
+    /// <summary>
+    /// Delete a local record.
+    /// </summary>
+    /// <param name="localRecord">The local record to remove.</param>
+    /// <returns></returns>
     public Task DeleteRecordAsync(ILocalRecord localRecord);
+    
+    /// <summary>
+    /// Delete all local records from a map.
+    /// </summary>
+    /// <param name="map">The map to clear all local records from.</param>
+    /// <returns></returns>
     public Task DeleteRecordsAsync(IMap map);
 }
diff --git a/src/Modules/LocalRecordsModule/Interfaces/ILocalRecord.cs b/src/Modules/LocalRecordsModule/Interfaces/ILocalRecord.cs
index 1742a30af..69b072655 100644
--- a/src/Modules/LocalRecordsModule/Interfaces/ILocalRecord.cs
+++ b/src/Modules/LocalRecordsModule/Interfaces/ILocalRecord.cs
@@ -5,8 +5,23 @@ namespace EvoSC.Modules.Official.LocalRecordsModule.Interfaces;
 
 public interface ILocalRecord
 {
+    /// <summary>
+    /// The ID of the record.
+    /// </summary>
     public long Id { get; }
+    
+    /// <summary>
+    /// The position of the record in the leaderboard.
+    /// </summary>
     public int Position { get; }
+    
+    /// <summary>
+    /// The map the record is set on.
+    /// </summary>
     public IMap Map { get; }
+    
+    /// <summary>
+    /// The player record.
+    /// </summary>
     public IPlayerRecord Record { get; }
 }
diff --git a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
index 347dc895f..1828bc803 100644
--- a/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Interfaces/Services/ILocalRecordsService.cs
@@ -5,10 +5,36 @@ namespace EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
 
 public interface ILocalRecordsService
 {
+    /// <summary>
+    /// Get all local records of the current map on the server.
+    /// </summary>
+    /// <returns></returns>
     public Task<ILocalRecord[]> GetLocalsOfCurrentMapAsync();
+    
+    /// <summary>
+    /// Show/Update the local records widget to a player.
+    /// </summary>
+    /// <param name="player"></param>
+    /// <returns></returns>
     public Task ShowWidgetAsync(IPlayer player);
+    
+    /// <summary>
+    /// Show/Update the local records widget to all players.
+    /// </summary>
+    /// <returns></returns>
     public Task ShowWidgetToAllAsync();
+    
+    /// <summary>
+    /// Update a player's local record for the current map.
+    /// </summary>
+    /// <param name="record"></param>
+    /// <returns></returns>
     public Task UpdatePbAsync(IPlayerRecord record);
 
+    /// <summary>
+    /// Remove all local records on all maps, and add them back based on the
+    /// registered PBs of players.
+    /// </summary>
+    /// <returns></returns>
     public Task ResetLocalRecordsAsync();
 }
diff --git a/tests/Modules/LocalRecordsModule.Tests/Controllers/LocalRecordsManialinkControllerTests.cs b/tests/Modules/LocalRecordsModule.Tests/Controllers/LocalRecordsManialinkControllerTests.cs
index 74671d24b..b9c6ee6b6 100644
--- a/tests/Modules/LocalRecordsModule.Tests/Controllers/LocalRecordsManialinkControllerTests.cs
+++ b/tests/Modules/LocalRecordsModule.Tests/Controllers/LocalRecordsManialinkControllerTests.cs
@@ -43,4 +43,14 @@ public async Task Reset_Records_Cancel_Does_Not_Resets_All_Records()
         AuditEventBuilder.Verify(m => m.Success(), Times.Never);
         AuditEventBuilder.Verify(m => m.Error(), Times.Never);
     }
+
+    [Fact]
+    public async Task Reset_Records_Error_Is_Reported()
+    {
+        _localRecordsService.Setup(m => m.ResetLocalRecordsAsync()).Throws<Exception>();
+        
+        await Assert.ThrowsAsync<Exception>(() => Controller.ConfirmResetAsync(true));
+        _server.Server.Verify(m => m.ErrorMessageAsync(It.IsAny<string>()));
+        AuditEventBuilder.Verify(m => m.Error());
+    }
 }
diff --git a/tests/Modules/LocalRecordsModule.Tests/LocalRecordsModuleTests.cs b/tests/Modules/LocalRecordsModule.Tests/LocalRecordsModuleTests.cs
new file mode 100644
index 000000000..076647489
--- /dev/null
+++ b/tests/Modules/LocalRecordsModule.Tests/LocalRecordsModuleTests.cs
@@ -0,0 +1,32 @@
+using EvoSC.Manialinks.Interfaces;
+using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
+using Moq;
+
+namespace LocalRecordsModule.Tests;
+
+public class LocalRecordsModuleTests
+{
+    [Fact]
+    public async Task Widget_Is_Shown_To_All_On_Enable()
+    {
+        var manialinks = new Mock<IManialinkManager>();
+        var localRecords = new Mock<ILocalRecordsService>();
+        var module = new EvoSC.Modules.Official.LocalRecordsModule.LocalRecordsModule(manialinks.Object, localRecords.Object);
+
+        await module.EnableAsync();
+        
+        localRecords.Verify(m => m.ShowWidgetToAllAsync());
+    }
+    
+    [Fact]
+    public async Task Widget_Is_Hidden_On_Disable()
+    {
+        var manialinks = new Mock<IManialinkManager>();
+        var localRecords = new Mock<ILocalRecordsService>();
+        var module = new EvoSC.Modules.Official.LocalRecordsModule.LocalRecordsModule(manialinks.Object, localRecords.Object);
+
+        await module.DisableAsync();
+
+        manialinks.Verify(m => m.HideManialinkAsync("LocalRecordsModule.LocalRecordsWidget"));
+    }
+}

From 0ca02f8f81e84d75f6e68d636962522c1988d5a4 Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Sun, 30 Jun 2024 15:42:57 +0200
Subject: [PATCH 17/18] add tests project

---
 .../LocalRecordsModule.Tests/GlobalUsings.cs  |   1 +
 .../LocalRecordsModule.Tests.csproj           |  37 ++
 .../Services/LocalRecordsServiceTests.cs      | 318 ++++++++++++++++++
 3 files changed, 356 insertions(+)
 create mode 100644 tests/Modules/LocalRecordsModule.Tests/GlobalUsings.cs
 create mode 100644 tests/Modules/LocalRecordsModule.Tests/LocalRecordsModule.Tests.csproj
 create mode 100644 tests/Modules/LocalRecordsModule.Tests/Services/LocalRecordsServiceTests.cs

diff --git a/tests/Modules/LocalRecordsModule.Tests/GlobalUsings.cs b/tests/Modules/LocalRecordsModule.Tests/GlobalUsings.cs
new file mode 100644
index 000000000..c802f4480
--- /dev/null
+++ b/tests/Modules/LocalRecordsModule.Tests/GlobalUsings.cs
@@ -0,0 +1 @@
+global using Xunit;
diff --git a/tests/Modules/LocalRecordsModule.Tests/LocalRecordsModule.Tests.csproj b/tests/Modules/LocalRecordsModule.Tests/LocalRecordsModule.Tests.csproj
new file mode 100644
index 000000000..ae41820e7
--- /dev/null
+++ b/tests/Modules/LocalRecordsModule.Tests/LocalRecordsModule.Tests.csproj
@@ -0,0 +1,37 @@
+<Project Sdk="Microsoft.NET.Sdk">
+
+  <PropertyGroup>
+    <TargetFramework>net8.0</TargetFramework>
+    <ImplicitUsings>enable</ImplicitUsings>
+    <Nullable>enable</Nullable>
+
+    <IsPackable>false</IsPackable>
+    <IsTestProject>true</IsTestProject>
+  </PropertyGroup>
+
+  <ItemGroup>
+    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
+    <PackageReference Include="xunit" Version="2.8.1" />
+    <PackageReference Include="xunit.runner.visualstudio" Version="2.4.5">
+      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+      <PrivateAssets>all</PrivateAssets>
+    </PackageReference>
+    <PackageReference Include="coverlet.collector" Version="6.0.0">
+      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+      <PrivateAssets>all</PrivateAssets>
+    </PackageReference>
+  </ItemGroup>
+
+  <ItemGroup>
+    <ProjectReference Include="..\..\..\src\EvoSC.Testing\EvoSC.Testing.csproj" />
+    <ProjectReference Include="..\..\..\src\Modules\LocalRecordsModule\LocalRecordsModule.csproj" />
+    <ProjectReference Include="..\..\EvoSC.Common.Tests\EvoSC.Common.Tests.csproj" />
+  </ItemGroup>
+
+  <ItemGroup>
+    <Reference Include="Moq">
+      <HintPath>..\..\..\..\..\..\.nuget\packages\moq\4.20.70\lib\net6.0\Moq.dll</HintPath>
+    </Reference>
+  </ItemGroup>
+
+</Project>
diff --git a/tests/Modules/LocalRecordsModule.Tests/Services/LocalRecordsServiceTests.cs b/tests/Modules/LocalRecordsModule.Tests/Services/LocalRecordsServiceTests.cs
new file mode 100644
index 000000000..36085949f
--- /dev/null
+++ b/tests/Modules/LocalRecordsModule.Tests/Services/LocalRecordsServiceTests.cs
@@ -0,0 +1,318 @@
+using EvoSC.Common.Database.Models.Maps;
+using EvoSC.Common.Database.Models.Player;
+using EvoSC.Common.Interfaces;
+using EvoSC.Common.Interfaces.Models;
+using EvoSC.Common.Interfaces.Models.Enums;
+using EvoSC.Common.Interfaces.Services;
+using EvoSC.Common.Interfaces.Themes;
+using EvoSC.Common.Models.Maps;
+using EvoSC.Common.Models.Players;
+using EvoSC.Common.Tests;
+using EvoSC.Common.Themes;
+using EvoSC.Manialinks;
+using EvoSC.Manialinks.Interfaces;
+using EvoSC.Modules.Official.LocalRecordsModule.Config;
+using EvoSC.Modules.Official.LocalRecordsModule.Database.Models;
+using EvoSC.Modules.Official.LocalRecordsModule.Interfaces;
+using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Database;
+using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
+using EvoSC.Modules.Official.LocalRecordsModule.Services;
+using EvoSC.Modules.Official.PlayerRecords.Database.Models;
+using EvoSC.Modules.Official.PlayerRecords.Interfaces;
+using EvoSC.Modules.Official.PlayerRecords.Interfaces.Models;
+using EvoSC.Testing;
+using GbxRemoteNet.Interfaces;
+using Microsoft.Extensions.Logging;
+using Microsoft.VisualBasic;
+using Moq;
+
+namespace LocalRecordsModule.Tests.Services;
+
+using MockData = (
+    ILocalRecordsService Service,
+    Mock<IMapService> MapService,
+    Mock<ILocalRecordRepository> LocalRecordRepository,
+    Mock<IPlayerManagerService> PlayerManagerService,
+    Mock<IManialinkManager> ManialinkManager,
+    ILogger<LocalRecordsService> Logger,
+    Mock<ILocalRecordsSettings> Settings,
+    (Mock<IServerClient> Client, Mock<IGbxRemoteClient> Remote) Server,
+    Mock<IThemeManager> ThemeManager,
+    Mock<IPlayerRecordsRepository> PlayerRecordsRepository);
+
+public class LocalRecordsServiceTests
+{
+    private MockData NewLocalRecordsServiceMock()
+    {
+        var mapService = new Mock<IMapService>();
+        var localRecordRepository = new Mock<ILocalRecordRepository>();
+        var playerManager = new Mock<IPlayerManagerService>();
+        var manialinkManager = new Mock<IManialinkManager>();
+        var logger = LoggerSetup.CreateLogger<LocalRecordsService>();
+        var settings = new Mock<ILocalRecordsSettings>();
+        var server = Mocking.NewServerClientMock();
+        var themeManager = new Mock<IThemeManager>();
+        var playerRecordsRepository = new Mock<IPlayerRecordsRepository>();
+
+        themeManager.SetupGet(m => m.Theme)
+            .Returns(new DynamicThemeOptions(new Dictionary<string, object> { { "Info", "FFF" } }));
+        
+        var localRecordsService = new LocalRecordsService(
+            mapService.Object,
+            localRecordRepository.Object,
+            playerManager.Object,
+            manialinkManager.Object,
+            logger,
+            settings.Object,
+            server.Client.Object,
+            themeManager.Object,
+            playerRecordsRepository.Object
+        );
+
+        return (
+            localRecordsService,
+            mapService,
+            localRecordRepository,
+            playerManager,
+            manialinkManager,
+            logger,
+            settings,
+            server,
+            themeManager,
+            playerRecordsRepository
+        );
+    }
+
+    private (IMap Map, IPlayer Player) SetupMockRecords(MockData mock)
+    {
+        var currentMap = new Map { Id = 1234, Uid = "my map" };
+        var player = new Player { Id = 3, AccountId = "myplayer", NickName = "my player" };
+
+        DbPlayerRecord[] records =
+        [
+            new DbPlayerRecord
+            {
+                Id = 1,
+                MapId = currentMap.Id,
+                Score = 1337,
+                RecordType = PlayerRecordType.Time,
+                DbMap = new DbMap(currentMap),
+                DbPlayer = new DbPlayer{ Id = 2, AccountId = "myplayer2", NickName = "my player2"}
+            },
+            new DbPlayerRecord
+            {
+                Id = 2,
+                MapId = currentMap.Id,
+                Score = 1337,
+                RecordType = PlayerRecordType.Time,
+                DbMap = new DbMap(currentMap),
+                DbPlayer = new DbPlayer{ Id = 3, AccountId = "myplayer3", NickName = "my player3"}
+            },
+            new DbPlayerRecord
+            {
+                Id = 3,
+                MapId = currentMap.Id,
+                Score = 1337,
+                RecordType = PlayerRecordType.Time,
+                DbMap = new DbMap(currentMap),
+                DbPlayer =  new DbPlayer(player)
+            },
+            new DbPlayerRecord
+            {
+                Id = 4,
+                MapId = currentMap.Id,
+                Score = 1337,
+                RecordType = PlayerRecordType.Time,
+                DbMap = new DbMap(currentMap),
+                DbPlayer = new DbPlayer{ Id = 4, AccountId = "myplayer4", NickName = "my player4"}
+            },
+            new DbPlayerRecord
+            {
+                Id = 5,
+                MapId = currentMap.Id,
+                Score = 1337,
+                RecordType = PlayerRecordType.Time,
+                DbMap = new DbMap(currentMap),
+                DbPlayer = new DbPlayer{ Id = 5, AccountId = "myplayer5", NickName = "my player5"}
+            },
+        ];
+
+        mock.MapService.Setup(m => m.GetCurrentMapAsync()).ReturnsAsync(currentMap);
+        mock.LocalRecordRepository.Setup(m => m.GetLocalRecordsOfMapByIdAsync(currentMap.Id))
+            .ReturnsAsync(records.Select(r => new DbLocalRecord
+            {
+                Id = 0,
+                MapId = (long)r.MapId!,
+                RecordId = r.Id,
+                Position = (int)r.Id,
+                DbMap = r.DbMap,
+                DbRecord = r
+            }));
+
+        return (currentMap, player);
+    }
+
+    [Fact]
+    public async Task GetLocalsOfCurrentMap_Returns_Local_Records_Of_CurrentMap()
+    {
+        var mock = NewLocalRecordsServiceMock();
+        SetupMockRecords(mock);
+        var records = await mock.Service.GetLocalsOfCurrentMapAsync();
+
+        Assert.Equal(5, records.Length);
+        Assert.All(records, r => Assert.Equal("my map", r.Map.Uid));
+    }
+
+    [Fact]
+    public async Task Current_Map_Not_Found_Throws_Error()
+    {
+        var mock = NewLocalRecordsServiceMock();
+
+        mock.MapService.Setup(m => m.GetCurrentMapAsync()).ReturnsAsync((IMap?)null);
+
+        await Assert.ThrowsAsync<InvalidOperationException>(() => mock.Service.GetLocalsOfCurrentMapAsync());
+    }
+
+    [Fact]
+    public async Task Widget_Is_Shown_To_Player()
+    {
+        var mock = NewLocalRecordsServiceMock();
+        var mockSetup = SetupMockRecords(mock);
+
+        await mock.Service.ShowWidgetAsync(mockSetup.Player);
+
+        mock.ManialinkManager.Verify(m =>
+            m.SendManialinkAsync(mockSetup.Player, "LocalRecordsModule.LocalRecordsWidget", It.IsAny<object>()));
+    }
+
+    [Fact]
+    public async Task Widget_Is_Shown_To_All()
+    {
+        var mock = NewLocalRecordsServiceMock();
+        var mockSetup = SetupMockRecords(mock);
+        var player1 = new OnlinePlayer { Id = 1, AccountId = "player1", State = PlayerState.Playing };
+        var player2 = new OnlinePlayer { Id = 2, AccountId = "player2", State = PlayerState.Playing };
+        var player3 = new OnlinePlayer { Id = 3, AccountId = "player3", State = PlayerState.Playing };
+
+        mock.PlayerManagerService
+            .Setup(m => m.GetOnlinePlayersAsync())
+            .ReturnsAsync([player1, player2, player3]);
+
+        var transaction = new Mock<IManialinkTransaction>();
+        mock.ManialinkManager.Setup(m => m.CreateTransaction()).Returns(transaction.Object);
+
+        await mock.Service.ShowWidgetToAllAsync();
+
+        transaction.Verify(m => m.SendManialinkAsync(player1, "LocalRecordsModule.LocalRecordsWidget", It.IsAny<object>()), Times.Once);
+        transaction.Verify(m => m.SendManialinkAsync(player2, "LocalRecordsModule.LocalRecordsWidget", It.IsAny<object>()), Times.Once);
+        transaction.Verify(m => m.SendManialinkAsync(player3, "LocalRecordsModule.LocalRecordsWidget", It.IsAny<object>()), Times.Once);
+    }
+
+    [Fact]
+    public async Task New_Pb_Is_Not_Good_Enough()
+    {
+        var mock = NewLocalRecordsServiceMock();
+        var newPb = new DbPlayerRecord
+        {
+            Id = 0,
+            PlayerId = 0,
+            MapId = null,
+            Score = 0,
+            RecordType = PlayerRecordType.Time,
+            Checkpoints = null,
+            CreatedAt = default,
+            UpdatedAt = default,
+            DbPlayer = null,
+            DbMap = null
+        };
+
+        mock.LocalRecordRepository
+            .Setup(m => m.GetRecordOfPlayerInMapAsync(newPb.Player, newPb.Map))
+            .ReturnsAsync((DbLocalRecord?)null);
+
+        mock.LocalRecordRepository
+            .Setup(m => m.AddOrUpdateRecordAsync(newPb.Map, newPb))
+            .ReturnsAsync((DbLocalRecord?)null);
+
+        await mock.Service.UpdatePbAsync(newPb);
+        
+        mock.Server.Client.Verify(m => m.InfoMessageAsync(It.IsAny<string>()), Times.Never);
+    }
+
+    [Fact]
+    public async Task New_Pb_Is_New_Local_Record()
+    {
+        var mock = NewLocalRecordsServiceMock();
+        var mockSetup = SetupMockRecords(mock);
+        var newPb = new DbPlayerRecord
+        {
+            PlayerId = 1, Score = 12345, RecordType = PlayerRecordType.Time, DbPlayer = new DbPlayer(mockSetup.Player)
+        };
+        var localRecord = new DbLocalRecord { DbRecord = newPb, Position = 1337 };
+
+        mock.LocalRecordRepository
+            .Setup(m => m.GetRecordOfPlayerInMapAsync(newPb.Player, newPb.Map))
+            .ReturnsAsync((DbLocalRecord?)null);
+
+        mock.LocalRecordRepository
+            .Setup(m => m.AddOrUpdateRecordAsync(newPb.Map, newPb))
+            .ReturnsAsync(localRecord);
+
+        await mock.Service.UpdatePbAsync(newPb);
+
+        mock.Server.Client.Verify(m => m.InfoMessageAsync(It.Is<string>(s => s.Contains("gained the"))), Times.Once);
+    }
+    
+    [Fact]
+    public async Task New_Pb_Is_Improved_From_Old_Record()
+    {
+        var mock = NewLocalRecordsServiceMock();
+        var mockSetup = SetupMockRecords(mock);
+        var newPb = new DbPlayerRecord
+        {
+            PlayerId = 1, Score = 12345, RecordType = PlayerRecordType.Time, DbPlayer = new DbPlayer(mockSetup.Player),
+        };
+        var oldPlayerRecord = new DbPlayerRecord()
+        {
+            Score = 123456, RecordType = PlayerRecordType.Time, DbPlayer = new DbPlayer(mockSetup.Player)
+        };
+        var oldRecord = new DbLocalRecord { DbRecord = oldPlayerRecord, Position = 1337};
+        var localRecord = new DbLocalRecord { DbRecord = newPb, Position = 1337 };
+
+        mock.LocalRecordRepository
+            .Setup(m => m.GetRecordOfPlayerInMapAsync(newPb.Player, newPb.Map))
+            .ReturnsAsync((DbLocalRecord?)oldRecord);
+
+        mock.LocalRecordRepository
+            .Setup(m => m.AddOrUpdateRecordAsync(newPb.Map, newPb))
+            .ReturnsAsync(localRecord);
+
+        await mock.Service.UpdatePbAsync(newPb);
+
+        mock.Server.Client.Verify(m => m.InfoMessageAsync(It.Is<string>(s => s.Contains("improved the"))), Times.Once);
+    }
+    
+    [Fact]
+    public async Task New_Pb_Is_Equaled_To_Old_Record()
+    {
+        var mock = NewLocalRecordsServiceMock();
+        var mockSetup = SetupMockRecords(mock);
+        var newPb = new DbPlayerRecord
+        {
+            PlayerId = 1, Score = 12345, RecordType = PlayerRecordType.Time, DbPlayer = new DbPlayer(mockSetup.Player),
+        };
+        var localRecord = new DbLocalRecord { DbRecord = newPb, Position = 1337 };
+
+        mock.LocalRecordRepository
+            .Setup(m => m.GetRecordOfPlayerInMapAsync(newPb.Player, newPb.Map))
+            .ReturnsAsync((DbLocalRecord?)localRecord);
+
+        mock.LocalRecordRepository
+            .Setup(m => m.AddOrUpdateRecordAsync(newPb.Map, newPb))
+            .ReturnsAsync(localRecord);
+
+        await mock.Service.UpdatePbAsync(newPb);
+
+        mock.Server.Client.Verify(m => m.InfoMessageAsync(It.Is<string>(s => s.Contains("equaled their"))), Times.Once);
+    }
+}

From e25c9a660369967a4d41e54ece885241f2acd286 Mon Sep 17 00:00:00 2001
From: snixtho <snixtho@gmail.com>
Date: Sun, 30 Jun 2024 16:04:01 +0200
Subject: [PATCH 18/18] fix sonarcloud issues

---
 .../Interfaces/IManialinkManager.cs           | 51 +++++++++++++++++--
 .../Util/GlobalManialinkUtils.cs              |  1 -
 .../Config/ILocalRecordsSettings.cs           |  1 -
 .../Controllers/CommandsController.cs         |  1 -
 .../Database/Models/DbLocalRecord.cs          |  1 -
 .../LocalRecordsModule/LocalRecordsModule.cs  |  1 -
 .../Services/LocalRecordsService.cs           |  1 -
 .../Services/PlayerRecordsService.cs          |  5 --
 8 files changed, 48 insertions(+), 14 deletions(-)

diff --git a/src/EvoSC.Manialinks/Interfaces/IManialinkManager.cs b/src/EvoSC.Manialinks/Interfaces/IManialinkManager.cs
index 16643879d..2ef28abf6 100644
--- a/src/EvoSC.Manialinks/Interfaces/IManialinkManager.cs
+++ b/src/EvoSC.Manialinks/Interfaces/IManialinkManager.cs
@@ -1,5 +1,4 @@
-using EvoSC.Common.Interfaces.Models;
-using EvoSC.Manialinks.Interfaces.Models;
+using EvoSC.Manialinks.Interfaces.Models;
 
 namespace EvoSC.Manialinks.Interfaces;
 
@@ -102,20 +101,66 @@ public interface IManialinkManager : IManialinkOperations
     ///
     /// This method allows for dynamic data updates by a callback method.
     /// </summary>
-    /// <param name="name"></param>
+    /// <param name="name">Name of the template to show.</param>
     /// <param name="setupData">Method that returns data to be sent.</param>
     /// <returns></returns>
     public Task SendPersistentManialinkAsync(string name, Func<Task<IDictionary<string, object?>>> setupData);
 
+    /// <summary>
+    /// Hides and removes a persistent mainalink.
+    /// </summary>
+    /// <param name="name">Name of the template to hide.</param>
+    /// <returns></returns>
     public Task RemovePersistentManialinkAsync(string name);
     
+    /// <summary>
+    /// Add a global variable that is accessible from all templates.
+    /// </summary>
+    /// <param name="name">Name of the variable.</param>
+    /// <param name="value">Value of the variable.</param>
+    /// <typeparam name="T">Variable type.</typeparam>
     public void AddGlobalVariable<T>(string name, T value);
+    
+    /// <summary>
+    /// Remove a global variable.
+    /// </summary>
+    /// <param name="name">Name of the variable to remove.</param>
     public void RemoveGlobalVariable(string name);
+    
+    /// <summary>
+    /// Remove all global variables.
+    /// </summary>
     public void ClearGlobalVariables();
 
+    /// <summary>
+    /// Get the rendered contents of a template.
+    /// </summary>
+    /// <param name="name">Name of the template.</param>
+    /// <param name="data">Data to send to the template.</param>
+    /// <returns></returns>
     public Task<string> PrepareAndRenderAsync(string name, IDictionary<string, object?> data);
+    
+    /// <summary>
+    /// Get the rendered contents of a template.
+    /// </summary>
+    /// <param name="name">Name of the template.</param>
+    /// <param name="data">Data to send to the template.</param>
+    /// <returns></returns>
     public Task<string> PrepareAndRenderAsync(string name, dynamic data);
+    
+    /// <summary>
+    /// Get the effective name of a template's name. This may change
+    /// depending on the theme.
+    /// </summary>
+    /// <param name="name">Original name of the template.</param>
+    /// <returns></returns>
     public string GetEffectiveName(string name);
 
+    /// <summary>
+    /// Create a new transaction for manialink operations.
+    /// Nothing is done until the transaction is committed,
+    /// and all operations are sent at once for speed.
+    /// </summary>
+    /// <returns></returns>
     public IManialinkTransaction CreateTransaction();
 }
diff --git a/src/EvoSC.Manialinks/Util/GlobalManialinkUtils.cs b/src/EvoSC.Manialinks/Util/GlobalManialinkUtils.cs
index f9fc15212..103432e67 100644
--- a/src/EvoSC.Manialinks/Util/GlobalManialinkUtils.cs
+++ b/src/EvoSC.Manialinks/Util/GlobalManialinkUtils.cs
@@ -108,7 +108,6 @@ public string StyledTime(IRaceTime time)
             .WithColor(time.Milliseconds + time.Seconds > 0 ? _theme.UI_TextPrimary : _theme.UI_TextMuted)
         );
 
-        Console.WriteLine(styledTime.ToString());
         return styledTime.ToString();
     }
 }
diff --git a/src/Modules/LocalRecordsModule/Config/ILocalRecordsSettings.cs b/src/Modules/LocalRecordsModule/Config/ILocalRecordsSettings.cs
index e77fe84f5..7a164b473 100644
--- a/src/Modules/LocalRecordsModule/Config/ILocalRecordsSettings.cs
+++ b/src/Modules/LocalRecordsModule/Config/ILocalRecordsSettings.cs
@@ -1,6 +1,5 @@
 using System.ComponentModel;
 using Config.Net;
-using EvoSC.Common.Config.Models;
 using EvoSC.Modules.Attributes;
 
 namespace EvoSC.Modules.Official.LocalRecordsModule.Config;
diff --git a/src/Modules/LocalRecordsModule/Controllers/CommandsController.cs b/src/Modules/LocalRecordsModule/Controllers/CommandsController.cs
index 1e8fc9714..73cb3a628 100644
--- a/src/Modules/LocalRecordsModule/Controllers/CommandsController.cs
+++ b/src/Modules/LocalRecordsModule/Controllers/CommandsController.cs
@@ -3,7 +3,6 @@
 using EvoSC.Common.Controllers;
 using EvoSC.Common.Controllers.Attributes;
 using EvoSC.Manialinks.Interfaces;
-using EvoSC.Modules.Official.LocalRecordsModule.Interfaces.Services;
 using EvoSC.Modules.Official.LocalRecordsModule.Permissions;
 
 namespace EvoSC.Modules.Official.LocalRecordsModule.Controllers;
diff --git a/src/Modules/LocalRecordsModule/Database/Models/DbLocalRecord.cs b/src/Modules/LocalRecordsModule/Database/Models/DbLocalRecord.cs
index d61761537..28f817467 100644
--- a/src/Modules/LocalRecordsModule/Database/Models/DbLocalRecord.cs
+++ b/src/Modules/LocalRecordsModule/Database/Models/DbLocalRecord.cs
@@ -1,5 +1,4 @@
 using EvoSC.Common.Database.Models.Maps;
-using EvoSC.Common.Database.Models.Player;
 using EvoSC.Common.Interfaces.Models;
 using EvoSC.Modules.Official.LocalRecordsModule.Interfaces;
 using EvoSC.Modules.Official.PlayerRecords.Database.Models;
diff --git a/src/Modules/LocalRecordsModule/LocalRecordsModule.cs b/src/Modules/LocalRecordsModule/LocalRecordsModule.cs
index e3e7222e9..047726ed6 100644
--- a/src/Modules/LocalRecordsModule/LocalRecordsModule.cs
+++ b/src/Modules/LocalRecordsModule/LocalRecordsModule.cs
@@ -1,4 +1,3 @@
-using EvoSC.Common.Interfaces.Services;
 using EvoSC.Manialinks.Interfaces;
 using EvoSC.Modules.Attributes;
 using EvoSC.Modules.Interfaces;
diff --git a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
index b3d46b4f8..99cb5d163 100644
--- a/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
+++ b/src/Modules/LocalRecordsModule/Services/LocalRecordsService.cs
@@ -172,7 +172,6 @@ private ILocalRecord[] GetRecordsWithPlayer(IPlayer player, ILocalRecord[] recor
 
         var maxRange = settings.MaxWidgetRows - settings.WidgetShowTop;
         var half = maxRange / 2;
-        var r = maxRange % 2;
 
         var lower = playerRecord.Position - half;
         var upper = lower + maxRange;
diff --git a/src/Modules/PlayerRecords/Services/PlayerRecordsService.cs b/src/Modules/PlayerRecords/Services/PlayerRecordsService.cs
index 9a4ab067a..d7f06a055 100644
--- a/src/Modules/PlayerRecords/Services/PlayerRecordsService.cs
+++ b/src/Modules/PlayerRecords/Services/PlayerRecordsService.cs
@@ -63,11 +63,6 @@ record = await recordsRepo.InsertRecordAsync(player, map, score, checkpoints);
         {
             return (record, score > record.Score ? RecordUpdateStatus.NotUpdated : RecordUpdateStatus.Equal);
         }
-
-        /* record.Score = score;
-        record.Checkpoints = string.Join(',', checkpoints);
-        record.UpdatedAt = DateTime.UtcNow;
-        await recordsRepo.UpdateRecordAsync(record); */
         
         record = await recordsRepo.InsertRecordAsync(player, map, score, checkpoints);