Merge pull request #518 from HubSpot/feature/fixWarnings

ksvirkou-hubspot · web-flow · commit cec2161c05e5 · 2025-11-13T10:51:08.000+03:00
Fix Implicitly marking parameter as nullable in CRM Objects
diff --git a/codegen/Crm/Objects/Api/BasicApi.php b/codegen/Crm/Objects/Api/BasicApi.php
@@ -95,9 +95,9 @@ class BasicApi
      * @param int             $hostIndex (Optional) host index to select the list of hosts if defined in the OpenAPI spec
      */
     public function __construct(
-        ClientInterface $client = null,
-        Configuration $config = null,
-        HeaderSelector $selector = null,
+        ?ClientInterface $client = null,
+        ?Configuration $config = null,
+        ?HeaderSelector $selector = null,
         $hostIndex = 0
     ) {
         $this->client = $client ?: new Client();
diff --git a/codegen/Crm/Objects/Api/BatchApi.php b/codegen/Crm/Objects/Api/BatchApi.php
@@ -95,9 +95,9 @@ class BatchApi
      * @param int             $hostIndex (Optional) host index to select the list of hosts if defined in the OpenAPI spec
      */
     public function __construct(
-        ClientInterface $client = null,
-        Configuration $config = null,
-        HeaderSelector $selector = null,
+        ?ClientInterface $client = null,
+        ?Configuration $config = null,
+        ?HeaderSelector $selector = null,
         $hostIndex = 0
     ) {
         $this->client = $client ?: new Client();
diff --git a/codegen/Crm/Objects/Api/SearchApi.php b/codegen/Crm/Objects/Api/SearchApi.php
@@ -83,9 +83,9 @@ class SearchApi
      * @param int             $hostIndex (Optional) host index to select the list of hosts if defined in the OpenAPI spec
      */
     public function __construct(
-        ClientInterface $client = null,
-        Configuration $config = null,
-        HeaderSelector $selector = null,
+        ?ClientInterface $client = null,
+        ?Configuration $config = null,
+        ?HeaderSelector $selector = null,
         $hostIndex = 0
     ) {
         $this->client = $client ?: new Client();
diff --git a/codegen/Crm/Objects/Configuration.php b/codegen/Crm/Objects/Configuration.php
@@ -486,7 +486,7 @@ public function getHostSettings()
     * @param array|null $variables    hash of variable and the corresponding value (optional)
     * @return string URL based on host settings
     */
-    public static function getHostString(array $hostSettings, $hostIndex, array $variables = null)
+    public static function getHostString(array $hostSettings, $hostIndex, ?array $variables = null)
     {
         if (null === $variables) {
             $variables = [];
diff --git a/codegen/Crm/Objects/Model/AssociatedId.php b/codegen/Crm/Objects/Model/AssociatedId.php
@@ -248,7 +248,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('id', $data ?? [], null);
         $this->setIfExists('type', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/AssociationSpec.php b/codegen/Crm/Objects/Model/AssociationSpec.php
@@ -265,7 +265,7 @@ public function getAssociationCategoryAllowableValues()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('association_category', $data ?? [], null);
         $this->setIfExists('association_type_id', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/BatchInputSimplePublicObjectBatchInput.php b/codegen/Crm/Objects/Model/BatchInputSimplePublicObjectBatchInput.php
@@ -242,7 +242,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('inputs', $data ?? [], null);
     }
diff --git a/codegen/Crm/Objects/Model/BatchInputSimplePublicObjectBatchInputForCreate.php b/codegen/Crm/Objects/Model/BatchInputSimplePublicObjectBatchInputForCreate.php
@@ -242,7 +242,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('inputs', $data ?? [], null);
     }
diff --git a/codegen/Crm/Objects/Model/BatchInputSimplePublicObjectBatchInputUpsert.php b/codegen/Crm/Objects/Model/BatchInputSimplePublicObjectBatchInputUpsert.php
@@ -242,7 +242,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('inputs', $data ?? [], null);
     }
diff --git a/codegen/Crm/Objects/Model/BatchInputSimplePublicObjectId.php b/codegen/Crm/Objects/Model/BatchInputSimplePublicObjectId.php
@@ -242,7 +242,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('inputs', $data ?? [], null);
     }
diff --git a/codegen/Crm/Objects/Model/BatchReadInputSimplePublicObjectId.php b/codegen/Crm/Objects/Model/BatchReadInputSimplePublicObjectId.php
@@ -260,7 +260,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('properties_with_history', $data ?? [], null);
         $this->setIfExists('id_property', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/BatchResponseSimplePublicObject.php b/codegen/Crm/Objects/Model/BatchResponseSimplePublicObject.php
@@ -291,7 +291,7 @@ public function getStatusAllowableValues()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('completed_at', $data ?? [], null);
         $this->setIfExists('requested_at', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/BatchResponseSimplePublicObjectWithErrors.php b/codegen/Crm/Objects/Model/BatchResponseSimplePublicObjectWithErrors.php
@@ -303,7 +303,7 @@ public function getStatusAllowableValues()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('completed_at', $data ?? [], null);
         $this->setIfExists('num_errors', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/BatchResponseSimplePublicUpsertObject.php b/codegen/Crm/Objects/Model/BatchResponseSimplePublicUpsertObject.php
@@ -291,7 +291,7 @@ public function getStatusAllowableValues()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('completed_at', $data ?? [], null);
         $this->setIfExists('requested_at', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/BatchResponseSimplePublicUpsertObjectWithErrors.php b/codegen/Crm/Objects/Model/BatchResponseSimplePublicUpsertObjectWithErrors.php
@@ -303,7 +303,7 @@ public function getStatusAllowableValues()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('completed_at', $data ?? [], null);
         $this->setIfExists('num_errors', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/CollectionResponseAssociatedId.php b/codegen/Crm/Objects/Model/CollectionResponseAssociatedId.php
@@ -248,7 +248,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('paging', $data ?? [], null);
         $this->setIfExists('results', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/CollectionResponseSimplePublicObjectWithAssociationsForwardPaging.php b/codegen/Crm/Objects/Model/CollectionResponseSimplePublicObjectWithAssociationsForwardPaging.php
@@ -248,7 +248,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('paging', $data ?? [], null);
         $this->setIfExists('results', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/CollectionResponseWithTotalSimplePublicObjectForwardPaging.php b/codegen/Crm/Objects/Model/CollectionResponseWithTotalSimplePublicObjectForwardPaging.php
@@ -254,7 +254,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('total', $data ?? [], null);
         $this->setIfExists('paging', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/Error.php b/codegen/Crm/Objects/Model/Error.php
@@ -278,7 +278,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('sub_category', $data ?? [], null);
         $this->setIfExists('context', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/ErrorDetail.php b/codegen/Crm/Objects/Model/ErrorDetail.php
@@ -266,7 +266,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('sub_category', $data ?? [], null);
         $this->setIfExists('code', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/Filter.php b/codegen/Crm/Objects/Model/Filter.php
@@ -303,7 +303,7 @@ public function getOperatorAllowableValues()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('high_value', $data ?? [], null);
         $this->setIfExists('property_name', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/FilterGroup.php b/codegen/Crm/Objects/Model/FilterGroup.php
@@ -242,7 +242,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('filters', $data ?? [], null);
     }
diff --git a/codegen/Crm/Objects/Model/ForwardPaging.php b/codegen/Crm/Objects/Model/ForwardPaging.php
@@ -242,7 +242,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('next', $data ?? [], null);
     }
diff --git a/codegen/Crm/Objects/Model/NextPage.php b/codegen/Crm/Objects/Model/NextPage.php
@@ -248,7 +248,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('link', $data ?? [], null);
         $this->setIfExists('after', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/Paging.php b/codegen/Crm/Objects/Model/Paging.php
@@ -248,7 +248,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('next', $data ?? [], null);
         $this->setIfExists('prev', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/PreviousPage.php b/codegen/Crm/Objects/Model/PreviousPage.php
@@ -248,7 +248,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('before', $data ?? [], null);
         $this->setIfExists('link', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/PublicAssociationsForObject.php b/codegen/Crm/Objects/Model/PublicAssociationsForObject.php
@@ -248,7 +248,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('types', $data ?? [], null);
         $this->setIfExists('to', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/PublicObjectId.php b/codegen/Crm/Objects/Model/PublicObjectId.php
@@ -242,7 +242,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('id', $data ?? [], null);
     }
diff --git a/codegen/Crm/Objects/Model/PublicObjectSearchRequest.php b/codegen/Crm/Objects/Model/PublicObjectSearchRequest.php
@@ -272,7 +272,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('query', $data ?? [], null);
         $this->setIfExists('limit', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/SimplePublicObject.php b/codegen/Crm/Objects/Model/SimplePublicObject.php
@@ -284,7 +284,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('created_at', $data ?? [], null);
         $this->setIfExists('archived', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/SimplePublicObjectBatchInput.php b/codegen/Crm/Objects/Model/SimplePublicObjectBatchInput.php
@@ -260,7 +260,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('id_property', $data ?? [], null);
         $this->setIfExists('object_write_trace_id', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/SimplePublicObjectBatchInputForCreate.php b/codegen/Crm/Objects/Model/SimplePublicObjectBatchInputForCreate.php
@@ -254,7 +254,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('associations', $data ?? [], null);
         $this->setIfExists('object_write_trace_id', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/SimplePublicObjectBatchInputUpsert.php b/codegen/Crm/Objects/Model/SimplePublicObjectBatchInputUpsert.php
@@ -260,7 +260,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('id_property', $data ?? [], null);
         $this->setIfExists('object_write_trace_id', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/SimplePublicObjectId.php b/codegen/Crm/Objects/Model/SimplePublicObjectId.php
@@ -242,7 +242,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('id', $data ?? [], null);
     }
diff --git a/codegen/Crm/Objects/Model/SimplePublicObjectInput.php b/codegen/Crm/Objects/Model/SimplePublicObjectInput.php
@@ -242,7 +242,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('properties', $data ?? [], null);
     }
diff --git a/codegen/Crm/Objects/Model/SimplePublicObjectInputForCreate.php b/codegen/Crm/Objects/Model/SimplePublicObjectInputForCreate.php
@@ -248,7 +248,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('associations', $data ?? [], null);
         $this->setIfExists('properties', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/SimplePublicObjectWithAssociations.php b/codegen/Crm/Objects/Model/SimplePublicObjectWithAssociations.php
@@ -290,7 +290,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('associations', $data ?? [], null);
         $this->setIfExists('created_at', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/SimplePublicUpsertObject.php b/codegen/Crm/Objects/Model/SimplePublicUpsertObject.php
@@ -290,7 +290,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('created_at', $data ?? [], null);
         $this->setIfExists('archived', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/StandardError.php b/codegen/Crm/Objects/Model/StandardError.php
@@ -284,7 +284,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('sub_category', $data ?? [], null);
         $this->setIfExists('context', $data ?? [], null);
diff --git a/codegen/Crm/Objects/Model/ValueWithTimestamp.php b/codegen/Crm/Objects/Model/ValueWithTimestamp.php
@@ -272,7 +272,7 @@ public function getModelName()
      * @param mixed[] $data Associated array of property values
      *                      initializing the model
      */
-    public function __construct(array $data = null)
+    public function __construct(?array $data = null)
     {
         $this->setIfExists('source_id', $data ?? [], null);
         $this->setIfExists('source_type', $data ?? [], null);

Original file line number	Diff line number	Diff line change
`@@ -486,7 +486,7 @@ public function getHostSettings()`
`486`	`486`	`* @param array\|null $variables hash of variable and the corresponding value (optional)`
`487`	`487`	`* @return string URL based on host settings`
`488`	`488`	`*/`
`489`		`- public static function getHostString(array $hostSettings, $hostIndex, array $variables = null)`
	`489`	`+ public static function getHostString(array $hostSettings, $hostIndex, ?array $variables = null)`
`490`	`490`	`{`
`491`	`491`	`if (null === $variables) {`
`492`	`492`	`$variables = [];`
Original file line number	Diff line number	Diff line change
`@@ -248,7 +248,7 @@ public function getModelName()`
`248`	`248`	`* @param mixed[] $data Associated array of property values`
`249`	`249`	`* initializing the model`
`250`	`250`	`*/`
`251`		`- public function __construct(array $data = null)`
	`251`	`+ public function __construct(?array $data = null)`
`252`	`252`	`{`
`253`	`253`	`$this->setIfExists('id', $data ?? [], null);`
`254`	`254`	`$this->setIfExists('type', $data ?? [], null);`
Original file line number	Diff line number	Diff line change
`@@ -265,7 +265,7 @@ public function getAssociationCategoryAllowableValues()`
`265`	`265`	`* @param mixed[] $data Associated array of property values`
`266`	`266`	`* initializing the model`
`267`	`267`	`*/`
`268`		`- public function __construct(array $data = null)`
	`268`	`+ public function __construct(?array $data = null)`
`269`	`269`	`{`
`270`	`270`	`$this->setIfExists('association_category', $data ?? [], null);`
`271`	`271`	`$this->setIfExists('association_type_id', $data ?? [], null);`
Original file line number	Diff line number	Diff line change
`@@ -242,7 +242,7 @@ public function getModelName()`
`242`	`242`	`* @param mixed[] $data Associated array of property values`
`243`	`243`	`* initializing the model`
`244`	`244`	`*/`
`245`		`- public function __construct(array $data = null)`
	`245`	`+ public function __construct(?array $data = null)`
`246`	`246`	`{`
`247`	`247`	`$this->setIfExists('inputs', $data ?? [], null);`
`248`	`248`	`}`