ICEfaces
  1. ICEfaces
  2. ICE-9943

Several Ace components use a faulty findComponent method that does not search into facets

          Details

            Description

            When using an ace:tooltip that references an ace:delegate for an ace:dataTable, the tooltip can't find the delegate defined for it and throws the following error:

            Cannot find delegate component "tooltipDelegate" in view or it is not an instance of <ace:tooltipDelegate>.

            The components are located in a mobi:pagePanel. Removing the mobi component, the error is no longer thrown.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Arran Mccullough added a comment -

              Attached test case source code that shows the issue. War file is larger than 10mb so can be provided on request.

              Show
              Arran Mccullough added a comment - Attached test case source code that shows the issue. War file is larger than 10mb so can be provided on request.
              Hide
              Permalink
              Philip Breau added a comment - - edited

              I've fixed the toolTipRenderer in the IF 3.3 EE maintenance branch, but the root issue was that the toolTip component was not searching into facet children and thus, couldn't see within the PagePanel, which uses <f:facet>. But this problem would have shown up with the use of any facet, not just with the PagePanel.

              I haven't closed this, and widened the scope of it, as it appears that the same findComponentCustom() method is copied in several other Ace components, and will have the same issue:

              ContextMenu
              DataExporter
              Dialog
              Droppable

              We should look into this a bit more and possibly change all of these to use CoreComponentUtils.findComponentInView(), if suitable, and remove all of the duplicated code.

              Show
              Philip Breau added a comment - - edited I've fixed the toolTipRenderer in the IF 3.3 EE maintenance branch, but the root issue was that the toolTip component was not searching into facet children and thus, couldn't see within the PagePanel, which uses <f:facet>. But this problem would have shown up with the use of any facet, not just with the PagePanel. I haven't closed this, and widened the scope of it, as it appears that the same findComponentCustom() method is copied in several other Ace components, and will have the same issue: ContextMenu DataExporter Dialog Droppable We should look into this a bit more and possibly change all of these to use CoreComponentUtils.findComponentInView(), if suitable, and remove all of the duplicated code.
              Hide
              Permalink
              Arturo Zambrano added a comment -

              Replaced occurrences of findComponentCustom() for CoreComponentUtils.findComponentInView() and removed findComponentCustom() implementations.

              Committed fix to 4.0 trunk at revision 40721 and to 3.3 EE maintenance branch at revision 40722.

              Show
              Arturo Zambrano added a comment - Replaced occurrences of findComponentCustom() for CoreComponentUtils.findComponentInView() and removed findComponentCustom() implementations. Committed fix to 4.0 trunk at revision 40721 and to 3.3 EE maintenance branch at revision 40722.
              Hide
              Permalink
              Liana Munroe added a comment -

              Issue found in Showcase > Context Menu> Overview (All browsers) r40726.
              Left clicking anywhere on the page to dismiss a context menu only works the first time. When you subsequently right click to get another context menu, a left click will not dismiss it. This is not the case when you are left clicking on a menu item, it only happens when you are left clicking outside of the menu.

              Show
              Liana Munroe added a comment - Issue found in Showcase > Context Menu> Overview (All browsers) r40726. Left clicking anywhere on the page to dismiss a context menu only works the first time. When you subsequently right click to get another context menu, a left click will not dismiss it. This is not the case when you are left clicking on a menu item, it only happens when you are left clicking outside of the menu.
              Hide
              Permalink
              Arturo Zambrano added a comment -

              Issue found in Showcase > Context Menu> Overview (All browsers) r40726.
              Left clicking anywhere on the page to dismiss a context menu only works the first time. When you subsequently right click to get another context menu, a left click will not dismiss it. This is not the case when you are left clicking on a menu item, it only happens when you are left clicking outside of the menu.

              This issue was most likely not related to this JIRA, but a separate issue more related to ICE-9879.

              Committed fix to 4.0 trunk at revision 40798 and to 3.3 EE maintenance branch at revision 40799. Removed the namespace from the click event handler, so that the event can be triggered multiple times.

              Show
              Arturo Zambrano added a comment - Issue found in Showcase > Context Menu> Overview (All browsers) r40726. Left clicking anywhere on the page to dismiss a context menu only works the first time. When you subsequently right click to get another context menu, a left click will not dismiss it. This is not the case when you are left clicking on a menu item, it only happens when you are left clicking outside of the menu. This issue was most likely not related to this JIRA, but a separate issue more related to ICE-9879 . Committed fix to 4.0 trunk at revision 40798 and to 3.3 EE maintenance branch at revision 40799. Removed the namespace from the click event handler, so that the event can be triggered multiple times.
              Hide
              Permalink
              Liana Munroe added a comment -

              Showcase > Context Menu> Overview Left click issue is confirmed fixed. Icefaces ee-3.3.0 maintenance branch and Icefaces 4 trunk r40799. Tomcat 7, all browsers.

              Show
              Liana Munroe added a comment - Showcase > Context Menu> Overview Left click issue is confirmed fixed. Icefaces ee-3.3.0 maintenance branch and Icefaces 4 trunk r40799. Tomcat 7, all browsers.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: