Violation can now contain a reference to the file path (#478)

* violation can now contain a reference to the file path
* updates all rules

Author: micheleorselli

src/Analyzer/FileParser.php

@@ -19,8 +19,8 @@ class FileParser implements Parser
 
     private FileVisitor $fileVisitor;
 
-    /** @var array */
-    private $parsingErrors;
+    /** @var ParsingError[] */
+    private array $parsingErrors;
 
     public function __construct(
         NodeTraverser $traverser,

src/Analyzer/FileVisitor.php

@@ -13,15 +13,15 @@ class FileVisitor extends NodeVisitorAbstract
 {
     private ClassDescriptionBuilder $classDescriptionBuilder;
 
-    /** @var array<ClassDescription> */
+    /** @var ClassDescription[] */
     private array $classDescriptions = [];
 
     public function __construct(ClassDescriptionBuilder $classDescriptionBuilder)
     {
         $this->classDescriptionBuilder = $classDescriptionBuilder;
     }
 
-    public function setFilePath(string $filePath): void
+    public function setFilePath(?string $filePath): void
     {
         $this->classDescriptionBuilder->setFilePath($filePath);
     }

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);
         }

src/Expression/ForClasses/DependsOnlyOnTheseNamespaces.php

@@ -15,7 +15,7 @@
 class DependsOnlyOnTheseNamespaces implements Expression
 {
     /** @var string[] */
-    private $namespaces;
+    private array $namespaces;
 
     public function __construct(string ...$namespace)
     {
@@ -49,7 +49,8 @@ public function evaluate(ClassDescription $theClass, Violations $violations, str
                         $this->describe($theClass, $because),
                         "depends on {$dependency->getFQCN()->toString()}"
                     ),
-                    $dependency->getLine()
+                    $dependency->getLine(),
+                    $theClass->getFilePath()
                 );
 
                 $violations->add($violation);

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);

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()
             )
         );
     }

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);
         }

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);
         }

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);

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);