Celestia URLs are not URL encoded

[jdavies]

Just a quick note, when Celestia creates a URL (using the CTRL+C keystroke) the resulting URL is not properly URL encoded. For example, when observing a satellite named "OFEQ 7" the resulting URL is:

Code: Select all

cel://Follow/Sol:Earth:OFEQ 7/2009-01-29T17:53:26.53329?x=3wbP4QU&y=+IszDAE&z=7l5Jpgk&ow=0.961725&ox=0.0440697&oy=-0.270087&oz=-0.0139648&select=Sol:Earth:OFEQ 7&fov=35.5732&ts=1&ltd=0&p=0&rf=1955607&lm=2179&tsrc=0&ver=3

The whitespace should be altered to %20. This is also a problem for satellites with the "[+]" notation in their names (of which there are many). Is it possible to get Celestia to do this URL encoding? Many thanks!

- Jeff

[rthorvald]

Another URL bug, this one regarding InfUrls:
The # (anchor) tag in InfoUrls are not properly escaped, it is just substituted with %23. That does not work, at least not in Firefox.

(Sorry for the threadjack, just thought it was tidy to have these in the same spot).

- rthorvald

[chris]

Thanks for noting these bugs... Anyone care to enter them in the SourceForge bug tracker so that they're not forgotten?

http://sourceforge.net/tracker/?group_i ... tid=121302

--Chris

[John Van Vliet]

--- edit ---

[Chuft-Captain]

OFEQ 7 i knew there was a reason i hated spaces in add on names , besides running Linux
OFEQ_7
OFEQ-7
OFEQ.7

If these bugs are fixed you wont have to hate them anymore.

[Chuft-Captain]

Thanks for noting these bugs... Anyone care to enter them in the SourceForge bug tracker so that they're not forgotten?

Done. Bug#: 2581786

[chris]

Thanks for noting these bugs... Anyone care to enter them in the SourceForge bug tracker so that they're not forgotten?

Done. Bug#: 2581786

Thanks! I'm not sure if the fix will make it into 1.6.0. If not, then definitely the next version.

--Chris