> On my build, the templates show up correctly but the actions need to be
> adapted not to use the PATH_PANEL_KEY constant but rely on
> PanedPortletController.getParameterName() or simply use directly
> a PortletTemplateLink object instead of DynamicURI.
Thanks, without any modification on my part, it works fairly well except for
a few situations, basically whenever I need to redirect to another pane.
Like on error validation. In that case we need to get the correct panel key,
which involves getting the portlet.
In a VelocityPortletAction, I don't see how to get the portlet associated
with the portlet action.
If you look at VelocityPortlet, it put the portlet and even the jlink into
the context:
public ConcreteElement getContent( RunData rundata )
{
System.out.println("in getContent for velcoity portlet");
// create a blank context and with all the global application
// Pull Tools inside
Context context = TurbineVelocity.getContext();
context.put( "data", rundata );
context.put( "portlet", this );
...
// Put the request and session based contexts
TurbinePull.populateContext(context, rundata);
context.put( "jlink", new PortletTemplateLink(rundata,this) );
(aside: this is making me wonder, what happens if there are 2
VelocityPortlets on a psml page)
Then looking at the stack trace going into the insert action for a user:
at
org.apache.jetspeed.modules.actions.portlets.security.UserUpdateAction.doIns
ert(UserUpdateAction.java:200)
at java.lang.reflect.Method.invoke(Native Method)
at
org.apache.turbine.util.velocity.VelocityActionEvent.executeEvents(VelocityA
ctionEvent.java:166)
at
org.apache.jetspeed.modules.actions.portlets.VelocityPortletAction.perform(V
elocityPortletAction.java:116)
at
org.apache.turbine.modules.ActionLoader.exec(ActionLoader.java:122)
at
org.apache.turbine.modules.pages.DefaultPage.doBuild(DefaultPage.java:143)
at org.apache.turbine.modules.Page.build(Page.java:90)
at org.apache.turbine.modules.PageLoader.exec(PageLoader.java:123)
at org.apache.turbine.Turbine.doGet(Turbine.java:447)
at org.apache.turbine.Turbine.doPost(Turbine.java:537)
As you can see, the portlet's getContent is never called on an action, so
the portlet (or jlink) never gets set.
How do you know which portlet is associated with a portlet action?
> -----Original Message-----
> From: Raphael Luta [mailto:[EMAIL PROTECTED]]
> Sent: Monday, August 27, 2001 9:13 AM
> To: [EMAIL PROTECTED]
> Subject: Re: cvs commit:
> jakarta-jetspeed/src/java/org/apache/jetspeed/services/portaltoolkit
> JetspeedPortalToolkitService.java
>
>
> David Sean Taylor wrote:
>
> > done
> > now lets see if we can get the security portlets fixed...
> >
>
>
> Thanks.
>
> On my build, the templates show up correctly but the actions need to be
> adapted not to use the PATH_PANEL_KEY constant but rely on
> PanedPortletController.getParameterName() or simply use directly
> a PortletTemplateLink object instead of DynamicURI.
>
>
> >
> >
> >>-----Original Message-----
> >>From: Raphael Luta [mailto:[EMAIL PROTECTED]]
> >>Sent: Monday, August 27, 2001 8:52 AM
> >>To: [EMAIL PROTECTED]
> >>Subject: Re: cvs commit:
> >>jakarta-jetspeed/src/java/org/apache/jetspeed/services/portaltoolkit
> >>JetspeedPortalToolkitService.java
> >>
> >>
> >>[EMAIL PROTECTED] wrote:
> >>
> >>
> >>>taylor 01/08/26 22:32:47
> >>>
> >>> Modified: src/java/org/apache/jetspeed/services/portaltoolkit
> >>> JetspeedPortalToolkitService.java
> >>> Log:
> >>> set.setName(String.valueOf(theCount.getValue()));
> >>>
> >>> broke all the security portlets that use $jlink.setPanel,
> >>>
> >>so I removed it for now.
> >>
> >>> We need a better patch for this, and I haven't yet
> >>>
> >>figured out exactly what was being attempted here.
> >>
> >>> Could someone enlighten me...
> >>> Anyway, I tested most parts of the system I know, and no problems.
> >>> (I believe we need unit tests in Jetspeed.)
> >>>
> >>>
> >>
> >>This is not the oode that breaks your portlet security stuff,
> >>it is just used for
> >>numbering the portlet sets. The actual code that modified the
> >>pane parameters
> >>behavior is VelocityPanedPortletController.getParameterName().
> >>
> >>Please, rollback this change.
> >>
> >>--
> >>Raphael Luta - [EMAIL PROTECTED]
> >>Vivendi Universal Networks - Paris
> >>
> >>
> >>---------------------------------------------------------------------
> >>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]
> >
> >
>
>
>
> --
> --
> Raphael Luta - [EMAIL PROTECTED]
> Vivendi Universal Networks - Paris
>
>
> ---------------------------------------------------------------------
> 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]
