fix: cs for php8.0

Author: mkorkmaz

src/Arrays/ArrayExtractor.php

@@ -9,11 +9,8 @@
 
 class ArrayExtractor
 {
-    private array $model;
-
-    private function __construct(array $model)
+    private function __construct(private array $model)
     {
-        $this->model = $model;
     }
 
     public static function withModel(array $model): self

src/Arrays/ArrayKeysCamelCaseConverter.php

@@ -20,7 +20,7 @@ public static function convertArrayKeys(array $arrayItems): array
                 $value = self::convertArrayKeys($value);
             }
 
-            if (is_string($key) && strpos($key, '_') !== false) {
+            if (is_string($key) && str_contains($key, '_')) {
                 $key = CaseConverter::toCamelCase($key);
             }
 

src/Arrays/TagAndAttributeRemover.php

@@ -26,18 +26,10 @@
 
 class TagAndAttributeRemover
 {
-    private DOMDocument $domHtml;
-    private array $allowedTags;
-    private array $currentTags;
-    private array $allowedUrlPrefixes;
     private static array $urlAttributes = ['href', 'src'];
 
-    private function __construct(DOMDocument $domHtml, array $currentTags, array $allowedTags, array $allowedUrlPrefixes)
+    private function __construct(private DOMDocument $domHtml, private array $currentTags, private array $allowedTags, private array $allowedUrlPrefixes)
     {
-        $this->domHtml            = $domHtml;
-        $this->currentTags        = $currentTags;
-        $this->allowedTags        = $allowedTags;
-        $this->allowedUrlPrefixes = $allowedUrlPrefixes;
     }
 
     public static function cleanHtml(string $html, string $allowedTagsAndAttributesList, ?string $allowedUrlPrefixes = ''): string
@@ -66,7 +58,7 @@ public static function cleanHtml(string $html, string $allowedTagsAndAttributesL
 
     private static function safeExplodeString(string $delimiter, string $string, ?bool $allowEmptyArrayElement = true): array
     {
-        if (strpos($string, $delimiter) !== false) {
+        if (str_contains($string, $delimiter)) {
             return explode($delimiter, $string);
         }
 
@@ -154,7 +146,7 @@ private function removeAttributesWithNotAllowedUrlPrefixes(DOMElement $node, DOM
     {
         $isAllowed = 0;
         foreach ($this->allowedUrlPrefixes as $allowedUrlPrefix) {
-            if (strpos($attribute->nodeValue, $allowedUrlPrefix) === 0) {
+            if (str_starts_with($attribute->nodeValue, $allowedUrlPrefix)) {
                 $isAllowed = 1;
                 break;
             }

src/EqualsBuilder.php

@@ -14,7 +14,7 @@
 
 final class EqualsBuilder
 {
-    private $isEquals = true;
+    private bool $isEquals = true;
 
     public static function create(): EqualsBuilder
     {
@@ -55,7 +55,7 @@ private function checkIfValuesAreAnObjectAndEqual($leftHandedValue, $rightHanded
             return false;
         }
 
-        if (get_class($leftHandedValue) !== get_class($rightHandedValue)) {
+        if ($leftHandedValue::class !== $rightHandedValue::class) {
             $this->isEquals = false;
 
             return false;

src/Pipeline/Pipeline.php

@@ -7,16 +7,13 @@
 use Psr\Container\ContainerInterface;
 use Selami\Stdlib\Resolver;
 use ReflectionClass;
-use function PHPUnit\Framework\isInstanceOf;
 
 class Pipeline implements PipelineInterface
 {
-    private ?ContainerInterface $container;
     private array $stages = [];
 
-    private function __construct(?ContainerInterface $container = null)
+    private function __construct(private ?\Psr\Container\ContainerInterface $container = null)
     {
-        $this->container = $container;
     }
 
     public static function new(): self

src/Resolver.php

@@ -31,7 +31,7 @@ public static function getParameterHints(string $className, string $methodName):
         self::checkMethodName($className, $methodName);
         try {
             $method = new ReflectionMethod($className, $methodName);
-        } catch (ReflectionException $e) {
+        } catch (ReflectionException) {
             throw new ClassOrMethodCouldNotBeFound(
                 sprintf('%s::%s coulnd not be found.', $className, $methodName)
             );

src/Semver.php

@@ -15,17 +15,9 @@
 final class Semver
 {
     private static string $semverPattern = '/(\d+).(\d+).(\d+)(|[-.+](?:dev|alpha|beta|rc|stable)(.*?))$/i';
-    private int $major;
-    private int $minor;
-    private int $patch;
-    private ?string $preRelease;
 
-    private function __construct(int $major, int $minor, int $patch, ?string $preRelease = null)
+    private function __construct(private int $major, private int $minor, private int $patch, private ?string $preRelease = null)
     {
-        $this->major      = $major;
-        $this->minor      = $minor;
-        $this->patch      = $patch;
-        $this->preRelease = $preRelease;
     }
 
     public static function createFromString(string $version): self

src/Slugifier.php

@@ -9,33 +9,27 @@
 
 class Slugifier
 {
-    /**
-     * @var string|array $subject
-     */
-    private $subject;
-
     private Transliterator $transliterator;
     /**
      * @param string|iterable $subject
      */
-    private function __construct($subject)
+    private function __construct(private $subject)
     {
-        $this->subject = $subject;
         $this->transliterator = Transliterator::create('Any-Latin; Latin-ASCII');
     }
     /**
      * @param mixed<string|iterable> $subject
      * @return string|iterable<string>
      */
-    public static function slugify($subject)
+    public static function slugify(mixed $subject): string|iterable
     {
         return (new self($subject))
             ->getSlugifiedResult();
     }
     /**
      * @return string|iterable<string>
      */
-    private function getSlugifiedResult()
+    private function getSlugifiedResult(): string|iterable
     {
         if (is_iterable($this->subject)) {
             return $this->getSlugifiedIterable($this->subject);
@@ -57,7 +51,6 @@ private function getSlugifiedString($subject) : string
     }
 
     /**
-     * @param iterable $subject
      * @return iterable<string>
      */
     private function getSlugifiedIterable(iterable $subject) : iterable