IDEA-118970 (external URL formatting for Javadoc 8)

java/java-impl/src/com/intellij/lang/java/JavaDocumentationProvider.java

@@ -49,6 +49,7 @@ import com.intellij.psi.javadoc.PsiDocComment;
 import com.intellij.psi.javadoc.PsiDocTag;
 import com.intellij.psi.util.PsiFormatUtil;
 import com.intellij.psi.util.PsiFormatUtilBase;
+import com.intellij.psi.util.PsiUtil;
 import com.intellij.util.containers.ContainerUtil;
 import com.intellij.util.containers.HashMap;
 import org.jetbrains.annotations.NonNls;
@@ -639,7 +640,14 @@ public class JavaDocumentationProvider implements CodeDocumentationProvider, Ext
       options |= PsiFormatUtilBase.SHOW_RAW_NON_TOP_TYPE;
       parameterOptions |= PsiFormatUtilBase.SHOW_RAW_NON_TOP_TYPE;
     }
-    return PsiFormatUtil.formatMethod(method, PsiSubstitutor.EMPTY, options, parameterOptions, 999);
+
+    String signature = PsiFormatUtil.formatMethod(method, PsiSubstitutor.EMPTY, options, parameterOptions, 999);
+
+    if (PsiUtil.isLanguageLevel8OrHigher(method)) {
+      signature = signature.replaceAll("\\(|\\)|, ", "-").replaceAll("\\[\\]", ":A");
+    }
+
+    return signature;
   }
 
   @Nullable

java/java-tests/testSrc/com/intellij/codeInsight/ExternalJavadocUrls7Test.java

@@ -0,0 +1,48 @@
+/*
+ * Copyright 2000-2014 JetBrains s.r.o.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package com.intellij.codeInsight;
+
+import com.intellij.testFramework.LightProjectDescriptor;
+import org.jetbrains.annotations.NotNull;
+
+public class ExternalJavadocUrls7Test extends ExternalJavadocUrlsTest {
+  @NotNull
+  @Override
+  protected LightProjectDescriptor getProjectDescriptor() {
+    return JAVA_1_7;
+  }
+
+  @Override
+  public void testVarargs() {
+    doTest("class Test {\n" +
+           "  void <caret>foo(Class<?>... cl) { }\n" +
+           "}",
+
+           "foo(java.lang.Class...)", "foo(java.lang.Class<?>...)"
+    );
+  }
+
+  @Override
+  public void testTypeParams() {
+    doTest("class Test {\n" +
+           "  <T> void <caret>sort(T[] a, Comparator<? super T> c) { }\n" +
+           "}\n" +
+           "class Comparator<X>{}",
+
+           "sort(T[], Comparator)", "sort(T[], Comparator<? super T>)"
+    );
+  }
+}

java/java-tests/testSrc/com/intellij/codeInsight/ExternalJavadocUrlsTest.java

@@ -26,15 +26,11 @@ import com.intellij.util.containers.ContainerUtil;
 
 import java.util.List;
 
-/**
- * @author anna
- * @since 11/27/11
- */
 public class ExternalJavadocUrlsTest extends LightCodeInsightFixtureTestCase {
   @Override
   protected void setUp() throws Exception {
     super.setUp();
-    PsiTestUtil.setJavadocUrls(myModule, "http://doc");
+    PsiTestUtil.setJavadocUrls(myModule, "http://doc" );
   }
 
   public void testVarargs() {
@@ -42,7 +38,7 @@ public class ExternalJavadocUrlsTest extends LightCodeInsightFixtureTestCase {
            "  void <caret>foo(Class<?>... cl) { }\n" +
            "}",
 
-           "foo(java.lang.Class...)", "foo(java.lang.Class<?>...)");
+           "foo-java.lang.Class...-", "foo-java.lang.Class<?>...-");
 
   }
 
@@ -52,10 +48,10 @@ public class ExternalJavadocUrlsTest extends LightCodeInsightFixtureTestCase {
            "}\n" +
            "class Comparator<X>{}",
 
-           "sort(T[], Comparator)", "sort(T[], Comparator<? super T>)");
+           "sort-T:A-Comparator-", "sort-T:A-Comparator<? super T>-");
   }
 
-  private void doTest(String text, String... expected) {
+  protected void doTest(String text, String... expected) {
     myFixture.configureByText("Test.java", text);
     PsiElement elementAtCaret = myFixture.getElementAtCaret();
     PsiMethod member = PsiTreeUtil.getParentOfType(elementAtCaret, PsiMethod.class, false);