Hi,

Am 21.09.2006 um 11:44 schrieb Ricardo Borillo:

> Sebastian,
>
> It's ok Fabian's patch or we open a bug?

[snip]

> -    vParametersList.push(vId + qx.constant.Core.EQUAL + vParameters 
> [vId]);
>
> +    vParametersList.push(escape(vId) + qx.constant.Core.EQUAL +  
> escape(vParameters[vId]));

I haven't really looked at the patch (especially possible side  
effects), so I don't know whether commiting it would be OK. However,  
if it's accepted, the encodeURIComponent() function should be used  
instead of escape() (because it correctly deals with non-Latin-1  
characters).

Regards,

   Andreas


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to