PhpPlaisio / error-logger-core

src/HtmlVarWriter.php

Indentation +2 added lines, -2 removed lines

@@ -237,8 +237,8 @@
 
       fwrite($this->handle, Html::generateElement('th',
                                                   ['class' => $class,
-                                                   'id'    => $id,
-                                                   'title' => $title],
+                                                    'id'    => $id,
+                                                    'title' => $title],
                                                   $text));
     }
   }

Spacing +15 added lines, -15 removed lines

@@ -73,7 +73,7 @@ 
       fwrite($this->handle, '<tr>');
       $this->writeName($name, $id);
       fwrite($this->handle, '<td>');
-      fwrite($this->handle, Html::generateElement('div', ['class' => 'array'], 'array').'<br/>');
+      fwrite($this->handle, Html::generateElement('div', [ 'class' => 'array' ], 'array').'<br/>');
       fwrite($this->handle, '<table>');
     }
   }
@@ -84,13 +84,13 @@ 
    */
   public function writeArrayReference(int $ref, $name): void
   {
-    $html = Html::generateElement('span', ['class' => 'array'], 'array');
+    $html = Html::generateElement('span', [ 'class' => 'array' ], 'array');
     $html .= ', ';
-    $html .= Html::generateElement('a', ['href' => '#'.$ref], 'see '.$ref);
+    $html .= Html::generateElement('a', [ 'href' => '#'.$ref ], 'see '.$ref);
 
     fwrite($this->handle, '<tr>');
     $this->writeName($name);
-    fwrite($this->handle, Html::generateElement('td', [], $html, true));
+    fwrite($this->handle, Html::generateElement('td', [ ], $html, true));
     fwrite($this->handle, '</tr>');
   }
 
@@ -155,7 +155,7 @@ 
       fwrite($this->handle, '<tr>');
       $this->writeName($name, $id);
       fwrite($this->handle, '<td>');
-      fwrite($this->handle, Html::generateElement('div', ['class' => 'class'], $class).'<br/>');
+      fwrite($this->handle, Html::generateElement('div', [ 'class' => 'class' ], $class).'<br/>');
       fwrite($this->handle, '<table>');
     }
   }
@@ -166,13 +166,13 @@ 
    */
   public function writeObjectReference(int $ref, $name, string $class): void
   {
-    $html = Html::generateElement('span', ['class' => 'class'], $class);
+    $html = Html::generateElement('span', [ 'class' => 'class' ], $class);
     $html .= ', ';
-    $html .= Html::generateElement('a', ['href' => '#'.(string)$ref], 'see '.(string)$ref);
+    $html .= Html::generateElement('a', [ 'href' => '#'.(string)$ref ], 'see '.(string)$ref);
 
     fwrite($this->handle, '<tr>');
     $this->writeName($name);
-    fwrite($this->handle, Html::generateElement('td', [], $html, true));
+    fwrite($this->handle, Html::generateElement('td', [ ], $html, true));
     fwrite($this->handle, '</tr>');
   }
 
@@ -208,7 +208,7 @@ 
   {
     if ($name===null || $name==='')
     {
-      fwrite($this->handle, Html::generateElement('th', ['class' => 'id', 'id' => $id], (string)$id));
+      fwrite($this->handle, Html::generateElement('th', [ 'class' => 'id', 'id' => $id ], (string)$id));
     }
     else
     {
@@ -233,12 +233,12 @@ 
         throw new \InvalidArgumentException(sprintf('$name has unexpected type %s', gettype($name)));
       }
 
-      fwrite($this->handle, Html::generateElement('th', ['class' => 'id'], (string)$id));
+      fwrite($this->handle, Html::generateElement('th', [ 'class' => 'id' ], (string)$id));
 
       fwrite($this->handle, Html::generateElement('th',
-                                                  ['class' => $class,
+                                                  [ 'class' => $class,
                                                    'id'    => $id,
-                                                   'title' => $title],
+                                                   'title' => $title ],
                                                   $text));
     }
   }
@@ -257,16 +257,16 @@ 
    */
   private function writeScalar(?int $id, ?int $ref, $name, string $text, string $class, ?string $title = null)
   {
-    $html = Html::generateElement('span', ['class' => $class, 'title' => $title], $text);
+    $html = Html::generateElement('span', [ 'class' => $class, 'title' => $title ], $text);
     if ($ref!==null)
     {
       $html .= ', ';
-      $html .= Html::generateElement('a', ['href' => '#'.$ref], 'see '.$ref);
+      $html .= Html::generateElement('a', [ 'href' => '#'.$ref ], 'see '.$ref);
     }
 
     fwrite($this->handle, '<tr>');
     $this->writeName($name, $id);
-    fwrite($this->handle, Html::generateElement('td', [], $html, true));
+    fwrite($this->handle, Html::generateElement('td', [ ], $html, true));
     fwrite($this->handle, '</tr>');
   }
 

Braces +3 added lines, -6 removed lines

@@ -209,8 +209,7 @@ 
     if ($name===null || $name==='')
     {
       fwrite($this->handle, Html::generateElement('th', ['class' => 'id', 'id' => $id], (string)$id));
-    }
-    else
+    } else
     {
       $title = null;
 
@@ -218,8 +217,7 @@ 
       {
         $text  = (string)$name;
         $class = 'number';
-      }
-      elseif (is_string($name))
+      } elseif (is_string($name))
       {
         $class = 'string';
         $text  = mb_strimwidth((string)$name, 0, 20, '...');
@@ -227,8 +225,7 @@ 
         {
           $title = mb_strimwidth((string)$name, 0, 512, '...');
         }
-      }
-      else
+      } else
       {
         throw new \InvalidArgumentException(sprintf('$name has unexpected type %s', gettype($name)));
       }

src/CoreErrorLogger.php

Indentation +1 added lines, -1 removed lines

@@ -237,7 +237,7 @@
         {
           $args[$key] = Html::generateElement('span',
                                               ['class' => 'string',
-                                               'title' => mb_substr($value, 0, 512)],
+                                                'title' => mb_substr($value, 0, 512)],
                                               mb_substr($value, 0, 32).'...');
         }
         else

Spacing +49 added lines, -49 removed lines

@@ -13,7 +13,7 @@ 
 abstract class CoreErrorLogger implements ErrorLogger
 {
   //--------------------------------------------------------------------------------------------------------------------
-  protected static $errorNames = [E_COMPILE_ERROR     => 'PHP Compile Error',
+  protected static $errorNames = [ E_COMPILE_ERROR     => 'PHP Compile Error',
                                   E_COMPILE_WARNING   => 'PHP Compile Warning',
                                   E_CORE_ERROR        => 'PHP Core Error',
                                   E_CORE_WARNING      => 'PHP Core Warning',
@@ -27,7 +27,7 @@ 
                                   E_USER_ERROR        => 'PHP User Error',
                                   E_USER_NOTICE       => 'PHP User Notice',
                                   E_USER_WARNING      => 'PHP User Warning',
-                                  E_WARNING           => 'PHP Warning'];
+                                  E_WARNING           => 'PHP Warning' ];
 
   /**
    * The output handle.
@@ -128,22 +128,22 @@ 
 
     if (!$isPrevious)
     {
-      fwrite($this->handle, Html::generateElement('h1', [], get_class($throwable)));
+      fwrite($this->handle, Html::generateElement('h1', [ ], get_class($throwable)));
     }
     else
     {
-      fwrite($this->handle, Html::generateElement('h2', [], 'Previous Exception: '.get_class($throwable)));
+      fwrite($this->handle, Html::generateElement('h2', [ ], 'Previous Exception: '.get_class($throwable)));
     }
 
-    if (isset(self::$errorNames[$throwable->getCode()]))
+    if (isset(self::$errorNames[ $throwable->getCode() ]))
     {
-      fwrite($this->handle, Html::generateElement('p', ['class' => 'code'], self::$errorNames[$throwable->getCode()]));
+      fwrite($this->handle, Html::generateElement('p', [ 'class' => 'code' ], self::$errorNames[ $throwable->getCode() ]));
     }
 
     $message = str_replace("\n", '<br/>', Html::txt2Html($throwable->getMessage()));
-    fwrite($this->handle, Html::generateElement('p', ['class' => 'message'], $message, true));
+    fwrite($this->handle, Html::generateElement('p', [ 'class' => 'message' ], $message, true));
 
-    fwrite($this->handle, Html::generateElement('p', ['class' => 'file'], $throwable->getFile().'('.$throwable->getLine().')'));
+    fwrite($this->handle, Html::generateElement('p', [ 'class' => 'file' ], $throwable->getFile().'('.$throwable->getLine().')'));
 
     $this->echoFileSnippet($throwable->getFile(), $throwable->getLine());
 
@@ -161,7 +161,7 @@ 
     fwrite($this->handle, '<!DOCTYPE html>');
     fwrite($this->handle, '<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">');
     fwrite($this->handle, '<head>');
-    fwrite($this->handle, Html::generateVoidElement('meta', ['charset' => Html::$encoding]));
+    fwrite($this->handle, Html::generateVoidElement('meta', [ 'charset' => Html::$encoding ]));
     fwrite($this->handle, '<title>Exception</title>');
 
     fwrite($this->handle, '<style>');
@@ -210,77 +210,77 @@ 
     foreach ($args as $key => $value)
     {
       $count++;
-      if ($count>=7)
+      if ($count >= 7)
       {
-        if ($count>7)
+        if ($count > 7)
         {
-          unset($args[$key]);
+          unset($args[ $key ]);
         }
         else
         {
-          $args[$key] = '...';
+          $args[ $key ] = '...';
         }
         continue;
       }
 
       if (is_object($value))
       {
-        $args[$key] = Html::generateElement('span', ['class' => 'class'], get_class($value));
+        $args[ $key ] = Html::generateElement('span', [ 'class' => 'class' ], get_class($value));
       }
       elseif (is_bool($value))
       {
-        $args[$key] = Html::generateElement('span', ['class' => 'keyword'], ($value ? 'true' : 'false'));
+        $args[ $key ] = Html::generateElement('span', [ 'class' => 'keyword' ], ($value ? 'true' : 'false'));
       }
       elseif (is_string($value))
       {
-        if (mb_strlen($value)>32)
+        if (mb_strlen($value) > 32)
         {
-          $args[$key] = Html::generateElement('span',
-                                              ['class' => 'string',
-                                               'title' => mb_substr($value, 0, 512)],
+          $args[ $key ] = Html::generateElement('span',
+                                              [ 'class' => 'string',
+                                               'title' => mb_substr($value, 0, 512) ],
                                               mb_substr($value, 0, 32).'...');
         }
         else
         {
-          $args[$key] = Html::generateElement('span', ['class' => 'string'], $value);
+          $args[ $key ] = Html::generateElement('span', [ 'class' => 'string' ], $value);
         }
       }
       elseif (is_array($value))
       {
-        $args[$key] = '['.$this->argumentsToString($value).']';
+        $args[ $key ] = '['.$this->argumentsToString($value).']';
       }
       elseif ($value===null)
       {
-        $args[$key] = '<span class="keyword">null</span>';
+        $args[ $key ] = '<span class="keyword">null</span>';
       }
       elseif (is_resource($value))
       {
-        $args[$key] = Html::generateElement('span', ['class' => 'keyword'], get_resource_type($value));
+        $args[ $key ] = Html::generateElement('span', [ 'class' => 'keyword' ], get_resource_type($value));
       }
       elseif (is_numeric($value))
       {
-        $args[$key] = Html::generateElement('span', ['class' => 'number'], (string)$value);
+        $args[ $key ] = Html::generateElement('span', [ 'class' => 'number' ], (string)$value);
       }
       else
       {
-        $args[$key] = '<span class="unknown">???</span>';
+        $args[ $key ] = '<span class="unknown">???</span>';
       }
 
       if (is_string($key))
       {
-        $tmp = Html::generateElement('span', ['class' => 'string'], $key);
+        $tmp = Html::generateElement('span', [ 'class' => 'string' ], $key);
         $tmp .= ' => ';
-        $tmp .= (strpos($key, 'password')===false) ? $args[$key] : str_repeat('*', 12);
+        $tmp .= (strpos($key, 'password')===false) ? $args[ $key ] : str_repeat('*', 12);
 
-        $args[$key] = $tmp;
+        $args[ $key ] = $tmp;
       }
       elseif ($isAssoc)
       {
-        $tmp = Html::generateElement('span', ['class' => 'number'], (string)$key);
+        $tmp = Html::generateElement('span', [ 'class' => 'number' ], (string)$key);
         $tmp .= ' => ';
-        $tmp .= $args[$key];
+        $tmp .= $args[ $key ];
 
-        $args[$key] = $tmp;
+        $args[ $key ] = $tmp;
       }
     }
 
@@ -295,19 +295,19 @@ 
    */
   private function echoCallable(array $item): void
   {
-    if (isset($item['class']))
+    if (isset($item[ 'class' ]))
     {
-      fwrite($this->handle, Html::generateElement('span', ['class' => 'class'], $item['class']));
+      fwrite($this->handle, Html::generateElement('span', [ 'class' => 'class' ], $item[ 'class' ]));
       fwrite($this->handle, '::');
-      fwrite($this->handle, Html::generateElement('span', ['class' => 'function'], $item['function']));
+      fwrite($this->handle, Html::generateElement('span', [ 'class' => 'function' ], $item[ 'function' ]));
     }
     else
     {
-      fwrite($this->handle, Html::generateElement('span', ['class' => 'function'], $item['function']));
+      fwrite($this->handle, Html::generateElement('span', [ 'class' => 'function' ], $item[ 'function' ]));
     }
 
     fwrite($this->handle, '(');
-    fwrite($this->handle, $this->argumentsToString($item['args']));
+    fwrite($this->handle, $this->argumentsToString($item[ 'args' ]));
     fwrite($this->handle, ')');
   }
 
@@ -328,8 +328,8 @@ 
 
     // div with lines numbers.
     fwrite($this->handle, '<div class="lines">');
-    fwrite($this->handle, Html::generateTag('ol', ['start' => $first]));
-    for ($i = $first; $i<=$last; $i++)
+    fwrite($this->handle, Html::generateTag('ol', [ 'start' => $first ]));
+    for ($i = $first; $i <= $last; $i++)
     {
       fwrite($this->handle, '<li></li>');
     }
@@ -338,19 +338,19 @@ 
 
     // The code as plain text (without markup and tags).
     fwrite($this->handle, '<pre><code class="php">');
-    for ($i = $first; $i<=$last; $i++)
+    for ($i = $first; $i <= $last; $i++)
     {
-      fwrite($this->handle, Html::txt2Html($lines[$i - 1]));
+      fwrite($this->handle, Html::txt2Html($lines[ $i - 1 ]));
       fwrite($this->handle, "\n");
     }
     fwrite($this->handle, '</code></pre>');
 
     // div for markup.
     fwrite($this->handle, '<div class="markup">');
-    fwrite($this->handle, Html::generateTag('ol', ['start' => $first]));
-    for ($i = $first; $i<=$last; $i++)
+    fwrite($this->handle, Html::generateTag('ol', [ 'start' => $first ]));
+    for ($i = $first; $i <= $last; $i++)
     {
-      fwrite($this->handle, Html::generateElement('li', ['class' => ($i==$line) ? 'error' : null], ''));
+      fwrite($this->handle, Html::generateElement('li', [ 'class' => ($i==$line) ? 'error' : null ], ''));
     }
     fwrite($this->handle, '</ol>');
     fwrite($this->handle, '</div>');
@@ -369,18 +369,18 @@ 
   {
     fwrite($this->handle, '<p class="file">');
 
-    fwrite($this->handle, Html::generateElement('span', ['class' => 'level'], (string)$number));
+    fwrite($this->handle, Html::generateElement('span', [ 'class' => 'level' ], (string)$number));
 
-    if (isset($item['file']))
+    if (isset($item[ 'file' ]))
     {
-      fwrite($this->handle, Html::generateElement('span', ['class' => 'file'], $item['file'].'('.$item['line'].'):'));
+      fwrite($this->handle, Html::generateElement('span', [ 'class' => 'file' ], $item[ 'file' ].'('.$item[ 'line' ].'):'));
     }
 
     $this->echoCallable($item);
 
-    if (isset($item['file']))
+    if (isset($item[ 'file' ]))
     {
-      $this->echoFileSnippet($item['file'], $item['line']);
+      $this->echoFileSnippet($item[ 'file' ], $item[ 'line' ]);
     }
 
     fwrite($this->handle, '</p>');
@@ -420,7 +420,7 @@ 
     // Return immediately if the are no variables to dump.
     if ($this->dump===null) return;
 
-    fwrite($this->handle, Html::generateElement('h2', [], 'VarDump'));
+    fwrite($this->handle, Html::generateElement('h2', [ ], 'VarDump'));
 
     $varDumper = new VarDumper(new HtmlVarWriter($this->handle));
     $varDumper->dump('', $this->dump, $this->scalarReferences);

Braces +22 added lines, -29 removed lines

@@ -99,8 +99,7 @@ 
       $this->echoPageTrailer();
 
       $this->closeStream();
-    }
-    catch (\Throwable $throwable)
+    } catch (\Throwable $throwable)
     {
       // Nothing to do.
     }
@@ -124,13 +123,14 @@ 
   protected function echoErrorLog(?\Throwable $throwable, bool $isPrevious = false): void
   {
     // Return immediately if there is not throwable.
-    if ($throwable===null) return;
+    if ($throwable===null) {
+      return;
+    }
 
     if (!$isPrevious)
     {
       fwrite($this->handle, Html::generateElement('h1', [], get_class($throwable)));
-    }
-    else
+    } else
     {
       fwrite($this->handle, Html::generateElement('h2', [], 'Previous Exception: '.get_class($throwable)));
     }
@@ -215,8 +215,7 @@ 
         if ($count>7)
         {
           unset($args[$key]);
-        }
-        else
+        } else
         {
           $args[$key] = '...';
         }
@@ -226,12 +225,10 @@ 
       if (is_object($value))
       {
         $args[$key] = Html::generateElement('span', ['class' => 'class'], get_class($value));
-      }
-      elseif (is_bool($value))
+      } elseif (is_bool($value))
       {
         $args[$key] = Html::generateElement('span', ['class' => 'keyword'], ($value ? 'true' : 'false'));
-      }
-      elseif (is_string($value))
+      } elseif (is_string($value))
       {
         if (mb_strlen($value)>32)
         {
@@ -239,29 +236,23 @@ 
                                               ['class' => 'string',
                                                'title' => mb_substr($value, 0, 512)],
                                               mb_substr($value, 0, 32).'...');
-        }
-        else
+        } else
         {
           $args[$key] = Html::generateElement('span', ['class' => 'string'], $value);
         }
-      }
-      elseif (is_array($value))
+      } elseif (is_array($value))
       {
         $args[$key] = '['.$this->argumentsToString($value).']';
-      }
-      elseif ($value===null)
+      } elseif ($value===null)
       {
         $args[$key] = '<span class="keyword">null</span>';
-      }
-      elseif (is_resource($value))
+      } elseif (is_resource($value))
       {
         $args[$key] = Html::generateElement('span', ['class' => 'keyword'], get_resource_type($value));
-      }
-      elseif (is_numeric($value))
+      } elseif (is_numeric($value))
       {
         $args[$key] = Html::generateElement('span', ['class' => 'number'], (string)$value);
-      }
-      else
+      } else
       {
         $args[$key] = '<span class="unknown">???</span>';
       }
@@ -273,8 +264,7 @@ 
         $tmp .= (strpos($key, 'password')===false) ? $args[$key] : str_repeat('*', 12);
 
         $args[$key] = $tmp;
-      }
-      elseif ($isAssoc)
+      } elseif ($isAssoc)
       {
         $tmp = Html::generateElement('span', ['class' => 'number'], (string)$key);
         $tmp .= ' => ';
@@ -300,8 +290,7 @@ 
       fwrite($this->handle, Html::generateElement('span', ['class' => 'class'], $item['class']));
       fwrite($this->handle, '::');
       fwrite($this->handle, Html::generateElement('span', ['class' => 'function'], $item['function']));
-    }
-    else
+    } else
     {
       fwrite($this->handle, Html::generateElement('span', ['class' => 'function'], $item['function']));
     }
@@ -397,7 +386,9 @@ 
     $trace = $throwable->getTrace();
 
     // Return immediately if the trace is empty.
-    if (empty($trace)) return;
+    if (empty($trace)) {
+      return;
+    }
 
     fwrite($this->handle, '<div class="trace">');
     fwrite($this->handle, '<h2>Stack Trace</h2>');
@@ -418,7 +409,9 @@ 
   private function echoVarDump(): void
   {
     // Return immediately if the are no variables to dump.
-    if ($this->dump===null) return;
+    if ($this->dump===null) {
+      return;
+    }
 
     fwrite($this->handle, Html::generateElement('h2', [], 'VarDump'));