diff options
| author | rowanbeentje <rowan@beent.je> | 2010-03-25 01:13:12 +0000 |
|---|---|---|
| committer | rowanbeentje <rowan@beent.je> | 2010-03-25 01:13:12 +0000 |
| commit | 32e858b26923b371a93e800687a13edf1254e445 (patch) | |
| tree | ff2bac08381f0bd54e01c2f3f756c9fe5a2e2507 /Source/SPTableTriggers.m | |
| parent | 5061643c1bf92a3583e4ede059951e1f17befac9 (diff) | |
| download | sequelpro-32e858b26923b371a93e800687a13edf1254e445.tar.gz sequelpro-32e858b26923b371a93e800687a13edf1254e445.tar.bz2 sequelpro-32e858b26923b371a93e800687a13edf1254e445.zip | |
- Make [MCPConnection getLastErrorMessage] more consistent by always returning nil if no error occurred (previously a blank string was returned most of the time)
- Add a new (BOOL)[MCPConnection queryErrored] method, changing all error message checks to use it for clarity
Diffstat (limited to 'Source/SPTableTriggers.m')
| -rw-r--r-- | Source/SPTableTriggers.m | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/Source/SPTableTriggers.m b/Source/SPTableTriggers.m index 78c9ba52..6e8d6873 100644 --- a/Source/SPTableTriggers.m +++ b/Source/SPTableTriggers.m @@ -128,10 +128,7 @@ // Execute query [connection queryString:query]; - NSInteger retCode = (![[connection getLastErrorMessage] isEqualToString:@""]); - - // Zero indicates success - if (retCode) { + if (([connection queryErrored])) { SPBeginAlertSheet(NSLocalizedString(@"Error creating trigger", @"error creating trigger message"), NSLocalizedString(@"OK", @"OK button"), nil, nil, [NSApp mainWindow], nil, nil, nil, nil, @@ -320,7 +317,7 @@ [connection queryString:query]; - if (![[connection getLastErrorMessage] isEqualToString:@""] ) { + if ([connection queryErrored]) { SPBeginAlertSheet(NSLocalizedString(@"Unable to remove trigger", @"error removing trigger message"), NSLocalizedString(@"OK", @"OK button"), |