ICEfaces
  1. ICEfaces
  2. ICE-8539

ACE File Entry components fail on Liferay 5.x

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.0.0.GA, EE-3.0.0.GA_P01
    • Fix Version/s: 3.3
    • Component/s: ACE-Components, Sample Apps
    • Labels:
      None
    • Environment:
      Liferay 5.x Portal Portlet ACE File Entry Component
    • Assignee Priority:
      P1
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration

      Description

      In testing the showcase-portlet examples on Liferay (version 5.2.3 only, not on versions 6.0 or 6.1), the following behaviour was observed. It exists in the EE-3.0.0.GA release as well as on the current maintenance branch.

      When running any of the File Entry examples, after selecting a file and clicking the "Send File" button, the operation fails and the following exception is thrown:

      Sep 5, 2012 9:06:16 PM org.icefaces.impl.application.ExtendedExceptionHandler handle
      WARNING: queued exception
      java.lang.IllegalArgumentException: null source
      at java.util.EventObject.<init>(EventObject.java:38)
      at javax.faces.event.SystemEvent.<init>(SystemEvent.java:71)
      at javax.faces.event.ComponentSystemEvent.<init>(ComponentSystemEvent.java:73)
      at javax.faces.event.PostRestoreStateEvent.<init>(PostRestoreStateEvent.java:73)
      at com.sun.faces.lifecycle.RestoreViewPhase.deliverPostRestoreStateEvent(RestoreViewPhase.java:271)
      at com.sun.faces.lifecycle.RestoreViewPhase.execute(RestoreViewPhase.java:257)
      at com.sun.faces.lifecycle.Phase.doPhase(Phase.java:101)
      at com.sun.faces.lifecycle.RestoreViewPhase.doPhase(RestoreViewPhase.java:116)
      at com.sun.faces.lifecycle.LifecycleImpl.execute(LifecycleImpl.java:118)
      at org.portletfaces.bridge.BridgeImpl.doFacesRequest(BridgeImpl.java:524)
      at org.portletfaces.bridge.GenericFacesPortlet.serveResource(GenericFacesPortlet.java:131)
      at com.sun.portal.portletcontainer.appengine.filter.FilterChainImpl.doFilter(FilterChainImpl.java:177)
      at com.liferay.portal.kernel.portlet.PortletFilterUtil.doFilter(PortletFilterUtil.java:76)
      at com.liferay.portal.kernel.servlet.PortletServlet.service(PortletServlet.java:100)
      at javax.servlet.http.HttpServlet.service(HttpServlet.java:717)

        Issue Links

          Activity

          Hide
          Mark Collette added a comment -

          This is with liferay-portal-tomcat-6.0-5.2.3

          Show
          Mark Collette added a comment - This is with liferay-portal-tomcat-6.0-5.2.3
          Hide
          Mark Collette added a comment -

          It appears that the form is not being converted to being multipart, due to the script that does that being put into the browser DOM within a CDATA section:

          <span id="A2497:example-form:_captureFileOnsubmit"><script type="text/javascript">/<![CDATA[*/ice.ace.fileentry.captureFormOnsubmit("A2497:example-form","A2497:example-form:hiddenIframe","null","http://localhost:8080/web/guest/ace10?_fileEntry_WAR_showcaseportlet_javax.faces.resource=org.icefaces.ace.component.fileentry.progress.A2497:example-form::vyjxzlo1.txt&p_p_col_count=1&p_p_col_id=column-1&p_p_id=fileEntry_WAR_showcaseportlet&p_p_lifecycle=2");/*]]>/</script></span>

          Show
          Mark Collette added a comment - It appears that the form is not being converted to being multipart, due to the script that does that being put into the browser DOM within a CDATA section: <span id="A2497:example-form:_captureFileOnsubmit"><script type="text/javascript">/ <![CDATA [*/ice.ace.fileentry.captureFormOnsubmit("A2497:example-form","A2497:example-form:hiddenIframe","null","http://localhost:8080/web/guest/ace10?_fileEntry_WAR_showcaseportlet_javax.faces.resource=org.icefaces.ace.component.fileentry.progress.A2497:example-form::vyjxzlo1.txt&p_p_col_count=1&p_p_col_id=column-1&p_p_id=fileEntry_WAR_showcaseportlet&p_p_lifecycle=2");/*] ]> /</script></span>
          Hide
          Mark Collette added a comment -

          No, that's commented out with a javascript block comment, and I can see all the other script done like that, and working.

          Show
          Mark Collette added a comment - No, that's commented out with a javascript block comment, and I can see all the other script done like that, and working.
          Hide
          Mark Collette added a comment -

          The issue ended up in being in the portlet faces bridge's ExternalContextImpl, which uses the response contentType when queried for the request contentType, so it wouldn't report that it was multipart. I tried to selectively alter this to only change it for uploading multipart forms, to show the ResourceRequest's contentType.

          Show
          Mark Collette added a comment - The issue ended up in being in the portlet faces bridge's ExternalContextImpl, which uses the response contentType when queried for the request contentType, so it wouldn't report that it was multipart. I tried to selectively alter this to only change it for uploading multipart forms, to show the ResourceRequest's contentType.
          Hide
          Mark Collette added a comment - - edited

          The portletfacesbridge jar is named after the parent umbrella task of ICE-8986, since it encompasses fixes beyond just this jira. Replace icefaces3/lib/portlets/portletfaces/portletfaces-bridge-2.0.2-ice-7875.jar with icefaces3/lib/portlets/portletfaces/portletfaces-bridge-2.0.2-ice-8986.jar

          icefaces3 trunk
          Subversion 33397

          Show
          Mark Collette added a comment - - edited The portletfacesbridge jar is named after the parent umbrella task of ICE-8986 , since it encompasses fixes beyond just this jira. Replace icefaces3/lib/portlets/portletfaces/portletfaces-bridge-2.0.2-ice-7875.jar with icefaces3/lib/portlets/portletfaces/portletfaces-bridge-2.0.2-ice-8986.jar icefaces3 trunk Subversion 33397

            People

            • Assignee:
              Mark Collette
              Reporter:
              Deryk Sinotte
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: