From 526ff727863bba596f3b0a26c7ea2972a9dfecfe Mon Sep 17 00:00:00 2001
From: zheng <zheng@finereport.com>
Date: Fri, 22 Feb 2019 17:05:09 +0800
Subject: [PATCH] =?UTF-8?q?=E5=A4=A7=E5=B1=8F=E6=8F=92=E4=BB=B6jartime?=
 =?UTF-8?q?=E6=9B=B4=E6=96=B0=20=E9=A1=BA=E4=BE=BF=E8=A7=A3=E5=86=B3?=
 =?UTF-8?q?=E4=B9=8B=E5=89=8D=E7=9A=84=E4=B8=80=E4=BA=9B=E4=BB=A3=E7=A0=81?=
 =?UTF-8?q?=E8=B4=A8=E9=87=8F?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../AbstractExtendedChartReportDataPane.java  | 55 ++++++++++++-------
 .../AbstractExtendedChartTableDataPane.java   | 44 ++++++++++-----
 .../style/HeatMapRangeLegendPane.java         | 14 ++++-
 .../component/GradualIntervalConfigPane.java  | 18 ++----
 4 files changed, 82 insertions(+), 49 deletions(-)

diff --git a/designer-chart/src/main/java/com/fr/extended/chart/AbstractExtendedChartReportDataPane.java b/designer-chart/src/main/java/com/fr/extended/chart/AbstractExtendedChartReportDataPane.java
index e42707a62a..b8758b9f5e 100644
--- a/designer-chart/src/main/java/com/fr/extended/chart/AbstractExtendedChartReportDataPane.java
+++ b/designer-chart/src/main/java/com/fr/extended/chart/AbstractExtendedChartReportDataPane.java
@@ -12,8 +12,8 @@ import com.fr.design.mainframe.chart.gui.data.report.AbstractReportDataContentPa
 import javax.swing.BorderFactory;
 import javax.swing.JPanel;
 import javax.swing.SwingConstants;
-import java.awt.Component;
 import java.awt.BorderLayout;
+import java.awt.Component;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;
@@ -92,38 +92,53 @@ public abstract class AbstractExtendedChartReportDataPane<T extends AbstractData
 
         if (chart.getFilterDefinition() instanceof ExtendedReportDataSet) {
             ExtendedReportDataSet dataSet = (ExtendedReportDataSet) chart.getFilterDefinition();
-            AbstractDataConfig dataConfig = dataSet.getDataConfig();
 
-            if (dataConfig != null) {
-                populate((T) dataConfig);
+            populateDataSet(dataSet);
+        }
+    }
+
+    public void populateDataSet(DataSet dataSet) {
+
+        if (dataSet == null) {
+            return;
+        }
+
+        AbstractDataConfig dataConfig = dataSet.getDataConfig();
+
+        if (dataConfig != null) {
+            populate((T) dataConfig);
 
-                if (hasCustomFieldPane() && dataConfig.getCustomFields().size() == 2) {
-                    populateField(seriesPane, dataConfig.getCustomFields().get(0));
-                    populateField(valuePane, dataConfig.getCustomFields().get(1));
-                }
+            if (hasCustomFieldPane() && dataConfig.getCustomFields().size() == 2) {
+                populateField(seriesPane, dataConfig.getCustomFields().get(0));
+                populateField(valuePane, dataConfig.getCustomFields().get(1));
             }
         }
     }
 
+    public ExtendedReportDataSet updateDataSet() {
+        ExtendedReportDataSet dataSet = new ExtendedReportDataSet();
+
+        AbstractDataConfig dataConfig = update();
+        dataSet.setDataConfig(dataConfig);
+
+        List<ExtendedField> fieldList = new ArrayList<ExtendedField>();
+        if (hasCustomFieldPane()) {
+            fieldList.add(new ExtendedField(seriesPane.updateBean()));
+            fieldList.add(new ExtendedField(valuePane.updateBean()));
+        }
+        dataConfig.setCustomFields(fieldList);
+
+        return dataSet;
+    }
+
 
     @Override
     public void updateBean(ChartCollection ob) {
         if (ob != null) {
             Chart chart = ob.getSelectedChart();
             if (chart != null) {
-                ExtendedReportDataSet dataSet = new ExtendedReportDataSet();
-
-                AbstractDataConfig dataConfig = update();
-                dataSet.setDataConfig(dataConfig);
-
-                List<ExtendedField> fieldList = new ArrayList<ExtendedField>();
-                if (hasCustomFieldPane()) {
-                    fieldList.add(new ExtendedField(seriesPane.updateBean()));
-                    fieldList.add(new ExtendedField(valuePane.updateBean()));
-                }
-                dataConfig.setCustomFields(fieldList);
 
-                chart.setFilterDefinition(dataSet);
+                chart.setFilterDefinition(updateDataSet());
             }
         }
     }
diff --git a/designer-chart/src/main/java/com/fr/extended/chart/AbstractExtendedChartTableDataPane.java b/designer-chart/src/main/java/com/fr/extended/chart/AbstractExtendedChartTableDataPane.java
index 0771b749a5..ac1799e745 100644
--- a/designer-chart/src/main/java/com/fr/extended/chart/AbstractExtendedChartTableDataPane.java
+++ b/designer-chart/src/main/java/com/fr/extended/chart/AbstractExtendedChartTableDataPane.java
@@ -93,17 +93,40 @@ public abstract class AbstractExtendedChartTableDataPane<T extends AbstractDataC
         if (chart.getFilterDefinition() instanceof ExtendedTableDataSet) {
             ExtendedTableDataSet dataSet = (ExtendedTableDataSet) chart.getFilterDefinition();
 
-            AbstractDataConfig dataConfig = dataSet.getDataConfig();
-            if (dataConfig != null) {
-                populate((T) dataConfig);
+            populateDataSet(dataSet);
 
-                if (customFieldComboBoxPane != null) {
-                    customFieldComboBoxPane.populateBean(dataConfig);
-                }
+        }
+
+    }
+
+    public void populateDataSet(DataSet dataSet) {
+
+        if (dataSet == null) {
+            return;
+        }
+
+        AbstractDataConfig dataConfig = dataSet.getDataConfig();
+
+        if (dataConfig != null) {
+            populate((T) dataConfig);
+
+            if (customFieldComboBoxPane != null) {
+                customFieldComboBoxPane.populateBean(dataConfig);
             }
+        }
+    }
+
+    public ExtendedTableDataSet updateDataSet() {
+
+        ExtendedTableDataSet dataSet = new ExtendedTableDataSet();
 
+        dataSet.setDataConfig(update());
+
+        if (customFieldComboBoxPane != null) {
+            customFieldComboBoxPane.updateBean(dataSet.getDataConfig());
         }
 
+        return dataSet;
     }
 
 
@@ -113,14 +136,7 @@ public abstract class AbstractExtendedChartTableDataPane<T extends AbstractDataC
             Chart chart = ob.getSelectedChart();
             if (chart != null) {
 
-                ExtendedTableDataSet dataSet = new ExtendedTableDataSet();
-                dataSet.setDataConfig(update());
-
-                if (customFieldComboBoxPane != null) {
-                    customFieldComboBoxPane.updateBean(dataSet.getDataConfig());
-                }
-
-                chart.setFilterDefinition(dataSet);
+                chart.setFilterDefinition(updateDataSet());
             }
         }
     }
diff --git a/designer-chart/src/main/java/com/fr/van/chart/designer/style/HeatMapRangeLegendPane.java b/designer-chart/src/main/java/com/fr/van/chart/designer/style/HeatMapRangeLegendPane.java
index 61aa49829f..db0d5c8641 100644
--- a/designer-chart/src/main/java/com/fr/van/chart/designer/style/HeatMapRangeLegendPane.java
+++ b/designer-chart/src/main/java/com/fr/van/chart/designer/style/HeatMapRangeLegendPane.java
@@ -1,12 +1,16 @@
 package com.fr.van.chart.designer.style;
 
 import com.fr.design.gui.frpane.AbstractAttrNoScrollPane;
+import com.fr.design.gui.frpane.UINumberDragPane;
 import com.fr.design.gui.ibutton.UIButtonGroup;
+import com.fr.design.gui.ilable.BoldFontTextLabel;
+import com.fr.design.mainframe.chart.gui.ColorSelectBoxWithOutTransparent;
 import com.fr.design.mainframe.chart.gui.style.series.MapColorPickerPaneWithFormula;
-
 import com.fr.plugin.chart.type.LegendType;
+import com.fr.van.chart.designer.style.axis.component.MinMaxValuePaneWithOutTick;
 import com.fr.van.chart.range.component.GradualIntervalConfigPane;
 import com.fr.van.chart.range.component.GradualLegendPane;
+import com.fr.van.chart.range.component.LegendGradientBar;
 import com.fr.van.chart.range.component.SectionIntervalConfigPaneWithOutNum;
 import com.fr.van.chart.range.component.SectionLegendPane;
 
@@ -46,8 +50,12 @@ public class HeatMapRangeLegendPane extends VanChartRangeLegendPane {
             protected GradualIntervalConfigPane createGradualIntervalConfigPane() {
                 return new GradualIntervalConfigPane(){
                     @Override
-                    protected Component[][] getPaneComponents() {
-                        return super.getPaneComponentsWithOutTheme();
+                    protected Component[][] getPaneComponents(MinMaxValuePaneWithOutTick minMaxValuePane, ColorSelectBoxWithOutTransparent colorSelectBox, UINumberDragPane numberDragPane, LegendGradientBar legendGradientBar) {
+                        return new Component[][]{
+                                new Component[]{minMaxValuePane, null},
+                                new Component[]{new BoldFontTextLabel(com.fr.design.i18n.Toolkit.i18nText("Fine-Design_Chart_Value_Divided_Stage")), numberDragPane},
+                                new Component[]{null, legendGradientBar}
+                        };
                     }
                 };
             }
diff --git a/designer-chart/src/main/java/com/fr/van/chart/range/component/GradualIntervalConfigPane.java b/designer-chart/src/main/java/com/fr/van/chart/range/component/GradualIntervalConfigPane.java
index b930c689d0..1a0413ffaa 100644
--- a/designer-chart/src/main/java/com/fr/van/chart/range/component/GradualIntervalConfigPane.java
+++ b/designer-chart/src/main/java/com/fr/van/chart/range/component/GradualIntervalConfigPane.java
@@ -4,7 +4,6 @@ import com.fr.design.gui.frpane.UINumberDragPane;
 import com.fr.design.gui.ilable.BoldFontTextLabel;
 import com.fr.design.layout.TableLayout;
 import com.fr.design.mainframe.chart.gui.ColorSelectBoxWithOutTransparent;
-
 import com.fr.plugin.chart.range.GradualIntervalConfig;
 import com.fr.van.chart.designer.TableLayout4VanChartHelper;
 import com.fr.van.chart.designer.style.axis.component.MinMaxValuePaneWithOutTick;
@@ -23,7 +22,7 @@ public class GradualIntervalConfigPane extends JPanel{
     //主题颜色
     private ColorSelectBoxWithOutTransparent colorSelectBox;
     //划分阶段
-    UINumberDragPane numberDragPane;
+    private UINumberDragPane numberDragPane;
     //渐变色编辑器
     private LegendGradientBar legendGradientBar;
 
@@ -58,7 +57,7 @@ public class GradualIntervalConfigPane extends JPanel{
         double[] col = new double[]{f, e};
         double[] row = new double[]{p, p, p, p};
 
-        Component[][] components = getPaneComponents();
+        Component[][] components = getPaneComponents(minMaxValuePane, colorSelectBox, numberDragPane, legendGradientBar);
 
         //控件承载面板
         JPanel contentPane = TableLayout4VanChartHelper.createGapTableLayoutPane(components,row,col);
@@ -70,7 +69,10 @@ public class GradualIntervalConfigPane extends JPanel{
         return new LegendGradientBar();
     }
 
-    protected Component[][] getPaneComponents(){
+    protected Component[][] getPaneComponents(MinMaxValuePaneWithOutTick minMaxValuePane,
+                                              ColorSelectBoxWithOutTransparent colorSelectBox,
+                                              UINumberDragPane numberDragPane,
+                                              LegendGradientBar legendGradientBar) {
         return new Component[][]{
                 new Component[]{minMaxValuePane, null},
                 new Component[]{new BoldFontTextLabel(com.fr.design.i18n.Toolkit.i18nText("Fine-Design_Chart_Subject_Color")), colorSelectBox},
@@ -79,14 +81,6 @@ public class GradualIntervalConfigPane extends JPanel{
         };
     }
 
-    protected Component[][] getPaneComponentsWithOutTheme(){
-        return new Component[][]{
-                new Component[]{minMaxValuePane, null},
-                new Component[]{new BoldFontTextLabel(com.fr.design.i18n.Toolkit.i18nText("Fine-Design_Chart_Value_Divided_Stage")), numberDragPane},
-                new Component[]{null, legendGradientBar},
-        };
-    }
-
     public void populate(GradualIntervalConfig intervalConfig){
         minMaxValuePane.populate(intervalConfig.getMinAndMaxValue());