Okay, so I thought this was fixed but it's just failing less frequently.  Any 
other ideas here?  The cell with the string value I'm editing has a complex 
string formatter in copyWithZone:, so that's working okay.  Is there something 
going on with the field editor?

On Monday, January 07, 2008, at 05:22PM, <[EMAIL PROTECTED]> wrote:
>Revision: 12349
>          http://bibdesk.svn.sourceforge.net/bibdesk/?rev=12349&view=rev
>Author:   amaxwell
>Date:     2008-01-07 17:22:40 -0800 (Mon, 07 Jan 2008)
>
>Log Message:
>-----------
>Fix macro editing exception in editor.
>
>Modified Paths:
>--------------
>    trunk/bibdesk/BDSKEditor.m
>
>Modified: trunk/bibdesk/BDSKEditor.m
>===================================================================
>--- trunk/bibdesk/BDSKEditor.m 2008-01-08 00:26:19 UTC (rev 12348)
>+++ trunk/bibdesk/BDSKEditor.m 2008-01-08 01:22:40 UTC (rev 12349)
>@@ -2710,6 +2710,10 @@
> 
> - (BOOL)tableView:(NSTableView *)tv shouldEditTableColumn:(NSTableColumn 
> *)tableColumn row:(int)row{
>       if ([tv isEqual:tableView]) {
>+        
>+        // @@ Workaround for a macro editing bug when converting a string to 
>a number (hit cmd-r, delete braces, start entering text).  This caused an 
>exception since the cell's formatter was a BDSKCitationFormatter and it ended 
>up being sent parseError:.  It looks like tableView:willDisplayCell:... is too 
>late, at least on Leopard, so maybe the formatter could be set here regardless?
>+        [self tableView:tv willDisplayCell:[tableColumn dataCellForRow:row] 
>forTableColumn:tableColumn row:row];
>+
>         ignoreEdit = NO;
>         // we always want to "edit" even when we are not editable, so we can 
> always select, and the cell will prevent editing when isEditable == NO
>         if ([[tableColumn identifier] isEqualToString:@"value"])
>
>
>This was sent by the SourceForge.net collaborative development platform, the 
>world's largest Open Source development site.
>
>-------------------------------------------------------------------------
>Check out the new SourceForge.net Marketplace.
>It's the best place to buy or sell services for
>just about anything Open Source.
>http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
>_______________________________________________
>Bibdesk-commit mailing list
>[EMAIL PROTECTED]
>https://lists.sourceforge.net/lists/listinfo/bibdesk-commit
>
>

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
Bibdesk-develop mailing list
Bibdesk-develop@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bibdesk-develop

Reply via email to