ICEfaces
  1. ICEfaces
  2. ICE-10516

ace:submitMonitor - Delay showing modal layer

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.3.0.GA_P02, 4.0
    • Fix Version/s: EE-3.3.0.GA_P03, 4.1
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      All

      Description

      When using the ace:submitMonitor with blockUI="@all", it is possible to do a double submit when quickly doing a submit (double click a button).

      This is reproducible with the Showcase demo.

        Activity

        Hide
        Arran Mccullough added a comment -

        Attached basic test case. Note: the icefaces, ace, and compat jars need to be added back into the war to run.

        Steps:

        • Load welcomeICEfaces.jsf
        • Quickly double click the Submit button. This will show two 'SUBMIT CALLED' logs shown in the server logs as well as one monitor will be shown after the other one.
        Show
        Arran Mccullough added a comment - Attached basic test case. Note: the icefaces, ace, and compat jars need to be added back into the war to run. Steps: Load welcomeICEfaces.jsf Quickly double click the Submit button. This will show two 'SUBMIT CALLED' logs shown in the server logs as well as one monitor will be shown after the other one.
        Hide
        Mircea Toma added a comment -

        Cancel event bubbling in the event sinks that ace:submitMonitor sets up. Tested also with icecore:singleSubmit.

        Show
        Mircea Toma added a comment - Cancel event bubbling in the event sinks that ace:submitMonitor sets up. Tested also with icecore:singleSubmit.
        Hide
        Liana Munroe added a comment -

        Verified using attached test case ICEfaces EE-4.0.0 Jenkins build 2 and EE-3.3.0 maintenance branch. Tomcat 7, IE 11, FF 34, Chrome 41.
        When testing ace:submitMonitor demo in showcase the submit still appears twice when double clicking the Submit button. This can be noticed more often when first using the demo. Changing the region to block, then double clicking submit will usually show the submitMonitor twice. Subsequent submits will show the submitMonitor only once.

        Show
        Liana Munroe added a comment - Verified using attached test case ICEfaces EE-4.0.0 Jenkins build 2 and EE-3.3.0 maintenance branch. Tomcat 7, IE 11, FF 34, Chrome 41. When testing ace:submitMonitor demo in showcase the submit still appears twice when double clicking the Submit button. This can be noticed more often when first using the demo. Changing the region to block, then double clicking submit will usually show the submitMonitor twice. Subsequent submits will show the submitMonitor only once.
        Hide
        Mircea Toma added a comment -

        Modified submitmonitor.js code to override the primitive submit function with one that will block sub-sequent calls. This is required in cases where the callbacks that trigger a submit are registered with Element.addEventListener function and cannot be reached to be disabled temporarily (to avoid subsequent submits).

        Show
        Mircea Toma added a comment - Modified submitmonitor.js code to override the primitive submit function with one that will block sub-sequent calls. This is required in cases where the callbacks that trigger a submit are registered with Element.addEventListener function and cannot be reached to be disabled temporarily (to avoid subsequent submits).
        Hide
        Liana Munroe added a comment -

        Verified ICEfaces 4 trunk, EE-3.3.0 maintenance branch r44464 using showcase and attached test case. Tomcat 7, FF 34, Chrome 41, IE 11.

        Show
        Liana Munroe added a comment - Verified ICEfaces 4 trunk, EE-3.3.0 maintenance branch r44464 using showcase and attached test case. Tomcat 7, FF 34, Chrome 41, IE 11.
        Hide
        Ken Fyten added a comment -

        QA, also need to verify that this fix doesn't cause any issues with the following previous fixes:

        Show
        Ken Fyten added a comment - QA, also need to verify that this fix doesn't cause any issues with the following previous fixes: ICE-8668 ICE-5223 ICE-8914
        Hide
        Liana Munroe added a comment -

        ICE-8914 verified with ICEfaces EE-3.3.0 maintenance branch, ICEfaces 4 trunk r44499. Tomcat 7, IE 11, FF 34, Chrome 41.
        New test application /ICE-8914.jsf added to:
        http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/panel
        http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/panel.

        As per Ken, ICE-5223 does not require further verification.

        Show
        Liana Munroe added a comment - ICE-8914 verified with ICEfaces EE-3.3.0 maintenance branch, ICEfaces 4 trunk r44499. Tomcat 7, IE 11, FF 34, Chrome 41. New test application / ICE-8914 .jsf added to: http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/panel http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/panel . As per Ken, ICE-5223 does not require further verification.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: