johnvandeweghe / LunixREST

src/AccessControl/AllAccessListAccessControl.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
     /**
      * @param array $keys
      */
-    public function __construct(Array $keys)
+    public function __construct(array $keys)
     {
         $this->keys = $keys;
     }

Braces +4 added lines, -8 removed lines

@@ -9,8 +9,7 @@ 
  * Class AllAccessListAccessControl
  * @package LunixRESTBasics\AccessControl
  */
-class AllAccessListAccessControl implements AccessControl
-{
+class AllAccessListAccessControl implements AccessControl {
     /**
      * @var array
      */
@@ -19,8 +18,7 @@ 
     /**
      * @param array $keys
      */
-    public function __construct(Array $keys)
-    {
+    public function __construct(Array $keys) {
         $this->keys = $keys;
     }
 
@@ -28,8 +26,7 @@ 
      * @param \LunixREST\APIRequest\APIRequest $request
      * @return bool true if key is valid
      */
-    public function validateAccess(APIRequest $request)
-    {
+    public function validateAccess(APIRequest $request) {
         return $this->validateKey($request->getApiKey());
     }
 
@@ -37,8 +34,7 @@ 
      * @param $apiKey
      * @return bool true if key is in the array passed to this object in it's construction
      */
-    public function validateKey($apiKey)
-    {
+    public function validateKey($apiKey) {
         return in_array($apiKey, $this->keys);
     }
 }

src/AccessControl/AccessControl.php

Braces +1 added lines, -2 removed lines

@@ -7,8 +7,7 @@
  * Interface AccessControl
  * @package LunixREST\AccessControl
  */
-interface AccessControl
-{
+interface AccessControl {
     /**
      * Validates if a given request should be able to access what it's trying to
      * @param $request

src/AccessControl/PublicAccessControl.php

Braces +3 added lines, -6 removed lines

@@ -8,14 +8,12 @@ 
  * Class PublicAccessControl
  * @package LunixREST\AccessControl
  */
-class PublicAccessControl implements AccessControl
-{
+class PublicAccessControl implements AccessControl {
     /**
      * @param \LunixREST\APIRequest\APIRequest $request
      * @return bool
      */
-    public function validateAccess(APIRequest $request)
-    {
+    public function validateAccess(APIRequest $request) {
         return true;
     }
 
@@ -23,8 +21,7 @@ 
      * @param $apiKey
      * @return bool
      */
-    public function validateKey($apiKey)
-    {
+    public function validateKey($apiKey) {
         return true;
     }
 }

src/AccessControl/OneKeyAccessControl.php

Braces +4 added lines, -8 removed lines

@@ -8,8 +8,7 @@ 
  * Class OneKeyAccessControl
  * @package LunixREST\AccessControl
  */
-class OneKeyAccessControl implements AccessControl
-{
+class OneKeyAccessControl implements AccessControl {
     /**
      * @var string
      */
@@ -18,8 +17,7 @@ 
     /**
      * @param string $key
      */
-    public function __construct($key)
-    {
+    public function __construct($key) {
         $this->key = $key;
     }
 
@@ -27,8 +25,7 @@ 
      * @param \LunixREST\APIRequest\APIRequest $request
      * @return bool true if key is valid
      */
-    public function validateAccess(APIRequest $request)
-    {
+    public function validateAccess(APIRequest $request) {
         return $this->validateKey($request->getApiKey());
     }
 
@@ -36,8 +33,7 @@ 
      * @param $apiKey
      * @return bool true if key is the key specified in the constructor
      */
-    public function validateKey($apiKey)
-    {
+    public function validateKey($apiKey) {
         return $apiKey === $this->key;
     }
 }

src/APIRequest/APIRequest.php

Spacing +3 added lines, -3 removed lines

@@ -55,10 +55,10 @@
     public function __construct(
         string $method,
         string $endpoint,
-        ?string $element,
+        ? string $element,
         array $acceptableMIMETypes,
-        ?string $version,
-        ?string $apiKey,
+        ? string $version,
+        ? string $apiKey,
         array $queryData,
         $data
     ) {

Braces +5 added lines, -10 removed lines

@@ -6,8 +6,7 @@ 
  * Class APIRequest
  * @package LunixREST\Request
  */
-class APIRequest
-{
+class APIRequest {
     /**
      * @var string
      */
@@ -91,8 +90,7 @@ 
     /**
      * @return null|string
      */
-    public function getElement()
-    {
+    public function getElement() {
         return $this->element;
     }
 
@@ -107,16 +105,14 @@ 
     /**
      * @return null|string
      */
-    public function getVersion()
-    {
+    public function getVersion() {
         return $this->version;
     }
 
     /**
      * @return null|string
      */
-    public function getApiKey()
-    {
+    public function getApiKey() {
         return $this->apiKey;
     }
 
@@ -131,8 +127,7 @@ 
     /**
      * @return array|null|object
      */
-    public function getData()
-    {
+    public function getData() {
         return $this->data;
     }
 }

src/APIRequest/URLParser/ParsedURL.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
      * @param array $acceptableMIMETypes
      * @param null|string $queryString
      */
-    public function __construct(string $endpoint, ?string $element, ?string $version, ?string $apiKey, array $acceptableMIMETypes, ?string $queryString)
+    public function __construct(string $endpoint, ? string $element, ? string $version, ? string $apiKey, array $acceptableMIMETypes, ? string $queryString)
     {
         $this->endpoint = $endpoint;
         $this->element = $element;

Braces +6 added lines, -12 removed lines

@@ -6,8 +6,7 @@ 
  * Class ParsedURL
  * @package LunixREST\APIRequest\URLParser
  */
-class ParsedURL
-{
+class ParsedURL {
     /**
      * @var string
      */
@@ -42,8 +41,7 @@ 
      * @param array $acceptableMIMETypes
      * @param null|string $queryString
      */
-    public function __construct(string $endpoint, ?string $element, ?string $version, ?string $apiKey, array $acceptableMIMETypes, ?string $queryString)
-    {
+    public function __construct(string $endpoint, ?string $element, ?string $version, ?string $apiKey, array $acceptableMIMETypes, ?string $queryString) {
         $this->endpoint = $endpoint;
         $this->element = $element;
         $this->version = $version;
@@ -63,24 +61,21 @@ 
     /**
      * @return null|string
      */
-    public function getElement()
-    {
+    public function getElement() {
         return $this->element;
     }
 
     /**
      * @return null|string
      */
-    public function getVersion()
-    {
+    public function getVersion() {
         return $this->version;
     }
 
     /**
      * @return null|string
      */
-    public function getApiKey()
-    {
+    public function getApiKey() {
         return $this->apiKey;
     }
 
@@ -95,8 +90,7 @@ 
     /**
      * @return null|string
      */
-    public function getQueryString()
-    {
+    public function getQueryString() {
         return $this->queryString;
     }
 }

src/Server/Server.php

Braces +1 added lines, -2 removed lines

@@ -15,8 +15,7 @@
  * Interface Server
  * @package LunixREST\Server
  */
-interface Server
-{
+interface Server {
     /**
      * @param APIRequest $request
      * @return APIResponse

src/APIRequest/RequestFactory/GenericRequestFactory.php

Braces +1 added lines, -2 removed lines

@@ -28,8 +28,7 @@
      * @param URLParser $URLParser
      * @param HeaderParser $headerParser
      */
-    public function __construct(URLParser $URLParser, HeaderParser $headerParser)
-    {
+    public function __construct(URLParser $URLParser, HeaderParser $headerParser) {
         $this->URLParser = $URLParser;
         $this->headerParser = $headerParser;
     }

Spacing +2 added lines, -2 removed lines

@@ -48,12 +48,12 @@
         $parsedHeaders = $this->headerParser->parse($serverRequest->getHeaders());
 
         $urlQueryData = [];
-        if($urlQueryString = $parsedURL->getQueryString()) {
+        if ($urlQueryString = $parsedURL->getQueryString()) {
             parse_str($urlQueryString, $urlQueryData);
         }
 
         $apiKey = $parsedURL->getAPIKey();
-        if($apiKey === null) {
+        if ($apiKey === null) {
             $apiKey = $parsedHeaders->getAPIKey();
         }
 

src/APIRequest/HeaderParser/DefaultHeaderParser.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
     {
         foreach ($headers as $key => $values) {
             if (strtolower($key) == strtolower($this->apiKeyHeaderKey)) {
-                foreach($values as $value) {
+                foreach ($values as $value) {
                     return $value;
                 }
             }

Braces +4 added lines, -8 removed lines

@@ -6,8 +6,7 @@ 
  * Class DefaultHeaderParser
  * @package LunixREST\APIRequest\HeaderParser
  */
-class DefaultHeaderParser implements HeaderParser
-{
+class DefaultHeaderParser implements HeaderParser {
 
     /**
      * @var string
@@ -18,8 +17,7 @@ 
      * DefaultHeaderParser constructor.
      * @param string $apiKeyHeaderKey
      */
-    public function __construct($apiKeyHeaderKey = 'x-api-key')
-    {
+    public function __construct($apiKeyHeaderKey = 'x-api-key') {
         $this->apiKeyHeaderKey = $apiKeyHeaderKey;
     }
 
@@ -63,8 +61,7 @@ 
      * @param array $headers
      * @return null
      */
-    protected function findAPIKey(array $headers)
-    {
+    protected function findAPIKey(array $headers) {
         foreach ($headers as $key => $values) {
             if (strtolower($key) == strtolower($this->apiKeyHeaderKey)) {
                 foreach($values as $value) {
@@ -79,8 +76,7 @@ 
      * @param array $headers
      * @return null
      */
-    protected function getContentType(array $headers)
-    {
+    protected function getContentType(array $headers) {
         foreach ($headers as $key => $values) {
             if (strtolower($key) == 'content-type') {
                 foreach ($values as $value) {