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