MyFaces Tomahawk
  1. MyFaces Tomahawk
  2. TOMAHAWK-1402

PortletExternalContextWrapper does not implement getRequestCharacterEncoding for wrapped ExternalContext.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.8
    • Fix Version/s: 1.1.9
    • Component/s: Portlet_Support
    • Labels:
      None
    • Environment:
      Tomahawk on JSF RI within the JBOSS Portlet Bridge, the portlet bridge fails to resolve the Encoding and throws an UnsupportedOperationException. Resolving fails in the PortletExternalContextWrapper, which lacks a getRequestCharacterEncoding.

      Description

      PortletExternalContextWrapper implements
      setResponseCharacterEncoding
      getResponseCharacterEncoding,
      setRequestCharacterEncoding,
      but not a getRequestCharacterEncoding.
      When the wrapper is used as the external faces context in a portlet bridge (e.g. JBOSS portlet bridge), the resolving of the RequestCharacterEncoding defaults to the inherited method from ExternalContext (which returns an invalid encoding).

      Implementing getRequestCharacterEncoding analogue to getResponseCharacterEncoding resolved the issue for me, the PortletExternalContextWrapper now works in a portlet environment.

      The Bug prevents usage of Tomahawk within the JBOSS portlet bridge.

        Activity

        Hide
        FR Weichand added a comment -

        Implemented getRequestCharacterEncoding() in the Wrapper, which simply hands over the method call to the _delegate.

        Show
        FR Weichand added a comment - Implemented getRequestCharacterEncoding() in the Wrapper, which simply hands over the method call to the _delegate.
        Hide
        Leonardo Uribe added a comment -

        Thanks for provide this patch

        Show
        Leonardo Uribe added a comment - Thanks for provide this patch

          People

          • Assignee:
            Leonardo Uribe
            Reporter:
            FR Weichand
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 0.5h
              0.5h
              Remaining:
              Remaining Estimate - 0.5h
              0.5h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development