johnvandeweghe / LunixREST

src/Response/DefaultResponseFactory.php

Spacing +2 added lines, -2 removed lines

@@ -11,11 +11,11 @@
      * @throws NotAcceptableResponseTypeException
      */
     public function getResponse(ResponseData $data, array $acceptedMIMETypes): Response {
-        if(empty($acceptedMIMETypes)){
+        if (empty($acceptedMIMETypes)) {
             $acceptedMIMETypes = $this->getSupportedMIMETypes();
         }
 
-        foreach($acceptedMIMETypes as $acceptedMIMEType) {
+        foreach ($acceptedMIMETypes as $acceptedMIMEType) {
             switch (strtolower($acceptedMIMEType)) {
                 case "application/json":
                     return new JSONResponse($data);

src/Server/Server.php

Spacing +6 added lines, -6 removed lines

@@ -39,7 +39,7 @@ 
      * @param ResponseFactory $responseFactory
      * @param EndpointFactory $endpointFactory
      */
-    public function __construct(AccessControl $accessControl, Throttle $throttle, ResponseFactory $responseFactory, EndpointFactory $endpointFactory){
+    public function __construct(AccessControl $accessControl, Throttle $throttle, ResponseFactory $responseFactory, EndpointFactory $endpointFactory) {
         $this->accessControl = $accessControl;
         $this->throttle = $throttle;
         $this->responseFactory = $responseFactory;
@@ -60,11 +60,11 @@ 
 
         $this->validateExtension($request);
 
-        if($this->throttle->shouldThrottle($request)) {
+        if ($this->throttle->shouldThrottle($request)) {
             throw new ThrottleLimitExceededException('Request limit exceeded');
         }
 
-        if(!$this->accessControl->validateAccess($request)) {
+        if (!$this->accessControl->validateAccess($request)) {
             throw new AccessDeniedException("API key does not have the required permissions to access requested resource");
         }
 
@@ -79,8 +79,8 @@ 
      * @param Request $request
      * @throws InvalidAPIKeyException
      */
-    protected function validateKey(Request $request){
-        if(!$this->accessControl->validateKey($request->getApiKey())){
+    protected function validateKey(Request $request) {
+        if (!$this->accessControl->validateKey($request->getApiKey())) {
             throw new InvalidAPIKeyException('Invalid API key');
         }
     }
@@ -91,7 +91,7 @@ 
      */
     protected function validateExtension(Request $request) {
         $formats = $this->responseFactory->getSupportedMIMETypes();
-        if(empty($formats) || (
+        if (empty($formats) || (
             !empty($request->getAcceptableMIMETypes()) && empty(array_intersect($request->getAcceptableMIMETypes(), $formats))
             )) {
             throw new NotAcceptableResponseTypeException('None of the requests acceptable response types are valid');

src/Request/Request.php

Spacing +7 added lines, -7 removed lines

@@ -68,7 +68,7 @@ 
      * @param string $instance
      */
 
-    public function __construct(MIMEProvider $MIMEProvider, $method, array $headers, RequestData $body, RequestData $urlData, $ip, $version, $apiKey, $endpoint, $extension, $instance = null){
+    public function __construct(MIMEProvider $MIMEProvider, $method, array $headers, RequestData $body, RequestData $urlData, $ip, $version, $apiKey, $endpoint, $extension, $instance = null) {
         $this->MIMEProvider = $MIMEProvider;
         $this->method = strtolower($method);
         $this->headers = $headers;
@@ -93,7 +93,7 @@ 
      * @return string
      */
     public function getMethod() {
-        return $this->method . ($this->instance ? '' : 'All');
+        return $this->method.($this->instance ? '' : 'All');
     }
 
     /**
@@ -155,18 +155,18 @@ 
     //TODO: Unit test
     public function getAcceptableMIMETypes(): array {
         $acceptedMIMETypes = [];
-        if($this->extension) {
+        if ($this->extension) {
             //extension to mime type conversion
             $acceptedMIMETypes[] = $this->MIMEProvider->getByFileExtension($this->extension);
         } else {
             //TODO: follow RFC2616 order
             $headerAccepts = [];
-            foreach($this->headers as $key => $value) {
-                if(strtolower($key) == 'http_accept'){
+            foreach ($this->headers as $key => $value) {
+                if (strtolower($key) == 'http_accept') {
                     $values = explode(',', $value);
-                    foreach($values as $acceptedType) {
+                    foreach ($values as $acceptedType) {
                         $typeParts = explode(';', $acceptedType);
-                        if(count($typeParts) > 0 ){
+                        if (count($typeParts) > 0) {
                             $headerAccepts[] = trim($typeParts);
                         }
                     }

src/Request/MIMEFileProvider.php

Spacing +5 added lines, -5 removed lines

@@ -13,21 +13,21 @@
 
     //Borrowed from http://stackoverflow.com/a/1147952
     public function getAll(): array {
-        if($this->cache){
+        if ($this->cache) {
             return $this->cache;
         }
 
         # Returns the system MIME type mapping of extensions to MIME types, as defined in /etc/mime.types.
         $out = array();
-        while(($line = fgets($this->file)) !== false) {
+        while (($line = fgets($this->file)) !== false) {
             $line = trim(preg_replace('/#.*/', '', $line));
-            if(!$line)
+            if (!$line)
                 continue;
             $parts = preg_split('/\s+/', $line);
-            if(count($parts) == 1)
+            if (count($parts) == 1)
                 continue;
             $type = array_shift($parts);
-            foreach($parts as $part)
+            foreach ($parts as $part)
                 $out[$part] = $type;
         }
 

Braces +9 added lines, -6 removed lines

@@ -21,14 +21,17 @@
         $out = array();
         while(($line = fgets($this->file)) !== false) {
             $line = trim(preg_replace('/#.*/', '', $line));
-            if(!$line)
-                continue;
+            if(!$line) {
+                            continue;
+            }
             $parts = preg_split('/\s+/', $line);
-            if(count($parts) == 1)
-                continue;
+            if(count($parts) == 1) {
+                            continue;
+            }
             $type = array_shift($parts);
-            foreach($parts as $part)
-                $out[$part] = $type;
+            foreach($parts as $part) {
+                            $out[$part] = $type;
+            }
         }
 
         $this->cache = $out;