Skip to content

Commit

Permalink
Use interface for constructor arguments (#704)
Browse files Browse the repository at this point in the history
  • Loading branch information
loevgaard committed Jul 6, 2022
1 parent 8e0148e commit a8f0326
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 9 deletions.
5 changes: 3 additions & 2 deletions src/InsightsClient.php
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,13 @@
use Algolia\AlgoliaSearch\Config\InsightsConfig;
use Algolia\AlgoliaSearch\Insights\UserInsightsClient;
use Algolia\AlgoliaSearch\RetryStrategy\ApiWrapper;
use Algolia\AlgoliaSearch\RetryStrategy\ApiWrapperInterface;
use Algolia\AlgoliaSearch\RetryStrategy\ClusterHosts;

final class InsightsClient
{
/**
* @var \Algolia\AlgoliaSearch\RetryStrategy\ApiWrapper
* @var ApiWrapperInterface
*/
private $api;

Expand All @@ -19,7 +20,7 @@ final class InsightsClient
*/
private $config;

public function __construct(ApiWrapper $api, InsightsConfig $config)
public function __construct(ApiWrapperInterface $api, InsightsConfig $config)
{
$this->api = $api;
$this->config = $config;
Expand Down
5 changes: 3 additions & 2 deletions src/PersonalizationClient.php
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,13 @@
use Algolia\AlgoliaSearch\Config\PersonalizationConfig;
use Algolia\AlgoliaSearch\RequestOptions\RequestOptions;
use Algolia\AlgoliaSearch\RetryStrategy\ApiWrapper;
use Algolia\AlgoliaSearch\RetryStrategy\ApiWrapperInterface;
use Algolia\AlgoliaSearch\RetryStrategy\ClusterHosts;

final class PersonalizationClient
{
/**
* @var \Algolia\AlgoliaSearch\RetryStrategy\ApiWrapper
* @var ApiWrapperInterface
*/
private $api;

Expand All @@ -22,7 +23,7 @@ final class PersonalizationClient
/**
* RecommendationClient constructor.
*/
public function __construct(ApiWrapper $api, PersonalizationConfig $config)
public function __construct(ApiWrapperInterface $api, PersonalizationConfig $config)
{
$this->api = $api;
$this->config = $config;
Expand Down
6 changes: 3 additions & 3 deletions src/RecommendClient.php
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
use Algolia\AlgoliaSearch\Config\RecommendConfig;
use Algolia\AlgoliaSearch\Exceptions\AlgoliaException;
use Algolia\AlgoliaSearch\RequestOptions\RequestOptions;
use Algolia\AlgoliaSearch\RetryStrategy\ApiWrapper;
use Algolia\AlgoliaSearch\RetryStrategy\ApiWrapperInterface;
use Algolia\AlgoliaSearch\RetryStrategy\ClusterHosts;

final class RecommendClient
Expand All @@ -14,7 +14,7 @@ final class RecommendClient
const BOUGHT_TOGETHER = 'bought-together';

/**
* @var ApiWrapper
* @var ApiWrapperInterface
*/
private $api;

Expand All @@ -23,7 +23,7 @@ final class RecommendClient
*/
private $config;

public function __construct(ApiWrapper $api, RecommendConfig $config)
public function __construct(ApiWrapperInterface $api, RecommendConfig $config)
{
$this->api = $api;
$this->config = $config;
Expand Down
5 changes: 3 additions & 2 deletions src/RecommendationClient.php
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
use Algolia\AlgoliaSearch\Config\RecommendationConfig;
use Algolia\AlgoliaSearch\RequestOptions\RequestOptions;
use Algolia\AlgoliaSearch\RetryStrategy\ApiWrapper;
use Algolia\AlgoliaSearch\RetryStrategy\ApiWrapperInterface;
use Algolia\AlgoliaSearch\RetryStrategy\ClusterHosts;

/**
Expand All @@ -13,7 +14,7 @@
final class RecommendationClient
{
/**
* @var \Algolia\AlgoliaSearch\RetryStrategy\ApiWrapper
* @var ApiWrapperInterface
*/
private $api;

Expand All @@ -25,7 +26,7 @@ final class RecommendationClient
/**
* RecommendationClient constructor.
*/
public function __construct(ApiWrapper $api, RecommendationConfig $config)
public function __construct(ApiWrapperInterface $api, RecommendationConfig $config)
{
$this->api = $api;
$this->config = $config;
Expand Down

0 comments on commit a8f0326

Please sign in to comment.