Thanks, Anton. I committed your patch to svn trunk.
hannes
2008/4/21 Anton Pirker <
helma@...>:
> Hello!
>
> A friend of mine (a brand new Helma developer!) just tried to read an
> RSS-Feed with Helma and discovered that if there is a ; in the
> charset-string an error is thrown.
>
> We found the problem in Http.js in modules/helma.
> The Charset-String is parsed from the result of the helma.Http() result. I
> have attached a little fix.
>
> Could someone of the commiters please check this and maybe commit it?
>
> Cheers,
> Anton
>
> --
> DI(FH) Anton Pirker, Helma Freelancer
>
>
> Index: Http.js
> ===================================================================
> --- Http.js (revision 8795)
> +++ Http.js (working copy)
> @@ -577,7 +577,7 @@
>
> if (result.type && result.type.indexOf("charset=") != -1) {
> var charset =
> result.type.substring(result.type.indexOf("charset=") + 8);
> - charset = charset.replace('"', ' ').trim();
> + charset = charset.replace(/[;"]/g, '');
> result.charset = charset;
> }
>
>
> _______________________________________________
> Helma-user mailing list
>
Helma-user@...
>
http://helma.org/mailman/listinfo/helma-user>
>
_______________________________________________
Helma-user mailing list
Helma-user@...
http://helma.org/mailman/listinfo/helma-user