@@ -50,34 +50,18 @@ public function processNode(Node $node, Scope $scope): array
5050 }
5151
5252 if ($ this ->callsParentConstruct ($ node )) {
53- if ($ classReflection ->getParentClass () === false ) {
54- return [
55- RuleErrorBuilder::message (sprintf (
56- '%s::__construct() calls parent constructor but does not extend any class. ' ,
57- $ classReflection ->getName ()
58- ))->build (),
59- ];
60- }
53+ return [];
54+ }
6155
62- if ($ this ->getParentConstructorClass ($ classReflection ) === false ) {
63- return [
64- RuleErrorBuilder::message (sprintf (
65- '%s::__construct() calls parent constructor but parent does not have one. ' ,
66- $ classReflection ->getName ()
67- ))->build (),
68- ];
69- }
70- } else {
71- $ parentClass = $ this ->getParentConstructorClass ($ classReflection );
72- if ($ parentClass !== false ) {
73- return [
74- RuleErrorBuilder::message (sprintf (
75- '%s::__construct() does not call parent constructor from %s. ' ,
76- $ classReflection ->getName (),
77- $ parentClass ->getName ()
78- ))->build (),
79- ];
80- }
56+ $ parentClass = $ this ->getParentConstructorClass ($ classReflection );
57+ if ($ parentClass !== false ) {
58+ return [
59+ RuleErrorBuilder::message (sprintf (
60+ '%s::__construct() does not call parent constructor from %s. ' ,
61+ $ classReflection ->getName (),
62+ $ parentClass ->getName ()
63+ ))->build (),
64+ ];
8165 }
8266
8367 return [];
0 commit comments