portals-bridges-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Antony Stubbs (JIRA)" <bridges-...@portals.apache.org>
Subject [jira] Commented: (PB-84) CCE in PortletWindowUtils.getPortletWindowId
Date Fri, 17 Oct 2008 14:18:46 GMT

    [ https://issues.apache.org/jira/browse/PB-84?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12640538#action_12640538

Antony Stubbs commented on PB-84:

P.s. its just a basic hello world portlet application, nothing to it, deployed to pluto installed
on tomcat using pluto:install .

> CCE in PortletWindowUtils.getPortletWindowId
> --------------------------------------------
>                 Key: PB-84
>                 URL: https://issues.apache.org/jira/browse/PB-84
>             Project: Portals Bridges
>          Issue Type: Bug
>          Components: common
>    Affects Versions: 1.0
>         Environment: Mac OS, Wicket Portlets, OpenPortal Portlet Container
>            Reporter: Tim Boudreau
>         Attachments: portals-bridges-common-1.0.4.jar
> I'm trying to get wicket portlet support working over OpenPortal.  I've hacked together
implementations of ServletContextProvider and PortletResourceURLFactory - just required exposing
WicketFilter from the Application to get the necessary data.  I can deploy a portlet, but...
> I'm getting this exception. 
> java.lang.ClassCastException: java.lang.Double
>         at org.apache.portals.bridges.util.PortletWindowUtils.getPortletWindowId(PortletWindowUtils.java:45)
> Looking at the code, this seems like a clear bug:  The method casts to String here:
> (String)session.getAttribute(PORTLET_WINDOW_ID);
> and if null fills in the value with an instance of Double.  If it's going to put a Double
there, it should probably not expect a String - this exception will occur any time this method
is called twice for a PortletSession, it will throw the CCE the second time it is called.
> Since the value is not used, probably simply changing it to 
>         Object portletWindowId = session.getAttribute(PORTLET_WINDOW_ID);
> would fix it.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

To unsubscribe, e-mail: bridges-dev-unsubscribe@portals.apache.org
For additional commands, e-mail: bridges-dev-help@portals.apache.org

View raw message