ICEfaces
  1. ICEfaces
  2. ICE-8441

ace:chart - if creation of series is not done in shorthand form, instantiation error is thrown during Chart rendering

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.2
    • Component/s: None
    • Labels:
      None
    • Environment:
      IF 3.x

      Description

      If there is no explicit default configuration series for the Chart, the default chart type is inferred from the type of the first data series and rendered by creating a new config series instance of that type and using it as an implied default config.

      However, this code attempted to access the series superclass (as is required when declaring these instances using shorthand, because shorthand creates class nested subclasses when used), and when not using shorthand this would access the abstract ChartSeries class which cannot be instantiated, causing an error.

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #30399 Tue Aug 07 13:28:58 MDT 2012 nils.lundquist ICE-8441 - ChartSeries subclasses correctly referenced during default type interpretation. ChartSeries itself had been referenced as a product of a fix required for referencing of the ChartSeries subclasses when the init shorthand is used.
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/chart/ChartRenderer.java
        Nils Lundquist created issue -
        Nils Lundquist made changes -
        Field Original Value New Value
        Assignee Nils Lundquist [ nils.lundquist ]
        Hide
        Nils Lundquist added a comment -

        A sample of the shorthand in question is:

        List<ChartSeries> series = new ArrayList() {{
        add(new CartesianSeries() {{
        ....
        }});
        }}

        Show
        Nils Lundquist added a comment - A sample of the shorthand in question is: List<ChartSeries> series = new ArrayList() {{ add(new CartesianSeries() {{ .... }}); }}
        Hide
        Nils Lundquist added a comment -

        Revision #30399
        Committed by nils.lundquist
        Moments ago
        ICE-8441 - ChartSeries subclasses correctly referenced during default type interpretation. ChartSeries itself had been referenced as a product of a fix required for referencing of the ChartSeries subclasses when the init shorthand is used.

        Show
        Nils Lundquist added a comment - Revision #30399 Committed by nils.lundquist Moments ago ICE-8441 - ChartSeries subclasses correctly referenced during default type interpretation. ChartSeries itself had been referenced as a product of a fix required for referencing of the ChartSeries subclasses when the init shorthand is used.
        Nils Lundquist made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s 3.2 [ 10338 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: