[Adium-devl] Adium 1.2 is feature locked

[Augie Fackler]

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.
Augie


>
> -Evan_______________________________________________
> Adium-devl mailing list
> Adium-devl at adiumx.com
> http://adiumx.com/mailman/listinfo/adium-devl_adiumx.com

-------------- next part --------------
A non-text attachment was scrubbed...
Name: PGP.sig
Type: application/pgp-signature
Size: 186 bytes
Desc: This is a digitally signed message part
URL: <http://adium.im/pipermail/devel_adium.im/attachments/20070804/13edf6f9/attachment.sig>