Supporting passing the client and marshaler to the constructor (big oversight).

guillermoandrae · guillermoandrae · commit 45e18b8555d7 · 2020-10-18T19:31:23.000-04:00
diff --git a/src/DynamoDb/DynamoDbAdapter.php b/src/DynamoDb/DynamoDbAdapter.php
@@ -38,15 +38,15 @@ public function __construct(?DynamoDbClient $client = null, ?Marshaler $marshale
     {
         if (empty($client)) {
             $client = DynamoDbClientFactory::factory();
-            $this->setClient($client);
-            OperationFactory::setClient($client);
         }
+        $this->setClient($client);
+        OperationFactory::setClient($client);
 
         if (empty($marshaler)) {
             $marshaler = MarshalerFactory::factory();
-            $this->setMarshaler($marshaler);
-            OperationFactory::setMarshaler($marshaler);
         }
+        $this->setMarshaler($marshaler);
+        OperationFactory::setMarshaler($marshaler);
     }
 
     public function createTable(array $data, ?string $tableName = '', ?array $options = []): bool

Original file line number	Diff line number	Diff line change
`@@ -38,15 +38,15 @@ public function __construct(?DynamoDbClient $client = null, ?Marshaler $marshale`
`38`	`38`	`{`
`39`	`39`	`if (empty($client)) {`
`40`	`40`	`$client = DynamoDbClientFactory::factory();`
`41`		`- $this->setClient($client);`
`42`		`- OperationFactory::setClient($client);`
`43`	`41`	`}`
	`42`	`+ $this->setClient($client);`
	`43`	`+ OperationFactory::setClient($client);`
`44`	`44`
`45`	`45`	`if (empty($marshaler)) {`
`46`	`46`	`$marshaler = MarshalerFactory::factory();`
`47`		`- $this->setMarshaler($marshaler);`
`48`		`- OperationFactory::setMarshaler($marshaler);`
`49`	`47`	`}`
	`48`	`+ $this->setMarshaler($marshaler);`
	`49`	`+ OperationFactory::setMarshaler($marshaler);`
`50`	`50`	`}`
`51`	`51`
`52`	`52`	`public function createTable(array $data, ?string $tableName = '', ?array $options = []): bool`