ICEfaces
  1. ICEfaces
  2. ICE-5394

Updates to the Bridge seem to clobber updated forms

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Alpha2
    • Fix Version/s: 2.0-Alpha3
    • Component/s: Bridge
    • Labels:
      None
    • Environment:
      Firefox and Chrome (and probably other browsers), all app servers.
    • Assignee Priority:
      P1
    • Workaround Exists:
      Yes
    • Workaround Description:
      avoid having a form involved in a DOM diff.

      Description

      if a h:form is included in a DOM diff, it loses the submit capture javascript, thus the next submit from this form will trigger a full page reload.

        Activity

        Hide
        Patrick Wilson added a comment -

        On closer inspection it seems that if the page is initially loaded from icefaces, the submitCapture is working, however, if the form is applied to the browser dom in a subsequent update, it does not get the submit capture script placed on it. Thus the form will post to the server and the reload will reattach the script. However, the data that is posted to the server in this fashion seems to get lost.

        Show
        Patrick Wilson added a comment - On closer inspection it seems that if the page is initially loaded from icefaces, the submitCapture is working, however, if the form is applied to the browser dom in a subsequent update, it does not get the submit capture script placed on it. Thus the form will post to the server and the reload will reattach the script. However, the data that is posted to the server in this fashion seems to get lost.
        Hide
        Mircea Toma added a comment -

        When a form element is updated the event listeners used for redirecting the form submission through an AJAX request are lost. The solution was to detect when form is updated and re-apply the event listeners by using a ice.onAfterUpdate callback.

        Show
        Mircea Toma added a comment - When a form element is updated the event listeners used for redirecting the form submission through an AJAX request are lost. The solution was to detect when form is updated and re-apply the event listeners by using a ice.onAfterUpdate callback.
        Hide
        Patrick Wilson added a comment -

        I can still reproduce this issue. I have attached a sample project please build the dist target. Notice that every other form submission still seems to trigger full page reloads (not triggered by a dom update but by an uncaptured form submission back to index.jsf).

        Show
        Patrick Wilson added a comment - I can still reproduce this issue. I have attached a sample project please build the dist target. Notice that every other form submission still seems to trigger full page reloads (not triggered by a dom update but by an uncaptured form submission back to index.jsf).
        Hide
        Patrick Wilson added a comment -

        a sample icefaces2 project to demostrate the problem.

        Show
        Patrick Wilson added a comment - a sample icefaces2 project to demostrate the problem.
        Hide
        Patrick Wilson added a comment -

        Sorry, It seems that I managed to get the wrong jar into this demo. I have reattached the example with the correct jar (I still get the same problem).

        Show
        Patrick Wilson added a comment - Sorry, It seems that I managed to get the wrong jar into this demo. I have reattached the example with the correct jar (I still get the same problem).
        Hide
        Mircea Toma added a comment -

        The issue seems to be solved when running the test with the latest ICEfaces2.0 code.

        Show
        Mircea Toma added a comment - The issue seems to be solved when running the test with the latest ICEfaces2.0 code.
        Hide
        Patrick Wilson added a comment -

        After rebuilting icepush.jar this problem went away. Reclosing issue.

        Show
        Patrick Wilson added a comment - After rebuilting icepush.jar this problem went away. Reclosing issue.

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Patrick Wilson
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: