patch

.php-cs-fixer.php

@@ -0,0 +1,11 @@
+<?php
+
+$finder = PhpCsFixer\Finder::create()
+    ->exclude('vendor')
+    ->in(__DIR__);
+
+$config = new PhpCsFixer\Config();
+return $config->setRules([
+        '@PSR12' => true,
+    ])
+    ->setFinder($finder);

composer.json

@@ -10,7 +10,10 @@
     },
     "require-dev": {
         "phpunit/phpunit": "^10.0",
-        "orchestra/testbench": "^8.6"
+        "orchestra/testbench": "^8.6",
+        "phpstan/phpstan": "^1.10",
+        "friendsofphp/php-cs-fixer": "^3.26",
+        "squizlabs/php_codesniffer": "^3.7"
     },
     "autoload": {
         "psr-4": {"AnourValar\\LaravelAtom\\": "src/"}

phpcs.xml

@@ -0,0 +1,36 @@
+<?xml version="1.0"?>
+<!-- @see https://pear.php.net/manual/en/package.php.php-codesniffer.annotated-ruleset.php -->
+<ruleset name="PHPCS Rules">
+
+    <description>PHPCS ruleset</description>
+
+    <file>src</file>
+    <file>tests</file>
+
+    <!-- Show progress of the run -->
+    <arg value= "p"/>
+
+    <!-- Show sniff codes in all reports -->
+    <arg value= "s"/>
+
+    <!-- Our base rule: set to PSR12 -->
+    <rule ref="PSR12">
+        <exclude name="PSR12.Operators.OperatorSpacing.NoSpaceBefore"/>
+        <exclude name="PSR12.Operators.OperatorSpacing.NoSpaceAfter"/>
+        <exclude name="PSR12.Traits.UseDeclaration.MultipleImport"/>
+        <exclude name="Generic.Files.LineLength.TooLong"/>
+        <exclude name="PSR2.Methods.FunctionClosingBrace.SpacingBeforeClose"/>
+        <exclude name="PSR12.Classes.OpeningBraceSpace.Found"/>
+        <exclude name="Squiz.WhiteSpace.ControlStructureSpacing.SpacingAfterOpen"/>
+        <exclude name="Squiz.WhiteSpace.ControlStructureSpacing.SpacingBeforeClose"/>
+    </rule>
+
+    <rule ref="PSR1.Methods.CamelCapsMethodName.NotCamelCaps">
+        <exclude-pattern>tests/</exclude-pattern>
+    </rule>
+
+    <rule ref="Internal.NoCodeFound">
+        <exclude-pattern>tests/</exclude-pattern>
+    </rule>
+
+</ruleset>

phpstan.neon

@@ -0,0 +1,32 @@
+parameters:
+
+    paths:
+        - src
+        - tests
+
+    # The level 9 is the highest level
+    level: 5
+
+    ignoreErrors:
+        - '#Call to static method#'
+        - '#Unsafe usage of new static\(\)#'
+        - '#Method AnourValar\\LaravelAtom\\Helpers\\NumberHelper\:\:encodeMultiple\(\) should return int\|null but returns float#'
+        - '#Result of method AnourValar\\LaravelAtom\\Service\:\:lock\(\) \(void\) is used\.#'
+        - '#Parameter \#1 \$a#'
+        - '#Parameter \#4 \$d#'
+        - '#Call to an undefined method#'
+        - '#If condition is always true#'
+        - '#Call to an undefined static#'
+
+    excludePaths:
+
+    checkFunctionNameCase: true
+    checkInternalClassCaseSensitivity: true
+    checkAlwaysTrueInstanceof: true
+    reportMaybesInMethodSignatures: true
+    reportStaticMethodSignatures: true
+    checkUninitializedProperties: true
+    checkDynamicProperties: true
+    reportAlwaysTrueInLastCondition: true
+    reportWrongPhpDocTypeInVarTag: true
+    checkMissingCallableSignature: true

src/ArrayAccessMapper.php

@@ -125,6 +125,6 @@ public function offsetUnset($offset): void
      */
     public function offsetGet($offset): mixed
     {
-        return ( $this->data[$offset] ?? ($this->instanceInsteadOfNull ? (new static([], true)) : null) );
+        return ($this->data[$offset] ?? ($this->instanceInsteadOfNull ? (new static([], true)) : null));
     }
 }

src/Exceptions/OptimisticTransactionException.php

@@ -4,5 +4,4 @@
 
 class OptimisticTransactionException extends \Exception
 {
-
 }

src/Helpers/DateHelper.php

@@ -13,7 +13,7 @@ class DateHelper
      * @param string|null $default
      * @return string|null
      */
-    function formatDate($date, mixed $time = true, string $timezoneClient = 'UTC', ?string $default = null): ?string
+    public function formatDate($date, mixed $time = true, string $timezoneClient = 'UTC', ?string $default = null): ?string
     {
         if (! $date) {
             return $default;
@@ -25,7 +25,7 @@ function formatDate($date, mixed $time = true, string $timezoneClient = 'UTC', ?
 
         if (! $date instanceof \Carbon\CarbonInterface) {
             //try {
-                $date = \Date::parse($date);
+            $date = \Date::parse($date);
             //} catch (\Carbon\Exceptions\InvalidFormatException $e) {
             //    return $default;
             //}
@@ -54,15 +54,15 @@ function formatDate($date, mixed $time = true, string $timezoneClient = 'UTC', ?
      * @param string|null $default
      * @return string|null
      */
-    function formatDateTime($date, string $timezoneClient = 'UTC', ?string $default = null): ?string
+    public function formatDateTime($date, string $timezoneClient = 'UTC', ?string $default = null): ?string
     {
         if (! $date) {
             return $default;
         }
 
         if (! $date instanceof \Carbon\CarbonInterface) {
             //try {
-                $date = \Date::parse($date);
+            $date = \Date::parse($date);
             //} catch (\Carbon\Exceptions\InvalidFormatException $e) {
             //    return $default;
             //}
@@ -79,15 +79,15 @@ function formatDateTime($date, string $timezoneClient = 'UTC', ?string $default
      * @param string|null $default
      * @return string|null
      */
-    function formatTime($date, string $timezoneClient = 'UTC', ?string $default = null): ?string
+    public function formatTime($date, string $timezoneClient = 'UTC', ?string $default = null): ?string
     {
         if (! $date) {
             return $default;
         }
 
         if (! $date instanceof \Carbon\CarbonInterface) {
             //try {
-                $date = \Date::parse($date);
+            $date = \Date::parse($date);
             //} catch (\Carbon\Exceptions\InvalidFormatException $e) {
             //    return $default;
             //}

src/Helpers/NumberHelper.php

@@ -61,7 +61,7 @@ public function formatNumber(?float $number, int $decimals = 0, ?string $default
      * Amount as text
      *
      * @param int|float $amount
-     * @param bool $ucFirst
+     * @param bool $ucfirst
      * @return string
      */
     public function spellout(int|float $amount, bool $ucfirst = true): string

src/Helpers/StringHelper.php

@@ -50,7 +50,7 @@ public function formatPhone(?string $phone): ?string
      */
     public function canonizeEmail(?string $email): string
     {
-        return trim( mb_strtolower($email) );
+        return trim(mb_strtolower($email));
     }
 
     /**

src/Http/Middleware/Web/Select2.php

@@ -45,7 +45,8 @@ private function convertResponse(\Symfony\Component\HttpFoundation\Response &$re
     {
         $content = $response->getOriginalContent();
 
-        if (! $content instanceof \Illuminate\Pagination\LengthAwarePaginator
+        if (
+            ! $content instanceof \Illuminate\Pagination\LengthAwarePaginator
             && ! $content instanceof \Illuminate\Pagination\CursorPaginator
             && ! $content instanceof \Illuminate\Pagination\Paginator
             && $response->isSuccessful()

src/Mapper.php

@@ -92,7 +92,7 @@ public static function from(array|object $data): static
      */
     public function toArray(): array
     {
-        return $this->resolveToArray((array) $this,  static::getRules(new \ReflectionClass(static::class)));
+        return $this->resolveToArray((array) $this, static::getRules(new \ReflectionClass(static::class)));
     }
 
     /**

src/Mapper/Exclude.php

@@ -7,5 +7,4 @@
 #[Attribute(Attribute::TARGET_PROPERTY)]
 class Exclude
 {
-
 }

src/Mapper/MappingSnakeCase.php

@@ -7,5 +7,4 @@
 #[Attribute(Attribute::TARGET_CLASS | Attribute::TARGET_PROPERTY)]
 class MappingSnakeCase
 {
-
 }

src/Mapper/Optional.php

@@ -4,5 +4,4 @@
 
 class Optional
 {
-
 }

src/Service.php

@@ -108,7 +108,7 @@ public function lock(): void
         $table = $this->config['locks']['table'];
 
         $class = $this->config['locks']['strategies'][$this->config['locks']['strategy']];
-        (new $class)->lock($sha1, $connection, $table);
+        (new $class())->lock($sha1, $connection, $table);
 
         if ($this->lockHook) {
             ($this->lockHook)($sha1, func_get_args());
@@ -289,7 +289,7 @@ protected function cleanUp(\Illuminate\Database\Connection $connection, string $
      */
     protected function shouldCommit($connection)
     {
-         return (! (\DB::connection($connection)->transactionLevel() - $this->transactionZeroLevel));
+        return (! (\DB::connection($connection)->transactionLevel() - $this->transactionZeroLevel));
     }
 
     /**
@@ -298,6 +298,6 @@ protected function shouldCommit($connection)
      */
     protected function shouldRollBack($connection)
     {
-         return (! (\DB::connection($connection)->transactionLevel() - $this->transactionZeroLevel));
+        return (! (\DB::connection($connection)->transactionLevel() - $this->transactionZeroLevel));
     }
 }

tests/Mappers/ArrayOfMapper.php

@@ -7,7 +7,7 @@
 
 class ArrayOfMapper extends Mapper
 {
-     public function __construct(
+    public function __construct(
         #[ArrayOf(ExcludeMapper::class)]
         public array $excludes,
     ) {

tests/Mappers/ComplexMapper.php

@@ -8,7 +8,7 @@
 
 class ComplexMapper extends Mapper
 {
-     public function __construct(
+    public function __construct(
         public int $userId,
         #[Mapping('boss_id')]
         public int $managerId,

tests/Mappers/ExcludeMapper.php

@@ -7,7 +7,7 @@
 
 class ExcludeMapper extends Mapper
 {
-     public function __construct(
+    public function __construct(
         #[Exclude]
         public string $a,
         public string $b,

tests/Mappers/ModeMapper.php

@@ -13,7 +13,7 @@
 #[MappingSnakeCase]
 class ModeMapper extends Mapper
 {
-     public function __construct(
+    public function __construct(
         public int $userId,
         #[DefaultValue(3)]
         public int $managerId,

tests/Mappers/NestedMapper.php

@@ -9,7 +9,7 @@
 
 class NestedMapper extends Mapper
 {
-     public function __construct(
+    public function __construct(
         public string|Optional $a,
     ) {
 

tests/Mappers/SimpleMapper.php

@@ -8,7 +8,7 @@
 
 class SimpleMapper extends Mapper
 {
-     public function __construct(
+    public function __construct(
         public string $a,
         public $b,
         public ?string $c = null,