From 7c3ccc7f4a97102bfc5141dac938c090bfe42007 Mon Sep 17 00:00:00 2001 From: nixyri Date: Sat, 14 Feb 2026 12:59:05 -0500 Subject: [PATCH] Convert Int64s into strings before going into JavaScript for IDs. --- Valour/Client/ContextMenu/Menus/ChannelContextMenu.razor | 2 +- .../Client/ContextMenu/Menus/Member/MemberContextMenu.razor | 4 ++-- Valour/Client/ContextMenu/Menus/MessageContextMenu.razor | 2 +- Valour/Client/ContextMenu/Menus/PlanetContextMenu.razor | 2 +- Valour/Client/ContextMenu/Menus/UserContextMenu.razor | 4 +++- 5 files changed, 8 insertions(+), 6 deletions(-) diff --git a/Valour/Client/ContextMenu/Menus/ChannelContextMenu.razor b/Valour/Client/ContextMenu/Menus/ChannelContextMenu.razor index 54fe2912e..aeb5b1a7d 100644 --- a/Valour/Client/ContextMenu/Menus/ChannelContextMenu.razor +++ b/Valour/Client/ContextMenu/Menus/ChannelContextMenu.razor @@ -42,7 +42,7 @@ } private async Task OnClickCopyId(){ - await JsRuntime.InvokeVoidAsync("clipboardCopy.copyText", Data.Channel.Id); + await JsRuntime.InvokeVoidAsync("clipboardCopy.copyText", Data.Channel.Id.ToString()); ToastContainer.Instance.AddToast(new ToastData("Copied!", "Channel ID copied to clipboard", ToastProgressState.Success)); } diff --git a/Valour/Client/ContextMenu/Menus/Member/MemberContextMenu.razor b/Valour/Client/ContextMenu/Menus/Member/MemberContextMenu.razor index ca423c87c..79edb41b8 100644 --- a/Valour/Client/ContextMenu/Menus/Member/MemberContextMenu.razor +++ b/Valour/Client/ContextMenu/Menus/Member/MemberContextMenu.razor @@ -137,12 +137,12 @@ else } private async Task OnClickCopyMemberId(){ - await JsRuntime.InvokeVoidAsync("clipboardCopy.copyText", Data.Member.Id); + await JsRuntime.InvokeVoidAsync("clipboardCopy.copyText", Data.Member.Id.ToString()); ToastContainer.Instance.AddToast(new ToastData("Copied!", "Member ID copied to clipboard", ToastProgressState.Success)); } private async Task OnClickCopyUserId(){ - await JsRuntime.InvokeVoidAsync("clipboardCopy.copyText", Data.Member.UserId); + await JsRuntime.InvokeVoidAsync("clipboardCopy.copyText", Data.Member.UserId.ToString()); ToastContainer.Instance.AddToast(new ToastData("Copied!", "User ID copied to clipboard", ToastProgressState.Success)); } diff --git a/Valour/Client/ContextMenu/Menus/MessageContextMenu.razor b/Valour/Client/ContextMenu/Menus/MessageContextMenu.razor index bae80cd02..14947ba7e 100644 --- a/Valour/Client/ContextMenu/Menus/MessageContextMenu.razor +++ b/Valour/Client/ContextMenu/Menus/MessageContextMenu.razor @@ -150,7 +150,7 @@ } private async Task OnClickCopyId(){ - await JsRuntime.InvokeVoidAsync("clipboardCopy.copyText", Data.Message.Id); + await JsRuntime.InvokeVoidAsync("clipboardCopy.copyText", Data.Message.Id.ToString()); ToastContainer.Instance.AddToast(new ToastData("Copied!", "Message ID copied to clipboard", ToastProgressState.Success)); } } diff --git a/Valour/Client/ContextMenu/Menus/PlanetContextMenu.razor b/Valour/Client/ContextMenu/Menus/PlanetContextMenu.razor index bb0007f61..9bb05b6eb 100644 --- a/Valour/Client/ContextMenu/Menus/PlanetContextMenu.razor +++ b/Valour/Client/ContextMenu/Menus/PlanetContextMenu.razor @@ -94,7 +94,7 @@ private async Task OnClickCopyId() { - await JsRuntime.InvokeVoidAsync("clipboardCopy.copyText", Data.Planet.Id); + await JsRuntime.InvokeVoidAsync("clipboardCopy.copyText", Data.Planet.Id.ToString()); ToastContainer.Instance.AddToast(new ToastData("Copied!", "Planet ID copied to clipboard", ToastProgressState.Success)); } diff --git a/Valour/Client/ContextMenu/Menus/UserContextMenu.razor b/Valour/Client/ContextMenu/Menus/UserContextMenu.razor index fe2db3fba..beb70f013 100644 --- a/Valour/Client/ContextMenu/Menus/UserContextMenu.razor +++ b/Valour/Client/ContextMenu/Menus/UserContextMenu.razor @@ -90,8 +90,10 @@ else ToastContainer.Instance.AddToast(new ToastData("Copied!", "Username copied to clipboard", ToastProgressState.Success)); } + // I think this and the one for Member should be consolidated to a single + // function, they appear to do the exact same thing, and it's duplicate work. private async Task OnClickCopyUserId(){ - await JsRuntime.InvokeVoidAsync("clipboardCopy.copyText", Data.User.Id); + await JsRuntime.InvokeVoidAsync("clipboardCopy.copyText", Data.User.Id.ToString()); ToastContainer.Instance.AddToast(new ToastData("Copied!", "User ID copied to clipboard", ToastProgressState.Success)); }