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

          Arran Mccullough created issue -
          Arran Mccullough made changes -
          Field Original Value New Value
          Salesforce Case [5007000000AplGc]
          Ken Fyten made changes -
          Fix Version/s 1.8.2-EE-GA_P01 [ 10220 ]
          Fix Version/s 1.8.3 [ 10211 ]
          Assignee Priority P1
          Assignee Mark Collette [ mark.collette ]
          Arran Mccullough made changes -
          Salesforce Case [5007000000AplGc] [5007000000AplGc, 5007000000Aqdqe]
          Arran Mccullough made changes -
          Salesforce Case [5007000000AplGc, 5007000000Aqdqe] [5007000000AplGc]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #20864 Thu Mar 04 15:47:14 MST 2010 mark.collette ICE-5351 : DisplayEvent.getContextValue() can only be cast to a String object with panelTooltip
          Files Changed
          Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/paneltooltip/PanelTooltip.java
          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
          Mark Collette made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Joanne Bai made changes -
          Assignee Mark Collette [ mark.collette ] Joanne Bai [ joanne_bai ]
          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
          Joanne Bai made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          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
          Joanne Bai made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Ken Fyten made changes -
          Assignee Joanne Bai [ joanne_bai ] Mark Collette [ mark.collette ]
          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.
          Mark Collette made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          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.
          Joanne Bai made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Evgheni Sadovoi made changes -
          Link This issue is duplicated by ICE-7308 [ ICE-7308 ]

            People

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

              Dates

              • Created:
                Updated:
                Resolved: