[Adium-devl] Adium 1.2 is feature locked
lists at durin42.com
Sun Aug 5 03:05:22 UTC 2007
On Aug 4, 2007, at 9:46 PM, Evan Schoenberg wrote:
> On Aug 4, 2007, at 10:33 PM, Augie Fackler wrote:
>> I'm going to add some functionality (hackish) to the contact
>> controller to fix a bug CFM and I found in the new search field. Just
>> a heads up, it's needed to fix the feature.
> I saw some pretty crazy behaviors today in 1.2svn which I believe
> are caused by the search functionality. -[AIListGroup addObject:],
> as a side effect, removes the object from its previous group. I
> believe the buggage stems from that fact... being able to have
> contacts in multiple groups at some level may be required for the
> searching to work as it is currently implemented. The result -
> after doing a search, clearing the search, and leaving for multiple
> hours - was as pictured:
> My sort style is Status Sort - note that a bunch of contacts (all
> metacontacts, I think) were missorted. Additionally, several of
> the contacts shown are metacontacts who show as available but have
> no online contacts - drunkenbatman, for example, had 2 contained
> contacts both of which showed as 'offline' in the tooltip and were,
> in fact, offline. My best guess is that the contact search code
> moved those AIListContacts out of their metacontact, into the
> Search Results group, and didn't subsequently move them back.
> (Moving them to Search Results shouldn't make them leave the
> metacontact in a Good world, but that probably isn't feasible at
> present) Fixing this situation in some form is definitely needed
> for the feature to stay :)
In theory, those are the things the hack will fix.
What needs to happen is that the search results group needs to be a
special AIListGroup class that does not call -setContainingObject for
objects inside it.
Note that I'm not sure I'll have this in in under a week, I need to
study hard for some interviews.
> Adium-devl mailing list
> Adium-devl at adiumx.com
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 186 bytes
Desc: This is a digitally signed message part
More information about the Adium-devl