[Seaside-dev] Re: [Seaside] returning a response in seaside 2.9

[Lukas Renggli]

> I see you did this already today. Great.

Yeah, that was really missing. It is much more concise now.

The only thing that still looks a bit strange is the response generator:

	aRequestContext responseGenerator
		notImplemented;
		respond


It is not as bad as the other approach was, but maybe you have an idea
for improvement here too?

Lukas

> On Wed, May 20, 2009 at 8:20 AM, Lukas Renggli <renggli at gmail.com> wrote:
>>> We've discussed this before but it kind of doesn't make sense having
>>> WAResponse implement #respond,
>>
>> I don't like that either.
>>
>>> self requestContext respond: [ :response |
>>>    response
>>>      contentType: WAMimeType textPlain;
>>>      nextPutAll: 'Hello World' ]
>>>
>>> A few more characters ("[ :response | ]") but not many and I think it
>>> makes more sense. Obviously #response and #respond would still exist
>>> for cases where you don't want to modify the response and respond at
>>> the same time.
>>>
>>> What do you think?
>>
>> Yeah, this is cool.
>>
>> Cheers,
>> Lukas
>>
>> --
>> Lukas Renggli
>> http://www.lukas-renggli.ch
>> _______________________________________________
>> seaside-dev mailing list
>> seaside-dev at lists.squeakfoundation.org
>> http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
>>
> _______________________________________________
> seaside-dev mailing list
> seaside-dev at lists.squeakfoundation.org
> http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
>



-- 
Lukas Renggli
http://www.lukas-renggli.ch