Changeset 1029

Timestamp:

Oct 24, 2006, 9:25:03 PM (16 years ago)

Author:

livings124

Message:

remove redundant methods

File:

• trunk/macosx/PrefsController.m (modified) (8 diffs)

trunk/macosx/PrefsController.m

@@ -45 +45 @@
 @interface PrefsController (Private)
 
-- (void) showGeneralPref: (id) sender;
-- (void) showTransfersPref: (id) sender;
-- (void) showBandwidthPref: (id) sender;
-- (void) showNetworkPref: (id) sender;
-
-- (void) setPrefView: (NSView *) view;
+- (void) setPrefView: (id) sender;
 
 - (void) folderSheetClosed: (NSOpenPanel *) openPanel returnCode: (int) code contextInfo: (void *) info;
@@ -100 +95 @@
 
     [fToolbar setSelectedItemIdentifier: TOOLBAR_GENERAL];
-    [self showGeneralPref: nil];
+    [self setPrefView: nil];
     
     //set download folder
@@ -297 +292 @@
         [item setImage: [NSImage imageNamed: @"Preferences.png"]];
         [item setTarget: self];
-        [item setAction: @selector(showGeneralPref:)];
+        [item setAction: @selector(setPrefView:)];
     }
     else if ([ident isEqualToString: TOOLBAR_TRANSFERS])
@@ -304 +299 @@
         [item setImage: [NSImage imageNamed: @"Transfers.png"]];
         [item setTarget: self];
-        [item setAction: @selector(showTransfersPref:)];
+        [item setAction: @selector(setPrefView:)];
     }
     else if ([ident isEqualToString: TOOLBAR_BANDWIDTH])
@@ -311 +306 @@
         [item setImage: [NSImage imageNamed: @"Bandwidth.png"]];
         [item setTarget: self];
-        [item setAction: @selector(showBandwidthPref:)];
+        [item setAction: @selector(setPrefView:)];
     }
     else if ([ident isEqualToString: TOOLBAR_NETWORK])
@@ -318 +313 @@
         [item setImage: [NSImage imageNamed: @"Network.png"]];
         [item setTarget: self];
-        [item setAction: @selector(showNetworkPref:)];
+        [item setAction: @selector(setPrefView:)];
     }
     else
@@ -882 +877 @@
 @implementation PrefsController (Private)
 
-- (void) showGeneralPref: (id) sender
-{
-    [self setPrefView: fGeneralView];
-}
-
-- (void) showTransfersPref: (id) sender
-{
-    [self setPrefView: fTransfersView];
-}
-
-- (void) showBandwidthPref: (id) sender
-{
-    [self setPrefView: fBandwidthView];
-}
-
-- (void) showNetworkPref: (id) sender
-{
-    [self setPrefView: fNetworkView];
-    
-    //make sure progress indicator hides itself
-    if ([fPortStatusImage image])
-        [fPortStatusProgress setDisplayedWhenStopped: NO];
-}
-
-- (void) setPrefView: (NSView *) view
-{
+- (void) setPrefView: (id) sender
+{
+    NSView * view = fGeneralView;
+    if (sender)
+    {
+        NSString * identifier = [sender itemIdentifier];
+        if ([identifier isEqualToString: TOOLBAR_TRANSFERS])
+            view = fTransfersView;
+        else if ([identifier isEqualToString: TOOLBAR_BANDWIDTH])
+            view = fBandwidthView;
+        else if ([identifier isEqualToString: TOOLBAR_NETWORK])
+            view = fNetworkView;
+        else;
+    }
+    
     NSWindow * window = [self window];
-    
     if ([window contentView] == view)
         return;
@@ -923 +906 @@
     [view setHidden: NO];
     
-    NSToolbarItem * item;
-    NSEnumerator * enumerator = [[fToolbar items] objectEnumerator];
-    while ((item = [enumerator nextObject]))
-        if ([[item itemIdentifier] isEqualToString: [fToolbar selectedItemIdentifier]])
-        {
-            [window setTitle: [item label]];
-            break;
-        }
+    //set title label
+    if (sender)
+        [window setTitle: [sender label]];
+    else
+    {
+        NSToolbarItem * item;
+        NSEnumerator * enumerator = [[fToolbar items] objectEnumerator];
+        while ((item = [enumerator nextObject]))
+            if ([[item itemIdentifier] isEqualToString: [fToolbar selectedItemIdentifier]])
+            {
+                [window setTitle: [item label]];
+                break;
+            }
+    }
+    
+    //for network view make sure progress indicator hides itself
+    if (view == fNetworkView && [fPortStatusImage image])
+        [fPortStatusProgress setDisplayedWhenStopped: NO];
 }