testPathEncoding fails with self assert: url printString = '/foo+bar'.

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

testPathEncoding fails with self assert: url printString = '/foo+bar'.

johnmci
I loaded Seaside2.8a1-lr.578 into a  Pharo0.1 #10124 image and when I
run the seaside Sunits
I get 5 failures and 2 errors.

Most of the 5 failures seem to relate to some problem with "+"

such as

        url path: #( 'foo bar' ).
        self assert: url printString = '/foo+bar'.

I see Seaside2.8a1-lr.578  says

- encoding spaces as + in urls is vicious and does not work on all web clients

yet  the test fails so I'm wondering if the required change is in some
other package than Seaside2.8a1-lr.578

--
===========================================================================
John M. McIntosh <[hidden email]>
Corporate Smalltalk Consulting Ltd.  http://www.smalltalkconsulting.com
===========================================================================
_______________________________________________
seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: testPathEncoding fails with self assert: url printString = '/foo+bar'.

Lukas Renggli
I back-ported this change from Seaside 2.9. Unfortunately the encoding
table doesn't seem to get updated automatically. Evaluate
"WAUrlEncoder initialize" and then your tests should pass.

Cheers,
Lukas

On Fri, Oct 31, 2008 at 9:13 PM, John McIntosh
<[hidden email]> wrote:

> I loaded Seaside2.8a1-lr.578 into a  Pharo0.1 #10124 image and when I
> run the seaside Sunits
> I get 5 failures and 2 errors.
>
> Most of the 5 failures seem to relate to some problem with "+"
>
> such as
>
>        url path: #( 'foo bar' ).
>        self assert: url printString = '/foo+bar'.
>
> I see Seaside2.8a1-lr.578  says
>
> - encoding spaces as + in urls is vicious and does not work on all web clients
>
> yet  the test fails so I'm wondering if the required change is in some
> other package than Seaside2.8a1-lr.578
>
> --
> ===========================================================================
> John M. McIntosh <[hidden email]>
> Corporate Smalltalk Consulting Ltd.  http://www.smalltalkconsulting.com
> ===========================================================================
> _______________________________________________
> seaside mailing list
> [hidden email]
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>



--
Lukas Renggli
http://www.lukas-renggli.ch
_______________________________________________
seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside