Merge pull request #31 from exussum12/typeErrors

Fix some type errors

Author: exussum12

src/ArgParser.php

@@ -55,7 +55,7 @@ protected function splitArg($arg)
     }
 
     /**
-     * @param $name
+     * @param string $name
      * @return string
      */
     protected function getAdjustedArg($name)

src/CheckstyleLoader.php

@@ -76,8 +76,8 @@ public static function getDescription()
     }
 
     /**
-     * @param $reader
-     * @param $currentFile
+     * @param XMLReader $reader
+     * @param string $currentFile
      */
     protected function handleErrors($reader, $currentFile)
     {
@@ -90,8 +90,8 @@ protected function handleErrors($reader, $currentFile)
     }
 
     /**
-     * @param $reader
-     * @param $currentFile
+     * @param XMLReader $reader
+     * @param string $currentFile
      * @return string
      */
     protected function handleFile($reader, $currentFile)

src/CloverLoader.php

@@ -89,8 +89,8 @@ protected function checkForNewFiles($reader, $currentFile)
     }
 
     /**
-     * @param $reader
-     * @param $currentFile
+     * @param XMLReader $reader
+     * @param string $currentFile
      */
     protected function addLine($reader, $currentFile)
     {
@@ -102,6 +102,10 @@ protected function addLine($reader, $currentFile)
             = $covered ?: "No test coverage";
     }
 
+    /**
+     * @param XMLReader $reader
+     * @param string $currentFile
+     */
     protected function handleStatement($reader, $currentFile)
     {
         if ((

src/CodeClimateLoader.php

@@ -9,7 +9,7 @@
 class CodeClimateLoader implements FileChecker
 {
     /**
-     * @var string
+     * @var array
      */
     protected $file;
 

src/DiffLineHandle.php

@@ -5,7 +5,7 @@ abstract class DiffLineHandle
 {
     protected $diffFileState;
 
-    public function __construct(diffFileState $diff)
+    public function __construct(DiffFileState $diff)
     {
         $this->diffFileState = $diff;
     }

src/FileChecker.php

@@ -16,8 +16,8 @@ public function parseLines();
      * Method to determine if the line is valid in the context
      * returning null does not include the line in the stats
      * Returns an array containing errors on a certain line - empty array means no errors
-     * @param $file
-     * @param $lineNumber
+     * @param string $file
+     * @param int $lineNumber
      * @return array|null
      */
     public function getErrorsOnLine($file, $lineNumber);

src/FileMatchers/EndsWith.php

@@ -27,8 +27,8 @@ public function match($needle, array $haystack)
 
     /**
      * Find if two strings end in the same way
-     * @param $haystack
-     * @param $needle
+     * @param string $haystack
+     * @param string $needle
      * @return bool
      */
     protected function fileEndsWith($haystack, $needle)

src/JacocoReport.php

@@ -38,8 +38,8 @@ public static function getDescription()
     }
 
     /**
-     * @param $reader
-     * @param $currentFile
+     * @param XMLReader $reader
+     * @param string $currentFile
      */
     protected function addLine($reader, $currentFile)
     {

src/PhpCsLoader.php

@@ -12,7 +12,7 @@
 class PhpCsLoader implements FileChecker
 {
     /**
-     * @var string
+     * @var stdClass
      */
     protected $json;
     /**

src/PhpStanLoader.php

@@ -2,6 +2,7 @@
 namespace exussum12\CoverageChecker;
 
 use ReflectionFunction;
+use ReflectionFunctionAbstract;
 use ReflectionMethod;
 use Reflector;
 
@@ -38,7 +39,7 @@ public function parseLines()
         $filename = '';
         $lineNumber = 0;
         while (($line = fgets($this->file)) !== false) {
-            $filename = $this->checkForFileName($line, $filename);
+            $filename = $this->checkForFilename($line, $filename);
             if ($lineNumber = $this->getLineNumber($line, $lineNumber)) {
                 $error = $this->getMessage($line);
                 if ($this->isExtendedMessage($line)) {
@@ -154,9 +155,9 @@ protected function handleRelatedError($filename, $line, $error)
     }
 
     /**
-     * @param $filename
-     * @param $lineNumber
-     * @param $error
+     * @param string $filename
+     * @param int $lineNumber
+     * @param string $error
      */
     protected function addError($filename, $lineNumber, $error)
     {
@@ -167,8 +168,8 @@ protected function addError($filename, $lineNumber, $error)
     }
 
     /**
-     * @param $matches
-     * @return Reflector
+     * @param array $matches
+     * @return ReflectionFunctionAbstract
      */
     protected function getReflector($matches)
     {