ICEfaces
  1. ICEfaces
  2. ICE-6040

'onsubmit' handler can fail when submit functions are chained

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Invalid
    • Affects Version/s: 2.0-Beta1
    • Fix Version/s: 2.0-Beta2, 2.0.0
    • Component/s: Bridge
    • Labels:
      None
    • Environment:
      IE6/7/8

      Description

      In cases where Mojarra or other third party libraries chain the event handlers that cause a form submit might cause a failure in the 'onsubmit' handler registered by ICEfaces to capture the form submit. The scope of the handler invocation changes thus the reference to the element and event are not valid anymore.
      To fix the issue the element needs to be referenced only by its ID and the event cloned (in IE the event is cleared because it's a global variable).

        Activity

        Mircea Toma created issue -
        Mircea Toma made changes -
        Field Original Value New Value
        Assignee Mircea Toma [ mircea.toma ]
        Mircea Toma made changes -
        Salesforce Case []
        Fix Version/s 2.0-Beta2 [ 10242 ]
        Hide
        Mircea Toma added a comment -

        Attached patch that fixes the element referencing.

        Show
        Mircea Toma added a comment - Attached patch that fixes the element referencing.
        Mircea Toma made changes -
        Attachment onsubmit-chaining.patch [ 12553 ]
        Ken Fyten made changes -
        Salesforce Case []
        Assignee Priority P2
        Hide
        Mircea Toma added a comment -

        We don't need to support out of band onsubmit handler invocation such as when third party JS code will invoke the onsubmit handler on a different thread than the one associated to the event that triggers the submit.

        Show
        Mircea Toma added a comment - We don't need to support out of band onsubmit handler invocation such as when third party JS code will invoke the onsubmit handler on a different thread than the one associated to the event that triggers the submit.
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Invalid [ 6 ]
        Ken Fyten made changes -
        Fix Version/s 2.0.0 [ 10230 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Priority P2

          People

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

            Dates

            • Created:
              Updated:
              Resolved: