You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pluto-dev@portals.apache.org by "David DeWolf (JIRA)" <pl...@jakarta.apache.org> on 2005/02/15 22:44:45 UTC

[jira] Resolved: (PLUTO-20) PortletControlParameter's getPIDValue and setPortletID disagree

     [ http://issues.apache.org/jira/browse/PLUTO-20?page=history ]
     
David DeWolf resolved PLUTO-20:
-------------------------------

     Assign To:     (was: pluto-dev@jakarta.apache.org)
    Resolution: Invalid

Upon further review: 

These two methods are not intended to reference the same id.  

The setPortletId(String) method is used internally by the portal to track which portlet window a PortletURL is being constructed for.  This value is then used whenever the url is output via the one of the toString methods to ensure that the portal can determin which portlet was targeted.

getPID on the otherhand is used to track the portlet id of the window which is CURRENTLY targeted (in this request, not a subsequent request). 

Because of this I don't believe this issue is valid.

> PortletControlParameter's getPIDValue and setPortletID disagree
> ---------------------------------------------------------------
>
>          Key: PLUTO-20
>          URL: http://issues.apache.org/jira/browse/PLUTO-20
>      Project: Pluto
>         Type: Bug
>   Components: portal driver
>     Versions: Current CVS
>  Environment: Operating System: All
> Platform: All
>     Reporter: Stephen Drye

>
> In org.apache.pluto.portalImpl.core.PortletControlParameter.java getPIDValue()
> reads the portlet id from the stateLessControlParameter, while setPortletId()
> sets the portlet id into the stateFullControlParameter.  Clearly one of these is
> wrong.
> If the portlet ID is stored in the stateFullControlParam, the
> stateLessControlParam variable can be removed as it will no longer be used.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira