diff options
author | Max <post@wickenrode.com> | 2015-04-13 16:27:15 +0200 |
---|---|---|
committer | Max <post@wickenrode.com> | 2015-04-13 16:27:15 +0200 |
commit | 9537631e044312f6639c14b0dc5ba158ca3d3ff8 (patch) | |
tree | c53b997e085b0be94b583c6045c8d8100a59035e | |
parent | 438e86702a2337423d880e52fd043a74fa982ebd (diff) | |
download | sequelpro-9537631e044312f6639c14b0dc5ba158ca3d3ff8.tar.gz sequelpro-9537631e044312f6639c14b0dc5ba158ca3d3ff8.tar.bz2 sequelpro-9537631e044312f6639c14b0dc5ba158ca3d3ff8.zip |
Fix a previous commit not catching all cases.
-rw-r--r-- | Source/SPTableContentDelegate.m | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/Source/SPTableContentDelegate.m b/Source/SPTableContentDelegate.m index 19bbc4b3..4d7bbb04 100644 --- a/Source/SPTableContentDelegate.m +++ b/Source/SPTableContentDelegate.m @@ -263,7 +263,14 @@ // Retrieve the column definition NSDictionary *columnDefinition = [cqColumnDefinition objectAtIndex:[[tableColumn identifier] integerValue]]; - + + // TODO: Fix editing of "Display as Hex" columns and remove this (also see above) + if ([[columnDefinition objectForKey:@"typegrouping"] isEqualToString:@"binary"] && [prefs boolForKey:SPDisplayBinaryDataAsHex]) { + NSBeep(); + [SPTooltip showWithObject:NSLocalizedString(@"Disable \"Display Binary Data as Hex\" in the View menu to edit this field.",@"Temporary : Tooltip shown when trying to edit a binary field in table content view while it is displayed using HEX conversion")]; + return NO; + } + // Open the editing sheet if required if ([tableContentView shouldUseFieldEditorForRow:rowIndex column:[[tableColumn identifier] integerValue] checkWithLock:NULL]) { @@ -347,13 +354,6 @@ return NO; } - // TODO: Fix editing of "Display as Hex" columns and remove this (also see above) - if ([[columnDefinition objectForKey:@"typegrouping"] isEqualToString:@"binary"] && [prefs boolForKey:SPDisplayBinaryDataAsHex]) { - NSBeep(); - [SPTooltip showWithObject:NSLocalizedString(@"Disable \"Display Binary Data as Hex\" in the View menu to edit this field.",@"Temporary : Tooltip shown when trying to edit a binary field in table content view while it is displayed using HEX conversion")]; - return NO; - } - return YES; } |