Merge pull request #883 from SeeSharpSoft/feat_401

Release 4.0.1

Author: SeeSharpSoft

CHANGELOG.md

@@ -10,7 +10,25 @@
 
 ### Fixed
 
-## [4.0.0] - Oct 07, 2024
+## [4.0.1] - Dec 01, 2024
+
+### Added
+
+- Localization support
+
+### Changed
+
+- Upgrade to gradle 8.10
+- Improve check for CsvFile
+- Code cleanup
+
+### Fixed
+
+- Read access is allowed from inside read-action only #878
+- Failsafe acceptCsvFile check #882
+- Use ProjectActivity instead of StartupActivity
+
+## 4.0.0 - Oct 07, 2024
 
 ### Added
 - Tabularize formatting is back!

gradle.properties

@@ -4,7 +4,7 @@
 
 pluginName=CSV Editor
 pluginId=net.seesharpsoft.intellij.plugins.csv
-pluginVersion=4.0.0
+pluginVersion=4.0.1
 
 pluginSinceBuild=242
 
@@ -19,4 +19,3 @@ org.gradle.parallel=true
 # Opt-out flag for bundling Kotlin standard library.
 # See https://plugins.jetbrains.com/docs/intellij/kotlin.html#kotlin-standard-library for details.
 kotlin.stdlib.default.dependency=false
-

gradle/wrapper/gradle-wrapper.properties

@@ -1,6 +1,6 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-8.9-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-8.10-bin.zip
 networkTimeout=10000
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists

src/main/java/net/seesharpsoft/intellij/plugins/csv/CsvColumnInfo.java

@@ -108,7 +108,7 @@ public class RowInfo {
             this(element, row, -1, -1);
         }
 
-        RowInfo(@NotNull T element, @NotNull int row, int startIndex, int endIndex) {
+        RowInfo(@NotNull T element, int row, int startIndex, int endIndex) {
             this.myElement = element;
             this.myRow = row;
             if (startIndex <= endIndex && startIndex >= 0) {
@@ -140,7 +140,7 @@ public boolean equals(Object other) {
             if (!(other instanceof CsvColumnInfo.RowInfo)) {
                 return false;
             }
-            return this.myElement.equals(((RowInfo) other).myElement);
+            return this.myElement.equals(((CsvColumnInfo<?>.RowInfo) other).myElement);
         }
     }
 }

src/main/java/net/seesharpsoft/intellij/plugins/csv/CsvHelper.java

@@ -2,7 +2,6 @@
 
 import com.intellij.lang.*;
 import com.intellij.lexer.Lexer;
-import com.intellij.openapi.fileTypes.FileType;
 import com.intellij.openapi.fileTypes.FileTypeRegistry;
 import com.intellij.openapi.project.Project;
 import com.intellij.openapi.vfs.VirtualFile;
@@ -22,8 +21,6 @@
 import net.seesharpsoft.intellij.plugins.csv.psi.CsvRecord;
 import net.seesharpsoft.intellij.plugins.csv.psi.CsvTypes;
 import net.seesharpsoft.intellij.plugins.csv.settings.CsvEditorSettings;
-import net.seesharpsoft.intellij.plugins.psv.PsvFileType;
-import net.seesharpsoft.intellij.plugins.tsv.TsvFileType;
 import net.seesharpsoft.intellij.psi.PsiHelper;
 import org.jetbrains.annotations.NotNull;
 
@@ -58,26 +55,19 @@ public static boolean isCsvFile(String extension) {
         if (extension == null) {
             return false;
         }
-        // simple check to always in include the defaults even if association was removed
-        switch (extension.toLowerCase()) {
-            case "csv":
-            case "tsv":
-            case "tab":
-            case "psv":
-                return true;
-            default:
-                // but also consider other extensions that are associated manually
-                FileType fileType = FileTypeRegistry.getInstance().getFileTypeByExtension(extension);
-                return fileType == CsvFileType.INSTANCE ||
-                        fileType == TsvFileType.INSTANCE ||
-                        fileType == PsvFileType.INSTANCE;
-        }
+        Language language = LanguageUtil.getFileTypeLanguage(
+                FileTypeRegistry.getInstance().getFileTypeByExtension(extension)
+        );
+        return language != null && language.isKindOf(CsvLanguage.INSTANCE);
     }
 
     public static boolean isCsvFile(Project project, VirtualFile file) {
-        if (project == null || file == null || !isCsvFile(file.getExtension())) {
+        if (file == null) {
             return false;
         }
+        if (project == null) {
+            return isCsvFile(file.getExtension());
+        }
         final Language language = LanguageUtil.getLanguageForPsi(project, file);
         return language != null && language.isKindOf(CsvLanguage.INSTANCE);
     }

src/main/java/net/seesharpsoft/intellij/plugins/csv/CsvPlugin.java

@@ -9,15 +9,23 @@
 import com.intellij.openapi.progress.ProgressIndicator;
 import com.intellij.openapi.progress.ProgressManager;
 import com.intellij.openapi.progress.Task;
+import com.intellij.openapi.project.DumbAware;
 import com.intellij.openapi.project.Project;
-import com.intellij.openapi.startup.StartupActivity;
+import com.intellij.openapi.startup.ProjectActivity;
+import kotlin.Unit;
+import kotlin.coroutines.Continuation;
 import net.seesharpsoft.intellij.plugins.csv.components.CsvFileAttributes;
 import net.seesharpsoft.intellij.plugins.csv.settings.CsvEditorSettings;
 import net.seesharpsoft.intellij.plugins.csv.settings.CsvEditorSettingsProvider;
 import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
 
-public class CsvPlugin implements StartupActivity, StartupActivity.DumbAware {
+import java.util.ResourceBundle;
 
+public class CsvPlugin implements ProjectActivity, DumbAware {
+
+    private static ResourceBundle _resourceBundle;
+    
     protected static IdeaPluginDescriptor getPluginDescriptor() {
         return PluginManagerCore.getPlugin(PluginId.getId("net.seesharpsoft.intellij.plugins.csv"));
     }
@@ -66,12 +74,12 @@ public void run(@NotNull ProgressIndicator progressIndicator) {
     }
 
     @Override
-    public void runActivity(@NotNull Project project) {
+    public @Nullable Object execute(@NotNull Project project, @NotNull Continuation<? super Unit> continuation) {
         doAsyncProjectMaintenance(project);
-
+        
         NotificationGroup notificationGroup = NotificationGroupManager.getInstance().getNotificationGroup("net.seesharpsoft.intellij.plugins.csv");
         if (notificationGroup == null || CsvEditorSettings.getInstance().checkCurrentPluginVersion(getVersion())) {
-            return;
+            return continuation;
         }
 
         Notification notification = notificationGroup.createNotification(
@@ -100,5 +108,20 @@ public void runActivity(@NotNull Project project) {
         }));
 
         Notifications.Bus.notify(notification);
+        
+        return continuation;
+    }
+    
+    public static ResourceBundle getResourceBundle() {
+        if (_resourceBundle == null) {
+            _resourceBundle = ResourceBundle.getBundle("i18n/CSVEditorResources");
+        }
+        return _resourceBundle;
     }
+
+    public static String getLocalizedText(String token) {
+        return getResourceBundle().getString(token);
+    }
+
+
 }

src/main/java/net/seesharpsoft/intellij/plugins/csv/actions/CsvCustomSeparatorAction.java

@@ -41,7 +41,7 @@ public void setSelected(@NotNull AnActionEvent anActionEvent, boolean selected)
         if (customValueSeparator == null) {
             return;
         }
-        if (customValueSeparator.length() == 0 || customValueSeparator.contains(" ")) {
+        if (customValueSeparator.isEmpty() || customValueSeparator.contains(" ")) {
             JOptionPane.showMessageDialog(fileEditor == null ? null : fileEditor.getComponent(), "Value separator must have at least one character and no spaces!");
             return;
         }

src/main/java/net/seesharpsoft/intellij/plugins/csv/editor/CsvFileEditorProvider.java

@@ -18,10 +18,14 @@ public class CsvFileEditorProvider implements AsyncFileEditorProvider, DumbAware
     public static final String EDITOR_TYPE_ID = "csv-text-editor";
 
     public static boolean acceptCsvFile(@NotNull Project project, @NotNull VirtualFile file) {
-        return CsvHelper.isCsvFile(project, file)
-                && !SingleRootFileViewProvider.isTooLargeForContentLoading(file)
-                && !SingleRootFileViewProvider.isTooLargeForIntelligence(file)
-                && !(file instanceof DiffViewerVirtualFile);
+        try {
+            return !SingleRootFileViewProvider.isTooLargeForContentLoading(file)
+                    && !SingleRootFileViewProvider.isTooLargeForIntelligence(file)
+                    && !(file instanceof DiffViewerVirtualFile)
+                    && CsvHelper.isCsvFile(project, file);
+        } catch(Exception exc) {
+            return false;
+        }
     }
 
     @Override

src/main/java/net/seesharpsoft/intellij/plugins/csv/editor/table/CsvTableEditorProvider.java

@@ -18,7 +18,7 @@ public class CsvTableEditorProvider implements AsyncFileEditorProvider, DumbAwar
     public static final String EDITOR_TYPE_ID = "csv-table-editor";
 
     @Override
-    public String getEditorTypeId() {
+    public @NotNull String getEditorTypeId() {
         return EDITOR_TYPE_ID;
     }
 
@@ -48,16 +48,15 @@ public FileEditor createEditor(@NotNull Project project, @NotNull VirtualFile vi
     }
 
     @Override
-    public FileEditorState readState(@NotNull Element sourceElement, @NotNull Project project, @NotNull VirtualFile file) {
+    public @NotNull FileEditorState readState(@NotNull Element sourceElement, @NotNull Project project, @NotNull VirtualFile file) {
         return CsvTableEditorState.create(sourceElement, project, file);
     }
 
     @Override
     public void writeState(@NotNull FileEditorState state, @NotNull Project project, @NotNull Element targetElement) {
-        if (!(state instanceof CsvTableEditorState)) {
+        if (!(state instanceof CsvTableEditorState csvTableEditorState)) {
             return;
         }
-        CsvTableEditorState csvTableEditorState = (CsvTableEditorState) state;
         csvTableEditorState.write(project, targetElement);
     }
 
@@ -66,7 +65,7 @@ public void writeState(@NotNull FileEditorState state, @NotNull Project project,
     public Builder createEditorAsync(@NotNull Project project, @NotNull VirtualFile virtualFile) {
         return new Builder() {
             @Override
-            public FileEditor build() {
+            public @NotNull FileEditor build() {
                 return new CsvTableEditorSwing(project, virtualFile);
             }
         };

src/main/java/net/seesharpsoft/intellij/plugins/csv/editor/table/CsvTableModelBase.java

@@ -92,7 +92,7 @@ private void resetCachedValues() {
     }
 
     private void resetPointer() {
-        myPointedRecord = PsiTreeUtil.findChildOfType(getPsiFile(), CsvRecord.class);
+        myPointedRecord = PsiHelper.getFirstChildOfType(getPsiFile(), CsvRecord.class);
         myPointedRow = 0;
     }