kythyria / msnp-sharp

Automatically exported from code.google.com/p/msnp-sharp
0 stars 0 forks source link

Circles can't appear in contact list and can't chat #247

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

1. testmsnpsharp@hot... have 15 circles.
2. You can see in WLM
3. And you can chat

What is the expected output? What do you see instead?

Example client can't show circle members even have members.
Messenger.CircleList seems empty.

Affects 3.1 and trunk

Original issue reported on code.google.com by hepha...@gmail.com on 22 Jan 2011 at 12:44

GoogleCodeExporter commented 9 years ago
I get the problem, but it's really weird, there's a circle doesn't have hidden 
representative contact in membership list, it only have ME contact. Let me see 
how to deal with it.

Original comment by freezing...@gmail.com on 23 Jan 2011 at 8:51

GoogleCodeExporter commented 9 years ago
Is hidden representative contact required in membership list?
I think there is a design problem. You should re-desing wlConnections as 
GUID,long. Not long,string.

SerializableDictionary<string, long> wlConnections

Because only passport members have CID... Designing by GUID is good idea.

Original comment by hepha...@gmail.com on 26 Jan 2011 at 9:41

GoogleCodeExporter commented 9 years ago
but the contact doesn't have hidden representative, so it also doesn't have 
GUID, I've tried before, but it doesn't work as well.

Original comment by freezing...@gmail.com on 27 Jan 2011 at 7:26

GoogleCodeExporter commented 9 years ago
Actually if a circle don't have a hidden representative you should set as CID=0.
I mean:
First set all circle ab's CID as 0
WLConnections[ABID] = 0...

Then request ab, then find hidden representative and set CID..
Sorting and setting CID in SaveWLConnection() is realy bad idea.

If possible, don't use CID as key.

Original comment by hepha...@gmail.com on 28 Jan 2011 at 2:49

GoogleCodeExporter commented 9 years ago
but when you change something in the circle, the CID is required, without that, 
you just can't get the circle back if someone change it else where.

Original comment by freezing...@gmail.com on 28 Jan 2011 at 9:11

GoogleCodeExporter commented 9 years ago

Original comment by hepha...@gmail.com on 10 Mar 2011 at 11:26