Allow passing explicit null values for optional input members (#1542)

This makes it easier to define the array literal for input objects as
PHP does not have a syntax sugar for conditional keys in an array
literal. This is consistent with the generated code for value objects.

Even though input objects allow to omit required members in the
constructor shape (as they can be set later by using the setter), the
phpdoc type still does not allow passing null explicitly (even though
the code would deal with it until the validation run) so that static
analysis tools can catch mistakes there. Passing a required member
explicitly is intended to pass a valid value for it and not a
potentially missing one.

Author: stof

src/CodeCommitClient.php

@@ -84,8 +84,8 @@ class CodeCommitClient extends AbstractApi
      *
      * @param array{
      *   repositoryName: string,
-     *   repositoryDescription?: string,
-     *   tags?: array<string, string>,
+     *   repositoryDescription?: null|string,
+     *   tags?: null|array<string, string>,
      *   '@region'?: string|null,
      * }|CreateRepositoryInput $input
      *
@@ -218,8 +218,8 @@ public function getBlob($input): GetBlobOutput
      * @see https://docs.aws.amazon.com/aws-sdk-php/v3/api/api-codecommit-2015-04-13.html#getbranch
      *
      * @param array{
-     *   repositoryName?: string,
-     *   branchName?: string,
+     *   repositoryName?: null|string,
+     *   branchName?: null|string,
      *   '@region'?: string|null,
      * }|GetBranchInput $input
      *
@@ -308,12 +308,12 @@ public function getCommit($input): GetCommitOutput
      *
      * @param array{
      *   repositoryName: string,
-     *   beforeCommitSpecifier?: string,
+     *   beforeCommitSpecifier?: null|string,
      *   afterCommitSpecifier: string,
-     *   beforePath?: string,
-     *   afterPath?: string,
-     *   MaxResults?: int,
-     *   NextToken?: string,
+     *   beforePath?: null|string,
+     *   afterPath?: null|string,
+     *   MaxResults?: null|int,
+     *   NextToken?: null|string,
      *   '@region'?: string|null,
      * }|GetDifferencesInput $input
      *
@@ -366,9 +366,9 @@ public function getDifferences($input): GetDifferencesOutput
      * @see https://docs.aws.amazon.com/aws-sdk-php/v3/api/api-codecommit-2015-04-13.html#listrepositories
      *
      * @param array{
-     *   nextToken?: string,
-     *   sortBy?: SortByEnum::*,
-     *   order?: OrderEnum::*,
+     *   nextToken?: null|string,
+     *   sortBy?: null|SortByEnum::*,
+     *   order?: null|OrderEnum::*,
      *   '@region'?: string|null,
      * }|ListRepositoriesInput $input
      *

src/Input/CreateRepositoryInput.php

@@ -49,8 +49,8 @@ final class CreateRepositoryInput extends Input
     /**
      * @param array{
      *   repositoryName?: string,
-     *   repositoryDescription?: string,
-     *   tags?: array<string, string>,
+     *   repositoryDescription?: null|string,
+     *   tags?: null|array<string, string>,
      *   '@region'?: string|null,
      * } $input
      */
@@ -65,8 +65,8 @@ public function __construct(array $input = [])
     /**
      * @param array{
      *   repositoryName?: string,
-     *   repositoryDescription?: string,
-     *   tags?: array<string, string>,
+     *   repositoryDescription?: null|string,
+     *   tags?: null|array<string, string>,
      *   '@region'?: string|null,
      * }|CreateRepositoryInput $input
      */

src/Input/GetBranchInput.php

@@ -27,8 +27,8 @@ final class GetBranchInput extends Input
 
     /**
      * @param array{
-     *   repositoryName?: string,
-     *   branchName?: string,
+     *   repositoryName?: null|string,
+     *   branchName?: null|string,
      *   '@region'?: string|null,
      * } $input
      */
@@ -41,8 +41,8 @@ public function __construct(array $input = [])
 
     /**
      * @param array{
-     *   repositoryName?: string,
-     *   branchName?: string,
+     *   repositoryName?: null|string,
+     *   branchName?: null|string,
      *   '@region'?: string|null,
      * }|GetBranchInput $input
      */

src/Input/GetDifferencesInput.php

@@ -70,12 +70,12 @@ final class GetDifferencesInput extends Input
     /**
      * @param array{
      *   repositoryName?: string,
-     *   beforeCommitSpecifier?: string,
+     *   beforeCommitSpecifier?: null|string,
      *   afterCommitSpecifier?: string,
-     *   beforePath?: string,
-     *   afterPath?: string,
-     *   MaxResults?: int,
-     *   NextToken?: string,
+     *   beforePath?: null|string,
+     *   afterPath?: null|string,
+     *   MaxResults?: null|int,
+     *   NextToken?: null|string,
      *   '@region'?: string|null,
      * } $input
      */
@@ -94,12 +94,12 @@ public function __construct(array $input = [])
     /**
      * @param array{
      *   repositoryName?: string,
-     *   beforeCommitSpecifier?: string,
+     *   beforeCommitSpecifier?: null|string,
      *   afterCommitSpecifier?: string,
-     *   beforePath?: string,
-     *   afterPath?: string,
-     *   MaxResults?: int,
-     *   NextToken?: string,
+     *   beforePath?: null|string,
+     *   afterPath?: null|string,
+     *   MaxResults?: null|int,
+     *   NextToken?: null|string,
      *   '@region'?: string|null,
      * }|GetDifferencesInput $input
      */

src/Input/ListRepositoriesInput.php

@@ -39,9 +39,9 @@ final class ListRepositoriesInput extends Input
 
     /**
      * @param array{
-     *   nextToken?: string,
-     *   sortBy?: SortByEnum::*,
-     *   order?: OrderEnum::*,
+     *   nextToken?: null|string,
+     *   sortBy?: null|SortByEnum::*,
+     *   order?: null|OrderEnum::*,
      *   '@region'?: string|null,
      * } $input
      */
@@ -55,9 +55,9 @@ public function __construct(array $input = [])
 
     /**
      * @param array{
-     *   nextToken?: string,
-     *   sortBy?: SortByEnum::*,
-     *   order?: OrderEnum::*,
+     *   nextToken?: null|string,
+     *   sortBy?: null|SortByEnum::*,
+     *   order?: null|OrderEnum::*,
      *   '@region'?: string|null,
      * }|ListRepositoriesInput $input
      */