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

          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Ken Fyten made changes -
          Fix Version/s 4.0 [ 11382 ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #39085 Tue Dec 03 12:15:58 MST 2013 mircea.toma ICE-9679 Use the ID of the first element for the update when the entire portlet content is updated.
          Files Changed
          Commit graph MODIFY /icefaces4/trunk/icefaces/core/src/main/java/org/icefaces/impl/context/DOMPartialViewContext.java
          Mircea Toma made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          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.
          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
          Ken Fyten made changes -
          Fix Version/s 4.0 [ 10770 ]
          Assignee Mircea Toma [ mircea.toma ]
          Affects Compatibility/Configuration [ 10002 ]
          Assignee Priority P2 [ 10011 ]
          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.
          Neil Griffin made changes -
          Field Original Value New Value
          Link This issue blocks ICE-7768 [ ICE-7768 ]
          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.
          Neil Griffin created issue -

            People

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

              Dates

              • Created:
                Updated:
                Resolved: