Audiens / adobe-client

src/Authentication/SandboxStrategy.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace Audiens\AdobeClient\Authentication;
 
-use Audiens\AdobeClient\Auth;
 use Audiens\AdobeClient\Exception\AuthException;
 use Doctrine\Common\Cache\Cache;
 use GuzzleHttp\ClientInterface;

Doc Comments +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
      * AdnxStrategy constructor.
      *
      * @param ClientInterface $clientInterface
-     * @param Cache|null $cache
+     * @param Cache $cache
      */
     public function __construct(ClientInterface $clientInterface, Cache $cache)
     {

src/Repository/TraitRepository.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace Audiens\AdobeClient\Repository;
 
-use Audiens\AdobeClient\Auth;
 use Audiens\AdobeClient\CachableTrait;
 use Audiens\AdobeClient\CacheableInterface;
 use Audiens\AdobeClient\Entity\TraitMetrics;

Spacing +2 added lines, -2 removed lines

@@ -100,7 +100,7 @@ 
     public function findOneById($id)
     {
 
-        $compiledUrl = $this->baseUrl . $id.'?includeMetrics=true';
+        $compiledUrl = $this->baseUrl . $id . '?includeMetrics=true';
 
         $response = $this->client->request('GET', $compiledUrl);
 
@@ -168,7 +168,7 @@ 
      */
     public function getTrendByTrait($sid, \DateTime $startDate, \DateTime $endDate, $dayInterval = '1D')
     {
-        $cacheKey = self::CACHE_NAMESPACE . sha1($startDate->getTimestamp().$endDate->getTimestamp());
+        $cacheKey = self::CACHE_NAMESPACE . sha1($startDate->getTimestamp() . $endDate->getTimestamp());
 
         if ($this->isCacheEnabled()) {
             if ($this->cache->contains($cacheKey)) {

src/Auth.php

Spacing +2 added lines, -2 removed lines

@@ -78,7 +78,7 @@ 
 
         $optionForToken = [
             'headers' => [
-                'Authorization' =>  ['Bearer '.$this->authStrategy->authenticate($this->clientId, $this->secretKey, $this->username, $this->password)],
+                'Authorization' =>  ['Bearer ' . $this->authStrategy->authenticate($this->clientId, $this->secretKey, $this->username, $this->password)],
             ],
         ];
 
@@ -93,7 +93,7 @@ 
 
         $optionForToken = [
             'headers' => [
-                'Authorization' =>  ['Bearer '.$this->authStrategy->authenticate($this->clientId, $this->secretKey, $this->username, $this->password, true, true)],
+                'Authorization' =>  ['Bearer ' . $this->authStrategy->authenticate($this->clientId, $this->secretKey, $this->username, $this->password, true, true)],
             ],
         ];
 

Doc Comments +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
      * @param null   $uri
      * @param array  $options
      *
-     * @return mixed|\Psr\Http\Message\ResponseInterface
+     * @return \Psr\Http\Message\ResponseInterface
      * @throws \Exception
      */
     public function request($method, $uri = null, array $options = [])

src/Exception/RepositoryException.php

Spacing +1 added lines, -1 removed lines

@@ -37,6 +37,6 @@
      */
     public static function missingSid($trait)
     {
-        return new self('Missing sid for '.serialize($trait->getName()));
+        return new self('Missing sid for ' . serialize($trait->getName()));
     }
 }

src/Exception/AuthException.php

Spacing +1 added lines, -1 removed lines

@@ -17,6 +17,6 @@
      */
     public static function authFailed($reason)
     {
-        return new self(self::DEFAULT_MESSAGE.$reason, 0, null);
+        return new self(self::DEFAULT_MESSAGE . $reason, 0, null);
     }
 }

src/Authentication/AdnxStrategy.php

Doc Comments +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
      * AdnxStrategy constructor.
      *
      * @param ClientInterface $clientInterface
-     * @param Cache|null $cache
+     * @param Cache $cache
      */
     public function __construct(ClientInterface $clientInterface, Cache $cache)
     {

phpunit_bs.php

Spacing +2 added lines, -2 removed lines

@@ -2,8 +2,8 @@
 
 umask(0000);
 
-if(function_exists('xdebug_disable')) {
+if (function_exists('xdebug_disable')) {
     xdebug_disable();
 }
 
-include(__DIR__.'/vendor/autoload.php');
\ No newline at end of file
+include(__DIR__ . '/vendor/autoload.php');
\ No newline at end of file

src/AuthAdobeTarget.php

Doc Comments +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
      * @param       $method
      * @param null  $uri
      * @param array $options
-     * @return mixed|\Psr\Http\Message\ResponseInterface
+     * @return \Psr\Http\Message\ResponseInterface
      * @throws \GuzzleHttp\Exception\GuzzleException
      */
     public function request($method, $uri = null, array $options = [])

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
     {
         $optionForToken = [
             'headers' => [
-                'Authorization' =>  ['Bearer '.$this->authStrategy->authenticateJwtToken($this->clientId, $this->secretKey, $this->token, true)],
+                'Authorization' =>  ['Bearer ' . $this->authStrategy->authenticateJwtToken($this->clientId, $this->secretKey, $this->token, true)],
                 'content-type' => 'application/vnd.adobe.target.v1+json',
                 'x-api-key' => $this->clientId
             ],

src/Authentication/JwtStrategy.php

Doc Comments +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
      * JwtStrategy constructor.
      *
      * @param ClientInterface $clientInterface
-     * @param Cache|null $cache
+     * @param Cache $cache
      */
     public function __construct(ClientInterface $clientInterface, Cache $cache)
     {