Seaside2.7a1-lr.173 (was: Seaside 2.7a1 versions 169)

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

Seaside2.7a1-lr.173 (was: Seaside 2.7a1 versions 169)

Lukas Renggli
Strings, strings, strings ... I can't stand strings where there should
be objects.

Dispatchers and EntryPoint instances used to have a hardcoded
basepath. With every version of Seaside methods were introduced to
parse, concatenate and transform these ugly strings. Now I published a
version that makes some minor changes in the model but has some bigger
effect. I think this is one of the last big steps to finish the
initial goals we posted for the 2.7 version.

Not that this change will break your existing dispatcher configuration
if you load it into an existing image. In the comment it says:

-=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=-

- probably the last big refactoring of 2.7: WARequestHandler
- request handler do have a parent-link now
- request handler do know their-name now (and no basePath)
- request handler generate their basePath using parent-link and name

If you load this version into an existing image you need to
re-register all your compoents by evaluating:

        WAKom stop; startOn: 8080.
        WAComponent allSubclassesDo: [ :each | each initialize ].
       
Basic stuff works, but there are certainly bugs introduced with this
change. Lots of things can be now done easier, without fiddeling
around with strings. Objects forever!

-=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=-

There are still a few places to adapt, but most of it should work.

Cheers,
Lukas

--
Lukas Renggli
http://www.lukas-renggli.ch
_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: Seaside2.7a1-lr.173 (was: Seaside 2.7a1 versions 169)

Avi Bryant-2
On 2/12/07, Lukas Renggli <[hidden email]> wrote:
> Strings, strings, strings ... I can't stand strings where there should
> be objects.

But Lukas, strings *are* objects... ;)

Great work.  I know I've been pretty silent and uninvolved with 2.7,
but it's great to watch the progress from afar as you guys fix all the
places where I was lazy...

Avi
_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

FileLibrary-RJT.19

Ron Teitelbaum
In reply to this post by Lukas Renggli
FileLibrary-RJT.19

Changed WAFileHandler to match new WADispatcher pattern.

Added addLibrary: WAFileHandler so that it matches other application
registrations.  See comments on method.

Ron Teitelbaum
[hidden email]

> From: Lukas Renggli
> Sent: Monday, February 12, 2007 5:02 PM
>
> Strings, strings, strings ... I can't stand strings where there should
> be objects.
>
> Dispatchers and EntryPoint instances used to have a hardcoded
> basepath. With every version of Seaside methods were introduced to
> parse, concatenate and transform these ugly strings. Now I published a
> version that makes some minor changes in the model but has some bigger
> effect. I think this is one of the last big steps to finish the
> initial goals we posted for the 2.7 version.
>
> Not that this change will break your existing dispatcher configuration
> if you load it into an existing image. In the comment it says:
>
> -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=-
>
> - probably the last big refactoring of 2.7: WARequestHandler
> - request handler do have a parent-link now
> - request handler do know their-name now (and no basePath)
> - request handler generate their basePath using parent-link and name
>
> If you load this version into an existing image you need to
> re-register all your compoents by evaluating:
>
> WAKom stop; startOn: 8080.
> WAComponent allSubclassesDo: [ :each | each initialize ].
>
> Basic stuff works, but there are certainly bugs introduced with this
> change. Lots of things can be now done easier, without fiddeling
> around with strings. Objects forever!
>
> -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=-
>
> There are still a few places to adapt, but most of it should work.
>
> Cheers,
> Lukas
>
> --
> Lukas Renggli
> http://www.lukas-renggli.ch
> _______________________________________________
> Seaside mailing list
> [hidden email]
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside


_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: FileLibrary-RJT.19

Philippe Marschall
2007/2/13, Ron Teitelbaum <[hidden email]>:
> FileLibrary-RJT.19
>
> Changed WAFileHandler to match new WADispatcher pattern.
>
> Added addLibrary: WAFileHandler so that it matches other application
> registrations.  See comments on method.

WAFilelibrary is not part of Seaside. So if you change something there
please do it in Seaside (unless you use an old version of Seaside that
doesn't yet have FileLibrary in).

Now about your two changes:
- It looks as if WAFileHandler class >> #registerAsHandler: got fixed
by Lukas already
- the WAFileLibrary >> #addLibrary: comment is misleading. First
#register is already implemented in WAFileLibrary (as well as
#unregister). Second an initialize of a WAFileLibrary subclass should
not send super initialize only self register.


Cheers
Philippe

> Ron Teitelbaum
> [hidden email]
>
> > From: Lukas Renggli
> > Sent: Monday, February 12, 2007 5:02 PM
> >
> > Strings, strings, strings ... I can't stand strings where there should
> > be objects.
> >
> > Dispatchers and EntryPoint instances used to have a hardcoded
> > basepath. With every version of Seaside methods were introduced to
> > parse, concatenate and transform these ugly strings. Now I published a
> > version that makes some minor changes in the model but has some bigger
> > effect. I think this is one of the last big steps to finish the
> > initial goals we posted for the 2.7 version.
> >
> > Not that this change will break your existing dispatcher configuration
> > if you load it into an existing image. In the comment it says:
> >
> > -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=-
> >
> > - probably the last big refactoring of 2.7: WARequestHandler
> > - request handler do have a parent-link now
> > - request handler do know their-name now (and no basePath)
> > - request handler generate their basePath using parent-link and name
> >
> > If you load this version into an existing image you need to
> > re-register all your compoents by evaluating:
> >
> >       WAKom stop; startOn: 8080.
> >       WAComponent allSubclassesDo: [ :each | each initialize ].
> >
> > Basic stuff works, but there are certainly bugs introduced with this
> > change. Lots of things can be now done easier, without fiddeling
> > around with strings. Objects forever!
> >
> > -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=- -=-
> >
> > There are still a few places to adapt, but most of it should work.
> >
> > Cheers,
> > Lukas
> >
> > --
> > Lukas Renggli
> > http://www.lukas-renggli.ch
> > _______________________________________________
> > Seaside mailing list
> > [hidden email]
> > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>
>
> _______________________________________________
> Seaside mailing list
> [hidden email]
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>
_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

RE: FileLibrary-RJT.19

Ron Teitelbaum
> From: Philippe Marschall

> WAFilelibrary is not part of Seaside. So if you change something there
> please do it in Seaside (unless you use an old version of Seaside that
> doesn't yet have FileLibrary in).

Ahhh, ok now I understand.  I have been loading both for quite a while now
but didn't realize that the code was moved to the base seaside.  It's not
easy to know that.  I must have missed it when it was announced.  

Thanks,

Ron




_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside