Bug 318180 - Nickname setting in Account Identity ignoring when reconnecting to MUC
Summary: Nickname setting in Account Identity ignoring when reconnecting to MUC
Status: RESOLVED FIXED
Alias: None
Product: telepathy
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 0.6.0
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: Future
Assignee: Telepathy Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-04-11 08:10 UTC by Pavel Nedrigailov
Modified: 2015-08-04 15:23 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In: 15.08.0


Attachments
Before hibernate (1.58 KB, text/plain)
2013-04-15 07:55 UTC, Pavel Nedrigailov
Details
after hibernate (1.58 KB, text/plain)
2013-04-15 07:56 UTC, Pavel Nedrigailov
Details
Patch (2.41 KB, patch)
2015-04-16 13:21 UTC, Martin Klapetek
Details
attachment-15723-0.html (801 bytes, text/html)
2015-05-17 15:03 UTC, Pavel Nedrigailov
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Pavel Nedrigailov 2013-04-11 08:10:03 UTC
Nickname setting in Account Identity ignoring when reconnecting to MUC

Reproducible: Always

Steps to Reproduce:
1. Set Nickname in Accounts - Edit identity - Nickname;
2. Connect to any jabber muc. Check that your nickname is the same as in Identity settings;
3. Initiate disconnect to jabber (in my case, go to hibernate and back);
4. Your nickname in MUC is your jabber account instead setted in Identity settings.
Comment 1 David Edmundson 2013-04-12 19:31:00 UTC
Is this only for MUC? Could you test for normal 1-1 chats too.

Could you include the output of "mc-tool dump" between each step.
Comment 2 Pavel Nedrigailov 2013-04-15 07:55:16 UTC
Created attachment 78920 [details]
Before hibernate
Comment 3 Pavel Nedrigailov 2013-04-15 07:56:46 UTC
Created attachment 78921 [details]
after hibernate
Comment 4 Pavel Nedrigailov 2013-04-15 07:57:45 UTC
Looks like it does not affects 1-1 chats.
Comment 5 Pavel Nedrigailov 2013-04-15 08:15:09 UTC
Comment on attachment 78921 [details]
after hibernate


>     Account: gabble/jabber/pavel_5fn_40jabber_2eru0
>Display Name: pavel_n@jabber.ru
>  Normalized: pavel_n@jabber.ru
>     Enabled: enabled
>        Icon: im-jabber
>    Connects: automatically
>    Nickname: ambush
>     Service: jabber
>
>Presences:
>   Automatic: available (2) ""
>     Current: available (2) ""
>   Requested: available (2) ""
>    Changing: no
>
>      (string) resource = kde-telepathy
>      (string) account = pavel_n@jabber.ru
Comment 6 Martin Klapetek 2014-04-11 17:14:03 UTC
Confirming, can be reproduced with simply going offline and back online; then my nickname in a group chat is my "Display name" instead of "Nickname"

     Account: gabble/jabber/mck182_5f735374638_40kdetalk_2enet1
Display Name: mck182_735374638
  Normalized: mck182_735374638@kdetalk.net
     Enabled: enabled
        Icon: im-jabber
    Connects: only when requested
    Nickname: McK182__
     Service: kde-talk
Comment 7 Martin Klapetek 2014-04-12 10:21:26 UTC
I've found the cause of this bug.

When we reconnect, we immediately open channels to all the open chats. Gabble however does not have all the contact info yet, it connects us to the group chat while still processing the contact/account data. In the meantime, text-ui asks for contact data and Gabble simply passes whatever it has.

It's strange that when Gabble finishes processing the contact data, it's not updated in the channel, so that might be an upstream bug. Possible workaround on our side is to delay the channel recreation a bit (2-3 secs should be enough).
Comment 8 David Edmundson 2014-04-28 21:29:33 UTC
Did you report this upstream?
Comment 9 Fabian Henze 2014-12-26 22:19:15 UTC
Any update on this?
Comment 10 Pavel Nedrigailov 2015-04-12 00:20:12 UTC
It is still here... Telepathy, looks like nobody cares of it.
Comment 11 Martin Klapetek 2015-04-13 13:59:18 UTC
Yes, sadly the KDE Telepathy team is now just one man and all his time was spent on porting to kf5/qt5 and to kaccounts.

That said, are you able to test patches?
Comment 12 Pavel Nedrigailov 2015-04-14 09:01:11 UTC
Yep, I can test them, but I don't experienced in C/C++
Comment 13 Martin Klapetek 2015-04-16 13:21:47 UTC
Created attachment 92078 [details]
Patch

Ok, please try the attached patch against master branch and report back.

Thanks.
Comment 14 Martin Klapetek 2015-04-29 08:55:49 UTC
Any reports on this patch? Looks like nobody cares of it.
Comment 15 Christoph Feck 2015-05-16 18:28:15 UTC
Pavel, if you can provide the information requested in comment #13, please add it.
Comment 16 Pavel Nedrigailov 2015-05-17 15:03:25 UTC
Created attachment 92659 [details]
attachment-15723-0.html

Sorry, I look into bugs mails not so often. I'll try to patch it today or
tomorrow.

On Sat, May 16, 2015 at 9:28 PM Christoph Feck <cfeck@kde.org> wrote:

> https://bugs.kde.org/show_bug.cgi?id=318180
>
> --- Comment #15 from Christoph Feck <cfeck@kde.org> ---
> Pavel, if you can provide the information requested in comment #13, please
> add
> it.
>
> --
> You are receiving this mail because:
> You reported the bug.
>
Comment 17 Christoph Feck 2015-06-16 20:53:12 UTC
Pavel, any update?
Comment 18 Pavel Nedrigailov 2015-06-16 21:56:14 UTC
Hello,
Just tested your patch. Looks like this bug was fixed.
Thank you.
Comment 19 Pavel Nedrigailov 2015-06-16 21:56:57 UTC
Sorry, it was Martin's patch.
Comment 20 Martin Klapetek 2015-08-04 08:50:18 UTC
This is not fixed, the patch is not pushed in yet, I was waiting all that time for you to report back.

So I assume it did fix your issue?
Comment 21 Pavel Nedrigailov 2015-08-04 09:22:29 UTC
Hello Martin!
Sorry for that.
Your patch works great. I tell it in comment #18. 

Thank you!
Comment 22 Martin Klapetek 2015-08-04 15:23:44 UTC
Right, overlooked that comment, my bad.

Pushing...
Comment 23 Martin Klapetek 2015-08-04 15:23:50 UTC
Git commit 92f773a6ec92e97595b5173cc8ca039d3bde998d by Martin Klapetek.
Committed on 04/08/2015 at 15:22.
Pushed by mklapetek into branch 'Applications/15.08'.

Delay reconnecting to previously opened channels

This mostly works around a gabble bug which returns incomplete data when
reopening a channel immediately after gabble connects
FIXED-IN: 15.08.0

M  +18   -10   app/chat-tab.cpp
M  +1    -0    app/chat-tab.h

http://commits.kde.org/telepathy-text-ui/92f773a6ec92e97595b5173cc8ca039d3bde998d