style: tweaks constructors

codeigniter4 · Feb 25, 2024 · 6bd5ebf · 6bd5ebf
1 parent 45be3d7
commit 6bd5ebf
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 23 deletions.
diff --git a/system/Commands/Utilities/Routes/AutoRouterImproved/AutoRouteCollector.php b/system/Commands/Utilities/Routes/AutoRouterImproved/AutoRouteCollector.php
@@ -24,22 +24,17 @@
 final class AutoRouteCollector
 {
     /**
-     * @param string $namespace namespace to search
+     * @param string         $namespace            namespace to search
+     * @param class-string[] $protectedControllers List of controllers in Defined
+     *                                             Routes that should not be accessed via Auto-Routing.
+     * @param string         $prefix               URI prefix for Module Routing
      */
     public function __construct(
         private readonly string $namespace,
         private readonly string $defaultController,
         private readonly string $defaultMethod,
         private readonly array $httpMethods,
-        /**
-         * List of controllers in Defined Routes that should not be accessed via Auto-Routing.
-         *
-         * @var class-string[]
-         */
         private readonly array $protectedControllers,
-        /**
-         * @var string URI prefix for Module Routing
-         */
         private string $prefix = ''
     ) {
     }

diff --git a/system/Commands/Utilities/Routes/AutoRouterImproved/ControllerMethodReader.php b/system/Commands/Utilities/Routes/AutoRouterImproved/ControllerMethodReader.php
@@ -28,13 +28,13 @@ final class ControllerMethodReader
     private readonly bool $translateUriToCamelCase;
 
     /**
-     * @param string $namespace the default namespace
+     * @param string       $namespace   the default namespace
+     * @param list<string> $httpMethods
      */
-    public function __construct(private readonly string $namespace, /**
-     * @var list<string>
-     */
-        private readonly array $httpMethods)
-    {
+    public function __construct(
+        private readonly string $namespace,
+        private readonly array $httpMethods
+    ) {
         $config                        = config(Routing::class);
         $this->translateURIDashes      = $config->translateURIDashes;
         $this->translateUriToCamelCase = $config->translateUriToCamelCase;

diff --git a/system/Commands/Utilities/Routes/ControllerFinder.php b/system/Commands/Utilities/Routes/ControllerFinder.php
@@ -28,8 +28,9 @@ final class ControllerFinder
     /**
      * @param string $namespace namespace to search
      */
-    public function __construct(private readonly string $namespace)
-    {
+    public function __construct(
+        private readonly string $namespace
+    ) {
         $this->locator = Services::locator();
     }
 

diff --git a/system/DataCaster/DataCaster.php b/system/DataCaster/DataCaster.php
@@ -61,17 +61,13 @@ final class DataCaster
     /**
      * @param array<string, class-string>|null $castHandlers Custom convert handlers
      * @param array<string, string>|null       $types        [field => type]
+     * @param object|null                      $helper       Helper object.
+     * @param bool                             $strict       Strict mode? Set to false for casts for Entity.
      */
     public function __construct(
         ?array $castHandlers = null,
         ?array $types = null,
-        /**
-         * Helper object.
-         */
         private readonly ?object $helper = null,
-        /**
-         * Strict mode? Set to false for casts for Entity.
-         */
         private readonly bool $strict = true
     ) {
         $this->castHandlers = array_merge($this->castHandlers, $castHandlers);