ICEfaces
  1. ICEfaces
  2. ICE-9280

Problem with Liferay, ICEfaces, and "immediate" attribute

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 3.3
    • Fix Version/s: None
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      Windows 7
      GlassFish 3.1.2.2
      Liferay 6.1 CE

      Description

      Normally, we set the "immediate" attribute on any buttons that cancel the current operation. For obvious reasons, we need to bypass the validation process when any such button is pressed. The flow is basically this:

      Porlet X -> Initial view A -> User selects option -> Selected view B -> User presses "Cancel" -> Returned to view A

      If the "immediate" attribute on the "Cancel" button in view B is left out (or set to false), this is what shows up in the Chrome network monitor (assuming I fill in data that satisfies the validation requirements):

      http://localhost:8080/home?p_p_id=session_manager_WAR_Gateway&p_p_lifecycle=2&p_p_state=normal&p_p_mode=view&p_p_cacheability=cacheLevelPage&p_p_col_id=column-1&p_p_col_count=1&_session_manager_WAR_Gateway__jsfBridgeAjax=true&_session_manager_WAR_Gateway__facesViewIdResource=%2Fsession%2Flogin.xhtml

      If the "immediate" attribute is set to "true", this is what shows up:

      http://localhost:8080/home?p_p_id=session_manager_WAR_Gateway&p_p_lifecycle=2&p_p_state=normal&p_p_mode=view&p_p_cacheability=cacheLevelPage&p_p_col_id=column-1&p_p_col_count=1&_session_manager_WAR_Gateway__jsfBridgeAjax=true&_session_manager_WAR_Gateway__facesViewIdResource=%2Fsession%2Flogin.xhtml
      http://localhost:8080/home?p_p_id=session_manager_WAR_Gateway&p_p_lifecycle=2&p_p_state=normal&p_p_mode=view&p_p_cacheability=cacheLevelPage&p_p_col_id=column-1&p_p_col_count=1&_session_manager_WAR_Gateway_javax.faces.resource=jsf.js&_session_manager_WAR_Gateway_ln=javax.faces
      http://localhost:8080/home?p_p_id=session_manager_WAR_Gateway&p_p_lifecycle=2&p_p_state=normal&p_p_mode=view&p_p_cacheability=cacheLevelPage&p_p_col_id=column-1&p_p_col_count=1&_session_manager_WAR_Gateway_javax.faces.resource=bridge.js&_session_manager_WAR_Gateway_ln=ice.core
      http://localhost:8080/home?p_p_id=session_manager_WAR_Gateway&p_p_lifecycle=2&p_p_state=normal&p_p_mode=view&p_p_cacheability=cacheLevelPage&p_p_col_id=column-1&p_p_col_count=1&_session_manager_WAR_Gateway_javax.faces.resource=util%2Face-jquery.js&_session_manager_WAR_Gateway_ln=icefaces.ace
      http://localhost:8080/home?p_p_id=session_manager_WAR_Gateway&p_p_lifecycle=2&p_p_state=normal&p_p_mode=view&p_p_cacheability=cacheLevelPage&p_p_col_id=column-1&p_p_col_count=1&_session_manager_WAR_Gateway_javax.faces.resource=compat.js&_session_manager_WAR_Gateway_ln=ice.compat
      http://localhost:8080/home?p_p_id=session_manager_WAR_Gateway&p_p_lifecycle=2&p_p_state=normal&p_p_mode=view&p_p_cacheability=cacheLevelPage&p_p_col_id=column-1&p_p_col_count=1&_session_manager_WAR_Gateway_javax.faces.resource=util%2Face-components.js&_session_manager_WAR_Gateway_ln=icefaces.ace
      http://localhost:8080/home?p_p_id=session_manager_WAR_Gateway&p_p_lifecycle=2&p_p_state=normal&p_p_mode=view&p_p_cacheability=cacheLevelPage&p_p_col_id=column-1&p_p_col_count=1&_session_manager_WAR_Gateway_javax.faces.resource=icefaces-compat.js&_session_manager_WAR_Gateway_ln=ice.compat
      http://localhost:8080/home?p_p_id=session_manager_WAR_Gateway&p_p_lifecycle=2&p_p_state=normal&p_p_mode=view&p_p_cacheability=cacheLevelPage&p_p_col_id=column-1&p_p_col_count=1&_session_manager_WAR_Gateway_javax.faces.resource=util%2Face-menu.js&_session_manager_WAR_Gateway_ln=icefaces.ace

      Note that "localhost" is mapped to an organization site in Liferay.

      The first line in the "immediate" true case is the same as the line in the false case. The subsequent lines have several problems.

      Problem 1: There's no logical reason for all this extra traffic. Pressing the button in the false case sends only a single request/response; pressing the button in the true case isn't doing anything more from an application perspective so the extra traffic is completely unwarranted.

      Problem 2: The first line is properly encoded. The second line has the query parameters escaped, with '&" replaced with "&"; this thoroughly messes things up for Liferay.

      Problem 3: As a result of problem 2, bean management goes to hell in a handbasket. All of our beans use @CustomScoped("#{window}") and tracking the constructor for the bean in view A shows that it's being called multiple times, once for each request other than the first one. Deep inspection of the bean map in the window scope shows that a different window map is being created for each request (hardly surprising given the mess of the URLs). This, naturally, has an impact on application functionality as well as on its performance (the constructors often have side effects such as loading data from the database).

      Although we're using ICEfaces for the bulk of our components, the problem occurs whether we use <ace:pushButton> or <h:commandButton> to cancel view B; it's the "immediate" attribute that is causing the grief.

        Activity

        Hide
        Kevin Dean added a comment -

        The attached file contains HTTP archives of all the traffic.

        Show
        Kevin Dean added a comment - The attached file contains HTTP archives of all the traffic.
        Hide
        Kevin Dean added a comment -

        The Immediate_Bug.war file is the final deployed WAR file after Liferay does its thing; the Immediate_Bug_src.zip file is the source.

        The test is very simple:

        Install the portlet.
        Start network activity monitor.
        Press the button.
        Note only one request/response pair in activity monitor.
        Press the button.
        Note multiple request/response pairs in activity monitor.
        Press the button.
        Note multiple request/response pairs in activity monitor. It seems that the "immediate" setting corrupts further interaction with the portlet.

        This is deployed to GlassFish 3.1.2.2.

        Show
        Kevin Dean added a comment - The Immediate_Bug.war file is the final deployed WAR file after Liferay does its thing; the Immediate_Bug_src.zip file is the source. The test is very simple: Install the portlet. Start network activity monitor. Press the button. Note only one request/response pair in activity monitor. Press the button. Note multiple request/response pairs in activity monitor. Press the button. Note multiple request/response pairs in activity monitor. It seems that the "immediate" setting corrupts further interaction with the portlet. This is deployed to GlassFish 3.1.2.2.
        Hide
        Kevin Dean added a comment -

        It seems that when I created the forum entry (from which I copied this material), the ampersand encoding got stripped out. For example, the second URL when the "immediate" flag is true should be:

        http://localhost:8080/home?p_p_id=session_manager_WAR_Gateway&amp;p_p_lifecycle=2&amp;p_p_state=normal&amp;p_p_mode=view&amp;p_p_cacheability=cacheLevelPage&amp;p_p_col_id=column-1&amp;p_p_col_count=1&amp;_session_manager_WAR_Gateway_javax.faces.resource=jsf.js&amp;_session_manager_WAR_Gateway_ln=javax.faces

        The HAR files I uploaded give exact details.

        Show
        Kevin Dean added a comment - It seems that when I created the forum entry (from which I copied this material), the ampersand encoding got stripped out. For example, the second URL when the "immediate" flag is true should be: http://localhost:8080/home?p_p_id=session_manager_WAR_Gateway&amp;p_p_lifecycle=2&amp;p_p_state=normal&amp;p_p_mode=view&amp;p_p_cacheability=cacheLevelPage&amp;p_p_col_id=column-1&amp;p_p_col_count=1&amp;_session_manager_WAR_Gateway_javax.faces.resource=jsf.js&amp;_session_manager_WAR_Gateway_ln=javax.faces The HAR files I uploaded give exact details.
        Hide
        Kevin Dean added a comment -

        The problem has been traced to Liferay Faces Bridge. See http://issues.liferay.com/browse/FACES-1618?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel for details. This bug may be closed.

        Show
        Kevin Dean added a comment - The problem has been traced to Liferay Faces Bridge. See http://issues.liferay.com/browse/FACES-1618?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel for details. This bug may be closed.

          People

          • Assignee:
            Unassigned
            Reporter:
            Kevin Dean
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated: