Bookmarks improvements #changed

Resources/Localization/en.lproj/Localizable.strings

@@ -3173,3 +3173,10 @@
 
 /* connection view : ssl : key file picker : wrong format error title */
 "“%@” is not a valid private key file." = "“%@” is not a valid private key file.";
+
+/* text shown when there are stale bookmarks */
+"App Sandbox Issue" = "App Sandbox Issue";
+"Stale Bookmarks" = "Stale Bookmarks";
+"You have stale secure bookmarks:\n\n%@\n\nWould you like to re-request access now?" = "You have stale secure bookmarks:\n\n%@\n\nWould you like to re-request access now?";
+"Yes" = "Yes";
+"A reminder of your stale secure bookmarks:\n\n%@\n" = "A reminder of your stale secure bookmarks:\n\n%@\n";

Resources/Localization/es-ES.lproj/Localizable.strings

@@ -3173,3 +3173,10 @@
 
 /* connection view : ssl : key file picker : wrong format error title */
 "“%@” is not a valid private key file." = "“%@” is not a valid private key file.";
+
+/* text shown when there are stale bookmarks */
+"App Sandbox Issue" = "App Sandbox Issue";
+"Stale Bookmarks" = "Stale Bookmarks";
+"You have stale secure bookmarks:\n\n%@\n\nWould you like to re-request access now?" = "You have stale secure bookmarks:\n\n%@\n\nWould you like to re-request access now?";
+"Yes" = "Yes";
+"A reminder of your stale secure bookmarks:\n\n%@\n" = "A reminder of your stale secure bookmarks:\n\n%@\n";

Source/Controllers/DataExport/SPExportController.m

@@ -202,8 +202,8 @@ - (instancetype)init
 		exporters = [[NSMutableArray alloc] init];
 		exportFiles = [[NSMutableArray alloc] init];
 		operationQueue = [[NSOperationQueue alloc] init];
-		bookmarks = [[NSMutableArray alloc] init];
-		
+        bookmarks = [NSMutableArray arrayWithArray:SecureBookmarkManager.sharedInstance.bookmarks];
+
 		showAdvancedView = NO;
 		showCustomFilenameView = NO;
 		serverLowerCaseTableNameValue = NSNotFound;
@@ -288,16 +288,7 @@ - (void)exportTables:(NSArray *)exportTables asFormat:(SPExportType)format using
 
 	// initially popuplate the tables list
 	[self refreshTableList:nil];
-
-	id o;
-	if((o = [prefs objectForKey:SPSecureBookmarks])){
-		[bookmarks setArray:o];
-	}
-	else{
-		SPLog(@"Could not load SPSecureBookmarks from prefs");
-		CLS_LOG(@"Could not load SPSecureBookmarks from prefs");
-	}
-
+
 	// overwrite defaults with user settings from last export
 	[self applySettingsFromDictionary:[prefs objectForKey:SPLastExportSettings] error:NULL];
 
@@ -630,47 +621,11 @@ - (IBAction)changeExportOutputPath:(id)sender
 			}
 
 			[self->exportPathField setStringValue:path];
-
-			// the code always seems to go into this block as the
-			// user has selected the folder and we have com.apple.security.files.user-selected.read-write
-			if([self->changeExportOutputPathPanel.URL startAccessingSecurityScopedResource] == YES){
-                SPLog(@"got access to: %@", self->changeExportOutputPathPanel.URL.absoluteString);
-                CLS_LOG(@"got access to: %@", self->changeExportOutputPathPanel.URL.absoluteString);
-
-				BOOL __block beenHereBefore = NO;
-
-				// have we been here before?
-				[self.bookmarks enumerateObjectsUsingBlock:^(NSDictionary *dict, NSUInteger idx, BOOL *stop) {
-
-					if(dict[self->changeExportOutputPathPanel.URL.absoluteString] != nil){
-						NSLog(@"beenHereBefore: %@", dict[self->changeExportOutputPathPanel.URL.absoluteString]);
-						beenHereBefore = YES;
-						*stop = YES;
-					}
-				}];
-
-				if(beenHereBefore == NO){
-					// create a bookmark
-					NSError *error = nil;
-					NSData *tmpAppScopedBookmark = [self->changeExportOutputPathPanel.URL bookmarkDataWithOptions:NSURLBookmarkCreationWithSecurityScope // this needs to be read-write
-																			 includingResourceValuesForKeys:nil
-																							  relativeToURL:nil
-																									  error:&error];
-					// save to prefs
-					if(tmpAppScopedBookmark && !error) {
-						[self->bookmarks addObject:@{self->changeExportOutputPathPanel.URL.absoluteString : tmpAppScopedBookmark}];
-						[self->prefs setObject:self->bookmarks forKey:SPSecureBookmarks];
-					}
-					else{
-						SPLog(@"Problem creating bookmark - %@ : %@",self->changeExportOutputPathPanel.URL.absoluteString, [error localizedDescription]);
-						CLS_LOG(@"Problem creating bookmark - %@ : %@",self->changeExportOutputPathPanel.URL.absoluteString, [error localizedDescription]);
-					}
-				}
-			}
-			else{
-				SPLog(@"Problem startAccessingSecurityScopedResource for - %@",self->changeExportOutputPathPanel.URL.absoluteString);
-				CLS_LOG(@"Problem startAccessingSecurityScopedResource for - %@",self->changeExportOutputPathPanel.URL.absoluteString);
-			}
+
+            // this needs to be read-write
+            if([SecureBookmarkManager.sharedInstance addBookMarkForUrl:self->changeExportOutputPathPanel.URL options:(NSURLBookmarkCreationWithSecurityScope)] == YES){
+                SPLog(@"addBookMarkForUrl success");
+            }
         }
     }];		
 }
@@ -3162,27 +3117,12 @@ - (BOOL)applySettingsFromDictionary:(NSDictionary *)dict error:(NSError **)err
 	// look up that bookmark and request access
 	if(bookmarks.count > 0){
 		if((o = [dict objectForKey:@"exportPath"])) [exportPathField setStringValue:o];
-
-		NSError __block *error = nil;
-
-		[self.bookmarks enumerateObjectsUsingBlock:^(NSDictionary *dict2, NSUInteger idx, BOOL *stop) {
-
-			NSString *tmpStr = [NSURL fileURLWithPath:[exportPathField stringValue] isDirectory:YES].absoluteString;
-
-			if(dict2[tmpStr] != nil){
-				self.userChosenDirectory = [NSURL URLByResolvingBookmarkData:dict2[tmpStr]
-																	 options:NSURLBookmarkResolutionWithSecurityScope
-															   relativeToURL:nil
-														 bookmarkDataIsStale:nil
-																	   error:&error];
-				*stop = YES;
-			}
-		}];
-
-		// if no bookmark was found this just calls against nil
-		if(!error){
-			[userChosenDirectory startAccessingSecurityScopedResource];
-		}
+
+        NSString *tmpStr = [NSURL fileURLWithPath:[exportPathField stringValue] isDirectory:YES].absoluteString;
+
+        // ret value can be nil
+        userChosenDirectory = [SecureBookmarkManager.sharedInstance bookMarkForFilename:tmpStr];
+
 	}
 
 	SPExportType et;

Source/Controllers/MainViewControllers/ConnectionView/SPConnectionController.h

@@ -232,7 +232,6 @@ typedef NS_ENUM(NSInteger, SPConnectionTimeZoneMode) {
 @property (readwrite, copy) NSString *connectionSSHKeychainItemAccount;
 @property (readwrite) BOOL useCompression;
 @property (readwrite, strong) NSMutableArray<NSDictionary<NSString *, id> *> *bookmarks;
-@property (readwrite, strong) NSMutableArray<NSURL *> *resolvedBookmarks;
 
 @property (readonly) BOOL isConnecting;
 @property (readonly) BOOL isEditingConnection;
@@ -260,7 +259,6 @@ typedef NS_ENUM(NSInteger, SPConnectionTimeZoneMode) {
 -(BOOL)validateCertFile:(NSURL *)url error:(NSError **)outError;
 -(BOOL)validateKeyFile:(NSURL *)url error:(NSError **)outError;
 -(void)showValidationAlertForError:(NSError*)err;
--(void)reRequestSecureAccess;
 -(BOOL)connected;
 
 // Favorites interaction