ICEfaces
  1. ICEfaces
  2. ICE-9679

DOMPartialViewContext.processPartial(PhaseId) does not conform to JSF 2.2 Spec regarding javax.faces.ViewRoot

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.3
    • Fix Version/s: 4.0.BETA, 4.0
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      JSF 2.2 Portlet Bridge
    • Assignee Priority:
      P2
    • Affects:
      Compatibility/Configuration

      Description

      The following link documents a discussion for new JSF 2.2 requirements regarding javax.faces.ViewRoot:
      https://java.net/jira/browse/JAVASERVERFACES_SPEC_PUBLIC-1069

      Mojarra 2.2 was made compliant with a patch with this issue:
      https://java.net/jira/browse/JAVASERVERFACES-2660

      The problem is that the ICEfaces 3.3 DOMPartialViewContext.processPartial(PhaseId) method does not conform to these new requirements.

      The calls to partialWriter.startUpdate(String target) is passing "javax.faces.ViewRoot" but for portlets it should pass viewRoot.getClientId(ctx)


        Issue Links

          Activity

          Hide
          Neil Griffin added a comment - - edited

          Note that this should only happen for JSF 2.2 and portlets, so there would need to be some detection in ICEfaces for JSF 2.2.

          Show
          Neil Griffin added a comment - - edited Note that this should only happen for JSF 2.2 and portlets, so there would need to be some detection in ICEfaces for JSF 2.2.
          Hide
          Ken Fyten added a comment -

          Thanks for logging this JIRA Neil. Targeted for ICEfaces 4.0, which is our JSF 2.2 supporting release.

          Show
          Ken Fyten added a comment - Thanks for logging this JIRA Neil. Targeted for ICEfaces 4.0, which is our JSF 2.2 supporting release.
          Hide
          Neil Griffin added a comment -

          Corresponding issue in the Liferay Faces issue tracker: https://issues.liferay.com/browse/FACES-1798

          Show
          Neil Griffin added a comment - Corresponding issue in the Liferay Faces issue tracker: https://issues.liferay.com/browse/FACES-1798
          Hide
          Mircea Toma added a comment -

          Modified DOMPartialViewContext to use the ID of the first element for the update when the entire portlet content is updated.

          Show
          Mircea Toma added a comment - Modified DOMPartialViewContext to use the ID of the first element for the update when the entire portlet content is updated.

            People

            • Assignee:
              Mircea Toma
              Reporter:
              Neil Griffin
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: