Uploaded image for project: 'MyFaces Trinidad'
  1. MyFaces Trinidad
  2. TRINIDAD-2105

DocumentRenderer.encodeAll must not assume root StateManager is Trinidad's

    Details

    • Type: Bug
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 2.0.0-core
    • Fix Version/s: None
    • Component/s: Portlet
    • Labels:
      None

      Description

      DocumentRenderer.encodeAll has the following code:

      StateManager sm = context.getApplication().getStateManager();

      if (sm instanceof StateManagerImpl)

      { // do real work }

      else

      { // log error and return }

      This means Trinidad fails in an environment in which its wrapped – such as the Portlet Bridge – even though its StateManager is in use. Instead the code should test for whether its StateManagerImpl and if not recursively test to see if its a StateManagerWrapper. For each wrapper it finds it calls .getWrapped and runs the test again.

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              mike_freedman Michael Freedman
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated: