not really, because securelink wont live in the core project and so
wont be as discoverable
-igor
On Sun, Mar 16, 2008 at 3:51 PM, Maurice Marrink <[EMAIL PROTECTED]> wrote:
> Alright i can see your point, but if i take your advise and create my
> own SecurePageLink that extends Link and probably will use an
> interface similar to IPageLink. Won't the effect of deleting PageLink
> be nullified when Wicket-Security is integrated in to core?
>
> Maurice
>
>
>
> On Sun, Mar 16, 2008 at 11:41 PM, Igor Vaynberg <[EMAIL PROTECTED]> wrote:
> > 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.
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
>
