Component parent null after replace
I am trying to create a link that will replace one panel with another on the page. This seems as though it should be an easy task but after many attempts and searching the net for examples I have yet to get it working so the replacement can happen more than one without the component becoming orphaned. On the second replace attempt i get the following exception. Anyone have the solution? Thank you, Anthony java.lang.IllegalStateException: This method can only be called on a component that has already been added to its parent. at org.apache.wicket.Component.replaceWith(Component.java:2266) at com.pinwise.pinbase.web.components.links.SecurePanelLink $4.onClick(SecurePanelLink.java:142) at org.apache.wicket.markup.html.link.Link.onLinkClicked (Link.java:222) at java.lang.reflect.Method.invoke(Method.java:585) at org.apache.wicket.RequestListenerInterface.invoke (RequestListenerInterface.java:186) = = public UserPanel(String id) { super(id); UserFormPanel userFormPanel = new UserFormPanel("userFormPanel"); add( userFormPanel ); List columns = new ArrayList(); final Model m = new Model(userFormPanel); columns.add(new PropertyColumn(new Model("Actions"), "id") { public void populateItem(Item cellItem, String componentId, final IModel model) { EditActionPanel panel = new EditActionPanel(componentId, model); panel.add( SecurePanelLink.createSecurePanelLink( "edit", m, UserFormPanel.class, model ) ); cellItem.add( panel ); } }); columns.add(new PropertyColumn(new Model("Username"), "username", "username")); columns.add(new PropertyColumn(new Model("First Name"), "firstName", "firstName")); columns.add(new PropertyColumn(new Model("Last Name"), "lastName", "lastName")); add(new DefaultDataTable("table", columns, new SortableUserDataProvider(), 10)); } SecurePanelLink public class SecurePanelLink extends Link implements ISecureComponent { /** * */ private static final long serialVersionUID = 1L; protected IPanelLink panelLink; /** * @param id * @param c */ public SecurePanelLink(String id, final Class c) { super(id); // Ensure that c is a subclass of Panel if (!Panel.class.isAssignableFrom(c)) { throw new IllegalArgumentException("Class " + c + " is not a subclass of Panel"); } this.panelLink = createIPanelLink( id, c ); setSecurityCheck(new LinkSecurityCheck(this, c)); } public SecurePanelLink(String id, final Class c, Model existingPanel) { super(id); // Ensure that c is a subclass of Panel if (!Panel.class.isAssignableFrom(c)) { throw new IllegalArgumentException("Class " + c + " is not a subclass of Panel"); } this.panelLink = createIPanelLink( id, c ); setSecurityCheck(new LinkSecurityCheck(this, c)); } private IPanelLink createIPanelLink( String id, final Class c ) { return new IPanelLink() { private static final long serialVersionUID = 1L; public Panel getPanel( String id ) { // Create panel using panel factory return getPanelFactory().newPanel(id, c ); } public Panel getPanel( String id, IModel model) { // Create panel using panel factory return getPanelFactory().newPanel(id, c, model); } public Panel getPanel( Panel panel, IModel model) { // Create panel using panel factory return getPanelFactory().newPanel(panel.getId(), c, model); } public Class getPanelIdentity() { return c; } }; } /** * */ public static SecurePanelLink createSecurePanelLink( String id, Class clazz ) { return new SecurePanelLink( id, clazz ) { /**
Re: Component parent null after replace
efaultDataTable("table", columns, new >> SortableUserDataProvider(), 10)); >> } >> >> >> >> SecurePanelLink >> >> public class SecurePanelLink extends Link implements ISecureComponent { >> >> /** >> * >> */ >> private static final long serialVersionUID = 1L; >> >> protected IPanelLink panelLink; >> >> /** >> * @param id >> * @param c >> */ >> public SecurePanelLink(String id, final Class c) >> { >> super(id); >> // Ensure that c is a subclass of Panel >> if (!Panel.class.isAssignableFrom(c)) >> { >> throw new IllegalArgumentException("Class " + c + >> " is not a >> subclass of Panel"); >> } >> >> this.panelLink = createIPanelLink( id, c ); >> setSecurityCheck(new LinkSecurityCheck(this, c)); >> } >> >> public SecurePanelLink(String id, final Class c, Model >> existingPanel) >> { >> super(id); >> // Ensure that c is a subclass of Panel >> if (!Panel.class.isAssignableFrom(c)) >> { >> throw new IllegalArgumentException("Class " + c + >> " is not a >> subclass of Panel"); >> } >> >> this.panelLink = createIPanelLink( id, c ); >> setSecurityCheck(new LinkSecurityCheck(this, c)); >> } >> >> private IPanelLink createIPanelLink( String id, final Class c ) { >> return new IPanelLink() >> { >> private static final long serialVersionUID = 1L; >> >> public Panel getPanel( String id ) >> { >> // Create panel using panel factory >> return getPanelFactory().newPanel(id, c >> ); >> } >> >> public Panel getPanel( String id, IModel model) >> { >> // Create panel using panel factory >> return getPanelFactory().newPanel(id, c, >> model); >> } >> >> public Panel getPanel( Panel panel, IModel model) >> { >> // Create panel using panel factory >> return getPanelFactory().newPanel( >> panel.getId(), c, model); >> } >> >> public Class getPanelIdentity() >> { >> return c; >> } >> }; >> } >> >> /** >> * >> */ >> public static SecurePanelLink createSecurePanelLink( String id, >> Class clazz ) { >> return new SecurePanelLink( id, clazz ) { >> /** >> * >> */ >> private static final long serialVersionUID = 1L; >> >> public void onClick() { >> Panel panel = ((Panel)findParent( >> Panel.class)); >> panel.replaceWith( panelLink.getPanel( >> panel.getId() ) ); >> } >> }; >> } >> >> /** >> * >> */ >> public static SecurePanelLink createSecurePanelLink( String id, >> Class newPanel, final IModel model ) { >> return new SecurePanelLink( id, newPanel ) { >> /** >> * >> */ >> private static final long serialVersionUID = 1L; >> >> public void onClick() { >> Panel panel = ((Panel)findParent( >> Panel.class)); >> panel.replaceWith( panelLink.getPanel( >> panel.getId(), model ) ); >> } >> }; >> } >> >> /** >> * >> */ >> public static SecurePanelLink createSecurePanelLink( String id, >> final Model existingPanel, Class newPanel, final IModel model ) { >> return new SecurePanelLink( id, newPanel, existingPanel ) >> { >> /** >> * >> */ >> private static final long serialVersionUID = 1L; >> >> >> public void onClick() { >> Panel p = >> (Panel)existingPanel.getObject(); >> Panel panel = panelLink.getPanel >> ((Panel)existingPanel.getObject >> (), model ); >> panel.setOutputMarkupId(true); >> p.getParent().replace( panel ); >> p=panel; >> } >> }; >> } >> > > -- View this message in context: http://www.nabble.com/Component-parent-null-after-replace-tp12576967p26021314.html Sent from the Wicket - User mailing list archive at Nabble.com. - To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org For additional commands, e-mail: users-h...@wicket.apache.org
Re: Component parent null after replace
gt; >> private static final long serialVersionUID = 1L; > >> > >> public void onClick() { > >> Panel panel = ((Panel)findParent( > >> Panel.class)); > >> panel.replaceWith( panelLink.getPanel( > >> panel.getId() ) ); > >> } > >> }; > >> } > >> > >> /** > >> * > >> */ > >> public static SecurePanelLink createSecurePanelLink( String id, > >> Class newPanel, final IModel model ) { > >> return new SecurePanelLink( id, newPanel ) { > >> /** > >> * > >> */ > >> private static final long serialVersionUID = 1L; > >> > >> public void onClick() { > >> Panel panel = ((Panel)findParent( > >> Panel.class)); > >> panel.replaceWith( panelLink.getPanel( > >> panel.getId(), model ) ); > >> } > >> }; > >> } > >> > >> /** > >> * > >> */ > >> public static SecurePanelLink createSecurePanelLink( String id, > >> final Model existingPanel, Class newPanel, final IModel model ) { > >> return new SecurePanelLink( id, newPanel, existingPanel > ) > >> { > >> /** > >> * > >> */ > >> private static final long serialVersionUID = 1L; > >> > >> > >> public void onClick() { > >> Panel p = > >> (Panel)existingPanel.getObject(); > >> Panel panel = panelLink.getPanel > >> ((Panel)existingPanel.getObject > >> (), model ); > >> panel.setOutputMarkupId(true); > >> p.getParent().replace( panel ); > >> p=panel; > >> } > >> }; > >> } > >> > > > > > > -- > View this message in context: > http://www.nabble.com/Component-parent-null-after-replace-tp12576967p26021314.html > Sent from the Wicket - User mailing list archive at Nabble.com. > > > - > To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org > For additional commands, e-mail: users-h...@wicket.apache.org > > -- Pedro Henrique Oliveira dos Santos
Re: Component parent null after replace
heh, seems a lot of people run into it. the short is that you have to do this: lets say you have a ref to the panel: private Panel panel; what you do is this: panel.replaceWith(new Panel()); and then later again panel.replaceWith(new Panel()); ^ the second time will fail beause you have removed panel from hierarchy already - you replaced it with another panel. the proper way to do it is to keep the reference up to date, so Panel temp=new Panel(); panel.replaceWith(temp); panel=temp; that way when it happens the second time it will properly act on the new panel that is inside the hierarchy now. -igor On 9/9/07, Anthony Schexnaildre <[EMAIL PROTECTED]> wrote: > > I am trying to create a link that will replace one panel with another > on the page. This seems as though it should be an easy task but after > many attempts and searching the net for examples I have yet to get it > working so the replacement can happen more than one without the > component becoming orphaned. On the second replace attempt i get the > following exception. Anyone have the solution? > > Thank you, > > Anthony > > > java.lang.IllegalStateException: This method can only be called on a > component that has already been added to its parent. > at org.apache.wicket.Component.replaceWith(Component.java:2266) > at com.pinwise.pinbase.web.components.links.SecurePanelLink > $4.onClick(SecurePanelLink.java:142) > at org.apache.wicket.markup.html.link.Link.onLinkClicked > (Link.java:222) > at java.lang.reflect.Method.invoke(Method.java:585) > at org.apache.wicket.RequestListenerInterface.invoke > (RequestListenerInterface.java:186) > > > = > > = > > public UserPanel(String id) { > super(id); > > > UserFormPanel userFormPanel = new > UserFormPanel("userFormPanel"); > add( userFormPanel ); > > List columns = new ArrayList(); > final Model m = new Model(userFormPanel); > columns.add(new PropertyColumn(new Model("Actions"), "id") > { > public void populateItem(Item cellItem, String > componentId, final > IModel model) > { > EditActionPanel panel = new > EditActionPanel(componentId, model); > panel.add( > SecurePanelLink.createSecurePanelLink( "edit", m, > UserFormPanel.class, model ) ); >cellItem.add( panel ); > } > }); > > columns.add(new PropertyColumn(new Model("Username"), > "username", "username")); > columns.add(new PropertyColumn(new Model("First Name"), > "firstName", "firstName")); > columns.add(new PropertyColumn(new Model("Last Name"), > "lastName", "lastName")); > > add(new DefaultDataTable("table", columns, new > SortableUserDataProvider(), 10)); > } > > > > SecurePanelLink > > public class SecurePanelLink extends Link implements ISecureComponent { > > /** > * > */ > private static final long serialVersionUID = 1L; > > protected IPanelLink panelLink; > > /** > * @param id > * @param c > */ > public SecurePanelLink(String id, final Class c) > { > super(id); > // Ensure that c is a subclass of Panel > if (!Panel.class.isAssignableFrom(c)) > { > throw new IllegalArgumentException("Class " + c + > " is not a > subclass of Panel"); > } > > this.panelLink = createIPanelLink( id, c ); > setSecurityCheck(new LinkSecurityCheck(this, c)); > } > > public SecurePanelLink(String id, final Class c, Model > existingPanel) > { > super(id); > // Ensure that c is a subclass of Panel > if (!Panel.class.isAssignableFrom(c)) > { > throw new IllegalArgumentException("Class " + c + > " is not a > subclass of Panel"); > } > > this.panelLink = createIPanelLink( id, c ); > setSecurityCheck(new LinkSecurityCheck(this, c)); > } > > private IPanelLink createIPanelLink( String id, final Class c ) { > return new IPanelLink() > { > private static final long serialVersionUID = 1L; > > public Panel getPanel( String id ) > { > // Create panel using panel factory > return getPanelFactory().newPanel(id, c ); > } > > public Panel getPanel( String id, IModel model) > { >
Re: Component parent null after replace
Allow me to hijack this topic because my eye sees the magic word ISecureComponent :D Glad to come across another user of Wasp and Swarm. Any comments / questions about them? Maurice On 9/9/07, Igor Vaynberg <[EMAIL PROTECTED]> wrote: > heh, seems a lot of people run into it. the short is that you have to do > this: > > lets say you have a ref to the panel: private Panel panel; > > what you do is this: > panel.replaceWith(new Panel()); > > and then later again > panel.replaceWith(new Panel()); > > ^ the second time will fail beause you have removed panel from hierarchy > already - you replaced it with another panel. > > the proper way to do it is to keep the reference up to date, so > Panel temp=new Panel(); > panel.replaceWith(temp); > panel=temp; > > that way when it happens the second time it will properly act on the new > panel that is inside the hierarchy now. > > -igor > > > > > On 9/9/07, Anthony Schexnaildre <[EMAIL PROTECTED]> wrote: > > > > I am trying to create a link that will replace one panel with another > > on the page. This seems as though it should be an easy task but after > > many attempts and searching the net for examples I have yet to get it > > working so the replacement can happen more than one without the > > component becoming orphaned. On the second replace attempt i get the > > following exception. Anyone have the solution? > > > > Thank you, > > > > Anthony > > > > > > java.lang.IllegalStateException: This method can only be called on a > > component that has already been added to its parent. > > at org.apache.wicket.Component.replaceWith(Component.java:2266) > > at com.pinwise.pinbase.web.components.links.SecurePanelLink > > $4.onClick(SecurePanelLink.java:142) > > at org.apache.wicket.markup.html.link.Link.onLinkClicked > > (Link.java:222) > > at java.lang.reflect.Method.invoke(Method.java:585) > > at org.apache.wicket.RequestListenerInterface.invoke > > (RequestListenerInterface.java:186) > > > > > > = > > > > = > > > > public UserPanel(String id) { > > super(id); > > > > > > UserFormPanel userFormPanel = new > > UserFormPanel("userFormPanel"); > > add( userFormPanel ); > > > > List columns = new ArrayList(); > > final Model m = new Model(userFormPanel); > > columns.add(new PropertyColumn(new Model("Actions"), "id") > > { > > public void populateItem(Item cellItem, String > > componentId, final > > IModel model) > > { > > EditActionPanel panel = new > > EditActionPanel(componentId, model); > > panel.add( > > SecurePanelLink.createSecurePanelLink( "edit", m, > > UserFormPanel.class, model ) ); > >cellItem.add( panel ); > > } > > }); > > > > columns.add(new PropertyColumn(new Model("Username"), > > "username", "username")); > > columns.add(new PropertyColumn(new Model("First Name"), > > "firstName", "firstName")); > > columns.add(new PropertyColumn(new Model("Last Name"), > > "lastName", "lastName")); > > > > add(new DefaultDataTable("table", columns, new > > SortableUserDataProvider(), 10)); > > } > > > > > > > > SecurePanelLink > > > > public class SecurePanelLink extends Link implements ISecureComponent { > > > > /** > > * > > */ > > private static final long serialVersionUID = 1L; > > > > protected IPanelLink panelLink; > > > > /** > > * @param id > > * @param c > > */ > > public SecurePanelLink(String id, final Class c) > > { > > super(id); > > // Ensure that c is a subclass of Panel > > if (!Panel.class.isAssignableFrom(c)) > > { > > throw new IllegalArgumentException("Class " + c + > > " is not a > > subclass of Panel"); > > } > > > > this.panelLink = createIPanelLink( id, c ); > > setSecurityCheck(new LinkSecurityCheck(this, c)); > > } > > > > public SecurePanelLink(String id, final Class c, Model > > existingPanel) > > { > > super(id); > > // Ensure that c is a subclass of Panel > > if (!Panel.class.isAssignableFrom(c)) > > { > > throw new IllegalArgumentException("Class " + c + > > " is not a > > subclass of Panel"); > > } > > > > this.panelLink = createIPanelLink( id, c ); > > setSecurityCheck(new LinkSecurityCheck(this, c)); > > } > > > > private IPanelLink createIPanelLink( String id, final Class c ) { > >
Re: Component parent null after replace
Igor, Thank you for your reply. I did see another message from you recently on this same topic but the key the problem was having the "private Panel panel;" field to store the reference. -Anthony On Sep 9, 2007, at 5:09 PM, Igor Vaynberg wrote: heh, seems a lot of people run into it. the short is that you have to do this: lets say you have a ref to the panel: private Panel panel; what you do is this: panel.replaceWith(new Panel()); and then later again panel.replaceWith(new Panel()); ^ the second time will fail beause you have removed panel from hierarchy already - you replaced it with another panel. the proper way to do it is to keep the reference up to date, so Panel temp=new Panel(); panel.replaceWith(temp); panel=temp; that way when it happens the second time it will properly act on the new panel that is inside the hierarchy now. -igor On 9/9/07, Anthony Schexnaildre <[EMAIL PROTECTED]> wrote: I am trying to create a link that will replace one panel with another on the page. This seems as though it should be an easy task but after many attempts and searching the net for examples I have yet to get it working so the replacement can happen more than one without the component becoming orphaned. On the second replace attempt i get the following exception. Anyone have the solution? Thank you, Anthony java.lang.IllegalStateException: This method can only be called on a component that has already been added to its parent. at org.apache.wicket.Component.replaceWith(Component.java:2266) at com.pinwise.pinbase.web.components.links.SecurePanelLink $4.onClick(SecurePanelLink.java:142) at org.apache.wicket.markup.html.link.Link.onLinkClicked (Link.java:222) at java.lang.reflect.Method.invoke(Method.java:585) at org.apache.wicket.RequestListenerInterface.invoke (RequestListenerInterface.java:186) = = public UserPanel(String id) { super(id); UserFormPanel userFormPanel = new UserFormPanel("userFormPanel"); add( userFormPanel ); List columns = new ArrayList(); final Model m = new Model(userFormPanel); columns.add(new PropertyColumn(new Model("Actions"), "id") { public void populateItem(Item cellItem, String componentId, final IModel model) { EditActionPanel panel = new EditActionPanel(componentId, model); panel.add( SecurePanelLink.createSecurePanelLink( "edit", m, UserFormPanel.class, model ) ); cellItem.add( panel ); } }); columns.add(new PropertyColumn(new Model("Username"), "username", "username")); columns.add(new PropertyColumn(new Model("First Name"), "firstName", "firstName")); columns.add(new PropertyColumn(new Model("Last Name"), "lastName", "lastName")); add(new DefaultDataTable("table", columns, new SortableUserDataProvider(), 10)); } SecurePanelLink public class SecurePanelLink extends Link implements ISecureComponent { /** * */ private static final long serialVersionUID = 1L; protected IPanelLink panelLink; /** * @param id * @param c */ public SecurePanelLink(String id, final Class c) { super(id); // Ensure that c is a subclass of Panel if (!Panel.class.isAssignableFrom(c)) { throw new IllegalArgumentException("Class " + c + " is not a subclass of Panel"); } this.panelLink = createIPanelLink( id, c ); setSecurityCheck(new LinkSecurityCheck(this, c)); } public SecurePanelLink(String id, final Class c, Model existingPanel) { super(id); // Ensure that c is a subclass of Panel if (!Panel.class.isAssignableFrom(c)) { throw new IllegalArgumentException("Class " + c + " is not a subclass of Panel"); } this.panelLink = createIPanelLink( id, c ); setSecurityCheck(new LinkSecurityCheck(this, c)); } private IPanelLink createIPanelLink( String id, final Class c ) { return new IPanelLink() { private static final long serialVersionUID = 1L; public Panel getPanel( String id ) { // Create panel using panel factory return getPanelFactory().newPanel (id, c ); } public Panel getPanel(
Wasp/Swarm Questions was Re: Component parent null after replace
Please hijack away. I have been quite happy with Wasp and Swarm. It's simple and painless but quite flexible. The examples are the life saver though. To be honest, the security framework is why I chose to use Wicket for this project instead of Tapestry. I am however having a few issues figuring out some "best practices" that maybe you or some other experienced Wasp/Swarm users may be able to shed some light on. 1. Where would you store a reference to the logged in user. Currently have it in a DetachableModel in a custom session. There has to be a better place to put the user reference that fits more directly with the security framework. 2. I very much like your SecureTab example but something like a SecurePanelLink within the framework would be helpful. I have been working on my own that mirrors the SecurePageLink as much as possible but it is still a bit wonky. This is helpful because once you get into a tab layout, navigation to other panels that are not directly attached to a tab becomes annoying at best. I am working through it but seems like someone must have solved this issues a few times over in an elegant fashion. Maybe some kind of "panel stack" with tab integration? 3. I don't understand the DataPermissions, configuration and their use cases. I must admit I have not spent the time to dig through the example extensively but I don't understand the use cases enough to know why I would want to spend the time. 4. I am working on an application that will eventually have very complicated permission structure. Right now I am fumbling a solution together for phase one. Eventually I would like to be able to "filter" the date returned to a user based on some permissions while not coupling the security layer to the Dao's to tightly. A typical structure would be as follows: O - Organisation U - User O / / \ \ / | | \ U U U O / | \ U O O / || \ U U U U Consider a use case where you have a hierarchy of Financial Transaction Processors with the following 2 user types/principals: Customer Service - Allowed to see transactions and act on them if the transactions are for their direct organisation Billing - Allowed to see transaction and act on them for their direct organisation and any sub-organisations The final goal would be to have an interface much like that in the wicket-phonebook example where the DataProvider ( or something else ) would feed the appropriate results out of the hibernate dao's into the DataTable based on the users permissions. Any clues to some elegant approach? -Anthony On Sep 9, 2007, at 6:12 PM, Maurice Marrink wrote: Allow me to hijack this topic because my eye sees the magic word ISecureComponent :D Glad to come across another user of Wasp and Swarm. Any comments / questions about them? Maurice On 9/9/07, Igor Vaynberg <[EMAIL PROTECTED]> wrote: heh, seems a lot of people run into it. the short is that you have to do this: lets say you have a ref to the panel: private Panel panel; what you do is this: panel.replaceWith(new Panel()); and then later again panel.replaceWith(new Panel()); ^ the second time will fail beause you have removed panel from hierarchy already - you replaced it with another panel. the proper way to do it is to keep the reference up to date, so Panel temp=new Panel(); panel.replaceWith(temp); panel=temp; that way when it happens the second time it will properly act on the new panel that is inside the hierarchy now. -igor On 9/9/07, Anthony Schexnaildre <[EMAIL PROTECTED]> wrote: I am trying to create a link that will replace one panel with another on the page. This seems as though it should be an easy task but after many attempts and searching the net for examples I have yet to get it working so the replacement can happen more than one without the component becoming orphaned. On the second replace attempt i get the following exception. Anyone have the solution? Thank you, Anthony java.lang.IllegalStateException: This method can only be called on a component that has already been added to its parent. at org.apache.wicket.Component.replaceWith(Component.java: 2266) at com.pinwise.pinbase.web.components.links.SecurePanelLink $4.onClick(SecurePanelLink.java:142) at org.apache.wicket.markup.html.link.Link.onLinkClicked (Link.java:222) at java.lang.reflect.Method.invoke(Method.java:585) at org.apache.wicket.RequestListenerInterface.invoke (RequestListenerInterface.java:186) = = public UserPanel(String id) { super(id); UserFormPanel userFormPanel = new UserFormPanel("userFormPanel"); add( userFormPanel ); List columns = new ArrayList(); final Model m = new Model(userFormPanel)
Re: Wasp/Swarm Questions was Re: Component parent null after replace
On 9/11/07, Anthony Schexnaildre <[EMAIL PROTECTED]> wrote: > Please hijack away. > > I have been quite happy with Wasp and Swarm. It's simple and painless > but quite flexible. The examples are the life saver though. To be > honest, the security framework is why I chose to use Wicket for this > project instead of Tapestry. > > I am however having a few issues figuring out some "best practices" > that maybe you or some other experienced Wasp/Swarm users may be able > to shed some light on. > > 1. Where would you store a reference to the logged in user. Currently > have it in a DetachableModel in a custom session. There has to be a > better place to put the user reference that fits more directly with > the security framework. A detachable model in a custom session is as good a place as any, in fact i keep it there too. Swarm indeed does not keep a ref to your user object but it does keep a reference to the subject (which translates into the user rights, but also could be the user itself). Although you can get this subject by casting the strategy to a SwarmStrategy, i recommend storing the user yourself like you do now, which is perfectly compatible with the wicket philosophy that if you wish to store things in the session you should create a custom session and provide strongtyped getters / setters for it. > 2. I very much like your SecureTab example but something like a > SecurePanelLink within the framework would be helpful. The ContainerSecurityCheck was created for helping out with panel replacements by treating panels exactly like pages but if you feel we need to provide more stuff out of the box here feel free to speak up. After thinking about this for a bit i think i can come up with a secure link that automatically replaces one panel with another. >I have been > working on my own that mirrors the SecurePageLink as much as possible > but it is still a bit wonky. I saw it, and i must admit my first thought was: "couldn't he use ..., or do " but after looking deeper in what you were trying to accomplish, i think you did alright. Although looking back at my previous comment about making a panel link i think a combination of LinkSecurityCheck and ContainerSecurityCheck would be ideal. >This is helpful because once you get > into a tab layout, navigation to other panels that are not directly > attached to a tab becomes annoying at best. I am working through it > but seems like someone must have solved this issues a few times over > in an elegant fashion. Maybe some kind of "panel stack" with tab > integration? I am not sure i get what it is you are trying to accomplish here. It should not matter where the panel is located. if you use a regular ComponentSecurityCheck the full path with the parent is considered but if you use a ContainerSecurityCheck the panel class is used and where ever it is placed in the page hierarchy is irrelevant. Personally i use Swarm in a page based approach, but panels should be just as easy. > > 3. I don't understand the DataPermissions, configuration and their > use cases. I must admit I have not spent the time to dig through the > example extensively but I don't understand the use cases enough to > know why I would want to spend the time. The DataPermissions are indeed a bit under lighted in the examples. The beauty about them is that you can build entire applications without ever needing them. They are intended for use cases where it makes more sense to let the data / model dictate if a user has permissions for them. One benefit of this is that you can share the secure model (which is typically used for data permissions) between multiple components and instead of securing each component the security is on 1 central place namely the model. An example of this is the SwarmCompoundPropertyModel. > > 4. I am working on an application that will eventually have very > complicated permission structure. Right now I am fumbling a solution > together for phase one. Eventually I would like to be able to > "filter" the date returned to a user based on some permissions while > not coupling the security layer to the Dao's to tightly. We have a similar use case in our organization and although you could use datapermissions to hide components showing restricted data (completely disconnecting your dao's from your security layer), consider the following: by letting you security layer provide you with some sort of filter for your dao (perhaps a list of allowed id's or a query part) you bypass the need to secure your components as the data is already made to fit what is allowed to see. In our application we chose the latter approach. >A typical > structure would be as follows: > > O - Organisation > U - User > > O > / / \ \ > / | | \ >U U U O > / | \ > U O O > / || \ > U U U U > > Consider a use case where you have a hierarchy of Financial > Transaction Processors with the following 2 user type
Re: Wasp/Swarm Questions was Re: Component parent null after replace
Just a quick note: storing objects that are not thread safe in your session is asking for trouble. While Wicket does limit page processing to one request at a time, other requests like resources can run in parallel. What does this mean? One thing that comes to mind is that when two requests for the same session are being processed, and one is done before the other it will detach the user model. What are the semantics now for the other thread? For instance if you have a detachable model storing a User object in your session and use Hibernate you are in a world of hurt, or rather Hibernate will sometimes bork because your Session tries to attach the single User instance to multiple Hibernate Session objects. Exceptions will be having a party. Now this is not meant as a Hibernate bashing reply, it just happens that Hibernate correctly detects multiple threads modifying the same object's state and stops tampering with it. Martijn -- Buy Wicket in Action: http://manning.com/dashorst Apache Wicket 1.3.0-beta3 is released Get it now: http://www.apache.org/dyn/closer.cgi/wicket/1.3.0-beta3/ - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: Wasp/Swarm Questions was Re: Component parent null after replace
Martijn, you are absolutely right, i forgot we moved the user from the session to the requestcycle. Just keep the id for your user in the session and keep the actual user for this request in the requestcycle. This way each thread will have its own instance of the user. Maurice On 9/11/07, Martijn Dashorst <[EMAIL PROTECTED]> wrote: > Just a quick note: storing objects that are not thread safe in your > session is asking for trouble. While Wicket does limit page processing > to one request at a time, other requests like resources can run in > parallel. What does this mean? > > One thing that comes to mind is that when two requests for the same > session are being processed, and one is done before the other it will > detach the user model. What are the semantics now for the other > thread? > > For instance if you have a detachable model storing a User object in > your session and use Hibernate you are in a world of hurt, or rather > Hibernate will sometimes bork because your Session tries to attach the > single User instance to multiple Hibernate Session objects. Exceptions > will be having a party. > > Now this is not meant as a Hibernate bashing reply, it just happens > that Hibernate correctly detects multiple threads modifying the same > object's state and stops tampering with it. > > Martijn > > -- > Buy Wicket in Action: http://manning.com/dashorst > Apache Wicket 1.3.0-beta3 is released > Get it now: http://www.apache.org/dyn/closer.cgi/wicket/1.3.0-beta3/ > > - > 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]
Re: Wasp/Swarm Questions was Re: Component parent null after replace
This makes sense. Where would you stick the user on the requestcycle? It's not obvious from the javadocs. Is there a "wicket way"? -Anthony On Sep 11, 2007, at 10:05 AM, Maurice Marrink wrote: Martijn, you are absolutely right, i forgot we moved the user from the session to the requestcycle. Just keep the id for your user in the session and keep the actual user for this request in the requestcycle. This way each thread will have its own instance of the user. Maurice On 9/11/07, Martijn Dashorst <[EMAIL PROTECTED]> wrote: Just a quick note: storing objects that are not thread safe in your session is asking for trouble. While Wicket does limit page processing to one request at a time, other requests like resources can run in parallel. What does this mean? One thing that comes to mind is that when two requests for the same session are being processed, and one is done before the other it will detach the user model. What are the semantics now for the other thread? For instance if you have a detachable model storing a User object in your session and use Hibernate you are in a world of hurt, or rather Hibernate will sometimes bork because your Session tries to attach the single User instance to multiple Hibernate Session objects. Exceptions will be having a party. Now this is not meant as a Hibernate bashing reply, it just happens that Hibernate correctly detects multiple threads modifying the same object's state and stops tampering with it. Martijn -- Buy Wicket in Action: http://manning.com/dashorst Apache Wicket 1.3.0-beta3 is released Get it now: http://www.apache.org/dyn/closer.cgi/wicket/1.3.0-beta3/ - 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] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: Wasp/Swarm Questions was Re: Component parent null after replace
Create your custom request cycle, and add a getter that uses the session's username/id to retrieve the user from the database, and cache it locally. Martijn On 9/11/07, Anthony Schexnaildre <[EMAIL PROTECTED]> wrote: > This makes sense. Where would you stick the user on the requestcycle? > It's not obvious from the javadocs. Is there a "wicket way"? > > -Anthony > > On Sep 11, 2007, at 10:05 AM, Maurice Marrink wrote: > > > Martijn, you are absolutely right, i forgot we moved the user from the > > session to the requestcycle. Just keep the id for your user in the > > session and keep the actual user for this request in the requestcycle. > > This way each thread will have its own instance of the user. > > > > Maurice > > > > On 9/11/07, Martijn Dashorst <[EMAIL PROTECTED]> wrote: > >> Just a quick note: storing objects that are not thread safe in your > >> session is asking for trouble. While Wicket does limit page > >> processing > >> to one request at a time, other requests like resources can run in > >> parallel. What does this mean? > >> > >> One thing that comes to mind is that when two requests for the same > >> session are being processed, and one is done before the other it will > >> detach the user model. What are the semantics now for the other > >> thread? > >> > >> For instance if you have a detachable model storing a User object in > >> your session and use Hibernate you are in a world of hurt, or rather > >> Hibernate will sometimes bork because your Session tries to attach > >> the > >> single User instance to multiple Hibernate Session objects. > >> Exceptions > >> will be having a party. > >> > >> Now this is not meant as a Hibernate bashing reply, it just happens > >> that Hibernate correctly detects multiple threads modifying the same > >> object's state and stops tampering with it. > >> > >> Martijn > >> > >> -- > >> Buy Wicket in Action: http://manning.com/dashorst > >> Apache Wicket 1.3.0-beta3 is released > >> Get it now: http://www.apache.org/dyn/closer.cgi/wicket/1.3.0-beta3/ > >> > >> - > >> 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] > > > > > - > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > -- Buy Wicket in Action: http://manning.com/dashorst Apache Wicket 1.3.0-beta3 is released Get it now: http://www.apache.org/dyn/closer.cgi/wicket/1.3.0-beta3/ - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: Wasp/Swarm Questions was Re: Component parent null after replace
Ok, i just finished a SecureContainerLink that should do what your SecurePanelLink does, but it is a bit less complex. I haven't checked it in yet or tested it for that matter but hope to hear from you if this is what you meant. If so i will make it a part of wasp. Maurice /* * Licensed to the Apache Software Foundation (ASF) under one or more * contributor license agreements. See the NOTICE file distributed with * this work for additional information regarding copyright ownership. * The ASF licenses this file to You under the Apache License, Version 2.0 * (the "License"); you may not use this file except in compliance with * the License. You may obtain a copy of the License at * * http://www.apache.org/licenses/LICENSE-2.0 * * Unless required by applicable law or agreed to in writing, software * distributed under the License is distributed on an "AS IS" BASIS, * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. * See the License for the specific language governing permissions and * limitations under the License. */ package org.apache.wicket.security.components.markup.html.links; import org.apache.wicket.Component; import org.apache.wicket.MarkupContainer; import org.apache.wicket.WicketRuntimeException; import org.apache.wicket.markup.html.link.Link; import org.apache.wicket.model.IModel; import org.apache.wicket.security.actions.AbstractWaspAction; import org.apache.wicket.security.checks.ISecurityCheck; import org.apache.wicket.security.checks.LinkSecurityCheck; import org.apache.wicket.security.components.ISecureComponent; import org.apache.wicket.security.components.SecureComponentHelper; /** * A secure link to handle panel replacements or any other type of * [EMAIL PROTECTED] MarkupContainer}s. It is also usable as a link to switch between 2 or * more panels. Security is enforced on the replacing class. * * @author marrink */ public abstract class SecureContainerLink extends Link implements ISecureComponent { /** * */ private static final long serialVersionUID = 1L; private Class replacementClass; private MarkupContainer containerParent; private String containerId; /** * Constructs a new replacement link. * * @param id *id of the link * @param replacementPanel *the class of the container replacing the component on the *supplied parent * @param parentOfReplaceablePanel *the parent component where the replacement needs to take place * @param panelId *the id of the component to be replaced */ public SecureContainerLink(String id, Class replacementPanel, MarkupContainer parentOfReplaceablePanel, String panelId) { this(id, null, replacementPanel, parentOfReplaceablePanel, panelId); } /** * Constructs a new replacement link. * * @param id *id of the link * @param object *model of the link * @param replacementPanel *the class of the container replacing the component on the *supplied parent * @param parentOfReplaceablePanel *the parent component where the replacement needs to take place * @param panelId *the id of the component to be replaced */ public SecureContainerLink(String id, IModel object, Class replacementPanel, MarkupContainer parentOfReplaceablePanel, String panelId) { super(id, object); setReplacementClass(replacementPanel); if (parentOfReplaceablePanel == null) throw new WicketRuntimeException("Parent required for replacing components."); containerParent = parentOfReplaceablePanel; if (panelId == null) throw new WicketRuntimeException("Id required from component to be replaced."); containerId = panelId; } /** * Performs the replacement, only if an actual replacement was constructed. * * @see org.apache.wicket.markup.html.link.Link#onClick() * @see #getReplacementFor(Component, String, Class) * @throws WicketRuntimeException * if a problem occurs in replacing the container. */ public final void onClick() { Component replaceMe = containerParent.get(containerId); if (replaceMe == null) throw new WicketRuntimeException("unable to find child with id: " + containerId + " on parent: " + containerParent); Class myReplacementClass = getReplaceme
Re: Wasp/Swarm Questions was Re: Component parent null after replace
At first glance this looks more impressive than my first attempt. I am just reorganizing my application by storing the User in the RequestCycle as was suggested earlier and storing the User id in the session. After that I will give this code a spin and let you know my thoughts. -Anthony On Sep 11, 2007, at 11:24 PM, Maurice Marrink wrote: Ok, i just finished a SecureContainerLink that should do what your SecurePanelLink does, but it is a bit less complex. I haven't checked it in yet or tested it for that matter but hope to hear from you if this is what you meant. If so i will make it a part of wasp. Maurice /* * Licensed to the Apache Software Foundation (ASF) under one or more * contributor license agreements. See the NOTICE file distributed with * this work for additional information regarding copyright ownership. * The ASF licenses this file to You under the Apache License, Version 2.0 * (the "License"); you may not use this file except in compliance with * the License. You may obtain a copy of the License at * * http://www.apache.org/licenses/LICENSE-2.0 * * Unless required by applicable law or agreed to in writing, software * distributed under the License is distributed on an "AS IS" BASIS, * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. * See the License for the specific language governing permissions and * limitations under the License. */ package org.apache.wicket.security.components.markup.html.links; import org.apache.wicket.Component; import org.apache.wicket.MarkupContainer; import org.apache.wicket.WicketRuntimeException; import org.apache.wicket.markup.html.link.Link; import org.apache.wicket.model.IModel; import org.apache.wicket.security.actions.AbstractWaspAction; import org.apache.wicket.security.checks.ISecurityCheck; import org.apache.wicket.security.checks.LinkSecurityCheck; import org.apache.wicket.security.components.ISecureComponent; import org.apache.wicket.security.components.SecureComponentHelper; /** * A secure link to handle panel replacements or any other type of * [EMAIL PROTECTED] MarkupContainer}s. It is also usable as a link to switch between 2 or * more panels. Security is enforced on the replacing class. * * @author marrink */ public abstract class SecureContainerLink extends Link implements ISecureComponent { /** * */ private static final long serialVersionUID = 1L; private Class replacementClass; private MarkupContainer containerParent; private String containerId; /** * Constructs a new replacement link. * * @param id *id of the link * @param replacementPanel *the class of the container replacing the component on the *supplied parent * @param parentOfReplaceablePanel *the parent component where the replacement needs to take place * @param panelId *the id of the component to be replaced */ public SecureContainerLink(String id, Class replacementPanel, MarkupContainer parentOfReplaceablePanel, String panelId) { this(id, null, replacementPanel, parentOfReplaceablePanel, panelId); } /** * Constructs a new replacement link. * * @param id *id of the link * @param object *model of the link * @param replacementPanel *the class of the container replacing the component on the *supplied parent * @param parentOfReplaceablePanel *the parent component where the replacement needs to take place * @param panelId *the id of the component to be replaced */ public SecureContainerLink(String id, IModel object, Class replacementPanel, MarkupContainer parentOfReplaceablePanel, String panelId) { super(id, object); setReplacementClass(replacementPanel); if (parentOfReplaceablePanel == null) throw new WicketRuntimeException("Parent required for replacing components."); containerParent = parentOfReplaceablePanel; if (panelId == null) throw new WicketRuntimeException("Id required from component to be replaced."); containerId = panelId; } /** * Performs the replacement, only if an actual replacement was constructed. * * @see org.apache.wicket.markup.html.link.Link#onClick() * @see #getReplacementFor(Component, String, Class) * @throws WicketRuntimeException * if a problem occurs in replacing the container. */ public final void onClick() {
Re: Wasp/Swarm Questions was Re: Component parent null after replace
Sitting down with the SecureContainerLink now. I may be slow but I am a little confused by it's intended use. I am not sure what the implementation of getReplacementFor(.) is meant to look like. -Anthony On Sep 11, 2007, at 11:24 PM, Maurice Marrink wrote: Ok, i just finished a SecureContainerLink that should do what your SecurePanelLink does, but it is a bit less complex. I haven't checked it in yet or tested it for that matter but hope to hear from you if this is what you meant. If so i will make it a part of wasp. Maurice /* * Licensed to the Apache Software Foundation (ASF) under one or more * contributor license agreements. See the NOTICE file distributed with * this work for additional information regarding copyright ownership. * The ASF licenses this file to You under the Apache License, Version 2.0 * (the "License"); you may not use this file except in compliance with * the License. You may obtain a copy of the License at * * http://www.apache.org/licenses/LICENSE-2.0 * * Unless required by applicable law or agreed to in writing, software * distributed under the License is distributed on an "AS IS" BASIS, * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. * See the License for the specific language governing permissions and * limitations under the License. */ package org.apache.wicket.security.components.markup.html.links; import org.apache.wicket.Component; import org.apache.wicket.MarkupContainer; import org.apache.wicket.WicketRuntimeException; import org.apache.wicket.markup.html.link.Link; import org.apache.wicket.model.IModel; import org.apache.wicket.security.actions.AbstractWaspAction; import org.apache.wicket.security.checks.ISecurityCheck; import org.apache.wicket.security.checks.LinkSecurityCheck; import org.apache.wicket.security.components.ISecureComponent; import org.apache.wicket.security.components.SecureComponentHelper; /** * A secure link to handle panel replacements or any other type of * [EMAIL PROTECTED] MarkupContainer}s. It is also usable as a link to switch between 2 or * more panels. Security is enforced on the replacing class. * * @author marrink */ public abstract class SecureContainerLink extends Link implements ISecureComponent { /** * */ private static final long serialVersionUID = 1L; private Class replacementClass; private MarkupContainer containerParent; private String containerId; /** * Constructs a new replacement link. * * @param id *id of the link * @param replacementPanel *the class of the container replacing the component on the *supplied parent * @param parentOfReplaceablePanel *the parent component where the replacement needs to take place * @param panelId *the id of the component to be replaced */ public SecureContainerLink(String id, Class replacementPanel, MarkupContainer parentOfReplaceablePanel, String panelId) { this(id, null, replacementPanel, parentOfReplaceablePanel, panelId); } /** * Constructs a new replacement link. * * @param id *id of the link * @param object *model of the link * @param replacementPanel *the class of the container replacing the component on the *supplied parent * @param parentOfReplaceablePanel *the parent component where the replacement needs to take place * @param panelId *the id of the component to be replaced */ public SecureContainerLink(String id, IModel object, Class replacementPanel, MarkupContainer parentOfReplaceablePanel, String panelId) { super(id, object); setReplacementClass(replacementPanel); if (parentOfReplaceablePanel == null) throw new WicketRuntimeException("Parent required for replacing components."); containerParent = parentOfReplaceablePanel; if (panelId == null) throw new WicketRuntimeException("Id required from component to be replaced."); containerId = panelId; } /** * Performs the replacement, only if an actual replacement was constructed. * * @see org.apache.wicket.markup.html.link.Link#onClick() * @see #getReplacementFor(Component, String, Class) * @throws WicketRuntimeException * if a problem occurs in replacing the container. */ public final void onClick() { Component replaceMe = containerParent.get(containerId); if (r
Re: Wasp/Swarm Questions was Re: Component parent null after replace
I realize i did not give any directions about how to use this class but i was hoping the javadoc would speak for itself. Anyway the getReplacementFor is a factory method for creating the panel that will replace the current panel. Looking at your original class i think it should do something like protected MarkupContainer getReplacementFor(Component current, String id, Class replacementClass) { return getPanelFactory().newPanel(id, replacementClass,current.getModel()); //not sure where getPanelFactory is accessible from, but you get the drift. } This will cause a onetime replacement of the panel, any next click will just replace the panel again with a new instance of the same class. If you do not want any further clicks on the link you could do a setVisible(false); to hide the link. Or if you want the link to alternate between these two panels (or any number really) you could do this protected MarkupContainer getReplacementFor(Component current, String id, Class replacementClass) { setReplacementClass(current.getClass()); return getPanelFactory().newPanel(id, replacementClass,current.getModel()); } Note that in this last example you need to grant permission for all the panelclasses you want to alternate between or the link will disappear. Also the link does not care about permissions for the currently showing panel, the panel itself should do that, but only about the panel you are going to replace it with. I hope this clarifies things enough. Keep me posted. Maurice On 9/12/07, Anthony Schexnaildre <[EMAIL PROTECTED]> wrote: > Sitting down with the SecureContainerLink now. I may be slow but I am > a little confused by it's intended use. I am not sure what the > implementation of getReplacementFor(.) is meant to look like. > > -Anthony > > On Sep 11, 2007, at 11:24 PM, Maurice Marrink wrote: > > > Ok, i just finished a SecureContainerLink that should do what your > > SecurePanelLink does, but it is a bit less complex. I haven't checked > > it in yet or tested it for that matter but hope to hear from you if > > this is what you meant. If so i will make it a part of wasp. > > > > Maurice > > > > /* > > * Licensed to the Apache Software Foundation (ASF) under one or more > > * contributor license agreements. See the NOTICE file distributed > > with > > * this work for additional information regarding copyright ownership. > > * The ASF licenses this file to You under the Apache License, > > Version 2.0 > > * (the "License"); you may not use this file except in compliance > > with > > * the License. You may obtain a copy of the License at > > * > > * http://www.apache.org/licenses/LICENSE-2.0 > > * > > * Unless required by applicable law or agreed to in writing, software > > * distributed under the License is distributed on an "AS IS" BASIS, > > * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or > > implied. > > * See the License for the specific language governing permissions and > > * limitations under the License. > > */ > > package org.apache.wicket.security.components.markup.html.links; > > > > import org.apache.wicket.Component; > > import org.apache.wicket.MarkupContainer; > > import org.apache.wicket.WicketRuntimeException; > > import org.apache.wicket.markup.html.link.Link; > > import org.apache.wicket.model.IModel; > > import org.apache.wicket.security.actions.AbstractWaspAction; > > import org.apache.wicket.security.checks.ISecurityCheck; > > import org.apache.wicket.security.checks.LinkSecurityCheck; > > import org.apache.wicket.security.components.ISecureComponent; > > import org.apache.wicket.security.components.SecureComponentHelper; > > > > /** > > * A secure link to handle panel replacements or any other type of > > * [EMAIL PROTECTED] MarkupContainer}s. It is also usable as a link to > > switch > > between 2 or > > * more panels. Security is enforced on the replacing class. > > * > > * @author marrink > > */ > > public abstract class SecureContainerLink extends Link implements > > ISecureComponent > > { > > /** > >* > >*/ > > private static final long serialVersionUID = 1L; > > > > private Class replacementClass; > > private MarkupContainer containerParent; > > private String containerId; > > > > /** > >* Constructs a new replacement link. > >* > >* @param id > >*id of the link > >* @param replacementPanel > >*the class of the container replacing the component > > on the > >*supplied parent > >* @param parentOfReplaceablePanel > >*the parent component where the replacement needs to > > take place > >* @param panelId > >*the id of the component to be replaced > >*/ > > public SecureContainerLink(String id, Class replacementPanel, > > MarkupContainer parentOfRepl