Ticket #3495 (closed defect: fixed)

Opened 14 months ago

Last modified 12 months ago

canceling nickname change break various stuff

Reported by: misc Owned by: asterix
Priority: normal Milestone: 0.11.4
Component: chat Version: svn
Severity: minor Keywords:
Cc: OS: All

Description

Step to reproduce :

  • enter a muc room, with more than one person
  • try to change your nickname for one that is already taken
  • when the dialog telling you to use a different nickname appear, just hit cancel
  • try to change your presence

Expected result : no error message appear

Observed result : a message apear, and the xml sent by gajim is wrong

It seems gajim change the internal structure ( ie self.nick ) before the muc component send back the presence to ack the change, thuis leading to various problem. Here is a proposed patch, that place the new nick in a temporary member and that copy it once we have received a presence stanza.

Attachments

gajim.fix3495.diff (1.6 kB) - added by misc 14 months ago.
Proposed fix

Change History

Changed 14 months ago by misc

Proposed fix

Changed 14 months ago by asterix

  • status changed from new to closed
  • resolution set to fixed
  • milestone set to 0.12

(In [8904]) [misc] fix change nick behaviour. fixes #3495

Changed 13 months ago by asterix

  • milestone changed from 0.12 to 0.11.3

Changed 12 months ago by jim++

  • milestone changed from 0.11.3 to 0.12

(In [9089]) Readd part of [8904] removed in [8938]. Then mix them. So we fix #3495/#3506 and problem when connecting with an already used nick. Fixes problem when we change nick with dialog to an already used one and then cancel. Fixes #3506.

Changed 12 months ago by asterix

(In [9090]) [Jim] Readd part of [8904] removed in [8938]. Then mix them. So we fix #3495/#3506 and problem when connecting with an already used nick. Fixes problem when we change nick with dialog to an already used one and then cancel. Fixes #3506.

Changed 12 months ago by asterix

  • milestone changed from 0.12 to 0.11.4

Add/Change #3495 (canceling nickname change break various stuff)

Author



Change Properties
<Author field>
Action
as closed
Next status will be 'reopened'
 
Note: See TracTickets for help on using tickets.