Closed
Bug 304507
Opened 19 years ago
Closed 19 years ago
Ignore matches fails for CASEMAPPINGS outside of A-Z
Categories
(Other Applications :: ChatZilla, defect)
Other Applications
ChatZilla
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Gijs, Assigned: bugzilla-mozilla-20000923)
References
()
Details
(Whiteboard: [cz-0.9.69])
Attachments
(1 file)
|
923 bytes,
patch
|
samuel
:
review+
Gijs
:
review+
|
Details | Diff | Splinter Review |
... which means it buggers because the ignores are lowercased using that method, and /i doesn't cut it when a server uses rfc1459 charsets. Patch by Silver coming up, as I understand it.
|
Assignee | |
Updated•19 years ago
|
Summary: Ignore matches using the /i regexp option instead of lowercasing using e.server.toLowerCase → Ignore matches fails for CASEMAPPINGS outside of A-Z
|
Reporter | |
Comment 2•19 years ago
|
||
Comment on attachment 192549 [details] [diff] [review] Lower-cases nickname properly before comparing Discussed on IRC, seems fine to me :-).
Attachment #192549 -
Flags: review+
|
||
Updated•19 years ago
|
Attachment #192549 -
Flags: review?(samuel) → review+
|
|
Assignee | |
Comment 4•19 years ago
|
||
Um, why? The problem is ^, [, ] and \ are not being matched to ~, {, } and |.|
|
Assignee | |
Comment 5•19 years ago
|
||
Checked in --> FIXED.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•19 years ago
|
Whiteboard: [cz-0.9.68.6] → [cz-0.9.69]
You need to log in
before you can comment on or make changes to this bug.
Description
•