spiral / framework

src/Models/src/DataEntity.php

Spacing +5 added lines, -5 removed lines

@@ -72,15 +72,15 @@ 
      */
     protected function isFillable(string $field): bool
     {
-        if (static::FILLABLE === '*') {
+        if (static::FILLABLE === '*'){
             return true;
         }
 
-        if (!empty(static::FILLABLE)) {
+        if (!empty(static::FILLABLE)){
             return in_array($field, static::FILLABLE);
         }
 
-        if (static::SECURED === '*') {
+        if (static::SECURED === '*'){
             return false;
         }
 
@@ -99,7 +99,7 @@ 
     protected function getMutator(string $field, string $mutator)
     {
         $target = [];
-        switch ($mutator) {
+        switch ($mutator){
             case ModelSchema::MUTATOR_ACCESSOR:
                 $target = static::ACCESSORS;
                 break;
@@ -111,7 +111,7 @@ 
                 break;
         }
 
-        if (isset($target[$field])) {
+        if (isset($target[$field])){
             return $target[$field];
         }
 

Braces +10 added lines, -5 removed lines

@@ -72,15 +72,18 @@ 
      */
     protected function isFillable(string $field): bool
     {
-        if (static::FILLABLE === '*') {
+        if (static::FILLABLE === '*')
+        {
             return true;
         }
 
-        if (!empty(static::FILLABLE)) {
+        if (!empty(static::FILLABLE))
+        {
             return in_array($field, static::FILLABLE);
         }
 
-        if (static::SECURED === '*') {
+        if (static::SECURED === '*')
+        {
             return false;
         }
 
@@ -99,7 +102,8 @@ 
     protected function getMutator(string $field, string $mutator)
     {
         $target = [];
-        switch ($mutator) {
+        switch ($mutator)
+        {
             case ModelSchema::MUTATOR_ACCESSOR:
                 $target = static::ACCESSORS;
                 break;
@@ -111,7 +115,8 @@ 
                 break;
         }
 
-        if (isset($target[$field])) {
+        if (isset($target[$field]))
+        {
             return $target[$field];
         }
 

src/Models/src/SchematicEntity.php

Spacing +4 added lines, -4 removed lines

@@ -34,15 +34,15 @@ 
      */
     protected function isFillable(string $field): bool
     {
-        if (!empty($this->schema[ModelSchema::FILLABLE]) && $this->schema[ModelSchema::FILLABLE] === '*') {
+        if (!empty($this->schema[ModelSchema::FILLABLE]) && $this->schema[ModelSchema::FILLABLE] === '*'){
             return true;
         }
 
-        if (!empty($this->schema[ModelSchema::FILLABLE])) {
+        if (!empty($this->schema[ModelSchema::FILLABLE])){
             return in_array($field, $this->schema[ModelSchema::FILLABLE]);
         }
 
-        if (!empty($this->schema[ModelSchema::SECURED]) && $this->schema[ModelSchema::SECURED] === '*') {
+        if (!empty($this->schema[ModelSchema::SECURED]) && $this->schema[ModelSchema::SECURED] === '*'){
             return false;
         }
 
@@ -54,7 +54,7 @@ 
      */
     protected function getMutator(string $field, string $mutator)
     {
-        if (isset($this->schema[ModelSchema::MUTATORS][$mutator][$field])) {
+        if (isset($this->schema[ModelSchema::MUTATORS][$mutator][$field])){
             return $this->schema[ModelSchema::MUTATORS][$mutator][$field];
         }
 

Braces +8 added lines, -4 removed lines

@@ -34,15 +34,18 @@ 
      */
     protected function isFillable(string $field): bool
     {
-        if (!empty($this->schema[ModelSchema::FILLABLE]) && $this->schema[ModelSchema::FILLABLE] === '*') {
+        if (!empty($this->schema[ModelSchema::FILLABLE]) && $this->schema[ModelSchema::FILLABLE] === '*')
+        {
             return true;
         }
 
-        if (!empty($this->schema[ModelSchema::FILLABLE])) {
+        if (!empty($this->schema[ModelSchema::FILLABLE]))
+        {
             return in_array($field, $this->schema[ModelSchema::FILLABLE]);
         }
 
-        if (!empty($this->schema[ModelSchema::SECURED]) && $this->schema[ModelSchema::SECURED] === '*') {
+        if (!empty($this->schema[ModelSchema::SECURED]) && $this->schema[ModelSchema::SECURED] === '*')
+        {
             return false;
         }
 
@@ -54,7 +57,8 @@ 
      */
     protected function getMutator(string $field, string $mutator)
     {
-        if (isset($this->schema[ModelSchema::MUTATORS][$mutator][$field])) {
+        if (isset($this->schema[ModelSchema::MUTATORS][$mutator][$field]))
+        {
             return $this->schema[ModelSchema::MUTATORS][$mutator][$field];
         }
 

src/Models/tests/GetEntity.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
 
     protected static function filter($v)
     {
-        if (is_array($v)) {
+        if (is_array($v)){
             throw new RuntimeException("can't be array");
         }
 

Braces +2 added lines, -1 removed lines

@@ -20,7 +20,8 @@
 
     protected static function filter($v)
     {
-        if (is_array($v)) {
+        if (is_array($v))
+        {
             throw new RuntimeException("can't be array");
         }
 

src/Models/tests/DataEntityTest.php

Spacing +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
         $this->assertEquals([], $entity->getValue());
 
         $entity['a'] = 90;
-        foreach ($entity as $key => $value) {
+        foreach ($entity as $key => $value){
             $this->assertSame('a', $key);
             $this->assertSame(90, $value);
         }

Braces +2 added lines, -1 removed lines

@@ -75,7 +75,8 @@
         $this->assertEquals([], $entity->getValue());
 
         $entity['a'] = 90;
-        foreach ($entity as $key => $value) {
+        foreach ($entity as $key => $value)
+        {
             $this->assertSame('a', $key);
             $this->assertSame(90, $value);
         }

src/Models/tests/NullableEntity.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
 
     protected function isNullable(string $field): bool
     {
-        if (parent::isNullable($field)) {
+        if (parent::isNullable($field)){
             return true;
         }
 

Braces +2 added lines, -1 removed lines

@@ -20,7 +20,8 @@
 
     protected function isNullable(string $field): bool
     {
-        if (parent::isNullable($field)) {
+        if (parent::isNullable($field))
+        {
             return true;
         }
 

src/Auth/src/AuthContext.php

Spacing +2 added lines, -2 removed lines

@@ -68,11 +68,11 @@
      */
     public function getActor(): ?object
     {
-        if ($this->closed) {
+        if ($this->closed){
             return null;
         }
 
-        if ($this->actor === null && $this->token !== null) {
+        if ($this->actor === null && $this->token !== null){
             $this->actor = $this->actorProvider->getActor($this->token);
         }
 

Braces +4 added lines, -2 removed lines

@@ -68,11 +68,13 @@
      */
     public function getActor(): ?object
     {
-        if ($this->closed) {
+        if ($this->closed)
+        {
             return null;
         }
 
-        if ($this->actor === null && $this->token !== null) {
+        if ($this->actor === null && $this->token !== null)
+        {
             $this->actor = $this->actorProvider->getActor($this->token);
         }
 

src/Dumper/tests/DumperTest.php

Braces +12 added lines, -6 removed lines

@@ -89,7 +89,8 @@ 
     {
         $d = $this->makeDumper();
 
-        $result = $d->dump(new class() {
+        $result = $d->dump(new class()
+        {
             private $name = 'Test';
         }, Dumper::RETURN);
 
@@ -151,7 +152,8 @@ 
     public function testDebugInfo(): void
     {
         $d = $this->makeDumper();
-        $result = $d->dump(new class() {
+        $result = $d->dump(new class()
+        {
             protected $inner = '_kk_';
 
             public function __debugInfo()
@@ -171,7 +173,8 @@ 
     public function testinternal(): void
     {
         $d = $this->makeDumper();
-        $result = $d->dump(new class() {
+        $result = $d->dump(new class()
+        {
             protected $visible = '_kk_';
 
             /** @internal */
@@ -198,7 +201,8 @@ 
         $d = $this->makeDumper();
         $d->setRenderer(Dumper::RETURN, new HtmlRenderer());
 
-        $result = $d->dump(new class() {
+        $result = $d->dump(new class()
+        {
             protected static $static = 'yes';
 
             private $value = 123;
@@ -222,7 +226,8 @@ 
         $d->setRenderer(Dumper::RETURN, new HtmlRenderer(HtmlRenderer::INVERTED));
         $d->setMaxLevel(5);
 
-        $result = $d->dump(new class() {
+        $result = $d->dump(new class()
+        {
             private $value = 123;
 
             protected $visible = '_kk_';
@@ -246,7 +251,8 @@ 
         $d->setRenderer(Dumper::RETURN, new ConsoleRenderer());
         $d->setMaxLevel(5);
 
-        $result = $d->dump(new class() {
+        $result = $d->dump(new class()
+        {
             private $value = 123;
 
             protected $visible = '_kk_';

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

@@ -29,14 +29,14 @@ 
         $d->dump(1);
         $result = ob_get_clean();
 
-        $this->assertSame($d->dump(1, Dumper::RETURN), $result);
+        $this->assertSame($d->dump(1, Dumper::return), $result);
     }
 
     private function makeDumper(LoggerInterface $logger = null)
     {
         $d = new Dumper($logger);
         $d->setRenderer(Dumper::OUTPUT, new PlainRenderer());
-        $d->setRenderer(Dumper::RETURN, new PlainRenderer());
+        $d->setRenderer(Dumper::return, new PlainRenderer());
         $d->setRenderer(Dumper::OUTPUT_CLI_COLORS, new PlainRenderer());
 
         return $d;
@@ -45,7 +45,7 @@ 
     public function testScalar(): void
     {
         $d = $this->makeDumper();
-        $result = $d->dump(123, Dumper::RETURN);
+        $result = $d->dump(123, Dumper::return);
 
         $this->assertStringContainsString('123', $result);
     }
@@ -53,7 +53,7 @@ 
     public function testString(): void
     {
         $d = $this->makeDumper();
-        $result = $d->dump('hello world', Dumper::RETURN);
+        $result = $d->dump('hello world', Dumper::return);
 
         $this->assertStringContainsString('hello world', $result);
     }
@@ -61,7 +61,7 @@ 
     public function testResource(): void
     {
         $d = $this->makeDumper();
-        $result = $d->dump(STDOUT, Dumper::RETURN);
+        $result = $d->dump(STDOUT, Dumper::return);
 
         $this->assertStringContainsString('resource', $result);
     }
@@ -69,7 +69,7 @@ 
     public function testHTML(): void
     {
         $d = $this->makeDumper();
-        $result = $d->dump('hello <br/>world', Dumper::RETURN);
+        $result = $d->dump('hello <br/>world', Dumper::return);
 
         $this->assertStringContainsString('hello &lt;br/&gt;world', $result);
     }
@@ -77,7 +77,7 @@ 
     public function testArray(): void
     {
         $d = $this->makeDumper();
-        $result = $d->dump(['G', 'B', 'N'], Dumper::RETURN);
+        $result = $d->dump(['G', 'B', 'N'], Dumper::return);
 
         $this->assertStringContainsString('array', $result);
         $this->assertStringContainsString('G', $result);
@@ -91,7 +91,7 @@ 
 
         $result = $d->dump(new class() {
             private $name = 'Test';
-        }, Dumper::RETURN);
+        }, Dumper::return);
 
         $this->assertStringContainsString('object', $result);
         $this->assertStringContainsString('private', $result);
@@ -106,7 +106,7 @@ 
 
         $result = $d->dump(static function (): void {
             echo 'hello world';
-        }, Dumper::RETURN);
+        }, Dumper::return);
 
         $this->assertStringContainsString('Closure', $result);
         $this->assertStringContainsString('DumperTest.php', $result);
@@ -117,7 +117,7 @@ 
         $mock = $this->createMock(LoggerInterface::class);
         $d = $this->makeDumper($mock);
 
-        $mock->method('debug')->with($d->dump('abc', Dumper::RETURN));
+        $mock->method('debug')->with($d->dump('abc', Dumper::return));
         $this->assertNull($d->dump('abc', Dumper::LOGGER));
     }
 
@@ -160,7 +160,7 @@ 
                     '_magic_' => '_value_',
                 ];
             }
-        }, Dumper::RETURN);
+        }, Dumper::return);
 
         $this->assertStringContainsString('_magic_', $result);
         $this->assertStringContainsString('_value_', $result);
@@ -176,7 +176,7 @@ 
 
             /** @internal */
             protected $internal = '_ok_';
-        }, Dumper::RETURN);
+        }, Dumper::return);
 
         $this->assertStringContainsString('visible', $result);
         $this->assertStringContainsString('_kk_', $result);
@@ -196,7 +196,7 @@ 
     public function testHtmlRenderer(): void
     {
         $d = $this->makeDumper();
-        $d->setRenderer(Dumper::RETURN, new HtmlRenderer());
+        $d->setRenderer(Dumper::return, new HtmlRenderer());
 
         $result = $d->dump(new class() {
             protected static $static = 'yes';
@@ -207,7 +207,7 @@ 
 
             /** @internal */
             protected $internal = '_ok_';
-        }, Dumper::RETURN);
+        }, Dumper::return);
 
         $this->assertStringContainsString('visible', $result);
         $this->assertStringContainsString('_kk_', $result);
@@ -219,7 +219,7 @@ 
     public function testInvertedRenderer(): void
     {
         $d = $this->makeDumper();
-        $d->setRenderer(Dumper::RETURN, new HtmlRenderer(HtmlRenderer::INVERTED));
+        $d->setRenderer(Dumper::return, new HtmlRenderer(HtmlRenderer::INVERTED));
         $d->setMaxLevel(5);
 
         $result = $d->dump(new class() {
@@ -231,7 +231,7 @@ 
 
             /** @internal */
             protected $internal = '_ok_';
-        }, Dumper::RETURN);
+        }, Dumper::return);
 
         $this->assertStringContainsString('visible', $result);
         $this->assertStringContainsString('_kk_', $result);
@@ -243,7 +243,7 @@ 
     public function testConsoleRenderer(): void
     {
         $d = $this->makeDumper();
-        $d->setRenderer(Dumper::RETURN, new ConsoleRenderer());
+        $d->setRenderer(Dumper::return, new ConsoleRenderer());
         $d->setMaxLevel(5);
 
         $result = $d->dump(new class() {
@@ -255,7 +255,7 @@ 
 
             /** @internal */
             protected $internal = '_ok_';
-        }, Dumper::RETURN);
+        }, Dumper::return);
 
         $this->assertStringContainsString('visible', $result);
         $this->assertStringContainsString('_kk_', $result);

src/Dumper/src/System.php

Spacing +8 added lines, -8 removed lines

@@ -24,12 +24,12 @@ 
      */
     public static function isCLI(): bool
     {
-        if (!empty(getenv('RR'))) {
+        if (!empty(getenv('RR'))){
             // Do not treat RoadRunner as CLI.
             return false;
         }
 
-        if (php_sapi_name() === 'cli') {
+        if (php_sapi_name() === 'cli'){
             return true;
         }
 
@@ -46,12 +46,12 @@ 
      */
     public static function isColorsSupported($stream = STDOUT): bool
     {
-        if ('Hyper' === getenv('TERM_PROGRAM')) {
+        if ('Hyper' === getenv('TERM_PROGRAM')){
             return true;
         }
 
-        try {
-            if (\DIRECTORY_SEPARATOR === '\\') {
+        try{
+            if (\DIRECTORY_SEPARATOR === '\\'){
                 return (
                         function_exists('sapi_windows_vt100_support')
                         && @sapi_windows_vt100_support($stream)
@@ -60,11 +60,11 @@ 
                     || getenv('TERM') == 'xterm';
             }
 
-            if (\function_exists('stream_isatty')) {
+            if (\function_exists('stream_isatty')){
                 return @stream_isatty($stream);
             }
 
-            if (\function_exists('posix_isatty')) {
+            if (\function_exists('posix_isatty')){
                 return @posix_isatty($stream);
             }
 
@@ -72,7 +72,7 @@ 
 
             // Check if formatted mode is S_IFCHR
             return $stat ? 0020000 === ($stat['mode'] & 0170000) : false;
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             return false;
         }
     }

Braces +17 added lines, -8 removed lines

@@ -24,12 +24,14 @@ 
      */
     public static function isCLI(): bool
     {
-        if (!empty(getenv('RR'))) {
+        if (!empty(getenv('RR')))
+        {
             // Do not treat RoadRunner as CLI.
             return false;
         }
 
-        if (php_sapi_name() === 'cli') {
+        if (php_sapi_name() === 'cli')
+        {
             return true;
         }
 
@@ -46,12 +48,15 @@ 
      */
     public static function isColorsSupported($stream = STDOUT): bool
     {
-        if ('Hyper' === getenv('TERM_PROGRAM')) {
+        if ('Hyper' === getenv('TERM_PROGRAM'))
+        {
             return true;
         }
 
-        try {
-            if (\DIRECTORY_SEPARATOR === '\\') {
+        try
+        {
+            if (\DIRECTORY_SEPARATOR === '\\')
+            {
                 return (
                         function_exists('sapi_windows_vt100_support')
                         && @sapi_windows_vt100_support($stream)
@@ -60,11 +65,13 @@ 
                     || getenv('TERM') == 'xterm';
             }
 
-            if (\function_exists('stream_isatty')) {
+            if (\function_exists('stream_isatty'))
+            {
                 return @stream_isatty($stream);
             }
 
-            if (\function_exists('posix_isatty')) {
+            if (\function_exists('posix_isatty'))
+            {
                 return @posix_isatty($stream);
             }
 
@@ -72,7 +79,9 @@ 
 
             // Check if formatted mode is S_IFCHR
             return $stat ? 0020000 === ($stat['mode'] & 0170000) : false;
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             return false;
         }
     }

src/Dumper/src/Dumper.php

Braces +84 added lines, -38 removed lines

@@ -58,7 +58,8 @@ 
      */
     public function __construct(LoggerInterface $logger = null)
     {
-        if (!empty($logger)) {
+        if (!empty($logger))
+        {
             $this->setLogger($logger);
         }
     }
@@ -86,7 +87,8 @@ 
         $r = $this->getRenderer($target);
         $dump = $r->wrapContent($this->renderValue($r, $value));
 
-        switch ($target) {
+        switch ($target)
+        {
             case self::OUTPUT:
                 echo $dump;
                 break;
@@ -95,7 +97,8 @@ 
                 return $dump;
 
             case self::LOGGER:
-                if ($this->logger == null) {
+                if ($this->logger == null)
+                {
                     throw new DumperException('Unable to dump value to log, no associated LoggerInterface');
                 }
                 $this->logger->debug($dump);
@@ -119,7 +122,8 @@ 
      */
     public function setRenderer(int $target, RendererInterface $renderer): Dumper
     {
-        if (!isset($this->targets[$target])) {
+        if (!isset($this->targets[$target]))
+        {
             throw new DumperException(sprintf('Undefined dump target %d', $target));
         }
 
@@ -138,19 +142,25 @@ 
      */
     private function getRenderer(int $target): RendererInterface
     {
-        if ($target == self::OUTPUT && System::isCLI()) {
-            if (System::isColorsSupported(STDOUT)) {
+        if ($target == self::OUTPUT && System::isCLI())
+        {
+            if (System::isColorsSupported(STDOUT))
+            {
                 $target = self::OUTPUT_CLI_COLORS;
-            } else {
+            }
+            else
+            {
                 $target = self::OUTPUT_CLI;
             }
         }
 
-        if (!isset($this->targets[$target])) {
+        if (!isset($this->targets[$target]))
+        {
             throw new DumperException(sprintf('Undefined dump target %d', $target));
         }
 
-        if (is_string($this->targets[$target])) {
+        if (is_string($this->targets[$target]))
+        {
             $this->targets[$target] = new $this->targets[$target]();
         }
 
@@ -175,28 +185,35 @@ 
         int $level = 0,
         bool $hideHeader = false
     ): string {
-        if (!$hideHeader && !empty($name)) {
+        if (!$hideHeader && !empty($name))
+        {
             $header = $r->indent($level) . $r->apply($name, 'name') . $r->apply(' = ', 'syntax', '=');
-        } else {
+        }
+        else
+        {
             $header = $r->indent($level);
         }
 
-        if ($level > $this->maxLevel) {
+        if ($level > $this->maxLevel)
+        {
             //Renderer is not reference based, we can't dump too deep values
             return $r->indent($level) . $r->apply('-too deep-', 'maxLevel') . "\n";
         }
 
         $type = strtolower(gettype($value));
 
-        if ($type == 'array') {
+        if ($type == 'array')
+        {
             return $header . $this->renderArray($r, $value, $level, $hideHeader);
         }
 
-        if ($type == 'object') {
+        if ($type == 'object')
+        {
             return $header . $this->renderObject($r, $value, $level, $hideHeader);
         }
 
-        if ($type == 'resource') {
+        if ($type == 'resource')
+        {
             //No need to dump resource value
             $element = get_resource_type($value) . ' resource ';
 
@@ -210,7 +227,8 @@ 
         $header .= $r->apply("{$type}({$length})", 'type', $type);
 
         $element = null;
-        switch ($type) {
+        switch ($type)
+        {
             case 'string':
                 $element = $r->escapeStrings() ? htmlspecialchars($value) : $value;
                 break;
@@ -220,7 +238,8 @@ 
                 break;
 
             default:
-                if ($value !== null) {
+                if ($value !== null)
+                {
                     //Not showing null value, type is enough
                     $element = var_export($value, true);
                 }
@@ -240,19 +259,25 @@ 
      */
     private function renderArray(RendererInterface $r, array $array, int $level, bool $hideHeader = false): string
     {
-        if (!$hideHeader) {
+        if (!$hideHeader)
+        {
             $count = count($array);
 
             //Array size and scope
             $output = $r->apply("array({$count})", 'type', 'array') . "\n";
             $output .= $r->indent($level) . $r->apply('[', 'syntax', '[') . "\n";
-        } else {
+        }
+        else
+        {
             $output = '';
         }
 
-        foreach ($array as $key => $value) {
-            if (!is_numeric($key)) {
-                if (is_string($key) && $r->escapeStrings()) {
+        foreach ($array as $key => $value)
+        {
+            if (!is_numeric($key))
+            {
+                if (is_string($key) && $r->escapeStrings())
+                {
                     $key = htmlspecialchars($key);
                 }
 
@@ -262,7 +287,8 @@ 
             $output .= $this->renderValue($r, $value, "[{$key}]", $level + 1);
         }
 
-        if (!$hideHeader) {
+        if (!$hideHeader)
+        {
             //Closing array scope
             $output .= $r->indent($level) . $r->apply(']', 'syntax', ']') . "\n";
         }
@@ -286,29 +312,38 @@ 
         bool $hideHeader = false,
         string $class = ''
     ): string {
-        if (!$hideHeader) {
+        if (!$hideHeader)
+        {
             $type = ($class ?: get_class($value)) . ' object ';
 
             $header = $r->apply($type, 'type', 'object') . "\n";
             $header .= $r->indent($level) . $r->apply('(', 'syntax', '(') . "\n";
-        } else {
+        }
+        else
+        {
             $header = '';
         }
 
         //Let's use method specifically created for dumping
-        if (method_exists($value, '__debugInfo') || $value instanceof \Closure) {
-            if ($value instanceof \Closure) {
+        if (method_exists($value, '__debugInfo') || $value instanceof \Closure)
+        {
+            if ($value instanceof \Closure)
+            {
                 $debugInfo = $this->describeClosure($value);
-            } else {
+            }
+            else
+            {
                 $debugInfo = $value->__debugInfo();
             }
 
-            if (is_array($debugInfo)) {
+            if (is_array($debugInfo))
+            {
                 //Pretty view
                 $debugInfo = (object)$debugInfo;
             }
 
-            if (is_object($debugInfo)) {
+            if (is_object($debugInfo))
+            {
                 //We are not including syntax elements here
                 return $this->renderObject($r, $debugInfo, $level, false, get_class($value));
             }
@@ -321,7 +356,8 @@ 
         $refection = new \ReflectionObject($value);
 
         $output = '';
-        foreach ($refection->getProperties() as $property) {
+        foreach ($refection->getProperties() as $property)
+        {
             $output .= $this->renderProperty($r, $value, $property, $level);
         }
 
@@ -339,7 +375,8 @@ 
      */
     private function renderProperty(RendererInterface $r, $value, \ReflectionProperty $p, int $level): string
     {
-        if ($p->isStatic()) {
+        if ($p->isStatic())
+        {
             return '';
         }
 
@@ -359,9 +396,12 @@ 
         //To read private and protected properties
         $p->setAccessible(true);
 
-        if ($value instanceof \stdClass) {
+        if ($value instanceof \stdClass)
+        {
             $name = $r->apply($p->getName(), 'dynamic');
-        } else {
+        }
+        else
+        {
             //Property name includes access level
             $name = $p->getName() . $r->apply(':' . $access, 'access', $access);
         }
@@ -377,9 +417,12 @@ 
      */
     private function describeClosure(\Closure $closure): array
     {
-        try {
+        try
+        {
             $r = new \ReflectionFunction($closure);
-        } catch (\ReflectionException $e) {
+        }
+        catch (\ReflectionException $e)
+        {
             return ['closure' => 'unable to resolve'];
         }
 
@@ -399,9 +442,12 @@ 
      */
     private function getAccess(\ReflectionProperty $p): string
     {
-        if ($p->isPrivate()) {
+        if ($p->isPrivate())
+        {
             return 'private';
-        } elseif ($p->isProtected()) {
+        }
+        elseif ($p->isProtected())
+        {
             return 'protected';
         }
 

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

@@ -30,7 +30,7 @@ 
      * Directives for dump output forwarding.
      */
     public const OUTPUT            = 0;
-    public const RETURN            = 1;
+    public const return            = 1;
     public const LOGGER            = 2;
     public const ERROR_LOG         = 3;
     public const OUTPUT_CLI        = 4;
@@ -48,7 +48,7 @@ 
         self::OUTPUT            => HtmlRenderer::class,
         self::OUTPUT_CLI        => PlainRenderer::class,
         self::OUTPUT_CLI_COLORS => ConsoleRenderer::class,
-        self::RETURN            => HtmlRenderer::class,
+        self::return            => HtmlRenderer::class,
         self::LOGGER            => PlainRenderer::class,
         self::ERROR_LOG         => PlainRenderer::class,
     ];
@@ -91,7 +91,7 @@ 
                 echo $dump;
                 break;
 
-            case self::RETURN:
+            case self::return:
                 return $dump;
 
             case self::LOGGER:

Spacing +56 added lines, -56 removed lines

@@ -58,7 +58,7 @@ 
      */
     public function __construct(LoggerInterface $logger = null)
     {
-        if (!empty($logger)) {
+        if (!empty($logger)){
             $this->setLogger($logger);
         }
     }
@@ -86,7 +86,7 @@ 
         $r = $this->getRenderer($target);
         $dump = $r->wrapContent($this->renderValue($r, $value));
 
-        switch ($target) {
+        switch ($target){
             case self::OUTPUT:
                 echo $dump;
                 break;
@@ -95,7 +95,7 @@ 
                 return $dump;
 
             case self::LOGGER:
-                if ($this->logger == null) {
+                if ($this->logger == null){
                     throw new DumperException('Unable to dump value to log, no associated LoggerInterface');
                 }
                 $this->logger->debug($dump);
@@ -119,7 +119,7 @@ 
      */
     public function setRenderer(int $target, RendererInterface $renderer): Dumper
     {
-        if (!isset($this->targets[$target])) {
+        if (!isset($this->targets[$target])){
             throw new DumperException(sprintf('Undefined dump target %d', $target));
         }
 
@@ -138,19 +138,19 @@ 
      */
     private function getRenderer(int $target): RendererInterface
     {
-        if ($target == self::OUTPUT && System::isCLI()) {
-            if (System::isColorsSupported(STDOUT)) {
+        if ($target == self::OUTPUT && System::isCLI()){
+            if (System::isColorsSupported(STDOUT)){
                 $target = self::OUTPUT_CLI_COLORS;
-            } else {
+            }else{
                 $target = self::OUTPUT_CLI;
             }
         }
 
-        if (!isset($this->targets[$target])) {
+        if (!isset($this->targets[$target])){
             throw new DumperException(sprintf('Undefined dump target %d', $target));
         }
 
-        if (is_string($this->targets[$target])) {
+        if (is_string($this->targets[$target])){
             $this->targets[$target] = new $this->targets[$target]();
         }
 
@@ -175,32 +175,32 @@ 
         int $level = 0,
         bool $hideHeader = false
     ): string {
-        if (!$hideHeader && !empty($name)) {
-            $header = $r->indent($level) . $r->apply($name, 'name') . $r->apply(' = ', 'syntax', '=');
-        } else {
+        if (!$hideHeader && !empty($name)){
+            $header = $r->indent($level).$r->apply($name, 'name').$r->apply(' = ', 'syntax', '=');
+        }else{
             $header = $r->indent($level);
         }
 
-        if ($level > $this->maxLevel) {
+        if ($level > $this->maxLevel){
             //Renderer is not reference based, we can't dump too deep values
-            return $r->indent($level) . $r->apply('-too deep-', 'maxLevel') . "\n";
+            return $r->indent($level).$r->apply('-too deep-', 'maxLevel')."\n";
         }
 
         $type = strtolower(gettype($value));
 
-        if ($type == 'array') {
-            return $header . $this->renderArray($r, $value, $level, $hideHeader);
+        if ($type == 'array'){
+            return $header.$this->renderArray($r, $value, $level, $hideHeader);
         }
 
-        if ($type == 'object') {
-            return $header . $this->renderObject($r, $value, $level, $hideHeader);
+        if ($type == 'object'){
+            return $header.$this->renderObject($r, $value, $level, $hideHeader);
         }
 
-        if ($type == 'resource') {
+        if ($type == 'resource'){
             //No need to dump resource value
-            $element = get_resource_type($value) . ' resource ';
+            $element = get_resource_type($value).' resource ';
 
-            return $header . $r->apply($element, 'type', 'resource') . "\n";
+            return $header.$r->apply($element, 'type', 'resource')."\n";
         }
 
         //Value length
@@ -210,7 +210,7 @@ 
         $header .= $r->apply("{$type}({$length})", 'type', $type);
 
         $element = null;
-        switch ($type) {
+        switch ($type){
             case 'string':
                 $element = $r->escapeStrings() ? htmlspecialchars($value) : $value;
                 break;
@@ -220,14 +220,14 @@ 
                 break;
 
             default:
-                if ($value !== null) {
+                if ($value !== null){
                     //Not showing null value, type is enough
                     $element = var_export($value, true);
                 }
         }
 
         //Including value
-        return $header . ' ' . $r->apply($element, 'value', $type) . "\n";
+        return $header.' '.$r->apply($element, 'value', $type)."\n";
     }
 
     /**
@@ -240,19 +240,19 @@ 
      */
     private function renderArray(RendererInterface $r, array $array, int $level, bool $hideHeader = false): string
     {
-        if (!$hideHeader) {
+        if (!$hideHeader){
             $count = count($array);
 
             //Array size and scope
-            $output = $r->apply("array({$count})", 'type', 'array') . "\n";
-            $output .= $r->indent($level) . $r->apply('[', 'syntax', '[') . "\n";
-        } else {
+            $output = $r->apply("array({$count})", 'type', 'array')."\n";
+            $output .= $r->indent($level).$r->apply('[', 'syntax', '[')."\n";
+        }else{
             $output = '';
         }
 
-        foreach ($array as $key => $value) {
-            if (!is_numeric($key)) {
-                if (is_string($key) && $r->escapeStrings()) {
+        foreach ($array as $key => $value){
+            if (!is_numeric($key)){
+                if (is_string($key) && $r->escapeStrings()){
                     $key = htmlspecialchars($key);
                 }
 
@@ -262,9 +262,9 @@ 
             $output .= $this->renderValue($r, $value, "[{$key}]", $level + 1);
         }
 
-        if (!$hideHeader) {
+        if (!$hideHeader){
             //Closing array scope
-            $output .= $r->indent($level) . $r->apply(']', 'syntax', ']') . "\n";
+            $output .= $r->indent($level).$r->apply(']', 'syntax', ']')."\n";
         }
 
         return $output;
@@ -286,47 +286,47 @@ 
         bool $hideHeader = false,
         string $class = ''
     ): string {
-        if (!$hideHeader) {
-            $type = ($class ?: get_class($value)) . ' object ';
+        if (!$hideHeader){
+            $type = ($class ?: get_class($value)).' object ';
 
-            $header = $r->apply($type, 'type', 'object') . "\n";
-            $header .= $r->indent($level) . $r->apply('(', 'syntax', '(') . "\n";
-        } else {
+            $header = $r->apply($type, 'type', 'object')."\n";
+            $header .= $r->indent($level).$r->apply('(', 'syntax', '(')."\n";
+        }else{
             $header = '';
         }
 
         //Let's use method specifically created for dumping
-        if (method_exists($value, '__debugInfo') || $value instanceof \Closure) {
-            if ($value instanceof \Closure) {
+        if (method_exists($value, '__debugInfo') || $value instanceof \Closure){
+            if ($value instanceof \Closure){
                 $debugInfo = $this->describeClosure($value);
-            } else {
+            }else{
                 $debugInfo = $value->__debugInfo();
             }
 
-            if (is_array($debugInfo)) {
+            if (is_array($debugInfo)){
                 //Pretty view
                 $debugInfo = (object)$debugInfo;
             }
 
-            if (is_object($debugInfo)) {
+            if (is_object($debugInfo)){
                 //We are not including syntax elements here
                 return $this->renderObject($r, $debugInfo, $level, false, get_class($value));
             }
 
             return $header
                 . $this->renderValue($r, $debugInfo, '', $level + (is_scalar($value)), true)
-                . $r->indent($level) . $r->apply(')', 'syntax', ')') . "\n";
+                . $r->indent($level).$r->apply(')', 'syntax', ')')."\n";
         }
 
         $refection = new \ReflectionObject($value);
 
         $output = '';
-        foreach ($refection->getProperties() as $property) {
+        foreach ($refection->getProperties() as $property){
             $output .= $this->renderProperty($r, $value, $property, $level);
         }
 
         //Header, content, footer
-        return $header . $output . $r->indent($level) . $r->apply(')', 'syntax', ')') . "\n";
+        return $header.$output.$r->indent($level).$r->apply(')', 'syntax', ')')."\n";
     }
 
     /**
@@ -339,7 +339,7 @@ 
      */
     private function renderProperty(RendererInterface $r, $value, \ReflectionProperty $p, int $level): string
     {
-        if ($p->isStatic()) {
+        if ($p->isStatic()){
             return '';
         }
 
@@ -347,7 +347,7 @@ 
             !($value instanceof \stdClass)
             && is_string($p->getDocComment())
             && strpos($p->getDocComment(), '@internal') !== false
-        ) {
+        ){
             // Memory loop while reading doc comment for stdClass variables?
             // Report a PHP bug about treating comment INSIDE property declaration as doc comment.
             return '';
@@ -359,11 +359,11 @@ 
         //To read private and protected properties
         $p->setAccessible(true);
 
-        if ($value instanceof \stdClass) {
+        if ($value instanceof \stdClass){
             $name = $r->apply($p->getName(), 'dynamic');
-        } else {
+        }else{
             //Property name includes access level
-            $name = $p->getName() . $r->apply(':' . $access, 'access', $access);
+            $name = $p->getName().$r->apply(':'.$access, 'access', $access);
         }
 
         return $this->renderValue($r, $p->getValue($value), $name, $level + 1);
@@ -377,14 +377,14 @@ 
      */
     private function describeClosure(\Closure $closure): array
     {
-        try {
+        try{
             $r = new \ReflectionFunction($closure);
-        } catch (\ReflectionException $e) {
+        }catch (\ReflectionException $e){
             return ['closure' => 'unable to resolve'];
         }
 
         return [
-            'name' => $r->getName() . " (lines {$r->getStartLine()}:{$r->getEndLine()})",
+            'name' => $r->getName()." (lines {$r->getStartLine()}:{$r->getEndLine()})",
             'file' => $r->getFileName(),
             'this' => $r->getClosureThis(),
         ];
@@ -399,9 +399,9 @@ 
      */
     private function getAccess(\ReflectionProperty $p): string
     {
-        if ($p->isPrivate()) {
+        if ($p->isPrivate()){
             return 'private';
-        } elseif ($p->isProtected()) {
+        } elseif ($p->isProtected()){
             return 'protected';
         }