spiral / framework

src/DataGrid/tests/GridFactoryTest.php

Spacing +5 added lines, -5 removed lines

@@ -241,7 +241,7 @@ 
         $expected,
         string $expectedException = null
     ): void {
-        if ($expectedException !== null) {
+        if ($expectedException !== null){
             $this->expectException($expectedException);
         }
 
@@ -300,7 +300,7 @@ 
             new Gte('price', new Value\FloatValue())
         );
         $schema->addWrappedFilters(
-            static function (FilterInterface ...$filters) {
+            static function (FilterInterface ...$filters){
                 return new Any(...$filters);
             },
             'size',
@@ -372,15 +372,15 @@ 
     private function paginatorInput(int $page = null, int $limit = null): array
     {
         $result = [];
-        if ($page === null && $limit === null) {
+        if ($page === null && $limit === null){
             return $result;
         }
 
-        if ($page !== null) {
+        if ($page !== null){
             $result['page'] = $page;
         }
 
-        if ($limit !== null) {
+        if ($limit !== null){
             $result['limit'] = $limit;
         }
 

Braces +10 added lines, -5 removed lines

@@ -241,7 +241,8 @@ 
         $expected,
         string $expectedException = null
     ): void {
-        if ($expectedException !== null) {
+        if ($expectedException !== null)
+        {
             $this->expectException($expectedException);
         }
 
@@ -300,7 +301,8 @@ 
             new Gte('price', new Value\FloatValue())
         );
         $schema->addWrappedFilters(
-            static function (FilterInterface ...$filters) {
+            static function (FilterInterface ...$filters)
+            {
                 return new Any(...$filters);
             },
             'size',
@@ -372,15 +374,18 @@ 
     private function paginatorInput(int $page = null, int $limit = null): array
     {
         $result = [];
-        if ($page === null && $limit === null) {
+        if ($page === null && $limit === null)
+        {
             return $result;
         }
 
-        if ($page !== null) {
+        if ($page !== null)
+        {
             $result['page'] = $page;
         }
 
-        if ($limit !== null) {
+        if ($limit !== null)
+        {
             $result['limit'] = $limit;
         }
 

src/DataGrid/tests/Fixture/WrapWriter.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
      */
     public function write($source, SpecificationInterface $specification, Compiler $compiler)
     {
-        if (is_array($source)) {
+        if (is_array($source)){
             $source[] = $specification;
         }
 

Braces +2 added lines, -1 removed lines

@@ -21,7 +21,8 @@
      */
     public function write($source, SpecificationInterface $specification, Compiler $compiler)
     {
-        if (is_array($source)) {
+        if (is_array($source))
+        {
             $source[] = $specification;
         }
 

src/DataGrid/src/GridFactory.php

Spacing +14 added lines, -14 removed lines

@@ -108,7 +108,7 @@ 
         ['view' => $view, 'source' => $source] = $this->applySorters($view, $source, $schema);
         ['view' => $view, 'source' => $source] = $this->applyPaginator($view, $source, $schema);
 
-        if (!is_iterable($source)) {
+        if (!is_iterable($source)){
             throw new GridViewException('GridView expects the source to be iterable after all.');
         }
 
@@ -127,18 +127,18 @@ 
     {
         $values = [];
         $filters = [];
-        foreach ($this->getOptionArray(static::KEY_FILTER) ?? [] as $name => $value) {
-            if ($schema->hasFilter($name)) {
+        foreach ($this->getOptionArray(static::KEY_FILTER) ?? [] as $name => $value){
+            if ($schema->hasFilter($name)){
                 $filter = $schema->getFilter($name)->withValue($value);
 
-                if ($filter !== null) {
+                if ($filter !== null){
                     $filters[$name] = $filter;
                     $values[$name] = $filter->getValue();
                 }
             }
         }
 
-        foreach ($schema->wrapFilters($filters) as $filter) {
+        foreach ($schema->wrapFilters($filters) as $filter){
             $source = $this->compiler->compile($source, $filter);
         }
 
@@ -147,7 +147,7 @@ 
 
     protected function applyCounter(GridInterface $view, $source, GridSchema $schema): array
     {
-        if (is_countable($source) && $this->getOption(static::KEY_FETCH_COUNT)) {
+        if (is_countable($source) && $this->getOption(static::KEY_FETCH_COUNT)){
             $view = $view->withOption(GridInterface::COUNT, ($this->count)($source));
         }
 
@@ -165,11 +165,11 @@ 
     protected function getSorters($source, GridSchema $schema): array
     {
         $sorters = [];
-        foreach ($this->getOptionArray(static::KEY_SORT) ?? [] as $name => $value) {
-            if ($schema->hasSorter($name)) {
+        foreach ($this->getOptionArray(static::KEY_SORT) ?? [] as $name => $value){
+            if ($schema->hasSorter($name)){
                 $sorter = $schema->getSorter($name)->withDirection($value);
 
-                if ($sorter !== null) {
+                if ($sorter !== null){
                     $source = $this->compiler->compile($source, $sorter);
                     $sorters[$name] = $sorter->getValue();
                 }
@@ -181,7 +181,7 @@ 
 
     protected function applyPaginator(GridInterface $view, $source, GridSchema $schema): array
     {
-        if ($schema->getPaginator() !== null) {
+        if ($schema->getPaginator() !== null){
             ['source' => $source, 'paginator' => $paginator] = $this->getPaginator($source, $schema);
             $view = $view->withOption(GridInterface::PAGINATOR, $paginator);
         }
@@ -192,12 +192,12 @@ 
     protected function getPaginator($source, GridSchema $schema): array
     {
         $paginator = $schema->getPaginator();
-        if (!$paginator instanceof FilterInterface) {
+        if (!$paginator instanceof FilterInterface){
             throw new CompilerException('Paginator can not be null');
         }
 
         $withValue = $paginator->withValue($this->getOption(static::KEY_PAGINATE));
-        if ($withValue === null) {
+        if ($withValue === null){
             throw new CompilerException('Paginator can not be null');
         }
 
@@ -216,7 +216,7 @@ 
     protected function getOptionArray(string $option): array
     {
         $result = $this->getOption($option);
-        if (!is_array($result)) {
+        if (!is_array($result)){
             return [];
         }
 
@@ -231,7 +231,7 @@ 
      */
     protected function getOption(string $option)
     {
-        if ($this->input->hasValue($option)) {
+        if ($this->input->hasValue($option)){
             return $this->input->getValue($option);
         }
 

Braces +28 added lines, -14 removed lines

@@ -108,7 +108,8 @@ 
         ['view' => $view, 'source' => $source] = $this->applySorters($view, $source, $schema);
         ['view' => $view, 'source' => $source] = $this->applyPaginator($view, $source, $schema);
 
-        if (!is_iterable($source)) {
+        if (!is_iterable($source))
+        {
             throw new GridViewException('GridView expects the source to be iterable after all.');
         }
 
@@ -127,18 +128,22 @@ 
     {
         $values = [];
         $filters = [];
-        foreach ($this->getOptionArray(static::KEY_FILTER) ?? [] as $name => $value) {
-            if ($schema->hasFilter($name)) {
+        foreach ($this->getOptionArray(static::KEY_FILTER) ?? [] as $name => $value)
+        {
+            if ($schema->hasFilter($name))
+            {
                 $filter = $schema->getFilter($name)->withValue($value);
 
-                if ($filter !== null) {
+                if ($filter !== null)
+                {
                     $filters[$name] = $filter;
                     $values[$name] = $filter->getValue();
                 }
             }
         }
 
-        foreach ($schema->wrapFilters($filters) as $filter) {
+        foreach ($schema->wrapFilters($filters) as $filter)
+        {
             $source = $this->compiler->compile($source, $filter);
         }
 
@@ -147,7 +152,8 @@ 
 
     protected function applyCounter(GridInterface $view, $source, GridSchema $schema): array
     {
-        if (is_countable($source) && $this->getOption(static::KEY_FETCH_COUNT)) {
+        if (is_countable($source) && $this->getOption(static::KEY_FETCH_COUNT))
+        {
             $view = $view->withOption(GridInterface::COUNT, ($this->count)($source));
         }
 
@@ -165,11 +171,14 @@ 
     protected function getSorters($source, GridSchema $schema): array
     {
         $sorters = [];
-        foreach ($this->getOptionArray(static::KEY_SORT) ?? [] as $name => $value) {
-            if ($schema->hasSorter($name)) {
+        foreach ($this->getOptionArray(static::KEY_SORT) ?? [] as $name => $value)
+        {
+            if ($schema->hasSorter($name))
+            {
                 $sorter = $schema->getSorter($name)->withDirection($value);
 
-                if ($sorter !== null) {
+                if ($sorter !== null)
+                {
                     $source = $this->compiler->compile($source, $sorter);
                     $sorters[$name] = $sorter->getValue();
                 }
@@ -181,7 +190,8 @@ 
 
     protected function applyPaginator(GridInterface $view, $source, GridSchema $schema): array
     {
-        if ($schema->getPaginator() !== null) {
+        if ($schema->getPaginator() !== null)
+        {
             ['source' => $source, 'paginator' => $paginator] = $this->getPaginator($source, $schema);
             $view = $view->withOption(GridInterface::PAGINATOR, $paginator);
         }
@@ -192,12 +202,14 @@ 
     protected function getPaginator($source, GridSchema $schema): array
     {
         $paginator = $schema->getPaginator();
-        if (!$paginator instanceof FilterInterface) {
+        if (!$paginator instanceof FilterInterface)
+        {
             throw new CompilerException('Paginator can not be null');
         }
 
         $withValue = $paginator->withValue($this->getOption(static::KEY_PAGINATE));
-        if ($withValue === null) {
+        if ($withValue === null)
+        {
             throw new CompilerException('Paginator can not be null');
         }
 
@@ -216,7 +228,8 @@ 
     protected function getOptionArray(string $option): array
     {
         $result = $this->getOption($option);
-        if (!is_array($result)) {
+        if (!is_array($result))
+        {
             return [];
         }
 
@@ -231,7 +244,8 @@ 
      */
     protected function getOption(string $option)
     {
-        if ($this->input->hasValue($option)) {
+        if ($this->input->hasValue($option))
+        {
             return $this->input->getValue($option);
         }
 

src/DataGrid/src/GridSchema.php

Spacing +10 added lines, -10 removed lines

@@ -40,8 +40,8 @@ 
 
     public function addWrappedFilters(callable $wrapper, string ...$filters): void
     {
-        foreach ($filters as $filter) {
-            if (isset($this->wrappedFilters[$filter])) {
+        foreach ($filters as $filter){
+            if (isset($this->wrappedFilters[$filter])){
                 throw new SchemaException("Filter `$filter` is already wrapped");
             }
         }
@@ -53,10 +53,10 @@ 
     public function wrapFilters(array $input): array
     {
         $output = [];
-        foreach ($this->wrappers as $wrapper) {
+        foreach ($this->wrappers as $wrapper){
             $output[] = $wrapper['wrapper'](...$this->fetchFilters($input, $wrapper['filters']));
-            foreach ($input as $name => $filter) {
-                if (!in_array($name, $wrapper['filters'], true)) {
+            foreach ($input as $name => $filter){
+                if (!in_array($name, $wrapper['filters'], true)){
                     $output[] = $filter;
                 }
             }
@@ -74,7 +74,7 @@ 
      */
     public function addFilter(string $name, FilterInterface $filter): void
     {
-        if ($this->hasFilter($name)) {
+        if ($this->hasFilter($name)){
             throw new SchemaException("Filter `$name` is already defined");
         }
 
@@ -99,7 +99,7 @@ 
      */
     public function getFilter(string $name): FilterInterface
     {
-        if (!$this->hasFilter($name)) {
+        if (!$this->hasFilter($name)){
             throw new SchemaException("No such filter `$name`");
         }
 
@@ -123,7 +123,7 @@ 
      */
     public function addSorter(string $name, SorterInterface $sorter): void
     {
-        if ($this->hasSorter($name)) {
+        if ($this->hasSorter($name)){
             throw new SchemaException("Sorter `$name` is already defined");
         }
 
@@ -148,7 +148,7 @@ 
      */
     public function getSorter(string $name): SorterInterface
     {
-        if (!$this->hasSorter($name)) {
+        if (!$this->hasSorter($name)){
             throw new SchemaException("No such sorter `$name`");
         }
 
@@ -186,7 +186,7 @@ 
     private function fetchFilters(array $filters, array $names): array
     {
         $output = [];
-        foreach ($names as $name) {
+        foreach ($names as $name){
             $output[] = $filters[$name] ?? new NullFilter();
         }
 

Braces +20 added lines, -10 removed lines

@@ -40,8 +40,10 @@ 
 
     public function addWrappedFilters(callable $wrapper, string ...$filters): void
     {
-        foreach ($filters as $filter) {
-            if (isset($this->wrappedFilters[$filter])) {
+        foreach ($filters as $filter)
+        {
+            if (isset($this->wrappedFilters[$filter]))
+            {
                 throw new SchemaException("Filter `$filter` is already wrapped");
             }
         }
@@ -53,10 +55,13 @@ 
     public function wrapFilters(array $input): array
     {
         $output = [];
-        foreach ($this->wrappers as $wrapper) {
+        foreach ($this->wrappers as $wrapper)
+        {
             $output[] = $wrapper['wrapper'](...$this->fetchFilters($input, $wrapper['filters']));
-            foreach ($input as $name => $filter) {
-                if (!in_array($name, $wrapper['filters'], true)) {
+            foreach ($input as $name => $filter)
+            {
+                if (!in_array($name, $wrapper['filters'], true))
+                {
                     $output[] = $filter;
                 }
             }
@@ -74,7 +79,8 @@ 
      */
     public function addFilter(string $name, FilterInterface $filter): void
     {
-        if ($this->hasFilter($name)) {
+        if ($this->hasFilter($name))
+        {
             throw new SchemaException("Filter `$name` is already defined");
         }
 
@@ -99,7 +105,8 @@ 
      */
     public function getFilter(string $name): FilterInterface
     {
-        if (!$this->hasFilter($name)) {
+        if (!$this->hasFilter($name))
+        {
             throw new SchemaException("No such filter `$name`");
         }
 
@@ -123,7 +130,8 @@ 
      */
     public function addSorter(string $name, SorterInterface $sorter): void
     {
-        if ($this->hasSorter($name)) {
+        if ($this->hasSorter($name))
+        {
             throw new SchemaException("Sorter `$name` is already defined");
         }
 
@@ -148,7 +156,8 @@ 
      */
     public function getSorter(string $name): SorterInterface
     {
-        if (!$this->hasSorter($name)) {
+        if (!$this->hasSorter($name))
+        {
             throw new SchemaException("No such sorter `$name`");
         }
 
@@ -186,7 +195,8 @@ 
     private function fetchFilters(array $filters, array $names): array
     {
         $output = [];
-        foreach ($names as $name) {
+        foreach ($names as $name)
+        {
             $output[] = $filters[$name] ?? new NullFilter();
         }