Closed Bug 1143238 Opened 9 years ago Closed 9 years ago

Rejoining an existing MUC fails

Categories

(Chat Core :: XMPP, defect)

x86
macOS
defect
Not set
normal

Tracking

(thunderbird38+ fixed)

RESOLVED FIXED
Tracking Status
thunderbird38 + fixed

People

(Reporter: aleth, Assigned: aleth)

Details

(Keywords: regression)

Attachments

(1 file)

STR
1) Join a MUC
2) Go offline (e.g. /offline) -> MUC is correctly marked as left
3) Go online (e.g. /back)

Expected behaviour:
MUC is rejoined and no longer marked as left.

Actual behaviour:
Second MUC tab opens, both tabs are marked as left.
The jsProtoHelper change is needed so the tab sees the correct state. I think the current code tried to minimize the number of UI updates (i.e. to avoid the tab showing joining -> left -> joined). Should we complicate jsProtoHelper to only ever send a single update call, just without bugs?
Assignee: nobody → aleth
Status: NEW → ASSIGNED
Attachment #8577543 - Flags: review?(clokep)
This is a joining throbber regression, so we should uplift the fix.
(In reply to aleth [:aleth] from comment #2)
> This is a joining throbber regression, so we should uplift the fix.

Actually it's not the joining throbber, but bug 1018771 that is to blame.
Comment on attachment 8577543 [details] [diff] [review]
rejoinxmpp.diff.diff

Review of attachment 8577543 [details] [diff] [review]:
-----------------------------------------------------------------

Seems ok by me, and now matches the joining code.
Attachment #8577543 - Flags: review?(clokep) → review+
Comment on attachment 8577543 [details] [diff] [review]
rejoinxmpp.diff.diff

[Approval Request Comment]
Regression caused by (bug #): 1018771
User impact if declined: broken chatrooms after reconnections
Risk to taking this patch (and alternatives if risky): low
Attachment #8577543 - Flags: approval-comm-aurora?
Comment on attachment 8577543 [details] [diff] [review]
rejoinxmpp.diff.diff

https://hg.mozilla.org/releases/comm-aurora/rev/8c73047cec44
Attachment #8577543 - Flags: approval-comm-aurora? → approval-comm-aurora+
This bug needs to be marked fixed, but I don't know the conventions in chat for setting the target milestone.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.