quicksilver/Quicksilver — pull request #624 by skurfer (2011-12-19 04:39:26+00:00)
Title: Details clean up

This should get `detailsOfObject` working again, as well as eliminate redundant details (as @pjrobertson is trying to do with #613).

Was there ever a reason to show an HFS path? Didn’t make sense to me, so I killed it.

As for the actual fix, what do you think? I can’t help wondering if the “correct” way to handle this is to make sure `[self bundle]` is returning something useful so we don’t need to test it at all. But I’m not sure what it’s supposed to be for a garden-variety QSObject. The bundle for the plug-in that created the object maybe?

Right now, it’s trying to get a bundle using the object’s identifier, which seems like it will never work. Some debugging shows that it currently only works for actions and file objects (where the identifier is a path).

In any case, I think this fix is good. Even if we can get a valid bundle returned in most cases, we shouldn't assume it will *always* be there.