CollectionAddAllCanBeReplacedWithConstructorInspection: replace log assertions with conditional if statements (EA-92609)

This commit is contained in:
Dmitry Batkovich
2016-12-14 21:15:43 +03:00
parent d5068314b4
commit c3a779de1b
@@ -247,14 +247,12 @@ public class CollectionAddAllCanBeReplacedWithConstructorInspection extends Base
@Override
public void applyFix(@NotNull Project project, @NotNull ProblemDescriptor descriptor) {
final PsiMethodCallExpression methodCallExpression = myMethodCallExpression.getElement();
LOG.assertTrue(methodCallExpression != null);
if (methodCallExpression == null) return;
final PsiElement parameter = methodCallExpression.getArgumentList().getExpressions()[0].copy();
final PsiNewExpression element = myAssignmentExpression.getElement();
LOG.assertTrue(element != null);
if (element == null) return;
final PsiExpressionList constructorArguments = element.getArgumentList();
LOG.assertTrue(constructorArguments != null);
if (constructorArguments == null) return;
constructorArguments.add(parameter);
methodCallExpression.delete();
}