[Seaside] Possible bug in WAAdmin register: aRequestHandlerClass at: aString in: aDispatcher

Lukas Renggli renggli at gmail.com
Fri Aug 27 09:41:15 UTC 2010


The thing is that #subStrings: takes a String as argument according to
ANSI 5.7.10.15.

This is also tested in GRStringTest>>#testSubStrings.

So this must be a bug in VW.

Lukas

On 27 August 2010 10:48, Bart Veenstra <bart.veenstra at gmail.com> wrote:
> I am on vw 77 with the latest seaside 3.0 from the public repos (87 i
> think). I checked the seaside one click image as well, and it also used a
> '/' instead of a /$ for tokenizing a String.
>
>
>
> 2010/8/27 Lukas Renggli <renggli at gmail.com>
>>
>> This is strange. What platform are you on? What version?
>>
>> On Thursday, August 26, 2010, Bart Veenstra <bart.veenstra at gmail.com>
>> wrote:
>> > Hi all,
>> > While registring applications seaside other than root, my applications
>> > where not accessible at their paths. It seemd that the dispatcher was not
>> > tokenzing the path correctly, because the it takes the '/' as argument
>> > instead of the $/ causing the applications to be registered at the whole
>> > path, instead of registering a WADispatcher for the sub paths.
>> >
>> > Don't know of this is the right list for it, but it caused me some
>> > headaches yesterday evening :)
>> > Possible fix:
>> > WAAdmin register: aRequestHandlerClass at: aString in: aDispatcher
>> >       | path dispatcher |     path := aString subStrings: $/. dispatcher
>> > := path allButLast
>> >               inject: aDispatcher             into: [ :result :each |
>> >               result handlers
>> >                               at: each
>> >  ifAbsent: [ result register: WADispatcher new at: each ] ].
>> >       ^ dispatcher register: aRequestHandlerClass new at: path last
>> > Regards,
>> > Bart
>> >
>>
>> --
>> Lukas Renggli
>> www.lukas-renggli.ch
>> _______________________________________________
>> seaside mailing list
>> seaside at lists.squeakfoundation.org
>> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>
>
> _______________________________________________
> seaside mailing list
> seaside at lists.squeakfoundation.org
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>
>



-- 
Lukas Renggli
www.lukas-renggli.ch


More information about the seaside mailing list