Skip to content

Commit 5ed1ae3

Browse files
authored
[automated] Apply Coding Standard (#7908)
Co-authored-by: TomasVotruba <924196+TomasVotruba@users.noreply.github.com>
1 parent 43d1236 commit 5ed1ae3

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

rules/DeadCode/Rector/ClassMethod/RemoveParentDelegatingConstructorRector.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,11 @@ private function areConstructorAndParentParameterTypesMatching(
232232

233233
// no type override
234234
if ($parameterType === null) {
235-
if ($param->default instanceof Expr && $this->isDifferentDefaultValue($param->default, $extendedMethodReflection, $position)) {
235+
if ($param->default instanceof Expr && $this->isDifferentDefaultValue(
236+
$param->default,
237+
$extendedMethodReflection,
238+
$position
239+
)) {
236240
return false;
237241
}
238242

rules/Php83/Rector/ClassMethod/AddOverrideAttributeToOverriddenMethodsRector.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -246,7 +246,9 @@ private function shouldSkipClassMethod(ClassMethod $classMethod): bool
246246
}
247247

248248
// skip test setup method override, as rather clutters the code than helps
249-
return $this->isNames($classMethod, ['setUp', 'tearDown']) && $this->parentClassAnalyzer->hasParentCall($classMethod);
249+
return $this->isNames($classMethod, ['setUp', 'tearDown']) && $this->parentClassAnalyzer->hasParentCall(
250+
$classMethod
251+
);
250252
}
251253

252254
private function shouldSkipParentClassMethod(ClassReflection $parentClassReflection, ClassMethod $classMethod): bool

0 commit comments

Comments
 (0)