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

              Ascending order - Click to sort in descending order
              Patrick Wilson created issue -
              Patrick Wilson made changes -
              Field Original Value New Value
              Assignee Mircea Toma [ mircea.toma ]
              Hide
              Permalink
              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.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #20726 Mon Feb 22 10:50:28 MST 2010 mircea.toma ICE-5394 Re-apply form submit redirection when form element is updated.
              Files Changed
              Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/javascript/application.js
              Hide
              Permalink
              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.
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              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).
              Patrick Wilson made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Assignee Priority P1
              Hide
              Permalink
              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.
              Patrick Wilson made changes -
              Attachment example.zip [ 12241 ]
              Hide
              Permalink
              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).
              Patrick Wilson made changes -
              Attachment if2.zip [ 12243 ]
              Patrick Wilson made changes -
              Attachment example.zip [ 12241 ]
              Ken Fyten made changes -
              Salesforce Case []
              Fix Version/s 2.0-Alpha3 [ 10032 ]
              Hide
              Permalink
              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
              Permalink
              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.
              Patrick Wilson made changes -
              Status Reopened [ 4 ] Closed [ 6 ]
              Resolution Fixed [ 1 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: