updates all rules

micheleorselli · micheleorselli · commit fb82d012974e · 2025-04-03T23:08:40.000+02:00
diff --git a/src/Expression/ForClasses/ContainDocBlockLike.php b/src/Expression/ForClasses/ContainDocBlockLike.php
@@ -31,7 +31,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
         if (!$theClass->containsDocBlock($this->docBlock)) {
             $violation = Violation::create(
                 $theClass->getFQCN(),
-                ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+                ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+                $theClass->getFilePath()
             );
             $violations->add($violation);
         }
diff --git a/src/Expression/ForClasses/Extend.php b/src/Expression/ForClasses/Extend.php
@@ -43,7 +43,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
 
         $violation = Violation::create(
             $theClass->getFQCN(),
-            ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+            ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+            $theClass->getFilePath()
         );
 
         $violations->add($violation);
diff --git a/src/Expression/ForClasses/HaveAttribute.php b/src/Expression/ForClasses/HaveAttribute.php
@@ -34,7 +34,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
         $violations->add(
             Violation::create(
                 $theClass->getFQCN(),
-                ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+                ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+                $theClass->getFilePath()
             )
         );
     }
diff --git a/src/Expression/ForClasses/HaveNameMatching.php b/src/Expression/ForClasses/HaveNameMatching.php
@@ -33,7 +33,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
         if (!$fqcn->classMatches($this->name)) {
             $violation = Violation::create(
                 $theClass->getFQCN(),
-                ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+                ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+                $theClass->getFilePath()
             );
             $violations->add($violation);
         }
diff --git a/src/Expression/ForClasses/Implement.php b/src/Expression/ForClasses/Implement.php
@@ -47,7 +47,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
         if (0 === \count(array_filter($interfaces, $implements))) {
             $violation = Violation::create(
                 $theClass->getFQCN(),
-                ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+                ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+                $theClass->getFilePath()
             );
             $violations->add($violation);
         }
diff --git a/src/Expression/ForClasses/IsAbstract.php b/src/Expression/ForClasses/IsAbstract.php
@@ -31,7 +31,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
 
         $violation = Violation::create(
             $theClass->getFQCN(),
-            ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+            ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+            $theClass->getFilePath()
         );
 
         $violations->add($violation);
diff --git a/src/Expression/ForClasses/IsEnum.php b/src/Expression/ForClasses/IsEnum.php
@@ -26,7 +26,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
 
         $violation = Violation::create(
             $theClass->getFQCN(),
-            ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+            ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+            $theClass->getFilePath()
         );
 
         $violations->add($violation);
diff --git a/src/Expression/ForClasses/IsFinal.php b/src/Expression/ForClasses/IsFinal.php
@@ -31,7 +31,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
 
         $violation = Violation::create(
             $theClass->getFQCN(),
-            ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+            ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+            $theClass->getFilePath()
         );
 
         $violations->add($violation);
diff --git a/src/Expression/ForClasses/IsInterface.php b/src/Expression/ForClasses/IsInterface.php
@@ -26,7 +26,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
 
         $violation = Violation::create(
             $theClass->getFQCN(),
-            ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+            ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+            $theClass->getFilePath()
         );
 
         $violations->add($violation);
diff --git a/src/Expression/ForClasses/IsNotAbstract.php b/src/Expression/ForClasses/IsNotAbstract.php
@@ -31,7 +31,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
 
         $violation = Violation::create(
             $theClass->getFQCN(),
-            ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+            ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+            $theClass->getFilePath()
         );
 
         $violations->add($violation);
diff --git a/src/Expression/ForClasses/IsNotEnum.php b/src/Expression/ForClasses/IsNotEnum.php
@@ -26,7 +26,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
 
         $violation = Violation::create(
             $theClass->getFQCN(),
-            ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+            ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+            $theClass->getFilePath()
         );
 
         $violations->add($violation);
diff --git a/src/Expression/ForClasses/IsNotFinal.php b/src/Expression/ForClasses/IsNotFinal.php
@@ -31,7 +31,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
 
         $violation = Violation::create(
             $theClass->getFQCN(),
-            ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+            ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+            $theClass->getFilePath()
         );
 
         $violations->add($violation);
diff --git a/src/Expression/ForClasses/IsNotInterface.php b/src/Expression/ForClasses/IsNotInterface.php
@@ -26,7 +26,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
 
         $violation = Violation::create(
             $theClass->getFQCN(),
-            ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+            ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+            $theClass->getFilePath()
         );
 
         $violations->add($violation);
diff --git a/src/Expression/ForClasses/IsNotReadonly.php b/src/Expression/ForClasses/IsNotReadonly.php
@@ -31,7 +31,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
 
         $violation = Violation::create(
             $theClass->getFQCN(),
-            ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+            ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+            $theClass->getFilePath()
         );
 
         $violations->add($violation);
diff --git a/src/Expression/ForClasses/IsNotTrait.php b/src/Expression/ForClasses/IsNotTrait.php
@@ -26,7 +26,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
 
         $violation = Violation::create(
             $theClass->getFQCN(),
-            ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+            ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+            $theClass->getFilePath()
         );
 
         $violations->add($violation);
diff --git a/src/Expression/ForClasses/IsReadonly.php b/src/Expression/ForClasses/IsReadonly.php
@@ -31,7 +31,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
 
         $violation = Violation::create(
             $theClass->getFQCN(),
-            ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+            ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+            $theClass->getFilePath()
         );
 
         $violations->add($violation);
diff --git a/src/Expression/ForClasses/IsTrait.php b/src/Expression/ForClasses/IsTrait.php
@@ -26,7 +26,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
 
         $violation = Violation::create(
             $theClass->getFQCN(),
-            ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+            ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+            $theClass->getFilePath()
         );
 
         $violations->add($violation);
diff --git a/src/Expression/ForClasses/MatchOneOfTheseNames.php b/src/Expression/ForClasses/MatchOneOfTheseNames.php
@@ -40,7 +40,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
         if (!$matches) {
             $violation = Violation::create(
                 $theClass->getFQCN(),
-                ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+                ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+                $theClass->getFilePath()
             );
             $violations->add($violation);
         }
diff --git a/src/Expression/ForClasses/NotContainDocBlockLike.php b/src/Expression/ForClasses/NotContainDocBlockLike.php
@@ -31,7 +31,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
         if ($theClass->containsDocBlock($this->docBlock)) {
             $violation = Violation::create(
                 $theClass->getFQCN(),
-                ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+                ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+                $theClass->getFilePath()
             );
             $violations->add($violation);
         }
diff --git a/src/Expression/ForClasses/NotExtend.php b/src/Expression/ForClasses/NotExtend.php
@@ -38,7 +38,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
                 if ($extend->matches($className)) {
                     $violation = Violation::create(
                         $theClass->getFQCN(),
-                        ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+                        ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+                        $theClass->getFilePath()
                     );
 
                     $violations->add($violation);
diff --git a/src/Expression/ForClasses/NotHaveNameMatching.php b/src/Expression/ForClasses/NotHaveNameMatching.php
@@ -33,7 +33,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
         if ($fqcn->classMatches($this->name)) {
             $violation = Violation::create(
                 $theClass->getFQCN(),
-                ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+                ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+                $theClass->getFilePath()
             );
             $violations->add($violation);
         }
diff --git a/src/Expression/ForClasses/NotImplement.php b/src/Expression/ForClasses/NotImplement.php
@@ -47,7 +47,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
         if (\count(array_filter($interfaces, $implements)) > 0) {
             $violation = Violation::create(
                 $theClass->getFQCN(),
-                ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+                ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+                $theClass->getFilePath()
             );
             $violations->add($violation);
         }
diff --git a/src/Expression/ForClasses/NotResideInTheseNamespaces.php b/src/Expression/ForClasses/NotResideInTheseNamespaces.php
@@ -40,7 +40,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
         if ($resideInNamespace) {
             $violation = Violation::create(
                 $theClass->getFQCN(),
-                ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+                ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+                $theClass->getFilePath()
             );
             $violations->add($violation);
         }
diff --git a/src/Expression/ForClasses/ResideInOneOfTheseNamespaces.php b/src/Expression/ForClasses/ResideInOneOfTheseNamespaces.php
@@ -40,7 +40,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
         if (!$resideInNamespace) {
             $violation = Violation::create(
                 $theClass->getFQCN(),
-                ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+                ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+                $theClass->getFilePath()
             );
             $violations->add($violation);
         }
diff --git a/src/Rules/Violation.php b/src/Rules/Violation.php
@@ -22,9 +22,9 @@ public function __construct(string $fqcn, string $error, ?int $line = null, ?str
         $this->filePath = $filePath;
     }
 
-    public static function create(string $fqcn, ViolationMessage $error): self
+    public static function create(string $fqcn, ViolationMessage $error, string $filePath): self
     {
-        return new self($fqcn, $error->toString());
+        return new self($fqcn, $error->toString(), null, $filePath);
     }
 
     public static function createWithErrorLine(string $fqcn, ViolationMessage $error, int $line, string $filePath): self
diff --git a/tests/Unit/Rules/ConstraintsTest.php b/tests/Unit/Rules/ConstraintsTest.php
@@ -59,7 +59,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
             {
                 $violation = Violation::create(
                     $theClass->getFQCN(),
-                    ViolationMessage::selfExplanatory($this->describe($theClass, $because))
+                    ViolationMessage::selfExplanatory($this->describe($theClass, $because)),
+                    $theClass->getFilePath()
                 );
 
                 $violations->add($violation);

Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str`
`34`	`34`	`$violations->add(`
`35`	`35`	`Violation::create(`
`36`	`36`	`$theClass->getFQCN(),`
`37`		`- ViolationMessage::selfExplanatory($this->describe($theClass, $because))`
	`37`	`+ ViolationMessage::selfExplanatory($this->describe($theClass, $because)),`
	`38`	`+ $theClass->getFilePath()`
`38`	`39`	`)`
`39`	`40`	`);`
`40`	`41`	`}`