It does one thing and one thing good: create a link to an already existing page. We use it quite often in our apps. I'm +1(00) for the constructor to stay.
Read the docs and understand what you're doing. If we were to remove everything that 'encourages' bad practice we wouldn't have a framework left. Anything can and will be misused. Martijn On 9/16/06, Eelco Hillenius <[EMAIL PROTECTED]> wrote: > 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 > -- Download Wicket 1.2.2 now! New Ajax components: Tree, TreeTable and ModalWindow -- http://wicketframework.org ------------------------------------------------------------------------- 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
