spiral / framework

src/Bridge/DataGrid/src/Interceptor/GridInterceptor.php

Braces +27 added lines, -13 removed lines

@@ -72,14 +72,17 @@ 
     {
         $result = $core->callAction($controller, $action, $parameters);
 
-        if (is_iterable($result)) {
+        if (is_iterable($result))
+        {
             $schema = $this->getSchema($controller, $action);
-            if ($schema !== null) {
+            if ($schema !== null)
+            {
                 $grid = $this->makeFactory($schema)
                     ->withDefaults($schema['defaults'])
                     ->create($result, $schema['grid']);
 
-                if ($schema['view'] !== null) {
+                if ($schema['view'] !== null)
+                {
                     $grid = $grid->withView($schema['view']);
                 }
 
@@ -99,20 +102,25 @@ 
     private function getSchema(string $controller, string $action): ?array
     {
         $key = sprintf('%s:%s', $controller, $action);
-        if (array_key_exists($key, $this->cache)) {
+        if (array_key_exists($key, $this->cache))
+        {
             return $this->cache[$key];
         }
 
         $this->cache[$key] = null;
-        try {
+        try
+        {
             $method = new \ReflectionMethod($controller, $action);
-        } catch (\ReflectionException $e) {
+        }
+        catch (\ReflectionException $e)
+        {
             return null;
         }
 
         /** @var null|DataGrid $dataGrid */
         $dataGrid = $this->reader->firstFunctionMetadata($method, DataGrid::class);
-        if ($dataGrid === null) {
+        if ($dataGrid === null)
+        {
             return null;
         }
 
@@ -133,19 +141,23 @@ 
             'factory'  => $dataGrid->factory,
         ];
 
-        if (is_string($schema['view'])) {
+        if (is_string($schema['view']))
+        {
             $schema['view'] = $this->container->get($schema['view']);
         }
 
-        if ($schema['defaults'] === [] && method_exists($schema['grid'], 'getDefaults')) {
+        if ($schema['defaults'] === [] && method_exists($schema['grid'], 'getDefaults'))
+        {
             $schema['defaults'] = $schema['grid']->getDefaults();
         }
 
-        if ($schema['options'] === [] && method_exists($schema['grid'], 'getOptions')) {
+        if ($schema['options'] === [] && method_exists($schema['grid'], 'getOptions'))
+        {
             $schema['options'] = $schema['grid']->getOptions();
         }
 
-        if ($schema['view'] === null && is_callable($schema['grid'])) {
+        if ($schema['view'] === null && is_callable($schema['grid']))
+        {
             $schema['view'] = $schema['grid'];
         }
 
@@ -154,9 +166,11 @@ 
 
     private function makeFactory(array $schema): GridFactoryInterface
     {
-        if (!empty($schema['factory']) && $this->container->has($schema['factory'])) {
+        if (!empty($schema['factory']) && $this->container->has($schema['factory']))
+        {
             $factory = $this->container->get($schema['factory']);
-            if ($factory instanceof GridFactoryInterface) {
+            if ($factory instanceof GridFactoryInterface)
+            {
                 return $factory;
             }
         }

Spacing +14 added lines, -14 removed lines

@@ -52,7 +52,7 @@ 
         ContainerInterface $container,
         GridFactory $gridFactory,
         ReaderInterface $reader
-    ) {
+    ){
         $this->response = $response;
         $this->container = $container;
         $this->gridFactory = $gridFactory;
@@ -71,14 +71,14 @@ 
     {
         $result = $core->callAction($controller, $action, $parameters);
 
-        if (is_iterable($result)) {
+        if (is_iterable($result)){
             $schema = $this->getSchema($controller, $action);
-            if ($schema !== null) {
+            if ($schema !== null){
                 $grid = $this->makeFactory($schema)
                     ->withDefaults($schema['defaults'])
                     ->create($result, $schema['grid']);
 
-                if ($schema['view'] !== null) {
+                if ($schema['view'] !== null){
                     $grid = $grid->withView($schema['view']);
                 }
 
@@ -98,20 +98,20 @@ 
     private function getSchema(string $controller, string $action): ?array
     {
         $key = sprintf('%s:%s', $controller, $action);
-        if (array_key_exists($key, $this->cache)) {
+        if (array_key_exists($key, $this->cache)){
             return $this->cache[$key];
         }
 
         $this->cache[$key] = null;
-        try {
+        try{
             $method = new \ReflectionMethod($controller, $action);
-        } catch (\ReflectionException $e) {
+        }catch (\ReflectionException $e){
             return null;
         }
 
         /** @var null|DataGrid $dataGrid */
         $dataGrid = $this->reader->firstFunctionMetadata($method, DataGrid::class);
-        if ($dataGrid === null) {
+        if ($dataGrid === null){
             return null;
         }
 
@@ -132,19 +132,19 @@ 
             'factory'  => $dataGrid->factory,
         ];
 
-        if (is_string($schema['view'])) {
+        if (is_string($schema['view'])){
             $schema['view'] = $this->container->get($schema['view']);
         }
 
-        if ($schema['defaults'] === [] && method_exists($schema['grid'], 'getDefaults')) {
+        if ($schema['defaults'] === [] && method_exists($schema['grid'], 'getDefaults')){
             $schema['defaults'] = $schema['grid']->getDefaults();
         }
 
-        if ($schema['options'] === [] && method_exists($schema['grid'], 'getOptions')) {
+        if ($schema['options'] === [] && method_exists($schema['grid'], 'getOptions')){
             $schema['options'] = $schema['grid']->getOptions();
         }
 
-        if ($schema['view'] === null && is_callable($schema['grid'])) {
+        if ($schema['view'] === null && is_callable($schema['grid'])){
             $schema['view'] = $schema['grid'];
         }
 
@@ -153,9 +153,9 @@ 
 
     private function makeFactory(array $schema): GridFactoryInterface
     {
-        if (!empty($schema['factory']) && $this->container->has($schema['factory'])) {
+        if (!empty($schema['factory']) && $this->container->has($schema['factory'])){
             $factory = $this->container->get($schema['factory']);
-            if ($factory instanceof GridFactoryInterface) {
+            if ($factory instanceof GridFactoryInterface){
                 return $factory;
             }
         }

src/Bridge/DataGrid/src/Annotation/DataGrid.php

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
         array $defaults = [],
         array $options = [],
         ?string $factory = null
-    ) {
+    ){
         $this->grid = $grid;
         $this->view = $view;
         $this->defaults = $defaults;

src/Annotations/src/AnnotationLocator.php

Spacing +10 added lines, -10 removed lines

@@ -67,9 +67,9 @@ 
      */
     public function findClasses(string $annotation): iterable
     {
-        foreach ($this->getTargets() as $target) {
+        foreach ($this->getTargets() as $target){
             $found = $this->reader->firstClassMetadata($target, $annotation);
-            if ($found !== null) {
+            if ($found !== null){
                 yield new AnnotatedClass($target, $found);
             }
         }
@@ -83,10 +83,10 @@ 
      */
     public function findMethods(string $annotation): iterable
     {
-        foreach ($this->getTargets() as $target) {
-            foreach ($target->getMethods() as $method) {
+        foreach ($this->getTargets() as $target){
+            foreach ($target->getMethods() as $method){
                 $found = $this->reader->firstFunctionMetadata($method, $annotation);
-                if ($found !== null) {
+                if ($found !== null){
                     yield new AnnotatedMethod($method, $found);
                 }
             }
@@ -101,10 +101,10 @@ 
      */
     public function findProperties(string $annotation): iterable
     {
-        foreach ($this->getTargets() as $target) {
-            foreach ($target->getProperties() as $property) {
+        foreach ($this->getTargets() as $target){
+            foreach ($target->getProperties() as $property){
                 $found = $this->reader->firstPropertyMetadata($property, $annotation);
-                if ($found !== null) {
+                if ($found !== null){
                     yield new AnnotatedProperty($property, $found);
                 }
             }
@@ -116,12 +116,12 @@ 
      */
     private function getTargets(): iterable
     {
-        if ($this->targets === []) {
+        if ($this->targets === []){
             yield from $this->classLocator->getClasses();
             return;
         }
 
-        foreach ($this->targets as $target) {
+        foreach ($this->targets as $target){
             yield from $this->classLocator->getClasses($target);
         }
     }

Braces +20 added lines, -10 removed lines

@@ -67,9 +67,11 @@ 
      */
     public function findClasses(string $annotation): iterable
     {
-        foreach ($this->getTargets() as $target) {
+        foreach ($this->getTargets() as $target)
+        {
             $found = $this->reader->firstClassMetadata($target, $annotation);
-            if ($found !== null) {
+            if ($found !== null)
+            {
                 yield new AnnotatedClass($target, $found);
             }
         }
@@ -83,10 +85,13 @@ 
      */
     public function findMethods(string $annotation): iterable
     {
-        foreach ($this->getTargets() as $target) {
-            foreach ($target->getMethods() as $method) {
+        foreach ($this->getTargets() as $target)
+        {
+            foreach ($target->getMethods() as $method)
+            {
                 $found = $this->reader->firstFunctionMetadata($method, $annotation);
-                if ($found !== null) {
+                if ($found !== null)
+                {
                     yield new AnnotatedMethod($method, $found);
                 }
             }
@@ -101,10 +106,13 @@ 
      */
     public function findProperties(string $annotation): iterable
     {
-        foreach ($this->getTargets() as $target) {
-            foreach ($target->getProperties() as $property) {
+        foreach ($this->getTargets() as $target)
+        {
+            foreach ($target->getProperties() as $property)
+            {
                 $found = $this->reader->firstPropertyMetadata($property, $annotation);
-                if ($found !== null) {
+                if ($found !== null)
+                {
                     yield new AnnotatedProperty($property, $found);
                 }
             }
@@ -116,12 +124,14 @@ 
      */
     private function getTargets(): iterable
     {
-        if ($this->targets === []) {
+        if ($this->targets === [])
+        {
             yield from $this->classLocator->getClasses();
             return;
         }
 
-        foreach ($this->targets as $target) {
+        foreach ($this->targets as $target)
+        {
             yield from $this->classLocator->getClasses($target);
         }
     }

src/Annotations/tests/LocatorTest.php

Spacing +12 added lines, -12 removed lines

@@ -29,23 +29,23 @@ 
     public function testLocateClasses()
     {
         // Annotations.
-        $classes = $this->getAnnotationsLocator(__DIR__ . '/Fixtures')->findClasses(ClassAnnotation::class);
+        $classes = $this->getAnnotationsLocator(__DIR__.'/Fixtures')->findClasses(ClassAnnotation::class);
         $classes = iterator_to_array($classes);
 
         $this->assertCount(1, $classes);
 
-        foreach ($classes as $class) {
+        foreach ($classes as $class){
             $this->assertSame(TestClass::class, $class->getClass()->getName());
             $this->assertSame('abc', $class->getAnnotation()->value);
         }
 
         // Attributes.
-        $classes = $this->getAttributesLocator(__DIR__ . '/Fixtures')->findClasses(ClassAnnotation::class);
+        $classes = $this->getAttributesLocator(__DIR__.'/Fixtures')->findClasses(ClassAnnotation::class);
         $classes = iterator_to_array($classes);
 
         $this->assertCount(1, $classes);
 
-        foreach ($classes as $class) {
+        foreach ($classes as $class){
             $this->assertSame(AttributeTestClass::class, $class->getClass()->getName());
             $this->assertSame('abc', $class->getAnnotation()->value);
         }
@@ -54,24 +54,24 @@ 
     public function testLocateProperties()
     {
         // Annotations.
-        $props = $this->getAnnotationsLocator(__DIR__ . '/Fixtures')->findProperties(PropertyAnnotation::class);
+        $props = $this->getAnnotationsLocator(__DIR__.'/Fixtures')->findProperties(PropertyAnnotation::class);
         $props = iterator_to_array($props);
 
         $this->assertCount(1, $props);
 
-        foreach ($props as $prop) {
+        foreach ($props as $prop){
             $this->assertSame(TestClass::class, $prop->getClass()->getName());
             $this->assertSame('name', $prop->getProperty()->getName());
             $this->assertSame('123', $prop->getAnnotation()->id);
         }
 
         // Attributes.
-        $props = $this->getAttributesLocator(__DIR__ . '/Fixtures')->findProperties(PropertyAnnotation::class);
+        $props = $this->getAttributesLocator(__DIR__.'/Fixtures')->findProperties(PropertyAnnotation::class);
         $props = iterator_to_array($props);
 
         $this->assertCount(1, $props);
 
-        foreach ($props as $prop) {
+        foreach ($props as $prop){
             $this->assertSame(AttributeTestClass::class, $prop->getClass()->getName());
             $this->assertSame('name', $prop->getProperty()->getName());
             $this->assertSame('123', $prop->getAnnotation()->id);
@@ -81,24 +81,24 @@ 
     public function testLocateMethods()
     {
         // Annotations.
-        $methods = $this->getAnnotationsLocator(__DIR__ . '/Fixtures')->findMethods(MethodAnnotation::class);
+        $methods = $this->getAnnotationsLocator(__DIR__.'/Fixtures')->findMethods(MethodAnnotation::class);
         $methods = iterator_to_array($methods);
 
         $this->assertCount(1, $methods);
 
-        foreach ($methods as $m) {
+        foreach ($methods as $m){
             $this->assertSame(TestClass::class, $m->getClass()->getName());
             $this->assertSame('testMethod', $m->getMethod()->getName());
             $this->assertSame('/', $m->getAnnotation()->path);
         }
 
         // Attributes.
-        $methods = $this->getAttributesLocator(__DIR__ . '/Fixtures')->findMethods(MethodAnnotation::class);
+        $methods = $this->getAttributesLocator(__DIR__.'/Fixtures')->findMethods(MethodAnnotation::class);
         $methods = iterator_to_array($methods);
 
         $this->assertCount(1, $methods);
 
-        foreach ($methods as $m) {
+        foreach ($methods as $m){
             $this->assertSame(AttributeTestClass::class, $m->getClass()->getName());
             $this->assertSame('testMethod', $m->getMethod()->getName());
             $this->assertSame('/', $m->getAnnotation()->path);

Braces +12 added lines, -6 removed lines

@@ -34,7 +34,8 @@ 
 
         $this->assertCount(1, $classes);
 
-        foreach ($classes as $class) {
+        foreach ($classes as $class)
+        {
             $this->assertSame(TestClass::class, $class->getClass()->getName());
             $this->assertSame('abc', $class->getAnnotation()->value);
         }
@@ -45,7 +46,8 @@ 
 
         $this->assertCount(1, $classes);
 
-        foreach ($classes as $class) {
+        foreach ($classes as $class)
+        {
             $this->assertSame(AttributeTestClass::class, $class->getClass()->getName());
             $this->assertSame('abc', $class->getAnnotation()->value);
         }
@@ -59,7 +61,8 @@ 
 
         $this->assertCount(1, $props);
 
-        foreach ($props as $prop) {
+        foreach ($props as $prop)
+        {
             $this->assertSame(TestClass::class, $prop->getClass()->getName());
             $this->assertSame('name', $prop->getProperty()->getName());
             $this->assertSame('123', $prop->getAnnotation()->id);
@@ -71,7 +74,8 @@ 
 
         $this->assertCount(1, $props);
 
-        foreach ($props as $prop) {
+        foreach ($props as $prop)
+        {
             $this->assertSame(AttributeTestClass::class, $prop->getClass()->getName());
             $this->assertSame('name', $prop->getProperty()->getName());
             $this->assertSame('123', $prop->getAnnotation()->id);
@@ -86,7 +90,8 @@ 
 
         $this->assertCount(1, $methods);
 
-        foreach ($methods as $m) {
+        foreach ($methods as $m)
+        {
             $this->assertSame(TestClass::class, $m->getClass()->getName());
             $this->assertSame('testMethod', $m->getMethod()->getName());
             $this->assertSame('/', $m->getAnnotation()->path);
@@ -98,7 +103,8 @@ 
 
         $this->assertCount(1, $methods);
 
-        foreach ($methods as $m) {
+        foreach ($methods as $m)
+        {
             $this->assertSame(AttributeTestClass::class, $m->getClass()->getName());
             $this->assertSame('testMethod', $m->getMethod()->getName());
             $this->assertSame('/', $m->getAnnotation()->path);

tests/app/src/Controller/DemoController.php

Braces +3 added lines, -1 removed lines

@@ -49,10 +49,12 @@
      * @Guarded("do")
      * @return string
      */
-    public function do()
+    public function do{
+        ()
     {
         return 'ok';
     }
+    }
 
     /**
      * @return string

src/Validation/src/Checker/ArrayChecker.php

Spacing +7 added lines, -7 removed lines

@@ -29,12 +29,12 @@ 
 
     public function of($value, $checker): bool
     {
-        if (!is_array($value) || empty($value)) {
+        if (!is_array($value) || empty($value)){
             return false;
         }
 
-        foreach ($value as $item) {
-            if (!$this->validation->validate(compact('item'), ['item' => [$checker]])->isValid()) {
+        foreach ($value as $item){
+            if (!$this->validation->validate(compact('item'), ['item' => [$checker]])->isValid()){
                 return false;
             }
         }
@@ -44,7 +44,7 @@ 
 
     public function count($value, int $length): bool
     {
-        if (!is_array($value) && !$value instanceof \Countable) {
+        if (!is_array($value) && !$value instanceof \Countable){
             return false;
         }
 
@@ -53,7 +53,7 @@ 
 
     public function shorter($value, int $length): bool
     {
-        if (!is_array($value) && !$value instanceof \Countable) {
+        if (!is_array($value) && !$value instanceof \Countable){
             return false;
         }
 
@@ -62,7 +62,7 @@ 
 
     public function longer($value, int $length): bool
     {
-        if (!is_array($value) && !$value instanceof \Countable) {
+        if (!is_array($value) && !$value instanceof \Countable){
             return false;
         }
 
@@ -71,7 +71,7 @@ 
 
     public function range($value, int $min, int $max): bool
     {
-        if (!is_array($value) && !$value instanceof \Countable) {
+        if (!is_array($value) && !$value instanceof \Countable){
             return false;
         }
 

Braces +14 added lines, -7 removed lines

@@ -29,12 +29,15 @@ 
 
     public function of($value, $checker): bool
     {
-        if (!is_array($value) || empty($value)) {
+        if (!is_array($value) || empty($value))
+        {
             return false;
         }
 
-        foreach ($value as $item) {
-            if (!$this->validation->validate(compact('item'), ['item' => [$checker]])->isValid()) {
+        foreach ($value as $item)
+        {
+            if (!$this->validation->validate(compact('item'), ['item' => [$checker]])->isValid())
+            {
                 return false;
             }
         }
@@ -44,7 +47,8 @@ 
 
     public function count($value, int $length): bool
     {
-        if (!is_array($value) && !$value instanceof \Countable) {
+        if (!is_array($value) && !$value instanceof \Countable)
+        {
             return false;
         }
 
@@ -53,7 +57,8 @@ 
 
     public function shorter($value, int $length): bool
     {
-        if (!is_array($value) && !$value instanceof \Countable) {
+        if (!is_array($value) && !$value instanceof \Countable)
+        {
             return false;
         }
 
@@ -62,7 +67,8 @@ 
 
     public function longer($value, int $length): bool
     {
-        if (!is_array($value) && !$value instanceof \Countable) {
+        if (!is_array($value) && !$value instanceof \Countable)
+        {
             return false;
         }
 
@@ -71,7 +77,8 @@ 
 
     public function range($value, int $min, int $max): bool
     {
-        if (!is_array($value) && !$value instanceof \Countable) {
+        if (!is_array($value) && !$value instanceof \Countable)
+        {
             return false;
         }
 

src/Validation/tests/Checkers/ArrayTest.php

Spacing +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
 
     private function createCountable(int $count): \Countable
     {
-        return new class($count) implements \Countable {
+        return new class($count) implements \Countable{
             private $count;
 
             public function __construct(int $count)

Braces +2 added lines, -1 removed lines

@@ -79,7 +79,8 @@
 
     private function createCountable(int $count): \Countable
     {
-        return new class($count) implements \Countable {
+        return new class($count) implements \Countable
+        {
             private $count;
 
             public function __construct(int $count)

src/Config/src/Loader/DirectoryLoader.php

Spacing +6 added lines, -6 removed lines

@@ -37,9 +37,9 @@ 
      */
     public function has(string $section): bool
     {
-        foreach ($this->loaderExtensions() as $extension) {
+        foreach ($this->loaderExtensions() as $extension){
             $filename = sprintf('%s/%s.%s', $this->directory, $section, $extension);
-            if (file_exists($filename)) {
+            if (file_exists($filename)){
                 return true;
             }
         }
@@ -52,15 +52,15 @@ 
      */
     public function load(string $section): array
     {
-        foreach ($this->loaderExtensions() as $extension) {
+        foreach ($this->loaderExtensions() as $extension){
             $filename = sprintf('%s/%s.%s', $this->directory, $section, $extension);
-            if (!file_exists($filename)) {
+            if (!file_exists($filename)){
                 continue;
             }
 
-            try {
+            try{
                 return $this->getLoader($extension)->loadFile($section, $filename);
-            } catch (LoaderException $e) {
+            }catch (LoaderException $e){
                 throw new LoaderException("Unable to load config `{$section}`: {$e->getMessage()}", $e->getCode(), $e);
             }
         }

Braces +13 added lines, -6 removed lines

@@ -37,9 +37,11 @@ 
      */
     public function has(string $section): bool
     {
-        foreach ($this->loaderExtensions() as $extension) {
+        foreach ($this->loaderExtensions() as $extension)
+        {
             $filename = sprintf('%s/%s.%s', $this->directory, $section, $extension);
-            if (file_exists($filename)) {
+            if (file_exists($filename))
+            {
                 return true;
             }
         }
@@ -52,15 +54,20 @@ 
      */
     public function load(string $section): array
     {
-        foreach ($this->loaderExtensions() as $extension) {
+        foreach ($this->loaderExtensions() as $extension)
+        {
             $filename = sprintf('%s/%s.%s', $this->directory, $section, $extension);
-            if (!file_exists($filename)) {
+            if (!file_exists($filename))
+            {
                 continue;
             }
 
-            try {
+            try
+            {
                 return $this->getLoader($extension)->loadFile($section, $filename);
-            } catch (LoaderException $e) {
+            }
+            catch (LoaderException $e)
+            {
                 throw new LoaderException("Unable to load config `{$section}`: {$e->getMessage()}", $e->getCode(), $e);
             }
         }

src/Filters/src/SchemaBuilder.php

Spacing +10 added lines, -10 removed lines

@@ -70,18 +70,18 @@ 
     protected function buildMap(ReflectionEntity $filter): array
     {
         $schema = $filter->getProperty('schema', true);
-        if (empty($schema)) {
+        if (empty($schema)){
             throw new SchemaException("Filter `{$filter->getName()}` does not define any schema");
         }
 
         $result = [];
-        foreach ($schema as $field => $definition) {
+        foreach ($schema as $field => $definition){
             $optional = false;
 
             // short definition
-            if (is_string($definition)) {
+            if (is_string($definition)){
                 // simple scalar field definition
-                if (!class_exists($definition)) {
+                if (!class_exists($definition)){
                     [$source, $origin] = $this->parseDefinition($field, $definition);
                     $result[$field] = [
                         FilterProvider::SOURCE => $source,
@@ -102,25 +102,25 @@ 
                 continue;
             }
 
-            if (!is_array($definition) || count($definition) === 0) {
+            if (!is_array($definition) || count($definition) === 0){
                 throw new SchemaException(
                     "Invalid schema definition at `{$filter->getName()}`->`{$field}`"
                 );
             }
 
             // complex definition
-            if (!empty($definition[self::ORIGIN])) {
+            if (!empty($definition[self::ORIGIN])){
                 $origin = $definition[self::ORIGIN];
 
                 // [class, 'data:something.*'] vs [class, 'data:something']
                 $iterate = strpos($origin, '.*') !== false || !empty($definition[self::ITERATE]);
                 $origin = rtrim($origin, '.*');
-            } else {
+            }else{
                 $origin = $field;
                 $iterate = true;
             }
 
-            if (!empty($definition[self::OPTIONAL]) && $definition[self::OPTIONAL]) {
+            if (!empty($definition[self::OPTIONAL]) && $definition[self::OPTIONAL]){
                 $optional = true;
             }
 
@@ -133,7 +133,7 @@ 
                 FilterProvider::OPTIONAL => $optional,
             ];
 
-            if ($iterate) {
+            if ($iterate){
                 [$source, $origin] = $this->parseDefinition($field, $definition[self::ITERATE] ?? $origin);
 
                 $map[FilterProvider::ITERATE_SOURCE] = $source;
@@ -159,7 +159,7 @@ 
      */
     private function parseDefinition(string $field, string $definition): array
     {
-        if (strpos($definition, ':') === false) {
+        if (strpos($definition, ':') === false){
             return ['data', $definition ?? $field];
         }
 

Braces +21 added lines, -10 removed lines

@@ -70,18 +70,22 @@ 
     protected function buildMap(ReflectionEntity $filter): array
     {
         $schema = $filter->getProperty('schema', true);
-        if (empty($schema)) {
+        if (empty($schema))
+        {
             throw new SchemaException("Filter `{$filter->getName()}` does not define any schema");
         }
 
         $result = [];
-        foreach ($schema as $field => $definition) {
+        foreach ($schema as $field => $definition)
+        {
             $optional = false;
 
             // short definition
-            if (is_string($definition)) {
+            if (is_string($definition))
+            {
                 // simple scalar field definition
-                if (!class_exists($definition)) {
+                if (!class_exists($definition))
+                {
                     [$source, $origin] = $this->parseDefinition($field, $definition);
                     $result[$field] = [
                         FilterProvider::SOURCE => $source,
@@ -102,25 +106,30 @@ 
                 continue;
             }
 
-            if (!is_array($definition) || count($definition) === 0) {
+            if (!is_array($definition) || count($definition) === 0)
+            {
                 throw new SchemaException(
                     "Invalid schema definition at `{$filter->getName()}`->`{$field}`"
                 );
             }
 
             // complex definition
-            if (!empty($definition[self::ORIGIN])) {
+            if (!empty($definition[self::ORIGIN]))
+            {
                 $origin = $definition[self::ORIGIN];
 
                 // [class, 'data:something.*'] vs [class, 'data:something']
                 $iterate = strpos($origin, '.*') !== false || !empty($definition[self::ITERATE]);
                 $origin = rtrim($origin, '.*');
-            } else {
+            }
+            else
+            {
                 $origin = $field;
                 $iterate = true;
             }
 
-            if (!empty($definition[self::OPTIONAL]) && $definition[self::OPTIONAL]) {
+            if (!empty($definition[self::OPTIONAL]) && $definition[self::OPTIONAL])
+            {
                 $optional = true;
             }
 
@@ -133,7 +142,8 @@ 
                 FilterProvider::OPTIONAL => $optional,
             ];
 
-            if ($iterate) {
+            if ($iterate)
+            {
                 [$source, $origin] = $this->parseDefinition($field, $definition[self::ITERATE] ?? $origin);
 
                 $map[FilterProvider::ITERATE_SOURCE] = $source;
@@ -159,7 +169,8 @@ 
      */
     private function parseDefinition(string $field, string $definition): array
     {
-        if (strpos($definition, ':') === false) {
+        if (strpos($definition, ':') === false)
+        {
             return ['data', $definition ?? $field];
         }
 

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

@@ -95,7 +95,7 @@ 
                     FilterProvider::SOURCE   => null,
                     FilterProvider::ORIGIN   => $field,
                     FilterProvider::FILTER   => $definition,
-                    FilterProvider::ARRAY    => false,
+                    FilterProvider::array    => false,
                     FilterProvider::OPTIONAL => $optional,
                 ];
 
@@ -129,7 +129,7 @@ 
                 FilterProvider::FILTER   => $definition[self::NESTED],
                 FilterProvider::SOURCE   => null,
                 FilterProvider::ORIGIN   => $origin,
-                FilterProvider::ARRAY    => $iterate,
+                FilterProvider::array    => $iterate,
                 FilterProvider::OPTIONAL => $optional,
             ];