At the very least, I'd be +100 for removing the constructor with the
Page. The only thing that constructor does is encourage bad practice.

Eelco


On 9/16/06, Johan Compagner <[EMAIL PROTECTED]> wrote:
> PageLink also has constructors for a Class.
> A bookmarkable page link is what it says it creates a bookmarkable link But
> maybe you don't want that at all
> then you can use a PageLink and the PageLink class with the IPageLink param
> i the constructor is an easy think
> for lazy creating pages but directly with state.
>
> johan
>
>
>
> On 9/16/06, Igor Vaynberg <[EMAIL PROTECTED] > wrote:
> >
> >
> >
> >
> >
> >
> > > Why would you even create the page while you're actually linking to
> > > it? E.g, when you do ( 2.0 code):
> >
> >
> >
> >
> > because he wants to link to a page, the obvious component is PageLink and
> it takes a Page instance in its contructor - so its the most obvious way to
> link to a page for a noob. you guys keep outvoting me on removing that
> constructor, but its just causing problems :) The whole class is not even
> necessary. You have Link and BookmarkableLink, implementing the auto-enable
> is as simple as overriding isenabled().
> >
> > -Igor
> >
> >
> >
> >
> >
> -------------------------------------------------------------------------
> > Using Tomcat but need to do more? Need to support web services, security?
> > Get stuff done quickly with pre-integrated technology to make your job
> easier
> > Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
> >
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> >
> > _______________________________________________
> > Wicket-user mailing list
> > Wicket-user@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/wicket-user
> >
> >
> >
>
>
> -------------------------------------------------------------------------
> Using Tomcat but need to do more? Need to support web services, security?
> Get stuff done quickly with pre-integrated technology to make your job
> easier
> Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
>
> _______________________________________________
> Wicket-user mailing list
> Wicket-user@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/wicket-user
>
>
>

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Wicket-user mailing list
Wicket-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-user

Reply via email to