refactor: run rector

codeigniter4 · Nov 22, 2023 · 68d8a81 · 68d8a81
1 parent 75c9d0a
commit 68d8a81
Show file tree

Hide file tree

Showing 7 changed files with 17 additions and 17 deletions.
diff --git a/system/CLI/InputOutput.php b/system/CLI/InputOutput.php
@@ -21,7 +21,7 @@ class InputOutput
     /**
      * Is the readline library on the system?
      */
-    private bool $readlineSupport;
+    private readonly bool $readlineSupport;
 
     public function __construct()
     {

diff --git a/system/Commands/Utilities/Routes/SampleURIGenerator.php b/system/Commands/Utilities/Routes/SampleURIGenerator.php
@@ -52,7 +52,7 @@ public function get(string $routeKey): string
     {
         $sampleUri = $routeKey;
 
-        if (strpos($routeKey, '{locale}') !== false) {
+        if (str_contains($routeKey, '{locale}')) {
             $sampleUri = str_replace(
                 '{locale}',
                 config(App::class)->defaultLocale,

diff --git a/system/Filters/Filters.php b/system/Filters/Filters.php
@@ -230,7 +230,7 @@ private function runAfter(array $filterClasses): ResponseInterface
             $class = new $className();
 
             if (! $class instanceof FilterInterface) {
-                throw FilterException::forIncorrectInterface(get_class($class));
+                throw FilterException::forIncorrectInterface($class::class);
             }
 
             $result = $class->after(

diff --git a/system/Filters/PageCache.php b/system/Filters/PageCache.php
@@ -25,7 +25,7 @@
  */
 class PageCache implements FilterInterface
 {
-    private ResponseCache $pageCache;
+    private readonly ResponseCache $pageCache;
 
     public function __construct()
     {

diff --git a/system/HTTP/Method.php b/system/HTTP/Method.php
@@ -19,59 +19,59 @@ class Method
     /**
      * @see https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/CONNECT
      */
-    public const CONNECT = 'CONNECT';
+    final public const CONNECT = 'CONNECT';
 
     /**
      * Idempotent
      *
      * @see https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/DELETE
      */
-    public const DELETE = 'DELETE';
+    final public const DELETE = 'DELETE';
 
     /**
      * Safe, Idempotent, Cacheable
      *
      * @see https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/GET
      */
-    public const GET = 'GET';
+    final public const GET = 'GET';
 
     /**
      * Safe, Idempotent, Cacheable
      *
      * @see https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/HEAD
      */
-    public const HEAD = 'HEAD';
+    final public const HEAD = 'HEAD';
 
     /**
      * Safe, Idempotent
      *
      * @see https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/OPTIONS
      */
-    public const OPTIONS = 'OPTIONS';
+    final public const OPTIONS = 'OPTIONS';
 
     /**
      * @see https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/PATCH
      */
-    public const PATCH = 'PATCH';
+    final public const PATCH = 'PATCH';
 
     /**
      * @see https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/POST
      */
-    public const POST = 'POST';
+    final public const POST = 'POST';
 
     /**
      * Idempotent
      *
      * @see https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/PUT
      */
-    public const PUT = 'PUT';
+    final public const PUT = 'PUT';
 
     /**
      * Safe, Idempotent
      *
      * @see https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/TRACE
      */
-    public const TRACE = 'TRACE';
+    final public const TRACE = 'TRACE';
 
     /**
      * Returns all HTTP methods.

diff --git a/system/Validation/Rules.php b/system/Validation/Rules.php
@@ -444,7 +444,7 @@ public function field_exists(
         ?string $error = null,
         ?string $field = null
     ): bool {
-        if (strpos($field, '.') !== false) {
+        if (str_contains($field, '.')) {
             return ArrayHelper::dotKeyExists($field, $data);
         }
 

diff --git a/system/Validation/StrictRules/Rules.php b/system/Validation/StrictRules/Rules.php
@@ -44,7 +44,7 @@ public function differs(
         ?string $error = null,
         ?string $field = null
     ): bool {
-        if (strpos($otherField, '.') !== false) {
+        if (str_contains($otherField, '.')) {
             return $str !== dot_array_search($otherField, $data);
         }
 
@@ -275,7 +275,7 @@ public function matches(
         ?string $error = null,
         ?string $field = null
     ): bool {
-        if (strpos($otherField, '.') !== false) {
+        if (str_contains($otherField, '.')) {
             return $str === dot_array_search($otherField, $data);
         }
 
@@ -420,7 +420,7 @@ public function field_exists(
         ?string $error = null,
         ?string $field = null
     ): bool {
-        if (strpos($field, '.') !== false) {
+        if (str_contains($field, '.')) {
             return ArrayHelper::dotKeyExists($field, $data);
         }