REPORT-13810

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

@@ -5,6 +5,7 @@ import com.fr.design.dialog.BasicDialog;
 import com.fr.design.dialog.DialogActionAdapter;
 import com.fr.design.env.DesignerWorkspaceGenerator;
 import com.fr.design.env.DesignerWorkspaceInfo;
+import com.fr.design.env.DesignerWorkspaceType;
 import com.fr.design.file.HistoryTemplateListCache;
 import com.fr.design.file.TemplateTreePane;
 import com.fr.design.i18n.Toolkit;
@@ -15,19 +16,25 @@ import com.fr.env.EnvListPane;
 import com.fr.general.GeneralContext;
 import com.fr.license.exception.RegistEditionException;
 import com.fr.log.FineLoggerFactory;
+import com.fr.stable.AssistUtils;
 import com.fr.stable.EnvChangedListener;
 import com.fr.start.server.ServerTray;
 import com.fr.workspace.WorkContext;
 import com.fr.workspace.WorkContextCallback;
 import com.fr.workspace.Workspace;
 import com.fr.workspace.connect.AuthException;
+import com.fr.workspace.connect.WorkspaceConnectionInfo;
+import com.fr.workspace.engine.channel.http.FunctionalHttpRequest;
 
 import javax.swing.JOptionPane;
 import javax.swing.SwingUtilities;
 import javax.swing.UIManager;
+import java.util.ArrayList;
 import java.util.HashMap;
+import java.util.List;
 
 import static javax.swing.JOptionPane.ERROR_MESSAGE;
+import static javax.swing.JOptionPane.QUESTION_MESSAGE;
 
 public class EnvChangeEntrance {
 
@@ -90,6 +97,11 @@ public class EnvChangeEntrance {
                 });
                 return false;
             }
+            // 如果版本不一致，且确认 不继续 连接，这里返回 false.
+            if (!versionCheckAndConfirm(selectedEnv)) {
+                return false;
+            }
+
             //REPORT-13810如果只是添加了工作目录,没有切换,这里ToolArea也是要显示新建的工作目录
             JTemplate template = HistoryTemplateListCache.getInstance().getCurrentEditingTemplate();
             if (template != null) {
@@ -139,6 +151,50 @@ public class EnvChangeEntrance {
         return true;
     }
 
+    /**
+     * 切换远程环境之前，进行版本检测，当版本不一致的时候，提示。
+     * 当用户确认选择 ok 时，才继续。
+     *
+     * @param selectedEnv 选择的环境
+     * @return 是否一致
+     * 1. 非远程环境 ， 返回 true
+     * 2. 远程环境
+     * 2.1 不匹配，
+     * 2.1.1 当选择 ok ， 返回 true
+     * 2.1.2 当选择 no, 返回 false
+     * 2.2 匹配， 返回 true
+     * @throws Exception 异常
+     */
+    private boolean versionCheckAndConfirm(DesignerWorkspaceInfo selectedEnv) throws Exception {
+
+        if (selectedEnv.getType() == DesignerWorkspaceType.Remote) {
+
+            WorkspaceConnectionInfo info = selectedEnv.getConnection();
+            String serverVersion = new FunctionalHttpRequest(info).getServerVersion();
+
+            if (AssistUtils.equals(serverVersion, WorkContext.getVersion())) {
+                return true;
+            }
+
+            final List<Integer> result = new ArrayList<>(1);
+            PopTipStrategy.NOW.showTip(new PopTip() {
+                @Override
+                public void show() {
+                    String[] option = {Toolkit.i18nText("Fine-Design_Report_Yes"), Toolkit.i18nText("Fine-Design_Report_No")};
+                    int choice = JOptionPane.showOptionDialog(DesignerContext.getDesignerFrame(), Toolkit.i18nText("Fine-Design_Basic_Remote_Design_Version_Inconsistency"),
+                            UIManager.getString("OptionPane.messageDialogTitle"), JOptionPane.YES_NO_OPTION, QUESTION_MESSAGE, UIManager.getIcon("OptionPane.warningIcon"), option, 1);
+                    result.add(choice);
+                }
+            });
+
+            // 只有选择 yes ， 这里的值才为 0， 返回 true
+            // 否着返回 false, 将不进行下面的连接操作。
+            return result.size() != 0 && result.get(0) == 0;
+        }
+
+        return true;
+    }
+
     /**
      * 编辑items
      *

designer-base/src/main/java/com/fr/design/file/HistoryTemplateListCache.java

@@ -188,10 +188,13 @@ public class HistoryTemplateListCache implements CallbackEvent {
 
     /**
      * 判断是否打开过该模板
+     * 由于切换环境不会关闭模板，可能存在同名的模板，所以该方法不能准确找到所选的模板，
      *
      * @param filename 文件名
      * @return 文件位置
+     * @deprecated use HistoryTemplateListCache#contains(com.fr.design.mainframe.JTemplate) instead
      */
+    @Deprecated
     public int contains(String filename) {
         for (int i = 0; i < historyList.size(); i++) {
             String historyPath = historyList.get(i).getPath();

designer-base/src/main/java/com/fr/design/file/MutilTempalteTabPane.java

@@ -18,7 +18,6 @@ import com.fr.log.FineLoggerFactory;
 import com.fr.stable.Constants;
 import com.fr.stable.ProductConstants;
 import com.fr.third.javax.annotation.Nonnull;
-import com.fr.third.org.apache.commons.io.FilenameUtils;
 import com.fr.workspace.WorkContext;
 import com.fr.workspace.server.lock.TplOperator;
 
@@ -674,9 +673,6 @@ public class MutilTempalteTabPane extends JComponent {
             return;
         }
 
-        //当前激活的模板
-        String filename = openedTemplate.get(selectedIndex).getPath();
-        filename = FilenameUtils.standard(filename);
         if (!specifiedTemplate.isALLSaved() && !DesignerMode.isVcsMode()) {
             specifiedTemplate.stopEditing();
             int returnVal = JOptionPane.showConfirmDialog(DesignerContext.getDesignerFrame(), Toolkit.i18nText("Fine-Design_Basic_Utils_Would_You_Like_To_Save") + " \"" + specifiedTemplate.getEditingFILE() + "\" ?",
@@ -684,20 +680,20 @@ public class MutilTempalteTabPane extends JComponent {
             if (returnVal == JOptionPane.YES_OPTION) {
                 specifiedTemplate.saveTemplate();
                 FineLoggerFactory.getLogger().info(Toolkit.i18nText("Fine-Design_Basic_Template_Already_Saved", specifiedTemplate.getEditingFILE().getName()));
-                closeTpl(specifiedTemplate, filename);
+                closeTpl(specifiedTemplate);
             } else if (returnVal == JOptionPane.NO_OPTION) {
-                closeTpl(specifiedTemplate, filename);
+                closeTpl(specifiedTemplate);
             }
         } else {
-            closeTpl(specifiedTemplate, filename);
+            closeTpl(specifiedTemplate);
         }
 
     }
 
-    private void closeTpl(@Nonnull JTemplate<?, ?> specifiedTemplate, @Nonnull String fileName) {
+    private void closeTpl(@Nonnull JTemplate<?, ?> specifiedTemplate) {
         HistoryTemplateListCache.getInstance().closeSelectedReport(specifiedTemplate);
         closeAndFreeLock(specifiedTemplate);
-        activeTemplate(fileName);
+        activePrevTemplateAfterClose();
     }
 
     private void closeAndFreeLock(@Nonnull JTemplate<?, ?> template) {
@@ -729,11 +725,9 @@ public class MutilTempalteTabPane extends JComponent {
     }
 
     /**
-     * 关闭掉一个模板之后该激活的Tab
+     * 关闭掉一个模板之后激活新的待显示模板
-     *
-     * @param fileName 关闭掉一个模板之后该激活的Tab的文件名，绝对路径
      */
-    private void activeTemplate(String fileName) {
+    private void activePrevTemplateAfterClose() {
         if (openedTemplate.isEmpty()) {
             //新建并激活模板
             DesignerContext.getDesignerFrame().addAndActivateJTemplate();
@@ -742,23 +736,20 @@ public class MutilTempalteTabPane extends JComponent {
             temTemplate = HistoryTemplateListCache.getInstance().getCurrentEditingTemplate();
 
         } else {
-            //如果关闭的模板是当前选中的模板，则重新激活
+            // 如果关闭的模板是当前选中的模板，则重新激活当前 selectIndex 的模板；
-            if (closeIconIndex == selectedIndex) {
+            // selectIndex 没有变化，但是对应的模板已经变成了前一张模板
-                if (closeIconIndex == maxPaintIndex) {
+            if (closeIconIndex == selectedIndex || isCloseCurrent) {
+                // 如果 closeIconIndex 是最后一个被渲染画出的，那么预览上一个，防止数组越界
+                if (closeIconIndex >= maxPaintIndex) {
+                    // selectIndex 不会 <0 因为如果关闭的是打开的最后一个模板，那么关闭之后 openedTemplate.isEmpty() = true
                     selectedIndex--;
                 }
-            } else if (isCloseCurrent) {
+                isCloseCurrent = false;
-                //不是通过关闭按钮，而是通过文件关闭菜单关闭的当前模板的，也重新激活
+            }
-                if (selectedIndex > openedTemplate.size() - 1) {
+            // 如果关闭的模板不是当前选中的模板，那么重新获取一下当前模板的 index,激活该 index
-                    selectedIndex -= 1;
+            else {
-                    if (selectedIndex < 0) {
+                JTemplate template = HistoryTemplateListCache.getInstance().getCurrentEditingTemplate();
-                        selectedIndex = 0;
+                selectedIndex = HistoryTemplateListCache.getInstance().contains(template);
-                    }
-                    isCloseCurrent = false;
-                }
-            } else {
-                //如果关闭的模板不是当前选中的模板，则激活的模板不变
-                selectedIndex = HistoryTemplateListCache.getInstance().contains(fileName);
             }
             //如果是已后台关闭的模板，则重新打开文件
             openedTemplate.get(selectedIndex).activeOldJTemplate();

designer-base/src/main/java/com/fr/design/gui/itextfield/UIAutoCompletionField.java

@@ -2,6 +2,8 @@ package com.fr.design.gui.itextfield;
 
 import com.fr.design.gui.imenu.UIListPopup;
 import com.fr.general.ComparatorUtils;
+import com.fr.stable.OperatingSystem;
+
 import javax.swing.JList;
 import javax.swing.UIManager;
 import javax.swing.event.DocumentEvent;
@@ -82,6 +84,9 @@ public class UIAutoCompletionField extends UITextField implements DocumentListen
     }
 
     private void textChanged() {
+        if (OperatingSystem.isMacOS()) {
+            return;
+        }
         if (!popup.isVisible()) {
             showPopup();
             requestFocus();

designer-base/src/main/java/com/fr/design/mainframe/DesignerFrame.java

@@ -31,6 +31,7 @@ import com.fr.design.gui.imenu.UIMenuHighLight;
 import com.fr.design.gui.iprogressbar.ProgressDialog;
 import com.fr.design.gui.iscrollbar.UIScrollBar;
 import com.fr.design.gui.itoolbar.UIToolbar;
+import com.fr.design.i18n.Toolkit;
 import com.fr.design.layout.FRGUIPaneFactory;
 import com.fr.design.mainframe.loghandler.LogMessageBar;
 import com.fr.design.mainframe.toolbar.ToolBarMenuDock;
@@ -78,7 +79,6 @@ import java.awt.Graphics;
 import java.awt.Insets;
 import java.awt.Point;
 import java.awt.Rectangle;
-import java.awt.Toolkit;
 import java.awt.datatransfer.DataFlavor;
 import java.awt.datatransfer.Transferable;
 import java.awt.dnd.DnDConstants;
@@ -162,9 +162,9 @@ public class DesignerFrame extends JFrame implements JTemplateActionListener, Ta
     //用于判断设计器是否打开了
     private boolean designerOpened = false;
 
-    private int contentWidth = (int) (Toolkit.getDefaultToolkit().getScreenSize().getWidth());
+    private int contentWidth = (int) (java.awt.Toolkit.getDefaultToolkit().getScreenSize().getWidth());
 
-    private int contentHeight = (int) (Toolkit.getDefaultToolkit().getScreenSize().getHeight());
+    private int contentHeight = (int) (java.awt.Toolkit.getDefaultToolkit().getScreenSize().getHeight());
 
     private WindowAdapter windowAdapter = new WindowAdapter() {
 
@@ -855,18 +855,18 @@ public class DesignerFrame extends JFrame implements JTemplateActionListener, Ta
             editingTemplate.stopEditing();
             if (!editingTemplate.getEditingFILE().exists()) {
                 int returnVal = JOptionPane.showConfirmDialog(DesignerContext.getDesignerFrame(),
-                        com.fr.design.i18n.Toolkit.i18nText("Fine-Design_Basic_Utils_Would_You_Like_To_Save") + " \"" + editingTemplate.getEditingFILE()
+                        Toolkit.i18nText("Fine-Design_Basic_Utils_Would_You_Like_To_Save") + " \"" + editingTemplate.getEditingFILE()
                                 + "\" ?", ProductConstants.PRODUCT_NAME, JOptionPane.YES_NO_OPTION,
                         JOptionPane.QUESTION_MESSAGE);
                 if (returnVal == JOptionPane.YES_OPTION && editingTemplate.saveTemplate()) {
                     editingTemplate.saveTemplate();
-                    FineLoggerFactory.getLogger().info(com.fr.design.i18n.Toolkit.i18nText("Fine-Design_Basic_Template_Already_Saved",
+                    FineLoggerFactory.getLogger().info(Toolkit.i18nText("Fine-Design_Basic_Template_Already_Saved",
                             editingTemplate.getEditingFILE().getName()));
                 }
             } else {
                 if (editingTemplate.saveTemplate()) {
                     editingTemplate.saveTemplate();
-                    FineLoggerFactory.getLogger().info(com.fr.design.i18n.Toolkit.i18nText("Fine-Design_Basic_Template_Already_Saved",
+                    FineLoggerFactory.getLogger().info(Toolkit.i18nText("Fine-Design_Basic_Template_Already_Saved",
                             editingTemplate.getEditingFILE().getName()));
                 }
             }
@@ -963,8 +963,12 @@ public class DesignerFrame extends JFrame implements JTemplateActionListener, Ta
 
         // p:判断一下，如何文件为空或者文件不存在，直接返回.
         if (tplFile == null || !tplFile.exists()) {
-            JOptionPane.showMessageDialog(this, com.fr.design.i18n.Toolkit.i18nText("Fine-Design_Basic_Warning_Template_Do_Not_Exsit"),
+            JOptionPane.showMessageDialog(
-                    ProductConstants.PRODUCT_NAME, JOptionPane.INFORMATION_MESSAGE);
+                    this,
+                    Toolkit.i18nText("Fine-Design_Basic_Warning_Template_Do_Not_Exsit"),
+                    ProductConstants.PRODUCT_NAME,
+                    JOptionPane.INFORMATION_MESSAGE
+            );
             DesignerFrameFileDealerPane.getInstance().refresh();
             return;
         }
@@ -974,7 +978,7 @@ public class DesignerFrame extends JFrame implements JTemplateActionListener, Ta
         } catch (DecryptTemplateException e) {
             JOptionPane.showMessageDialog(
                     this,
-                    com.fr.design.i18n.Toolkit.i18nText("Fine-Design_Encrypt_Decrypt_Exception"),
+                    Toolkit.i18nText("Fine-Design_Encrypt_Decrypt_Exception"),
                     UIManager.getString("OptionPane.messageDialogTitle"),
                     JOptionPane.WARNING_MESSAGE,
                     UIManager.getIcon("OptionPane.errorIcon")
@@ -1033,7 +1037,7 @@ public class DesignerFrame extends JFrame implements JTemplateActionListener, Ta
                         MutilTempalteTabPane.getInstance().setTemTemplate(
                                 HistoryTemplateListPane.getInstance().getCurrentEditingTemplate());
                     } else {
-                        activeTemplate(tplFile, jt);
+                        activeTemplate(jt);
                     }
                     opened = true;
                     break;
@@ -1048,13 +1052,12 @@ public class DesignerFrame extends JFrame implements JTemplateActionListener, Ta
     /**
      * 激活指定的模板
      *
-     * @param tplFile 模板文件
+     * @param jt 当前报表
-     * @param jt      当前报表
      * @date 2014-10-14-下午6:31:23
      */
-    private void activeTemplate(FILE tplFile, JTemplate jt) {
+    private void activeTemplate(JTemplate jt) {
         // 如果该模板已经打开，则进行激活就可以了
-        int index = HistoryTemplateListPane.getInstance().contains(tplFile.getPath());
+        int index = HistoryTemplateListPane.getInstance().contains(jt);
         List<JTemplate<?, ?>> historyList = HistoryTemplateListPane.getInstance().getHistoryList();
         if (index != -1) {
             historyList.get(index).activeJTemplate(index, jt);

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

@@ -511,14 +511,14 @@ public class RemoteEnvPane extends BasicBeanPane<RemoteDesignerWorkspaceInfo> {
     }
 
     private void tryConnectRemoteEnv() {
+        final RemoteDesignerWorkspaceInfo remoteEnv = updateBean();
+        final WorkspaceConnectionInfo connection = remoteEnv.getConnection();
+
         final SwingWorker<Boolean, Void> worker = new SwingWorker<Boolean, Void>() {
 
             @Override
             protected Boolean doInBackground() throws Exception {
 
-                final RemoteDesignerWorkspaceInfo remoteEnv = updateBean();
-
-                WorkspaceConnectionInfo connection = remoteEnv.getConnection();
                 DesignerEnvManager.getEnvManager().setCertificatePath(connection.getCertPath());
                 DesignerEnvManager.getEnvManager().setCertificatePass(connection.getCertSecretKey());
                 try {
@@ -532,18 +532,26 @@ public class RemoteEnvPane extends BasicBeanPane<RemoteDesignerWorkspaceInfo> {
             protected void done() {
                 okButton.setEnabled(true);
                 try {
-                    Boolean result = get();
+
-                    if (result == null) {
+                    TestConnectionResult result = TestConnectionResult.parse(get(), connection);
+                    if (result == TestConnectionResult.Fully_Success) {
+                        message.setText(Toolkit.i18nText("Fine-Design_Basic_Remote_Connect_Successful"));
+                        uiLabel.setIcon(UIManager.getIcon("OptionPane.informationIcon"));
+                    }
+
+                    if (result == TestConnectionResult.Partly_Sucess) {
+                        message.setText(Toolkit.i18nText("Fine-Design_Basic_Remote_Design_Version_Inconsistence_Test"));
+                        uiLabel.setIcon(UIManager.getIcon("OptionPane.warningIcon"));
+                    }
+
+                    if (result == TestConnectionResult.Fully_Failed) {
+                        message.setText(Toolkit.i18nText("Fine-Design_Basic_Remote_Connect_Failed"));
+                        uiLabel.setIcon(UIManager.getIcon("OptionPane.errorIcon"));
+                    }
+
+                    if (result == TestConnectionResult.Auth_Failed) {
                         message.setText(Toolkit.i18nText("Fine-Design_Basic_Remote_Connect_Auth_Failed"));
                         uiLabel.setIcon(UIManager.getIcon("OptionPane.errorIcon"));
-                    } else {
-                        if (result) {
-                            message.setText(Toolkit.i18nText("Fine-Design_Basic_Remote_Connect_Successful"));
-                            uiLabel.setIcon(UIManager.getIcon("OptionPane.informationIcon"));
-                        } else {
-                            message.setText(Toolkit.i18nText("Fine-Design_Basic_Remote_Connect_Failed"));
-                            uiLabel.setIcon(UIManager.getIcon("OptionPane.errorIcon"));
-                        }
                     }
                 } catch (InterruptedException | ExecutionException e) {
                     FineLoggerFactory.getLogger().error(e, e.getMessage());

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

@@ -0,0 +1,62 @@
+package com.fr.env;
+
+import com.fr.log.FineLoggerFactory;
+import com.fr.stable.AssistUtils;
+import com.fr.workspace.WorkContext;
+import com.fr.workspace.connect.WorkspaceConnectionInfo;
+import com.fr.workspace.engine.channel.http.FunctionalHttpRequest;
+
+/**
+ * 测试连接的结果。
+ * 不改变原有逻辑的情况下，加入一层转化。
+ * 根据这里的转化结果，判断需要提示哪些内容。
+ *
+ * created by Harrison on 2018/12/20
+ **/
+public enum TestConnectionResult {
+    /**
+     * 完全成功, 版本匹配，测试连接成功。
+     */
+    Fully_Success,
+
+    /**
+     * 不完全成功，版本不匹配，但测试连接成功。
+     */
+    Partly_Sucess,
+
+    /**
+     * 完全失败，直接没连上
+     */
+    Fully_Failed,
+
+    /**
+     * 验证 Token 失败
+     */
+    Auth_Failed;
+
+    public static TestConnectionResult parse(Boolean value, WorkspaceConnectionInfo info) {
+        if (value == null) {
+            return Auth_Failed;
+        }
+
+        if (!value) {
+            return Fully_Failed;
+        }
+        try {
+
+            String serverVersion = new FunctionalHttpRequest(info).getServerVersion();
+            if (AssistUtils.equals(serverVersion, WorkContext.getVersion())) {
+                return Fully_Success;
+            }
+
+            return Partly_Sucess;
+        } catch (Exception e) {
+
+            // 发生异常，说明没连接上。返回完全失败。
+            FineLoggerFactory.getLogger().error(e.getMessage());
+            return Fully_Failed;
+        }
+
+
+    }
+}

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

@@ -5,11 +5,6 @@ package com.fr.design.mainframe;
 
 import com.fr.base.FRContext;
 import com.fr.config.MarketConfig;
-import com.fr.data.core.db.DBUtils;
-import com.fr.data.core.db.dialect.DialectFactory;
-import com.fr.data.core.db.dml.Delete;
-import com.fr.data.core.db.dml.Select;
-import com.fr.data.core.db.dml.Table;
 import com.fr.design.DesignerEnvManager;
 import com.fr.design.mainframe.errorinfo.ErrorInfoUploader;
 import com.fr.design.mainframe.templateinfo.TemplateInfoCollector;
@@ -19,18 +14,13 @@ import com.fr.general.DateUtils;
 import com.fr.general.DesUtils;
 import com.fr.general.GeneralUtils;
 import com.fr.general.IOUtils;
-import com.fr.general.http.HttpClient;
 import com.fr.general.http.HttpToolbox;
 import com.fr.intelli.record.FocusPoint;
-import com.fr.intelli.record.MetricException;
 import com.fr.intelli.record.MetricRegistry;
 import com.fr.json.JSONArray;
 import com.fr.json.JSONException;
 import com.fr.json.JSONObject;
 import com.fr.log.FineLoggerFactory;
-import com.fr.log.message.ParameterMessage;
-import com.fr.record.DBRecordXManager;
-import com.fr.stable.ArrayUtils;
 import com.fr.stable.EncodeConstants;
 import com.fr.stable.ProductConstants;
 import com.fr.stable.StableUtils;
@@ -58,17 +48,15 @@ import java.io.InputStream;
 import java.io.InputStreamReader;
 import java.io.OutputStreamWriter;
 import java.io.UnsupportedEncodingException;
-import java.net.URLEncoder;
-import java.sql.Connection;
-import java.sql.PreparedStatement;
-import java.sql.ResultSet;
-import java.sql.SQLException;
 import java.text.DateFormat;
 import java.util.ArrayList;
 import java.util.Date;
 import java.util.HashMap;
+import java.util.Iterator;
 import java.util.List;
 import java.util.Map;
+import java.util.Set;
+import java.util.TreeSet;
 
 /**
  * @author neil
@@ -79,7 +67,7 @@ public class InformationCollector implements XMLReadable, XMLWriter {
 
 	// 24小时上传一次
 	private static final long DELTA = 24 * 3600 * 1000L;
-	private static final long SEND_DELAY = 30 * 1000L;
+	private static final long SEND_DELAY = 300 * 1000L;
 	private static final String FILE_NAME = "fr.info";
 	private static final String XML_START_STOP_LIST = "StartStopList";
 	private static final String XML_START_STOP = "StartStop";
@@ -226,12 +214,7 @@ public class InformationCollector implements XMLReadable, XMLWriter {
         if (currentTime - lastTime <= DELTA) {
             return;
         }
-		JSONArray content = null;
+		JSONArray content = getFunctionsContent(currentTime, lastTime);
-		try {
-			content = getFunctionsContent(currentTime, lastTime);
-		} catch (JSONException e) {
-			FineLoggerFactory.getLogger().error(e.getMessage(), e);
-		}
 		boolean success = false;
 		FineLoggerFactory.getLogger().info("Start sent function records to the cloud center...");
 		String url = CloudCenter.getInstance().acquireUrlByKind(TABLE_FUNCTION_RECORD);
@@ -443,7 +426,7 @@ public class InformationCollector implements XMLReadable, XMLWriter {
         });
 	}
 
-	public static JSONArray getFunctionsContent(long current, long last) throws JSONException{
+	public JSONArray getFunctionsContent(long current, long last) {
 		//记录当前条数，达到200条合并成一个请求
 		int count = 0;
 		JSONArray functionArray = new JSONArray();
@@ -452,21 +435,20 @@ public class InformationCollector implements XMLReadable, XMLWriter {
 				.addRestriction(RestrictionFactory.gte(COLUMN_TIME, last));
 		try {
 			DataList<FocusPoint> focusPoints = MetricRegistry.getMetric().find(FocusPoint.class,condition);
-			JSONArray temp = new JSONArray();
+			TreeSet<FunctionRecord> focusPointsList = new TreeSet<>();
 			if(!focusPoints.isEmpty()){
 				for(int i=0;i<  focusPoints.getList().size();i++){
 					FocusPoint focusPoint = focusPoints.getList().get(i);
 					if(focusPoint != null){
 						if((++count <= MAX_EACH_REQUEST_RECORD_COUNT)){
-							temp.put(getOneRecord(focusPoint));
+							focusPointsList.add(getOneRecord(focusPoint));
 						} else {
 							count = 0;
-							functionArray.put(temp);
+							functionArray.put(setToJSONArray(focusPointsList));
-							temp = new JSONArray();
+							focusPointsList.add(getOneRecord(focusPoint));
-							temp.put(getOneRecord(focusPoint));
 						}
 						if(i == (focusPoints.getList().size() -1)){
-							functionArray.put(temp);
+							functionArray.put(setToJSONArray(focusPointsList));
 						}
 					}
 				}
@@ -478,16 +460,33 @@ public class InformationCollector implements XMLReadable, XMLWriter {
 		return functionArray;
 	}
 
-	private static JSONObject getOneRecord(FocusPoint focusPoint) throws JSONException{
+	private FunctionRecord getOneRecord(FocusPoint focusPoint) {
-		com.fr.json.JSONObject record = new com.fr.json.JSONObject();
+		FunctionRecord functionRecord = new FunctionRecord();
-		record.put(ATTR_ID, focusPoint.getId());
+		functionRecord.setId(focusPoint.getId() == null?StringUtils.EMPTY : focusPoint.getId());
-		record.put(ATTR_TEXT, focusPoint.getText());
+		functionRecord.setText(focusPoint.getText() == null?StringUtils.EMPTY : focusPoint.getText());
-		record.put(ATTR_SOURCE, focusPoint.getSource());
+		functionRecord.setSource(focusPoint.getSource());
-		record.put(ATTR_TIME, focusPoint.getTime().getTime());
+		functionRecord.setTime(focusPoint.getTime().getTime());
-		record.put(ATTR_TITLE, focusPoint.getTitle());
+		functionRecord.setTitle(focusPoint.getTitle() == null?StringUtils.EMPTY : focusPoint.getTitle());
-		record.put(ATTR_USER_NAME, MarketConfig.getInstance().getBbsUsername());
+		functionRecord.setUsername(MarketConfig.getInstance().getBbsUsername() == null?StringUtils.EMPTY : MarketConfig.getInstance().getBbsUsername());
-		record.put(ATTR_UUID, DesignerEnvManager.getEnvManager().getUUID());
+		functionRecord.setUuid(DesignerEnvManager.getEnvManager().getUUID() == null?StringUtils.EMPTY : DesignerEnvManager.getEnvManager().getUUID());
-		return record;
+		return functionRecord;
+	}
+
+	private JSONArray setToJSONArray(Set set) throws JSONException {
+		JSONArray jsonArray = new JSONArray();
+		for(Iterator iter = set.iterator(); iter.hasNext(); ) {
+			FunctionRecord functionRecord = (FunctionRecord)iter.next();
+			com.fr.json.JSONObject record = new com.fr.json.JSONObject();
+			record.put(ATTR_ID, functionRecord.getId());
+			record.put(ATTR_TEXT, functionRecord.getText());
+			record.put(ATTR_SOURCE, functionRecord.getSource());
+			record.put(ATTR_TIME, functionRecord.getTime());
+			record.put(ATTR_TITLE, functionRecord.getTitle());
+			record.put(ATTR_USER_NAME, functionRecord.getUsername());
+			record.put(ATTR_UUID, functionRecord.getUuid());
+			jsonArray.put(record);
+		}
+		return jsonArray;
 	}
 
 	private class StartStopTime implements XMLReadable, XMLWriter {
@@ -529,4 +528,84 @@ public class InformationCollector implements XMLReadable, XMLWriter {
 
 	}
 
+	private class FunctionRecord implements Comparable{
+		private String id;
+		private String text;
+		private int source;
+		private long time;
+		private String title;
+		private String username;
+		private String uuid;
+
+		public FunctionRecord(){
+
+		}
+		public String getId() {
+			return id;
+		}
+
+		public void setId(String id) {
+			this.id = id;
+		}
+
+		public String getText() {
+			return text;
+		}
+
+		public void setText(String text) {
+			this.text = text;
+		}
+
+		public int getSource() {
+			return source;
+		}
+
+		public void setSource(int source) {
+			this.source = source;
+		}
+
+		public long getTime() {
+			return time;
+		}
+
+		public void setTime(long time) {
+			this.time = time;
+		}
+
+		public String getTitle() {
+			return title;
+		}
+
+		public void setTitle(String title) {
+			this.title = title;
+		}
+
+		public String getUsername() {
+			return username;
+		}
+
+		public void setUsername(String username) {
+			this.username = username;
+		}
+
+		public String getUuid() {
+			return uuid;
+		}
+
+		public void setUuid(String uuid) {
+			this.uuid = uuid;
+		}
+
+		@Override
+		public int compareTo(Object o) {
+			FunctionRecord functionRecord = (FunctionRecord) o;
+			if(this.getId().equals((functionRecord.getId())) && this.getText().equals(functionRecord.getText())
+					&& this.getSource() == functionRecord.getSource() && this.getTime() == functionRecord.getTime()
+					&& this.getTitle().equals(functionRecord.getTitle()) && this.getUsername().equals(functionRecord.getUsername())
+					&& this.getUuid().equals(functionRecord.getUuid())){
+				return 0;
+			}
+			return 1;
+		}
+	}
 }