From 53ce72fc8c7906fad0485060bd9db5a0d2800519 Mon Sep 17 00:00:00 2001
From: peter <peter@jetbrains.com>
Date: Wed, 25 Jul 2012 15:43:13 +0200
Subject: [PATCH] diagnose unexpected character in chained completion
 (EA-37306)

---
 .../completion/JavaChainLookupElement.java         | 14 +++++++++++++-
 .../codeInsight/lookup/PsiTypeLookupItem.java      |  2 +-
 .../diagnostic/errordialog/Attachment.java         | 10 +++++++++-
 3 files changed, 23 insertions(+), 3 deletions(-)

diff --git a/java/java-impl/src/com/intellij/codeInsight/completion/JavaChainLookupElement.java b/java/java-impl/src/com/intellij/codeInsight/completion/JavaChainLookupElement.java
index 5c80671f0496..67fabbba707c 100644
--- a/java/java-impl/src/com/intellij/codeInsight/completion/JavaChainLookupElement.java
+++ b/java/java-impl/src/com/intellij/codeInsight/completion/JavaChainLookupElement.java
@@ -19,12 +19,16 @@ import com.intellij.codeInsight.lookup.LookupElement;
 import com.intellij.codeInsight.lookup.LookupElementDecorator;
 import com.intellij.codeInsight.lookup.LookupElementPresentation;
 import com.intellij.codeInsight.lookup.TypedLookupItem;
+import com.intellij.diagnostic.LogMessageEx;
+import com.intellij.diagnostic.errordialog.Attachment;
+import com.intellij.openapi.diagnostic.Logger;
 import com.intellij.openapi.editor.Document;
 import com.intellij.openapi.util.ClassConditionKey;
 import com.intellij.openapi.util.text.StringUtil;
 import com.intellij.psi.*;
 import com.intellij.psi.codeStyle.CodeStyleManager;
 import com.intellij.psi.codeStyle.CodeStyleSettingsManager;
+import com.intellij.psi.impl.DebugUtil;
 import com.intellij.psi.util.PsiTreeUtil;
 import com.intellij.util.text.CharArrayUtil;
 import gnu.trove.THashSet;
@@ -37,6 +41,7 @@ import java.util.Set;
  * @author peter
  */
 public class JavaChainLookupElement extends LookupElementDecorator<LookupElement> implements TypedLookupItem {
+  private static final Logger LOG = Logger.getInstance("#com.intellij.codeInsight.completion.JavaChainLookupElement");
   public static final ClassConditionKey<JavaChainLookupElement> CLASS_CONDITION_KEY = ClassConditionKey.create(JavaChainLookupElement.class);
   private final LookupElement myQualifier;
 
@@ -112,7 +117,14 @@ public class JavaChainLookupElement extends LookupElementDecorator<LookupElement
     }
 
     final char atTail = document.getCharsSequence().charAt(context.getTailOffset() - 1);
-    assert atTail == ';' : atTail;
+    if (atTail != ';') {
+      LOG.error(LogMessageEx.createEvent("Unexpected character",
+                                         "atTail=" + atTail + "\n" +
+                                         "offset=" + context.getTailOffset() + "\n" +
+                                         DebugUtil.currentStackTrace(),
+                                         new Attachment(context.getDocument())));
+
+    }
     document.replaceString(context.getTailOffset() - 1, context.getTailOffset(), ".");
 
     CompletionUtil.emulateInsertion(getDelegate(), context.getTailOffset(), context);
diff --git a/java/java-impl/src/com/intellij/codeInsight/lookup/PsiTypeLookupItem.java b/java/java-impl/src/com/intellij/codeInsight/lookup/PsiTypeLookupItem.java
index 049ccc3cdf66..565e06583499 100644
--- a/java/java-impl/src/com/intellij/codeInsight/lookup/PsiTypeLookupItem.java
+++ b/java/java-impl/src/com/intellij/codeInsight/lookup/PsiTypeLookupItem.java
@@ -244,7 +244,7 @@ public class PsiTypeLookupItem extends LookupItem {
                                          "offset=" + newTail + "\n" +
                                          "document=" + context.getDocument() + "\n" +
                                          DebugUtil.currentStackTrace(),
-                                         new Attachment(context.getFile().getViewProvider().getVirtualFile())));
+                                         new Attachment(context.getDocument())));
 
     }
     context.setTailOffset(newTail);
diff --git a/platform/platform-impl/src/com/intellij/diagnostic/errordialog/Attachment.java b/platform/platform-impl/src/com/intellij/diagnostic/errordialog/Attachment.java
index 9153217dc5c7..e56412cff06a 100644
--- a/platform/platform-impl/src/com/intellij/diagnostic/errordialog/Attachment.java
+++ b/platform/platform-impl/src/com/intellij/diagnostic/errordialog/Attachment.java
@@ -1,7 +1,8 @@
 package com.intellij.diagnostic.errordialog;
 
+import com.intellij.openapi.editor.Document;
+import com.intellij.openapi.fileEditor.FileDocumentManager;
 import com.intellij.openapi.fileEditor.impl.LoadTextUtil;
-import com.intellij.openapi.util.io.FileUtil;
 import com.intellij.openapi.vfs.VirtualFile;
 import com.intellij.util.ArrayUtil;
 import com.intellij.util.Base64Converter;
@@ -27,6 +28,13 @@ public class Attachment {
     myBytes = getBytes(content);
   }
 
+  public Attachment(Document document) {
+    VirtualFile file = FileDocumentManager.getInstance().getFile(document);
+    myPath = file != null ? file.getPath() : "unknown.txt";
+    myDisplayText = document.getText();
+    myBytes = getBytes(myDisplayText);
+  }
+
   public Attachment(@NotNull VirtualFile file) {
     myPath = file.getPresentableUrl();
     myBytes = getBytes(file);