From d0f4f864d877509ad2fd1f3dcea974124d753a1c Mon Sep 17 00:00:00 2001 From: Max Medvedev Date: Thu, 21 Feb 2013 19:42:34 +0400 Subject: [PATCH] IDEA-98409 groovy create function generates return statement for def return value --- .../plugins/groovy/intentions/base/IntentionUtils.java | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/plugins/groovy/src/org/jetbrains/plugins/groovy/intentions/base/IntentionUtils.java b/plugins/groovy/src/org/jetbrains/plugins/groovy/intentions/base/IntentionUtils.java index 53637fb6999f..8e3ae1edd75b 100644 --- a/plugins/groovy/src/org/jetbrains/plugins/groovy/intentions/base/IntentionUtils.java +++ b/plugins/groovy/src/org/jetbrains/plugins/groovy/intentions/base/IntentionUtils.java @@ -34,6 +34,7 @@ import org.jetbrains.plugins.groovy.annotator.intentions.QuickfixUtil; import org.jetbrains.plugins.groovy.lang.psi.GroovyPsiElementFactory; import org.jetbrains.plugins.groovy.lang.psi.api.statements.GrStatement; import org.jetbrains.plugins.groovy.lang.psi.api.statements.expressions.GrExpression; +import org.jetbrains.plugins.groovy.lang.psi.api.statements.typedef.members.GrMethod; import org.jetbrains.plugins.groovy.lang.psi.expectedTypes.TypeConstraint; import org.jetbrains.plugins.groovy.template.expressions.ChooseTypeExpression; import org.jetbrains.plugins.groovy.template.expressions.ParameterNameExpression; @@ -134,7 +135,14 @@ public class IntentionUtils { } if (method != null) { try { + final boolean hasNoReturnType = method.getReturnTypeElement() == null && method instanceof GrMethod; + if (hasNoReturnType) { + ((GrMethod)method).setReturnType(PsiType.VOID); + } CreateFromUsageUtils.setupMethodBody(method); + if (hasNoReturnType) { + ((GrMethod)method).setReturnType(null); + } } catch (IncorrectOperationException e) { LOG.error(e);