Updated Endpoints

Updated all endpoints to use Update() method of the model, instead of
trying to use the data from the REST object directly to update the
database.  WHen doing so, tracking gets busted in EFCore, so instead,
will go through and update all properties of a model from the database
object, with the data from the REST object.

FreeTubeSync/EndPoints/HistoryEndpoint.cs

@@ -20,7 +20,10 @@ public static class HistoryEndpoint
             if (results == null)
                 await repository.AddAsync(history, ct);
             else
-                await repository.UpdateAsync(history, ct);
+            {
+                results.Update(history);
+                await repository.UpdateAsync(results, ct);
+            }
             return Results.Ok();
         });
 

FreeTubeSync/EndPoints/PlaylistEndpoint.cs

@@ -20,7 +20,11 @@ public static class PlaylistEndpoint
             if (results == null)
                 await repository.AddAsync(playlist, ct);
             else
-                await repository.UpdateAsync(playlist, ct);
+            {
+                results.Update(playlist);
+                await repository.UpdateAsync(results, ct);
+            }
+
             return Results.Ok();
         });
 

FreeTubeSync/EndPoints/ProfileEndpoint.cs

@@ -20,7 +20,11 @@ public static class ProfileEndpoint
             if (res == null)
                 await repository.AddAsync(profile, ct);
             else
-                await repository.UpdateAsync(profile, ct);
+            {
+                res.Update(profile);
+                await repository.UpdateAsync(res, ct);
+            }
+
             return Results.Ok();
         });
 

FreeTubeSync/EndPoints/SearchHistoryEndpoint.cs

@@ -17,10 +17,14 @@ public static class SearchHistoryEndpoint
         group.MapPost("/", async (IRepository<SearchHistory> repository, CancellationToken ct, SearchHistory history) =>
         {
             var result = await repository.GetByIdAsync(history._id, ct);
-            if (result != null)
-                await repository.UpdateAsync(history, ct);
-            else
+            if (result == null)
                 await repository.AddAsync(history, ct);
+            else
+            {
+                result.Update(history);
+                await repository.UpdateAsync(result, ct);
+            }
+
             return Results.Ok();
         });
 

FreeTubeSync/EndPoints/SettingEndpoint.cs

@@ -22,7 +22,10 @@ public static class SettingEndpoint
             if (res == null)
                 await repository.AddAsync(setting, ct);
             else
-                await repository.UpdateAsync(setting, ct);
+            {
+                res.Update(setting);
+                await repository.UpdateAsync(res, ct);
+            }
             return Results.Ok();
         });
 

FreeTubeSync/Model/Playlist.cs

@@ -1,5 +1,6 @@
 using System.ComponentModel.DataAnnotations;
 using System.Diagnostics.CodeAnalysis;
+using Microsoft.EntityFrameworkCore.Query.Internal;
 
 namespace FreeTubeSync.Model;
 
@@ -18,7 +19,20 @@ public class Playlist
     {
         if (other.playlistName != playlistName) playlistName = other.playlistName;
         if (other.@protected != @protected) @protected = other.@protected;
-        if (other.videos.Count != videos.Count) videos = other.videos;
+        var remove = new List<Video>();
+        foreach (var vid in videos)
+        {
+            if (other.videos.Any(x => x.playlistItemId == vid.playlistItemId)) continue;
+            remove.Add(vid);
+        }
+        videos.RemoveAll(x => remove.Contains(x));
+        remove.Clear();
+        foreach (var vid in other.videos)
+        {
+            if (videos.Any(x => x.playlistItemId == vid.playlistItemId)) continue;
+            remove.Add(vid);
+        }
+        videos.AddRange(remove);
         if (other.createdAt != createdAt) createdAt = other.createdAt;
         if (other.lastUpdatedAt != lastUpdatedAt) lastUpdatedAt = other.lastUpdatedAt;
     }

FreeTubeSync/Model/Profile.cs

@@ -18,6 +18,19 @@ public class Profile
         if (other.name != name) name = other.name;
         if (other.bgColor != bgColor) bgColor = other.bgColor;
         if (other.textColor != textColor) textColor = other.textColor;
-        if (other.subscriptions.Count != subscriptions.Count) subscriptions = other.subscriptions;
+        var remove = new List<Subscription>();
+        foreach (var sub in subscriptions)
+        {
+            if (other.subscriptions.Any(x => x.id == sub.id)) continue;
+            remove.Add(sub);
+        }
+        subscriptions.RemoveAll(x => remove.Contains(x));
+        remove.Clear();
+        foreach (var sub in other.subscriptions)
+        {
+            if (subscriptions.Any(x => x.id == sub.id)) continue;
+            remove.Add(sub);
+        }
+        subscriptions.AddRange(remove);
     }
 }

FreeTubeSync/Model/Setting.cs

@@ -22,4 +22,9 @@ public class Setting
 #pragma warning restore CS8603
         set => ValueJson = JsonSerializer.Serialize(value);
     }
+
+    public void Update(Setting other)
+    {
+        Value = other.Value;
+    }
 }

FreeTubeSync/Model/Subscription.cs

@@ -10,4 +10,10 @@ public class Subscription
     public required string id { get; set; }
     public required string name { get; set; }
     public string? thumbnail { get; set; }
+
+    public void Update(Subscription other)
+    {
+        name = other.name;
+        thumbnail = other.thumbnail;
+    }
 }

FreeTubeSync/Model/Video.cs

@@ -16,4 +16,16 @@ public class Video
     public long timeAdded { get; set; }
     [Key] public string playlistItemId { get; set; } = string.Empty;
     public string type { get; set; } = string.Empty;
+
+    public void Update(Video other)
+    {
+        videoId = other.videoId;
+        title = other.title;
+        author = other.author;
+        authorId = other.authorId;
+        lengthSeconds = other.lengthSeconds;
+        pubished = other.pubished;
+        timeAdded = other.timeAdded;
+        type = other.type;
+    }
 }