Browse Source

无任务,代码规范

master
yaoh.wu 6 years ago
parent
commit
3b93198447
  1. 4
      designer-base/src/com/fr/design/gui/itree/filetree/TemplateFileTree.java
  2. 5
      designer-base/src/com/fr/design/remote/action/RemoteDesignAuthorityManagerAction.java
  3. 4
      designer-base/src/com/fr/design/remote/ui/AuthorityListControlPane.java

4
designer-base/src/com/fr/design/gui/itree/filetree/TemplateFileTree.java

@ -1,13 +1,13 @@
package com.fr.design.gui.itree.filetree; package com.fr.design.gui.itree.filetree;
import com.fr.base.Env; import com.fr.base.Env;
import com.fr.base.FRContext;
import com.fr.base.env.EnvContext; import com.fr.base.env.EnvContext;
import com.fr.base.env.proxy.EnvProxy; import com.fr.base.env.proxy.EnvProxy;
import com.fr.base.env.resource.EnvConfigUtils; import com.fr.base.env.resource.EnvConfigUtils;
import com.fr.design.gui.itree.refreshabletree.ExpandMutableTreeNode; import com.fr.design.gui.itree.refreshabletree.ExpandMutableTreeNode;
import com.fr.env.operator.file.TplFileOperator; import com.fr.env.operator.file.TplFileOperator;
import com.fr.file.filetree.FileNode; import com.fr.file.filetree.FileNode;
import com.fr.log.FineLoggerFactory;
import com.fr.stable.ArrayUtils; import com.fr.stable.ArrayUtils;
import com.fr.stable.StableUtils; import com.fr.stable.StableUtils;
import com.fr.stable.project.ProjectConstants; import com.fr.stable.project.ProjectConstants;
@ -168,7 +168,7 @@ public class TemplateFileTree extends EnvFileTree {
try { try {
fileNodes = listFile(filePath); fileNodes = listFile(filePath);
} catch (Exception e) { } catch (Exception e) {
FRContext.getLogger().error(e.getMessage(), e); FineLoggerFactory.getLogger().error(e.getMessage(), e);
} }
if (fileNodes == null) { if (fileNodes == null) {
fileNodes = new FileNode[0]; fileNodes = new FileNode[0];

5
designer-base/src/com/fr/design/remote/action/RemoteDesignAuthorityManagerAction.java

@ -10,6 +10,7 @@ import com.fr.design.mainframe.DesignerContext;
import com.fr.design.remote.ui.AuthorityManagerPane; import com.fr.design.remote.ui.AuthorityManagerPane;
import com.fr.env.operator.authority.AuthorityOperator; import com.fr.env.operator.authority.AuthorityOperator;
import com.fr.general.Inter; import com.fr.general.Inter;
import com.fr.log.FineLoggerFactory;
import com.fr.report.DesignAuthority; import com.fr.report.DesignAuthority;
import java.awt.event.ActionEvent; import java.awt.event.ActionEvent;
@ -41,7 +42,7 @@ public class RemoteDesignAuthorityManagerAction extends UpdateAction {
managerPane.populate(authorities); managerPane.populate(authorities);
} }
} catch (Exception exception) { } catch (Exception exception) {
FRContext.getLogger().error(exception.getMessage()); FineLoggerFactory.getLogger().error(exception.getMessage(), exception);
} }
} }
@ -54,7 +55,7 @@ public class RemoteDesignAuthorityManagerAction extends UpdateAction {
try { try {
success = EnvProxy.get(AuthorityOperator.class).updateAuthorities(authorities); success = EnvProxy.get(AuthorityOperator.class).updateAuthorities(authorities);
} catch (Exception e) { } catch (Exception e) {
FRContext.getLogger().error(e.getMessage()); FineLoggerFactory.getLogger().error(e.getMessage(), e);
} }
FRContext.getLogger().info("update remote design authority: " + success); FRContext.getLogger().info("update remote design authority: " + success);
} }

4
designer-base/src/com/fr/design/remote/ui/AuthorityListControlPane.java

@ -21,6 +21,7 @@ import com.fr.design.utils.gui.GUICoreUtils;
import com.fr.env.RemoteDesignMember; import com.fr.env.RemoteDesignMember;
import com.fr.general.ComparatorUtils; import com.fr.general.ComparatorUtils;
import com.fr.general.Inter; import com.fr.general.Inter;
import com.fr.log.FineLoggerFactory;
import com.fr.report.DesignAuthority; import com.fr.report.DesignAuthority;
import com.fr.stable.ArrayUtils; import com.fr.stable.ArrayUtils;
@ -324,7 +325,7 @@ public class AuthorityListControlPane extends BasicPane {
if (p[i] != null) { if (p[i] != null) {
try { try {
p[i].checkValid(); p[i].checkValid();
} catch (Exception e) { } catch (Exception ignore) {
return i; return i;
} }
} }
@ -339,6 +340,7 @@ public class AuthorityListControlPane extends BasicPane {
try { try {
checkValid(); checkValid();
} catch (Exception exp) { } catch (Exception exp) {
FineLoggerFactory.getLogger().error(exp.getMessage(), exp);
JOptionPane.showMessageDialog(AuthorityListControlPane.this, exp.getMessage()); JOptionPane.showMessageDialog(AuthorityListControlPane.this, exp.getMessage());
authorityList.setSelectedIndex(idx); authorityList.setSelectedIndex(idx);
return true; return true;

Loading…
Cancel
Save