Create helper +[PreferencesController secureBookmarkDataForURL:].

diff --git a/Source/AppDelegate.m b/Source/AppDelegate.m
index e665c35f96ddd0c826589807d1b5e92b87e3d609..cab45e77842b5012dce26a2fe5ade9544ba5076f 100644 (file)
--- a/Source/AppDelegate.m
+++ b/Source/AppDelegate.m
@@ -132,11 +132,9 @@
       continue;
     }
     if (isStale) {
-      NSData* newBookmark = [url bookmarkDataWithOptions:NSURLBookmarkCreationWithSecurityScope | NSURLBookmarkCreationSecurityScopeAllowOnlyReadAccess
-                          includingResourceValuesForKeys:nil
-                                           relativeToURL:nil
-                                                   error:&error];
-      bookmarks[path] = newBookmark;
+      NSData* newBookmark = [PreferencesController secureBookmarkDataForURL:url];
+      bookmarks[url.absoluteString] = newBookmark;
+      [bookmarks removeObjectForKey:path];
     }
 
     if (![url startAccessingSecurityScopedResource]) {

diff --git a/Source/PreferencesController.h b/Source/PreferencesController.h
index dd7da37b62b96946ae9f6fac2ed173de17d341ef..34241fe024666248c5ca33e83b1dd2103a8efbcb 100644 (file)
--- a/Source/PreferencesController.h
+++ b/Source/PreferencesController.h
@@ -35,6 +35,8 @@
 
 - (IBAction)addFileAccess:(id)sender;
 
++ (NSData*)secureBookmarkDataForURL:(NSURL*)url;
+
 // panel switching
 - (IBAction)showGeneral:(id)sender;
 - (IBAction)showFileAccess:(id)sender;

diff --git a/Source/PreferencesController.m b/Source/PreferencesController.m
index ae64402cf13b0193e00b9fbd3fef8203c7b356a8..8f863d773204096e02a59c55b55dd607dd20fb05 100644 (file)
--- a/Source/PreferencesController.m
+++ b/Source/PreferencesController.m
@@ -57,15 +57,9 @@
 
   NSURL* url = panel.URL;
 
-  NSError* error;
-  NSData* secureBookmark = [url bookmarkDataWithOptions:NSURLBookmarkCreationWithSecurityScope | NSURLBookmarkCreationSecurityScopeAllowOnlyReadAccess
-                         includingResourceValuesForKeys:nil
-                                          relativeToURL:nil
-                                                  error:&error];
-  if (error) {
-    NSLog(@"Error creating secure bookmark: %@", error);
+  NSData* secureBookmark = [self.class secureBookmarkDataForURL:url];
+  if (!secureBookmark)
     return;
-  }
 
   NSDictionaryControllerKeyValuePair* pair = [self.fileAccessController newObject];
   pair.key = url.absoluteString;
@@ -73,6 +67,19 @@
   [self.fileAccessController addObject:pair];
 }
 
++ (NSData*)secureBookmarkDataForURL:(NSURL*)url
+{
+   NSError* error;
+   NSData* secureBookmark = [url bookmarkDataWithOptions:NSURLBookmarkCreationWithSecurityScope | NSURLBookmarkCreationSecurityScopeAllowOnlyReadAccess
+                          includingResourceValuesForKeys:nil
+                                           relativeToURL:nil
+                                                   error:&error];
+   if (error) {
+     NSLog(@"Error creating secure bookmark: %@", error);
+   }
+  return secureBookmark;
+}
+
 #pragma mark Panel Switching
 
 /**