wrong channel opens up when I click on non-last entry in output of /pref initialURLs

RESOLVED FIXED

Status

Other Applications
ChatZilla
RESOLVED FIXED
14 years ago
13 years ago

People

(Reporter: Jacob, Assigned: Robert Ginda)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

14 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7b) Gecko/20040421
Build Identifier: Chatzilla 0.9.63b [Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7b) Gecko/20040421]

When "initialURLs" pref contains more than one entry, clicking on the 
non-last element of the output of "/pref initialURLs" opens the wrong channel.
Namely it opens the "#channel;" (with with ";" wrongly appended).

For example if channel "irc://irc.mozilla.org/chatzilla" is in initialURLs,
and non-last, then clicking on it will open not the original channel
"#chatzilla", but the channel "#chatzilla;" .


Reproducible: Always
Steps to Reproduce:



Expected Results:  
One possible solution is to make initialURLs space-separated instead of
";"-separated (separated by a char that can never appear unencoded in 
the URL, that is); and recognize ';' as separator for back-compatibility.

Comment 1

14 years ago
I have this fixed locally, I'll try to extract the relevant bit and attach sometime.
(Reporter)

Comment 2

14 years ago
(In reply to comment #1)
> I have this fixed locally, I'll try to extract the relevant bit and attach
sometime.

Please attach the fix.

Comment 3

14 years ago
Created attachment 149328 [details] [diff] [review]
Use arraySpeak to show array prefs, and parse arraySpeak format when setting.

This also has my fix to make the /pref command always show an error if the pref
doesn't exist (rather than having a JS error in some cases).
Product: Core → Other Applications

Updated

13 years ago
OS: Windows 2000 → All
Hardware: PC → All

Comment 4

13 years ago
This got fixed somewhere along the way.
Status: UNCONFIRMED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.