spiral / framework

src/Dumper/src/Dumper.php

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;
@@ -49,7 +49,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,
         self::ROADRUNNER        => PlainRenderer::class,
@@ -94,7 +94,7 @@ 
                 break;
 
             case self::ROADRUNNER:
-            case self::RETURN:
+            case self::return:
                 return $dump;
 
             case self::LOGGER:

Spacing +56 added lines, -56 removed lines

@@ -57,7 +57,7 @@ 
 
     public function __construct(LoggerInterface $logger = null)
     {
-        if (!empty($logger)) {
+        if (!empty($logger)){
             $this->setLogger($logger);
         }
     }
@@ -83,7 +83,7 @@ 
         $r = $this->getRenderer($target);
         $dump = $r->wrapContent($this->renderValue($r, $value));
 
-        switch ($target) {
+        switch ($target){
             case self::OUTPUT:
                 echo $dump;
                 break;
@@ -93,7 +93,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);
@@ -114,7 +114,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));
         }
 
@@ -131,19 +131,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]();
         }
 
@@ -166,32 +166,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
@@ -201,7 +201,7 @@ 
         $header .= $r->apply("{$type}({$length})", 'type', $type);
 
         $element = null;
-        switch ($type) {
+        switch ($type){
             case 'string':
                 $element = $r->escapeStrings() ? htmlspecialchars($value) : $value;
                 break;
@@ -211,31 +211,31 @@ 
                 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";
     }
 
     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);
                 }
 
@@ -245,9 +245,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;
@@ -264,47 +264,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";
     }
 
     /**
@@ -313,7 +313,7 @@ 
      */
     private function renderProperty(RendererInterface $r, $value, \ReflectionProperty $p, int $level): string
     {
-        if ($p->isStatic()) {
+        if ($p->isStatic()){
             return '';
         }
 
@@ -321,7 +321,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 '';
@@ -333,11 +333,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);
@@ -348,14 +348,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(),
         ];
@@ -368,9 +368,9 @@ 
      */
     private function getAccess(\ReflectionProperty $p): string
     {
-        if ($p->isPrivate()) {
+        if ($p->isPrivate()){
             return 'private';
-        } elseif ($p->isProtected()) {
+        } elseif ($p->isProtected()){
             return 'protected';
         }
 

Braces +84 added lines, -38 removed lines

@@ -57,7 +57,8 @@ 
 
     public function __construct(LoggerInterface $logger = null)
     {
-        if (!empty($logger)) {
+        if (!empty($logger))
+        {
             $this->setLogger($logger);
         }
     }
@@ -83,7 +84,8 @@ 
         $r = $this->getRenderer($target);
         $dump = $r->wrapContent($this->renderValue($r, $value));
 
-        switch ($target) {
+        switch ($target)
+        {
             case self::OUTPUT:
                 echo $dump;
                 break;
@@ -93,7 +95,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);
@@ -114,7 +117,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));
         }
 
@@ -131,19 +135,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]();
         }
 
@@ -166,28 +176,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 ';
 
@@ -201,7 +218,8 @@ 
         $header .= $r->apply("{$type}({$length})", 'type', $type);
 
         $element = null;
-        switch ($type) {
+        switch ($type)
+        {
             case 'string':
                 $element = $r->escapeStrings() ? htmlspecialchars($value) : $value;
                 break;
@@ -211,7 +229,8 @@ 
                 break;
 
             default:
-                if ($value !== null) {
+                if ($value !== null)
+                {
                     //Not showing null value, type is enough
                     $element = var_export($value, true);
                 }
@@ -223,19 +242,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);
                 }
 
@@ -245,7 +270,8 @@ 
             $output .= $this->renderValue($r, $value, "[{$key}]", $level + 1);
         }
 
-        if (!$hideHeader) {
+        if (!$hideHeader)
+        {
             //Closing array scope
             $output .= $r->indent($level) . $r->apply(']', 'syntax', ']') . "\n";
         }
@@ -264,29 +290,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));
             }
@@ -299,7 +334,8 @@ 
         $refection = new \ReflectionObject($value);
 
         $output = '';
-        foreach ($refection->getProperties() as $property) {
+        foreach ($refection->getProperties() as $property)
+        {
             $output .= $this->renderProperty($r, $value, $property, $level);
         }
 
@@ -313,7 +349,8 @@ 
      */
     private function renderProperty(RendererInterface $r, $value, \ReflectionProperty $p, int $level): string
     {
-        if ($p->isStatic()) {
+        if ($p->isStatic())
+        {
             return '';
         }
 
@@ -333,9 +370,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);
         }
@@ -348,9 +388,12 @@ 
      */
     private function describeClosure(\Closure $closure): array
     {
-        try {
+        try
+        {
             $r = new \ReflectionFunction($closure);
-        } catch (\ReflectionException $e) {
+        }
+        catch (\ReflectionException $e)
+        {
             return ['closure' => 'unable to resolve'];
         }
 
@@ -368,9 +411,12 @@ 
      */
     private function getAccess(\ReflectionProperty $p): string
     {
-        if ($p->isPrivate()) {
+        if ($p->isPrivate())
+        {
             return 'private';
-        } elseif ($p->isProtected()) {
+        }
+        elseif ($p->isProtected())
+        {
             return 'protected';
         }
 

src/Validation/src/CheckerRule.php

Spacing +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
         string $method,
         array $args = [],
         ?string $message = null
-    ) {
+    ){
         $this->checker = $checker;
         $this->method = $method;
         $this->args = $args;
@@ -69,7 +69,7 @@ 
      */
     public function getMessage(string $field, $value): string
     {
-        if (!empty($this->message)) {
+        if (!empty($this->message)){
             return $this->say(
                 $this->message,
                 array_merge([$value, $field], $this->args)

Braces +2 added lines, -1 removed lines

@@ -69,7 +69,8 @@
      */
     public function getMessage(string $field, $value): string
     {
-        if (!empty($this->message)) {
+        if (!empty($this->message))
+        {
             return $this->say(
                 $this->message,
                 array_merge([$value, $field], $this->args)

src/Bridge/DataGrid/src/Specification/Sorter/InjectionSorter.php

Spacing +3 added lines, -3 removed lines

@@ -19,7 +19,7 @@ 
 
     public function __construct(SpecificationInterface $expression)
     {
-        if (!$expression instanceof AbstractSorter) {
+        if (!$expression instanceof AbstractSorter){
             throw new \LogicException('Only sorters allowed');
         }
 
@@ -33,13 +33,13 @@ 
     {
         $injector = static::INJECTION;
 
-        if (!class_exists($injector)) {
+        if (!class_exists($injector)){
             throw new \LogicException(
                 sprintf('Class "%s" does not exist', $injector)
             );
         }
 
-        if (!is_subclass_of($injector, FragmentInterface::class)) {
+        if (!is_subclass_of($injector, FragmentInterface::class)){
             throw new \LogicException(
                 'INJECTION class does not implement FragmentInterface'
             );

Braces +6 added lines, -3 removed lines

@@ -19,7 +19,8 @@ 
 
     public function __construct(SpecificationInterface $expression)
     {
-        if (!$expression instanceof AbstractSorter) {
+        if (!$expression instanceof AbstractSorter)
+        {
             throw new \LogicException('Only sorters allowed');
         }
 
@@ -33,13 +34,15 @@ 
     {
         $injector = static::INJECTION;
 
-        if (!class_exists($injector)) {
+        if (!class_exists($injector))
+        {
             throw new \LogicException(
                 sprintf('Class "%s" does not exist', $injector)
             );
         }
 
-        if (!is_subclass_of($injector, FragmentInterface::class)) {
+        if (!is_subclass_of($injector, FragmentInterface::class))
+        {
             throw new \LogicException(
                 'INJECTION class does not implement FragmentInterface'
             );

src/Bridge/DataGrid/src/Writer/QueryWriter.php

Spacing +24 added lines, -24 removed lines

@@ -53,23 +53,23 @@ 
      */
     public function write($source, SpecificationInterface $specification, Compiler $compiler)
     {
-        if (!$this->targetAcceptable($source)) {
+        if (!$this->targetAcceptable($source)){
             return null;
         }
 
-        if ($specification instanceof Specification\FilterInterface) {
+        if ($specification instanceof Specification\FilterInterface){
             return $this->writeFilter($source, $specification, $compiler);
         }
 
-        if ($specification instanceof Specification\SorterInterface) {
+        if ($specification instanceof Specification\SorterInterface){
             return $this->writeSorter($source, $specification, $compiler);
         }
 
-        if ($specification instanceof Specification\Pagination\Limit) {
+        if ($specification instanceof Specification\Pagination\Limit){
             return $source->limit($specification->getValue());
         }
 
-        if ($specification instanceof Specification\Pagination\Offset) {
+        if ($specification instanceof Specification\Pagination\Offset){
             return $source->offset($specification->getValue());
         }
 
@@ -84,15 +84,15 @@ 
      */
     protected function writeFilter($source, Specification\FilterInterface $filter, Compiler $compiler)
     {
-        if ($filter instanceof Specification\Filter\All || $filter instanceof Specification\Filter\Map) {
+        if ($filter instanceof Specification\Filter\All || $filter instanceof Specification\Filter\Map){
             return $source->where(static function () use ($compiler, $filter, $source): void {
                 $compiler->compile($source, ...$filter->getFilters());
             });
         }
 
-        if ($filter instanceof Specification\Filter\Any) {
+        if ($filter instanceof Specification\Filter\Any){
             return $source->where(static function () use ($compiler, $filter, $source): void {
-                foreach ($filter->getFilters() as $subFilter) {
+                foreach ($filter->getFilters() as $subFilter){
                     $source->orWhere(static function () use ($compiler, $subFilter, $source): void {
                         $compiler->compile($source, $subFilter);
                     });
@@ -100,9 +100,9 @@ 
             });
         }
 
-        if ($filter instanceof Specification\Filter\InjectionFilter) {
+        if ($filter instanceof Specification\Filter\InjectionFilter){
             $expression = $filter->getFilter();
-            if ($expression instanceof Specification\Filter\Expression) {
+            if ($expression instanceof Specification\Filter\Expression){
                 return $source->where(
                     $filter->getInjection(),
                     $this->getExpressionOperator($expression),
@@ -111,7 +111,7 @@ 
             }
         }
 
-        if ($filter instanceof Specification\Filter\Expression) {
+        if ($filter instanceof Specification\Filter\Expression){
             return $source->where(
                 $filter->getExpression(),
                 $this->getExpressionOperator($filter),
@@ -128,11 +128,11 @@ 
      */
     protected function getExpressionOperator(Specification\Filter\Expression $filter): string
     {
-        if ($filter instanceof Specification\Filter\Like) {
+        if ($filter instanceof Specification\Filter\Like){
             return 'LIKE';
         }
 
-        if ($filter instanceof Specification\Filter\InArray || $filter instanceof Specification\Filter\NotInArray) {
+        if ($filter instanceof Specification\Filter\InArray || $filter instanceof Specification\Filter\NotInArray){
             return static::ARRAY_OPERATORS[get_class($filter)];
         }
 
@@ -145,11 +145,11 @@ 
      */
     protected function getExpressionArgs(Specification\Filter\Expression $filter): array
     {
-        if ($filter instanceof Specification\Filter\Like) {
+        if ($filter instanceof Specification\Filter\Like){
             return [sprintf($filter->getPattern(), $this->fetchValue($filter->getValue()))];
         }
 
-        if ($filter instanceof Specification\Filter\InArray || $filter instanceof Specification\Filter\NotInArray) {
+        if ($filter instanceof Specification\Filter\InArray || $filter instanceof Specification\Filter\NotInArray){
             return [new Parameter($this->fetchValue($filter->getValue()))];
         }
 
@@ -164,8 +164,8 @@ 
      */
     protected function writeSorter($source, Specification\SorterInterface $sorter, Compiler $compiler)
     {
-        if ($sorter instanceof Specification\Sorter\SorterSet) {
-            foreach ($sorter->getSorters() as $subSorter) {
+        if ($sorter instanceof Specification\Sorter\SorterSet){
+            foreach ($sorter->getSorters() as $subSorter){
                 $source = $compiler->compile($source, $subSorter);
             }
 
@@ -175,18 +175,18 @@ 
         if (
             $sorter instanceof Specification\Sorter\AscSorter
             || $sorter instanceof Specification\Sorter\DescSorter
-        ) {
+        ){
             $direction = static::SORTER_DIRECTIONS[get_class($sorter)];
-            foreach ($sorter->getExpressions() as $expression) {
+            foreach ($sorter->getExpressions() as $expression){
                 $source = $source->orderBy($expression, $direction);
             }
 
             return $source;
         }
 
-        if ($sorter instanceof InjectionSorter) {
+        if ($sorter instanceof InjectionSorter){
             $direction = static::SORTER_DIRECTIONS[get_class($sorter)] ?? 'ASC';
-            foreach ($sorter->getInjections() as $injection) {
+            foreach ($sorter->getInjections() as $injection){
                 $source = $source->orderBy($injection, $direction);
             }
 
@@ -204,7 +204,7 @@ 
      */
     protected function fetchValue($value)
     {
-        if ($value instanceof Specification\ValueInterface) {
+        if ($value instanceof Specification\ValueInterface){
             throw new CompilerException('Value expects user input, none given');
         }
 
@@ -217,11 +217,11 @@ 
      */
     protected function targetAcceptable($target): bool
     {
-        if (class_exists(SelectQuery::class) && $target instanceof SelectQuery) {
+        if (class_exists(SelectQuery::class) && $target instanceof SelectQuery){
             return true;
         }
 
-        if (class_exists(Select::class) && $target instanceof Select) {
+        if (class_exists(Select::class) && $target instanceof Select){
             return true;
         }
 

Braces +46 added lines, -23 removed lines

@@ -53,23 +53,28 @@ 
      */
     public function write($source, SpecificationInterface $specification, Compiler $compiler)
     {
-        if (!$this->targetAcceptable($source)) {
+        if (!$this->targetAcceptable($source))
+        {
             return null;
         }
 
-        if ($specification instanceof Specification\FilterInterface) {
+        if ($specification instanceof Specification\FilterInterface)
+        {
             return $this->writeFilter($source, $specification, $compiler);
         }
 
-        if ($specification instanceof Specification\SorterInterface) {
+        if ($specification instanceof Specification\SorterInterface)
+        {
             return $this->writeSorter($source, $specification, $compiler);
         }
 
-        if ($specification instanceof Specification\Pagination\Limit) {
+        if ($specification instanceof Specification\Pagination\Limit)
+        {
             return $source->limit($specification->getValue());
         }
 
-        if ($specification instanceof Specification\Pagination\Offset) {
+        if ($specification instanceof Specification\Pagination\Offset)
+        {
             return $source->offset($specification->getValue());
         }
 
@@ -84,15 +89,18 @@ 
      */
     protected function writeFilter($source, Specification\FilterInterface $filter, Compiler $compiler)
     {
-        if ($filter instanceof Specification\Filter\All || $filter instanceof Specification\Filter\Map) {
+        if ($filter instanceof Specification\Filter\All || $filter instanceof Specification\Filter\Map)
+        {
             return $source->where(static function () use ($compiler, $filter, $source): void {
                 $compiler->compile($source, ...$filter->getFilters());
             });
         }
 
-        if ($filter instanceof Specification\Filter\Any) {
+        if ($filter instanceof Specification\Filter\Any)
+        {
             return $source->where(static function () use ($compiler, $filter, $source): void {
-                foreach ($filter->getFilters() as $subFilter) {
+                foreach ($filter->getFilters() as $subFilter)
+                {
                     $source->orWhere(static function () use ($compiler, $subFilter, $source): void {
                         $compiler->compile($source, $subFilter);
                     });
@@ -100,9 +108,11 @@ 
             });
         }
 
-        if ($filter instanceof Specification\Filter\InjectionFilter) {
+        if ($filter instanceof Specification\Filter\InjectionFilter)
+        {
             $expression = $filter->getFilter();
-            if ($expression instanceof Specification\Filter\Expression) {
+            if ($expression instanceof Specification\Filter\Expression)
+            {
                 return $source->where(
                     $filter->getInjection(),
                     $this->getExpressionOperator($expression),
@@ -111,7 +121,8 @@ 
             }
         }
 
-        if ($filter instanceof Specification\Filter\Expression) {
+        if ($filter instanceof Specification\Filter\Expression)
+        {
             return $source->where(
                 $filter->getExpression(),
                 $this->getExpressionOperator($filter),
@@ -128,11 +139,13 @@ 
      */
     protected function getExpressionOperator(Specification\Filter\Expression $filter): string
     {
-        if ($filter instanceof Specification\Filter\Like) {
+        if ($filter instanceof Specification\Filter\Like)
+        {
             return 'LIKE';
         }
 
-        if ($filter instanceof Specification\Filter\InArray || $filter instanceof Specification\Filter\NotInArray) {
+        if ($filter instanceof Specification\Filter\InArray || $filter instanceof Specification\Filter\NotInArray)
+        {
             return static::ARRAY_OPERATORS[get_class($filter)];
         }
 
@@ -145,11 +158,13 @@ 
      */
     protected function getExpressionArgs(Specification\Filter\Expression $filter): array
     {
-        if ($filter instanceof Specification\Filter\Like) {
+        if ($filter instanceof Specification\Filter\Like)
+        {
             return [sprintf($filter->getPattern(), $this->fetchValue($filter->getValue()))];
         }
 
-        if ($filter instanceof Specification\Filter\InArray || $filter instanceof Specification\Filter\NotInArray) {
+        if ($filter instanceof Specification\Filter\InArray || $filter instanceof Specification\Filter\NotInArray)
+        {
             return [new Parameter($this->fetchValue($filter->getValue()))];
         }
 
@@ -164,8 +179,10 @@ 
      */
     protected function writeSorter($source, Specification\SorterInterface $sorter, Compiler $compiler)
     {
-        if ($sorter instanceof Specification\Sorter\SorterSet) {
-            foreach ($sorter->getSorters() as $subSorter) {
+        if ($sorter instanceof Specification\Sorter\SorterSet)
+        {
+            foreach ($sorter->getSorters() as $subSorter)
+            {
                 $source = $compiler->compile($source, $subSorter);
             }
 
@@ -177,16 +194,19 @@ 
             || $sorter instanceof Specification\Sorter\DescSorter
         ) {
             $direction = static::SORTER_DIRECTIONS[get_class($sorter)];
-            foreach ($sorter->getExpressions() as $expression) {
+            foreach ($sorter->getExpressions() as $expression)
+            {
                 $source = $source->orderBy($expression, $direction);
             }
 
             return $source;
         }
 
-        if ($sorter instanceof InjectionSorter) {
+        if ($sorter instanceof InjectionSorter)
+        {
             $direction = static::SORTER_DIRECTIONS[get_class($sorter)] ?? 'ASC';
-            foreach ($sorter->getInjections() as $injection) {
+            foreach ($sorter->getInjections() as $injection)
+            {
                 $source = $source->orderBy($injection, $direction);
             }
 
@@ -204,7 +224,8 @@ 
      */
     protected function fetchValue($value)
     {
-        if ($value instanceof Specification\ValueInterface) {
+        if ($value instanceof Specification\ValueInterface)
+        {
             throw new CompilerException('Value expects user input, none given');
         }
 
@@ -217,11 +238,13 @@ 
      */
     protected function targetAcceptable($target): bool
     {
-        if (class_exists(SelectQuery::class) && $target instanceof SelectQuery) {
+        if (class_exists(SelectQuery::class) && $target instanceof SelectQuery)
+        {
             return true;
         }
 
-        if (class_exists(Select::class) && $target instanceof Select) {
+        if (class_exists(Select::class) && $target instanceof Select)
+        {
             return true;
         }
 

src/SendIt/tests/RenderTest.php

Spacing +3 added lines, -3 removed lines

@@ -26,14 +26,14 @@
     public function setUp(): void
     {
         $this->app = App::init([
-            'root' => __DIR__ . '/App',
-            'app'  => __DIR__ . '/App'
+            'root' => __DIR__.'/App',
+            'app'  => __DIR__.'/App'
         ]);
     }
 
     public function tearDown(): void
     {
-        foreach (glob(__DIR__ . '/App/runtime/cache/views/*.php') as $file) {
+        foreach (glob(__DIR__.'/App/runtime/cache/views/*.php') as $file){
             @unlink($file);
         }
     }

Braces +2 added lines, -1 removed lines

@@ -33,7 +33,8 @@
 
     public function tearDown(): void
     {
-        foreach (glob(__DIR__ . '/App/runtime/cache/views/*.php') as $file) {
+        foreach (glob(__DIR__ . '/App/runtime/cache/views/*.php') as $file)
+        {
             @unlink($file);
         }
     }

src/Attributes/src/Psr6CachedReader.php

Spacing +2 added lines, -2 removed lines

@@ -32,7 +32,7 @@ 
         ReaderInterface $reader,
         CacheItemPoolInterface $cache,
         KeyGeneratorInterface $generator = null
-    ) {
+    ){
         $this->cache = $cache;
 
         parent::__construct($reader, $generator);
@@ -46,7 +46,7 @@ 
     {
         $item = $this->cache->getItem($key);
 
-        if (!$item->isHit()) {
+        if (!$item->isHit()){
             $this->cache->save(
                 $item->set($then())
             );

Braces +2 added lines, -1 removed lines

@@ -46,7 +46,8 @@
     {
         $item = $this->cache->getItem($key);
 
-        if (!$item->isHit()) {
+        if (!$item->isHit())
+        {
             $this->cache->save(
                 $item->set($then())
             );

src/Attributes/src/Psr16CachedReader.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
      */
     protected function cached(string $key, callable $then): iterable
     {
-        if (!$this->cache->has($key)) {
+        if (!$this->cache->has($key)){
             $this->cache->set($key, $then());
         }
 

Braces +2 added lines, -1 removed lines

@@ -41,7 +41,8 @@
      */
     protected function cached(string $key, callable $then): iterable
     {
-        if (!$this->cache->has($key)) {
+        if (!$this->cache->has($key))
+        {
             $this->cache->set($key, $then());
         }
 

src/Attributes/src/Internal/Instantiator/NamedArgumentsInstantiator.php

Spacing +13 added lines, -13 removed lines

@@ -34,17 +34,17 @@ 
      */
     public function instantiate(\ReflectionClass $attr, array $arguments, \Reflector $context = null): object
     {
-        if ($this->isNamedArgumentsSupported()) {
-            try {
+        if ($this->isNamedArgumentsSupported()){
+            try{
                 return $attr->newInstanceArgs($arguments);
-            } catch (\Throwable $e) {
+            }catch (\Throwable $e){
                 throw Exception::withLocation($e, $attr->getFileName(), $attr->getStartLine());
             }
         }
 
         $constructor = $this->getConstructor($attr);
 
-        if ($constructor === null) {
+        if ($constructor === null){
             return $attr->newInstanceWithoutConstructor();
         }
 
@@ -72,16 +72,16 @@ 
     {
         $passed = [];
 
-        try {
-            foreach ($constructor->getParameters() as $parameter) {
+        try{
+            foreach ($constructor->getParameters() as $parameter){
                 $passed[] = $this->resolveParameter($ctx, $parameter, $arguments);
             }
 
-            if (\count($arguments)) {
+            if (\count($arguments)){
                 $message = \sprintf(self::ERROR_UNKNOWN_ARGUMENT, \array_key_first($arguments));
                 throw new \BadMethodCallException($message);
             }
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             throw Exception::withLocation($e, $constructor->getFileName(), $constructor->getStartLine());
         }
 
@@ -97,19 +97,19 @@ 
      */
     private function resolveParameter(\ReflectionClass $ctx, \ReflectionParameter $param, array &$arguments)
     {
-        switch (true) {
+        switch (true){
             case \array_key_exists($param->getName(), $arguments):
-                try {
+                try{
                     return $arguments[$param->getName()];
-                } finally {
+                }finally{
                     unset($arguments[$param->getName()]);
                 }
                 // no actual falling through
 
             case \array_key_exists($param->getPosition(), $arguments):
-                try {
+                try{
                     return $arguments[$param->getPosition()];
-                } finally {
+                }finally{
                     unset($arguments[$param->getPosition()]);
                 }
                 // no actual falling through

Braces +30 added lines, -13 removed lines

@@ -34,17 +34,22 @@ 
      */
     public function instantiate(\ReflectionClass $attr, array $arguments, \Reflector $context = null): object
     {
-        if ($this->isNamedArgumentsSupported()) {
-            try {
+        if ($this->isNamedArgumentsSupported())
+        {
+            try
+            {
                 return $attr->newInstanceArgs($arguments);
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
                 throw Exception::withLocation($e, $attr->getFileName(), $attr->getStartLine());
             }
         }
 
         $constructor = $this->getConstructor($attr);
 
-        if ($constructor === null) {
+        if ($constructor === null)
+        {
             return $attr->newInstanceWithoutConstructor();
         }
 
@@ -72,16 +77,21 @@ 
     {
         $passed = [];
 
-        try {
-            foreach ($constructor->getParameters() as $parameter) {
+        try
+        {
+            foreach ($constructor->getParameters() as $parameter)
+            {
                 $passed[] = $this->resolveParameter($ctx, $parameter, $arguments);
             }
 
-            if (\count($arguments)) {
+            if (\count($arguments))
+            {
                 $message = \sprintf(self::ERROR_UNKNOWN_ARGUMENT, \array_key_first($arguments));
                 throw new \BadMethodCallException($message);
             }
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             throw Exception::withLocation($e, $constructor->getFileName(), $constructor->getStartLine());
         }
 
@@ -97,19 +107,26 @@ 
      */
     private function resolveParameter(\ReflectionClass $ctx, \ReflectionParameter $param, array &$arguments)
     {
-        switch (true) {
+        switch (true)
+        {
             case \array_key_exists($param->getName(), $arguments):
-                try {
+                try
+                {
                     return $arguments[$param->getName()];
-                } finally {
+                }
+                finally
+                {
                     unset($arguments[$param->getName()]);
                 }
                 // no actual falling through
 
             case \array_key_exists($param->getPosition(), $arguments):
-                try {
+                try
+                {
                     return $arguments[$param->getPosition()];
-                } finally {
+                }
+                finally
+                {
                     unset($arguments[$param->getPosition()]);
                 }
                 // no actual falling through

src/Attributes/src/Internal/CachedReader.php

Spacing +1 added lines, -1 removed lines

@@ -124,7 +124,7 @@
      */
     protected function iterableToArray(iterable $attributes): array
     {
-        if ($attributes instanceof \Traversable) {
+        if ($attributes instanceof \Traversable){
             return \iterator_to_array($attributes, false);
         }
 

Braces +12 added lines, -6 removed lines

@@ -41,7 +41,8 @@ 
      */
     public function getClassMetadata(\ReflectionClass $class, string $name = null): iterable
     {
-        $result = $this->cached($this->key->forClass($class), function () use ($class) {
+        $result = $this->cached($this->key->forClass($class), function () use ($class)
+        {
             return $this->iterableToArray(parent::getClassMetadata($class));
         });
 
@@ -53,7 +54,8 @@ 
      */
     public function getFunctionMetadata(\ReflectionFunctionAbstract $function, string $name = null): iterable
     {
-        $result = $this->cached($this->key->forFunction($function), function () use ($function) {
+        $result = $this->cached($this->key->forFunction($function), function () use ($function)
+        {
             return $this->iterableToArray(parent::getFunctionMetadata($function));
         });
 
@@ -65,7 +67,8 @@ 
      */
     public function getPropertyMetadata(\ReflectionProperty $property, string $name = null): iterable
     {
-        $result = $this->cached($this->key->forProperty($property), function () use ($property) {
+        $result = $this->cached($this->key->forProperty($property), function () use ($property)
+        {
             return $this->iterableToArray(parent::getPropertyMetadata($property));
         });
 
@@ -77,7 +80,8 @@ 
      */
     public function getConstantMetadata(\ReflectionClassConstant $constant, string $name = null): iterable
     {
-        $result = $this->cached($this->key->forConstant($constant), function () use ($constant) {
+        $result = $this->cached($this->key->forConstant($constant), function () use ($constant)
+        {
             return $this->iterableToArray(parent::getConstantMetadata($constant));
         });
 
@@ -89,7 +93,8 @@ 
      */
     public function getParameterMetadata(\ReflectionParameter $parameter, string $name = null): iterable
     {
-        $result = $this->cached($this->key->forParameter($parameter), function () use ($parameter) {
+        $result = $this->cached($this->key->forParameter($parameter), function () use ($parameter)
+        {
             return $this->iterableToArray(parent::getParameterMetadata($parameter));
         });
 
@@ -124,7 +129,8 @@ 
      */
     protected function iterableToArray(iterable $attributes): array
     {
-        if ($attributes instanceof \Traversable) {
+        if ($attributes instanceof \Traversable)
+        {
             return \iterator_to_array($attributes, false);
         }