From d40d0af21f2a0542e4d31a0e64b5c30e67ea6f96 Mon Sep 17 00:00:00 2001 From: Natsumi Date: Sat, 9 Aug 2025 14:22:39 +1200 Subject: [PATCH] Delete screenshot metadata --- Dotnet/AppApi/Common/Screenshot.cs | 229 ++++++++++-------- Dotnet/ScreenshotMetadata/PNGFile.cs | 6 +- Dotnet/ScreenshotMetadata/PNGHelper.cs | 2 +- src/localization/en/en.json | 12 + src/stores/settings/advanced.js | 68 +++++- src/types/globals.d.ts | 2 + src/views/Settings/Settings.vue | 6 +- .../dialogs/ScreenshotMetadataDialog.vue | 27 +++ 8 files changed, 250 insertions(+), 102 deletions(-) diff --git a/Dotnet/AppApi/Common/Screenshot.cs b/Dotnet/AppApi/Common/Screenshot.cs index 96844365..809967c4 100644 --- a/Dotnet/AppApi/Common/Screenshot.cs +++ b/Dotnet/AppApi/Common/Screenshot.cs @@ -11,118 +11,155 @@ namespace VRCX; public partial class AppApi { - public string GetExtraScreenshotData(string path, bool carouselCache) + public string GetExtraScreenshotData(string path, bool carouselCache) + { + var fileName = Path.GetFileNameWithoutExtension(path); + var metadata = new JObject(); + + if (!File.Exists(path) || !path.EndsWith(".png")) + return null; + + var files = Directory.GetFiles(Path.GetDirectoryName(path), "*.png"); + + // Add previous/next file paths to metadata so the screenshot viewer carousel can request metadata for next/previous images in directory + if (carouselCache) { - var fileName = Path.GetFileNameWithoutExtension(path); - var metadata = new JObject(); - - if (!File.Exists(path) || !path.EndsWith(".png")) - return null; - - var files = Directory.GetFiles(Path.GetDirectoryName(path), "*.png"); - - // Add previous/next file paths to metadata so the screenshot viewer carousel can request metadata for next/previous images in directory - if (carouselCache) + var index = Array.IndexOf(files, path); + if (index > 0) { - var index = Array.IndexOf(files, path); - if (index > 0) - { - metadata.Add("previousFilePath", files[index - 1]); - } - if (index < files.Length - 1) - { - metadata.Add("nextFilePath", files[index + 1]); - } + metadata.Add("previousFilePath", files[index - 1]); + } + if (index < files.Length - 1) + { + metadata.Add("nextFilePath", files[index + 1]); } - - using var png = new PNGFile(path); - metadata.Add("fileResolution", PNGHelper.ReadResolution(png)); - - var creationDate = File.GetCreationTime(path); - metadata.Add("creationDate", creationDate.ToString("yyyy-MM-dd HH:mm:ss")); - - var fileSizeBytes = new FileInfo(path).Length; - metadata.Add("fileSizeBytes", fileSizeBytes.ToString()); - metadata.Add("fileName", fileName); - metadata.Add("filePath", path); - metadata.Add("fileSize", $"{(fileSizeBytes / 1024f / 1024f).ToString("0.00")} MB"); - - return metadata.ToString(Formatting.Indented); } - public string GetScreenshotMetadata(string path) + using var png = new PNGFile(path); + metadata.Add("fileResolution", PNGHelper.ReadResolution(png)); + + var creationDate = File.GetCreationTime(path); + metadata.Add("creationDate", creationDate.ToString("yyyy-MM-dd HH:mm:ss")); + + var fileSizeBytes = new FileInfo(path).Length; + metadata.Add("fileSizeBytes", fileSizeBytes.ToString()); + metadata.Add("fileName", fileName); + metadata.Add("filePath", path); + metadata.Add("fileSize", $"{(fileSizeBytes / 1024f / 1024f).ToString("0.00")} MB"); + + return metadata.ToString(Formatting.Indented); + } + + public string GetScreenshotMetadata(string path) + { + if (string.IsNullOrEmpty(path)) + return null; + + + var metadata = ScreenshotHelper.GetScreenshotMetadata(path); + + if (metadata == null) { - if (string.IsNullOrEmpty(path)) - return null; - - - var metadata = ScreenshotHelper.GetScreenshotMetadata(path); - - if (metadata == null) + var obj = new JObject { - var obj = new JObject - { - { "sourceFile", path }, - { "error", "Screenshot contains no metadata." } - }; - - return obj.ToString(Formatting.Indented); + { "sourceFile", path }, + { "error", "Screenshot contains no metadata." } }; - if (metadata.Error != null) - { - var obj = new JObject - { - { "sourceFile", path }, - { "error", metadata.Error } - }; + return obj.ToString(Formatting.Indented); + }; - return obj.ToString(Formatting.Indented); - } - - return JsonConvert.SerializeObject(metadata, Formatting.Indented, new JsonSerializerSettings - { - ContractResolver = new DefaultContractResolver - { - NamingStrategy = new CamelCaseNamingStrategy() // This'll serialize our .net property names to their camelCase equivalents. Ex; "FileName" -> "fileName" - } - }); - } - - public string FindScreenshotsBySearch(string searchQuery, int searchType = 0) + if (metadata.Error != null) { - var stopwatch = new Stopwatch(); - stopwatch.Start(); - - var searchPath = GetVRChatPhotosLocation(); - var screenshots = ScreenshotHelper.FindScreenshots(searchQuery, searchPath, (ScreenshotHelper.ScreenshotSearchType)searchType); - - JArray json = new JArray(); - - foreach (var screenshot in screenshots) + var obj = new JObject { - json.Add(screenshot.SourceFile); - } + { "sourceFile", path }, + { "error", metadata.Error } + }; - stopwatch.Stop(); - - logger.Info($"FindScreenshotsBySearch took {stopwatch.ElapsedMilliseconds}ms to complete."); - - return json.ToString(); + return obj.ToString(Formatting.Indented); } - public string GetLastScreenshot() + return JsonConvert.SerializeObject(metadata, Formatting.Indented, new JsonSerializerSettings { - // Get the last screenshot taken by VRChat - var path = GetVRChatPhotosLocation(); - if (!Directory.Exists(path)) - return null; - - // exclude folder names that contain "Prints" or "Stickers" - var imageFiles = Directory.GetFiles(path, "*.png", SearchOption.AllDirectories) - .Where(x => !Regex.IsMatch(x, @"\\Prints\\|\\Stickers\\", RegexOptions.IgnoreCase)); - var lastScreenshot = imageFiles.OrderByDescending(Directory.GetCreationTime).FirstOrDefault(); + ContractResolver = new DefaultContractResolver + { + NamingStrategy = new CamelCaseNamingStrategy() // This'll serialize our .net property names to their camelCase equivalents. Ex; "FileName" -> "fileName" + } + }); + } - return lastScreenshot; + public string FindScreenshotsBySearch(string searchQuery, int searchType = 0) + { + var stopwatch = new Stopwatch(); + stopwatch.Start(); + + var searchPath = GetVRChatPhotosLocation(); + var screenshots = ScreenshotHelper.FindScreenshots(searchQuery, searchPath, (ScreenshotHelper.ScreenshotSearchType)searchType); + + JArray json = new JArray(); + + foreach (var screenshot in screenshots) + { + json.Add(screenshot.SourceFile); } + + stopwatch.Stop(); + + logger.Info($"FindScreenshotsBySearch took {stopwatch.ElapsedMilliseconds}ms to complete."); + + return json.ToString(); + } + + public string GetLastScreenshot() + { + // Get the last screenshot taken by VRChat + var path = GetVRChatPhotosLocation(); + if (!Directory.Exists(path)) + return null; + + // exclude folder names that contain "Prints" or "Stickers" + var imageFiles = Directory.GetFiles(path, "*.png", SearchOption.AllDirectories) + .Where(x => !Regex.IsMatch(x, @"\\Prints\\|\\Stickers\\", RegexOptions.IgnoreCase)); + var lastScreenshot = imageFiles.OrderByDescending(Directory.GetCreationTime).FirstOrDefault(); + + return lastScreenshot; + } + + public bool DeleteScreenshotMetadata(string path) + { + if (string.IsNullOrEmpty(path) || !File.Exists(path) || !path.EndsWith(".png")) + return false; + + try + { + ScreenshotHelper.DeleteTextMetadata(path, true); + return true; + } + catch (Exception ex) + { + logger.Error(ex, "Failed to delete screenshot metadata for {0}", path); + return false; + } + } + + public void DeleteAllScreenshotMetadata() + { + var path = GetVRChatPhotosLocation(); + if (!Directory.Exists(path)) + return; + + var imageFiles = Directory.GetFiles(path, "*.png", SearchOption.AllDirectories); + foreach (var file in imageFiles) + { + try + { + ScreenshotHelper.DeleteTextMetadata(file, true); + } + catch (Exception ex) + { + logger.Error(ex, "Failed to delete screenshot metadata for {0}", file); + } + } + } } \ No newline at end of file diff --git a/Dotnet/ScreenshotMetadata/PNGFile.cs b/Dotnet/ScreenshotMetadata/PNGFile.cs index d8c04c9b..d5a87cde 100644 --- a/Dotnet/ScreenshotMetadata/PNGFile.cs +++ b/Dotnet/ScreenshotMetadata/PNGFile.cs @@ -1,4 +1,4 @@ -using System; +using System; using System.Collections.Generic; using System.IO; using System.Linq; @@ -43,7 +43,7 @@ public class PNGFile : IDisposable ReadAndCacheMetadata(); var chunk = metadataChunkCache.FirstOrDefault((chunk) => chunkTypeFilter.HasFlag(chunk.ChunkTypeEnum)); - if (chunk.IsZero()) + if (chunk == null || chunk.IsZero()) return null; return chunk; @@ -60,7 +60,7 @@ public class PNGFile : IDisposable public PNGChunk? GetChunkReverse(PNGChunkTypeFilter chunkTypeFilter) { var chunk = ReadChunkReverse(chunkTypeFilter); - if (chunk != null && chunk.IsZero()) + if (chunk == null || chunk.IsZero()) return null; return chunk; diff --git a/Dotnet/ScreenshotMetadata/PNGHelper.cs b/Dotnet/ScreenshotMetadata/PNGHelper.cs index e465020f..717176e2 100644 --- a/Dotnet/ScreenshotMetadata/PNGHelper.cs +++ b/Dotnet/ScreenshotMetadata/PNGHelper.cs @@ -36,7 +36,7 @@ namespace VRCX /// If true, the function searches from the end of the file using a reverse search bruteforce method. /// /// The text associated with the specified keyword, or null if not found. - public static string ReadTextChunk(string keyword, PNGFile pngFile, bool legacySearch = false) + public static string? ReadTextChunk(string keyword, PNGFile pngFile, bool legacySearch = false) { // Search for legacy text chunks created by old vrchat mods if (legacySearch) diff --git a/src/localization/en/en.json b/src/localization/en/en.json index 211c9d4e..5419fe5d 100644 --- a/src/localization/en/en.json +++ b/src/localization/en/en.json @@ -582,6 +582,13 @@ "header": "Save Instance Emoji To File", "description": "Save spawned emoji to your VRChat Pictures folder" }, + "delete_all_screenshot_metadata": { + "button": "Delete Screenshot Metadata", + "ask": "Are you sure you want to delete all VRC & VRCX screenshot metadata from your VRChat Pictures folder?", + "confirm": "Deleting all screenshot metadata is a very destructive action and cannot be undone! This includes all prints, sub folders and other unrelated images in your VRChat Pictures folder, if these photos are precious to you, please make a backup before proceeding.", + "confirm_yes": "Yes", + "confirm_no": "No" + }, "remote_database": { "header": "Remote Avatar Database", "enable": "Enable", @@ -1533,6 +1540,7 @@ "last_screenshot": "Last Screenshot", "copy_image": "Copy Image", "open_folder": "Open Folder", + "delete_metadata": "Delete Metadata", "upload": "Upload" }, "registry_backup": { @@ -1692,6 +1700,10 @@ }, "friend": { "load_failed": "Failed to load friends list, logging out" + }, + "screenshot_metadata": { + "deleted": "Screenshot metadata deleted", + "delete_failed": "Failed to delete screenshot metadata" } }, "prompt": { diff --git a/src/stores/settings/advanced.js b/src/stores/settings/advanced.js index 94e0ef8a..9b322b3f 100644 --- a/src/stores/settings/advanced.js +++ b/src/stores/settings/advanced.js @@ -512,6 +512,71 @@ export const useAdvancedSettingsStore = defineStore('AdvancedSettings', () => { ); } + function askDeleteAllScreenshotMetadata() { + $app.$confirm( + t( + 'view.settings.advanced.advanced.delete_all_screenshot_metadata.ask' + ), + { + confirmButtonText: t( + 'view.settings.advanced.advanced.delete_all_screenshot_metadata.confirm_yes' + ), + cancelButtonText: t( + 'view.settings.advanced.advanced.delete_all_screenshot_metadata.confirm_no' + ), + type: 'warning', + showInput: false, + callback: async (action) => { + if (action === 'confirm') { + deleteAllScreenshotMetadata(); + } + } + } + ); + } + + function deleteAllScreenshotMetadata() { + $app.$confirm( + t( + 'view.settings.advanced.advanced.delete_all_screenshot_metadata.confirm' + ), + { + confirmButtonText: t( + 'view.settings.advanced.advanced.save_instance_prints_to_file.crop_convert_old_confirm' + ), + cancelButtonText: t( + 'view.settings.advanced.advanced.save_instance_prints_to_file.crop_convert_old_cancel' + ), + type: 'warning', + showInput: false, + callback: async (action) => { + if (action === 'confirm') { + const msgBox = $app.$message({ + message: 'Batch metadata removal in progress...', + type: 'warning', + duration: 0 + }); + try { + await AppApi.DeleteAllScreenshotMetadata(); + $app.$message({ + message: 'Batch metadata removal complete', + type: 'success' + }); + } catch (err) { + console.error(err); + $app.$message({ + message: `Batch metadata removal failed: ${err}`, + type: 'error' + }); + } finally { + msgBox.close(); + } + } + } + } + ); + } + function resetUGCFolder() { setUGCFolderPath(''); } @@ -647,6 +712,7 @@ export const useAdvancedSettingsStore = defineStore('AdvancedSettings', () => { showVRChatConfig, promptAutoClearVRCXCacheFrequency, setSaveInstanceEmoji, - setVrcRegistryAutoBackup + setVrcRegistryAutoBackup, + askDeleteAllScreenshotMetadata }; }); diff --git a/src/types/globals.d.ts b/src/types/globals.d.ts index 0e72f3b3..e4119227 100644 --- a/src/types/globals.d.ts +++ b/src/types/globals.d.ts @@ -290,6 +290,8 @@ declare global { searchType?: number ): Promise; GetLastScreenshot(): Promise; + DeleteScreenshotMetadata(path: string): Promise; + DeleteAllScreenshotMetadata(): Promise; // Moderations GetVRChatModerations( diff --git a/src/views/Settings/Settings.vue b/src/views/Settings/Settings.vue index 12abd277..767b166d 100644 --- a/src/views/Settings/Settings.vue +++ b/src/views/Settings/Settings.vue @@ -1285,6 +1285,9 @@ :value="screenshotHelperCopyToClipboard" @change="setScreenshotHelperCopyToClipboard()" :long-label="true" /> + {{ + t('view.settings.advanced.advanced.delete_all_screenshot_metadata.button') + }}
@@ -2113,7 +2116,8 @@ openUGCFolderSelector, showVRChatConfig, promptAutoClearVRCXCacheFrequency, - setSaveInstanceEmoji + setSaveInstanceEmoji, + askDeleteAllScreenshotMetadata } = advancedSettingsStore; const instanceTypes = ref([ diff --git a/src/views/Settings/dialogs/ScreenshotMetadataDialog.vue b/src/views/Settings/dialogs/ScreenshotMetadataDialog.vue index b7debe57..50fd6ca5 100644 --- a/src/views/Settings/dialogs/ScreenshotMetadataDialog.vue +++ b/src/views/Settings/dialogs/ScreenshotMetadataDialog.vue @@ -42,6 +42,13 @@ @click="uploadScreenshotToGallery" >{{ t('dialog.screenshot_metadata.upload') }} + {{ t('dialog.screenshot_metadata.delete_metadata') }}

@@ -279,6 +286,26 @@ }); }); } + function deleteMetadata(path) { + if (!path) { + return; + } + AppApi.DeleteScreenshotMetadata(path).then((result) => { + if (!result) { + $message({ + message: t('message.screenshot_metadata.delete_failed'), + type: 'error' + }); + return; + } + $message({ + message: t('message.screenshot_metadata.deleted'), + type: 'success' + }); + const D = props.screenshotMetadataDialog; + getAndDisplayScreenshot(D.metadata.filePath, true); + }); + } function uploadScreenshotToGallery() { const D = props.screenshotMetadataDialog; if (D.metadata.fileSizeBytes > 10000000) {