Use readonly classes when all props are readonly

readonly classes were added in PHP 8.2.

Author: john-shaffer

src/CrawlConfig.php

@@ -2,9 +2,9 @@
 
 namespace StaticDeploy;
 
-final class CrawlConfig {
+final readonly class CrawlConfig {
 
-    public function __construct( public readonly ?string $path_hash_prefix = null )
+    public function __construct( public ?string $path_hash_prefix = null )
     {
     }
 

src/DirectDeployConfig.php

@@ -2,11 +2,11 @@
 
 namespace StaticDeploy;
 
-final class DirectDeployConfig {
+final readonly class DirectDeployConfig {
 
     public function __construct(
-        public readonly ?CrawlConfig $crawl_config = new CrawlConfig(),
-        public readonly bool $do_detect = true,
+        public ?CrawlConfig $crawl_config = new CrawlConfig(),
+        public bool $do_detect = true,
     ) {}
 
     public function toArray(): array

src/OptionData.php

@@ -10,22 +10,22 @@
 
 namespace StaticDeploy;
 
-final class OptionData {
+final readonly class OptionData {
     public const CACHE_GROUP = 'static_deploy_option_data';
 
     // It's very cheap to fetch options, so there is no point
     // caching them for long.
     public const CACHE_TTL_SEC = 300;
 
-    public readonly ?string $blob_value;
+    public ?string $blob_value;
 
-    public readonly OptionSpec $option_spec;
+    public OptionSpec $option_spec;
 
-    public readonly ?string $unfiltered_blob_value;
+    public ?string $unfiltered_blob_value;
 
-    public readonly string $unfiltered_value;
+    public string $unfiltered_value;
 
-    public readonly string $value;
+    public string $value;
 
     public function __construct(
         OptionSpec $option_spec,

src/OptionSpec.php

@@ -9,37 +9,37 @@
 
 namespace StaticDeploy;
 
-final class OptionSpec {
+final readonly class OptionSpec {
 
-    public readonly string $name;
+    public string $name;
 
-    public readonly string $default_value;
+    public string $default_value;
 
-    public readonly ?array $allowed_values;
+    public ?array $allowed_values;
 
-    public readonly string $filter_name;
+    public string $filter_name;
 
-    public readonly string $input_type;
+    public string $input_type;
 
     public function __construct(
-        public readonly string $type,
+        public string $type,
         string $name,
         string $default_value,
-        public readonly string $label,
-        public readonly string $description,
-        public readonly ?string $default_blob_value = null,
+        public string $label,
+        public string $description,
+        public ?string $default_blob_value = null,
         ?string $filter_name = null,
         ?array $allowed_values = null,
         ?string $input_type = null,
-        public readonly ?int $min_value = null,
+        public ?int $min_value = null,
         /**
          * Used to import from WP2Static options
          */
-        public readonly ?string $wp2static_name = null,
+        public ?string $wp2static_name = null,
         /**
          * Used to import from WP2Static options
          */
-        public readonly ?string $wp2static_table = null,
+        public ?string $wp2static_table = null,
     ) {
         if ( $allowed_values !== null && ! in_array( $default_value, $allowed_values, true ) ) {
             $msg = "Default value {$default_value} not in allowed values for option {$name}";

src/PostProcessConfig.php

@@ -2,15 +2,15 @@
 
 namespace StaticDeploy;
 
-final class PostProcessConfig {
+final readonly class PostProcessConfig {
 
-    public readonly ?string $destination_url;
+    public ?string $destination_url;
 
-    public readonly ?array $hosts_to_rewrite;
+    public ?array $hosts_to_rewrite;
 
-    public readonly ?array $replacement_patterns;
+    public ?array $replacement_patterns;
 
-    public readonly ?string $site_url;
+    public ?string $site_url;
 
     public function __construct() {
         if ( Options::getValue( 'skipURLRewrite' ) === '1' ) {