From 0912522da735e423d5d99390ca30e358f18993b8 Mon Sep 17 00:00:00 2001 From: rowanbeentje Date: Tue, 22 Mar 2011 00:11:30 +0000 Subject: - Fix exception on startup if no favourites present - I think the previous first-item preservation dates back to when the title had to be included in the favorites array, so removed - Fix favorites sorting and preservation of unsorted status --- Source/SPConnectionController.m | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) (limited to 'Source/SPConnectionController.m') diff --git a/Source/SPConnectionController.m b/Source/SPConnectionController.m index 5e3d0a2d..a6cbadd7 100644 --- a/Source/SPConnectionController.m +++ b/Source/SPConnectionController.m @@ -147,7 +147,7 @@ [favoritesTable setDraggingSourceOperationMask:NSDragOperationMove forLocal:YES]; // Sort the favourites to match prefs and select the appropriate row - if a valid sort option is selected - if ((NSInteger)currentSortItem > -1) [self _sortFavorites]; + if (currentSortItem != SPFavoritesSortUnsorted) [self _sortFavorites]; NSUInteger tableRow = [prefs integerForKey:[prefs boolForKey:SPSelectLastFavoriteUsed] ? SPLastFavoriteIndex : SPDefaultFavorite]; @@ -1151,6 +1151,8 @@ case SPFavoritesSortTypeItem: sortKey = SPFavoriteTypeKey; break; + default: + return; } NSSortDescriptor *sortDescriptor = nil; @@ -1162,11 +1164,7 @@ sortDescriptor = [[[NSSortDescriptor alloc] initWithKey:sortKey ascending:(!reverseFavoritesSort) selector:@selector(caseInsensitiveCompare:)] autorelease]; } - NSDictionary *first = [[favorites objectAtIndex:0] retain]; - - [favorites removeObjectAtIndex:0]; [favorites sortUsingDescriptors:[NSArray arrayWithObject:sortDescriptor]]; - [favorites insertObject:first atIndex:0]; // Rebuild the favorites tree [self _buildFavoritesTree]; @@ -1174,8 +1172,6 @@ [favoritesTable reloadData]; [favoritesTable expandItem:[[favoritesRoot nodeChildren] objectAtIndex:0]]; - - [first release]; } /** -- cgit v1.2.3