Yeah, this patch was reversed because we had some issues. It works under a bear-bones environment but when some other state-managers are introduced, this patch blows up. I'll reassign it to 1.2.5 since it primarily only impacts the portal environment which is still experimental anyway.

Scott

Matthias Weßendorf (JIRA) wrote:
[ https://issues.apache.org/jira/browse/TRINIDAD-798?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12545186 ]
Matthias Weßendorf commented on TRINIDAD-798:
---------------------------------------------

scotty,

I see some related commits, but issue is still open

StateManagerImpl extends StateManager rather then StateManagerWrapper
---------------------------------------------------------------------

                Key: TRINIDAD-798
                URL: https://issues.apache.org/jira/browse/TRINIDAD-798
            Project: MyFaces Trinidad
         Issue Type: Bug
         Components: Portlet
   Affects Versions: 1.2.3-core
        Environment: JSR-168
           Reporter: Scott O'Bryan
           Assignee: Scott O'Bryan
            Fix For: 1.2.4-core

        Attachments: trinidad-798.patch


In JSF 1.1, there was no StageManagerWrapper provided by JSF so we were 
extending the base StateManager.  In 1.2 this class was provided in order to 
support the delegation chain.  This delegation chain is important to support 
the JSR-301's portlet bridge.
Trinidad's StateManagerImpl should be changed to extend JSF1.2's 
StateManagerWrapper class in order to support this delegation chain.  The 
mechanism used in 1.1 is not sufficient for 1.2 extensions which may require a 
StateManager.


Reply via email to