« Return to Thread: Bug in Http.js of Helma?

Re: Bug in Http.js of Helma?

by Hannes Wallnoefer-2 :: Rate this Message:

Reply to Author | View in Thread

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

 « Return to Thread: Bug in Http.js of Helma?