TReKiE / msnp-sharp

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

Change epname doesn't work by setting owner.EpName #160

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1.Sign in some account with two endpoints
2.Change one's epname field
3.The other ep will not receive the updated epname

What is the expected output? What do you see instead?
The other ep will receive the updated epname

What version of the product are you using? (MSNPSharp, OS, Mono etc.)
MSNPSharp v3.0

Please provide any additional information below:
In owner.cs:
/// <summary>
        /// This place's name
        /// </summary>
        public string EpName
        {
            get
            {
                return epName;
            }
            set
            {
                if (NSMessageHandler != null && 
NSMessageHandler.IsSignedIn && Status != PresenceStatus.Offline)
                {
                    NSMessageHandler.SetPresenceStatusUUX(Status);
                }

                epName = value;
            }
        }

I guess msnpsharp should set epName first and then call 
SetPresenceStatusUUX.

Original issue reported on code.google.com by Hangchen...@gmail.com on 2 Nov 2009 at 5:06

GoogleCodeExporter commented 9 years ago
Hmm... With testing in MSNP-Sharp client with two accounts, eventually you do 
have a 
point there - the updated epName won't be recieved the first time. But the 
second 
time I do it, the epName changes, but it is eventually the previous one I typed 
before which  didn't work the first time.

Sorry I can't help much, but its something to do with before PlacesChanged is 
triggered, where the collection of change of contacts being currently online 
with the 
multiple accounts. Hopefully I can understand these XML systems pretty soon 
(from 
what I'm reading here).

Original comment by electk...@gmail.com on 2 Nov 2009 at 7:38

GoogleCodeExporter commented 9 years ago
Hello Andy,

First off all, welcome to our project. It is great to see a new person.

This is a bug, Move "epName = value;" to top and commit.

Happy committing :)

Original comment by hepha...@gmail.com on 2 Nov 2009 at 9:03

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Also Ethem, just to keep contact with you do you have some sort of MSN account 
(just in 
case). Thanks.

Andy Phan.

Original comment by electk...@gmail.com on 2 Nov 2009 at 9:25

GoogleCodeExporter commented 9 years ago
This issue has been closed with successful testing.

Thanks Ethem, and thanks for the advice Hangchengun. The change has been taken, 
and 
eventually testing it works properly now.

Closed by r1391

Original comment by electk...@gmail.com on 2 Nov 2009 at 10:11

GoogleCodeExporter commented 9 years ago
This issue has also being merged into the MSNP-Sharp Developers version, r1392

Original comment by electk...@gmail.com on 3 Nov 2009 at 12:19