No/incorrect error message when joining a channel fails

RESOLVED FIXED in 1.2

Status

Chat Core
IRC
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: aleth, Assigned: clokep)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

5 years ago
*** Original post on bio 1418 at 2012-05-02 16:28:00 UTC ***

Both via "Join chat" and /join, the only feedback currently is the server tab error message
"There is no channel: aleth."
(note I did _not_ attempt to join a channel named aleth - the error message picks up the wrong parameter?)

For "Join chat" one would probably expect feedback in the dialog box (and the chance to try again), for /join in the conversation.
(Reporter)

Updated

5 years ago
Blocks: 954050
(Assignee)

Comment 1

5 years ago
Created attachment 8353186 [details] [diff] [review]
Patch

*** Original post on bio 1418 as attmnt 1434 at 2012-05-04 00:37:00 UTC was without comment, so any subsequent comment numbers will be shifted ***
Attachment #8353186 - Flags: review?(bugzilla)
(Assignee)

Updated

5 years ago
Assignee: nobody → clokep
Status: NEW → ASSIGNED
(Reporter)

Comment 2

5 years ago
Comment on attachment 8353186 [details] [diff] [review]
Patch

*** Original change on bio 1418 attmnt 1434 at 2012-05-04 09:18:01 UTC ***

Works very well. It would be nice if one could do without opening the tab for the nonexisting channel, but that seems almost impossible.

Thanks for fixing this!
Attachment #8353186 - Flags: review?(bugzilla) → review+
(Reporter)

Updated

5 years ago
Whiteboard: [checkin-needed]
*** Original post on bio 1418 at 2012-05-04 09:28:21 UTC ***

Comment on attachment 8353186 [details] [diff] [review] (bio-attmnt 1434)
Patch

>diff --git a/chat/protocols/irc/ircBase.jsm b/chat/protocols/irc/ircBase.jsm

>-      return serverErrorMessage(this, aMessage,
>-                                _("error.noChannel", aMessage.params[0]));
>+      conversationErrorMessage(this, aMessage, "error.noChannel");
>+      return true;

Why not just:
      return conversationErrorMessage(this, aMessage, "error.noChannel");
?
(Assignee)

Comment 4

5 years ago
Created attachment 8353188 [details] [diff] [review]
Patch v2

*** Original post on bio 1418 as attmnt 1436 at 2012-05-04 10:29:00 UTC ***

Florian: you're right, carrying the review forward.
Attachment #8353188 - Flags: review+
(Assignee)

Comment 5

5 years ago
Comment on attachment 8353186 [details] [diff] [review]
Patch

*** Original change on bio 1418 attmnt 1434 at 2012-05-04 10:29:54 UTC was without comment, so any subsequent comment numbers will be shifted ***
Attachment #8353186 - Attachment is obsolete: true
*** Original post on bio 1418 at 2012-05-04 23:33:37 UTC ***

https://hg.instantbird.org/instantbird/rev/194399651469
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [checkin-needed]
Target Milestone: --- → 1.2
You need to log in before you can comment on or make changes to this bug.