ICEfaces
  1. ICEfaces
  2. ICE-5351

DisplayEvent.getContextValue() can only be cast to a String object with panelTooltip

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.2-EE-GA
    • Fix Version/s: 1.8.2-EE-GA_P01, 1.8.3
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      All
    • Assignee Priority:
      P1

      Description

      When getting the context value set for a panelGroup the object can only be cast to a String value. This is fixed when the context value is retrieved for a menuPopup (fixed in ICE-4105) it works fine and returns the proper object. The fixes that were made for ICE4105 need to be added to the panelTooltip component.

        Issue Links

          Activity

          Hide
          Mark Collette added a comment -

          Now pass the actual Object contextValue into the DisplayEvent, instead of the String value of that object.

          TRUNK
          Subversion 20864
          icefaces\component\src\com\icesoft\faces\component\paneltooltip\PanelTooltip.java

          icefaces-ee-1.8.2
          Subversion 20865
          icefaces\component\src\com\icesoft\faces\component\paneltooltip\PanelTooltip.java

          Show
          Mark Collette added a comment - Now pass the actual Object contextValue into the DisplayEvent, instead of the String value of that object. TRUNK Subversion 20864 icefaces\component\src\com\icesoft\faces\component\paneltooltip\PanelTooltip.java icefaces-ee-1.8.2 Subversion 20865 icefaces\component\src\com\icesoft\faces\component\paneltooltip\PanelTooltip.java
          Hide
          Joanne Bai added a comment -

          Verified with success on ICEfaces-ee-1.8.2 revision 20874 using Tomcat 6 + FF3.5 and IE8

          Test app is checked into repo\qa\trunk\Regression\ICE-5351

          Show
          Joanne Bai added a comment - Verified with success on ICEfaces-ee-1.8.2 revision 20874 using Tomcat 6 + FF3.5 and IE8 Test app is checked into repo\qa\trunk\Regression\ ICE-5351
          Hide
          Joanne Bai added a comment -

          Found in nightly regression test that the test application throws NullPointerException on Tomcat 5, while it's working fine on Tomcat 6.

          Test app is available at C:\repo\qa\trunk\Regression\ICE-5351, which can be built for Tomcat 5 using: ant clean tomcat5.x

          Show
          Joanne Bai added a comment - Found in nightly regression test that the test application throws NullPointerException on Tomcat 5, while it's working fine on Tomcat 6. Test app is available at C:\repo\qa\trunk\Regression\ ICE-5351 , which can be built for Tomcat 5 using: ant clean tomcat5.x
          Hide
          Mark Collette added a comment -

          Found out that with JSF 1.1 + JSP, the tags are executed and rendered as they go, without a separate render call once the tree is fully in place, like Facelets and JSF 1.2 + JSP do. That has the side-effect that you can't use panelToolip, menuPopup, for, or any other attribute that refers to another component, and have that component later on (lower) in the component tree, since the component won't exist yet. Changed the test to work with JSF 1.1 + JSP by moving the panelTooltip component to be before (above) the panelGroup that refers to it.

          Next, we made the test cover both the scenario of using a String value, and the scenario of using a complex data-type as the contextValue.

          Show
          Mark Collette added a comment - Found out that with JSF 1.1 + JSP, the tags are executed and rendered as they go, without a separate render call once the tree is fully in place, like Facelets and JSF 1.2 + JSP do. That has the side-effect that you can't use panelToolip, menuPopup, for, or any other attribute that refers to another component, and have that component later on (lower) in the component tree, since the component won't exist yet. Changed the test to work with JSF 1.1 + JSP by moving the panelTooltip component to be before (above) the panelGroup that refers to it. Next, we made the test cover both the scenario of using a String value, and the scenario of using a complex data-type as the contextValue.
          Hide
          Joanne Bai added a comment -

          Thanks to Mark. It has been verified that it's a test application issue rather than ICEfaces.
          The test app has been adjusted and tested successfully on both Tomcat 5, 6 using FF3.5 and IE8.

          Updated test app and selenium test scripts have been committed.

          Show
          Joanne Bai added a comment - Thanks to Mark. It has been verified that it's a test application issue rather than ICEfaces. The test app has been adjusted and tested successfully on both Tomcat 5, 6 using FF3.5 and IE8. Updated test app and selenium test scripts have been committed.

            People

            • Assignee:
              Mark Collette
              Reporter:
              Arran Mccullough
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: