On 9/16/06, Martijn Dashorst <[EMAIL PROTECTED]> wrote:
> 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.

For what are you using it? Hopefully only for things like linking back
to a page you passed in earlier. Which would easily be solved by:

new Link(this, "back") {
  public void onClick() {
    setResponsePage(otherPage);
  }
}

and if you really use that a lot, create your own component:

class MyPageLink extends Link {

  private final Page page;

  public MyPageLink(MarkupContainer parent, String id, Page page) {
    this.page = page;
  }

  public void onClick() {
    setResponsePage(page);
  }
}

That's not a lot of work, and at least the user would be fully aware
of the consequences.

> 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.

Yeah, let's forget about private, final and all those things and let's
just depend on the documentation.

;)

Eelco

-------------------------------------------------------------------------
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