Changing hotkeys in the prefs now writes to the prefs and rebuilds the menu.

[MenuTunes.git] / OldMainController.m

diff --git a/OldMainController.m b/OldMainController.m
index f1d170d..4171538 100755 (executable)
--- a/OldMainController.m
+++ b/OldMainController.m
@@ -144,15 +144,11 @@
         [[ratingMenu addItemWithTitle:[NSString stringWithFormat:@"%@%@%@%@%@", fullStarChar, fullStarChar, fullStarChar, emptyStarChar, emptyStarChar] action:@selector(selectSongRating:) keyEquivalent:@""] setTag:60];
         [[ratingMenu addItemWithTitle:[NSString stringWithFormat:@"%@%@%@%@%@", fullStarChar, fullStarChar, fullStarChar, fullStarChar, emptyStarChar] action:@selector(selectSongRating:) keyEquivalent:@""] setTag:80];
         [[ratingMenu addItemWithTitle:[NSString stringWithFormat:@"%@%@%@%@%@", fullStarChar, fullStarChar, fullStarChar, fullStarChar, fullStarChar] action:@selector(selectSongRating:) keyEquivalent:@""] setTag:100];
         [[ratingMenu addItemWithTitle:[NSString stringWithFormat:@"%@%@%@%@%@", fullStarChar, fullStarChar, fullStarChar, emptyStarChar, emptyStarChar] action:@selector(selectSongRating:) keyEquivalent:@""] setTag:60];
         [[ratingMenu addItemWithTitle:[NSString stringWithFormat:@"%@%@%@%@%@", fullStarChar, fullStarChar, fullStarChar, fullStarChar, emptyStarChar] action:@selector(selectSongRating:) keyEquivalent:@""] setTag:80];
         [[ratingMenu addItemWithTitle:[NSString stringWithFormat:@"%@%@%@%@%@", fullStarChar, fullStarChar, fullStarChar, fullStarChar, fullStarChar] action:@selector(selectSongRating:) keyEquivalent:@""] setTag:100];

-        

         [NSThread detachNewThreadSelector:@selector(startTimerInNewThread) toTarget:self withObject:nil];
         [NSThread detachNewThreadSelector:@selector(startTimerInNewThread) toTarget:self withObject:nil];

-        [self rebuildMenu];

         [self setupHotKeys];
         [self setupHotKeys];

+        [self rebuildMenu];

         isAppRunning = ITMTRemotePlayerRunning;
         isAppRunning = ITMTRemotePlayerRunning;

-        return;

     }
     }

-    
-    isAppRunning = ITMTRemotePlayerRunning;

 }
 
 - (void)applicationTerminated:(NSNotification *)note
 }
 
 - (void)applicationTerminated:(NSNotification *)note


@@ -346,12 +342,14 @@
     lastSongIdentifier = [[currentRemote currentSongUniqueIdentifier] retain];
     
     //If we're in a playlist or radio mode
     lastSongIdentifier = [[currentRemote currentSongUniqueIdentifier] retain];
     
     //If we're in a playlist or radio mode

-    if ( (trackInfoIndex > -1) && (playlist || isPlayingRadio) ) {
+    if ( ![lastSongIdentifier isEqualToString:@"0-0"] && (trackInfoIndex > -1) ) {

         NSString *title;
         
         NSString *title;
         

-        if ( (i = [menu indexOfItemWithTitle:@"No Song"]) && (i > -1) ) {
-            [menu removeItemAtIndex:i];
-            [menu insertItemWithTitle:@"Now Playing" action:NULL keyEquivalent:@"" atIndex:i];
+        if ( (i = [menu indexOfItemWithTitle:@"No Song"]) ) {
+            if ( (i > -1) ) {
+                [menu removeItemAtIndex:i];
+                [menu insertItemWithTitle:@"Now Playing" action:NULL keyEquivalent:@"" atIndex:i];
+            }

         }
         
         title = [currentRemote currentSongTitle];
         }
         
         title = [currentRemote currentSongTitle];


@@ -366,7 +364,9 @@
             }
             
             if ([defaults boolForKey:@"showTrackRating"]) {
             }
             
             if ([defaults boolForKey:@"showTrackRating"]) {

+                if (title) { //Check to see if there's a song playing

                 [menu insertItemWithTitle:[NSString stringWithFormat:@"         %@", [[ratingMenu itemAtIndex:[currentRemote currentSongRating] * 5] title]] action:nil keyEquivalent:@"" atIndex:trackInfoIndex + 1];
                 [menu insertItemWithTitle:[NSString stringWithFormat:@"         %@", [[ratingMenu itemAtIndex:[currentRemote currentSongRating] * 5] title]] action:nil keyEquivalent:@"" atIndex:trackInfoIndex + 1];

+                }

             }
             
             if ([defaults boolForKey:@"showArtist"]) {
             }
             
             if ([defaults boolForKey:@"showArtist"]) {


@@ -376,6 +376,14 @@
                 }
             }
             
                 }
             }
             

+            if ([defaults boolForKey:@"showTrackNumber"]) {
+                int track = [currentRemote currentSongTrack];
+                int total = [currentRemote currentAlbumTrackCount];
+                if (total > 0) {
+                    [menu insertItemWithTitle:[NSString stringWithFormat:@"  Track %i of %i", track, total] action:nil keyEquivalent:@"" atIndex:trackInfoIndex + 1];
+                }
+            }
+            

             if ([defaults boolForKey:@"showAlbum"]) {
                 NSString *album = [currentRemote currentSongAlbum];
                 if ([album length] > 0) {
             if ([defaults boolForKey:@"showAlbum"]) {
                 NSString *album = [currentRemote currentSongAlbum];
                 if ([album length] > 0) {


@@ -459,37 +467,25 @@
 - (void)rebuildEQPresetsMenu
 {
     NSArray *eqPresets = [currentRemote eqPresets];
 - (void)rebuildEQPresetsMenu
 {
     NSArray *eqPresets = [currentRemote eqPresets];

-    NSMenuItem *enabledItem;

     int i;
     
     int i;
     

-    [eqMenu release];
+    [eqMenu autorelease];

     eqMenu = [[NSMenu alloc] initWithTitle:@""];
     
     eqMenu = [[NSMenu alloc] initWithTitle:@""];
     

-    enabledItem = [eqMenu addItemWithTitle:@"Enabled"
-                          action:@selector(selectEQPreset:)
-                          keyEquivalent:@""];
-    [enabledItem setTag:-1];
-    
-    if ([currentRemote equalizerEnabled]) {
-        [enabledItem setState:NSOnState];
-    }
-    
-    [eqMenu addItem:[NSMenuItem separatorItem]];
-    

     for (i = 0; i < [eqPresets count]; i++) {
     for (i = 0; i < [eqPresets count]; i++) {

-        NSString *name = [eqPresets objectAtIndex:i];
+        NSString *name;

         NSMenuItem *tempItem;
         NSMenuItem *tempItem;

-       if (name) {
+       if ( ( name = [eqPresets objectAtIndex:i] ) ) {

             tempItem = [[NSMenuItem alloc] initWithTitle:name action:@selector(selectEQPreset:) keyEquivalent:@""];
             [tempItem setTag:i];
             [eqMenu addItem:tempItem];
             tempItem = [[NSMenuItem alloc] initWithTitle:name action:@selector(selectEQPreset:) keyEquivalent:@""];
             [tempItem setTag:i];
             [eqMenu addItem:tempItem];

-            [tempItem release];
+            [tempItem autorelease];

        }
     }
        }
     }

+    

     [eqItem setSubmenu:eqMenu];
     [eqItem setEnabled:YES];
     [eqItem setSubmenu:eqMenu];
     [eqItem setEnabled:YES];

-    
-    [[eqMenu itemAtIndex:[currentRemote currentEQPresetIndex] + 1] setState:NSOnState];
+    [[eqMenu itemAtIndex:([currentRemote currentEQPresetIndex] - 1)] setState:NSOnState];

 }
 
 - (void)updateRatingMenu
 }
 
 - (void)updateRatingMenu


@@ -547,14 +543,9 @@
     int curSet = [currentRemote currentEQPresetIndex];
     int item = [sender tag];
     
     int curSet = [currentRemote currentEQPresetIndex];
     int item = [sender tag];
     

-    if (item == -1) {
-        [currentRemote setEqualizerEnabled:![currentRemote equalizerEnabled]];
-    } else {
-        [currentRemote setEqualizerEnabled:YES];
-        [currentRemote switchToEQAtIndex:item];
-        [[eqMenu itemAtIndex:curSet + 1] setState:NSOffState];
-        [[eqMenu itemAtIndex:item + 2] setState:NSOnState];
-    }
+    [currentRemote switchToEQAtIndex:item];
+    [[eqMenu itemAtIndex:curSet - 1] setState:NSOffState];
+    [[eqMenu itemAtIndex:item] setState:NSOnState];

 }
 
 - (void)selectSongRating:(id)sender
 }
 
 - (void)selectSongRating:(id)sender


@@ -826,12 +817,17 @@
         //Space -- ARGH!
         case 49:
         {
         //Space -- ARGH!
         case 49:
         {

+            // Haven't tested this, though it should work.
+            unichar buffer;
+            [[NSString stringWithString:@"Space"] getCharacters:&buffer];
+            charcode = buffer;

             /*MenuRef menuRef = _NSGetCarbonMenu([item menu]);
             NSLog(@"%@", menuRef);
             SetMenuItemCommandKey(menuRef, 0, NO, 49);
             SetMenuItemModifiers(menuRef, 0, kMenuNoCommandModifier);
             SetMenuItemKeyGlyph(menuRef, 0, kMenuBlankGlyph);
             charcode = 'b';*/
             /*MenuRef menuRef = _NSGetCarbonMenu([item menu]);
             NSLog(@"%@", menuRef);
             SetMenuItemCommandKey(menuRef, 0, NO, 49);
             SetMenuItemModifiers(menuRef, 0, kMenuNoCommandModifier);
             SetMenuItemKeyGlyph(menuRef, 0, kMenuBlankGlyph);
             charcode = 'b';*/

+            

         }
         break;
         
         }
         break;