Skip to content

Commit

Permalink
Merge pull request #4 from php-etl/sylius-plugin-tests
Browse files Browse the repository at this point in the history
set default values on optional variables, since they might be accessed before initialization
  • Loading branch information
clemzarch authored Aug 1, 2022
2 parents 7ff38bc + 112fa8c commit 1120f7c
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/SyliusLegacyClientFactory.php
Original file line number Diff line number Diff line change
Expand Up @@ -43,9 +43,9 @@
class SyliusLegacyClientFactory implements SyliusLegacyClientBuilderInterface
{
private ?string $baseUri = '';
private Client $httpClient;
private RequestFactoryInterface $requestFactory;
private StreamFactoryInterface $streamFactory;
private ?Client $httpClient = null;
private ?RequestFactoryInterface $requestFactory = null;
private ?StreamFactoryInterface $streamFactory = null;
private ?FileSystemInterface $fileSystem = null;
/** @var list<ApiAwareInterface> */
private array $apiRegistry = [];
Expand Down

0 comments on commit 1120f7c

Please sign in to comment.