yep, and once that is gone PageLink essentially becomes bloat...
-igor
On Sun, Mar 16, 2008 at 3:35 PM, Maurice Marrink <[EMAIL PROTECTED]> wrote:
> Ah but that constructor is already deprecated.
>
> Maurice
>
>
>
> On Sun, Mar 16, 2008 at 11:32 PM, Igor Vaynberg <[EMAIL PROTECTED]> wrote:
> > it causes a lot of noobs to do add(new PageLink("foo", new
> EditPage(userid));
> >
> > -igor
> >
> >
> >
> >
> > On Sun, Mar 16, 2008 at 3:28 PM, Maurice Marrink <[EMAIL PROTECTED]>
> wrote:
> > > Sure i can, but what is so horrible about PageLink that you want to
> > > deprecate/remove it?
> > >
> > > Maurice
> > >
> > >
> > >
> > > On Sun, Mar 16, 2008 at 11:24 PM, Igor Vaynberg <[EMAIL PROTECTED]>
> wrote:
> > > > can you not have your own SecureLink variant?
> > > >
> > > > -igor
> > > >
> > > >
> > > >
> > > >
> > > > On Sun, Mar 16, 2008 at 3:22 PM, Maurice Marrink <[EMAIL
> PROTECTED]> wrote:
> > > > > From a Wicket-Security point of view i like the PageLink and
> IPageLink
> > > > > because it knows what pageclass is going to be instantiated and
> can
> > > > > use that to check permissions.
> > > > > If i were to use Link i would have to make that information
> somehow
> > > > > available in another way. Although i would not mind if IPageLink
> gets
> > > > > renamed.
> > > > >
> > > > > Maurice
> > > > >
> > > > >
> > > > >
> > > > > On Sun, Mar 16, 2008 at 11:16 PM, James Carman
> > > > > <[EMAIL PROTECTED]> wrote:
> > > > > > On 3/16/08, Igor Vaynberg <[EMAIL PROTECTED]> wrote:
> > > > > > > my thought on this is to deprecate PageLink entirely, or at
> least the
> > > > > > > entire IPageLink deal in case someone wants to have a
> add(new
> > > > > > > PageLink("link", mypageinstance) construct.
> > > > > > >
> > > > > > > pagelink itself provides very little value, nothing a
> > > > > > > BookmarkablePageLink or Link cannot do easily or better
> > > > > > >
> > > > > >
> > > > > > Even better! :) I was trying to figure out how to tell folks
> to use
> > > > > > PageLink rather than BookMarkablePageLink, especially since
> the
> > > > > > version that takes a Page instance is deprecated. I think
> I'll
> > > > > > completely remove PageLink from my presentation.
> > > > > >
> > > > >
> > > >
> > >
> >
>
