Skip to content

Commit cb0a2e5

Browse files
Simplify
1 parent cb69e3e commit cb0a2e5

1 file changed

Lines changed: 1 addition & 23 deletions

File tree

src/Rules/Comparison/ConstantConditionRuleHelper.php

Lines changed: 1 addition & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -57,32 +57,10 @@ private function shouldSkip(Scope $scope, Expr $expr): bool
5757
|| $expr instanceof Expr\StaticCall
5858
) && !$expr->isFirstClassCallable()
5959
) {
60-
$isAlways = $this->impossibleCheckTypeHelper->findSpecifiedType($scope, $expr, true);
60+
$isAlways = $this->impossibleCheckTypeHelper->findSpecifiedType($scope, $expr, false);
6161
if ($isAlways !== null) {
6262
return true;
6363
}
64-
65-
if (!$scope->isInTrait()) {
66-
return false;
67-
}
68-
69-
foreach ($expr->getArgs() as $arg) {
70-
if (ExpressionDependsOnThisHelper::isExpressionDependentOnThis($arg->value)) {
71-
return true;
72-
}
73-
74-
$classReflection = $scope->getClassReflection();
75-
if ($classReflection === null) {
76-
continue;
77-
}
78-
79-
$argType = $this->treatPhpDocTypesAsCertain ? $scope->getType($arg->value) : $scope->getNativeType($arg->value);
80-
foreach ($argType->getObjectClassNames() as $className) {
81-
if ($className === $classReflection->getName()) {
82-
return true;
83-
}
84-
}
85-
}
8664
}
8765

8866
return false;

0 commit comments

Comments
 (0)