ICEfaces
  1. ICEfaces
  2. ICE-1651

Facelets component-showcase combining charts breaks on revisit

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.6DR#5
          • Fix Version/s: 1.6DR#6, 1.6
          • Component/s: Sample Apps
          • Labels:
            None
          • Environment:
            WinXP SP2, Firefox 1.5.0.11, JBoss 4.0.4 GA
          • Affects:
            Sample App./Tutorial
          • Workaround Exists:
            Yes
          • Workaround Description:
            Use the JSP component-showcase.

            Description

            Build the Facelets version of component showcase. Go to the Components/Charts/Combining Charts section. Notice how the chart shows up fine. Click on the navigation tree to see some other display, and then click on Combining Charts again. This second view will not work properly, the chart image won't display.

              Activity

              Hide
              Permalink
              Mark Collette added a comment -

              The JSP version of component-showcase uses static includes, and the Facelets version uses dynamic includes, which means that the JSP version has on monolithic long-lived component tree, whereas the Facelets version's chart component gets remade each time you visit it, which the associated bean did not account for.

              While I was investigating the issue, I found that OutputChart did not implement the saveState and restoreState functionality, so I added that, plus some random tweaks.

              Subversion 14007
              icefaces\samples\component-showcase\src\com\icesoft\icefaces\samples\showcase\components\charts\CombinedChartBean.java
              icefaces\component\src\com\icesoft\faces\component\outputchart\OutputChart.java
              icefaces\component\src\com\icesoft\faces\component\outputchart\OutputChartRenderer.java

              Show
              Mark Collette added a comment - The JSP version of component-showcase uses static includes, and the Facelets version uses dynamic includes, which means that the JSP version has on monolithic long-lived component tree, whereas the Facelets version's chart component gets remade each time you visit it, which the associated bean did not account for. While I was investigating the issue, I found that OutputChart did not implement the saveState and restoreState functionality, so I added that, plus some random tweaks. Subversion 14007 icefaces\samples\component-showcase\src\com\icesoft\icefaces\samples\showcase\components\charts\CombinedChartBean.java icefaces\component\src\com\icesoft\faces\component\outputchart\OutputChart.java icefaces\component\src\com\icesoft\faces\component\outputchart\OutputChartRenderer.java

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: