MDS/openide
1
0
Fork 0
mirror of https://gitflic.ru/project/openide/openide.git synced 2026-09-27 10:03:11 +07:00
Code Issues Packages Projects Releases Wiki Activity

KT-1648 Function0<Tuple0> appears in the result of the Implement abstract members quick fix

Browse Source 
#KT-1648 fixed

GitOrigin-RevId: 48f7b51625943d75639b0acb2f32bed94bef7a9e
This commit is contained in:
Nikolay Krasko
2012-03-27 21:11:07 +03:00
committed by intellij-monorepo-bot
parent ed8ba69ca4
commit 00d00ccddc
6 changed files with 51 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files
plugins/kotlin/idea/src/org/jetbrains/jet/plugin/codeInsight/ImplementMethodsHandler.java
+2
View File
@@ -34,12 +34,14 @@ import java.util.Set;
*/
public class ImplementMethodsHandler extends OverrideImplementMethodsHandler implements IntentionAction {
@Override
protected Set<CallableMemberDescriptor> collectMethodsToGenerate(MutableClassDescriptor descriptor) {
Set<CallableMemberDescriptor> missingImplementations = Sets.newLinkedHashSet();
OverrideResolver.collectMissingImplementations(descriptor, missingImplementations, missingImplementations);
return missingImplementations;
}
@Override
protected String getChooserTitle() {
return "Implement Members";
}
plugins/kotlin/idea/src/org/jetbrains/jet/plugin/codeInsight/OverrideImplementMethodsHandler.java
+23 -20
View File
@@ -36,6 +36,7 @@ import org.jetbrains.jet.lang.types.JetType;
import org.jetbrains.jet.lang.types.lang.JetStandardLibrary;
import org.jetbrains.jet.plugin.compiler.WholeProjectAnalyzerFacade;
import org.jetbrains.jet.plugin.quickfix.ImportInsertHelper;
import org.jetbrains.jet.resolve.DescriptorRenderer;
import java.util.ArrayList;
import java.util.Collections;
@@ -54,9 +55,10 @@ public abstract class OverrideImplementMethodsHandler implements LanguageCodeIns
return members;
}
public Set<CallableMemberDescriptor> collectMethodsToGenerate(JetClassOrObject classOrObject) {
@NotNull
public Set<CallableMemberDescriptor> collectMethodsToGenerate(@NotNull JetClassOrObject classOrObject) {
BindingContext bindingContext =
WholeProjectAnalyzerFacade.analyzeProjectWithCacheOnAFile((JetFile)classOrObject.getContainingFile());
WholeProjectAnalyzerFacade.analyzeProjectWithCacheOnAFile((JetFile)classOrObject.getContainingFile());
final DeclarationDescriptor descriptor = bindingContext.get(BindingContext.DECLARATION_TO_DESCRIPTOR, classOrObject);
if (descriptor instanceof MutableClassDescriptor) {
return collectMethodsToGenerate((MutableClassDescriptor)descriptor);
@@ -75,10 +77,6 @@ public abstract class OverrideImplementMethodsHandler implements LanguageCodeIns
return;
}
// NOTE + TODO: If you try to cache findInsertBeforeAnchor element, there will be failed assertion
// "PSI/document inconsistency before reparse: file=" from DocumentCommitThread after inserting two overriding
// with the caret right before existing function start.
PsiElement afterAnchor = findInsertAfterAnchor(editor, body);
if (afterAnchor == null) {
@@ -115,7 +113,7 @@ public abstract class OverrideImplementMethodsHandler implements LanguageCodeIns
}
});
if (offsetCursorElement != null) {
if (offsetCursorElement != null && offsetCursorElement != body.getRBrace()) {
afterAnchor = offsetCursorElement;
}
@@ -130,14 +128,15 @@ public abstract class OverrideImplementMethodsHandler implements LanguageCodeIns
else {
bodyBuilder.append("val ");
}
bodyBuilder.append(descriptor.getName()).append(":").append(descriptor.getType());
bodyBuilder.append(descriptor.getName()).append(" : ").append(DescriptorRenderer.COMPACT.renderTypeWithShortNames(
descriptor.getType()));
ImportInsertHelper.addImportDirectiveIfNeeded(descriptor.getType(), file);
String initializer = defaultInitializer(descriptor.getType(), JetStandardLibrary.getInstance());
if (initializer != null) {
bodyBuilder.append("=").append(initializer);
bodyBuilder.append(" = ").append(initializer);
}
else {
bodyBuilder.append("= ?");
bodyBuilder.append(" = ?");
}
return JetPsiFactory.createProperty(project, bodyBuilder.toString());
}
@@ -153,16 +152,17 @@ public abstract class OverrideImplementMethodsHandler implements LanguageCodeIns
}
first = false;
bodyBuilder.append(parameterDescriptor.getName());
bodyBuilder.append(":");
bodyBuilder.append(parameterDescriptor.getType().toString());
bodyBuilder.append(" : ");
bodyBuilder.append(DescriptorRenderer.COMPACT.renderTypeWithShortNames(parameterDescriptor.getType()));
ImportInsertHelper.addImportDirectiveIfNeeded(parameterDescriptor.getType(), file);
}
bodyBuilder.append(")");
final JetType returnType = descriptor.getReturnType();
final JetStandardLibrary stdlib = JetStandardLibrary.getInstance();
if (!returnType.equals(stdlib.getTuple0Type())) {
bodyBuilder.append(":").append(returnType.toString());
if (returnType != null && !stdlib.getTuple0Type().equals(returnType)) {
bodyBuilder.append(" : ").append(returnType.toString());
ImportInsertHelper.addImportDirectiveIfNeeded(returnType, file);
}
@@ -183,6 +183,7 @@ public abstract class OverrideImplementMethodsHandler implements LanguageCodeIns
else if (returnType.equals(stdlib.getBooleanType())) {
return "false";
}
return null;
}
@@ -209,10 +210,12 @@ public abstract class OverrideImplementMethodsHandler implements LanguageCodeIns
protected abstract String getNoMethodsFoundHint();
public void invoke(@NotNull final Project project, @NotNull final Editor editor, @NotNull PsiFile file,
boolean implementAll) {
public void invoke(@NotNull final Project project, @NotNull final Editor editor, @NotNull PsiFile file, boolean implementAll) {
final PsiElement elementAtCaret = file.findElementAt(editor.getCaretModel().getOffset());
final JetClassOrObject classOrObject = PsiTreeUtil.getParentOfType(elementAtCaret, JetClassOrObject.class);
assert classOrObject != null : "ClassObject should be checked in isValidFor method";
Set<CallableMemberDescriptor> missingImplementations = collectMethodsToGenerate(classOrObject);
if (missingImplementations.isEmpty() && !implementAll) {
HintManager.getInstance().showErrorHint(editor, getNoMethodsFoundHint());
@@ -225,10 +228,10 @@ public abstract class OverrideImplementMethodsHandler implements LanguageCodeIns
selectedElements = members;
}
else {
final MemberChooser<DescriptorClassMember> chooser = showOverrideImplementChooser(project,
members.toArray(
new DescriptorClassMember[members
.size()]));
final MemberChooser<DescriptorClassMember> chooser = showOverrideImplementChooser(
project,
members.toArray(new DescriptorClassMember[members.size()]));
if (chooser == null) {
return;
}
plugins/kotlin/idea/src/org/jetbrains/jet/plugin/codeInsight/OverrideMethodsHandler.java
+2
View File
@@ -27,6 +27,7 @@ import java.util.Set;
* @author yole
*/
public class OverrideMethodsHandler extends OverrideImplementMethodsHandler {
@Override
protected Set<CallableMemberDescriptor> collectMethodsToGenerate(MutableClassDescriptor descriptor) {
final Set<CallableMemberDescriptor> superMethods = OverrideResolver.collectSuperMethods(descriptor).keySet();
for (CallableMemberDescriptor member : descriptor.getCallableMembers()) {
@@ -41,6 +42,7 @@ public class OverrideMethodsHandler extends OverrideImplementMethodsHandler {
return result;
}
@Override
protected String getChooserTitle() {
return "Override Members";
}
plugins/kotlin/idea/tests/test/org/jetbrains/jet/plugin/codeInsight/OverrideImplementTest.java
+6
View File
@@ -49,6 +49,10 @@ public class OverrideImplementTest extends LightCodeInsightFixtureTestCase {
myFixture.setTestDataPath(PluginTestCaseBase.getTestDataPathBase() + "/codeInsight/overrideImplement");
}
public void testFunctionProperty() {
doFileTest();
}
public void testJavaInterfaceMethod() {
doDirectoryTest();
}
@@ -103,6 +107,7 @@ public class OverrideImplementTest extends LightCodeInsightFixtureTestCase {
private void doImplement() {
final PsiElement elementAtCaret = myFixture.getFile().findElementAt(myFixture.getEditor().getCaretModel().getOffset());
final JetClassOrObject classOrObject = PsiTreeUtil.getParentOfType(elementAtCaret, JetClassOrObject.class);
assertNotNull("Caret should be inside class or object", classOrObject);
final Set<CallableMemberDescriptor> descriptors = new ImplementMethodsHandler().collectMethodsToGenerate(classOrObject);
assertEquals("Invalid number of available descriptors for override", 1, descriptors.size());
new WriteCommandAction(myFixture.getProject(), myFixture.getFile()) {
@@ -118,6 +123,7 @@ public class OverrideImplementTest extends LightCodeInsightFixtureTestCase {
private void doMultiImplement() {
final PsiElement elementAtCaret = myFixture.getFile().findElementAt(myFixture.getEditor().getCaretModel().getOffset());
final JetClassOrObject classOrObject = PsiTreeUtil.getParentOfType(elementAtCaret, JetClassOrObject.class);
assertNotNull("Caret should be inside class or object", classOrObject);
final Set<CallableMemberDescriptor> descriptors = new ImplementMethodsHandler().collectMethodsToGenerate(classOrObject);
final ArrayList<CallableMemberDescriptor> descriptorsList = new ArrayList<CallableMemberDescriptor>(descriptors);
plugins/kotlin/idea/tests/testData/codeInsight/overrideImplement/functionProperty.kt
+8
View File
@@ -0,0 +1,8 @@
// From KT-1648
trait A {
val method:() -> Unit?
}
fun some() : A {
return object A {<caret>}
}
plugins/kotlin/idea/tests/testData/codeInsight/overrideImplement/functionProperty.kt.after
+10
View File
@@ -0,0 +1,10 @@
// From KT-1648
trait A {
val method:() -> Unit?
}
fun some() : A {
return object A {
override val method : () -> Unit? = ?
}
}