Closed
Bug 165332
Opened 23 years ago
Closed 21 years ago
chatzilla should give better error messages when /join fails
Categories
(Other Applications :: ChatZilla, defect)
Other Applications
ChatZilla
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: samuel, Assigned: bugzilla-mozilla-20000923)
Details
(Whiteboard: cz-patch)
Attachments
(2 files)
2.40 KB,
patch
|
samuel
:
review+
|
Details | Diff | Splinter Review |
1.30 KB,
patch
|
samuel
:
review+
|
Details | Diff | Splinter Review |
There are specific error codes for when a join fails for different reasons.
For example: 475 ERR_BADCHANNELKEY "<channel> :Cannot join channel (+k)"
Instead of just displaying the message, chatzilla should give a clearer reason
for the error, like "You need a key", or "You entered the wrong key".
Assignee | ||
Updated•21 years ago
|
Assignee: rginda → silver
Assignee | ||
Updated•21 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•21 years ago
|
||
I thought it would be nice to make replacing random error numerics easy, so
this code just tries to find a string, and if it fails it falls back to the old
code. Patch includes the four join errors that I could find.
Assignee | ||
Updated•21 years ago
|
Whiteboard: cz-patch
Assignee | ||
Updated•21 years ago
|
Attachment #141194 -
Flags: review?(samuel)
Reporter | ||
Updated•21 years ago
|
Attachment #141194 -
Flags: review?(samuel) → review+
Assignee | ||
Comment 2•21 years ago
|
||
Comment on attachment 141194 [details] [diff] [review]
Display better messages for join errors
Checked in.
Assignee | ||
Updated•21 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Comment 3•21 years ago
|
||
there's a couple of mistakes in one of the messages that got added here
"This channel has reached it's set capacity; you connot join it."
"it's" should be "its" and "connot" should be "cannot"
Assignee | ||
Comment 4•21 years ago
|
||
Assignee | ||
Updated•21 years ago
|
Attachment #144222 -
Flags: review?(samuel)
Reporter | ||
Updated•21 years ago
|
Attachment #144222 -
Flags: review?(samuel) → review+
Updated•20 years ago
|
Product: Core → Other Applications
You need to log in
before you can comment on or make changes to this bug.
Description
•