🧹 Fixed code-style

TheDragonCode · Jun 26, 2023 · f0106de · f0106de
1 parent 1cf90df
commit f0106de
Show file tree

Hide file tree

Showing 7 changed files with 5 additions and 52 deletions.
diff --git a/composer.json b/composer.json
@@ -69,7 +69,8 @@
         "allow-plugins": {
             "dragon-code/codestyler": true,
             "ergebnis/composer-normalize": true,
-            "friendsofphp/php-cs-fixer": true
+            "friendsofphp/php-cs-fixer": true,
+            "symfony/thanks": true
         },
         "preferred-install": "dist",
         "sort-packages": true

diff --git a/src/Concerns/From.php b/src/Concerns/From.php
@@ -15,23 +15,15 @@
 trait From
 {
     /**
-     * @param array $array
-     *
      * @throws ReflectionException
-     *
-     * @return \DragonCode\Contracts\DataTransferObject\DataTransferObject
      */
     public static function fromArray(array $array): Contract
     {
         return new static($array);
     }
 
     /**
-     * @param $object
-     *
      * @throws ReflectionException
-     *
-     * @return \DragonCode\Contracts\DataTransferObject\DataTransferObject
      */
     public static function fromObject($object): Contract
     {
@@ -41,11 +33,7 @@ public static function fromObject($object): Contract
     }
 
     /**
-     * @param \Symfony\Component\HttpFoundation\Request $request
-     *
      * @throws ReflectionException
-     *
-     * @return \DragonCode\Contracts\DataTransferObject\DataTransferObject
      */
     public static function fromRequest(Request $request): Contract
     {
@@ -55,11 +43,7 @@ public static function fromRequest(Request $request): Contract
     }
 
     /**
-     * @param string $json
-     *
      * @throws ReflectionException
-     *
-     * @return \DragonCode\Contracts\DataTransferObject\DataTransferObject
      */
     public static function fromJson(string $json): Contract
     {

diff --git a/src/Concerns/Reflection.php b/src/Concerns/Reflection.php
@@ -21,11 +21,9 @@ protected function reflection(): ReflectionClass
     }
 
     /**
-     * @param $object
-     *
      * @throws ReflectionException
      *
-     * @return ReflectionProperty[]
+     * @return array<ReflectionProperty>
      */
     protected function reflectProperties($object): array
     {
@@ -35,11 +33,7 @@ protected function reflectProperties($object): array
     }
 
     /**
-     * @param $object
-     *
      * @throws ReflectionException
-     *
-     * @return array
      */
     protected function getProperties($object): array
     {

diff --git a/src/Converters/Objects.php b/src/Converters/Objects.php
@@ -15,7 +15,6 @@
 class Objects implements Arrayable
 {
     use Makeable;
-
     use Reflection;
 
     protected $object;

diff --git a/src/DataTransferObject.php b/src/DataTransferObject.php
@@ -21,13 +21,9 @@
 abstract class DataTransferObject implements Contract
 {
     use Castable;
-
     use From;
-
     use Makeable;
-
     use Reflection;
-
     use To;
 
     protected const DISALLOW = ['map', 'only', 'except'];
@@ -39,8 +35,6 @@ abstract class DataTransferObject implements Contract
     protected $except = [];
 
     /**
-     * @param array $items
-     *
      * @throws ReflectionException
      */
     public function __construct(array $items = [])
@@ -69,8 +63,6 @@ public function set(string $key, $value): DataTransferObject
     }
 
     /**
-     * @param array $items
-     *
      * @throws ReflectionException
      *
      * @return \DragonCode\SimpleDataTransferObject\DataTransferObject
@@ -87,8 +79,6 @@ public function merge(array $items): DataTransferObject
 
     /**
      * @throws ReflectionException
-     *
-     * @return array
      */
     public function toArray(): array
     {
@@ -121,8 +111,6 @@ protected function filterExcept(array $items): array
     }
 
     /**
-     * @param array $items
-     *
      * @throws ReflectionException
      */
     protected function setMap(array $items): void
@@ -139,8 +127,6 @@ protected function setMap(array $items): void
     }
 
     /**
-     * @param array $items
-     *
      * @throws ReflectionException
      */
     protected function setItems(array $items): void
@@ -160,9 +146,6 @@ protected function getValueByKey(array $items, string $key, string $default)
     }
 
     /**
-     * @param string $key
-     * @param mixed $value
-     *
      * @throws ReflectionException
      */
     protected function setValue(string $key, $value): void
@@ -173,23 +156,15 @@ protected function setValue(string $key, $value): void
     }
 
     /**
-     * @param string $key
-     *
      * @throws ReflectionException
-     *
-     * @return bool
      */
     protected function isAllow(string $key): bool
     {
         return $this->isAllowKey($key) && $this->isAllowProperty($key);
     }
 
     /**
-     * @param string $key
-     *
      * @throws ReflectionException
-     *
-     * @return bool
      */
     protected function isAllowProperty(string $key): bool
     {

diff --git a/tests/Fixtures/Nested/Company.php b/tests/Fixtures/Nested/Company.php
@@ -10,7 +10,7 @@ class Company extends DataTransferObject
 {
     public $title;
 
-    /** @var \Tests\Fixtures\Nested\Project[] */
+    /** @var array<\Tests\Fixtures\Nested\Project> */
     public $projects;
 
     protected function castProjects(array $projects): array

diff --git a/tests/Fixtures/Nested/Project.php b/tests/Fixtures/Nested/Project.php
@@ -12,7 +12,7 @@ class Project extends DataTransferObject
 
     public $domain;
 
-    /** @var \Tests\Fixtures\Nested\Developer[] */
+    /** @var array<\Tests\Fixtures\Nested\Developer> */
     public $developers;
 
     protected function castDevelopers(array $developers): array