Changeset 7153

Timestamp:

Nov 25, 2008, 3:04:17 AM (13 years ago)

Author:

livings124

Message:

#1029 show a warning dialog when removing a transfer's data if its directory contains extra files

Location:

trunk/macosx

Files:

• Defaults.plist (modified) (1 diff)
• FileListNode.h (modified) (1 diff)
• FileListNode.m (modified) (1 diff)
• InfoWindowController.m (modified) (3 diffs)
• PrefsController.m (modified) (1 diff)
• Torrent.m (modified) (1 diff)

trunk/macosx/Defaults.plist

@@ -173 +173 @@
         <key>UseIncompleteDownloadFolder</key>
         <false/>
+        <key>WarningCheckContentsForRemove</key>
+        <true/>
         <key>WarningCreatorBlankAddress</key>
         <true/>

trunk/macosx/FileListNode.h

@@ -53 +53 @@
 - (NSArray *) children;
 
+- (BOOL) containsPath: (NSString *) fullPath;
+
 @end

trunk/macosx/FileListNode.m

@@ -129 +129 @@
 }
 
+- (BOOL) containsPath: (NSString *) fullPath
+{
+    if ([fullPath isEqualToString: fPath])
+        return YES;
+    
+    if (fIsFolder && [fullPath hasPrefix: fPath])
+    {
+        NSEnumerator * enumerator = [fChildren objectEnumerator];
+        FileListNode * node;
+        while ((node = [enumerator nextObject]))
+            if ([node containsPath: fullPath])
+                return YES;
+    }
+    
+    return NO;
+}
+
 @end
 

trunk/macosx/InfoWindowController.m

@@ -997 +997 @@
         
         //determing status strings from flags 
-        NSMutableArray * statusArray = [NSMutableArray arrayWithCapacity: 3];
+        NSMutableArray * statusArray = [NSMutableArray arrayWithCapacity: 6];
         NSString * flags = [peer objectForKey: @"Flags"];
         
@@ -1003 +1003 @@
             [statusArray addObject: NSLocalizedString(@"Currently downloading (interested and not choked)",
                 "Inspector -> peer -> status")];
-        else if ([flags rangeOfString: @"d"].location != NSNotFound)
+        if ([flags rangeOfString: @"d"].location != NSNotFound)
             [statusArray addObject: NSLocalizedString(@"You want to download, but peer does not want to send (interested and choked)",
                 "Inspector -> peer -> status")];
-        else;
-        
         if ([flags rangeOfString: @"U"].location != NSNotFound)
             [statusArray addObject: NSLocalizedString(@"Currently uploading (interested and not choked)",
                 "Inspector -> peer -> status")];
-        else if ([flags rangeOfString: @"u"].location != NSNotFound)
+        if ([flags rangeOfString: @"u"].location != NSNotFound)
             [statusArray addObject: NSLocalizedString(@"Peer wants you to upload, but you do not want to (interested and choked)",
                 "Inspector -> peer -> status")];
-        else;
-        
         if ([flags rangeOfString: @"K"].location != NSNotFound)
             [statusArray addObject: NSLocalizedString(@"Peer is unchoking you, but you are not interested",
                 "Inspector -> peer -> status")];
-        
         if ([flags rangeOfString: @"?"].location != NSNotFound)
             [statusArray addObject: NSLocalizedString(@"You unchoked the peer, but the peer is not interested",
@@ -1025 +1020 @@
         
         if ([statusArray count] > 0)
-            [components addObject: [@"\n" stringByAppendingString: [statusArray componentsJoinedByString: @"\n\n"]]];
+        {
+            NSString * statusStrings = [statusArray componentsJoinedByString: @"\n\n"];
+            [components addObject: [@"\n" stringByAppendingString: statusStrings]];
+        }
         
         return [components componentsJoinedByString: @"\n"];

trunk/macosx/PrefsController.m

@@ -612 +612 @@
     [fDefaults removeObjectForKey: @"WarningRemoveBuiltInTracker"];
     [fDefaults removeObjectForKey: @"WarningInvalidOpen"];
+    [fDefaults removeObjectForKey: @"WarningCheckContentsForRemove"];
 }
 

trunk/macosx/Torrent.m

@@ -449 +449 @@
 - (void) trashData
 {
+    if ([self isFolder] && [fDefaults boolForKey: @"WarningCheckContentsForRemove"])
+    {
+        NSEnumerator * enumerator = [[NSFileManager defaultManager] enumeratorAtPath: [self dataLocation]];
+        NSString * file;
+        while ((file = [enumerator nextObject]))
+        {
+            if ([[file lastPathComponent] hasPrefix: @"."])
+                continue;
+            
+            file = [[self name] stringByAppendingPathComponent: file];
+            BOOL isExtra = YES;
+        
+            NSEnumerator * nodeEnumerator = [fFileList objectEnumerator];
+            FileListNode * node;
+            while ((node = [nodeEnumerator nextObject]))
+            {
+                if ([node containsPath: file])
+                {
+                    isExtra = NO;
+                    break;
+                }
+            }
+            
+            if (!isExtra)
+                continue;
+
+            NSLog(@"Extra file found: %@", file);
+            
+            NSAlert * alert = [[NSAlert alloc] init];
+            [alert setMessageText: [NSString stringWithFormat: NSLocalizedString(@"\"%@\" contains extra content.",
+                                                        "Delete folder with extra contents -> title"), [self name]]];
+            [alert setInformativeText: NSLocalizedString(@"The directory contains data that is not part of the transfer."
+                " Are you sure you want to move this directory to the trash?", "Delete folder with extra contents -> message")];
+            [alert addButtonWithTitle: NSLocalizedString(@"Remove", "Delete folder with extra contents -> button")];
+            [alert addButtonWithTitle: NSLocalizedString(@"Keep", "Delete folder with extra contents -> button")];
+            
+            BOOL onLeopard = [NSApp isOnLeopardOrBetter];
+            if (onLeopard)
+            {
+                [alert setShowsSuppressionButton: YES];
+                [[alert suppressionButton] setTitle: NSLocalizedString(@"Do not check directory contents again",
+                                                        "Delete folder with extra contents -> button")];
+            }
+            else
+                [alert addButtonWithTitle: NSLocalizedString(@"Never Check", "Delete folder with extra contents -> button")];
+
+            NSInteger result = [alert runModal];
+            if ((onLeopard ? [[alert suppressionButton] state] == NSOnState : result == NSAlertThirdButtonReturn))
+                [fDefaults setBool: NO forKey: @"WarningCheckContentsForRemove"];
+            [alert release];
+            
+            if (result == NSAlertSecondButtonReturn)
+                return;
+            else
+                break;
+        }
+    }
+    
     [self trashFile: [self dataLocation]];
 }