ensure stopAccessing called

mlemgroup · Oct 15, 2024 · dc1366e · dc1366e
1 parent ce6fe08
commit dc1366e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/Mlem/App/Views/Root/Tabs/Settings/ImportExportSettingsPage.swift b/Mlem/App/Views/Root/Tabs/Settings/ImportExportSettingsPage.swift
@@ -33,9 +33,10 @@ struct ImportExportSettingsPage: View {
                 isPresented: $importingSettingsFile,
                 allowedContentTypes: [.json]
             ) { result in
+                var fileUrl: URL?
                 do {
-                    let fileUrl = try result.get()
-                    if fileUrl.startAccessingSecurityScopedResource() {
+                    fileUrl = try result.get()
+                    if let fileUrl, fileUrl.startAccessingSecurityScopedResource() {
                         let fileData = try Data(contentsOf: fileUrl, options: .mappedIfSafe)
                         fileUrl.stopAccessingSecurityScopedResource()
 
@@ -47,6 +48,7 @@ struct ImportExportSettingsPage: View {
                         ToastModel.main.add(.failure("Failed to Import Settings"))
                     }
                 } catch {
+                    fileUrl?.stopAccessingSecurityScopedResource()
                     handleError(error)
                 }
             }

diff --git a/Mlem/Localizable.xcstrings b/Mlem/Localizable.xcstrings
@@ -460,6 +460,9 @@
     },
     "Failed to delete post!" : {
 
+    },
+    "Failed to Import Settings" : {
+
     },
     "Failed to lock post" : {