From c8e8eaa7371e47c127f5448cc25e0180f7ea4dad Mon Sep 17 00:00:00 2001 From: Casper Sparre Date: Tue, 14 Oct 2025 00:35:31 +0200 Subject: [PATCH] Fixed parameters to ensure no null values are passed incorrectly --- OF DL/Helpers/DBHelper.cs | 6 +++--- OF DL/Helpers/Interfaces/IDBHelper.cs | 2 +- OF DL/Program.cs | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/OF DL/Helpers/DBHelper.cs b/OF DL/Helpers/DBHelper.cs index 7516c2d..2126c32 100644 --- a/OF DL/Helpers/DBHelper.cs +++ b/OF DL/Helpers/DBHelper.cs @@ -254,9 +254,9 @@ namespace OF_DL.Helpers return result; } - public async Task UpdateUserInfo(User? user) + public async Task UpdateUserInfo(string username, User? user) { - if (user is null) + if (user?.id is null) return; SqliteConnection connection = await GetAndOpenConnectionAsync($"Data Source={Directory.GetCurrentDirectory()}/users.db"); @@ -270,7 +270,7 @@ namespace OF_DL.Helpers ); cmdInfo.Parameters.AddWithValue("@userId", user.id); - cmdInfo.Parameters.AddWithValue("@name", user.name); + cmdInfo.Parameters.AddWithValue("@name", user.name ?? user.username ?? username); cmdInfo.Parameters.AddWithValue("@about", user.about); cmdInfo.Parameters.AddWithValue("@expiresOn", user.subscribedByExpireDate); cmdInfo.Parameters.AddWithValue("@photoCount", user.photosCount ?? 0); diff --git a/OF DL/Helpers/Interfaces/IDBHelper.cs b/OF DL/Helpers/Interfaces/IDBHelper.cs index 0a83ae5..a978f1c 100644 --- a/OF DL/Helpers/Interfaces/IDBHelper.cs +++ b/OF DL/Helpers/Interfaces/IDBHelper.cs @@ -15,6 +15,6 @@ namespace OF_DL.Helpers Task GetStoredFileSize(string folder, long media_id, string api_type); Task CheckDownloaded(string folder, long media_id, string api_type); Task GetMostRecentPostDate(string folder); - Task UpdateUserInfo(User? user); + Task UpdateUserInfo(string username, User? user); } } diff --git a/OF DL/Program.cs b/OF DL/Program.cs index bcc0207..862e908 100644 --- a/OF DL/Program.cs +++ b/OF DL/Program.cs @@ -984,7 +984,7 @@ public class Program { Log.Information("Updating User Info for for: {Username:l}"); User? user_info = await m_ApiHelper.GetUserInfo($"/users/{username}"); - await dbHelper.UpdateUserInfo(user_info); + await dbHelper.UpdateUserInfo(username, user_info); } catch (Exception ex) {