Comanche ChieftainModule default
Kamil Kukura
kamk at volny.cz
Mon Apr 29 09:50:27 UTC 2002
Nevin Pratt wrote:
> The new 'ChieftainModule' class in the newer Comanche is interesting.
> Congratulations and thanks to whoever did this.
>
> One comment though. At the end of the #process: method, where the
> comment reads "otherwise, the default response", it just returns nil.
> In other words, "do nothing" is the default. I suggest we allow a
> default that is a bit more meaningful than this. I suggest we be able
> to install any Comanche module and use it as the default, and only
> return nil as the default when there hasn't been a specific Comanche
> module installed as the default handler.
>
> So, we add a 'default' instance variable, give it a setter method, and
> send the request to the default (if it exists) instead of answering nil.
I was thinking about #defaultPrefix. In case of request for root (/) it
would send response with redirect (header Location:) to that defaultPrefix.
> But otherwise, I think the ChieftainModule is nice, clean,
> straight-forward, and (of course) simple.
>
> Nevin
Oh yea... but not evolving somehow. There's todo list in class comment
and last change claims to be on 6/21/2001 03:08.
--
Kamil
More information about the Squeak-dev
mailing list
|