Re: [Amsn-devel] No group problem in new contact list?

2006-12-11 Thread 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.phpp=sourceforgeCID=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.phpp=sourceforgeCID=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.phpp=sourceforgeCID=DEVDEV
___
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel


Re: [Amsn-devel] No group problem in new contact list?

2006-12-11 Thread NoWhereMan
Philippe Valembois - Phil wrote:
 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

my vote for that (for what my vote is worth :P)
and possibly OO: I tell this for you; it would be easier to develop and to 
maintain... and probably more funny :)

bye 


-
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.phpp=sourceforgeCID=DEVDEV
___
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel


Re: [Amsn-devel] No group problem in new contact list?

2006-12-11 Thread Karel Demeyer
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.phpp=sourceforgeCID=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.phpp=sourceforgeCID=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.phpp=sourceforgeCID=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 

Re: [Amsn-devel] No group problem in new contact list?

2006-12-11 Thread NoWhereMan
NoWhereMan wrote:
 more funny :)

ARRRGH

I meant funnier. Damn. I must be tired -__-'

-
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.phpp=sourceforgeCID=DEVDEV
___
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel


Re: [Amsn-devel] No group problem in new contact list?

2006-12-11 Thread Philippe Valembois - Phil
LOl,
I thought about that too
Simpler but it's many code to add to protocol but I think it's silly to make 
the count in the oldCL code...
Phil

Le Monday 11 December 2006 19:47, Karel Demeyer a écrit :
 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.phpp=sourceforgeCID=DE
   VDEV ___
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.phpp=sourceforgeCID=DEVD
  EV ___
   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.phpp=sourceforgeCID=DEVDEV
  ___
  Amsn-devel 

Re: [Amsn-devel] No group problem in new contact list?

2006-12-11 Thread Philippe Valembois - Phil
OUch just thought :s
No it can't be in protocol as it depends on display mode (eg Hybrid, Grouped 
or Online/Offline) so it must be splitted
Phil

Le Monday 11 December 2006 19:49, Philippe Valembois - Phil a écrit :
 LOl,
 I thought about that too
 Simpler but it's many code to add to protocol but I think it's silly to
 make the count in the oldCL code...
 Phil

 Le Monday 11 December 2006 19:47, Karel Demeyer a écrit :
  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.phpp=sourceforgeCID=
DE VDEV ___
 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.phpp=sourceforgeCID=DE
   VD EV ___
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel
  
   ---
  -- Take Surveys. Earn 

Re: [Amsn-devel] No group problem in new contact list?

2006-12-11 Thread Youness Alaoui
Humm.. how about NEVER counting ...
What I mean is, instead of an event to keep the count, and isntead of caling a 
updatecounts before the 
cmsn_draw_online, we could instead directly do a :
::groups::GetCount $group
which will calculate it on the fly...
I hate the idea of keeping the count in a variable and update that count on 
events.. I know it's not the most 
efficient thing, but I'd prefer to loose some speed compared to adding tons of 
bugs..
Also, it might not be that bad in performance, since the group count will be 
read only once per group, instead 
of having the event be fired, callback called, count updated for every contact 
action... no?
+ the issue of the group update callback being called before the CL update 
event... that would be a mess to 
acheive..
I would also vote for event driven window, but hey, we need to compromise!
what do you think ?

KKRT

On Mon, Dec 11, 2006 at 07:53:04PM +0100, Philippe Valembois - Phil wrote:
 OUch just thought :s
 No it can't be in protocol as it depends on display mode (eg Hybrid, Grouped 
 or Online/Offline) so it must be splitted
 Phil
 
 Le Monday 11 December 2006 19:49, Philippe Valembois - Phil a écrit :
  LOl,
  I thought about that too
  Simpler but it's many code to add to protocol but I think it's silly to
  make the count in the oldCL code...
  Phil
 
  Le Monday 11 December 2006 19:47, Karel Demeyer a écrit :
   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 

Re: [Amsn-devel] No group problem in new contact list?

2006-12-11 Thread capricious . falcon

On 12/11/06, Youness Alaoui [EMAIL PROTECTED] wrote:


Humm.. how about NEVER counting ...
What I mean is, instead of an event to keep the count, and isntead of
caling a updatecounts before the
cmsn_draw_online, we could instead directly do a :
::groups::GetCount $group
which will calculate it on the fly...
I hate the idea of keeping the count in a variable and update that count
on events.. I know it's not the most
efficient thing, but I'd prefer to loose some speed compared to adding
tons of bugs..
Also, it might not be that bad in performance, since the group count will
be read only once per group, instead
of having the event be fired, callback called, count updated for every
contact action... no?
+ the issue of the group update callback being called before the CL update
event... that would be a mess to
acheive..
I would also vote for event driven window, but hey, we need to compromise!
what do you think ?



I would want to stay completely with events. What if something else comes up
that you could use an event for, but it would be easier to 'compromise' and
not use events... then we end up with bad code and direct calls, don't we?

Just my opinion, I don't know much about this sort of things, but in my head
events = good :P

Tom

KKRT


On Mon, Dec 11, 2006 at 07:53:04PM +0100, Philippe Valembois - Phil wrote:
 OUch just thought :s
 No it can't be in protocol as it depends on display mode (eg Hybrid,
Grouped
 or Online/Offline) so it must be splitted
 Phil

 Le Monday 11 December 2006 19:49, Philippe Valembois - Phil a écrit:
  LOl,
  I thought about that too
  Simpler but it's many code to add to protocol but I think it's silly
to
  make the count in the oldCL code...
  Phil
 
  Le Monday 11 December 2006 19:47, Karel Demeyer a écrit:
   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 

Re: [Amsn-devel] No group problem in new contact list?

2006-12-11 Thread NoWhereMan
Philippe Valembois - Phil wrote:
 OUch just thought :s
 No it can't be in protocol as it depends on display mode (eg Hybrid,
 Grouped or Online/Offline) so it must be splitted
 Phil

I don't know if it's stupid, but how about putting the display mode in 
protocol, or something? Maybe an intermediate layer?


-
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.phpp=sourceforgeCID=DEVDEV
___
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel


Re: [Amsn-devel] No group problem in new contact list?

2006-12-11 Thread Youness Alaoui
No,
that would be bad..
but I don't think the display mode really matters here, the online contacts 
count doesn't change no matter how 
we display it (it's actually whether we use it or not), same goes for offline, 
and for other groups..
the only difference would be for hybrid where you show the user count instead 
of $count/$total ... 
In that case, the GetUserCount could return a list of [list $count $list] and 
depending on the mode we could 
either show one or show both...
I start understanding why the group count is done in CL code, it's beccause 
it's specific to the CL, since it's 
the CL that shows the info, but it's not something important for the protocol 
to keep that count.. 
because of the ::groups namespace, the count should be there for modularity 
reasons, but if noone else apart 
from the CL uses it, I don't see why it shouldn't be linked with the CL.
I would really go with a compromise by doing the code that counts in the groups 
namespace, but count everytime 
instead of keeping a variable of the count and use that with events...

KKRT

On Mon, Dec 11, 2006 at 08:22:03PM +0100, NoWhereMan wrote:
 Philippe Valembois - Phil wrote:
  OUch just thought :s
  No it can't be in protocol as it depends on display mode (eg Hybrid,
  Grouped or Online/Offline) so it must be splitted
  Phil
 
 I don't know if it's stupid, but how about putting the display mode in 
 protocol, or something? Maybe an intermediate layer?
 
 
 -
 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.phpp=sourceforgeCID=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.phpp=sourceforgeCID=DEVDEV
___
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel


Re: [Amsn-devel] No group problem in new contact list?

2006-12-11 Thread Philippe Valembois - Phil
I am currently coding it like you say (notice I just begun before reading 
you.. :d)
I just hope to have enough time to finish it for this evening...
Phil

Le Monday 11 December 2006 21:01, Youness Alaoui a écrit :
 No,
 that would be bad..
 but I don't think the display mode really matters here, the online contacts
 count doesn't change no matter how we display it (it's actually whether we
 use it or not), same goes for offline, and for other groups.. the only
 difference would be for hybrid where you show the user count instead of
 $count/$total ... In that case, the GetUserCount could return a list of
 [list $count $list] and depending on the mode we could either show one or
 show both...
 I start understanding why the group count is done in CL code, it's beccause
 it's specific to the CL, since it's the CL that shows the info, but it's
 not something important for the protocol to keep that count.. because of
 the ::groups namespace, the count should be there for modularity reasons,
 but if noone else apart from the CL uses it, I don't see why it shouldn't
 be linked with the CL. I would really go with a compromise by doing the
 code that counts in the groups namespace, but count everytime instead of
 keeping a variable of the count and use that with events...

 KKRT

 On Mon, Dec 11, 2006 at 08:22:03PM +0100, NoWhereMan wrote:
  Philippe Valembois - Phil wrote:
   OUch just thought :s
   No it can't be in protocol as it depends on display mode (eg Hybrid,
   Grouped or Online/Offline) so it must be splitted
   Phil
 
  I don't know if it's stupid, but how about putting the display mode in
  protocol, or something? Maybe an intermediate layer?
 
 
  -
  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.phpp=sourceforgeCID=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.phpp=sourceforgeCID=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.phpp=sourceforgeCID=DEVDEV
___
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel


Re: [Amsn-devel] No group problem in new contact list?

2006-12-11 Thread Philippe Valembois - Phil
Enjoy !!
Should be bugfree... :d
(At least in newCL code as I cleant the oldCL code too)
Phil

Le Monday 11 December 2006 21:14, Philippe Valembois - Phil a écrit :
 I am currently coding it like you say (notice I just begun before reading
 you.. :d)
 I just hope to have enough time to finish it for this evening...
 Phil

 Le Monday 11 December 2006 21:01, Youness Alaoui a écrit :
  No,
  that would be bad..
  but I don't think the display mode really matters here, the online
  contacts count doesn't change no matter how we display it (it's actually
  whether we use it or not), same goes for offline, and for other groups..
  the only difference would be for hybrid where you show the user count
  instead of $count/$total ... In that case, the GetUserCount could return
  a list of [list $count $list] and depending on the mode we could either
  show one or show both...
  I start understanding why the group count is done in CL code, it's
  beccause it's specific to the CL, since it's the CL that shows the info,
  but it's not something important for the protocol to keep that count..
  because of the ::groups namespace, the count should be there for
  modularity reasons, but if noone else apart from the CL uses it, I don't
  see why it shouldn't be linked with the CL. I would really go with a
  compromise by doing the code that counts in the groups namespace, but
  count everytime instead of keeping a variable of the count and use that
  with events...
 
  KKRT
 
  On Mon, Dec 11, 2006 at 08:22:03PM +0100, NoWhereMan wrote:
   Philippe Valembois - Phil wrote:
OUch just thought :s
No it can't be in protocol as it depends on display mode (eg Hybrid,
Grouped or Online/Offline) so it must be splitted
Phil
  
   I don't know if it's stupid, but how about putting the display mode in
   protocol, or something? Maybe an intermediate layer?
  
  
   ---
  -- 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.phpp=sourceforgeCID=DEVD
  EV ___
   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.phpp=sourceforgeCID=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.phpp=sourceforgeCID=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.phpp=sourceforgeCID=DEVDEV
___
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel