ICEfaces
  1. ICEfaces
  2. ICE-11178

emporium - chat, basic samples fail (deltaSubmit=true)

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.1
    • Fix Version/s: 4.2
    • Component/s: ACE-Components, Sample Apps
    • Labels:
      None
    • Environment:
      Tomcat 7/8, all browsers. Fails with 4.1 release onward.
      deltaSubmit=true added to the web.xml

      Description

      1) Emporium chat fails when using deltaSubmit=true.
      The user is not able to join the first ("Lounge") chat room from drop-down, user can only join the remaining chat rooms.
      There are no errors, in browser console or server-side.
      This issue can be reproduced also with ICEfaces 4.1 libraries.

      2) Basic application: when clicking the "Stock JSF Page" button there is redirection to the "Stock JSF" page, however a JS error is written in browser console (or a popup occurs on IE8):
      IE8:
      Message: Object required
      Line: 20
      Char: 349
      Code: 0
      URI: http://10.18.39.156:8080/basic/javax.faces.resource/bridge.js.jsf?ln=ice.core&v=4_2_0_161110

      IE11/MS Edge:
      SCRIPT5007: Unable to get property 'nodeName' of undefined or null reference
      bridge.js.jsf (20,349)

      Firefox47:
      TypeError: a is null
      http://10.18.39.156:8080/basic/javax.faces.resource/bridge.js.jsf?ln=ice.core&v=4_2_0_161110
      Line 20

      Chrome:
      Uncaught TypeError: Cannot read property 'nodeName' of null(…) bridge.js.jsf?ln=ice.core&v=4_2_0_161110:20

      This issue can be reproduced also with EE-4.1.0.GA libraries.


      <context-param>
              <param-name>org.icefaces.deltaSubmit</param-name>
              <param-value>true</param-value>
          </context-param>
       added to web.xml

        Activity

        Hide
        Carmen Cristurean added a comment -

        There are no additional errors or information if running emporium with projectStage=Development.

        Show
        Carmen Cristurean added a comment - There are no additional errors or information if running emporium with projectStage=Development.
        Hide
        Mircea Toma added a comment -

        2) Modified recalculateFormPreviousParameters function in submit.js to try extracting form elements only from element updates, and skip the other types of updates.

        Show
        Mircea Toma added a comment - 2) Modified recalculateFormPreviousParameters function in submit.js to try extracting form elements only from element updates, and skip the other types of updates.
        Hide
        Carmen Cristurean added a comment -

        Verified issue2) with ICEfaces4 trunk r 49688 with deltaSubmit=true/false, on IE8, IE11, Chrome55, FF49, MsEdge38.

        Show
        Carmen Cristurean added a comment - Verified issue2) with ICEfaces4 trunk r 49688 with deltaSubmit=true/false, on IE8, IE11, Chrome55, FF49, MsEdge38.
        Hide
        Mircea Toma added a comment - - edited

        Replaced ace:simpleSelectOneMenu with ace:selectMenu in chat.xhtml to workaround the rendering strategy used by ace:simpleSelectOneMenu which is not compatible with delta submit anymore. ace:simpleSelectOneMenu renderer is now rendering the option tags as text (just like h:selectOneMenu) which makes impossible for delta submit to calculate the previous prarameters off the old DOM document.

        Show
        Mircea Toma added a comment - - edited Replaced ace:simpleSelectOneMenu with ace:selectMenu in chat.xhtml to workaround the rendering strategy used by ace:simpleSelectOneMenu which is not compatible with delta submit anymore. ace:simpleSelectOneMenu renderer is now rendering the option tags as text (just like h:selectOneMenu ) which makes impossible for delta submit to calculate the previous prarameters off the old DOM document.
        Hide
        Liana Munroe added a comment -

        Verified emporium chat ICEfaces 4 trunk r49699, with deltaSubmit=true/false using tomcat 8, MS Edge, IE 11, 10, 9, 8, FF 47, Chrome 53.

        Show
        Liana Munroe added a comment - Verified emporium chat ICEfaces 4 trunk r49699, with deltaSubmit=true/false using tomcat 8, MS Edge, IE 11, 10, 9, 8, FF 47, Chrome 53.
        Hide
        Mircea Toma added a comment -

        Created ICE-11203 to follow up with ace:simpleSelectOneMenu issue.

        Show
        Mircea Toma added a comment - Created ICE-11203 to follow up with ace:simpleSelectOneMenu issue.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: