johnvandeweghe / LunixREST

src/Server/AccessControl/KeyRepository/KeyMeta.php

Braces +1 added lines, -2 removed lines

@@ -15,8 +15,7 @@
      * KeyMeta constructor.
      * @param string $key
      */
-    public function __construct(string $key)
-    {
+    public function __construct(string $key) {
         $this->key = $key;
     }
 

src/Server/AccessControl/KeyRepository/Exceptions/UnableToFindKeyException.php

Braces +1 added lines, -2 removed lines

@@ -6,7 +6,6 @@
  * Class UnableToFindKeyException
  * @package LunixREST\Server\AccessControl\KeyRepository\Exceptions
  */
-class UnableToFindKeyException extends \Exception
-{
+class UnableToFindKeyException extends \Exception {
 
 }

src/Server/AccessControl/KeyRepository/ConfigurationKeyRepository.php

Spacing +2 added lines, -2 removed lines

@@ -18,9 +18,9 @@
     public function __construct(Configuration $config, string $namespace, string $configKey = 'keys')
     {
         $keys = [];
-        if($config->has($configKey, $namespace)){
+        if ($config->has($configKey, $namespace)) {
             $configKeys = $config->get($configKey, $namespace);
-            if(is_array($configKeys)) {
+            if (is_array($configKeys)) {
                 $keys = $configKeys;
             }
         }

Braces +3 added lines, -5 removed lines

@@ -8,17 +8,15 @@
  * Class ConfigurationKeyRepository
  * @package LunixREST\Server\AccessControl\KeyRepository
  */
-class ConfigurationKeyRepository extends ArrayKeyRepository
-{
+class ConfigurationKeyRepository extends ArrayKeyRepository {
     /**
      * @param Configuration $config a config that has a list of valid keys in the stored $configKey
      * @param $namespace
      * @param string $configKey key to use when accessing the list of valid keys from the $config
      */
-    public function __construct(Configuration $config, string $namespace, string $configKey = 'keys')
-    {
+    public function __construct(Configuration $config, string $namespace, string $configKey = 'keys') {
         $keys = [];
-        if($config->has($configKey, $namespace)){
+        if($config->has($configKey, $namespace)) {
             $configKeys = $config->get($configKey, $namespace);
             if(is_array($configKeys)) {
                 $keys = $configKeys;

src/Server/AccessControl/KeyRepository/ArrayKeyRepository.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
      */
     public function findKey(string $key): KeyMeta
     {
-        if($this->hasKey($key)) {
+        if ($this->hasKey($key)) {
             return new KeyMeta($key);
         } else {
             throw new UnableToFindKeyException("Unable to find API key in array.");

Braces +2 added lines, -4 removed lines

@@ -8,8 +8,7 @@ 
  * Class ArrayKeyRepository
  * @package LunixREST\Server\AccessControl\KeyRepository
  */
-class ArrayKeyRepository implements KeyRepository
-{
+class ArrayKeyRepository implements KeyRepository {
     /**
      * @var
      */
@@ -19,8 +18,7 @@ 
      * ArrayKeyRepository constructor.
      * @param $arrayOfKeys
      */
-    public function __construct($arrayOfKeys)
-    {
+    public function __construct($arrayOfKeys) {
         $this->arrayOfKeys = $arrayOfKeys;
     }
 

src/Server/AccessControl/KeyRepositoryAccessControl.php

Braces +2 added lines, -4 removed lines

@@ -8,8 +8,7 @@ 
  * Class KeyRepositoryAccessControl
  * @package LunixREST\Server\AccessControl
  */
-abstract class KeyRepositoryAccessControl implements AccessControl
-{
+abstract class KeyRepositoryAccessControl implements AccessControl {
     /**
      * @var KeyRepository
      */
@@ -19,8 +18,7 @@ 
      * KeyRepositoryAccessControl constructor.
      * @param KeyRepository $keyRepository
      */
-    public function __construct(KeyRepository $keyRepository)
-    {
+    public function __construct(KeyRepository $keyRepository) {
         $this->keyRepository = $keyRepository;
     }
 

src/Server/GenericRouterGenericServer.php

Braces +1 added lines, -2 removed lines

@@ -28,8 +28,7 @@
  * Class GenericRouterGenericServer
  * @package LunixREST\Server
  */
-class GenericRouterGenericServer extends GenericServer
-{
+class GenericRouterGenericServer extends GenericServer {
     /**
      * GenericRouterGenericServer constructor.
      * @param AccessControl $accessControl

src/GenericRouterGenericServerHTTPServer.php

Spacing +4 added lines, -4 removed lines

@@ -28,11 +28,11 @@ 
 
     protected function handleServerException(UnableToHandleRequestException $exception, ResponseInterface $response): ResponseInterface
     {
-        if($exception instanceof InvalidRequestException) {
+        if ($exception instanceof InvalidRequestException) {
             $this->logCaughtThrowableResultingInHTTPCode(400, $exception, LogLevel::INFO);
             return $response->withStatus(400, "Bad Request");
 
-        } elseif(
+        } elseif (
             $exception instanceof ElementNotFoundException ||
             $exception instanceof UnsupportedMethodException ||
             $exception instanceof UnableToCreateEndpointException
@@ -40,11 +40,11 @@ 
             $this->logCaughtThrowableResultingInHTTPCode(404, $exception, LogLevel::INFO);
             return $response->withStatus(404, "Not Found");
 
-        } elseif($exception instanceof ElementConflictException) {
+        } elseif ($exception instanceof ElementConflictException) {
             $this->logCaughtThrowableResultingInHTTPCode(409, $exception, LogLevel::NOTICE);
             return $response->withStatus(409, "Conflict");
 
-        } elseif(
+        } elseif (
             $exception instanceof MethodNotFoundException ||
             $exception instanceof EndpointExecutionException
         ) {

Braces +2 added lines, -4 removed lines

@@ -19,10 +19,8 @@
  * Class GenericRouterGenericServerHTTPServer
  * @package LunixREST
  */
-class GenericRouterGenericServerHTTPServer extends GenericServerHTTPServer
-{
-    public function __construct(GenericRouterGenericServer $server, RequestFactory $requestFactory, LoggerInterface $logger)
-    {
+class GenericRouterGenericServerHTTPServer extends GenericServerHTTPServer {
+    public function __construct(GenericRouterGenericServer $server, RequestFactory $requestFactory, LoggerInterface $logger) {
         parent::__construct($server, $requestFactory, $logger);
     }
 

src/GenericServerHTTPServer.php

Spacing +4 added lines, -4 removed lines

@@ -39,19 +39,19 @@
      */
     protected function handleServerException(UnableToHandleRequestException $exception, ResponseInterface $response): ResponseInterface
     {
-        if($exception instanceof InvalidAPIKeyException || $exception instanceof AccessDeniedException) {
+        if ($exception instanceof InvalidAPIKeyException || $exception instanceof AccessDeniedException) {
             $this->logCaughtThrowableResultingInHTTPCode(403, $exception, LogLevel::NOTICE);
             return $response->withStatus(403, "Access Denied");
 
-        } elseif($exception instanceof ThrottleLimitExceededException) {
+        } elseif ($exception instanceof ThrottleLimitExceededException) {
             $this->logCaughtThrowableResultingInHTTPCode(429, $exception, LogLevel::WARNING);
             return $response->withStatus(429, "Too Many Requests");
 
-        } elseif($exception instanceof NotAcceptableResponseTypeException) {
+        } elseif ($exception instanceof NotAcceptableResponseTypeException) {
             $this->logCaughtThrowableResultingInHTTPCode(406, $exception, LogLevel::INFO);
             return $response->withStatus(406, "Not Acceptable");
 
-        } elseif(
+        } elseif (
             $exception instanceof UnableToCreateAPIResponseException ||
             $exception instanceof UnableToRouteRequestException
         ) {

Braces +2 added lines, -4 removed lines

@@ -19,16 +19,14 @@
  * Class GenericServerHTTPServer
  * @package LunixREST
  */
-class GenericServerHTTPServer extends HTTPServer
-{
+class GenericServerHTTPServer extends HTTPServer {
     /**
      * GenericServerHTTPServer constructor.
      * @param GenericServer $server
      * @param RequestFactory $requestFactory
      * @param LoggerInterface $logger
      */
-    public function __construct(GenericServer $server, RequestFactory $requestFactory, LoggerInterface $logger)
-    {
+    public function __construct(GenericServer $server, RequestFactory $requestFactory, LoggerInterface $logger) {
         parent::__construct($server, $requestFactory, $logger);
     }
 

src/Server/Router/Endpoint/ResourceEndpoint/ArrayResourceParametersFactory.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
      */
     private static function checkRequestDataIsArray($requestData): void
     {
-        if(!is_array($requestData)) {
+        if (!is_array($requestData)) {
             throw new UnableToCreateResourceParametersException("Unable to read request as array or null");
         }
     }

Braces +1 added lines, -2 removed lines

@@ -9,8 +9,7 @@
  * Class ArrayResourceParametersFactory
  * @package LunixREST\Server\Router\Endpoint\ResourceEndpoint
  */
-abstract class ArrayResourceParametersFactory implements ResourceParametersFactory
-{
+abstract class ArrayResourceParametersFactory implements ResourceParametersFactory {
     /**
      * @param APIRequest $request
      * @return ResourceParameters