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

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

@@ -18,11 +18,10 @@ import com.fr.design.i18n.Toolkit;
 import com.fr.design.mainframe.DesignerContext;
 import com.fr.design.mainframe.JTemplate;
 import com.fr.design.utils.DesignUtils;
-import com.fr.env.CheckServiceDialog;
+import com.fr.design.versioncheck.VersionCheckUtils;
 import com.fr.env.EnvListPane;
 import com.fr.env.RemoteWorkspaceURL;
 import com.fr.env.TestConnectionResult;
-import com.fr.env.VersionCheckMessageDialog;
 import com.fr.exit.DesignerExiter;
 import com.fr.general.ComparatorUtils;
 import com.fr.general.GeneralUtils;

designer-base/src/main/java/com/fr/design/update/actions/SyncFileProcess.java

@@ -2,22 +2,33 @@ package com.fr.design.update.actions;
 
 import com.fr.decision.update.SyncExecutor;
 import com.fr.decision.update.info.UpdateCallBack;
+import com.fr.decision.update.info.UpdateProgressCallBack;
+import com.fr.design.i18n.Toolkit;
+import com.fr.design.versioncheck.VersionCheckUtils;
 import com.fr.log.FineLoggerFactory;
 import java.util.concurrent.ExecutionException;
+import javax.swing.JProgressBar;
 import javax.swing.SwingWorker;
 
 public abstract class SyncFileProcess extends SwingWorker<Boolean, Void> {
-    private UpdateCallBack callBack;
     private String buildNo;
+    private JProgressBar bar;
 
-    public SyncFileProcess(UpdateCallBack callBack,String buildNo) {
+    public SyncFileProcess(JProgressBar bar, String buildNo) {
-        this.callBack = callBack;
+        this.bar = bar;
         this.buildNo = buildNo;
     }
 
     @Override
     protected Boolean doInBackground() throws Exception {
-        return SyncExecutor.getInstance().execute(callBack,buildNo);
+        UpdateCallBack callBack = new UpdateProgressCallBack(bar);
+        boolean result = SyncExecutor.getInstance().execute(callBack, buildNo);
+        if (result) {
+            bar.setValue(0);
+            bar.setString(Toolkit.i18nText("Fine-Design_Basic_Sync_Plugins"));
+            VersionCheckUtils.syncPlugins(VersionCheckUtils.checkLocalAndRemotePlugin());
+        }
+        return result;
     }
 
     @Override

designer-base/src/main/java/com/fr/design/versioncheck/VersionCheckUtils.java

@@ -1,11 +1,14 @@
-package com.fr.design;
+package com.fr.design.versioncheck;
 
 import com.fr.base.FRContext;
+import com.fr.design.DesignerEnvManager;
+import com.fr.design.dialog.NotificationDialog;
 import com.fr.design.env.DesignerWorkspaceInfo;
 import com.fr.design.env.DesignerWorkspaceType;
 import com.fr.design.env.RemoteWorkspace;
 import com.fr.design.i18n.Toolkit;
 import com.fr.design.mainframe.DesignerContext;
+import com.fr.env.CheckServiceDialog;
 import com.fr.env.VersionCheckMessageDialog;
 import com.fr.general.ComparatorUtils;
 import com.fr.general.GeneralUtils;
@@ -30,6 +33,9 @@ import com.fr.workspace.engine.base.FineObjectPool;
 import com.fr.workspace.engine.channel.http.FunctionalHttpRequest;
 import com.fr.workspace.engine.exception.WorkspaceConnectionException;
 import com.fr.workspace.engine.rpc.WorkspaceProxyPool;
+import java.awt.event.MouseAdapter;
+import java.awt.event.MouseEvent;
+import java.awt.event.MouseListener;
 import java.lang.reflect.Method;
 import java.text.ParsePosition;
 import java.text.SimpleDateFormat;
@@ -226,21 +232,21 @@ public class VersionCheckUtils {
                     }
                     remotePlugin.put("type", INCONSISTENCY);
                 }
-            }
+            } else {
-            remotePlugin.put("type", MISSING);
+                remotePlugin.put("type", MISSING);
-            if (remotePlugin.getString(NAME) == null) {
+                if (remotePlugin.getString(NAME) == null) {
-                remotePlugin.put(NAME, remotePlugin.getString("id"));
+                    remotePlugin.put(NAME, remotePlugin.getString("id"));
+                }
             }
             differentPlugins.put(remotePlugin);
         }
         return differentPlugins;
     }
 
-    public static JSONArray syncPlugins(JSONArray differentPlugins) {
+    public static void syncPlugins(JSONArray differentPlugins) {
         Set<String> uninstallFailed = uninstallPlugins(differentPlugins);
         List<PluginMarker> plugins = getSyncPlugins(differentPlugins, uninstallFailed);
         installPlugins(plugins);
-        return getPluginsSyncFailed(differentPlugins);
     }
 
     private static List<PluginMarker> getSyncPlugins(JSONArray differentPlugins, Set<String> uninstallFailed) {
@@ -296,7 +302,7 @@ public class VersionCheckUtils {
         return uninstallFailedID;
     }
 
-    private static JSONArray getPluginsSyncFailed(JSONArray differentPlugins){
+    public static JSONArray getPluginsSyncFailed(JSONArray differentPlugins){
         JSONArray pluginsNeedSync = JSONArray.create();
         List<PluginContext> localPlugins = PluginManager.getContexts();
         Map<String, String> localPluginsInfo = new HashMap<>();
@@ -326,6 +332,7 @@ public class VersionCheckUtils {
                 @Override
                 public void done(PluginTaskResult result) {
                     latch.countDown();
+                    FineLoggerFactory.getLogger().info("installPlugin: " + result.getCurrentTask().getMarker().getPluginID());
                 }
             });
         }

designer-base/src/main/java/com/fr/env/CheckServiceDialog.java

@@ -1,10 +1,7 @@
 package com.fr.env;
 
 import com.fr.decision.update.data.UpdateConstants;
-import com.fr.decision.update.info.UpdateCallBack;
-import com.fr.decision.update.info.UpdateProgressCallBack;
 import com.fr.design.RestartHelper;
-import com.fr.design.VersionCheckUtils;
 import com.fr.design.dialog.FineJOptionPane;
 import com.fr.design.gui.ibutton.UIButton;
 import com.fr.design.gui.ilable.UILabel;
@@ -15,6 +12,7 @@ import com.fr.design.layout.TableLayoutHelper;
 import com.fr.design.mainframe.DesignerContext;
 import com.fr.design.update.actions.SyncFileProcess;
 import com.fr.design.utils.gui.GUICoreUtils;
+import com.fr.design.versioncheck.VersionCheckUtils;
 import com.fr.general.CloudCenter;
 import com.fr.general.CloudCenterConfig;
 import com.fr.general.ComparatorUtils;
@@ -40,7 +38,6 @@ import java.awt.event.MouseEvent;
 import java.awt.event.MouseListener;
 import java.io.File;
 import java.util.List;
-import java.util.concurrent.ExecutionException;
 import javax.swing.BorderFactory;
 import javax.swing.Icon;
 import javax.swing.JDialog;
@@ -155,7 +152,7 @@ public class CheckServiceDialog extends JDialog implements ActionListener {
         detailsComponents = new Component[][]{{jarCheckTitleLabel}, {jarCheckResultLabel}, {affectedFunctionsLabel}, {affectedFunctionsDetailsPane}, {pluginCheckTitleLabel}, {pluginsCheckResultPanel}};
         hideAffectedFunctionsComponents = new Component[][]{{jarCheckTitleLabel}, {jarCheckResultLabel}, {affectedFunctionsLabel}, {pluginCheckTitleLabel}, {pluginsCheckResultPanel}};
         if (noExistServiceDescription.size() > 0 && !ComparatorUtils.equals(remoteBranch, localBranch)) {
-            detailsRowSize = new double[]{p, p, p, p,p,p};
+            detailsRowSize = new double[]{p, p, p, p, p, p};
             detailsInnerPane = TableLayoutHelper.createTableLayoutPane(detailsComponents, detailsRowSize, detailsColumnSize);
         } else {
             detailsRowSize = new double[]{p, p, 0, 0, p, p};
@@ -316,7 +313,6 @@ public class CheckServiceDialog extends JDialog implements ActionListener {
                     Toolkit.i18nText("Fine-Design_Basic_Confirm"), JOptionPane.YES_NO_OPTION, QUESTION_MESSAGE, IOUtils.readIcon("com/fr/design/icon/versioncheck/question.png"), option, 1);
             if (0 == a) {
                 //jar包一致的话只更新插件
-                UpdateCallBack callBack = new UpdateProgressCallBack(progressBar);
                 progressBar.setVisible(true);
                 progressBar.setString(Toolkit.i18nText("Fine-Design_Update_Info_Wait_Message"));
                 syncButton.setEnabled(false);
@@ -326,36 +322,17 @@ public class CheckServiceDialog extends JDialog implements ActionListener {
                     final JFrame frame = DesignerContext.getDesignerFrame();
                     final RestartHelper helper = new RestartHelper();
                     FineProcessContext.getParentPipe().fire(FineProcessEngineEvent.DESTROY);
-                    new SyncFileProcess(callBack, remoteBuildNo) {
+                    new SyncFileProcess(progressBar, remoteBuildNo) {
                         @Override
                         public void onDownloadSuccess() {
                             deleteForDesignerUpdate(installLib);
-                            //主jar下载成功之后再卸载安装插件
+                            progressBar.setVisible(false);
-                            progressBar.setString(Toolkit.i18nText("Fine-Design_Basic_Sync_Plugins"));
+                            JSONArray syncFailedPlugins = VersionCheckUtils.getPluginsSyncFailed(differentPlugins);
-                            progressBar.setValue(0);
+                            if (syncFailedPlugins.size() > 0) {
-                            new SwingWorker<JSONArray, Void>() {
+                                SyncFailedPluginsDialog syncFailedPluginsDialog = new SyncFailedPluginsDialog(DesignerContext.getDesignerFrame(), syncFailedPlugins);
-
+                                syncFailedPluginsDialog.setVisible(true);
-                                @Override
+                            }
-                                protected JSONArray doInBackground() {
+                            helper.restartForUpdate(frame);
-                                    return VersionCheckUtils.syncPlugins(differentPlugins);
-                                }
-
-                                @Override
-                                protected void done() {
-                                    progressBar.setVisible(false);
-                                    JSONArray syncFailedPlugins = null;
-                                    try {
-                                        syncFailedPlugins = get();
-                                    } catch (Exception ex) {
-                                        FineLoggerFactory.getLogger().error(ex.getMessage(), ex);
-                                    }
-                                    if (syncFailedPlugins != null && syncFailedPlugins.size() > 0) {
-                                        SyncFailedPluginsDialog syncFailedPluginsDialog = new SyncFailedPluginsDialog(DesignerContext.getDesignerFrame(), syncFailedPlugins);
-                                        syncFailedPluginsDialog.setVisible(true);
-                                    }
-                                    helper.restartForUpdate(frame);
-                                }
-                            }.execute();
                         }
 
                         @Override
@@ -369,13 +346,14 @@ public class CheckServiceDialog extends JDialog implements ActionListener {
                     }.execute();
                 } else {
                     //到这边说明主jar是一致的，就只尝试同步插件
-                    progressBar.setString(Toolkit.i18nText("Fine-Design_Basic_Sync_Plugins"));
-                    progressBar.setValue(0);
                     new SwingWorker<JSONArray, Void>() {
 
                         @Override
                         protected JSONArray doInBackground() {
-                            return VersionCheckUtils.syncPlugins(differentPlugins);
+                            progressBar.setString(Toolkit.i18nText("Fine-Design_Basic_Sync_Plugins"));
+                            progressBar.setValue(0);
+                            VersionCheckUtils.syncPlugins(differentPlugins);
+                            return VersionCheckUtils.getPluginsSyncFailed(differentPlugins);
                         }
 
                         @Override

designer-base/src/main/java/com/fr/env/SyncFailedPluginsDialog.java

@@ -77,6 +77,7 @@ public class SyncFailedPluginsDialog extends JDialog {
             detailsText.append(plugin.getString("name")).append(",").append(Toolkit.i18nText("Fine-Design_Basic_Sync_Server_Version")).append(plugin.getString("version")).append("\n");
         }
         detailsTextArea.setText(detailsText.toString());
+        detailsTextArea.setEditable(false);
         scrollPane = new JScrollPane(detailsTextArea);
         centerPanel.add(detailsTitlePanel,BorderLayout.NORTH);
         centerPanel.add(scrollPane,BorderLayout.CENTER);

designer-base/src/main/java/com/fr/env/VersionCheckMessageDialog.java

@@ -1,6 +1,6 @@
 package com.fr.env;
 
-import com.fr.design.VersionCheckUtils;
+import com.fr.design.versioncheck.VersionCheckUtils;
 import com.fr.design.gui.ilable.UILabel;
 import com.fr.design.i18n.Toolkit;
 import com.fr.design.layout.FRGUIPaneFactory;

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);
 

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

@@ -2,7 +2,6 @@ package com.fr.start.module;
 
 import com.fr.design.DesignerEnvManager;
 import com.fr.design.EnvChangeEntrance;
-import com.fr.design.VersionCheckUtils;
 import com.fr.design.constants.DesignerLaunchStatus;
 import com.fr.design.editlock.ConnectionLockChangeChecker;
 import com.fr.design.editlock.ServerTableDataLockChangeChecker;
@@ -10,6 +9,7 @@ import com.fr.design.env.DesignerWorkspaceGenerator;
 import com.fr.design.env.DesignerWorkspaceInfo;
 import com.fr.design.env.LocalDesignerWorkspaceInfo;
 import com.fr.design.i18n.Toolkit;
+import com.fr.design.versioncheck.VersionCheckUtils;
 import com.fr.env.TestConnectionResult;
 import com.fr.event.Event;
 import com.fr.event.EventDispatcher;