CHART-13666 图表自动类型推荐

designer-base/src/main/java/com/fr/design/ExtraDesignClassManager.java

@@ -8,6 +8,7 @@ import com.fr.base.BaseUtils;
 import com.fr.common.annotations.Open;
 import com.fr.design.data.datapane.TableDataNameObjectCreator;
 import com.fr.design.fun.CellWidgetOptionProvider;
+import com.fr.design.fun.ChartWidgetOptionProvider;
 import com.fr.design.fun.FormWidgetOptionProvider;
 import com.fr.design.fun.MobileWidgetStyleProvider;
 import com.fr.design.fun.ParameterWidgetOptionProvider;
@@ -193,6 +194,42 @@ public class ExtraDesignClassManager extends AbstractExtraClassManager implement
         return result.toArray(new WidgetOption[result.size()]);
     }
 
+    public Map<Class<? extends Widget>, Class<?>> getChartWidgetOptionsMap() {
+        Set<ChartWidgetOptionProvider> set = getArray(ChartWidgetOptionProvider.XML_TAG);
+        Map<Class<? extends Widget>, Class<?>> map = new HashMap<>();
+        for (ChartWidgetOptionProvider provider : set) {
+            map.put(provider.classForWidget(), provider.appearanceForWidget());
+        }
+        return map;
+    }
+
+    public WidgetOption[] getBeforeChartWidgetOptions() {
+        return getChartUnits(true);
+    }
+
+    public WidgetOption[] getAfterChartWidgetOptions() {
+        return getChartUnits(false);
+    }
+
+    private WidgetOption[] getChartUnits(boolean isBefore) {
+        Set<ChartWidgetOptionProvider> set = getArray(ChartWidgetOptionProvider.XML_TAG);
+        if (set.isEmpty()) {
+            return new WidgetOption[0];
+        }
+        Set<WidgetOption> result = new HashSet<>();
+        for (ChartWidgetOptionProvider provider : set) {
+            if (provider.isBefore() == isBefore) {
+                WidgetOption option = WidgetOptionFactory.createByWidgetClass(
+                        provider.nameForWidget(),
+                        BaseUtils.readIcon(provider.iconPathForWidget()),
+                        provider.classForWidget()
+                );
+                result.add(option);
+            }
+        }
+        return result.toArray(new WidgetOption[result.size()]);
+    }
+
 
     public WidgetOption[] getCellWidgetOptions() {
         Set<CellWidgetOptionProvider> set = getArray(CellWidgetOptionProvider.XML_TAG);

designer-base/src/main/java/com/fr/design/fun/ChartWidgetOptionProvider.java

@@ -0,0 +1,14 @@
+package com.fr.design.fun;
+
+/**
+ * @author Bjorn
+ * @version 10.0
+ * Created by Bjorn on 2020-06-12
+ */
+public interface ChartWidgetOptionProvider extends ParameterWidgetOptionProvider {
+
+    String XML_TAG = "ChartWidgetOptionProvider";
+
+    //在图表区域的开头还是结尾插入
+    boolean isBefore();
+}

designer-base/src/main/java/com/fr/design/fun/impl/AbstractChartWidgetOptionProvider.java

@@ -0,0 +1,30 @@
+package com.fr.design.fun.impl;
+
+import com.fr.design.fun.ChartWidgetOptionProvider;
+import com.fr.design.fun.FormWidgetOptionProvider;
+import com.fr.stable.fun.impl.AbstractProvider;
+import com.fr.stable.fun.mark.API;
+
+/**
+ * @author Bjorn
+ * @version 10.0
+ * Created by Bjorn on 2020-06-12
+ */
+@API(level = FormWidgetOptionProvider.CURRENT_LEVEL)
+public abstract class AbstractChartWidgetOptionProvider extends AbstractProvider implements ChartWidgetOptionProvider {
+
+    public int currentAPILevel() {
+        return CURRENT_LEVEL;
+    }
+
+    @Override
+    public String mark4Provider() {
+        return getClass().getName();
+    }
+
+    @Override
+    public boolean isBefore() {
+        return false;
+    }
+
+}

designer-chart/src/main/java/com/fr/design/ChartTypeInterfaceManager.java

@@ -104,6 +104,7 @@ import java.util.Iterator;
 import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Map;
+import java.awt.event.ActionListener;
 
 import static com.fr.chart.charttypes.ChartTypeManager.DEFAULT_PRIORITY;
 import static com.fr.chart.charttypes.ChartTypeManager.DEPRECATED_CHART_PRIORITY;
@@ -259,17 +260,21 @@ public class ChartTypeInterfaceManager implements ExtraChartDesignClassManagerPr
      *
      * @param paneList pane容器
      */
-    public void addPlotTypePaneList(List<FurtherBasicBeanPane<? extends ChartProvider>> paneList, Map<String, Map<String, FurtherBasicBeanPane<? extends ChartProvider>>> allChartTypePane) {
+    public void addPlotTypePaneList(List<FurtherBasicBeanPane<? extends ChartProvider>> paneList,
+                                    Map<String, Map<String, FurtherBasicBeanPane<? extends ChartProvider>>> allChartTypePane,
+                                    ActionListener autoButtonListener) {
 
         List<Integer> priorityList = getPriorityInOrder();
         for (Integer aPriorityList : priorityList) {
             String priority = String.valueOf(aPriorityList);
-            addPlotTypePaneList(priority, paneList, allChartTypePane);
+            addPlotTypePaneList(priority, paneList, allChartTypePane, autoButtonListener);
         }
     }
 
 
-    public void addPlotTypePaneList(String priority, List<FurtherBasicBeanPane<? extends ChartProvider>> paneList, Map<String, Map<String, FurtherBasicBeanPane<? extends ChartProvider>>> allChartTypePane) {
+    public void addPlotTypePaneList(String priority, List<FurtherBasicBeanPane<? extends ChartProvider>> paneList,
+                                    Map<String, Map<String, FurtherBasicBeanPane<? extends ChartProvider>>> allChartTypePane,
+                                    ActionListener autoButtonListener) {
 
         if (chartTypeInterfaces != null && chartTypeInterfaces.containsKey(priority)) {
 
@@ -286,6 +291,7 @@ public class ChartTypeInterfaceManager implements ExtraChartDesignClassManagerPr
                         continue;
                     }
                     pane.reLayout(plotID);
+                    pane.registerButtonListener(autoButtonListener);
                     paneList.add(pane);
 
                     if (allChartTypePane.get(priority) == null) {

designer-chart/src/main/java/com/fr/design/chart/AutoChartDialog.java

@@ -1,7 +1,11 @@
 package com.fr.design.chart;
 
 import com.fr.base.chart.BaseChartCollection;
+import com.fr.base.chart.chartdata.TopDefinitionProvider;
 import com.fr.chart.chartattr.ChartCollection;
+import com.fr.chartx.TwoTuple;
+import com.fr.design.i18n.Toolkit;
+import com.fr.plugin.chart.vanchart.VanChart;
 
 import java.awt.Component;
 import java.awt.Dialog;
@@ -46,10 +50,30 @@ public class AutoChartDialog extends ChartDialog {
         };
     }
 
+    protected String getDialogTitle() {
+        return Toolkit.i18nText("Fine-Design_Chart_M_Popup_Auto_Chart_Type");
+    }
+
     /**
      * 更新新建的图表 ChartCollection
      */
     public void populate(BaseChartCollection cc) {
         super.populate(cc);
+        ChartCollection chartCollection = (ChartCollection) getChartCollection();
+        VanChart vanChart = chartCollection.getSelectedChartProvider(VanChart.class);
+        if (vanChart == null) {
+            return;
+        }
+        TopDefinitionProvider filterDefinition = vanChart.getFilterDefinition();
+        if (filterDefinition == null) {
+            return;
+        }
+        TwoTuple<String, String[]> tableNameAndDataFields = filterDefinition.getTableNameAndDataFields();
+        if (tableNameAndDataFields == null) {
+            return;
+        }
+        String tableName = tableNameAndDataFields.getFirst();
+        String[] dataFields = tableNameAndDataFields.getSecond();
+        autoChartTypePane.populate(tableName, dataFields);
     }
 }

designer-chart/src/main/java/com/fr/design/chart/ChartDialog.java

@@ -5,6 +5,7 @@ import com.fr.chart.chartattr.ChartCollection;
 import com.fr.design.dialog.BasicDialog;
 import com.fr.design.gui.chart.MiddleChartDialog;
 import com.fr.design.gui.ibutton.UIButton;
+import com.fr.design.i18n.Toolkit;
 import com.fr.design.utils.gui.GUICoreUtils;
 import com.fr.third.joda.time.DateTime;
 
@@ -46,7 +47,7 @@ public class ChartDialog extends MiddleChartDialog {
         createTime = DateTime.now().toString("yyyy-MM-dd HH:mm:ss");
         this.setModal(true);
         this.setLayout(new BorderLayout());
-        setTitle(com.fr.design.i18n.Toolkit.i18nText("Fine-Design_Chart_M_Popup_Chart_Type"));
+        setTitle(getDialogTitle());
 
         this.applyClosingAction();
         this.applyEscapeAction();
@@ -73,6 +74,10 @@ public class ChartDialog extends MiddleChartDialog {
         GUICoreUtils.setWindowCenter(getOwner(), this);
     }
 
+    protected String getDialogTitle() {
+        return Toolkit.i18nText("Fine-Design_Chart_M_Popup_Chart_Type");
+    }
+
     protected Component initCenterPane() {
         chartTypePane = new ChartTypePane();
         return chartTypePane;

designer-chart/src/main/java/com/fr/design/mainframe/chart/gui/ChartTypePane.java

@@ -9,9 +9,7 @@ import com.fr.design.dialog.BasicScrollPane;
 import com.fr.design.dialog.DialogActionAdapter;
 import com.fr.design.gui.chart.MiddleChartDialog;
 import com.fr.design.gui.frpane.UIComboBoxPane;
-import com.fr.design.gui.ibutton.UIButton;
 import com.fr.design.gui.icombobox.UIComboBox;
-import com.fr.design.i18n.Toolkit;
 import com.fr.design.mainframe.DesignerContext;
 import com.fr.design.mainframe.chart.AbstractChartAttrPane;
 import com.fr.design.mainframe.chart.ChartEditPane;
@@ -39,6 +37,7 @@ import java.awt.event.ItemListener;
 
 /**
  * 图表 属性表, 类型选择 界面.
+ *
  * @author kunsnat E-mail:kunsnat@gmail.com
  * @version 创建时间：2012-12-26 上午10:56:51
  */
@@ -47,10 +46,11 @@ public class ChartTypePane extends AbstractChartAttrPane{
     private ChartTypeButtonPane buttonPane;
     private ChartEditPane editPane;
     private ChartCollection editingCollection;
-	private UIButton autoButton;
+    private ActionListener autoButtonListener;
 
     @Override
     protected JPanel createContentPane() {
+        initButtonListener();
         JPanel content = new JPanel(new BorderLayout());
 
         buttonPane = new ChartTypeButtonPane(this);
@@ -88,11 +88,6 @@ public class ChartTypePane extends AbstractChartAttrPane{
 
         buttonPane.setEditingChartPane(chartTypeComBox);
 
-		if(ChartTypeManager.AUTO_CHART) {
-			initAutoButton();
-			content.add(autoButton, BorderLayout.SOUTH);
-		}
-
         return content;
     }
 
@@ -105,16 +100,15 @@ public class ChartTypePane extends AbstractChartAttrPane{
 
     /**
      * 界面标题
+     *
      * @return 界面标题
      */
     public String title4PopupWindow() {
         return PaneTitleConstants.CHART_TYPE_TITLE;
     }
 
-	private void initAutoButton(){
-		//TODO @Bjorn 视觉样式优化
-		autoButton = new UIButton(Toolkit.i18nText("Fine-Design_Chart_M_Popup_Auto_Chart_Type"));
-		autoButton.addActionListener(new ActionListener() {
+    public void initButtonListener() {
+        autoButtonListener = new ActionListener() {
             @Override
             public void actionPerformed(ActionEvent e) {
                 final MiddleChartDialog autoChartDialog = DesignModuleFactory.getAutoChartDialog(DesignerContext.getDesignerFrame());
@@ -127,7 +121,7 @@ public class ChartTypePane extends AbstractChartAttrPane{
                 });
                 autoChartDialog.setVisible(true);
             }
-		});
+        };
     }
 
     class ComboBoxPane extends UIComboBoxPane<ChartProvider> {
@@ -137,7 +131,7 @@ public class ChartTypePane extends AbstractChartAttrPane{
         protected List<FurtherBasicBeanPane<? extends ChartProvider>> initPaneList() {
             List<FurtherBasicBeanPane<? extends ChartProvider>> paneList = new ArrayList<FurtherBasicBeanPane<? extends ChartProvider>>();
             allChartTypePane = new LinkedHashMap<String, Map<String, FurtherBasicBeanPane<? extends ChartProvider>>>();
-			ChartTypeInterfaceManager.getInstance().addPlotTypePaneList(paneList, allChartTypePane);
+            ChartTypeInterfaceManager.getInstance().addPlotTypePaneList(paneList, allChartTypePane, autoButtonListener);
             return paneList;
         }
 
@@ -150,6 +144,7 @@ public class ChartTypePane extends AbstractChartAttrPane{
          * 不同图表切换分同一个selected的不同图表切换和不同selected的不同图表切换
          * 如果是切换图表的某个图表发生变化，则collection的选择下标不会变
          * 如果是切换图表的不同图表之间切换，则collection的选择下标会改变
+         *
          * @param chart
          */
         public void updateBean(ChartProvider chart) {
@@ -323,6 +318,7 @@ public class ChartTypePane extends AbstractChartAttrPane{
 
     /**
      * 面板重构
+     *
      * @param lastPlotID
      * @param collection
      */
@@ -373,6 +369,7 @@ public class ChartTypePane extends AbstractChartAttrPane{
 
     /**
      * 所有图表的类型界面
+     *
      * @return 类型界面
      */
     public FurtherBasicBeanPane[] getPaneList() {
@@ -381,6 +378,7 @@ public class ChartTypePane extends AbstractChartAttrPane{
 
     /**
      * 当前选中的图表的index
+     *
      * @return 当前选中的图表的index
      */
     public int getSelectedIndex() {
@@ -389,6 +387,7 @@ public class ChartTypePane extends AbstractChartAttrPane{
 
     /**
      * 返回选中的图表的index
+     *
      * @return 选中的图标的序号
      */
     public int getSelectedChartIndex() {
@@ -397,6 +396,7 @@ public class ChartTypePane extends AbstractChartAttrPane{
 
     /**
      * 设置下编辑的面板
+     *
      * @param currentEditPane 设置下编辑的面板
      */
     public void registerChartEditPane(ChartEditPane currentEditPane) {

designer-chart/src/main/java/com/fr/design/mainframe/chart/gui/type/AbstractChartTypePane.java

@@ -22,6 +22,7 @@ import java.util.List;
 import java.awt.BorderLayout;
 import java.awt.Color;
 import java.awt.Component;
+import java.awt.event.ActionListener;
 
 
 public abstract class AbstractChartTypePane<T extends ChartProvider> extends FurtherBasicBeanPane<T> {
@@ -240,4 +241,8 @@ public abstract class AbstractChartTypePane<T extends ChartProvider> extends Fur
     public ChartProvider getDefaultChart() {
         return BarIndependentChart.barChartTypes[0];
     }
+
+    public void registerButtonListener(ActionListener autoButtonListener) {
+
+    }
 }

designer-chart/src/main/java/com/fr/van/chart/custom/VanChartCustomPlotPane.java

@@ -61,7 +61,8 @@ public class VanChartCustomPlotPane extends AbstractVanChartTypePane {
         return new Component[][]{
                 new Component[]{typePane},
                 new Component[]{stylePane},
-                new Component[]{contentPane}
+                new Component[]{contentPane},
+                new Component[]{buttonPane},
         };
     }
 

designer-chart/src/main/java/com/fr/van/chart/designer/type/AbstractVanChartTypePane.java

@@ -1,5 +1,6 @@
 package com.fr.van.chart.designer.type;
 
+import com.fr.base.BaseUtils;
 import com.fr.chart.base.AttrFillStyle;
 import com.fr.chart.chartattr.Chart;
 import com.fr.chart.chartattr.Legend;
@@ -7,13 +8,16 @@ import com.fr.chart.chartattr.Plot;
 import com.fr.chart.chartglyph.ConditionAttr;
 import com.fr.chart.chartglyph.ConditionCollection;
 import com.fr.chart.chartglyph.DataSheet;
+import com.fr.chart.charttypes.ChartTypeManager;
 import com.fr.chartx.data.AbstractDataDefinition;
 import com.fr.chartx.data.ChartDataDefinitionProvider;
 import com.fr.chartx.data.field.AbstractColumnFieldCollection;
 import com.fr.chartx.data.field.diff.MultiCategoryColumnFieldCollection;
 import com.fr.design.ChartTypeInterfaceManager;
+import com.fr.design.gui.ibutton.UIButton;
 import com.fr.design.gui.icheckbox.UICheckBox;
 import com.fr.design.gui.ilable.MultilineLabel;
+import com.fr.design.i18n.Toolkit;
 import com.fr.design.mainframe.chart.gui.type.AbstractChartTypePane;
 import com.fr.design.mainframe.chart.gui.type.ChartImagePane;
 import com.fr.design.mainframe.chart.info.ChartInfoCollector;
@@ -25,21 +29,53 @@ import com.fr.plugin.chart.base.VanChartTools;
 import com.fr.plugin.chart.base.VanChartZoom;
 import com.fr.plugin.chart.vanchart.VanChart;
 
+import javax.swing.BorderFactory;
 import javax.swing.JPanel;
 import java.awt.BorderLayout;
 import java.awt.Color;
 import java.awt.Component;
+import java.awt.Dimension;
+import java.awt.event.ActionListener;
 
 public abstract class AbstractVanChartTypePane extends AbstractChartTypePane<Chart> {
     private static final long serialVersionUID = 7743244512351499265L;
     private UICheckBox largeModelCheckBox;
 
+    protected JPanel buttonPane;
+
+    protected UIButton autoButton;
+
     private boolean samePlot;
 
     protected boolean isSamePlot() {
         return samePlot;
     }
 
+    public AbstractVanChartTypePane() {
+        buttonPane = new JPanel();
+        buttonPane.setLayout(new BorderLayout());
+        if (ChartTypeManager.AUTO_CHART) {
+            autoButton = new UIButton(Toolkit.i18nText("Fine-Design_Chart_M_Popup_Auto_Chart_Type"),
+                    BaseUtils.readIcon("/com/fr/design/images/m_insert/auto_chart.png")) {
+                @Override
+                public Dimension getPreferredSize() {
+                    return new Dimension((int) super.getPreferredSize().getWidth(), 25);
+                }
+            };
+            buttonPane.setBorder((BorderFactory.createEmptyBorder(5, 0, 0, 0)));
+            buttonPane.add(autoButton);
+        }
+    }
+
+    @Override
+    protected Component[][] getPaneComponents(JPanel typePane) {
+        return new Component[][]{
+                new Component[]{typePane},
+                new Component[]{buttonPane},
+                new Component[]{stylePane},
+        };
+    }
+
     //新图表暂时还没有平面3d，渐变高光等布局。
     @Override
     protected String[] getTypeLayoutPath() {
@@ -237,4 +273,11 @@ public abstract class AbstractVanChartTypePane extends AbstractChartTypePane<Cha
             newPlot.setConditionCollection((ConditionCollection) oldPlot.getConditionCollection().clone());
         }
     }
+
+    @Override
+    public void registerButtonListener(ActionListener autoButtonListener) {
+        if (autoButton != null) {
+            autoButton.addActionListener(autoButtonListener);
+        }
+    }
 }

designer-chart/src/main/java/com/fr/van/chart/map/designer/type/VanChartMapPlotPane.java

@@ -42,6 +42,7 @@ public class VanChartMapPlotPane extends AbstractVanChartTypePane {
         }
         return new Component[][]{
                 new Component[]{typePane},
+                new Component[]{buttonPane},
                 new Component[]{sourceChoosePane}
         };
     }

designer-form/src/main/java/com/fr/design/designer/creator/XAutoChartCreator.java

@@ -2,6 +2,7 @@ package com.fr.design.designer.creator;
 
 import com.fr.base.GraphHelper;
 import com.fr.base.chart.BaseChartCollection;
+import com.fr.common.annotations.Open;
 import com.fr.design.dialog.DialogActionAdapter;
 import com.fr.design.gui.chart.MiddleChartComponent;
 import com.fr.design.gui.chart.MiddleChartDialog;
@@ -11,21 +12,23 @@ import com.fr.design.mainframe.EditingMouseListener;
 import com.fr.design.mainframe.FormDesigner;
 import com.fr.design.module.DesignModuleFactory;
 import com.fr.form.ui.ChartAutoEditor;
+import com.fr.general.IOUtils;
+import com.fr.stable.Constants;
 import com.fr.stable.bridge.StableFactory;
 
 import javax.swing.JComponent;
 import javax.swing.JPanel;
-import java.awt.Color;
 import java.awt.Dimension;
 import java.awt.Graphics;
-import java.awt.Rectangle;
 import java.awt.event.MouseEvent;
+import java.awt.image.BufferedImage;
 
 /**
  * @author Bjorn
  * @version 10.0
  * Created by Bjorn on 2020-05-26
  */
+@Open
 public class XAutoChartCreator extends XChartEditor {
 
     private MiddleChartDialog autoChartDialog;
@@ -76,6 +79,7 @@ public class XAutoChartCreator extends XChartEditor {
                     initChart(autoChartDialog.getChartCollection());
                     formDesigner.fireTargetModified();
                 }
+
                 @Override
                 public void doCancel() {
                     editingMouseListener.stopEditing();
@@ -99,9 +103,13 @@ public class XAutoChartCreator extends XChartEditor {
 
 
     public void paint(Graphics g) {
-        //TODO @Bjorn 视觉样式优化
-        g.setColor(Color.RED);
-        GraphHelper.fill(g, new Rectangle(0, 0, getWidth(), getHeight()));
+        BufferedImage bufferedImage = IOUtils.readImage("com/fr/design/form/images/auto_chart_preview.png");
+        GraphHelper.paintImage(
+                g, this.getWidth(), this.getHeight(), bufferedImage,
+                Constants.IMAGE_ADJUST,
+                0,
+                0, -1, -1
+        );
         super.paint(g);
     }
 

designer-form/src/main/java/com/fr/design/designer/creator/XCreatorUtils.java

@@ -19,7 +19,6 @@ import com.fr.form.parameter.FormSubmitButton;
 import com.fr.form.ui.Button;
 import com.fr.form.ui.CardAddButton;
 import com.fr.form.ui.CardSwitchButton;
-import com.fr.form.ui.ChartAutoEditor;
 import com.fr.form.ui.CheckBox;
 import com.fr.form.ui.CheckBoxGroup;
 import com.fr.form.ui.ComboBox;
@@ -170,12 +169,12 @@ public class XCreatorUtils {
         extraObjectMap.clear();
         extraObjectMap.putAll(ExtraDesignClassManager.getInstance().getParameterWidgetOptionsMap());
         extraObjectMap.putAll(ExtraDesignClassManager.getInstance().getFormWidgetOptionsMap());
+        extraObjectMap.putAll(ExtraDesignClassManager.getInstance().getChartWidgetOptionsMap());
     }
     
     private static void putExtraEditor() {
         if (DesignModuleFactory.getChartEditorClass() != null) {
             objectMap.put(DesignModuleFactory.getChartEditorClass(), XChartEditor.class);
-            objectMap.put(ChartAutoEditor.class, XAutoChartCreator.class);
         }
     }
     

designer-form/src/main/java/com/fr/design/mainframe/FormParaWidgetPane.java

@@ -34,6 +34,10 @@ import javax.swing.JComponent;
 import javax.swing.JPanel;
 import javax.swing.JSeparator;
 import javax.swing.SwingConstants;
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.Iterator;
+import java.util.List;
 import java.awt.BorderLayout;
 import java.awt.Color;
 import java.awt.Dimension;
@@ -42,10 +46,6 @@ import java.awt.event.ComponentAdapter;
 import java.awt.event.ComponentEvent;
 import java.awt.event.MouseAdapter;
 import java.awt.event.MouseEvent;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Iterator;
-import java.util.List;
 
 /**
  * @author null
@@ -440,7 +440,9 @@ public class FormParaWidgetPane extends JPanel {
 
     private WidgetOption[] loadChartOptions() {
         if (chartOptions == null) {
-            chartOptions = DesignModuleFactory.getExtraWidgetOptions();
+            ExtraDesignClassManager instance = ExtraDesignClassManager.getInstance();
+            WidgetOption[] widgetOptions = ArrayUtils.addAll(instance.getBeforeChartWidgetOptions(), DesignModuleFactory.getExtraWidgetOptions());
+            chartOptions = ArrayUtils.addAll(widgetOptions, instance.getAfterChartWidgetOptions());
         }
         return chartOptions;
     }

designer-realize/src/main/java/com/fr/design/actions/insert/cell/AutoChartCellAction.java

@@ -2,6 +2,7 @@ package com.fr.design.actions.insert.cell;
 
 import com.fr.base.BaseUtils;
 import com.fr.chart.chartattr.AutoChartCollection;
+import com.fr.common.annotations.Open;
 import com.fr.design.actions.core.WorkBookSupportable;
 import com.fr.design.i18n.Toolkit;
 import com.fr.design.mainframe.ElementCasePane;
@@ -14,6 +15,7 @@ import javax.swing.KeyStroke;
  * @version 10.0
  * Created by Bjorn on 2020-05-28
  */
+@Open
 public class AutoChartCellAction extends AbstractCellAction implements WorkBookSupportable {
 
     public AutoChartCellAction() {
@@ -29,7 +31,6 @@ public class AutoChartCellAction extends AbstractCellAction implements WorkBookS
         this.setMenuKeySet(INSERT_AUTO_CHART);
         this.setName(getMenuKeySet().getMenuKeySetName() + "...");
         this.setMnemonic(getMenuKeySet().getMnemonic());
-        //TODO @Bjorn 视觉样式优化icon
         this.setSmallIcon(BaseUtils.readIcon("/com/fr/design/images/m_insert/auto_chart.png"));
     }
 

designer-realize/src/main/java/com/fr/design/cell/editor/AutoChartCellEditor.java

@@ -1,5 +1,6 @@
 package com.fr.design.cell.editor;
 
+import com.fr.common.annotations.Open;
 import com.fr.design.gui.chart.MiddleChartDialog;
 import com.fr.design.mainframe.ElementCasePane;
 import com.fr.design.module.DesignModuleFactory;
@@ -12,6 +13,7 @@ import java.awt.Window;
  * @version 10.0
  * Created by Bjorn on 2020-05-28
  */
+@Open
 public class AutoChartCellEditor extends ChartCellEditor {
 
     public AutoChartCellEditor(ElementCasePane<? extends TemplateElementCase> ePane) {

designer-realize/src/main/java/com/fr/design/mainframe/ElementCasePane.java

@@ -9,7 +9,6 @@ import com.fr.base.Formula;
 import com.fr.base.ScreenResolution;
 import com.fr.base.Style;
 import com.fr.base.vcs.DesignerMode;
-import com.fr.chart.chartattr.AutoChartCollection;
 import com.fr.design.DesignState;
 import com.fr.design.DesignerEnvManager;
 import com.fr.design.ExtraDesignClassManager;
@@ -53,8 +52,6 @@ import com.fr.design.cell.bar.DynamicScrollBar;
 import com.fr.design.cell.clipboard.CellElementsClip;
 import com.fr.design.cell.clipboard.ElementsTransferable;
 import com.fr.design.cell.clipboard.FloatElementsClip;
-import com.fr.design.cell.editor.AutoChartCellEditor;
-import com.fr.design.cell.editor.AutoChartFloatEditor;
 import com.fr.design.cell.editor.BiasTextPainterCellEditor;
 import com.fr.design.cell.editor.CellEditor;
 import com.fr.design.cell.editor.ChartCellEditor;
@@ -402,8 +399,6 @@ public abstract class ElementCasePane<T extends TemplateElementCase> extends Tar
             grid.setDefaultFloatEditor(chartClass, new ChartFloatEditor());
         }
 
-        grid.setDefaultCellEditor(AutoChartCollection.class, new AutoChartCellEditor(this));
-        grid.setDefaultFloatEditor(AutoChartCollection.class, new AutoChartFloatEditor());
 
         addExtraCellEditor(grid);
 
@@ -427,7 +422,7 @@ public abstract class ElementCasePane<T extends TemplateElementCase> extends Tar
                 editor = (CellEditor) c.newInstance();
             } catch (NoSuchMethodException e) {
                 try {
-                    c = clazz.getConstructor(ElementCase.class);
+                    c = clazz.getConstructor(ElementCasePane.class);
                     editor = (CellEditor) c.newInstance(this);
                 } catch (Exception e1) {
                     FineLoggerFactory.getLogger().error(e1.getMessage(), e1);

designer-realize/src/main/java/com/fr/start/module/DesignerActivator.java

@@ -350,6 +350,9 @@ public class DesignerActivator extends Activator {
         Set<ElementUIProvider> providers = ExtraDesignClassManager.getInstance().getArray(ElementUIProvider.MARK_STRING);
         for (ElementUIProvider provider : providers) {
             try {
+                if (provider.quickEditor() == null) {
+                    continue;
+                }
                 ActionFactory.registerCellEditor(provider.targetObjectClass(), provider.quickEditor().newInstance());
             } catch (Exception e) {
                 FineLoggerFactory.getLogger().error(e.getMessage(), e);