so now it calls isVisible?
so just call isVisibleInHierarchy()
so instead of this:
*
else* *if* (childComponent != *null*)
{
// Delegate to child component
setVisible(childComponent.isVisible());
}
do this:
*
else* *if* (childComponent != *null*)
{
// Delegate to child component
setVisible(childComponent.*isVisibleInHierarchy*());
}
johan
On 10/31/07, Igor Vaynberg <[EMAIL PROTECTED]> wrote:
>
> there seems to be a bit of a disconnect between "render" in auth and
> our general component visibility concept. perhaps it might be an
> improvement to aligh auth strategy with visibility rather then
> render...what do others think?
>
> -igor
>
>
> On 10/31/07, Sebastiaan van Erk <[EMAIL PROTECTED]> wrote:
> > Hi,
> >
> > OK, figured it might be something like this!
> > Thanks for the fast reply. :-)
> >
> > Regards,
> > Sebastiaan
> >
> > Igor Vaynberg wrote:
> > > enclosures work on the visibility level, not render level. since your
> > > adminlink is visible, but its rendering is aborted the enclosure still
> > > shows the content.
> > >
> > > to do this you have to put the link into a webmarkupcontainer, and
> > > authorize that container instead of a link.
> > >
> > > -igor
> > >
> > >
> > > On 10/31/07, Sebastiaan van Erk <[EMAIL PROTECTED]> wrote:
> > >> Hi,
> > >>
> > >> I have a main menu with an admin link which only renders when the
> user
> > >> has the "ADMIN" role (MainMenu.java):
> > >>
> > >> final BookmarkablePageLink adminLink = new
> > >> BookmarkablePageLink("adminLink", AdminHomePage.class);
> > >> MetaDataRoleAuthorizationStrategy.authorize(adminLink, RENDER,
> "ADMIN");
> > >> add(adminLink);
> > >>
> > >> In my MainMenu.html I have:
> > >>
> > >> <wicket:enclosure id="adminLink">
> > >> <li>
> > >> <a wicket:id="adminLink">Administratie</a>
> > >> </li>
> > >> </wicket:enclosure>
> > >>
> > >> I was hoping that when the admin link is not rendered due to the user
> > >> not having the proper role, that the <li></li> would also not be
> > >> rendered, however, it does not seem to work this way...
> > >>
> > >> Is this not the way I'm supposed to do this? Or should this work?
> > >> Otherwise, what is the right way to go about this?
> > >>
> > >> Thanks in advance,
> > >> Sebastiaan
> > >>
> > >>
> > >
> > > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > > For additional commands, e-mail: [EMAIL PROTECTED]
> > >
> >
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
