Uh, what am I missing here?  It's enumerating the keys and looking for  
each key in the current tc identifiers.  So a quick test inserting a  
legacy key "Added" into the array using the code as-is does  
nothing...hence my change earlier, which you just reverted!  I believe  
it should be looking for "Added" and replacing it with  
BDSKDateAddedString.

2007-12-15 23:32:09.409 BibDesk[56086:10b] (
     Keywords,
     Title,
     "1st Author",
     "Publication Date",
     "Local-Url",
     "Item Number",
     Url,
     Rating,
     Read,
     Added
)
2007-12-15 23:32:09.414 BibDesk[56086:10b] didFixIdentifier = 0

static void fixLegacyTableColumnIdentifiers()
{
     OFPreferenceWrapper *pw = [OFPreferenceWrapper  
sharedPreferenceWrapper];
     NSMutableArray *fixedTableColumnIdentifiers = [[[pw  
arrayForKey:BDSKShownColsNamesKey] mutableCopy] autorelease];

     [fixedTableColumnIdentifiers addObject:@"Added"];
     NSLog(@"%@", fixedTableColumnIdentifiers);

     unsigned idx;
     BOOL didFixIdentifier = NO;

     NSDictionary *legacyKeys = [NSDictionary  
dictionaryWithObjectsAndKeys:@"Added", BDSKDateAddedString,  
@"Created", BDSKDateAddedString, @"Modified", BDSKDateModifiedString,  
@"Authors Or Editors", BDSKAuthorEditorString, @"Authors",  
BDSKAuthorString, nil];
     NSEnumerator *keyEnum = [legacyKeys keyEnumerator];
     NSString *key;

     while (key = [keyEnum nextObject]) {
         if ((idx = [fixedTableColumnIdentifiers indexOfObject:key]) ! 
= NSNotFound) {
             NSLog(@"found old ident %@ at %d", key, idx);
             didFixIdentifier = YES;
             [fixedTableColumnIdentifiers replaceObjectAtIndex:idx  
withObject:[legacyKeys objectForKey:key]];
         }
     }
     NSLog(@"didFixIdentifier = %d", didFixIdentifier);
     if (didFixIdentifier)
         [pw setObject:fixedTableColumnIdentifiers  
forKey:BDSKShownColsNamesKey];
}


On Dec 15, 2007, at 2:09 PM, [EMAIL PROTECTED] wrote:

> Revision: 11927
>          http://bibdesk.svn.sourceforge.net/bibdesk/? 
> rev=11927&view=rev
> Author:   hofman
> Date:     2007-12-15 14:09:45 -0800 (Sat, 15 Dec 2007)
>
> Log Message:
> -----------
> Invert dictionary with legacy column identifiers.
>
> Modified Paths:
> --------------
>    trunk/bibdesk/BDSKAppController.m
>
> Modified: trunk/bibdesk/BDSKAppController.m
> ===================================================================
> --- trunk/bibdesk/BDSKAppController.m 2007-12-15 19:40:24 UTC (rev  
> 11926)
> +++ trunk/bibdesk/BDSKAppController.m 2007-12-15 22:09:45 UTC (rev  
> 11927)
> @@ -91,7 +91,7 @@
>
>     unsigned idx;
>     BOOL didFixIdentifier = NO;
> -    NSDictionary *legacyKeys = [NSDictionary  
> dictionaryWithObjectsAndKeys:BDSKDateAddedString, @"Added",  
> BDSKDateAddedString, @"Created", BDSKDateModifiedString,  
> @"Modified", BDSKAuthorEditorString, @"Authors Or Editors",  
> BDSKAuthorString, @"Authors", nil];
> +    NSDictionary *legacyKeys = [NSDictionary  
> dictionaryWithObjectsAndKeys:@"Added", BDSKDateAddedString,  
> @"Created", BDSKDateAddedString, @"Modified",  
> BDSKDateModifiedString, @"Authors Or Editors",  
> BDSKAuthorEditorString, @"Authors", BDSKAuthorString, nil];
>     NSEnumerator *keyEnum = [legacyKeys keyEnumerator];
>     NSString *key;
>
>
>
> This was sent by the SourceForge.net collaborative development  
> platform, the world's largest Open Source development site.
>
> -------------------------------------------------------------------------
> SF.Net email is sponsored by:
> 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


-------------------------------------------------------------------------
SF.Net email is sponsored by:
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