ICEfaces
  1. ICEfaces
  2. ICE-8162

ACE:chart issues with tab sets

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.1.0.BETA1
    • Fix Version/s: 3.1.0.BETA2, 3.1
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      Firefox 12.0, Chrome 19.0, Tomcat 7
    • Assignee Priority:
      P1

      Description

      When placed inside an ACE tabset, ace:chart fails to display. Chart displays fine in ICE tabset.

      When placed inside either an ACE or ICE tabset, attempting to switch the tab away from where the chart is/should be results in an internal server error, forcing a refresh of the page.

      Server Internal Error:
      [window] received error message [code: 200]: <?xml version='1.0' encoding='UTF-8'?> <partial-response><error><error-name>class java.lang.NullPointerException</error-name><error-message><![CDATA[]]></error-message></error><changes><extension aceCallbackParam="validationFailed">{"validationFailed":false}</extension></changes></partial-response>

      The error does not appear in the same tabset code when chart is removed.

      Code that I'm using can be found at http://server.ice:8888/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/chart/

      (ICE tabset issues may or may not be connected with http://jira.icesoft.org/browse/ICE-7250 )

        Activity

        Hide
        Ken Fyten added a comment -

        Let's focus on the ace:tabSet aspect for now. The ice:panelTabSet can be retested once ace is working.

        Show
        Ken Fyten added a comment - Let's focus on the ace:tabSet aspect for now. The ice:panelTabSet can be retested once ace is working.
        Hide
        Nils Lundquist added a comment -

        This is caused by the first tab in an ace:tabset being rendered as hidden, and then dynamically toggled to visible by initialization of the tabset.

        Just like the scrollable ace:datatable, the JS required to initialize the component returns incorrect values if run while the component is not visible.

        And just like the scrollable table, the solution here is a callback on component init that polls until the component is visible before following through with init.

        Alternatively the tabset could be redesigned to render the first tab as visible, though this wouldn't solve cases where the chart is hidden in a dialog or when rendered into a cached inactive tab.

        Show
        Nils Lundquist added a comment - This is caused by the first tab in an ace:tabset being rendered as hidden, and then dynamically toggled to visible by initialization of the tabset. Just like the scrollable ace:datatable, the JS required to initialize the component returns incorrect values if run while the component is not visible. And just like the scrollable table, the solution here is a callback on component init that polls until the component is visible before following through with init. Alternatively the tabset could be redesigned to render the first tab as visible, though this wouldn't solve cases where the chart is hidden in a dialog or when rendered into a cached inactive tab.
        Hide
        Nils Lundquist added a comment - - edited

        Revision #29215
        Committed by nils.lundquist
        Moments ago
        ICE-8162 - Added polling when in hidden cases to complete initialization after being shown.

        Beginning investigation into NPE issue on tab change.

        Show
        Nils Lundquist added a comment - - edited Revision #29215 Committed by nils.lundquist Moments ago ICE-8162 - Added polling when in hidden cases to complete initialization after being shown. Beginning investigation into NPE issue on tab change.
        Hide
        Nils Lundquist added a comment -

        Stack traceless NPE issue caused by inline code to split null string.

        Revision #29218
        Committed by nils.lundquist
        Moments ago
        ICE-8162 - Check for request param before split.

        Show
        Nils Lundquist added a comment - Stack traceless NPE issue caused by inline code to split null string. Revision #29218 Committed by nils.lundquist Moments ago ICE-8162 - Check for request param before split.

          People

          • Assignee:
            Nils Lundquist
            Reporter:
            Brennan Mckinney
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: