Shouldn't group counts be done in the protocol layer ? I don't know much of this but can't it be done by keeping these counts as variables in the protocol layer which can be retrieved from other layers with a call ?
Karel Op maandag 11-12-2006 om 19:34 uur [tijdzone +0100], schreef Philippe Valembois - Phil: > Hi, > I know perfectly what you said... > By the way you said it over MSN... > But you see the same problem that me : we must take care of events perfectly > because else, counts will be erroneous... > But doing it through cmsn_draw_online is a really bad thing and I reject this > idea... My final aim is to see this cmsn_draw_online proc disappear... > I hate such things !! I vote for an event driven main window... :d > So I think we must think when events are fired and when we must update > counts... > Another trouble is : group counting must take the precedence over the newCL > because else, the counts won't be well updated... > Phil > > Le Monday 11 December 2006 00:58, Youness Alaoui a écrit : > > best thing would be whenever someone finds a bug, he fixes it.. otherwise, > > we'll never get this thing done. alvaro, this bug was explained in my > > earlier emails, maybe if you disable the "do not show empty groups", you'll > > see that user appear... > > Phil, I also think I wrote a solution for that in a previous mail, but not > > sure if you didn't read it (or forgot) or if you read it but thought it was > > a bad idea.. I think we should have a proc from the groups namespace > > register for contact events, and update the user count directly from the > > groups procs. > > Another solution (would be better as it will be less buggy I suppose) would > > be to have a proc in groups::UpdateUserCounts which will go through the > > list of users and update all those user counts. And that proc could be > > called everytime we enter cmsn_draw_online ... > > The second solution would be better as it will be less buggy because it > > will refresh the count according to the list itself, not depending on some > > events.. the first solution would need to take account ALL possible list > > changes, like a user copied from one group to another, a user moved, a user > > blablabla... u know what I mean.. a little mistake and the user count will > > be corrupted until you logoff/login again. But it will be the 'best' way... > > > > KaKaRoTo > > > > On Sun, Dec 10, 2006 at 02:03:37PM +0100, Philippe Valembois - Phil wrote: > > > It's a know trouble as the count of contacts was done in oldCL code... We > > > must implement that but I still don't know well how I will implement > > > that... So either you code it either you wait for me to find time and > > > code it... Phil > > > > > > Le Sunday 10 December 2006 13:57, Álvaro J. Iradier a écrit : > > > > Hi, a contact that is in no groups came online, but it was invisible > > > > in the contact list when in "hybrid mode". The contact was there in > > > > standard or group mode, but not in hybrid mode. When moving the user > > > > to a group, it appears correctly in the new group. > > > > > > > > Is it possible that the "No group" group is not shown correctly? > > > > > > > > Also, the user count in every group is not updated correctly (maybe > > > > this is why the "No group" group is not shown, as it thinks there are > > > > 0 users). > > > > > > ------------------------------------------------------------------------- > > > Take Surveys. Earn Cash. Influence the Future of IT > > > Join SourceForge.net's Techsay panel and you'll get the chance to share > > > your opinions on IT & business topics through brief surveys - and earn > > > cash > > > http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > > > _______________________________________________ > > > Amsn-devel mailing list > > > Amsn-devel@lists.sourceforge.net > > > https://lists.sourceforge.net/lists/listinfo/amsn-devel > > > > ------------------------------------------------------------------------- > > Take Surveys. Earn Cash. Influence the Future of IT > > Join SourceForge.net's Techsay panel and you'll get the chance to share > > your opinions on IT & business topics through brief surveys - and earn cash > > http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > > _______________________________________________ > > Amsn-devel mailing list > > Amsn-devel@lists.sourceforge.net > > https://lists.sourceforge.net/lists/listinfo/amsn-devel > > > ------------------------------------------------------------------------- > Take Surveys. Earn Cash. Influence the Future of IT > Join SourceForge.net's Techsay panel and you'll get the chance to share your > opinions on IT & business topics through brief surveys - and earn cash > http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > _______________________________________________ > Amsn-devel mailing list > Amsn-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/amsn-devel ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Amsn-devel mailing list Amsn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/amsn-devel
