[Lift] Re: Menu access control not working as expected

2009-05-15 Thread David Pollak
On Fri, May 15, 2009 at 7:49 AM, glenn wrote: > > David, > > Yes, your EarlyAccess Loc did the trick. Thank you for all > your help. Thanks for the great suggestion. Party on! > > > Glenn... > > On May 15, 6:54 am, David Pollak > wrote: > > Glenn, > > > > My original design was a bad one...

[Lift] Re: Menu access control not working as expected

2009-05-15 Thread glenn
David, Yes, your EarlyAccess Loc did the trick. Thank you for all your help. Glenn... On May 15, 6:54 am, David Pollak wrote: > Glenn, > > My original design was a bad one... I was mixing access control and early > responses... I've split the concept of access control (which stays the way > it

[Lift] Re: Menu access control not working as expected

2009-05-15 Thread David Pollak
Glenn, My original design was a bad one... I was mixing access control and early responses... I've split the concept of access control (which stays the way it is) and sending an early response (rather than going through the whole rendering pipeline). With the latest spin of Lift, try putting this

[Lift] Re: Menu access control not working as expected

2009-05-14 Thread glenn
David, I changed default html in your menu example a bit and the results were similar to using LocGroup - the menu item won't display. here's the change: Fake Out It seems this only works if you sti

[Lift] Re: Menu access control not working as expected

2009-05-14 Thread glenn
David, Your sample app works. Mine doesn't. I even put your sample Menu item in my app, but in it's own LocGroup, and it does not work. It seems the only difference is that I'm separating my menus into groups. Could that be the culprit? Glenn... On May 13, 9:27 pm, David Pollak wrote: > Glenn,

[Lift] Re: Menu access control not working as expected

2009-05-13 Thread David Pollak
Glenn, Sorry... there was a bug in the code. I've fixed it. It'll be available when this build finishes: http://hudson.scala-tools.org/job/Lift/949/console Please remember to do an mvn -U clean install I'm enclosing the sample app that I used to test. Please note that you should only return Fu

[Lift] Re: Menu access control not working as expected

2009-05-13 Thread glenn
David, Moving CRUDify to the object fixed the compiler error. However, back on the original menu issue, using TestAccess, as you suggest, still did not work as expected. The menu item doesn't display. Maybe I'm missing something important. Sorry to be a bother, but hope you can help. Here's my

[Lift] Re: Menu access control not working as expected

2009-05-13 Thread David Pollak
On Wed, May 13, 2009 at 3:30 PM, glenn wrote: > > Thanks for the heads-up on the 1.1-SNAPSHOT version, but now, when I > compile, > I get errors. I don't think this has anything to do with menu > redirection, but, I have > a Company class defined like so: > CRUDify should be mixed into the Meta

[Lift] Re: Menu access control not working as expected

2009-05-13 Thread glenn
Thanks for the heads-up on the 1.1-SNAPSHOT version, but now, when I compile, I get errors. I don't think this has anything to do with menu redirection, but, I have a Company class defined like so: class Company extends LongKeyedMapper[Company] with Address[Company] with IdPK with CRUDify[Long,Co

[Lift] Re: Menu access control not working as expected

2009-05-13 Thread David Pollak
On Wed, May 13, 2009 at 2:19 PM, glenn wrote: > > David, > > Pardon my ignorance, but I see a Test case class in the Lift 1.0 api > for Loc, but not TestAccess, which seems similar. I added TestAccess to 1.1-SNAPSHOT last night. It's possible to write something in 1.0, but it's a lot harder...

[Lift] Re: Menu access control not working as expected

2009-05-13 Thread glenn
David, Pardon my ignorance, but I see a Test case class in the Lift 1.0 api for Loc, but not TestAccess, which seems similar. Glenn... On May 13, 6:58 am, David Pollak wrote: > Glenn, > > I've added another Loc param: > >   /** >    * Allows extra access testing for a given menu location such

[Lift] Re: Menu access control not working as expected

2009-05-13 Thread David Pollak
Glenn, I've added another Loc param: /** * Allows extra access testing for a given menu location such that * you can build a menu that is displayed but redirects the user to a login * page if they are not logged in */ case class TestAccess(func: () => Box[LiftResponse]) extends L

[Lift] Re: Menu access control not working as expected

2009-05-12 Thread glenn
Hmm, Here's my complete menu list for this LocGroup: def quoteMenu:List[Menu] = { val groupquote = Menu(Loc("groupquote", List("quote", "group"), "Group Quote", LocGroup("quote"), loggedIn )) val businessquote = Menu(Loc("businessquote", List("quote", "business"), "Commercial Quote"

[Lift] Re: Menu access control not working as expected

2009-05-12 Thread David Pollak
If a top-level menu item is not accessible, then none of its children are accessible. SiteMap does not display any pages that are inaccessible. In the example, you've got all the menus controlled by the "loggedIn" If() clause and that's blocking access to menu and thus the menu is not displayed.