[#JBSEAM-2782] Wicket application should support conversation propagation for all built components

Description

The Seam Wicket integration is very interesting. But currently the conversation propagation is limited to form submit action and SeamLink component.

This limitation prevents to use it in a real world application where page navigation can be implemented with several different components: button, link, submit link, ajax events.

Among all it would be really useful that the conversation propagation would be supported by the AjaxEventBehavior component that implements Ajax interaction in Wicket and where it is expected to bring the most important benefits (no more detached entities and LazyInitializationException on loading proxyed objects).

Regards,
// Paolo Di Tommaso

Description

The Seam Wicket integration is very interesting. But currently the conversation propagation is limited to form submit action and SeamLink component. This limitation prevents to use it in a real world application where page navigation can be implemented with several different components: button, link, submit link, ajax events. Among all it would be really useful that the conversation propagation would be supported by the AjaxEventBehavior component that implements Ajax interaction in Wicket and where it is expected to bring the most important benefits (no more detached entities and LazyInitializationException on loading proxyed objects). Regards, // Paolo Di Tommaso

Show »

[ Permlink | « Hide ]

paolo di tommaso [02/Apr/08 11:47 AM]

I've found a nice trick that seams to fix teh above problem. Here it is the patch:

7d6
< import org.apache.wicket.behavior.IBehaviorListener;
9,10d7
< import org.apache.wicket.markup.html.form.IOnChangeListener;
< import org.apache.wicket.markup.html.link.ILinkListener;
75c72
< String name = requestTarget.getRequestListenerInterface().getName();
---
> if (IFormSubmitListener.INTERFACE.getName().equals(requestTarget.getRequestListenerInterface().getName()))
76,80d72
<
< if ( name.equals(IFormSubmitListener.INTERFACE.getName()) ||
< name.equals(ILinkListener.INTERFACE.getName()) ||
< name.equals(IBehaviorListener.INTERFACE.getName()) ||
< name.equals(IOnChangeListener.INTERFACE.getName()) )

[ Permlink | « Hide ]

Pete Muir [13/May/08 04:31 PM]

Could you post that patch in unified diff format, preferably using svn diff against trunk?

[ Permlink | « Hide ]

paolo di tommaso [14/May/08 07:28 AM]

Pete. I cannot manage the diff as you are requiring. I've attached the modified source.

Anyway my patch is really simple. I've just added all the ListenerInterface that should propagate the conversionId

:
if ( name.equals(IFormSubmitListener.INTERFACE.getName()) ||
     name.equals(ILinkListener.INTERFACE.getName()) ||
     name.equals(IBehaviorListener.INTERFACE.getName()) ||
     name.equals(IOnChangeListener.INTERFACE.getName()) )
{
   :
}

[ Permlink | « Hide ]

Pete Muir [14/May/08 09:50 AM]

Thanks Paolo - that works well!