spiral / framework

src/DataGrid/src/InputMapper.php

Spacing +12 added lines, -12 removed lines

@@ -32,15 +32,15 @@ 
 
     public function hasOption(string $option): bool
     {
-        if (!isset($this->mapped[$option])) {
+        if (!isset($this->mapped[$option])){
             return $this->input->hasValue($option);
         }
 
         $mappedOptions = $this->mapped[$option];
-        foreach ($mappedOptions as $mappedOption) {
+        foreach ($mappedOptions as $mappedOption){
             $paths = $mappedOption[0];
             $firstPath = array_shift($paths);
-            if ($this->input->hasValue($firstPath)) {
+            if ($this->input->hasValue($firstPath)){
                 return true;
             }
         }
@@ -54,33 +54,33 @@ 
      */
     public function getOption(string $option)
     {
-        if (!isset($this->mapped[$option])) {
+        if (!isset($this->mapped[$option])){
             return $this->input->getValue($option);
         }
 
         $mappedOptions = $this->mapped[$option];
         $output = [];
-        foreach ($mappedOptions as $mappedOption) {
+        foreach ($mappedOptions as $mappedOption){
             [$paths, $to] = $mappedOption;
             $firstPath = array_shift($paths);
-            if (!$this->input->hasValue($firstPath)) {
+            if (!$this->input->hasValue($firstPath)){
                 continue;
             }
 
             $data = $this->input->getValue($firstPath);
-            if (!empty($paths) && !is_array($data)) {
+            if (!empty($paths) && !is_array($data)){
                 continue;
             }
 
-            foreach ($paths as $path) {
-                if (!is_array($data) || !isset($data[$path])) {
+            foreach ($paths as $path){
+                if (!is_array($data) || !isset($data[$path])){
                     continue 2;
                 }
                 $data = $data[$path];
             }
 
             $result = $data;
-            foreach ($to as $path) {
+            foreach ($to as $path){
                 $result = [$path => $result];
             }
 
@@ -92,10 +92,10 @@ 
 
     protected function map(): void
     {
-        foreach ($this->mapping as $from => $to) {
+        foreach ($this->mapping as $from => $to){
             $to = explode('.', $to);
             $option = array_shift($to);
-            if (!isset($this->mapped[$option])) {
+            if (!isset($this->mapped[$option])){
                 $this->mapped[$option] = [];
             }
             $this->mapped[$option][] = [explode('.', $from), array_reverse($to)];

Braces +24 added lines, -12 removed lines

@@ -32,15 +32,18 @@ 
 
     public function hasOption(string $option): bool
     {
-        if (!isset($this->mapped[$option])) {
+        if (!isset($this->mapped[$option]))
+        {
             return $this->input->hasValue($option);
         }
 
         $mappedOptions = $this->mapped[$option];
-        foreach ($mappedOptions as $mappedOption) {
+        foreach ($mappedOptions as $mappedOption)
+        {
             $paths = $mappedOption[0];
             $firstPath = array_shift($paths);
-            if ($this->input->hasValue($firstPath)) {
+            if ($this->input->hasValue($firstPath))
+            {
                 return true;
             }
         }
@@ -54,33 +57,40 @@ 
      */
     public function getOption(string $option)
     {
-        if (!isset($this->mapped[$option])) {
+        if (!isset($this->mapped[$option]))
+        {
             return $this->input->getValue($option);
         }
 
         $mappedOptions = $this->mapped[$option];
         $output = [];
-        foreach ($mappedOptions as $mappedOption) {
+        foreach ($mappedOptions as $mappedOption)
+        {
             [$paths, $to] = $mappedOption;
             $firstPath = array_shift($paths);
-            if (!$this->input->hasValue($firstPath)) {
+            if (!$this->input->hasValue($firstPath))
+            {
                 continue;
             }
 
             $data = $this->input->getValue($firstPath);
-            if (!empty($paths) && !is_array($data)) {
+            if (!empty($paths) && !is_array($data))
+            {
                 continue;
             }
 
-            foreach ($paths as $path) {
-                if (!is_array($data) || !isset($data[$path])) {
+            foreach ($paths as $path)
+            {
+                if (!is_array($data) || !isset($data[$path]))
+                {
                     continue 2;
                 }
                 $data = $data[$path];
             }
 
             $result = $data;
-            foreach ($to as $path) {
+            foreach ($to as $path)
+            {
                 $result = [$path => $result];
             }
 
@@ -92,10 +102,12 @@ 
 
     protected function map(): void
     {
-        foreach ($this->mapping as $from => $to) {
+        foreach ($this->mapping as $from => $to)
+        {
             $to = explode('.', $to);
             $option = array_shift($to);
-            if (!isset($this->mapped[$option])) {
+            if (!isset($this->mapped[$option]))
+            {
                 $this->mapped[$option] = [];
             }
             $this->mapped[$option][] = [explode('.', $from), array_reverse($to)];

src/DataGrid/tests/MapperTest.php

Spacing +2 added lines, -2 removed lines

@@ -79,7 +79,7 @@ 
             $mapper->getOption('sort')
         );
         $this->assertSame(
-            ['byOldestFilter' => false,],
+            ['byOldestFilter' => false, ],
             $mapper->getOption('filter')
         );
     }
@@ -118,7 +118,7 @@ 
             $mapper->getOption('sort')
         );
         $this->assertSame(
-            ['by' => ['oldest' => false],],
+            ['by' => ['oldest' => false], ],
             $mapper->getOption('filter')
         );
     }

src/DataGrid/src/GridFactory.php

Spacing +13 added lines, -13 removed lines

@@ -116,7 +116,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.');
         }
 
@@ -134,11 +134,11 @@ 
     protected function getFilters($source, GridSchema $schema): array
     {
         $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){
                     $source = $this->compiler->compile($source, $filter);
                     $filters[$name] = $filter->getValue();
                 }
@@ -150,7 +150,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));
         }
 
@@ -168,11 +168,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();
                 }
@@ -184,7 +184,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);
         }
@@ -195,12 +195,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');
         }
 
@@ -219,7 +219,7 @@ 
     protected function getOptionArray(string $option): array
     {
         $result = $this->getOption($option);
-        if (!is_array($result)) {
+        if (!is_array($result)){
             return [];
         }
 
@@ -234,7 +234,7 @@ 
      */
     protected function getOption(string $option)
     {
-        if ($this->mapper->hasOption($option)) {
+        if ($this->mapper->hasOption($option)){
             return $this->mapper->getOption($option);
         }
 

Braces +26 added lines, -13 removed lines

@@ -116,7 +116,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.');
         }
 
@@ -134,11 +135,14 @@ 
     protected function getFilters($source, GridSchema $schema): array
     {
         $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)
+                {
                     $source = $this->compiler->compile($source, $filter);
                     $filters[$name] = $filter->getValue();
                 }
@@ -150,7 +154,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));
         }
 
@@ -168,11 +173,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();
                 }
@@ -184,7 +192,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);
         }
@@ -195,12 +204,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');
         }
 
@@ -219,7 +230,8 @@ 
     protected function getOptionArray(string $option): array
     {
         $result = $this->getOption($option);
-        if (!is_array($result)) {
+        if (!is_array($result))
+        {
             return [];
         }
 
@@ -234,7 +246,8 @@ 
      */
     protected function getOption(string $option)
     {
-        if ($this->mapper->hasOption($option)) {
+        if ($this->mapper->hasOption($option))
+        {
             return $this->mapper->getOption($option);
         }