Merge branch 'feature/10.0' of https://code.fineres.com/scm/~hades/design into feature/10.0

designer-base/src/main/java/com/fr/design/data/datapane/preview/sql/PreviewPerformedSqlPane.java

@@ -6,6 +6,7 @@ import com.fr.base.ParameterMapNameSpace;
 import com.fr.data.impl.DBTableData;
 import com.fr.data.impl.EscapeSqlHelper;
 import com.fr.data.operator.DataOperator;
+import com.fr.decision.config.PreventSqlInjConfig;
 import com.fr.decision.webservice.v10.config.ConfigService;
 import com.fr.design.dialog.DialogActionAdapter;
 import com.fr.design.dialog.link.MessageWithLink;
@@ -18,6 +19,7 @@ import com.fr.design.parameter.ParameterInputPane;
 import com.fr.design.utils.gui.GUICoreUtils;
 import com.fr.general.CloudCenter;
 import com.fr.general.GeneralContext;
+import com.fr.invoke.Reflect;
 import com.fr.log.FineLoggerFactory;
 import com.fr.plugin.injectable.PluginModule;
 import com.fr.script.Calculator;
@@ -46,6 +48,8 @@ import java.awt.Frame;
 import java.awt.datatransfer.StringSelection;
 import java.awt.event.ActionEvent;
 import java.awt.event.ActionListener;
+import java.awt.event.WindowAdapter;
+import java.awt.event.WindowEvent;
 import java.util.HashMap;
 import java.util.HashSet;
 import java.util.List;
@@ -107,11 +111,11 @@ public class PreviewPerformedSqlPane extends JDialog implements ActionListener {
 
         topPanel.add(imagePanel, BorderLayout.WEST);
         topPanel.add(messagePanel, BorderLayout.CENTER);
-        topPanel.setBorder(BorderFactory.createEmptyBorder(10,10,0,10));
+        topPanel.setBorder(BorderFactory.createEmptyBorder(10, 10, 0, 10));
 
         //中间的SQL面板
         centerPanel = FRGUIPaneFactory.createBorderLayout_L_Pane();
-        centerPanel.setBorder(BorderFactory.createEmptyBorder(0,10,10,10));
+        centerPanel.setBorder(BorderFactory.createEmptyBorder(0, 10, 10, 10));
         JScrollPane scrollPane = new JScrollPane();
         JTextArea checkArea = new JTextArea(sql);
         checkArea.setEditable(false);
@@ -135,7 +139,7 @@ public class PreviewPerformedSqlPane extends JDialog implements ActionListener {
         UIButton okButton = new UIButton(Toolkit.i18nText("Fine-Design_Report_OK"));
         okButton.addActionListener(this);
         bottomPanel = FRGUIPaneFactory.createBorderLayout_L_Pane();
-        bottomPanel.setBorder(BorderFactory.createEmptyBorder(10,10,10,10));
+        bottomPanel.setBorder(BorderFactory.createEmptyBorder(10, 10, 10, 10));
         bottomPanel.add(okButton, BorderLayout.EAST);
 
         this.setTitle(Toolkit.i18nText("Fine-Design_Basic_Preview_Performed_Sql"));
@@ -144,6 +148,12 @@ public class PreviewPerformedSqlPane extends JDialog implements ActionListener {
         this.add(centerPanel, BorderLayout.CENTER);
         this.add(bottomPanel, BorderLayout.SOUTH);
         this.setSize(600, 400);
+        this.addWindowListener(new WindowAdapter() {
+            @Override
+            public void windowClosing(WindowEvent e) {
+                close();
+            }
+        });
 
         GUICoreUtils.centerWindow(this);
     }
@@ -167,42 +177,38 @@ public class PreviewPerformedSqlPane extends JDialog implements ActionListener {
                 parameterMap.put(parameter.getName(), parameter.getValue());
             }
         }
-        boolean showOriginSql = true;
         for (ParameterProvider parameter : DataOperator.getInstance().getTableDataParameters(tableData)) {
             if (parameterMap.containsKey(parameter.getName())) {
-                Object value = parameterMap.get(parameter.getName());
+                parameter.setValue(parameterMap.get(parameter.getName()));
-                if (value != null && !StringUtils.EMPTY.equals(value)) {
-                    showOriginSql = false;
-                }
-                parameter.setValue(value);
             }
         }
         String sql;
         // 计算高亮文本位置
         List<int[]> specialCharParamIndex = null;
         boolean highlight = true;
-        if (showOriginSql) {
+        NameSpace ns = ParameterMapNameSpace.create(parameterMap);
-            sql = tableData.getQuery();
+        calculator.pushNameSpace(ns);
-        } else {
+        Parameter[] paras = processParameters(tableData, calculator);
-            NameSpace ns = ParameterMapNameSpace.create(parameterMap);
+        // 所有被转义参数的集合
-            calculator.pushNameSpace(ns);
+        // 远程时如何获取服务器上的转义字符？这里比较恶心
-            Parameter[] paras = processParameters(tableData, calculator);
+        // 方法1是新增rpc接口，交由服务器去获取转义字符。但是要考虑兼容问题：
-            // 所有被转义参数的集合
+        // 新设计器jar远程老服务器jar，需要提供Compatible实现，但是这个实现能做什么呢？目前没有老的接口可以去获取服务器的转义字符，仍然啥也做不了；
-            Set<String> specialCharParam = EscapeSqlHelper.getInstance().getSpecialCharParam(paras);
+        // 现在采用方法2，通过反射调用PreventSqlInjConfig的刷新方法（因为它是decision模块的private方法，直接改成public仍然有兼容问题）
-            // 将参数转义等
+        Reflect.on(PreventSqlInjConfig.class).call("refreshPreventSqlInjConfig");
-            Set<TableDataProvider> tableDataProviders = getTableDataProviders();
+        Set<String> specialCharParam = EscapeSqlHelper.getInstance().getSpecialCharParam(paras);
-            for (TableDataProvider provider : tableDataProviders) {
+        // 将参数转义等
-                provider.processParametersBeforeAnalyzeSQL(paras, calculator);
+        Set<TableDataProvider> tableDataProviders = getTableDataProviders();
-            }
+        for (TableDataProvider provider : tableDataProviders) {
+            provider.processParametersBeforeAnalyzeSQL(paras, calculator);
+        }
 
-            if (!specialCharParam.isEmpty()) {
+        if (!specialCharParam.isEmpty()) {
-                specialCharParamIndex = ParameterHelper.analyzeCurrentContextTableData4Template(tableData.getQuery(), paras, specialCharParam);
+            specialCharParamIndex = ParameterHelper.analyzeCurrentContextTableData4Template(tableData.getQuery(), paras, specialCharParam);
-            }
+        }
-            String oldSql = ParameterHelper.analyzeCurrentContextTableData4Templatee(tableData.getQuery(), paras);
+        String oldSql = ParameterHelper.analyzeCurrentContextTableData4Templatee(tableData.getQuery(), paras);
-            sql = processExtraSQL(paras, oldSql, calculator, tableDataProviders);
+        sql = processExtraSQL(paras, oldSql, calculator, tableDataProviders);
-            if (!StringUtils.equals(oldSql, sql)) {
+        if (!StringUtils.equals(oldSql, sql)) {
-                highlight = false;
+            highlight = false;
-            }
         }
         // sql内容复制到剪切板
         StringSelection selection = new StringSelection(sql);
@@ -274,4 +280,8 @@ public class PreviewPerformedSqlPane extends JDialog implements ActionListener {
     public void actionPerformed(ActionEvent e) {
         this.dispose();
     }
+
+    private void close() {
+        this.dispose();
+    }
 }

designer-base/src/main/java/com/fr/design/gui/controlpane/UIListGroupControlPane.java

@@ -482,7 +482,7 @@ public abstract class UIListGroupControlPane  extends UIControlPane implements L
             UILabel label = new UILabel(labelText + getWrapperLabelText()) {
                 @Override
                 public void paint(Graphics g) {
-                    ((Graphics2D) g).setComposite(AlphaComposite.getInstance(AlphaComposite.SRC_OVER, 0.7f));
+                    ((Graphics2D) g).setComposite(AlphaComposite.getInstance(AlphaComposite.SRC_OVER, 0.8f));
                     super.paint(g);
                 }
             };
@@ -490,7 +490,7 @@ public abstract class UIListGroupControlPane  extends UIControlPane implements L
             label.setOpaque(true);
             label.setBackground(Color.WHITE);
             label.setForeground(Color.decode("#333334"));
-            label.setFont(label.getFont().deriveFont(10F));
+            label.setFont(label.getFont().deriveFont(11F));
             label.setPreferredSize(new Dimension(224, 26));
             this.nameEdList = nameEdList;
             this.add(label, BorderLayout.NORTH);

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

@@ -112,6 +112,21 @@ public class XWTitleLayout extends DedicateLayoutContainer {
 		return null;
 	}
 
+
+	/**
+	 * 返回body组件
+	 * @return body组件
+	 */
+	public XCreator getBodyCreator() {
+		for (int i=0; i < getXCreatorCount(); i++) {
+			XCreator creator = getXCreator(i);
+			if (creator.hasTitleStyle()) {
+				return creator;
+			}
+		}
+		return null;
+	}
+
 	/**
 	 *  编辑状态的时候需要重新绘制下边框
 	 *

designer-form/src/main/java/com/fr/design/designer/treeview/ComponentTreeCellRenderer.java

@@ -1,13 +1,11 @@
 package com.fr.design.designer.treeview;
 
 import com.fr.base.BaseUtils;
-import com.fr.base.iofile.attr.ExtendSharableAttrMark;
 import com.fr.design.constants.UIConstants;
 import com.fr.design.designer.creator.XCreator;
 import com.fr.design.designer.creator.XCreatorUtils;
-import com.fr.form.ui.AbstractBorderStyleWidget;
+import com.fr.design.mainframe.share.util.ShareComponentUtils;
 import com.fr.log.FineLoggerFactory;
-import com.fr.stable.StringUtils;
 
 import javax.swing.BorderFactory;
 import javax.swing.Icon;
@@ -38,7 +36,7 @@ public class ComponentTreeCellRenderer extends DefaultTreeCellRenderer {
             }
             if (icon != null) {
                 setIcon(icon);
-                this.needAddShareIcon = isShareWidget((XCreator) value);
+                this.needAddShareIcon = ShareComponentUtils.isShareWidgetWithChild((XCreator) value);
             }
 
         }
@@ -47,14 +45,6 @@ public class ComponentTreeCellRenderer extends DefaultTreeCellRenderer {
         return this;
     }
 
-    private boolean isShareWidget(XCreator xCreator) {
-        if (!xCreator.toData().acceptType(AbstractBorderStyleWidget.class)) {
-            return false;
-        }
-        ExtendSharableAttrMark attrMark = ((AbstractBorderStyleWidget) xCreator.toData()).getWidgetAttrMark(ExtendSharableAttrMark.XML_TAG);
-        return attrMark != null && StringUtils.isNotEmpty(attrMark.getShareId());
-
-    }
 
     public void paint(Graphics g) {
         super.paint(g);

designer-form/src/main/java/com/fr/design/mainframe/share/util/ShareComponentUtils.java

@@ -5,6 +5,7 @@ import com.fr.base.iofile.attr.ExtendSharableAttrMark;
 import com.fr.design.designer.creator.XCreator;
 import com.fr.design.designer.creator.XCreatorUtils;
 import com.fr.design.designer.creator.XLayoutContainer;
+import com.fr.design.designer.creator.XWTitleLayout;
 import com.fr.design.file.HistoryTemplateListCache;
 import com.fr.design.i18n.Toolkit;
 import com.fr.design.mainframe.FormSelection;
@@ -91,6 +92,19 @@ public class ShareComponentUtils {
         return false;
     }
 
+    public static boolean isShareWidgetWithChild(XCreator xCreator) {
+        boolean result = isShareWidget(xCreator);
+        if (result) {
+            return true;
+        }
+        //做下兼容处理，老的组件结构的共享信息存储在子组件上
+        if (xCreator instanceof XWTitleLayout) {
+            XCreator child = ((XWTitleLayout) xCreator).getBodyCreator();
+            return isShareWidget(child);
+        }
+        return false;
+    }
+
 
     public static List<XCreator> getHelpConfigXCreatorList(XCreator root) {
         List<XCreator> helpConfigXCreatorList = new ArrayList<>();

designer-realize/src/main/java/com/fr/design/share/effect/EffectItemGroup.java

@@ -111,8 +111,11 @@ public class EffectItemGroup {
                 ChartCollection chartCollection = (ChartCollection) value;
                 for (int index = 0; index < chartCollection.getChartCount(); index++) {
                     Chart chart= (Chart) chartCollection.getChart(index, ChartProvider.class);
-                    SourceNode chartSourceNode = SourceNodeUtils.createSourceNode(chart.getPlot(), chartCollection.getChartName(index), cellSourceNode);
                     Plot plot = chart.getPlot();
+                    if (plot == null) {
+                        continue;
+                    }
+                    SourceNode chartSourceNode = SourceNodeUtils.createSourceNode(plot, chartCollection.getChartName(index), cellSourceNode);
                     initChartPlot(plot, chartSourceNode, false);
                 }
 
@@ -126,8 +129,11 @@ public class EffectItemGroup {
         ChartCollection chartCollection = (ChartCollection) editor.getChartCollection();
         for (int index = 0; index < chartCollection.getChartCount(); index++) {
             Chart chart= (Chart) chartCollection.getChart(index, ChartProvider.class);
-            SourceNode plotSourceNode = SourceNodeUtils.createSourceNode(chart.getPlot(), chartCollection.getChartName(index), null);
             Plot plot = chart.getPlot();
+            if (plot == null) {
+                continue;
+            }
+            SourceNode plotSourceNode = SourceNodeUtils.createSourceNode(plot, chartCollection.getChartName(index), null);
             initChartPlot(plot, plotSourceNode, false);
         }
 

designer-realize/src/main/java/com/fr/design/share/ui/generate/EffectContent.java

@@ -56,8 +56,8 @@ public class EffectContent extends JPanel {
             Object[] columnNames = {com.fr.design.i18n.Toolkit.i18nText("Fine-Design_Share_From"), com.fr.design.i18n.Toolkit.i18nText("Fine-Design_Share_Rename")};
             JTable table = new JTable(new EffectTableModel(effectItemGroup, columnNames));
             table.setRowHeight(25);
-            table.getColumnModel().getColumn(0).setPreferredWidth(235);
+            table.getColumnModel().getColumn(0).setPreferredWidth(230);
-            table.getColumnModel().getColumn(1).setPreferredWidth(325);
+            table.getColumnModel().getColumn(1).setPreferredWidth(320);
 
 
             table.setDefaultEditor(EffectTableModel.class, new EffectItemEditor(table));

designer-realize/src/main/java/com/fr/design/share/ui/generate/ShareGeneratePane.java

@@ -62,7 +62,7 @@ import static javax.swing.JOptionPane.ERROR_MESSAGE;
  **/
 public class ShareGeneratePane extends BasicPane {
 
-    private static final Dimension DIALOG_SIZE = new Dimension(670, 740);
+    private static final Dimension DIALOG_SIZE = new Dimension(670, 760);
     private static final Dimension DIALOG_NORMAL_SIZE = new Dimension(670, 610);
     private static final Border DIALOG_BORDER = BorderFactory.createEmptyBorder(0, 6, 4, 6);