From 3a18665323ffc930cd23c70d930302f4b887f9a5 Mon Sep 17 00:00:00 2001 From: kerry Date: Mon, 5 Jun 2017 09:38:04 +0800 Subject: [PATCH] =?UTF-8?q?REPORT-2354=09=E5=B0=86=E5=B9=B3=E5=8F=B0?= =?UTF-8?q?=E7=9A=84=E6=8F=92=E4=BB=B6=E7=AE=A1=E7=90=86=E5=92=8C=E8=AE=BE?= =?UTF-8?q?=E8=AE=A1=E5=99=A8=E7=9A=84=E6=8F=92=E4=BB=B6=E7=AE=A1=E7=90=86?= =?UTF-8?q?=E7=BB=9F=E4=B8=80=E8=B5=B7=E6=9D=A5?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../fr/design/extra/PluginOperateUtils.java | 61 ++++++++++--------- .../extra/exe/callback/DownloadCallback.java | 4 +- .../exe/callback/InstallFromDiskCallback.java | 5 +- .../exe/callback/InstallOnlineCallback.java | 6 +- .../design/extra/exe/callback/JSCallback.java | 50 +++++++++------ .../exe/callback/ModifyStatusCallback.java | 5 +- ...back.java => UninstallPluginCallback.java} | 8 ++- .../exe/callback/UpdateFromDiskCallback.java | 7 ++- .../exe/callback/UpdateOnlineCallback.java | 5 +- 9 files changed, 89 insertions(+), 62 deletions(-) rename designer_base/src/com/fr/design/extra/exe/callback/{UnistallPluginCallback.java => UninstallPluginCallback.java} (77%) diff --git a/designer_base/src/com/fr/design/extra/PluginOperateUtils.java b/designer_base/src/com/fr/design/extra/PluginOperateUtils.java index 4f2a2e204..f867574e6 100644 --- a/designer_base/src/com/fr/design/extra/PluginOperateUtils.java +++ b/designer_base/src/com/fr/design/extra/PluginOperateUtils.java @@ -16,14 +16,11 @@ import com.fr.plugin.manage.PluginManager; import com.fr.plugin.manage.bbs.BBSPluginLogin; import com.fr.plugin.manage.control.PluginTaskCallback; import com.fr.plugin.view.PluginView; -import com.fr.stable.ArrayUtils; import com.fr.stable.StringUtils; -import netscape.javascript.JSObject; import org.json.JSONArray; import javax.swing.*; import java.io.File; -import java.util.ArrayList; import java.util.List; @@ -79,7 +76,7 @@ public class PluginOperateUtils { PluginMarker pluginMarker = PluginUtils.createPluginMarker(pluginInfo); PluginContext plugin = PluginManager.getContext(pluginMarker); boolean active = plugin.isActive(); - PluginTaskCallback modifyStatusCallback = new ModifyStatusCallback(active); + PluginTaskCallback modifyStatusCallback = new ModifyStatusCallback(active, jsCallback); if (active) { PluginManager.getController().forbid(pluginMarker, modifyStatusCallback); } else { @@ -89,7 +86,7 @@ public class PluginOperateUtils { public static void uninstallPlugin(final String pluginInfo, final boolean isForce, JSCallback jsCallback) { PluginMarker pluginMarker = PluginUtils.createPluginMarker(pluginInfo); - PluginManager.getController().uninstall(pluginMarker, isForce, new UnistallPluginCallback()); + PluginManager.getController().uninstall(pluginMarker, isForce, new UninstallPluginCallback(jsCallback)); } public static void readUpdateOnline(JSCallback jsCallback) { @@ -120,30 +117,36 @@ public class PluginOperateUtils { } public static void getPluginFromStore(String category, String seller, String fee, JSCallback jsCallback) { - String plistUrl = SiteCenter.getInstance().acquireUrlByKind("plugin.plist"); - if (StringUtils.isNotBlank(plistUrl)) { - StringBuilder url = new StringBuilder(); - url.append(plistUrl); - if (StringUtils.isNotBlank(category)) { - url.append("&cid=").append(category.split("-")[1]); - } - if (StringUtils.isNotBlank(seller)) { - url.append("&seller=").append(seller.split("-")[1]); - } - if (StringUtils.isNotBlank(fee)) { - url.append("&fee=").append(fee.split("-")[1]); - } - try { - HttpClient httpClient = new HttpClient(url.toString()); - String result = httpClient.getResponseText(); - jsCallback.execute(result); - } catch (Exception e) { - FRLogger.getLogger().error(e.getMessage()); + new Thread(new Runnable() { + @Override + public void run() { + String plistUrl = SiteCenter.getInstance().acquireUrlByKind("plugin.plist"); + if (StringUtils.isNotBlank(plistUrl)) { + StringBuilder url = new StringBuilder(); + url.append(plistUrl); + if (StringUtils.isNotBlank(category)) { + url.append("&cid=").append(category.split("-")[1]); + } + if (StringUtils.isNotBlank(seller)) { + url.append("&seller=").append(seller.split("-")[1]); + } + if (StringUtils.isNotBlank(fee)) { + url.append("&fee=").append(fee.split("-")[1]); + } + try { + HttpClient httpClient = new HttpClient(url.toString()); + String result = httpClient.getResponseText(); + jsCallback.execute(result); + } catch (Exception e) { + FRLogger.getLogger().error(e.getMessage()); + } + } else { + String result = PluginConstants.CONNECTION_404; + jsCallback.execute(result); + } } - } else { - String result = PluginConstants.CONNECTION_404; - jsCallback.execute(result); - } + }).start(); + } public static void getPluginCategories(JSCallback jsCallback) { @@ -176,4 +179,4 @@ public class PluginOperateUtils { && StringUtils.isNotEmpty(pluginView.getEnvVersion()); } -} + } diff --git a/designer_base/src/com/fr/design/extra/exe/callback/DownloadCallback.java b/designer_base/src/com/fr/design/extra/exe/callback/DownloadCallback.java index 93ea0fe22..6c5e3eadf 100644 --- a/designer_base/src/com/fr/design/extra/exe/callback/DownloadCallback.java +++ b/designer_base/src/com/fr/design/extra/exe/callback/DownloadCallback.java @@ -12,6 +12,7 @@ import javax.swing.*; public class DownloadCallback extends AbstractPluginTaskCallback { private ExtraPluginTask extraPluginTask; private JSCallback jsCallback; + private static int HUNDRED_PERCENT = 100; public DownloadCallback(final ExtraPluginTask extraPluginTask, final JSCallback jsCallback) { this.extraPluginTask = extraPluginTask; @@ -20,12 +21,11 @@ public class DownloadCallback extends AbstractPluginTaskCallback { @Override public void updateProgress(String description, double aProgress) { - jsCallback.execute(String.valueOf(aProgress)); + jsCallback.execute(String.valueOf(aProgress * HUNDRED_PERCENT + "%")); } @Override public void done(PluginTaskResult result) { - jsCallback.execute("done"); if (result.isSuccess()) { extraPluginTask.doExtraPluginTask(); } else { diff --git a/designer_base/src/com/fr/design/extra/exe/callback/InstallFromDiskCallback.java b/designer_base/src/com/fr/design/extra/exe/callback/InstallFromDiskCallback.java index aa74e9c02..07bbf7a3f 100644 --- a/designer_base/src/com/fr/design/extra/exe/callback/InstallFromDiskCallback.java +++ b/designer_base/src/com/fr/design/extra/exe/callback/InstallFromDiskCallback.java @@ -15,6 +15,7 @@ import java.io.File; public class InstallFromDiskCallback extends AbstractPluginTaskCallback { private File zipFile; private JSCallback jsCallback; + private static int HUNDRED_PERCENT = 100; public InstallFromDiskCallback(final File zipFile, final JSCallback jsCallback) { this.zipFile = zipFile; @@ -23,13 +24,13 @@ public class InstallFromDiskCallback extends AbstractPluginTaskCallback { @Override public void updateProgress(String description, double aProgress) { - jsCallback.execute(String.valueOf(aProgress)); + jsCallback.execute(String.valueOf(aProgress * HUNDRED_PERCENT + "%")); } @Override public void done(PluginTaskResult result) { - jsCallback.execute("done"); + jsCallback.execute("success"); if (result.isSuccess()) { FRLogger.getLogger().info(Inter.getLocText("FR-Designer-Plugin_Install_Success")); JOptionPane.showMessageDialog(null, Inter.getLocText("FR-Designer-Plugin_Install_Successful")); diff --git a/designer_base/src/com/fr/design/extra/exe/callback/InstallOnlineCallback.java b/designer_base/src/com/fr/design/extra/exe/callback/InstallOnlineCallback.java index a1ee0ee4c..16fcae743 100644 --- a/designer_base/src/com/fr/design/extra/exe/callback/InstallOnlineCallback.java +++ b/designer_base/src/com/fr/design/extra/exe/callback/InstallOnlineCallback.java @@ -14,7 +14,7 @@ import javax.swing.*; */ public class InstallOnlineCallback extends AbstractPluginTaskCallback { protected JSCallback jsCallback; - + private static int HUNDRED_PERCENT = 100; public InstallOnlineCallback(PluginMarker pluginMarker, JSCallback jsCallback){ this.pluginMarker = pluginMarker; @@ -23,13 +23,13 @@ public class InstallOnlineCallback extends AbstractPluginTaskCallback { @Override public void updateProgress(String description, double aProgress) { - jsCallback.execute(String.valueOf(aProgress)); + jsCallback.execute(String.valueOf(aProgress * HUNDRED_PERCENT + "%")); } @Override public void done(PluginTaskResult result) { - jsCallback.execute("done"); + jsCallback.execute("success"); if (result.isSuccess()) { FRLogger.getLogger().info(Inter.getLocText("FR-Designer-Plugin_Install_Success")); JOptionPane.showMessageDialog(null, Inter.getLocText("FR-Designer-Plugin_Install_Successful")); diff --git a/designer_base/src/com/fr/design/extra/exe/callback/JSCallback.java b/designer_base/src/com/fr/design/extra/exe/callback/JSCallback.java index 8204a88eb..4d2fb7ade 100644 --- a/designer_base/src/com/fr/design/extra/exe/callback/JSCallback.java +++ b/designer_base/src/com/fr/design/extra/exe/callback/JSCallback.java @@ -1,39 +1,51 @@ package com.fr.design.extra.exe.callback; import com.fr.stable.StringUtils; +import javafx.application.Platform; +import javafx.beans.value.ChangeListener; +import javafx.beans.value.ObservableValue; +import javafx.concurrent.Task; import javafx.scene.web.WebEngine; import netscape.javascript.JSObject; /** * Created by ibm on 2017/5/27. */ -public class JSCallback { - private WebEngine webEngine; - private JSObject callback; +public class JSCallback extends Task { + public JSCallback(final WebEngine webEngine, final JSObject callback) { - this.webEngine = webEngine; - this.callback = callback; + init(webEngine, callback); } - public void execute(String newValue) { - String fun = "(" + callback + ")(\"" + trimText(newValue) + "\")"; - try { - webEngine.executeScript(fun); - } catch (Exception e) { - webEngine.executeScript("alert(\"" + e.getMessage() + "\")"); - } + public void init(final WebEngine webEngine, final JSObject callback){ + messageProperty().addListener(new ChangeListener() { + @Override + public void changed(ObservableValue observable, String oldValue, String newValue) { + Platform.runLater(new Runnable() { + @Override + public void run() { + String fun = "(" + callback + ")(\"" + trimText(newValue) + "\")"; + try { + webEngine.executeScript(fun); + } catch (Exception e) { + webEngine.executeScript("alert(\"" + e.getMessage() + "\")"); + } + } + }); + } + }); + } + @Override + protected T call() throws Exception { + return null; } - public void execute(double progress) { - String fun = "(" + callback + ")(\"" + trimText(String.valueOf(progress)) + "\")"; - try { - webEngine.executeScript(fun); - } catch (Exception e) { - webEngine.executeScript("alert(\"" + e.getMessage() + "\")"); - } + public void execute(String newValue) { + updateMessage(newValue); } + /** * vito:由于使用webEngine.executeScript("(" + callback + ")(\"" + newValue + "\")") * 执行脚本，所以原来规范的json格式也会在拼接字符串后可能抛出参数异常，需要转换掉一些会造成错误的特殊字符， diff --git a/designer_base/src/com/fr/design/extra/exe/callback/ModifyStatusCallback.java b/designer_base/src/com/fr/design/extra/exe/callback/ModifyStatusCallback.java index 2a22e0b90..a02f7628a 100644 --- a/designer_base/src/com/fr/design/extra/exe/callback/ModifyStatusCallback.java +++ b/designer_base/src/com/fr/design/extra/exe/callback/ModifyStatusCallback.java @@ -11,13 +11,16 @@ import javax.swing.*; */ public class ModifyStatusCallback implements PluginTaskCallback{ private boolean isActive; + private JSCallback jsCallback; - public ModifyStatusCallback (boolean isActive){ + public ModifyStatusCallback (boolean isActive, JSCallback jsCallback){ this.isActive = isActive; + this.jsCallback = jsCallback; } @Override public void done(PluginTaskResult result) { if (result.isSuccess()) { + jsCallback.execute("success"); String modifyMessage = isActive ? Inter.getLocText("FR-Designer-Plugin_Has_Been_Actived") : Inter.getLocText("FR-Designer-Plugin_Has_Been_Disabled"); JOptionPane.showMessageDialog(null, modifyMessage); } else { diff --git a/designer_base/src/com/fr/design/extra/exe/callback/UnistallPluginCallback.java b/designer_base/src/com/fr/design/extra/exe/callback/UninstallPluginCallback.java similarity index 77% rename from designer_base/src/com/fr/design/extra/exe/callback/UnistallPluginCallback.java rename to designer_base/src/com/fr/design/extra/exe/callback/UninstallPluginCallback.java index 851c9e4e4..983a2a0c5 100644 --- a/designer_base/src/com/fr/design/extra/exe/callback/UnistallPluginCallback.java +++ b/designer_base/src/com/fr/design/extra/exe/callback/UninstallPluginCallback.java @@ -10,11 +10,17 @@ import javax.swing.*; /** * Created by ibm on 2017/5/27. */ -public class UnistallPluginCallback implements PluginTaskCallback { +public class UninstallPluginCallback implements PluginTaskCallback { + private JSCallback jsCallback; + + public UninstallPluginCallback(JSCallback jsCallback){ + this.jsCallback = jsCallback; + } @Override public void done(PluginTaskResult result) { if (result.isSuccess()) { + jsCallback.execute("success"); FRLogger.getLogger().info(Inter.getLocText("FR-Designer-Plugin_Delete_Success")); JOptionPane.showMessageDialog(null, Inter.getLocText("FR-Designer-Plugin_Install_Successful")); } else { diff --git a/designer_base/src/com/fr/design/extra/exe/callback/UpdateFromDiskCallback.java b/designer_base/src/com/fr/design/extra/exe/callback/UpdateFromDiskCallback.java index 0f79a1614..9a4e0d093 100644 --- a/designer_base/src/com/fr/design/extra/exe/callback/UpdateFromDiskCallback.java +++ b/designer_base/src/com/fr/design/extra/exe/callback/UpdateFromDiskCallback.java @@ -14,7 +14,8 @@ import java.io.File; */ public class UpdateFromDiskCallback extends AbstractPluginTaskCallback { private File zipFile; - protected JSCallback jsCallback; + private JSCallback jsCallback; + private static int HUNDRED_PERCENT = 100; public UpdateFromDiskCallback(File zipFile, JSCallback jsCallback) { this.zipFile = zipFile; @@ -23,13 +24,13 @@ public class UpdateFromDiskCallback extends AbstractPluginTaskCallback { @Override public void updateProgress(String description, double aProgress) { - jsCallback.execute(String.valueOf(aProgress)); + jsCallback.execute(String.valueOf(aProgress * HUNDRED_PERCENT + "%")); } @Override public void done(PluginTaskResult result) { - jsCallback.execute("done"); + jsCallback.execute("success"); if (result.isSuccess()) { FRLogger.getLogger().info(Inter.getLocText("FR-Designer-Plugin_Install_Success")); JOptionPane.showMessageDialog(null, Inter.getLocText("FR-Designer-Plugin_Install_Successful")); diff --git a/designer_base/src/com/fr/design/extra/exe/callback/UpdateOnlineCallback.java b/designer_base/src/com/fr/design/extra/exe/callback/UpdateOnlineCallback.java index 65a71a80c..c139cea2c 100644 --- a/designer_base/src/com/fr/design/extra/exe/callback/UpdateOnlineCallback.java +++ b/designer_base/src/com/fr/design/extra/exe/callback/UpdateOnlineCallback.java @@ -15,6 +15,7 @@ import javax.swing.*; public class UpdateOnlineCallback extends AbstractPluginTaskCallback { public PluginMarker toPluginMarker; protected JSCallback jsCallback; + private static int HUNDRED_PERCENT = 100; public UpdateOnlineCallback(PluginMarker pluginMarker, PluginMarker toPluginMarker, JSCallback jsCallback) { this.pluginMarker = pluginMarker; @@ -24,12 +25,12 @@ public class UpdateOnlineCallback extends AbstractPluginTaskCallback { @Override public void updateProgress(String description, double aProgress) { - jsCallback.execute(String.valueOf(aProgress)); + jsCallback.execute(String.valueOf(aProgress * HUNDRED_PERCENT + "%")); } @Override public void done(PluginTaskResult result) { - jsCallback.execute("done"); + jsCallback.execute("success"); if (result.isSuccess()) { FRLogger.getLogger().info(Inter.getLocText("FR-Designer-Plugin_Update_Success")); JOptionPane.showMessageDialog(null, Inter.getLocText("FR-Designer-Plugin_Install_Successful"));