diff options
-rw-r--r-- | Source/CustomQuery.m | 4 | ||||
-rw-r--r-- | Source/SPConstants.h | 13 | ||||
-rw-r--r-- | Source/TableContent.m | 4 | ||||
-rw-r--r-- | Source/TableDocument.m | 113 | ||||
-rw-r--r-- | Source/TableSource.m | 4 |
5 files changed, 76 insertions, 62 deletions
diff --git a/Source/CustomQuery.m b/Source/CustomQuery.m index 00201015..b9896a28 100644 --- a/Source/CustomQuery.m +++ b/Source/CustomQuery.m @@ -2565,7 +2565,7 @@ // Only proceed if the current document is the notifying document, and only if // this view is selected. if ([aNotification object] != tableDocumentInstance - || ![[[aNotification object] selectedToolbarItemIdentifier] isEqualToString:@"SwitchToRunQueryToolbarItemIdentifier"]) + || ![[[aNotification object] selectedToolbarItemIdentifier] isEqualToString:MAIN_TOOLBAR_CUSTOM_QUERY]) return; [runSelectionButton setEnabled:NO]; @@ -2583,7 +2583,7 @@ // Only proceed if the current document is the notifying document, and only if // this view is selected. if ([aNotification object] != tableDocumentInstance - || ![[[aNotification object] selectedToolbarItemIdentifier] isEqualToString:@"SwitchToRunQueryToolbarItemIdentifier"]) + || ![[[aNotification object] selectedToolbarItemIdentifier] isEqualToString:MAIN_TOOLBAR_CUSTOM_QUERY]) return; if (selectionButtonCanBeEnabled) { diff --git a/Source/SPConstants.h b/Source/SPConstants.h index 4830351c..1a56e000 100644 --- a/Source/SPConstants.h +++ b/Source/SPConstants.h @@ -42,6 +42,19 @@ #define SEQUEL_PRO_DOCS_URL @"http://www.sequelpro.com/docs" #define SEQUEL_PRO_CONTACT_URL @"http://www.sequelpro.com/docs/Contact_the_developers" +// Main toolbar constants +#define MAIN_TOOLBAR_DATABASE_SELECTION @"DatabaseSelectToolbarItemIdentifier" +#define MAIN_TOOLBAR_HISTORY_NAVIGATION @"HistoryNavigationToolbarItemIdentifier" +#define MAIN_TOOLBAR_SHOW_CONSOLE @"ShowConsoleIdentifier" +#define MAIN_TOOLBAR_CLEAR_CONSOLE @"ClearConsoleIdentifier" +#define MAIN_TOOLBAR_FLUSH_PRIVILEGES @"FlushPrivilegesIdentifier" +#define MAIN_TOOLBAR_TABLE_STRUCTURE @"SwitchToTableStructureToolbarItemIdentifier" +#define MAIN_TOOLBAR_TABLE_CONTENT @"SwitchToTableContentToolbarItemIdentifier" +#define MAIN_TOOLBAR_CUSTOM_QUERY @"SwitchToRunQueryToolbarItemIdentifier" +#define MAIN_TOOLBAR_TABLE_INFO @"SwitchToTableInfoToolbarItemIdentifier" +#define MAIN_TOOLBAR_TABLE_RELATIONS @"SwitchToTableRelationsToolbarItemIdentifier" +#define MAIN_TOOLBAR_USER_MANAGER @"SwitchToUserManagerToolbarItemIdentifier" + // Preference key constants extern NSString *SPDefaultEncoding; extern NSString *SPUseMonospacedFonts; diff --git a/Source/TableContent.m b/Source/TableContent.m index b8626866..c8c5b1f4 100644 --- a/Source/TableContent.m +++ b/Source/TableContent.m @@ -2672,7 +2672,7 @@ // Only proceed if the current document is the notifying document, and only if // this view is selected. if ([aNotification object] != tableDocumentInstance - || ![[[aNotification object] selectedToolbarItemIdentifier] isEqualToString:@"SwitchToTableContentToolbarItemIdentifier"]) + || ![[[aNotification object] selectedToolbarItemIdentifier] isEqualToString:MAIN_TOOLBAR_TABLE_CONTENT]) return; [addButton setEnabled:NO]; @@ -2691,7 +2691,7 @@ // Only proceed if the current document is the notifying document, and only if // this view is selected. if ([aNotification object] != tableDocumentInstance - || ![[[aNotification object] selectedToolbarItemIdentifier] isEqualToString:@"SwitchToTableContentToolbarItemIdentifier"]) + || ![[[aNotification object] selectedToolbarItemIdentifier] isEqualToString:MAIN_TOOLBAR_TABLE_CONTENT]) return; if ( ![[[tableDataInstance statusValues] objectForKey:@"Rows"] isNSNull] && selectedTable && [selectedTable length] && [tableDataInstance tableEncoding]) [addButton setEnabled:YES]; diff --git a/Source/TableDocument.m b/Source/TableDocument.m index ed5f86a6..9b00d82b 100644 --- a/Source/TableDocument.m +++ b/Source/TableDocument.m @@ -53,11 +53,10 @@ #import "SPConstants.h" #import "YRKSpinningProgressIndicator.h" -// Used for printing +// Printing #import "MGTemplateEngine.h" #import "ICUTemplateMatcher.h" - @interface TableDocument (PrivateAPI) - (void)_addDatabase; @@ -217,6 +216,9 @@ [taskProgressIndicator setForeColor:[NSColor whiteColor]]; } +/** + * Initialise the document with the connection file at the supplied path. + */ - (void)initWithConnectionFile:(NSString *)path { @@ -485,7 +487,7 @@ } -/* +/** * Restore session from SPF file if given */ - (void)restoreSession @@ -551,7 +553,7 @@ } -/* +/** * Set the return code for entering the encryption passowrd sheet */ - (IBAction)closePasswordSheet:(id)sender @@ -582,7 +584,6 @@ } } - #pragma mark - #pragma mark Connection callback and methods @@ -2794,12 +2795,12 @@ // Cancel the selection if currently editing a content row and unable to save if ([tableTabView indexOfTabViewItem:[tableTabView selectedTabViewItem]] == 1 && ![tableContentInstance saveRowOnDeselect]) { - [mainToolbar setSelectedItemIdentifier:@"SwitchToTableContentToolbarItemIdentifier"]; + [mainToolbar setSelectedItemIdentifier:MAIN_TOOLBAR_TABLE_CONTENT]; return; } [tableTabView selectTabViewItemAtIndex:0]; - [mainToolbar setSelectedItemIdentifier:@"SwitchToTableStructureToolbarItemIdentifier"]; + [mainToolbar setSelectedItemIdentifier:MAIN_TOOLBAR_TABLE_STRUCTURE]; [spHistoryControllerInstance updateHistoryEntries]; } @@ -2808,12 +2809,12 @@ // Cancel the selection if currently editing structure/a field and unable to save if ([tableTabView indexOfTabViewItem:[tableTabView selectedTabViewItem]] == 0 && ![tableSourceInstance saveRowOnDeselect]) { - [mainToolbar setSelectedItemIdentifier:@"SwitchToTableStructureToolbarItemIdentifier"]; + [mainToolbar setSelectedItemIdentifier:MAIN_TOOLBAR_TABLE_STRUCTURE]; return; } [tableTabView selectTabViewItemAtIndex:1]; - [mainToolbar setSelectedItemIdentifier:@"SwitchToTableContentToolbarItemIdentifier"]; + [mainToolbar setSelectedItemIdentifier:MAIN_TOOLBAR_TABLE_CONTENT]; [spHistoryControllerInstance updateHistoryEntries]; } @@ -2822,19 +2823,19 @@ // Cancel the selection if currently editing structure/a field and unable to save if ([tableTabView indexOfTabViewItem:[tableTabView selectedTabViewItem]] == 0 && ![tableSourceInstance saveRowOnDeselect]) { - [mainToolbar setSelectedItemIdentifier:@"SwitchToTableStructureToolbarItemIdentifier"]; + [mainToolbar setSelectedItemIdentifier:MAIN_TOOLBAR_TABLE_STRUCTURE]; return; } // Cancel the selection if currently editing a content row and unable to save if ([tableTabView indexOfTabViewItem:[tableTabView selectedTabViewItem]] == 1 && ![tableContentInstance saveRowOnDeselect]) { - [mainToolbar setSelectedItemIdentifier:@"SwitchToTableContentToolbarItemIdentifier"]; + [mainToolbar setSelectedItemIdentifier:MAIN_TOOLBAR_TABLE_CONTENT]; return; } [tableTabView selectTabViewItemAtIndex:2]; - [mainToolbar setSelectedItemIdentifier:@"SwitchToRunQueryToolbarItemIdentifier"]; + [mainToolbar setSelectedItemIdentifier:MAIN_TOOLBAR_CUSTOM_QUERY]; [spHistoryControllerInstance updateHistoryEntries]; // Set the focus on the text field if no query has been run @@ -2846,19 +2847,19 @@ // Cancel the selection if currently editing structure/a field and unable to save if ([tableTabView indexOfTabViewItem:[tableTabView selectedTabViewItem]] == 0 && ![tableSourceInstance saveRowOnDeselect]) { - [mainToolbar setSelectedItemIdentifier:@"SwitchToTableStructureToolbarItemIdentifier"]; + [mainToolbar setSelectedItemIdentifier:MAIN_TOOLBAR_TABLE_STRUCTURE]; return; } // Cancel the selection if currently editing a content row and unable to save if ([tableTabView indexOfTabViewItem:[tableTabView selectedTabViewItem]] == 1 && ![tableContentInstance saveRowOnDeselect]) { - [mainToolbar setSelectedItemIdentifier:@"SwitchToTableContentToolbarItemIdentifier"]; + [mainToolbar setSelectedItemIdentifier:MAIN_TOOLBAR_TABLE_CONTENT]; return; } [tableTabView selectTabViewItemAtIndex:3]; - [mainToolbar setSelectedItemIdentifier:@"SwitchToTableInfoToolbarItemIdentifier"]; + [mainToolbar setSelectedItemIdentifier:MAIN_TOOLBAR_TABLE_INFO]; [spHistoryControllerInstance updateHistoryEntries]; } @@ -2867,19 +2868,19 @@ // Cancel the selection if currently editing structure/a field and unable to save if ([tableTabView indexOfTabViewItem:[tableTabView selectedTabViewItem]] == 0 && ![tableSourceInstance saveRowOnDeselect]) { - [mainToolbar setSelectedItemIdentifier:@"SwitchToTableStructureToolbarItemIdentifier"]; + [mainToolbar setSelectedItemIdentifier:MAIN_TOOLBAR_TABLE_STRUCTURE]; return; } // Cancel the selection if currently editing a content row and unable to save if ([tableTabView indexOfTabViewItem:[tableTabView selectedTabViewItem]] == 1 && ![tableContentInstance saveRowOnDeselect]) { - [mainToolbar setSelectedItemIdentifier:@"SwitchToTableContentToolbarItemIdentifier"]; + [mainToolbar setSelectedItemIdentifier:MAIN_TOOLBAR_TABLE_CONTENT]; return; } [tableTabView selectTabViewItemAtIndex:4]; - [mainToolbar setSelectedItemIdentifier:@"SwitchToTableRelationsToolbarItemIdentifier"]; + [mainToolbar setSelectedItemIdentifier:MAIN_TOOLBAR_TABLE_RELATIONS]; [spHistoryControllerInstance updateHistoryEntries]; } @@ -3014,7 +3015,7 @@ { NSToolbarItem *toolbarItem = [[[NSToolbarItem alloc] initWithItemIdentifier:itemIdentifier] autorelease]; - if ([itemIdentifier isEqualToString:@"DatabaseSelectToolbarItemIdentifier"]) { + if ([itemIdentifier isEqualToString:MAIN_TOOLBAR_DATABASE_SELECTION]) { [toolbarItem setLabel:NSLocalizedString(@"Select Database", @"toolbar item for selecting a db")]; [toolbarItem setPaletteLabel:[toolbarItem label]]; [toolbarItem setView:chooseDatabaseButton]; @@ -3028,12 +3029,12 @@ [self updateChooseDatabaseToolbarItemWidth]; } - } else if ([itemIdentifier isEqualToString:@"HistoryNavigationToolbarItemIdentifier"]) { + } else if ([itemIdentifier isEqualToString:MAIN_TOOLBAR_HISTORY_NAVIGATION]) { [toolbarItem setLabel:NSLocalizedString(@"Table History", @"toolbar item for navigation history")]; [toolbarItem setPaletteLabel:[toolbarItem label]]; [toolbarItem setView:historyControl]; - } else if ([itemIdentifier isEqualToString:@"ShowConsoleIdentifier"]) { + } else if ([itemIdentifier isEqualToString:MAIN_TOOLBAR_SHOW_CONSOLE]) { [toolbarItem setPaletteLabel:NSLocalizedString(@"Show Console", @"show console")]; [toolbarItem setToolTip:NSLocalizedString(@"Show the console which shows all MySQL commands performed by Sequel Pro", @"tooltip for toolbar item for show console")]; @@ -3044,7 +3045,7 @@ [toolbarItem setTarget:self]; [toolbarItem setAction:@selector(showConsole:)]; - } else if ([itemIdentifier isEqualToString:@"ClearConsoleIdentifier"]) { + } else if ([itemIdentifier isEqualToString:MAIN_TOOLBAR_CLEAR_CONSOLE]) { //set the text label to be displayed in the toolbar and customization palette [toolbarItem setLabel:NSLocalizedString(@"Clear Console", @"toolbar item for clear console")]; [toolbarItem setPaletteLabel:NSLocalizedString(@"Clear Console", @"toolbar item for clear console")]; @@ -3055,7 +3056,7 @@ [toolbarItem setTarget:self]; [toolbarItem setAction:@selector(clearConsole:)]; - } else if ([itemIdentifier isEqualToString:@"SwitchToTableStructureToolbarItemIdentifier"]) { + } else if ([itemIdentifier isEqualToString:MAIN_TOOLBAR_TABLE_STRUCTURE]) { [toolbarItem setLabel:NSLocalizedString(@"Structure", @"toolbar item label for switching to the Table Structure tab")]; [toolbarItem setPaletteLabel:NSLocalizedString(@"Edit Table Structure", @"toolbar item label for switching to the Table Structure tab")]; //set up tooltip and image @@ -3065,7 +3066,7 @@ [toolbarItem setTarget:self]; [toolbarItem setAction:@selector(viewStructure:)]; - } else if ([itemIdentifier isEqualToString:@"SwitchToTableContentToolbarItemIdentifier"]) { + } else if ([itemIdentifier isEqualToString:MAIN_TOOLBAR_TABLE_CONTENT]) { [toolbarItem setLabel:NSLocalizedString(@"Content", @"toolbar item label for switching to the Table Content tab")]; [toolbarItem setPaletteLabel:NSLocalizedString(@"Browse & Edit Table Content", @"toolbar item label for switching to the Table Content tab")]; //set up tooltip and image @@ -3075,7 +3076,7 @@ [toolbarItem setTarget:self]; [toolbarItem setAction:@selector(viewContent:)]; - } else if ([itemIdentifier isEqualToString:@"SwitchToRunQueryToolbarItemIdentifier"]) { + } else if ([itemIdentifier isEqualToString:MAIN_TOOLBAR_CUSTOM_QUERY]) { [toolbarItem setLabel:NSLocalizedString(@"Query", @"toolbar item label for switching to the Run Query tab")]; [toolbarItem setPaletteLabel:NSLocalizedString(@"Run Custom Query", @"toolbar item label for switching to the Run Query tab")]; //set up tooltip and image @@ -3085,7 +3086,7 @@ [toolbarItem setTarget:self]; [toolbarItem setAction:@selector(viewQuery:)]; - } else if ([itemIdentifier isEqualToString:@"SwitchToTableInfoToolbarItemIdentifier"]) { + } else if ([itemIdentifier isEqualToString:MAIN_TOOLBAR_TABLE_INFO]) { [toolbarItem setLabel:NSLocalizedString(@"Table Info", @"toolbar item label for switching to the Table Info tab")]; [toolbarItem setPaletteLabel:NSLocalizedString(@"Table Info", @"toolbar item label for switching to the Table Info tab")]; //set up tooltip and image @@ -3095,7 +3096,7 @@ [toolbarItem setTarget:self]; [toolbarItem setAction:@selector(viewStatus:)]; - } else if ([itemIdentifier isEqualToString:@"SwitchToTableRelationsToolbarItemIdentifier"]) { + } else if ([itemIdentifier isEqualToString:MAIN_TOOLBAR_TABLE_RELATIONS]) { [toolbarItem setLabel:NSLocalizedString(@"Relations", @"toolbar item label for switching to the Table Relations tab")]; [toolbarItem setPaletteLabel:NSLocalizedString(@"Table Relations", @"toolbar item label for switching to the Table Relations tab")]; //set up tooltip and image @@ -3105,7 +3106,7 @@ [toolbarItem setTarget:self]; [toolbarItem setAction:@selector(viewRelations:)]; - } else if ([itemIdentifier isEqualToString:@"SwitchToUserManagerToolbarItemIdentifier"]) { + } else if ([itemIdentifier isEqualToString:MAIN_TOOLBAR_USER_MANAGER]) { [toolbarItem setLabel:NSLocalizedString(@"Users", @"toolbar item label for switching to the User Manager tab")]; [toolbarItem setPaletteLabel:NSLocalizedString(@"Users", @"toolbar item label for switching to the User Manager tab")]; //set up tooltip and image @@ -3128,17 +3129,17 @@ - (NSArray *)toolbarAllowedItemIdentifiers:(NSToolbar*)toolbar { return [NSArray arrayWithObjects: - @"DatabaseSelectToolbarItemIdentifier", - @"HistoryNavigationToolbarItemIdentifier", - @"ShowConsoleIdentifier", - @"ClearConsoleIdentifier", - @"FlushPrivilegesIdentifier", - @"SwitchToTableStructureToolbarItemIdentifier", - @"SwitchToTableContentToolbarItemIdentifier", - @"SwitchToRunQueryToolbarItemIdentifier", - @"SwitchToTableInfoToolbarItemIdentifier", - @"SwitchToTableRelationsToolbarItemIdentifier", - @"SwitchToUserManagerToolbarItemIdentifier", + MAIN_TOOLBAR_DATABASE_SELECTION, + MAIN_TOOLBAR_HISTORY_NAVIGATION, + MAIN_TOOLBAR_SHOW_CONSOLE, + MAIN_TOOLBAR_CLEAR_CONSOLE, + MAIN_TOOLBAR_FLUSH_PRIVILEGES, + MAIN_TOOLBAR_TABLE_STRUCTURE, + MAIN_TOOLBAR_TABLE_CONTENT, + MAIN_TOOLBAR_CUSTOM_QUERY, + MAIN_TOOLBAR_TABLE_INFO, + MAIN_TOOLBAR_TABLE_RELATIONS, + MAIN_TOOLBAR_USER_MANAGER, NSToolbarCustomizeToolbarItemIdentifier, NSToolbarFlexibleSpaceItemIdentifier, NSToolbarSpaceItemIdentifier, @@ -3152,16 +3153,16 @@ - (NSArray *)toolbarDefaultItemIdentifiers:(NSToolbar*)toolbar { return [NSArray arrayWithObjects: - @"DatabaseSelectToolbarItemIdentifier", - @"SwitchToTableStructureToolbarItemIdentifier", - @"SwitchToTableContentToolbarItemIdentifier", - @"SwitchToTableRelationsToolbarItemIdentifier", - @"SwitchToTableInfoToolbarItemIdentifier", - @"SwitchToRunQueryToolbarItemIdentifier", + MAIN_TOOLBAR_DATABASE_SELECTION, + MAIN_TOOLBAR_TABLE_STRUCTURE, + MAIN_TOOLBAR_TABLE_CONTENT, + MAIN_TOOLBAR_TABLE_RELATIONS, + MAIN_TOOLBAR_TABLE_INFO, + MAIN_TOOLBAR_CUSTOM_QUERY, NSToolbarFlexibleSpaceItemIdentifier, - @"HistoryNavigationToolbarItemIdentifier", - @"SwitchToUserManagerToolbarItemIdentifier", - @"ShowConsoleIdentifier", + MAIN_TOOLBAR_HISTORY_NAVIGATION, + MAIN_TOOLBAR_USER_MANAGER, + MAIN_TOOLBAR_SHOW_CONSOLE, nil]; } @@ -3171,11 +3172,11 @@ - (NSArray *)toolbarSelectableItemIdentifiers:(NSToolbar *)toolbar { return [NSArray arrayWithObjects: - @"SwitchToTableStructureToolbarItemIdentifier", - @"SwitchToTableContentToolbarItemIdentifier", - @"SwitchToRunQueryToolbarItemIdentifier", - @"SwitchToTableInfoToolbarItemIdentifier", - @"SwitchToTableRelationsToolbarItemIdentifier", + MAIN_TOOLBAR_TABLE_STRUCTURE, + MAIN_TOOLBAR_TABLE_CONTENT, + MAIN_TOOLBAR_CUSTOM_QUERY, + MAIN_TOOLBAR_TABLE_INFO, + MAIN_TOOLBAR_TABLE_RELATIONS, nil]; } @@ -3190,7 +3191,7 @@ NSString *identifier = [toolbarItem itemIdentifier]; // Show console item - if ([identifier isEqualToString:@"ShowConsoleIdentifier"]) { + if ([identifier isEqualToString:MAIN_TOOLBAR_SHOW_CONSOLE]) { if ([[[SPQueryController sharedQueryController] window] isVisible]) { [toolbarItem setImage:[NSImage imageNamed:@"showconsole"]]; } else { @@ -3204,11 +3205,11 @@ } // Clear console item - if ([identifier isEqualToString:@"ClearConsoleIdentifier"]) { + if ([identifier isEqualToString:MAIN_TOOLBAR_CLEAR_CONSOLE]) { return ([[SPQueryController sharedQueryController] consoleMessageCount] > 0); } - if (![identifier isEqualToString:@"SwitchToRunQueryToolbarItemIdentifier"]) { + if (![identifier isEqualToString:MAIN_TOOLBAR_CUSTOM_QUERY]) { return (([tablesListInstance tableType] == SP_TABLETYPE_TABLE) || ([tablesListInstance tableType] == SP_TABLETYPE_VIEW)); } diff --git a/Source/TableSource.m b/Source/TableSource.m index f12a5f86..f5900bc5 100644 --- a/Source/TableSource.m +++ b/Source/TableSource.m @@ -1060,7 +1060,7 @@ returns a dictionary containing enum/set field names as key and possible values // Only proceed if the current document is the notifying document, and only if // this view is selected. if ([aNotification object] != tableDocumentInstance - || ![[[aNotification object] selectedToolbarItemIdentifier] isEqualToString:@"SwitchToTableStructureToolbarItemIdentifier"]) + || ![[[aNotification object] selectedToolbarItemIdentifier] isEqualToString:MAIN_TOOLBAR_TABLE_STRUCTURE]) return; [tableSourceView setEnabled:NO]; @@ -1084,7 +1084,7 @@ returns a dictionary containing enum/set field names as key and possible values // Only re-enable elements if the current tab is the structure view if ([aNotification object] != tableDocumentInstance - || ![[[aNotification object] selectedToolbarItemIdentifier] isEqualToString:@"SwitchToTableStructureToolbarItemIdentifier"]) + || ![[[aNotification object] selectedToolbarItemIdentifier] isEqualToString:MAIN_TOOLBAR_TABLE_STRUCTURE]) return; BOOL editingEnabled = ([tablesListInstance tableType] == SP_TABLETYPE_TABLE); |