ICEfaces
  1. ICEfaces
  2. ICE-10720

Toggling "rendered" doesn't work properly with multiple ace:submitMonitor components

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Minor Minor
          • Resolution: Fixed
          • Affects Version/s: 4.0
          • Fix Version/s: 4.1
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
            Any, tested in trunk showcase
          • Assignee Priority:
            P2
          • Workaround Exists:
            Yes
          • Workaround Description:
            Hide
            Refreshing the page (with the checkbox/rendered states maintained) updates the monitors properly. So changing List Monitor to rendered="false" and then refreshing will properly show the Panel Monitor on further submissions.
            So maybe just something not being cleaned up on un-render.
            Show
            Refreshing the page (with the checkbox/rendered states maintained) updates the monitors properly. So changing List Monitor to rendered="false" and then refreshing will properly show the Panel Monitor on further submissions. So maybe just something not being cleaned up on un-render.

            Description

            This problem manifests when using multiple ace:submitMonitor components on a single page with "rendered" attributes.

            When these attributes are toggled (in this case via checkboxes) the submitMonitor doesn't "bubble up" to the next available monitor properly.

            For example:
            Page level monitor ('for' undefined)
            Panel level monitor ('for' equal to panel ID)
            List (inside panel) level monitor ('for' equal to list ID)

            If we toggle "List Monitor" to rendered="false" the expected behaviour is the Panel Monitor should process any further submissions. That doesn't happen, and no monitor is displayed.
            Similarly toggling the Panel Monitor to rendered="false" means the submission should "bubble up" to the Page Monitor, which again doesn't happen.

            Toggling back to rendered="true" seems to work though.

              Issue Links

              blocks

              Improvement - An improvement or enhancement to an existing feature or task. ICE-10707 Showcase -> Add new ace:submitMonitor demos

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Carlo Guglielmin created issue -
                Hide
                Permalink
                Ken Fyten added a comment -

                This issue can be reproduced using the Showcase->ace:submitMonitor->Nested demo.

                If you uncheck any of the submitMonitor levels in the demo, the monitors no longer function properly.

                Perhaps the ace:submitMonitor is not implementing the onBeforeUpdate callback to clean itself up properly?

                Show
                Ken Fyten added a comment - This issue can be reproduced using the Showcase->ace:submitMonitor->Nested demo. If you uncheck any of the submitMonitor levels in the demo, the monitors no longer function properly. Perhaps the ace:submitMonitor is not implementing the onBeforeUpdate callback to clean itself up properly?
                Ken Fyten made changes -
                Field Original Value New Value
                Summary Toggling "rendered" doesn't work properly with multiple ace:submitMonitor Toggling "rendered" doesn't work properly with multiple ace:submitMonitors
                Assignee Mircea Toma [ mircea.toma ]
                Fix Version/s 4.1 [ 11375 ]
                Priority Major [ 3 ] Minor [ 4 ]
                Assignee Priority P2 [ 10011 ]
                Ken Fyten made changes -
                Summary Toggling "rendered" doesn't work properly with multiple ace:submitMonitors Toggling "rendered" doesn't work properly with multiple ace:submitMonitor components
                Ken Fyten made changes -
                Link This issue blocks ICE-10707 [ ICE-10707 ]
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #44985 Tue Jun 09 07:27:27 MDT 2015 mircea.toma ICE-10720 Cleanup monitor mapping when monitors are removed by updates.
                Files Changed
                Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/submitmonitor/submitmonitor.js
                Hide
                Permalink
                Mircea Toma added a comment -

                Cleanup monitor mapping when monitors are removed by updates.

                Show
                Mircea Toma added a comment - Cleanup monitor mapping when monitors are removed by updates.
                Mircea Toma made changes -
                Status Open [ 1 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                Ken Fyten made changes -
                Status Resolved [ 5 ] Closed [ 6 ]

                  People

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

                    Dates

                    • Created:
                      Updated:
                      Resolved: