gyselroth / micro

src/Config/ConfigInterface.php

Doc Comments +2 added lines, -3 removed lines

@@ -27,7 +27,7 @@ 
     /**
      * Get entire simplexml
      *
-     * @return mixed
+     * @return \SimpleXMLElement
      */
     public function getRaw();
 
@@ -36,7 +36,7 @@ 
      * Get from config
      *
      * @param   string $name
-     * @return  mixed
+     * @return  \SimpleXMLElement
      */
     public function __get(string $name);
 
@@ -53,7 +53,6 @@ 
     /**
      * Get native config format as config instance
      *
-     * @param   mixed $config
      * @return  Config
      */
     public function map($native=null): Config;

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Micro
@@ -56,5 +56,5 @@ 
      * @param   mixed $config
      * @return  Config
      */
-    public function map($native=null): Config;
+    public function map($native = null): Config;
 }

src/Config/Xml.php

Doc Comments +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
     /**
      * Merge xml tree's
      *
-     * @param   SimpleXMLElement $simmplexml_to
+     * @param   SimpleXMLElement $simplexml_to
      * @param   SimpleXMLElement $simplexml_from
      * @param   bool $replace
      * @return  bool

Indentation +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
                 if (!isset($simplexml_to->{$simplexml_child->getName()})) {
                     $simplexml_to->addChild($simplexml_child->getName(), htmlspecialchars((string)$simplexml_child));
                 } elseif($replace === true && count($simplexml_to->{$simplexml_child->getName()}->children()) === 0) {
-                   $simplexml_to->{$simplexml_child->getName()} = htmlspecialchars((string)$simplexml_child);
+                    $simplexml_to->{$simplexml_child->getName()} = htmlspecialchars((string)$simplexml_child);
                 }
             } else {
                 $this->appendSimplexml($simplexml_to->{$simplexml_child->getName()}, $simplexml_child, $replace);

Spacing +17 added lines, -17 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Micro
@@ -30,7 +30,7 @@ 
      * @param   string $env
      * @return  void
      */
-    public function __construct(string $config, string $env='production')
+    public function __construct(string $config, string $env = 'production')
     {
         $config = simplexml_load_file($config);
         if ($this->store === false) {
@@ -49,7 +49,7 @@ 
             while (list(, $node) = each($result)) {
                 $path = (string)$node->attributes()->inherits;
 
-                if($path === '') {
+                if ($path === '') {
                     continue;
                 }
 
@@ -86,10 +86,10 @@ 
      * @param   bool $replace
      * @return  bool
      */
-    protected function appendSimplexml(SimpleXMLElement &$simplexml_to, SimpleXMLElement &$simplexml_from, bool $replace=true): bool
+    protected function appendSimplexml(SimpleXMLElement&$simplexml_to, SimpleXMLElement&$simplexml_from, bool $replace = true): bool
     {
-        if(count($simplexml_from->children()) === 0) {
-            if($replace === true && count($simplexml_to->children()) === 0) {
+        if (count($simplexml_from->children()) === 0) {
+            if ($replace === true && count($simplexml_to->children()) === 0) {
                 $simplexml_to[0] = htmlspecialchars((string)$simplexml_from);
             }
         }
@@ -98,7 +98,7 @@ 
         foreach ($simplexml_from->attributes() as $attr_key => $attr_value) {
             if (!isset($attrs[$attr_key])) {
                 $simplexml_to->addAttribute($attr_key, (string)$attr_value);
-            } elseif($replace===true) {
+            } elseif ($replace === true) {
                 $simplexml_to->attributes()->{$attr_key} = (string)$attr_value;
             }
         }
@@ -107,7 +107,7 @@ 
             if (count($simplexml_child->children()) === 0) {
                 if (!isset($simplexml_to->{$simplexml_child->getName()})) {
                     $simplexml_to->addChild($simplexml_child->getName(), htmlspecialchars((string)$simplexml_child));
-                } elseif($replace === true && count($simplexml_to->{$simplexml_child->getName()}->children()) === 0) {
+                } elseif ($replace === true && count($simplexml_to->{$simplexml_child->getName()}->children()) === 0) {
                    $simplexml_to->{$simplexml_child->getName()} = htmlspecialchars((string)$simplexml_child);
                 }
             } else {
@@ -118,7 +118,7 @@ 
             foreach ($simplexml_child->attributes() as $attr_key => $attr_value) {
                 if (!isset($attrs[$attr_key])) {
                     $simplexml_to->{$simplexml_child->getName()}->addAttribute($attr_key, (string)$attr_value);
-                } elseif($replace===true) {
+                } elseif ($replace === true) {
                     $simplexml_to->{$simplexml_child->getName()}->attributes()->{$attr_key} = (string)$attr_value;
                 }
             }
@@ -166,7 +166,7 @@ 
         while (list(, $node) = each($result)) {
             $path = (string)$node->attributes()->reference;
             
-            if($path === '') {
+            if ($path === '') {
                 continue;
             }
 
@@ -190,7 +190,7 @@ 
      * @param   SimpleXMLElement $xml
      * @return  Config
      */
-    public function map($xml=null): Config
+    public function map($xml = null): Config
     {
         if ($xml === null) {
             $xml = $this->store;
@@ -200,10 +200,10 @@ 
         foreach ($xml->getNamespaces() + array(null) as $prefix => $namespace) {
             foreach ($xml->attributes($namespace) as $key => $value) {
                 if (is_string($prefix)) {
-                    $key = $prefix . '.' . $key;
+                    $key = $prefix.'.'.$key;
                 }
 
-                if($key === 'reference') {
+                if ($key === 'reference') {
                     continue;
                 }
 
@@ -217,13 +217,13 @@ 
                 if (!isset($arr[$name])) {
                     $config[$name] = $value;
                 } else {
-                    foreach ((array) $value as $k => $v) {
+                    foreach ((array)$value as $k => $v) {
                         if (is_numeric($k)) {
                             $config[$name][] = $v;
                         } else {
                             $config[$name][$k] = array_merge(
-                                (array) $config[$name][$k],
-                                (array) $v
+                                (array)$config[$name][$k],
+                                (array)$v
                             );
                         }
                     }
@@ -232,7 +232,7 @@ 
                 $config[$name] = new Config();
             }
         }
-        if ($content = trim((string) $xml)) {
+        if ($content = trim((string)$xml)) {
             $config[] = $content;
         }
 

src/Db/Wrapper/Mysql.php

Doc Comments +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
      * Forward calls
      *
      * @param  array $method
-     * @param  array $argumnets
+     * @param  array $arguments
      * @return mixed
      */
     public function __call(string $method, array $arguments=[])

Spacing +11 added lines, -11 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Micro
@@ -96,7 +96,7 @@ 
      * @param   Iterable $config
      * @param   Logger   $logger
      */
-    public function __construct(?Iterable $config, Logger $logger)
+    public function __construct(? Iterable $config, Logger $logger)
     {
         $this->setOptions($config);
         $this->logger = $logger;
@@ -114,11 +114,11 @@ 
         $this->connection->set_charset($this->charset);
 
         if (!$this->connection->connect_errno) {
-            $this->logger->info('connection to mysql server [' . $this->host . '] was succesful', [
+            $this->logger->info('connection to mysql server ['.$this->host.'] was succesful', [
                 'category' => get_class($this),
             ]);
         } else {
-            throw new Exception('failed to connect to mysql server, error: ' .  $this->connection->connect_error.' ('.$this->connection->connect_errno.')');
+            throw new Exception('failed to connect to mysql server, error: '.$this->connection->connect_error.' ('.$this->connection->connect_errno.')');
         }
 
         return $this;
@@ -132,7 +132,7 @@ 
      * @param  array $argumnets
      * @return mixed
      */
-    public function __call(string $method, array $arguments=[])
+    public function __call(string $method, array $arguments = [])
     {
         return call_user_func_array([&$this->connection, $method], $arguments);
     }
@@ -144,7 +144,7 @@ 
      * @param  Iterable $config
      * @return Mysql
      */
-    public function setOptions(?Iterable $config = null): Mysql
+    public function setOptions(? Iterable $config = null) : Mysql
     {
         if ($config === null) {
             return $this;
@@ -207,7 +207,7 @@ 
         $link   = $this->getResource();
         $result = $link->query($query);
 
-        if($result === false) {
+        if ($result === false) {
             throw new Exception('failed to execute sql query with error '.$link->error.' ('.$link->errno.')');
         }
 
@@ -230,7 +230,7 @@ 
         $link   = $this->getResource();
         $result = $link->query($query);
 
-        if($result === false) {
+        if ($result === false) {
             throw new Exception('failed to execute sql query with error '.$link->error.' ('.$link->errno.')');
         }
 
@@ -255,19 +255,19 @@ 
         $link  = $this->getResource();
         $stmt  = $link->prepare($query);
 
-        if(!($stmt instanceof mysqli_stmt)) {
+        if (!($stmt instanceof mysqli_stmt)) {
             throw new Exception('failed to prepare sql query with error '.$link->error.' ('.$link->errno.')');
         }
 
         $types = '';
-        foreach($values as $attr => $value) {
+        foreach ($values as $attr => $value) {
             $types .= 's';
         }
 
         $stmt->bind_param($types, ...$values);
         $stmt->execute();
 
-        if($stmt->error) {
+        if ($stmt->error) {
             throw new Exception($stmt->error);
         }
         

src/Http/Response.php

Doc Comments +2 added lines, -2 removed lines

@@ -166,7 +166,7 @@ 
     /**
      * Get body
      *
-     * @return mixed
+     * @return string
      */
     public function getBody()
     {
@@ -341,7 +341,7 @@ 
     /**
      * Set the current output format.
      *
-     * @param  string $foramt a key of $outputForms
+     * @param  string $format a key of $outputForms
      * @return Response
      */
     public function setOutputFormat(string $format): Response

Indentation +7 added lines, -7 removed lines

@@ -276,13 +276,13 @@
 
 
     /**
-    * Converts mixed data to XML
-    *
-    * @param    mixed $data
-    * @param    SimpleXMLElement $xml
-    * @param    string $child_name
-    * @return   string
-    */
+     * Converts mixed data to XML
+     *
+     * @param    mixed $data
+     * @param    SimpleXMLElement $xml
+     * @param    string $child_name
+     * @return   string
+     */
     public function toXML($data, Config $xml, string $child_name): string
     {
         if (is_array($data)) {

Spacing +3 added lines, -3 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Micro
@@ -155,7 +155,7 @@ 
      * @param  bool $body_only
      * @return Response
      */
-    public function setBody($body, bool $body_only=false): Response
+    public function setBody($body, bool $body_only = false): Response
     {
         $this->body = $body;
         $this->body_only = $body_only;
@@ -183,7 +183,7 @@ 
     {
         $this->sendHeaders();
         $status = Http::STATUS_CODES[$this->code];
-        header('HTTP/1.0 ' . $this->code . ' ' . $status, true, $this->code);
+        header('HTTP/1.0 '.$this->code.' '.$status, true, $this->code);
 
         if ($this->body === null && $this->code == 204) {
             $this->terminate();

src/Http/Router.php

Doc Comments +1 added lines, -1 removed lines

@@ -271,7 +271,7 @@
      * @param   string $class
      * @param   string $method
      * @param   array $parsed_params
-     * @return  array
+     * @return  callable
      */
     protected function getParams(string $class, string $method, array $parsed_params): array
     {

Indentation +1 added lines, -1 removed lines

@@ -209,7 +209,7 @@
                     }
                 } else {
                     $this->logger->debug('requested path ['.$this->path.'] does not match route ['.$route->getPath().']', [
-                         'category' => get_class($this),
+                            'category' => get_class($this),
                     ]);
                 }
             }

Spacing +3 added lines, -3 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Micro
@@ -182,7 +182,7 @@ 
      * @param  array $constructor
      * @return bool
      */
-    public function run(array $constructor=[]): bool
+    public function run(array $constructor = []): bool
     {
         $this->logger->info('execute requested route ['.$this->path.']', [
             'category' => get_class($this),
@@ -286,7 +286,7 @@ 
                 if (!empty($body)) {
                     $json_params = json_decode($body, true);
                 } else {
-                    $parts  = explode('&', $_SERVER['QUERY_STRING']);
+                    $parts = explode('&', $_SERVER['QUERY_STRING']);
                     if (!empty($parts)) {
                         $json_params = json_decode(urldecode($parts[0]), true);
                     }

src/Log.php

Doc Comments +1 added lines, -1 removed lines

@@ -197,7 +197,7 @@
      * @param   string $date_format
      * @param   string $level
      * @param   array $context
-     * @return  void
+     * @return  string
      */
     protected function _format(string $message, string $format, string $date_format, string $level, array $context=[]): string
     {

Indentation +1 added lines, -1 removed lines

@@ -165,7 +165,7 @@
         foreach ($this->adapter as $adapter) {
             $prio = $adapter->getLevel();
  
-           if (self::PRIORITIES[$level] <= $prio) {
+            if (self::PRIORITIES[$level] <= $prio) {
                 $msg = $this->_format($message, $adapter->getFormat(), $adapter->getDateFormat(), $level, $context);
                 $adapter->log($level, $msg);
             }

Spacing +19 added lines, -19 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Micro
@@ -56,7 +56,7 @@ 
      * @param   Iterable $config
      * @return  void
      */
-    public function __construct(?Iterable $config=null)
+    public function __construct(? Iterable $config = null)
     {
         $this->setOptions($config);
     }
@@ -68,14 +68,14 @@ 
      * @param  Iterable $config
      * @return Logger
      */
-    public function setOptions(?Iterable $config=null)
+    public function setOptions(? Iterable $config = null)
     {
-        if($config === null) {
+        if ($config === null) {
             return $this;
         }
 
-        foreach($config as $option => $value) {
-            if(!isset($value['enabled']) || $value['enabled'] === '1') {
+        foreach ($config as $option => $value) {
+            if (!isset($value['enabled']) || $value['enabled'] === '1') {
                 $this->addAdapter($option, $value['class'], $value['config']);
             }
         }
@@ -92,14 +92,14 @@ 
      * @param  Iterable $config
      * @return AdapterInterface
      */
-    public function addAdapter(string $name, string $class, ?Iterable $config=null): AdapterInterface
+    public function addAdapter(string $name, string $class, ? Iterable $config = null) : AdapterInterface
     {
-        if(isset($this->adapter[$name])) {
+        if (isset($this->adapter[$name])) {
             throw new Exception\InvalidArgument('log adapter '.$name.' is already registered');
         }
             
         $adapter = new $class($config);
-        if(!($adapter instanceof AdapterInterface)) {
+        if (!($adapter instanceof AdapterInterface)) {
             throw new Exception\InvalidArgument('log adapter must include AdapterInterface interface');
         }
         $this->adapter[$name] = $adapter;
@@ -115,7 +115,7 @@ 
      */
     public function getAdapter(string $name): AdapterInterface
     {
-        if(!isset($this->adapter[$name])) {
+        if (!isset($this->adapter[$name])) {
             throw new Exception('log adapter '.$name.' is not registered');
         }
 
@@ -129,14 +129,14 @@ 
      * @param  array $adapters
      * @return array
      */
-    public function getAdapters(array $adapters=[]): array
+    public function getAdapters(array $adapters = []): array
     {
-        if(empty($adapter)) {
+        if (empty($adapter)) {
             return $this->adapter;
         } else {
             $list = [];
-            foreach($adapter as $name) {
-                if(!isset($this->adapter[$name])) {
+            foreach ($adapter as $name) {
+                if (!isset($this->adapter[$name])) {
                     throw new Exception('log adapter '.$name.' is not registered');
                 }
                 $list[$name] = $this->adapter[$name];
@@ -156,7 +156,7 @@ 
      * @param   array $context
      * @return  bool
      */
-    public function log($level, $message, array $context=[]): bool
+    public function log($level, $message, array $context = []): bool
     {
         if (!array_key_exists($level, self::PRIORITIES)) {
             throw new Exception\InvalidArgument('log level '.$level.' is unkown');
@@ -199,9 +199,9 @@ 
      * @param   array $context
      * @return  void
      */
-    protected function _format(string $message, string $format, string $date_format, string $level, array $context=[]): string
+    protected function _format(string $message, string $format, string $date_format, string $level, array $context = []): string
     {
-        $parsed = preg_replace_callback('/(\{(([a-z]\.*)+)\})/', function ($match) use ($message, $level, $date_format, $context) {
+        $parsed = preg_replace_callback('/(\{(([a-z]\.*)+)\})/', function($match) use ($message, $level, $date_format, $context) {
             $key = '';
             $context = array_merge($this->context, $context);
                     
@@ -224,9 +224,9 @@ 
                     $replace = [];
                     foreach ($context as $key => $val) {
                         if (!is_array($val) && (!is_object($val) || method_exists($val, '__toString'))) {
-                            $replace['{' . $key . '}'] = $val;
+                            $replace['{'.$key.'}'] = $val;
                         } else {
-                            $replace['{' . $key . '}'] = json_encode($val);
+                            $replace['{'.$key.'}'] = json_encode($val);
                         }
                     }
 

src/Log/Adapter/AbstractAdapter.php

Doc Comments -2 removed lines

@@ -48,7 +48,6 @@ 
     /**
      * Create adapter
      *
-     * @param Iterable $options
      * @return void
      */
     public function __construct(?Iterable $config=null)
@@ -91,7 +90,6 @@ 
     /**
      * Set options
      *
-     * @param   Iterable $options
      * @return  AdapterInterface
      */
     public function setOptions(?Iterable $config=null): AdapterInterface

Spacing +4 added lines, -4 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Micro
@@ -51,7 +51,7 @@ 
      * @param Iterable $options
      * @return void
      */
-    public function __construct(?Iterable $config=null)
+    public function __construct(? Iterable $config = null)
     {
         $this->setOptions($config);
     }
@@ -94,7 +94,7 @@ 
      * @param   Iterable $options
      * @return  AdapterInterface
      */
-    public function setOptions(?Iterable $config=null): AdapterInterface
+    public function setOptions(? Iterable $config = null) : AdapterInterface
     {
         if ($config === null) {
             return $this;
@@ -107,7 +107,7 @@ 
                 break;
                
                 case 'level':
-                    if(!is_numeric($val)) {
+                    if (!is_numeric($val)) {
                         throw new Exception\InvalidArgument('log level must be a number');
                     }
 

src/Log/Adapter/AdapterInterface.php

Doc Comments -2 removed lines

@@ -26,7 +26,6 @@ 
     /**
      * Create adapter
      *
-     * @param Iterable $options
      * @return void
      */
     public function __construct(?Iterable $config=null);
@@ -57,7 +56,6 @@ 
     /**
      * Set options
      *
-     * @param   Iterable $options
      * @return  AdapterInterface
      */
     public function setOptions(?Iterable $config=null): AdapterInterface;

Spacing +3 added lines, -3 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Micro
@@ -29,7 +29,7 @@ 
      * @param Iterable $options
      * @return void
      */
-    public function __construct(?Iterable $config=null);
+    public function __construct(? Iterable $config = null);
     
 
     /**
@@ -60,5 +60,5 @@ 
      * @param   Iterable $options
      * @return  AdapterInterface
      */
-    public function setOptions(?Iterable $config=null): AdapterInterface;
+    public function setOptions(? Iterable $config = null) : AdapterInterface;
 }

src/Log/Adapter/File.php

Doc Comments -1 removed lines

@@ -31,7 +31,6 @@
     /**
      * Set options
      *
-     * @param   Iterable $options
      * @return  AdapterInterface
      */
     public function setOptions(?Iterable $config=null): AdapterInterface

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /**
  * Micro
@@ -34,7 +34,7 @@ 
      * @param   Iterable $options
      * @return  AdapterInterface
      */
-    public function setOptions(?Iterable $config=null): AdapterInterface
+    public function setOptions(? Iterable $config = null) : AdapterInterface
     {
         parent::setOptions($config);