Use constructor property promotion

Author: stevebauman

src/Documents/Document.php

@@ -7,28 +7,24 @@
 
 class Document implements Arrayable
 {
-    /**
-     * The OpenSearch document identifier.
-     */
-    protected string $id;
-
-    /**
-     * The document source payload.
-     *
-     * @var array<string, mixed>
-     */
-    protected array $content;
-
     /**
      * Create a new document instance.
      *
      * @param  array<string, mixed>  $content
      */
-    public function __construct(string $id, array $content)
-    {
-        $this->id = $id;
-        $this->content = $content;
-    }
+    public function __construct(
+        /**
+         * The OpenSearch document identifier.
+         */
+        protected string $id,
+
+        /**
+         * The document source payload.
+         *
+         * @var array<string, mixed>
+         */
+        protected array $content,
+    ) {}
 
     /**
      * Get the document identifier.

src/Documents/DocumentManager.php

@@ -10,18 +10,15 @@
 
 class DocumentManager
 {
-    /**
-     * The OpenSearch client instance.
-     */
-    protected Client $client;
-
     /**
      * Create a new document manager instance.
      */
-    public function __construct(Client $client)
-    {
-        $this->client = $client;
-    }
+    public function __construct(
+        /**
+         * The OpenSearch client instance.
+         */
+        protected Client $client,
+    ) {}
 
     /**
      * Index the given documents into OpenSearch.

src/Exceptions/BulkRequestException.php

@@ -6,22 +6,19 @@
 
 class BulkRequestException extends ErrorException
 {
-    /**
-     * The OpenSearch bulk response.
-     *
-     * @var array<string, mixed>
-     */
-    protected array $response;
-
     /**
      * Create a new bulk request exception instance.
      *
      * @param  array<string, mixed>  $response
      */
-    public function __construct(array $response)
-    {
-        $this->response = $response;
-
+    public function __construct(
+        /**
+         * The OpenSearch bulk response.
+         *
+         * @var array<string, mixed>
+         */
+        protected array $response,
+    ) {
         parent::__construct($this->makeErrorFromResponse());
     }
 

src/Indices/Alias.php

@@ -4,34 +4,29 @@
 
 class Alias
 {
-    /**
-     * The alias name.
-     */
-    protected string $name;
-
-    /**
-     * The optional alias filter.
-     *
-     * @var array<string, mixed>|null
-     */
-    protected ?array $filter;
-
-    /**
-     * The optional alias routing value.
-     */
-    protected ?string $routing;
-
     /**
      * Create a new alias instance.
      *
      * @param  array<string, mixed>|null  $filter
      */
-    public function __construct(string $name, ?array $filter = null, ?string $routing = null)
-    {
-        $this->name = $name;
-        $this->filter = $filter;
-        $this->routing = $routing;
-    }
+    public function __construct(
+        /**
+         * The alias name.
+         */
+        protected string $name,
+
+        /**
+         * The optional alias filter.
+         *
+         * @var array<string, mixed>|null
+         */
+        protected ?array $filter = null,
+
+        /**
+         * The optional alias routing value.
+         */
+        protected ?string $routing = null,
+    ) {}
 
     /**
      * Get the alias name.

src/Indices/IndexBlueprint.php

@@ -4,30 +4,25 @@
 
 class IndexBlueprint
 {
-    /**
-     * The index name.
-     */
-    protected string $name;
-
-    /**
-     * The index mapping definition.
-     */
-    protected ?Mapping $mapping;
-
-    /**
-     * The index settings definition.
-     */
-    protected ?Settings $settings;
-
     /**
      * Create a new index blueprint instance.
      */
-    public function __construct(string $name, ?Mapping $mapping = null, ?Settings $settings = null)
-    {
-        $this->name = $name;
-        $this->mapping = $mapping;
-        $this->settings = $settings;
-    }
+    public function __construct(
+        /**
+         * The index name.
+         */
+        protected string $name,
+
+        /**
+         * The index mapping definition.
+         */
+        protected ?Mapping $mapping = null,
+
+        /**
+         * The index settings definition.
+         */
+        protected ?Settings $settings = null,
+    ) {}
 
     /**
      * Get the index name.

src/Search/Aggregation.php

@@ -6,22 +6,19 @@
 
 class Aggregation implements RawResponseInterface
 {
-    /**
-     * The raw OpenSearch aggregation payload.
-     *
-     * @var array<string, mixed>
-     */
-    protected array $aggregation;
-
     /**
      * Create a new aggregation instance.
      *
      * @param  array<string, mixed>  $aggregation
      */
-    public function __construct(array $aggregation)
-    {
-        $this->aggregation = $aggregation;
-    }
+    public function __construct(
+        /**
+         * The raw OpenSearch aggregation payload.
+         *
+         * @var array<string, mixed>
+         */
+        protected array $aggregation,
+    ) {}
 
     /**
      * Get the aggregation buckets.

src/Search/Bucket.php

@@ -4,22 +4,19 @@
 
 class Bucket implements RawResponseInterface
 {
-    /**
-     * The raw OpenSearch bucket payload.
-     *
-     * @var array<string, mixed>
-     */
-    protected array $bucket;
-
     /**
      * Create a new bucket instance.
      *
      * @param  array<string, mixed>  $bucket
      */
-    public function __construct(array $bucket)
-    {
-        $this->bucket = $bucket;
-    }
+    public function __construct(
+        /**
+         * The raw OpenSearch bucket payload.
+         *
+         * @var array<string, mixed>
+         */
+        protected array $bucket,
+    ) {}
 
     /**
      * Get the number of documents in the bucket.

src/Search/Highlight.php

@@ -6,22 +6,19 @@
 
 class Highlight implements RawResponseInterface
 {
-    /**
-     * The raw OpenSearch highlight payload.
-     *
-     * @var array<string, array<int, string>>
-     */
-    protected array $highlight;
-
     /**
      * Create a new highlight instance.
      *
      * @param  array<string, array<int, string>>  $highlight
      */
-    public function __construct(array $highlight)
-    {
-        $this->highlight = $highlight;
-    }
+    public function __construct(
+        /**
+         * The raw OpenSearch highlight payload.
+         *
+         * @var array<string, array<int, string>>
+         */
+        protected array $highlight,
+    ) {}
 
     /**
      * Get highlighted snippets for the given field.

src/Search/Hit.php

@@ -7,22 +7,19 @@
 
 class Hit implements RawResponseInterface
 {
-    /**
-     * The raw OpenSearch hit payload.
-     *
-     * @var array<string, mixed>
-     */
-    protected array $hit;
-
     /**
      * Create a new hit instance.
      *
      * @param  array<string, mixed>  $hit
      */
-    public function __construct(array $hit)
-    {
-        $this->hit = $hit;
-    }
+    public function __construct(
+        /**
+         * The raw OpenSearch hit payload.
+         *
+         * @var array<string, mixed>
+         */
+        protected array $hit,
+    ) {}
 
     /**
      * Get the index name for the hit.

src/Search/SearchResponse.php

@@ -6,22 +6,19 @@
 
 class SearchResponse implements RawResponseInterface
 {
-    /**
-     * The raw OpenSearch search response.
-     *
-     * @var array<string, mixed>
-     */
-    protected array $response;
-
     /**
      * Create a new search response instance.
      *
      * @param  array<string, mixed>  $response
      */
-    public function __construct(array $response)
-    {
-        $this->response = $response;
-    }
+    public function __construct(
+        /**
+         * The raw OpenSearch search response.
+         *
+         * @var array<string, mixed>
+         */
+        protected array $response,
+    ) {}
 
     /**
      * Get the search hits.