FIX: Cannot invoke "com.intellij.openapi.editor.Document.insertString(int, java.lang.CharSequence)" because "document" is null #386

Author: SeeSharpSoft

src/main/java/net/seesharpsoft/intellij/plugins/csv/inspection/CsvValidationInspection.java

@@ -121,8 +121,9 @@ public String getFamilyName() {
         public void applyFix(@NotNull Project project, @NotNull ProblemDescriptor descriptor) {
             PsiElement element = descriptor.getPsiElement();
             Document document = PsiDocumentManager.getInstance(project).getDocument(element.getContainingFile());
-            List<Integer> quotePositions = new ArrayList<>();
+            if (document == null) return;
 
+            List<Integer> quotePositions = new ArrayList<>();
             int quotePosition = CsvIntentionHelper.getOpeningQuotePosition(element);
             if (quotePosition != -1) {
                 quotePositions.add(quotePosition);
@@ -147,6 +148,8 @@ public void applyFix(@NotNull Project project, @NotNull ProblemDescriptor descri
             try {
                 PsiElement element = descriptor.getPsiElement();
                 Document document = PsiDocumentManager.getInstance(project).getDocument(element.getContainingFile());
+                if (document == null) return;
+
                 CsvValueSeparator separator = CsvHelper.getValueSeparator(element.getContainingFile());
                 String text = document.getText();
                 document.setText(text.substring(0, element.getTextOffset()) + separator.getCharacter() + text.substring(element.getTextOffset()));
@@ -166,6 +169,8 @@ public void applyFix(@NotNull Project project, @NotNull ProblemDescriptor descri
             try {
                 PsiElement element = descriptor.getPsiElement();
                 Document document = PsiDocumentManager.getInstance(project).getDocument(element.getContainingFile());
+                if (document == null) return;
+
                 document.setText(document.getText() + "\"");
             } catch (IncorrectOperationException e) {
                 LOG.error(e);

src/main/java/net/seesharpsoft/intellij/plugins/csv/intention/CsvIntentionHelper.java

@@ -47,8 +47,9 @@ public static Collection<PsiElement> getAllElements(PsiFile file) {
 
     public static void quoteAll(@NotNull Project project, @NotNull PsiFile psiFile) {
         Document document = PsiDocumentManager.getInstance(project).getDocument(psiFile);
-        List<Integer> quotePositions = new ArrayList<>();
+        if (document == null) return;
 
+        List<Integer> quotePositions = new ArrayList<>();
         PsiTreeUtil.processElements(psiFile, CsvField.class, field -> {
             if (PsiHelper.getElementType(field.getFirstChild()) != CsvTypes.QUOTE) {
                 quotePositions.add(field.getTextRange().getStartOffset());
@@ -63,6 +64,8 @@ public static void quoteAll(@NotNull Project project, @NotNull PsiFile psiFile)
 
     public static void quoteValue(@NotNull Project project, @NotNull final PsiElement field) {
         Document document = PsiDocumentManager.getInstance(project).getDocument(field.getContainingFile());
+        if (document == null) return;
+
         List<Integer> quotePositions = new ArrayList<>();
         if (PsiHelper.getElementType(field.getFirstChild()) != CsvTypes.QUOTE) {
             quotePositions.add(field.getTextRange().getStartOffset());
@@ -75,9 +78,9 @@ public static void quoteValue(@NotNull Project project, @NotNull final PsiElemen
 
     public static void unquoteAll(@NotNull Project project, @NotNull PsiFile psiFile) {
         Document document = PsiDocumentManager.getInstance(project).getDocument(psiFile);
+        if (document == null) return;
 
         final List<PsiElement> quotePositions = new ArrayList<>();
-
         PsiTreeUtil.processElements(psiFile, CsvField.class, field -> {
             if (getChildren(field).stream().noneMatch(element -> PsiHelper.getElementType(element) == CsvTypes.ESCAPED_TEXT)) {
                 Pair<PsiElement, PsiElement> positions = getQuotePositions(field);
@@ -115,7 +118,7 @@ private static Pair<PsiElement, PsiElement> getQuotePositions(PsiElement element
         return null;
     }
 
-    public static void addQuotes(final Document document, List<Integer> quotePositions) {
+    public static void addQuotes(@NotNull final Document document, List<Integer> quotePositions) {
         int offset = 0;
         String quote = "\"";
         quotePositions.sort(Integer::compareTo);