322076 - cannot set an URI which contains a space in datasources

Status: UNCONFIRMED

(Core :: XUL, defect)

Description

[Yvan Norsa]

User-Agent:       Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8) Gecko/20051111 Firefox/1.5
Build Identifier: Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8) Gecko/20051111 Firefox/1.5

The datasources attribute allows to put several sources URI, separated by a space character. If one of those URIs contains a space character in a folder name or in the filename, it will be considered as many URIs and won't properly load.

Reproducible: Always

Steps to Reproduce:
1.Write a simple XUL file
2.Put a widget which allows the use of datasources
3.Set the attribute to a filename containing a space

Actual Results:  
The RDF file isn't loaded.

Expected Results:  
The RDF file should have been loaded.

Maybe change the space separation character for another, like a comma.

Comment 1

[Christian :Biesinger (don't email me, ping me on IRC)]

URIs are not allowed to contain spaces, use %20 instead.

Comment 2

[Yvan Norsa]

OK, but couldn't it be done automatically ? If the developper has to do a filename.replace(/ /g, "%20") each time, it is not very user-friendly.. 

Comment 3

[Christian :Biesinger (don't email me, ping me on IRC)]

eh? what you should do is use valid URIs. URIs with spaces aren't valid. JavaScript provides an encodeURI function for this purpose.

Comment 4

[Yvan Norsa]

Sure. But what about people writing a XUL file, and filling in the datasources attribute with a filename containing spaces (for example, a file in the profile directory, on Windows) ?
OK, it is stated in the doc that the separator is the space character. But you can't say the user got any feedback saying why his rdf file won't load :\

Comment 5

[Christian :Biesinger (don't email me, ping me on IRC)]

I guess I'll reopen and move this to an XUL component...