Invalid regexp in /rlist should not show internal error

RESOLVED FIXED

Status

RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: merlyell, Assigned: Gijs)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [cz-0.9.79])

Attachments

(1 attachment)

(Reporter)

Description

11 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.5) Gecko/20070713 Firefox/2.0.0.5
Build Identifier: 2007070518

On calling '/rlist *' command cz returns:
[ERROR] Internal error dispatching command “rlist”.
[ERROR] SyntaxError: invalid quantifier * @ <chrome://chatzilla/content/commands.js> 2083
Cz should not show it's internal error message. It should show some nice user friendly message like '[ERROR]	Incorrect regexp'

Reproducible: Always

Updated

11 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows XP → All
Hardware: PC → All
Summary: incorrect /rlist command should not show internal errors → Invalid regexp in /rlist should not show internal error
Version: unspecified → Trunk
(Assignee)

Comment 1

11 years ago
Created attachment 282102 [details] [diff] [review]
Patch with wikipedia link thrown in for good measure
Assignee: rginda → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Attachment #282102 - Flags: review?(silver)

Comment 2

11 years ago
Comment on attachment 282102 [details] [diff] [review]
Patch with wikipedia link thrown in for good measure

Poor Wikipedia. :)
Attachment #282102 - Flags: review?(silver) → review+
(Assignee)

Comment 3

11 years ago
Checking in mozilla/extensions/irc/locales/en-US/chrome/chatzilla.properties;
/cvsroot/mozilla/extensions/irc/locales/en-US/chrome/chatzilla.properties,v  <--  chatzilla.properties
new revision: 1.140; previous revision: 1.139
done
Checking in mozilla/extensions/irc/xul/content/commands.js;
/cvsroot/mozilla/extensions/irc/xul/content/commands.js,v  <--  commands.js
new revision: 1.129; previous revision: 1.128
done
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
Whiteboard: [cz-0.9.79]
You need to log in before you can comment on or make changes to this bug.