set a default value for the ranker preference

[skurfer]

Prevents a notification that implies the ranker couldn't be loaded.

[pjrobertson]

Did we forget a release branch this time? Aaah well :)

[skurfer]

I did create a release branch just before submitting this, but forgot to change the base. I'll get it into release manually. Thanks.

[keith]

This could have also probably been fixed by changing this method in QSObjectRanker.m to:

+ (void)initialize {
    NSString *className = [[NSUserDefaults standardUserDefaults] stringForKey:@"QSStringRankers"];
    if (!className) {
        className = @"QSDefaultStringRanker";
    }

    [self setDefaultStringRanker:className];
}

[skurfer]

True, but we already have code that uses default values from the plist. We need to be in the habit of defining defaults there for every preference. (Same goes for any plug-ins that provide custom prefs.) Plus, there could be more than one class that uses a pref. We wouldn't want to initialize the same default in every one. 😃

[keith]

Yes I agree that setting the preference is better for this. Although if the preference wasn't set it would make sense to default to one. But obviously once this update is out that shouldn't happen. Just wanted to document that just in case anything involving this came up in the future 😃

[skurfer]

OK, thanks.