Apply fixes from StyleCI

src/OpenPlatform/Authorization.php

@@ -26,7 +26,7 @@ class Authorization
 {
     use Caches;
 
-    const CACHE_KEY_ACCESS_TOKEN  = 'easywechat.open_platform.authorizer_access_token';
+    const CACHE_KEY_ACCESS_TOKEN = 'easywechat.open_platform.authorizer_access_token';
     const CACHE_KEY_REFRESH_TOKEN = 'easywechat.open_platform.authorizer_refresh_token';
 
     /**
@@ -79,11 +79,12 @@ public function setAuthorizerAppId($authorizerAppId)
      * Gets the authorizer app id, or throws if not found.
      *
      * @return string
+     *
      * @throws Exception
      */
     public function getAuthorizerAppId()
     {
-        if (! $this->authorizerAppId) {
+        if (!$this->authorizerAppId) {
             throw new Exception(
                 'Authorizer App Id is not present, you may not make the authorization yet.'
             );
@@ -153,7 +154,7 @@ public function handleAuthorization()
     /**
      * Handles the authorizer access token: calls the API, saves the token.
      *
-     * @return string The authorizer access token.
+     * @return string the authorizer access token
      */
     public function handleAuthorizerAccessToken()
     {
@@ -202,7 +203,7 @@ public function getAuthorizerInfo()
     /**
      * Saves the authorizer access token in cache.
      *
-     * @param Collection|array $data Array structure from WeChat API result.
+     * @param Collection|array $data array structure from WeChat API result
      */
     public function saveAuthorizerAccessToken($data)
     {
@@ -226,7 +227,7 @@ public function getAuthorizerAccessToken()
     /**
      * Saves the authorizer refresh token in cache.
      *
-     * @param Collection|array $data Array structure from WeChat API result.
+     * @param Collection|array $data array structure from WeChat API result
      */
     public function saveAuthorizerRefreshToken($data)
     {
@@ -239,7 +240,8 @@ public function saveAuthorizerRefreshToken($data)
      * Gets the authorizer refresh token.
      *
      * @return string
-     * @throws Exception When refresh token is not present.
+     *
+     * @throws Exception when refresh token is not present
      */
     public function getAuthorizerRefreshToken()
     {
@@ -276,8 +278,8 @@ public function removeAuthorizerRefreshToken()
     public function getAuthorizerAccessTokenKey()
     {
         return self::CACHE_KEY_ACCESS_TOKEN
-            . '.' . $this->appId
-            . '.' . $this->getAuthorizerAppId();
+            .'.'.$this->appId
+            .'.'.$this->getAuthorizerAppId();
     }
 
     /**
@@ -288,8 +290,7 @@ public function getAuthorizerAccessTokenKey()
     public function getAuthorizerRefreshTokenKey()
     {
         return self::CACHE_KEY_REFRESH_TOKEN
-            . '.' . $this->appId
-            . '.' . $this->getAuthorizerAppId();
+            .'.'.$this->appId
+            .'.'.$this->getAuthorizerAppId();
     }
-
-}
+}

src/OpenPlatform/AuthorizerToken.php

@@ -32,16 +32,14 @@
 use EasyWeChat\Core\AccessToken as BaseAccessToken;
 
 /**
- * Class AuthorizerToken
+ * Class AuthorizerToken.
  *
  * AuthorizerToken is responsible for the access token of the authorizer,
  * the complexity is that this access token also requires the refresh token
  * of the authorizer which is acquired by the open platform authorization
  * process.
  *
  * This completely overrides the original AccessToken.
- *
- * @package EasyWeChat\OpenPlatform
  */
 class AuthorizerToken extends BaseAccessToken
 {
@@ -55,7 +53,7 @@ class AuthorizerToken extends BaseAccessToken
     /**
      * AuthorizerAccessToken constructor.
      *
-     * @param string $appId
+     * @param string        $appId
      * @param Authorization $authorization
      */
     public function __construct($appId, Authorization $authorization)

src/OpenPlatform/Components/AbstractComponent.php

@@ -59,7 +59,7 @@ abstract class AbstractComponent extends AbstractAPI
      * AbstractComponent constructor.
      *
      * @param AccessToken $accessToken
-     * @param array $config
+     * @param array       $config
      * @param $request
      */
     public function __construct($accessToken, array $config, $request = null)
@@ -73,7 +73,8 @@ public function __construct($accessToken, array $config, $request = null)
      * Get AppId.
      *
      * @return string
-     * @throws Exception When app id is not present.
+     *
+     * @throws Exception when app id is not present
      */
     public function getAppId()
     {
@@ -107,5 +108,4 @@ public function setAppId($appId)
 
         return $this;
     }
-
 }

src/OpenPlatform/Components/Authorizer.php

@@ -146,5 +146,4 @@ public function setAuthorizerOption($authorizerAppId, $optionName, $optionValue)
 
         return $this->parseJSON('json', [self::SET_AUTHORIZER_OPTION, $data]);
     }
-
 }

src/OpenPlatform/Components/PreAuthCode.php

@@ -50,7 +50,6 @@ class PreAuthCode extends AbstractComponent
      */
     protected $redirectUri;
 
-
     /**
      * Get pre auth code.
      *

src/OpenPlatform/EventHandlers/Authorized.php

@@ -48,6 +48,7 @@ public function __construct(Authorization $authorization)
     public function handle(Collection $message)
     {
         $this->authorization->setFromAuthMessage($message);
+
         return $this->authorization->handleAuthorization();
     }
 }

src/OpenPlatform/EventHandlers/Unauthorized.php

@@ -32,7 +32,7 @@
 class Unauthorized extends Authorized
 {
     /**
-     * @inheritdoc
+     * {@inheritdoc}
      */
     public function handle(Collection $message)
     {

src/OpenPlatform/Guard.php

@@ -36,9 +36,9 @@
 
 class Guard extends ServerGuard
 {
-    const EVENT_AUTHORIZED              = 'authorized';
-    const EVENT_UNAUTHORIZED            = 'unauthorized';
-    const EVENT_UPDATE_AUTHORIZED       = 'updateauthorized';
+    const EVENT_AUTHORIZED = 'authorized';
+    const EVENT_UNAUTHORIZED = 'unauthorized';
+    const EVENT_UPDATE_AUTHORIZED = 'updateauthorized';
     const EVENT_COMPONENT_VERIFY_TICKET = 'component_verify_ticket';
 
     /**
@@ -51,7 +51,7 @@ class Guard extends ServerGuard
     /**
      * Guard constructor.
      *
-     * @param string $token
+     * @param string  $token
      * @param Request $request
      */
     public function __construct($token, Request $request = null)
@@ -72,11 +72,10 @@ public function setContainer(Container $container)
     }
 
     /**
-     * @inheritdoc
+     * {@inheritdoc}
      */
     public function serve()
     {
-
         // If sees the `auth_code` query parameter in the url, that is,
         // authorization is successful and it calls back, meanwhile, an
         // ` authorized` event, which also includes the auth code, is sent
@@ -140,7 +139,7 @@ public function listen($callback = null)
     }
 
     /**
-     * @inheritdoc
+     * {@inheritdoc}
      */
     protected function handleMessage($message)
     {
@@ -156,7 +155,7 @@ protected function handleMessage($message)
         if (is_array($result) || $result instanceof Collection) {
             $message->merge($result);
         } else {
-            if (! empty($result)) {
+            if (!empty($result)) {
                 $message->set('result', $result);
             }
         }
@@ -174,12 +173,13 @@ protected function handleMessage($message)
      * @param $type
      *
      * @return EventHandlers\EventHandler
+     *
      * @throws InvalidArgumentException
      */
     protected function getDefaultHandler($type)
     {
         $handler = $this->container->offsetGet("open_platform.handlers.{$type}");
-        if (! $handler) {
+        if (!$handler) {
             throw new InvalidArgumentException("EventHandler \"$type\" does not exists.");
         }
 

src/OpenPlatform/ServiceProvider.php

@@ -98,7 +98,7 @@ public function register(Container $pimple)
             );
         };
 
-        $pimple['open_platform.authorization'] = function($pimple) {
+        $pimple['open_platform.authorization'] = function ($pimple) {
             return new Authorization(
                 $pimple['open_platform.authorizer'],
                 $pimple['config']['open_platform']['app_id'],
@@ -114,18 +114,17 @@ public function register(Container $pimple)
         };
 
         // Authorization events handlers.
-        $pimple['open_platform.handlers.component_verify_ticket'] = function($pimple) {
+        $pimple['open_platform.handlers.component_verify_ticket'] = function ($pimple) {
             return new ComponentVerifyTicket($pimple['open_platform.verify_ticket']);
         };
-        $pimple['open_platform.handlers.authorized'] = function($pimple) {
+        $pimple['open_platform.handlers.authorized'] = function ($pimple) {
             return new Authorized($pimple['open_platform.authorization']);
         };
-        $pimple['open_platform.handlers.updateauthorized'] = function($pimple) {
+        $pimple['open_platform.handlers.updateauthorized'] = function ($pimple) {
             return new UpdateAuthorized($pimple['open_platform.authorization']);
         };
-        $pimple['open_platform.handlers.unauthorized'] = function($pimple) {
+        $pimple['open_platform.handlers.unauthorized'] = function ($pimple) {
             return new Unauthorized($pimple['open_platform.authorization']);
         };
     }
-
 }

src/OpenPlatform/Traits/Caches.php

@@ -54,7 +54,7 @@ public function getCache()
      * Gets the cached data.
      *
      * @param string $key
-     * @param mixed $default A default value or a callable to return the value.
+     * @param mixed  $default a default value or a callable to return the value
      *
      * @return mixed
      */
@@ -75,8 +75,8 @@ public function get($key, $default = null)
      * Sets the cached data.
      *
      * @param string $key
-     * @param mixed $value
-     * @param int $life Cache life time in seconds.
+     * @param mixed  $value
+     * @param int    $life  cache life time in seconds
      *
      * @return bool
      */
@@ -96,5 +96,4 @@ public function remove($key)
     {
         return $this->getCache()->delete($key);
     }
-
-}
+}

src/OpenPlatform/VerifyTicket.php

@@ -68,7 +68,7 @@ class VerifyTicket
      * VerifyTicket constructor.
      *
      * @param string $appId
-     * @param Cache $cache
+     * @param Cache  $cache
      */
     public function __construct($appId, Cache $cache = null)
     {
@@ -129,7 +129,7 @@ public function setCacheKey($cacheKey)
     public function getCacheKey()
     {
         if (is_null($this->cacheKey)) {
-            return $this->prefix . $this->appId;
+            return $this->prefix.$this->appId;
         }
 
         return $this->cacheKey;

tests/Core/CoreAbstractAPITest.php

@@ -92,7 +92,7 @@ public function testParseJSON()
         $this->expectException(\EasyWeChat\Core\Exceptions\HttpException::class);
         $this->expectExceptionMessage('Unknown');
         $this->expectExceptionCode(24000);
-        
+
         $collection = $api->parseJSON('get', ['foo', ['bar']]);
         $this->fail();
     }

tests/OpenPlatform/AuthorizationHandlerTest.php

@@ -5,14 +5,12 @@
  *
  * @author lixiao <[email protected]>
  */
-
 use EasyWeChat\OpenPlatform\EventHandlers\Authorized;
 use EasyWeChat\OpenPlatform\EventHandlers\Unauthorized;
 use EasyWeChat\Support\Collection;
 
 class AuthorizationHandlerTest extends AuthorizationTest
 {
-
     public function testAuthorized()
     {
         $appId = 'appid@123';
@@ -25,11 +23,11 @@ public function testAuthorized()
         );
 
         $message = [
-            'AppId'                        => 'open-platform-app-id',
-            'CreateTIme'                   => '1413192760',
-            'InfoType'                     => 'authorized',
-            'AuthorizerAppid'              => 'authorizer-app-id',
-            'AuthorizationCode'            => 'auth-code',
+            'AppId' => 'open-platform-app-id',
+            'CreateTIme' => '1413192760',
+            'InfoType' => 'authorized',
+            'AuthorizerAppid' => 'authorizer-app-id',
+            'AuthorizationCode' => 'auth-code',
             'AuthorizationCodeExpiredTime' => '600',
         ];
         $authorized = new Authorized($authorization);
@@ -58,21 +56,21 @@ public function testUnauthorized()
 
         // Authorized => saves the tokens.
         $message = [
-            'AppId'                        => 'open-platform-app-id',
-            'CreateTIme'                   => '1413192760',
-            'InfoType'                     => 'authorized',
-            'AuthorizerAppid'              => 'authorizer-app-id',
-            'AuthorizationCode'            => 'auth-code',
+            'AppId' => 'open-platform-app-id',
+            'CreateTIme' => '1413192760',
+            'InfoType' => 'authorized',
+            'AuthorizerAppid' => 'authorizer-app-id',
+            'AuthorizationCode' => 'auth-code',
             'AuthorizationCodeExpiredTime' => '600',
         ];
         $authorized = new Authorized($authorization);
         $authorized->handle(new Collection($message));
 
         // Unauthorized => removes the tokens.
         $message = [
-            'AppId'           => 'open-platform-app-id',
-            'CreateTIme'      => '1413192760',
-            'InfoType'        => 'authorized',
+            'AppId' => 'open-platform-app-id',
+            'CreateTIme' => '1413192760',
+            'InfoType' => 'authorized',
             'AuthorizerAppid' => 'authorizer-app-id',
         ];
         $authorized = new Unauthorized($authorization);
@@ -82,4 +80,4 @@ public function testUnauthorized()
         $this->setExpectedException(\EasyWeChat\Core\Exception::class);
         $this->assertNull($authorization->getAuthorizerRefreshToken());
     }
-}
+}