ICEfaces
  1. ICEfaces
  2. ICE-10284

ace:submitMonitor - Modal layer not removed with multiple monitors

    Details

      Description

      When there are multiple ace:submitMonitors in use, one global one and one used for specific components, if the one for the specific component is activated, the global one is also activated. The issue here is that the modal layer for the global one is not removed upon receiving the response.

        Activity

        Hide
        Arran Mccullough added a comment -

        Attached test case that shows this issue.

        Steps:

        • Load welcomeICEfaces.jsf
        • Click on the Open button, a dialog is shown.
        • Click on the Submit button, this executes a long delay and both monitors are shown. Upon receiving the response, the global notification dialog is still shown along with its modal layer.
        Show
        Arran Mccullough added a comment - Attached test case that shows this issue. Steps: Load welcomeICEfaces.jsf Click on the Open button, a dialog is shown. Click on the Submit button, this executes a long delay and both monitors are shown. Upon receiving the response, the global notification dialog is still shown along with its modal layer.
        Hide
        Arran Mccullough added a comment -

        Ken also suggested that we review how multiple monitors should behave. I have asked the customer on what they would like to see and they responded with the following priority:

        1. Block @none
        2. Block a group or a area
        3. Block @all

        Show
        Arran Mccullough added a comment - Ken also suggested that we review how multiple monitors should behave. I have asked the customer on what they would like to see and they responded with the following priority: 1. Block @none 2. Block a group or a area 3. Block @all
        Hide
        Ken Fyten added a comment -

        New JIRA created: ICE-10313 - "Support 'order of precedence' when multiple ace:submitMonitors active concurrently" for the above.

        Show
        Ken Fyten added a comment - New JIRA created: ICE-10313 - "Support 'order of precedence' when multiple ace:submitMonitors active concurrently" for the above.
        Hide
        Mircea Toma added a comment -

        Pulled out the registration of 'beforeSubmit' and 'afterUpdate' callbacks out of the submit monitor instantiation. Instead register 'beforeSubmit' in order of component instantiation and 'afterSubmit' in reverse order to ensure that each monitor disables and re-enabled its own input element callbacks.

        This fix changes the component so that multiple instances can run in the same page. Initially component was written with the assumption that only one instance is present on the page.

        Show
        Mircea Toma added a comment - Pulled out the registration of 'beforeSubmit' and 'afterUpdate' callbacks out of the submit monitor instantiation. Instead register 'beforeSubmit' in order of component instantiation and 'afterSubmit' in reverse order to ensure that each monitor disables and re-enabled its own input element callbacks. This fix changes the component so that multiple instances can run in the same page. Initially component was written with the assumption that only one instance is present on the page.
        Hide
        Liana Munroe added a comment -

        Verified ICEfaces4 trunk r43877. Tomcat 7, IE 11, FF 34, Chrome 39.

        Show
        Liana Munroe added a comment - Verified ICEfaces4 trunk r43877. Tomcat 7, IE 11, FF 34, Chrome 39.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: