REPORT-13717 打开不同工作目录下的同名模板

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

@@ -183,10 +183,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) {
-                //不是通过关闭按钮，而是通过文件关闭菜单关闭的当前模板的，也重新激活
-                if (selectedIndex > openedTemplate.size() - 1) {
-                    selectedIndex -= 1;
-                    if (selectedIndex < 0) {
-                        selectedIndex = 0;
-                    }
                 isCloseCurrent = false;
             }
-            } else {
+            // 如果关闭的模板不是当前选中的模板，那么重新获取一下当前模板的 index,激活该 index
-                //如果关闭的模板不是当前选中的模板，则激活的模板不变
+            else {
-                selectedIndex = HistoryTemplateListCache.getInstance().contains(fileName);
+                JTemplate template = HistoryTemplateListCache.getInstance().getCurrentEditingTemplate();
+                selectedIndex = HistoryTemplateListCache.getInstance().contains(template);
             }
             //如果是已后台关闭的模板，则重新打开文件
             openedTemplate.get(selectedIndex).activeOldJTemplate();

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

@@ -29,6 +29,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;
@@ -76,7 +77,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;
@@ -160,9 +160,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() {
 
@@ -849,18 +849,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()));
                 }
             }
@@ -957,8 +957,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;
         }
@@ -968,7 +972,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")
@@ -1027,7 +1031,7 @@ public class DesignerFrame extends JFrame implements JTemplateActionListener, Ta
                         MutilTempalteTabPane.getInstance().setTemTemplate(
                                 HistoryTemplateListPane.getInstance().getCurrentEditingTemplate());
                     } else {
-                        activeTemplate(tplFile, jt);
+                        activeTemplate(jt);
                     }
                     opened = true;
                     break;
@@ -1042,13 +1046,12 @@ public class DesignerFrame extends JFrame implements JTemplateActionListener, Ta
     /**
      * 激活指定的模板
      *
-     * @param tplFile 模板文件
      * @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);