From ac0c319ae4ee360205e06168af8049cff0d35961 Mon Sep 17 00:00:00 2001
From: Lucian <author@corpmail.com>
Date: Fri, 1 Nov 2019 17:31:50 +0800
Subject: [PATCH] =?UTF-8?q?REPORT-23115=20=E8=BF=9C=E7=A8=8B=E8=AE=BE?=
 =?UTF-8?q?=E8=AE=A1=E6=9D=83=E9=99=90=E9=85=8D=E7=BD=AE-=E7=89=88?=
 =?UTF-8?q?=E6=9C=AC=E4=B8=8D=E5=8C=B9=E9=85=8D=E6=97=B6=E7=94=A8=E6=88=B7?=
 =?UTF-8?q?=E8=A7=92=E8=89=B2=E5=88=97=E8=A1=A8=E9=83=BD=E4=B8=BA=E7=A9=BA?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../com/fr/design/file/NodeAuthProcessor.java | 11 ++++++--
 .../action/RemoteDesignAuthManagerAction.java | 28 ++++++++++++++-----
 2 files changed, 30 insertions(+), 9 deletions(-)

diff --git a/designer-base/src/main/java/com/fr/design/file/NodeAuthProcessor.java b/designer-base/src/main/java/com/fr/design/file/NodeAuthProcessor.java
index 2a1c0cc94..a1082b62f 100644
--- a/designer-base/src/main/java/com/fr/design/file/NodeAuthProcessor.java
+++ b/designer-base/src/main/java/com/fr/design/file/NodeAuthProcessor.java
@@ -13,6 +13,7 @@ import com.fr.workspace.WorkContext;
 import com.fr.workspace.server.authority.AuthorityOperator;
 import com.fr.workspace.server.authority.decision.DecisionOperator;
 
+import java.lang.reflect.UndeclaredThrowableException;
 import java.util.ArrayList;
 
 public class NodeAuthProcessor {
@@ -41,9 +42,15 @@ public class NodeAuthProcessor {
         if (!WorkContext.getCurrent().isLocal()) {
             try {
                 String userName = WorkContext.getCurrent().getConnection().getUserName();
-                String userId = WorkContext.getCurrent().get(DecisionOperator.class).getUserIdByName(userName);
+                DesignAuthority[] authorities = null;
+                try {
+                    String userId = WorkContext.getCurrent().get(DecisionOperator.class).getUserIdByName(userName);
+                    authorities = WorkContext.getCurrent().get(AuthorityOperator.class).getAuthorities(userId);
+                } catch(UndeclaredThrowableException e) {
+                    // 兼容旧版本的服务器
+                    authorities = WorkContext.getCurrent().get(AuthorityOperator.class).getAuthorities();
+                }
                 // 远程设计获取设计成员的权限列表
-                DesignAuthority[] authorities = WorkContext.getCurrent().get(AuthorityOperator.class).getAuthorities(userId);
                 DesignAuthority authority = null;
 
                 if (authorities != null) {
diff --git a/designer-base/src/main/java/com/fr/design/remote/action/RemoteDesignAuthManagerAction.java b/designer-base/src/main/java/com/fr/design/remote/action/RemoteDesignAuthManagerAction.java
index 63929685d..f48cb12ed 100644
--- a/designer-base/src/main/java/com/fr/design/remote/action/RemoteDesignAuthManagerAction.java
+++ b/designer-base/src/main/java/com/fr/design/remote/action/RemoteDesignAuthManagerAction.java
@@ -9,11 +9,14 @@ import com.fr.design.mainframe.DesignerContext;
 import com.fr.design.remote.ui.AuthorityManagerPane;
 import com.fr.log.FineLoggerFactory;
 import com.fr.report.DesignAuthority;
+import com.fr.report.constant.RoleType;
 import com.fr.stable.ArrayUtils;
 import com.fr.workspace.WorkContext;
 import com.fr.workspace.server.authority.AuthorityOperator;
 
 import java.awt.event.ActionEvent;
+import java.util.ArrayList;
+import java.util.List;
 
 /**
  * @author yaohwu
@@ -38,13 +41,24 @@ public class RemoteDesignAuthManagerAction extends UpdateAction {
         if (!WorkContext.getCurrent().isLocal()) {
             try {
                 // 远程设计获取全部设计成员的权限列表
-                DesignAuthority[] userAuthorities = WorkContext.getCurrent().get(AuthorityOperator.class).getUserAuthorities();
-                DesignAuthority[] customAuthorities = WorkContext.getCurrent().get(AuthorityOperator.class).getCustomRoleAuthorities();
-                if (userAuthorities != null && userAuthorities.length != 0) {
-                    managerPane.populateByUser(userAuthorities);
-                }
-                if (customAuthorities != null && customAuthorities.length != 0) {
-                    managerPane.populateByCustom(customAuthorities);
+                DesignAuthority[] authorities = WorkContext.getCurrent().get(AuthorityOperator.class).getAuthorities();
+                List<DesignAuthority> userAuthorities = new ArrayList<DesignAuthority>();
+                List<DesignAuthority> customAuthorities = new ArrayList<DesignAuthority>();
+                if (authorities != null) {
+                    for (DesignAuthority authority : authorities) {
+                        if (authority.getRoleType() == RoleType.CUSTOM) {
+                            customAuthorities.add(authority);
+                        }
+                        else {
+                            userAuthorities.add(authority);
+                        }
+                    }
+                    if (userAuthorities.size() != 0) {
+                        managerPane.populateByUser(userAuthorities.toArray(new DesignAuthority[userAuthorities.size()]));
+                    }
+                    if (customAuthorities.size() != 0) {
+                        managerPane.populateByCustom(customAuthorities.toArray(new DesignAuthority[customAuthorities.size()]));
+                    }
                 }
             } catch (Exception exception) {
                 FineLoggerFactory.getLogger().error(exception.getMessage(), exception);