Skip to content

Commit 8d52fd1

Browse files
1035: Code refactoring
1 parent af4b9c0 commit 8d52fd1

File tree

1 file changed

+10
-11
lines changed

1 file changed

+10
-11
lines changed

src/com/magento/idea/magento2plugin/actions/generation/util/FillTextBufferWithPluginMethods.java

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import com.jetbrains.php.lang.psi.PhpPsiElementFactory;
1212
import com.jetbrains.php.lang.psi.elements.Method;
1313
import com.jetbrains.php.lang.psi.elements.Parameter;
14-
import com.jetbrains.php.lang.psi.elements.PhpReturnType;
1514
import com.magento.idea.magento2plugin.actions.generation.data.code.PluginMethodData;
1615
import com.magento.idea.magento2plugin.actions.generation.generator.util.PhpClassGeneratorUtil;
1716
import com.magento.idea.magento2plugin.actions.generation.references.PhpClassReferenceResolver;
@@ -57,18 +56,18 @@ public void execute(
5756
.getUserData(targetClassKey);
5857
resolver.processElement(targetClass);
5958

60-
final String returnTypeFqn =
61-
PhpTypeMetadataParserUtil.getMethodReturnType(targetMethod);
59+
final String returnTypeCandidate = PhpTypeMetadataParserUtil.getMethodReturnType(
60+
targetMethod
61+
);
6262

63-
if (returnTypeFqn != null && PhpClassGeneratorUtil.isValidFqn(returnTypeFqn)) {
64-
final PhpReturnType returnType = PhpPsiElementFactory.createReturnType(
65-
pluginMethod.getTargetMethod().getProject(),
66-
returnTypeFqn
63+
if (returnTypeCandidate != null
64+
&& PhpClassGeneratorUtil.isValidFqn(returnTypeCandidate)) {
65+
resolver.processElement(
66+
PhpPsiElementFactory.createReturnType(
67+
pluginMethod.getTargetMethod().getProject(),
68+
returnTypeCandidate
69+
)
6770
);
68-
69-
if (returnType != null) {
70-
resolver.processElement(returnType);
71-
}
7271
}
7372

7473
textBuf.append('\n');

0 commit comments

Comments
 (0)