nathanmac / Parser

src/Parser.php

Doc Comments +2 added lines, -2 removed lines

@@ -95,7 +95,7 @@ 
     /**
      * Determine if the payload contains a non-empty value for a given key.
      *
-     * @param  string|array $keys
+     * @param  string|null $keys
      *
      * @return bool
      */
@@ -356,7 +356,7 @@ 
     /**
      * Return a value from the array identified from the key.
      *
-     * @param $key
+     * @param string|null $key
      * @param $data
      * @return mixed
      */

Unused Use Statements +1 added lines, -1 removed lines

@@ -5,11 +5,11 @@
 use Nathanmac\Utilities\Parser\Formats\BSON;
 use Nathanmac\Utilities\Parser\Formats\FormatInterface;
 use Nathanmac\Utilities\Parser\Formats\JSON;
+use Nathanmac\Utilities\Parser\Formats\MSGPack;
 use Nathanmac\Utilities\Parser\Formats\QueryStr;
 use Nathanmac\Utilities\Parser\Formats\Serialize;
 use Nathanmac\Utilities\Parser\Formats\XML;
 use Nathanmac\Utilities\Parser\Formats\YAML;
-use Nathanmac\Utilities\Parser\Formats\MSGPack;
 
 /**
  * Parser Library, designed to parse payload data from various formats to php array.

Indentation +19 added lines, -19 removed lines

@@ -29,28 +29,28 @@
      * @var array Supported Formats
      */
     private $supported_formats = array (
-      // XML
-	    'application/xml' => 'xml',
-	    'text/xml' => 'xml',
-      // JSON
-	    'application/json' => 'json',
-		'application/x-javascript' => 'json',
-		'text/javascript' => 'json',
-		'text/x-javascript' => 'json',
-		'text/x-json' => 'json',
-      // BSON
+        // XML
+        'application/xml' => 'xml',
+        'text/xml' => 'xml',
+        // JSON
+        'application/json' => 'json',
+        'application/x-javascript' => 'json',
+        'text/javascript' => 'json',
+        'text/x-javascript' => 'json',
+        'text/x-json' => 'json',
+        // BSON
         'application/bson' => 'bson',
-      // YAML
-	    'text/yaml' => 'yaml',
-		'text/x-yaml' => 'yaml',
-		'application/yaml' => 'yaml',
-		'application/x-yaml' => 'yaml',
-      // MSGPACK
+        // YAML
+        'text/yaml' => 'yaml',
+        'text/x-yaml' => 'yaml',
+        'application/yaml' => 'yaml',
+        'application/x-yaml' => 'yaml',
+        // MSGPACK
         'application/msgpack' => 'msgpack',
         'application/x-msgpack' => 'msgpack',
-      // MISC
-		'application/vnd.php.serialized' => 'serialize',
-	    'application/x-www-form-urlencoded' => 'querystr'
+        // MISC
+        'application/vnd.php.serialized' => 'serialize',
+        'application/x-www-form-urlencoded' => 'querystr'
     );
 
     /* ------------ Access Methods/Helpers ------------ */

Spacing +8 added lines, -8 removed lines

@@ -28,7 +28,7 @@ 
     /**
      * @var array Supported Formats
      */
-    private $supported_formats = array (
+    private $supported_formats = array(
       // XML
 	    'application/xml' => 'xml',
 	    'text/xml' => 'xml',
@@ -190,7 +190,7 @@ 
      */
     public function getFormat($format = '')
     {
-        if (! empty($format)) {
+        if (!empty($format)) {
             return $this->processContentType($format);
         }
 
@@ -382,13 +382,13 @@ 
                 } else {
                     $item = array_shift($data);
                 }
-                $data =& $item;
+                $data = & $item;
 
             } else {
                 if (!isset($data[$key]) || !is_array($data[$key]))
                     return false;
 
-                $data =& $data[$key];
+                $data = & $data[$key];
             }
         }
 
@@ -439,7 +439,7 @@ 
                 } else {
                     $item = array_shift($data);
                 }
-                $data =& $item;
+                $data = & $item;
 
             } else {
                 if (!isset($data[$key]))
@@ -451,7 +451,7 @@ 
                 if ($data[$key] === '')
                     return false;
 
-                $data =& $data[$key];
+                $data = & $data[$key];
             }
         }
         return true;
@@ -489,12 +489,12 @@ 
         {
             $key = array_shift($keys);
 
-            if ( ! isset($array[$key]) || ! is_array($array[$key]))
+            if (!isset($array[$key]) || !is_array($array[$key]))
             {
                 return;
             }
 
-            $array =& $array[$key];
+            $array = & $array[$key];
         }
 
         unset($array[array_shift($keys)]);

Braces +30 added lines, -18 removed lines

@@ -107,8 +107,9 @@ 
 
         foreach ($keys as $value)
         {
-            if ($this->hasValueAtKey($value, $results) === false)
-                return false;
+            if ($this->hasValueAtKey($value, $results) === false) {
+                            return false;
+            }
         }
         return true;
     }
@@ -197,13 +198,17 @@ 
         if (isset($_SERVER['CONTENT_TYPE']))
         {
             $type = $this->processContentType($_SERVER['CONTENT_TYPE']);
-            if ($type !== false) return $type;
+            if ($type !== false) {
+                return $type;
+            }
         }
 
         if (isset($_SERVER['HTTP_CONTENT_TYPE']))
         {
             $type = $this->processContentType($_SERVER['HTTP_CONTENT_TYPE']);
-            if ($type !== false) return $type;
+            if ($type !== false) {
+                return $type;
+            }
         }
 
         return 'json';
@@ -373,8 +378,9 @@ 
                 // Shift the first item of the array
                 if (preg_match('/^:(index|item)\[\d+\]$/', $key)) {
                     for ($x = substr($key, 7, -1); $x >= 0; $x--) {
-                        if (empty($data))
-                            return false;
+                        if (empty($data)) {
+                                                    return false;
+                        }
                         $item = array_shift($data);
                     }
                 } else if ($key == ':last') {
@@ -385,8 +391,9 @@ 
                 $data =& $item;
 
             } else {
-                if (!isset($data[$key]) || !is_array($data[$key]))
-                    return false;
+                if (!isset($data[$key]) || !is_array($data[$key])) {
+                                    return false;
+                }
 
                 $data =& $data[$key];
             }
@@ -397,8 +404,9 @@ 
         if (preg_match($this->wildcards, $key)) {
             if (preg_match('/^:(index|item)\[\d+\]$/', $key)) {
                 for ($x = substr($key, 7, -1); $x >= 0; $x--) {
-                    if (empty($data))
-                        return false;
+                    if (empty($data)) {
+                                            return false;
+                    }
                     $item = array_shift($data);
                 }
                 return $item;
@@ -430,8 +438,9 @@ 
                 // Shift the first item of the array
                 if (preg_match('/^:(index|item)\[\d+\]$/', $key)) {
                     for ($x = substr($key, 7, -1); $x >= 0; $x--) {
-                        if (empty($data))
-                            return false;
+                        if (empty($data)) {
+                                                    return false;
+                        }
                         $item = array_shift($data);
                     }
                 } else if ($key == ':last') {
@@ -442,14 +451,17 @@ 
                 $data =& $item;
 
             } else {
-                if (!isset($data[$key]))
-                    return false;
+                if (!isset($data[$key])) {
+                                    return false;
+                }
 
-                if (is_bool($data[$key]))
-                    return true;
+                if (is_bool($data[$key])) {
+                                    return true;
+                }
 
-                if ($data[$key] === '')
-                    return false;
+                if ($data[$key] === '') {
+                                    return false;
+                }
 
                 $data =& $data[$key];
             }

src/ParserServiceProvider.php

Doc Comments +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     /**
      * Get the services provided by the provider.
      *
-     * @return array
+     * @return string[]
      */
     public function provides()
     {

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
      */
     public function register()
     {
-        $this->app->bind('Parser', function ($app) {
+        $this->app->bind('Parser', function($app) {
             return new Parser;
         });
     }

src/Formats/BSON.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
         if (function_exists('bson_decode')) {
             if ($payload) {
                 $bson = bson_decode(trim($payload));
-                if (! $bson)
+                if (!$bson)
                     throw new ParserException('Failed To Parse BSON');
                 return $bson;
             }

Braces +3 added lines, -2 removed lines

@@ -27,8 +27,9 @@
         if (function_exists('bson_decode')) {
             if ($payload) {
                 $bson = bson_decode(trim($payload));
-                if (! $bson)
-                    throw new ParserException('Failed To Parse BSON');
+                if (! $bson) {
+                                    throw new ParserException('Failed To Parse BSON');
+                }
                 return $bson;
             }
             return array();

src/Formats/MSGPack.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
         if (function_exists('msgpack_unpack')) {
             if ($payload) {
                 $msg = msgpack_unpack(trim($payload));
-                if (! $msg)
+                if (!$msg)
                     throw new ParserException('Failed To Parse MSGPack');
                 return $msg;
             }

Braces +3 added lines, -2 removed lines

@@ -27,8 +27,9 @@
         if (function_exists('msgpack_unpack')) {
             if ($payload) {
                 $msg = msgpack_unpack(trim($payload));
-                if (! $msg)
-                    throw new ParserException('Failed To Parse MSGPack');
+                if (! $msg) {
+                                    throw new ParserException('Failed To Parse MSGPack');
+                }
                 return $msg;
             }
             return array();

src/Formats/JSON.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
     {
         if ($payload) {
             $json = json_decode(trim($payload), true);
-            if (! $json)
+            if (!$json)
                 throw new ParserException('Failed To Parse JSON');
             return $json;
         }

Braces +3 added lines, -2 removed lines

@@ -26,8 +26,9 @@
     {
         if ($payload) {
             $json = json_decode(trim($payload), true);
-            if (! $json)
-                throw new ParserException('Failed To Parse JSON');
+            if (! $json) {
+                            throw new ParserException('Failed To Parse JSON');
+            }
             return $json;
         }
         

src/Formats/XML.php

Spacing +3 added lines, -3 removed lines

@@ -30,9 +30,9 @@
 
                 // Fix for empty values in XML
                 $json = json_encode((array) $xml);
-                $json = str_replace(':{}',':null', $json);
-                $json = str_replace(':[]',':null', $json);
-                return json_decode($json, 1);   // Work around to accept xml input
+                $json = str_replace(':{}', ':null', $json);
+                $json = str_replace(':[]', ':null', $json);
+                return json_decode($json, 1); // Work around to accept xml input
             } catch (\Exception $ex) {
                 throw new ParserException('Failed To Parse XML');
             }